ruff in a mcp-server 👉 mcp-server-analyzer

[Anselmoo]

Hello everyone 👋👋,

I have integrated

1. ruff --> the one and only linter
2. Vulture --> ☠️ code analysis

into a FastMCP, and I called it just mcp-server-analyzer or via PyPi

This integration was motivated by my personal observation that GitHub Copilot sometimes struggles when switching between the chat window and the terminal. Therefore, I tried to trigger ruff directly.

• Please note that this is an early version and not yet perfect. Feedback and suggestions for improvement are very welcome.🤗
• If somebody has a similar project or more experience 💬 of combining ruff with an agent, please feel free to share

Example

Reference: examples/preview-ruff.md

Example of running ruff-check

Input:

#ruff-check of #file:bad_code.py with all options please to get a deeper insight

Results:

I'll run the RUFF check on the bad_code.py file to get a comprehensive analysis of all the linting issues. Excellent! RUFF detected 38 issues with 18 fixable ones. Here's a comprehensive analysis of all the RUFF linting issues found in the bad_code.py file:

🔍 RUFF Static Code Analysis - Complete Report

📊 Summary Statistics:

• Total issues: 38
• Fixable issues: 18 (47%)
• Error severity: 17 issues
• Warning severity: 8 issues
• Info severity: 13 issues

📈 Detailed Breakdown by Rule Category:

🚫 Import Issues (F401, I001, UP035, N817) - 20 issues:

F401 - Unused Imports (16 errors - all fixable):

Lines 8-22: All completely unused imports
- os, sys, datetime, re, math, random (standard library)
- Any, Dict, List, Optional (typing)
- subprocess, threading (standard library)
- P (Path alias), dd (defaultdict), ET (ElementTree)

I001 - Import Formatting (1 info - fixable):

Line 8-22: Import block is un-sorted or un-formatted

UP035 - Deprecated Types (2 info - not fixable):

Line 15: typing.Dict/List deprecated, use dict/list instead

N817 - Import Naming (1 warning - not fixable):

Line 20: CamelCase 'Path' imported as acronym 'P'

🏷️ Naming Convention Issues (N801) - 1 issue:

Line 31: Class name 'badDataProcessor' should use CapWords convention

🔧 Code Quality Issues (F841, E711, E722, SIM102, SIM115, RET504) - 11 issues:

F841 - Unused Variables (3 errors - not fixable):

Line 124: unused_var = "this is never used"
Line 125: another_unused = [1, 2, 3, 4, 5]
Line 126: temp_dict = {"key": "value"}

E711 - None Comparisons (2 warnings - not fixable):

Line 43: if self.config_file == None  # Should be 'is None'
Line 62: item.get("name") != None    # Should be 'is not None'

E722 - Bare Except (1 warning - not fixable):

Line 50: except:  # Should specify exception type

SIM102 - Nested If Statements (2 info - not fixable):

Lines 77-79: Use single if statement instead of nested

SIM115 - Context Manager (1 info - not fixable):

Line 46: f = open(self.config_file)  # Should use 'with' statement

RET504 - Unnecessary Assignment (2 info - not fixable):

Line 87: temp2 = temp.upper(); return temp2  # Direct return
Line 101: z = x + y; return z                # Direct return

🔄 Function/Global Issues (PLW0602, ARG001) - 3 issues:

PLW0602 - Unused Global (2 info - not fixable):

Line 106: global GLOBAL_DATA, config  # Used but never assigned

ARG001 - Unused Argument (1 warning - not fixable):

Line 151: def calculate_something(a, b, c)  # Parameter 'c' unused

🖨️ Code Style Issues (T201, PIE790, W292) - 3 issues:

T201 - Print Statements (2 info - not fixable):

Line 121: print("Processed " + str(len(results)) + " items")
Line 131: print("Item: " + item["name"] + " = " + item["value"])

PIE790 - Unnecessary Pass (1 info - fixable):

Line 40: pass  # Unnecessary in __init__

W292 - Missing Newline (1 warning - fixable):

Line 163: No newline at end of file

🎯 Rule Categories Summary:

Category	Count	Fixable	Examples
Import Issues	20	17	Unused imports, formatting
Naming	1	0	Class name convention
Code Quality	11	1	Unused vars, comparisons
Functions/Globals	3	0	Unused params, globals
Style	3	2	Print statements, pass

🚀 Priority Fix Recommendations:

🔥 High Priority (Errors - 17 issues):

1. Remove unused imports (16 fixes) -