suggest_refactoring
Analyzes code to provide refactoring suggestions that improve structure, readability, and maintainability for cleaner, more efficient programming.
Instructions
提供代码重构建议，改善代码结构和可读性
Input Schema
TableJSON Schema
| Name | Required | Description | Default |
|---|---|---|---|
| code | Yes | 要重构的代码 | |
| language | Yes | 编程语言 | |
| focus | No | 重构重点 |
Implementation Reference
- src/tools/refactoringSuggester.ts:6-56 (handler)Core handler function `suggestRefactoring` that orchestrates analysis for performance, readability, maintainability, and language-specific refactoring suggestions, generating prioritized suggestions.export async function suggestRefactoring( code: string, language: string, focus: 'performance' | 'readability' | 'maintainability' | 'all' = 'all' ): Promise<RefactoringResult> { const suggestions: RefactoringSuggestion[] = []; // 根据重构重点进行分析 if (focus === 'all' || focus === 'performance') { await analyzePerformanceIssues(code, language, suggestions); } if (focus === 'all' || focus === 'readability') { await analyzeReadabilityIssues(code, language, suggestions); } if (focus === 'all' || focus === 'maintainability') { await analyzeMaintainabilityIssues(code, language, suggestions); } // 语言特定重构建议 switch (language.toLowerCase()) { case 'javascript': case 'typescript': await analyzeJavaScriptRefactoring(code, suggestions); break; case 'python': await analyzePythonRefactoring(code, suggestions); break; case 'java': await analyzeJavaRefactoring(code, suggestions); break; } // 按优先级排序 suggestions.sort((a, b) => { const priorityOrder = { 'high': 3, 'medium': 2, 'low': 1 }; return priorityOrder[b.priority] - priorityOrder[a.priority]; }); return { language, focus, totalSuggestions: suggestions.length, highPriority: suggestions.filter(s => s.priority === 'high').length, mediumPriority: suggestions.filter(s => s.priority === 'medium').length, lowPriority: suggestions.filter(s => s.priority === 'low').length, suggestions, summary: generateRefactoringSummary(suggestions, focus) }; }
- src/types/index.ts:61-82 (schema)TypeScript interfaces defining the input/output structure for refactoring results: RefactoringResult and RefactoringSuggestion.export interface RefactoringResult { language: string; focus: 'performance' | 'readability' | 'maintainability' | 'all'; totalSuggestions: number; highPriority: number; mediumPriority: number; lowPriority: number; suggestions: RefactoringSuggestion[]; summary: string; } export interface RefactoringSuggestion { type: 'performance' | 'readability' | 'maintainability' | 'modernization' | 'pythonic'; priority: 'high' | 'medium' | 'low'; title: string; description: string; line: number; originalCode: string; suggestedCode: string; impact: string; effort: 'low' | 'medium' | 'high'; }
- src/index.ts:107-129 (registration)MCP tool registration in the server's listTools handler, specifying name, description, and JSON schema for input parameters.{ name: 'suggest_refactoring', description: '提供代码重构建议，改善代码结构和可读性', inputSchema: { type: 'object', properties: { code: { type: 'string', description: '要重构的代码' }, language: { type: 'string', description: '编程语言' }, focus: { type: 'string', enum: ['performance', 'readability', 'maintainability', 'all'], description: '重构重点' } }, required: ['code', 'language'] } },
- src/index.ts:244-262 (handler)Server-side wrapper handler that validates tool arguments with Zod schema and invokes the core suggestRefactoring function, formatting response for MCP protocol.private async handleSuggestRefactoring(args: any) { const schema = z.object({ code: z.string(), language: z.string(), focus: z.enum(['performance', 'readability', 'maintainability', 'all']).default('all') }); const { code, language, focus } = schema.parse(args); const result = await suggestRefactoring(code, language, focus); return { content: [ { type: 'text', text: JSON.stringify(result, null, 2) } ] }; }
- Helper function for performance analysis, one of several specialized analyzers called by the main handler.async function analyzePerformanceIssues( code: string, language: string, suggestions: RefactoringSuggestion[] ): Promise<void> { const lines = code.split('\n'); lines.forEach((line, index) => { const lineNumber = index + 1; const trimmedLine = line.trim(); // 检测循环中的重复计算 if (isInLoop(lines, index) && hasExpensiveOperation(trimmedLine)) { suggestions.push({ type: 'performance', priority: 'high', title: '循环中的重复计算', description: '在循环中发现可能的重复计算，建议提取到循环外部', line: lineNumber, originalCode: line, suggestedCode: optimizeLoopCalculation(line, language), impact: '可显著提升性能，特别是在大数据集处理时', effort: 'low' }); } // 检测字符串拼接性能问题 if (hasInefficiientStringConcatenation(trimmedLine, language)) { suggestions.push({ type: 'performance', priority: 'medium', title: '低效的字符串拼接', description: '使用更高效的字符串拼接方法', line: lineNumber, originalCode: line, suggestedCode: optimizeStringConcatenation(line, language), impact: '在大量字符串操作时可提升性能', effort: 'low' }); } // 检测不必要的对象创建 if (hasUnnecessaryObjectCreation(trimmedLine, language)) { suggestions.push({ type: 'performance', priority: 'medium', title: '不必要的对象创建', description: '减少不必要的对象创建以降低内存压力', line: lineNumber, originalCode: line, suggestedCode: optimizeObjectCreation(line, language), impact: '减少内存使用和垃圾回收压力', effort: 'medium' }); } // 检测低效的数组操作 if (hasInefficientArrayOperation(trimmedLine, language)) { suggestions.push({ type: 'performance', priority: 'medium', title: '低效的数组操作', description: '使用更高效的数组操作方法', line: lineNumber, originalCode: line, suggestedCode: optimizeArrayOperation(line, language), impact: '在处理大数组时可显著提升性能', effort: 'low' }); } }); }