couchloop-eq-mcp 1.0.3 → 1.0.5
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/README.md +24 -11
- package/dist/clients/shrinkChatClient.js +1 -1
- package/dist/clients/shrinkChatClient.js.map +1 -1
- package/dist/developer/analyzers/bloat-detector.d.ts +89 -0
- package/dist/developer/analyzers/bloat-detector.d.ts.map +1 -0
- package/dist/developer/analyzers/bloat-detector.js +483 -0
- package/dist/developer/analyzers/bloat-detector.js.map +1 -0
- package/dist/developer/backup/auto-backup.d.ts +96 -0
- package/dist/developer/backup/auto-backup.d.ts.map +1 -0
- package/dist/developer/backup/auto-backup.js +346 -0
- package/dist/developer/backup/auto-backup.js.map +1 -0
- package/dist/developer/blockers/package-blocker.d.ts +33 -0
- package/dist/developer/blockers/package-blocker.d.ts.map +1 -0
- package/dist/developer/blockers/package-blocker.js +224 -0
- package/dist/developer/blockers/package-blocker.js.map +1 -0
- package/dist/developer/evaluators/ai-error-preventer.d.ts +54 -0
- package/dist/developer/evaluators/ai-error-preventer.d.ts.map +1 -0
- package/dist/developer/evaluators/ai-error-preventer.js +270 -0
- package/dist/developer/evaluators/ai-error-preventer.js.map +1 -0
- package/dist/developer/evaluators/build-context-detector.d.ts +44 -0
- package/dist/developer/evaluators/build-context-detector.d.ts.map +1 -0
- package/dist/developer/evaluators/build-context-detector.js +258 -0
- package/dist/developer/evaluators/build-context-detector.js.map +1 -0
- package/dist/developer/evaluators/package-evaluator.d.ts +37 -0
- package/dist/developer/evaluators/package-evaluator.d.ts.map +1 -0
- package/dist/developer/evaluators/package-evaluator.js +278 -0
- package/dist/developer/evaluators/package-evaluator.js.map +1 -0
- package/dist/developer/guards/file-guardian.d.ts +79 -0
- package/dist/developer/guards/file-guardian.d.ts.map +1 -0
- package/dist/developer/guards/file-guardian.js +309 -0
- package/dist/developer/guards/file-guardian.js.map +1 -0
- package/dist/developer/managers/context-manager.d.ts +61 -0
- package/dist/developer/managers/context-manager.d.ts.map +1 -0
- package/dist/developer/managers/context-manager.js +302 -0
- package/dist/developer/managers/context-manager.js.map +1 -0
- package/dist/developer/metrics/complexity-calculator.d.ts +52 -0
- package/dist/developer/metrics/complexity-calculator.d.ts.map +1 -0
- package/dist/developer/metrics/complexity-calculator.js +259 -0
- package/dist/developer/metrics/complexity-calculator.js.map +1 -0
- package/dist/developer/reports/review-summary.d.ts +49 -0
- package/dist/developer/reports/review-summary.d.ts.map +1 -0
- package/dist/developer/reports/review-summary.js +249 -0
- package/dist/developer/reports/review-summary.js.map +1 -0
- package/dist/developer/scanners/review-assistant.d.ts +41 -0
- package/dist/developer/scanners/review-assistant.d.ts.map +1 -0
- package/dist/developer/scanners/review-assistant.js +374 -0
- package/dist/developer/scanners/review-assistant.js.map +1 -0
- package/dist/developer/scanners/secret-scanner.d.ts +66 -0
- package/dist/developer/scanners/secret-scanner.d.ts.map +1 -0
- package/dist/developer/scanners/secret-scanner.js +287 -0
- package/dist/developer/scanners/secret-scanner.js.map +1 -0
- package/dist/developer/scanners/sql-injection-detector.d.ts +54 -0
- package/dist/developer/scanners/sql-injection-detector.d.ts.map +1 -0
- package/dist/developer/scanners/sql-injection-detector.js +174 -0
- package/dist/developer/scanners/sql-injection-detector.js.map +1 -0
- package/dist/developer/scanners/xss-detector.d.ts +60 -0
- package/dist/developer/scanners/xss-detector.d.ts.map +1 -0
- package/dist/developer/scanners/xss-detector.js +229 -0
- package/dist/developer/scanners/xss-detector.js.map +1 -0
- package/dist/developer/types/ai-errors.d.ts +34 -0
- package/dist/developer/types/ai-errors.d.ts.map +1 -0
- package/dist/developer/types/ai-errors.js +271 -0
- package/dist/developer/types/ai-errors.js.map +1 -0
- package/dist/developer/types/package.d.ts +32 -0
- package/dist/developer/types/package.d.ts.map +1 -0
- package/dist/developer/types/package.js +5 -0
- package/dist/developer/types/package.js.map +1 -0
- package/dist/developer/updaters/dependency-updater.d.ts +102 -0
- package/dist/developer/updaters/dependency-updater.d.ts.map +1 -0
- package/dist/developer/updaters/dependency-updater.js +472 -0
- package/dist/developer/updaters/dependency-updater.js.map +1 -0
- package/dist/developer/validators/cargo.d.ts +14 -0
- package/dist/developer/validators/cargo.d.ts.map +1 -0
- package/dist/developer/validators/cargo.js +132 -0
- package/dist/developer/validators/cargo.js.map +1 -0
- package/dist/developer/validators/gem.d.ts +14 -0
- package/dist/developer/validators/gem.d.ts.map +1 -0
- package/dist/developer/validators/gem.js +85 -0
- package/dist/developer/validators/gem.js.map +1 -0
- package/dist/developer/validators/go.d.ts +14 -0
- package/dist/developer/validators/go.d.ts.map +1 -0
- package/dist/developer/validators/go.js +138 -0
- package/dist/developer/validators/go.js.map +1 -0
- package/dist/developer/validators/maven.d.ts +14 -0
- package/dist/developer/validators/maven.d.ts.map +1 -0
- package/dist/developer/validators/maven.js +99 -0
- package/dist/developer/validators/maven.js.map +1 -0
- package/dist/developer/validators/npm.d.ts +14 -0
- package/dist/developer/validators/npm.d.ts.map +1 -0
- package/dist/developer/validators/npm.js +96 -0
- package/dist/developer/validators/npm.js.map +1 -0
- package/dist/developer/validators/nuget.d.ts +15 -0
- package/dist/developer/validators/nuget.d.ts.map +1 -0
- package/dist/developer/validators/nuget.js +107 -0
- package/dist/developer/validators/nuget.js.map +1 -0
- package/dist/developer/validators/pypi.d.ts +14 -0
- package/dist/developer/validators/pypi.d.ts.map +1 -0
- package/dist/developer/validators/pypi.js +118 -0
- package/dist/developer/validators/pypi.js.map +1 -0
- package/dist/developer/validators/registry-manager.d.ts +37 -0
- package/dist/developer/validators/registry-manager.d.ts.map +1 -0
- package/dist/developer/validators/registry-manager.js +89 -0
- package/dist/developer/validators/registry-manager.js.map +1 -0
- package/dist/developer/validators/version-checker.d.ts +145 -0
- package/dist/developer/validators/version-checker.d.ts.map +1 -0
- package/dist/developer/validators/version-checker.js +529 -0
- package/dist/developer/validators/version-checker.js.map +1 -0
- package/dist/server/index.js.map +1 -1
- package/dist/server/middleware/auth.d.ts +7 -9
- package/dist/server/middleware/auth.d.ts.map +1 -1
- package/dist/server/middleware/auth.js.map +1 -1
- package/dist/tools/check-versions.d.ts +100 -0
- package/dist/tools/check-versions.d.ts.map +1 -0
- package/dist/tools/check-versions.js +328 -0
- package/dist/tools/check-versions.js.map +1 -0
- package/dist/tools/detect-code-smell.d.ts +9 -0
- package/dist/tools/detect-code-smell.d.ts.map +1 -0
- package/dist/tools/detect-code-smell.js +231 -0
- package/dist/tools/detect-code-smell.js.map +1 -0
- package/dist/tools/index.d.ts +471 -0
- package/dist/tools/index.d.ts.map +1 -1
- package/dist/tools/index.js +178 -0
- package/dist/tools/index.js.map +1 -1
- package/dist/tools/journey.js +1 -1
- package/dist/tools/journey.js.map +1 -1
- package/dist/tools/pre-review-code.d.ts +71 -0
- package/dist/tools/pre-review-code.d.ts.map +1 -0
- package/dist/tools/pre-review-code.js +159 -0
- package/dist/tools/pre-review-code.js.map +1 -0
- package/dist/tools/preserve-context.d.ts +27 -0
- package/dist/tools/preserve-context.d.ts.map +1 -0
- package/dist/tools/preserve-context.js +98 -0
- package/dist/tools/preserve-context.js.map +1 -0
- package/dist/tools/protect-files.d.ts +224 -0
- package/dist/tools/protect-files.d.ts.map +1 -0
- package/dist/tools/protect-files.js +286 -0
- package/dist/tools/protect-files.js.map +1 -0
- package/dist/tools/scan-security.d.ts +38 -0
- package/dist/tools/scan-security.d.ts.map +1 -0
- package/dist/tools/scan-security.js +237 -0
- package/dist/tools/scan-security.js.map +1 -0
- package/dist/tools/validate_packages.d.ts +8 -0
- package/dist/tools/validate_packages.d.ts.map +1 -0
- package/dist/tools/validate_packages.js +159 -0
- package/dist/tools/validate_packages.js.map +1 -0
- package/dist/types/auth.d.ts +18 -18
- package/dist/types/auth.d.ts.map +1 -1
- package/dist/types/auth.js +91 -36
- package/dist/types/auth.js.map +1 -1
- package/dist/types/context.d.ts +46 -0
- package/dist/types/context.d.ts.map +1 -0
- package/dist/types/context.js +17 -0
- package/dist/types/context.js.map +1 -0
- package/dist/types/file-protection.d.ts +50 -0
- package/dist/types/file-protection.d.ts.map +1 -0
- package/dist/types/file-protection.js +9 -0
- package/dist/types/file-protection.js.map +1 -0
- package/dist/utils/errorHandler.d.ts.map +1 -1
- package/dist/utils/errorHandler.js +2 -1
- package/dist/utils/errorHandler.js.map +1 -1
- package/package.json +23 -2
- package/dist/db/migrate.d.ts +0 -4
- package/dist/db/migrate.d.ts.map +0 -1
- package/dist/db/migrate.js +0 -34
- package/dist/db/migrate.js.map +0 -1
- package/dist/db/migrations/schema.d.ts +0 -1074
- package/dist/db/migrations/schema.d.ts.map +0 -1
- package/dist/db/migrations/schema.js +0 -160
- package/dist/db/migrations/schema.js.map +0 -1
- package/dist/db/schema.d.ts +0 -1576
- package/dist/db/schema.d.ts.map +0 -1
- package/dist/db/schema.js +0 -204
- package/dist/db/schema.js.map +0 -1
- package/dist/db/seed.d.ts +0 -4
- package/dist/db/seed.d.ts.map +0 -1
- package/dist/db/seed.js +0 -57
- package/dist/db/seed.js.map +0 -1
- package/dist/db/seedOAuth.d.ts +0 -4
- package/dist/db/seedOAuth.d.ts.map +0 -1
- package/dist/db/seedOAuth.js +0 -76
- package/dist/db/seedOAuth.js.map +0 -1
- package/dist/governance/config.d.ts +0 -66
- package/dist/governance/config.d.ts.map +0 -1
- package/dist/governance/config.js +0 -238
- package/dist/governance/config.js.map +0 -1
- package/dist/governance/detectors/hallucination.d.ts +0 -61
- package/dist/governance/detectors/hallucination.d.ts.map +0 -1
- package/dist/governance/detectors/hallucination.js +0 -338
- package/dist/governance/detectors/hallucination.js.map +0 -1
- package/dist/governance/detectors/inconsistency.d.ts +0 -99
- package/dist/governance/detectors/inconsistency.d.ts.map +0 -1
- package/dist/governance/detectors/inconsistency.js +0 -548
- package/dist/governance/detectors/inconsistency.js.map +0 -1
- package/dist/governance/detectors/toneDrift.d.ts +0 -63
- package/dist/governance/detectors/toneDrift.d.ts.map +0 -1
- package/dist/governance/detectors/toneDrift.js +0 -421
- package/dist/governance/detectors/toneDrift.js.map +0 -1
- package/dist/governance/detectors/unsafeReasoning.d.ts +0 -54
- package/dist/governance/detectors/unsafeReasoning.d.ts.map +0 -1
- package/dist/governance/detectors/unsafeReasoning.js +0 -473
- package/dist/governance/detectors/unsafeReasoning.js.map +0 -1
- package/dist/governance/evaluationEngine.d.ts +0 -112
- package/dist/governance/evaluationEngine.d.ts.map +0 -1
- package/dist/governance/evaluationEngine.js +0 -265
- package/dist/governance/evaluationEngine.js.map +0 -1
- package/dist/governance/intervention.d.ts +0 -81
- package/dist/governance/intervention.d.ts.map +0 -1
- package/dist/governance/intervention.js +0 -405
- package/dist/governance/intervention.js.map +0 -1
- package/dist/server/oauth/anomalyDetection.d.ts +0 -146
- package/dist/server/oauth/anomalyDetection.d.ts.map +0 -1
- package/dist/server/oauth/anomalyDetection.js +0 -405
- package/dist/server/oauth/anomalyDetection.js.map +0 -1
- package/dist/server/oauth/authServer.d.ts +0 -61
- package/dist/server/oauth/authServer.d.ts.map +0 -1
- package/dist/server/oauth/authServer.js +0 -283
- package/dist/server/oauth/authServer.js.map +0 -1
- package/dist/server/oauth/dpop.d.ts +0 -135
- package/dist/server/oauth/dpop.d.ts.map +0 -1
- package/dist/server/oauth/dpop.js +0 -338
- package/dist/server/oauth/dpop.js.map +0 -1
- package/dist/server/oauth/gdpr/consent.d.ts +0 -173
- package/dist/server/oauth/gdpr/consent.d.ts.map +0 -1
- package/dist/server/oauth/gdpr/consent.js +0 -388
- package/dist/server/oauth/gdpr/consent.js.map +0 -1
- package/dist/server/oauth/gdpr/dataPortability.d.ts +0 -214
- package/dist/server/oauth/gdpr/dataPortability.d.ts.map +0 -1
- package/dist/server/oauth/gdpr/dataPortability.js +0 -486
- package/dist/server/oauth/gdpr/dataPortability.js.map +0 -1
- package/dist/server/oauth/gdpr/index.d.ts +0 -103
- package/dist/server/oauth/gdpr/index.d.ts.map +0 -1
- package/dist/server/oauth/gdpr/index.js +0 -273
- package/dist/server/oauth/gdpr/index.js.map +0 -1
- package/dist/server/oauth/gdpr/rightToErasure.d.ts +0 -184
- package/dist/server/oauth/gdpr/rightToErasure.d.ts.map +0 -1
- package/dist/server/oauth/gdpr/rightToErasure.js +0 -527
- package/dist/server/oauth/gdpr/rightToErasure.js.map +0 -1
- package/dist/server/oauth/monitoring/securityMonitor.d.ts +0 -218
- package/dist/server/oauth/monitoring/securityMonitor.d.ts.map +0 -1
- package/dist/server/oauth/monitoring/securityMonitor.js +0 -615
- package/dist/server/oauth/monitoring/securityMonitor.js.map +0 -1
- package/dist/server/oauth/pkce.d.ts +0 -61
- package/dist/server/oauth/pkce.d.ts.map +0 -1
- package/dist/server/oauth/pkce.js +0 -157
- package/dist/server/oauth/pkce.js.map +0 -1
- package/dist/server/oauth/providers/base.d.ts +0 -147
- package/dist/server/oauth/providers/base.d.ts.map +0 -1
- package/dist/server/oauth/providers/base.js +0 -312
- package/dist/server/oauth/providers/base.js.map +0 -1
- package/dist/server/oauth/providers/github.d.ts +0 -55
- package/dist/server/oauth/providers/github.d.ts.map +0 -1
- package/dist/server/oauth/providers/github.js +0 -225
- package/dist/server/oauth/providers/github.js.map +0 -1
- package/dist/server/oauth/providers/google.d.ts +0 -49
- package/dist/server/oauth/providers/google.d.ts.map +0 -1
- package/dist/server/oauth/providers/google.js +0 -153
- package/dist/server/oauth/providers/google.js.map +0 -1
- package/dist/server/oauth/providers/index.d.ts +0 -9
- package/dist/server/oauth/providers/index.d.ts.map +0 -1
- package/dist/server/oauth/providers/index.js +0 -24
- package/dist/server/oauth/providers/index.js.map +0 -1
- package/dist/server/oauth/refreshTokenRotation.d.ts +0 -114
- package/dist/server/oauth/refreshTokenRotation.d.ts.map +0 -1
- package/dist/server/oauth/refreshTokenRotation.js +0 -344
- package/dist/server/oauth/refreshTokenRotation.js.map +0 -1
- package/dist/server/oauth/security.d.ts +0 -101
- package/dist/server/oauth/security.d.ts.map +0 -1
- package/dist/server/oauth/security.js +0 -268
- package/dist/server/oauth/security.js.map +0 -1
- package/dist/server/oauth/tokenEncryption.d.ts +0 -80
- package/dist/server/oauth/tokenEncryption.d.ts.map +0 -1
- package/dist/server/oauth/tokenEncryption.js +0 -218
- package/dist/server/oauth/tokenEncryption.js.map +0 -1
- package/dist/tools/sendMessage-complex-backup.d.ts +0 -6
- package/dist/tools/sendMessage-complex-backup.d.ts.map +0 -1
- package/dist/tools/sendMessage-complex-backup.js +0 -545
- package/dist/tools/sendMessage-complex-backup.js.map +0 -1
- package/dist/tools/sendMessage-revised.d.ts +0 -11
- package/dist/tools/sendMessage-revised.d.ts.map +0 -1
- package/dist/tools/sendMessage-revised.js +0 -429
- package/dist/tools/sendMessage-revised.js.map +0 -1
- package/dist/tools/sendMessage-truly-simple.d.ts +0 -8
- package/dist/tools/sendMessage-truly-simple.d.ts.map +0 -1
- package/dist/tools/sendMessage-truly-simple.js +0 -299
- package/dist/tools/sendMessage-truly-simple.js.map +0 -1
|
@@ -0,0 +1,231 @@
|
|
|
1
|
+
/**
|
|
2
|
+
* MCP Tool: detect_code_smell
|
|
3
|
+
* Analyzes code for bloat, verbose patterns, and over-engineering
|
|
4
|
+
* Returns actionable suggestions to prevent technical debt
|
|
5
|
+
*/
|
|
6
|
+
import { z } from 'zod';
|
|
7
|
+
import { BloatDetector } from '../developer/analyzers/bloat-detector.js';
|
|
8
|
+
const inputSchema = z.object({
|
|
9
|
+
code: z.string().describe('Code snippet to analyze for bloat patterns'),
|
|
10
|
+
language: z.enum(['javascript', 'typescript', 'python']).default('javascript').describe('Programming language'),
|
|
11
|
+
threshold: z.number().optional().describe('Complexity threshold (0-100, default: 50)'),
|
|
12
|
+
includeMetrics: z.boolean().default(true).describe('Include detailed complexity metrics'),
|
|
13
|
+
includeSuggestions: z.boolean().default(true).describe('Include refactoring suggestions')
|
|
14
|
+
});
|
|
15
|
+
export const detectCodeSmellTool = {
|
|
16
|
+
name: 'detect_code_smell',
|
|
17
|
+
description: 'Detect code bloat, verbose patterns, and over-engineered code that creates technical debt. Analyzes complexity metrics and suggests simplifications.',
|
|
18
|
+
inputSchema: {
|
|
19
|
+
type: 'object',
|
|
20
|
+
properties: {
|
|
21
|
+
code: {
|
|
22
|
+
type: 'string',
|
|
23
|
+
description: 'Code snippet to analyze for bloat patterns'
|
|
24
|
+
},
|
|
25
|
+
language: {
|
|
26
|
+
type: 'string',
|
|
27
|
+
enum: ['javascript', 'typescript', 'python'],
|
|
28
|
+
default: 'javascript',
|
|
29
|
+
description: 'Programming language'
|
|
30
|
+
},
|
|
31
|
+
threshold: {
|
|
32
|
+
type: 'number',
|
|
33
|
+
description: 'Complexity threshold (0-100, default: 50)'
|
|
34
|
+
},
|
|
35
|
+
includeMetrics: {
|
|
36
|
+
type: 'boolean',
|
|
37
|
+
default: true,
|
|
38
|
+
description: 'Include detailed complexity metrics'
|
|
39
|
+
},
|
|
40
|
+
includeSuggestions: {
|
|
41
|
+
type: 'boolean',
|
|
42
|
+
default: true,
|
|
43
|
+
description: 'Include refactoring suggestions'
|
|
44
|
+
}
|
|
45
|
+
},
|
|
46
|
+
required: ['code']
|
|
47
|
+
}
|
|
48
|
+
};
|
|
49
|
+
export async function handleDetectCodeSmell(input) {
|
|
50
|
+
try {
|
|
51
|
+
const params = inputSchema.parse(input);
|
|
52
|
+
const detector = new BloatDetector(params.threshold || 50);
|
|
53
|
+
// Analyze code
|
|
54
|
+
const result = detector.analyze(params.code, params.language);
|
|
55
|
+
// Format response
|
|
56
|
+
const response = {
|
|
57
|
+
status: result.summary.score > 75 ? 'warning' : result.summary.score > 50 ? 'caution' : 'ok',
|
|
58
|
+
summary: {
|
|
59
|
+
totalIssues: result.summary.totalIssues,
|
|
60
|
+
criticalIssues: result.summary.criticalIssues,
|
|
61
|
+
complexityScore: result.summary.score,
|
|
62
|
+
recommendation: result.summary.recommendation
|
|
63
|
+
}
|
|
64
|
+
};
|
|
65
|
+
// Include metrics if requested
|
|
66
|
+
if (params.includeMetrics) {
|
|
67
|
+
response.metrics = {
|
|
68
|
+
cyclomaticComplexity: result.metrics.cyclomaticComplexity,
|
|
69
|
+
cognitiveComplexity: result.metrics.cognitiveComplexity,
|
|
70
|
+
nestingDepth: result.metrics.nestingDepth,
|
|
71
|
+
linesOfCode: result.metrics.linesOfCode,
|
|
72
|
+
maxFunctionLength: result.metrics.maxFunctionLength,
|
|
73
|
+
averageFunctionLength: result.metrics.averageFunctionLength,
|
|
74
|
+
functionCount: result.metrics.functionCount,
|
|
75
|
+
severity: result.metrics.severity
|
|
76
|
+
};
|
|
77
|
+
}
|
|
78
|
+
// Include issues
|
|
79
|
+
if (result.issues.length > 0) {
|
|
80
|
+
response.issues = result.issues.map(issue => ({
|
|
81
|
+
type: issue.type,
|
|
82
|
+
severity: issue.severity,
|
|
83
|
+
line: issue.line,
|
|
84
|
+
pattern: issue.pattern,
|
|
85
|
+
suggestion: issue.suggestion,
|
|
86
|
+
...(issue.example && { example: issue.example })
|
|
87
|
+
}));
|
|
88
|
+
}
|
|
89
|
+
// Include refactoring priority
|
|
90
|
+
if (params.includeSuggestions && result.refactoringPriority.length > 0) {
|
|
91
|
+
response.refactoringPriority = result.refactoringPriority;
|
|
92
|
+
}
|
|
93
|
+
// Add severity breakdown
|
|
94
|
+
const highSeverity = result.issues.filter(i => i.severity === 'high').length;
|
|
95
|
+
const mediumSeverity = result.issues.filter(i => i.severity === 'medium').length;
|
|
96
|
+
const lowSeverity = result.issues.filter(i => i.severity === 'low').length;
|
|
97
|
+
response.issueBreakdown = {
|
|
98
|
+
high: highSeverity,
|
|
99
|
+
medium: mediumSeverity,
|
|
100
|
+
low: lowSeverity
|
|
101
|
+
};
|
|
102
|
+
// Create summary message
|
|
103
|
+
let message = '';
|
|
104
|
+
if (result.summary.totalIssues === 0) {
|
|
105
|
+
message = 'Clean code! No significant bloat patterns detected.';
|
|
106
|
+
}
|
|
107
|
+
else if (result.summary.criticalIssues > 0) {
|
|
108
|
+
message = `Found ${result.summary.totalIssues} issues (${result.summary.criticalIssues} critical). Priority refactoring needed.`;
|
|
109
|
+
}
|
|
110
|
+
else {
|
|
111
|
+
message = `Found ${result.summary.totalIssues} issues. ${result.summary.recommendation}`;
|
|
112
|
+
}
|
|
113
|
+
response.message = message;
|
|
114
|
+
// Add detailed analysis for educational purposes
|
|
115
|
+
if (params.includeSuggestions) {
|
|
116
|
+
response.analysis = {
|
|
117
|
+
complexity: explainComplexity(result.metrics),
|
|
118
|
+
keyFindings: generateKeyFindings(result.issues),
|
|
119
|
+
actionItems: generateActionItems(result)
|
|
120
|
+
};
|
|
121
|
+
}
|
|
122
|
+
return response;
|
|
123
|
+
}
|
|
124
|
+
catch (error) {
|
|
125
|
+
console.error('Code smell detection error:', error);
|
|
126
|
+
return {
|
|
127
|
+
error: error instanceof Error ? error.message : 'Failed to analyze code',
|
|
128
|
+
status: 'error'
|
|
129
|
+
};
|
|
130
|
+
}
|
|
131
|
+
}
|
|
132
|
+
/**
|
|
133
|
+
* Explain complexity metrics in human terms
|
|
134
|
+
*/
|
|
135
|
+
function explainComplexity(metrics) {
|
|
136
|
+
const parts = [];
|
|
137
|
+
if (metrics.cyclomaticComplexity > 15) {
|
|
138
|
+
parts.push(`High cyclomatic complexity (${metrics.cyclomaticComplexity}). Code has many decision points.`);
|
|
139
|
+
}
|
|
140
|
+
else if (metrics.cyclomaticComplexity > 10) {
|
|
141
|
+
parts.push(`Moderate cyclomatic complexity (${metrics.cyclomaticComplexity}). Consider simplifying.`);
|
|
142
|
+
}
|
|
143
|
+
if (metrics.nestingDepth > 5) {
|
|
144
|
+
parts.push(`Deep nesting (${metrics.nestingDepth} levels). Use early returns to flatten structure.`);
|
|
145
|
+
}
|
|
146
|
+
else if (metrics.nestingDepth > 3) {
|
|
147
|
+
parts.push(`Moderate nesting (${metrics.nestingDepth} levels). Could be reduced.`);
|
|
148
|
+
}
|
|
149
|
+
if (metrics.maxFunctionLength > 100) {
|
|
150
|
+
parts.push(`Function length (${metrics.maxFunctionLength} lines). Consider breaking into smaller functions.`);
|
|
151
|
+
}
|
|
152
|
+
if (metrics.cognitiveComplexity > 20) {
|
|
153
|
+
parts.push(`High cognitive complexity (${metrics.cognitiveComplexity}). Difficult to understand and maintain.`);
|
|
154
|
+
}
|
|
155
|
+
return parts.length > 0
|
|
156
|
+
? parts.join(' ')
|
|
157
|
+
: `Code complexity is within acceptable ranges (score: ${metrics.score}/100).`;
|
|
158
|
+
}
|
|
159
|
+
/**
|
|
160
|
+
* Generate key findings summary
|
|
161
|
+
*/
|
|
162
|
+
function generateKeyFindings(issues) {
|
|
163
|
+
const findings = [];
|
|
164
|
+
const typeCounts = new Map();
|
|
165
|
+
for (const issue of issues) {
|
|
166
|
+
typeCounts.set(issue.type, (typeCounts.get(issue.type) || 0) + 1);
|
|
167
|
+
}
|
|
168
|
+
// Convert to findings
|
|
169
|
+
for (const [type, count] of typeCounts.entries()) {
|
|
170
|
+
if (count === 1) {
|
|
171
|
+
findings.push(`1 instance of ${type.replace(/_/g, ' ')}`);
|
|
172
|
+
}
|
|
173
|
+
else {
|
|
174
|
+
findings.push(`${count} instances of ${type.replace(/_/g, ' ')}`);
|
|
175
|
+
}
|
|
176
|
+
}
|
|
177
|
+
return findings;
|
|
178
|
+
}
|
|
179
|
+
/**
|
|
180
|
+
* Generate prioritized action items
|
|
181
|
+
*/
|
|
182
|
+
function generateActionItems(result) {
|
|
183
|
+
const actions = [];
|
|
184
|
+
let priority = 1;
|
|
185
|
+
// High-priority issues first
|
|
186
|
+
const highIssues = result.issues.filter((i) => i.severity === 'high');
|
|
187
|
+
for (const issue of highIssues.slice(0, 3)) {
|
|
188
|
+
actions.push({
|
|
189
|
+
priority,
|
|
190
|
+
action: `Fix ${issue.type}: ${issue.suggestion}`,
|
|
191
|
+
impact: 'Critical - prevents bugs and improves security'
|
|
192
|
+
});
|
|
193
|
+
priority++;
|
|
194
|
+
}
|
|
195
|
+
// Medium-priority issues
|
|
196
|
+
const mediumIssues = result.issues.filter((i) => i.severity === 'medium');
|
|
197
|
+
for (const issue of mediumIssues.slice(0, 2)) {
|
|
198
|
+
actions.push({
|
|
199
|
+
priority,
|
|
200
|
+
action: `Address ${issue.type}: ${issue.suggestion}`,
|
|
201
|
+
impact: 'Important - improves maintainability'
|
|
202
|
+
});
|
|
203
|
+
priority++;
|
|
204
|
+
}
|
|
205
|
+
// Refactoring priorities from metrics
|
|
206
|
+
if (result.metrics.cyclomaticComplexity > 15) {
|
|
207
|
+
actions.push({
|
|
208
|
+
priority,
|
|
209
|
+
action: 'Extract complex conditional logic into separate functions',
|
|
210
|
+
impact: 'High - reduces cognitive load and improves testability'
|
|
211
|
+
});
|
|
212
|
+
priority++;
|
|
213
|
+
}
|
|
214
|
+
if (result.metrics.nestingDepth > 5) {
|
|
215
|
+
actions.push({
|
|
216
|
+
priority,
|
|
217
|
+
action: 'Use early returns and extract nested blocks',
|
|
218
|
+
impact: 'High - makes code easier to follow'
|
|
219
|
+
});
|
|
220
|
+
priority++;
|
|
221
|
+
}
|
|
222
|
+
if (result.metrics.maxFunctionLength > 100) {
|
|
223
|
+
actions.push({
|
|
224
|
+
priority,
|
|
225
|
+
action: 'Break oversized functions into smaller, focused units (target: 20-30 lines)',
|
|
226
|
+
impact: 'Medium - improves readability and testability'
|
|
227
|
+
});
|
|
228
|
+
}
|
|
229
|
+
return actions;
|
|
230
|
+
}
|
|
231
|
+
//# sourceMappingURL=detect-code-smell.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"detect-code-smell.js","sourceRoot":"","sources":["../../src/tools/detect-code-smell.ts"],"names":[],"mappings":"AAAA;;;;GAIG;AAEH,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAC;AAExB,OAAO,EAAE,aAAa,EAAE,MAAM,0CAA0C,CAAC;AAEzE,MAAM,WAAW,GAAG,CAAC,CAAC,MAAM,CAAC;IAC3B,IAAI,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,CAAC,4CAA4C,CAAC;IACvE,QAAQ,EAAE,CAAC,CAAC,IAAI,CAAC,CAAC,YAAY,EAAE,YAAY,EAAE,QAAQ,CAAC,CAAC,CAAC,OAAO,CAAC,YAAY,CAAC,CAAC,QAAQ,CAAC,sBAAsB,CAAC;IAC/G,SAAS,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE,CAAC,QAAQ,CAAC,2CAA2C,CAAC;IACtF,cAAc,EAAE,CAAC,CAAC,OAAO,EAAE,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,QAAQ,CAAC,qCAAqC,CAAC;IACzF,kBAAkB,EAAE,CAAC,CAAC,OAAO,EAAE,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,QAAQ,CAAC,iCAAiC,CAAC;CAC1F,CAAC,CAAC;AAEH,MAAM,CAAC,MAAM,mBAAmB,GAAS;IACvC,IAAI,EAAE,mBAAmB;IACzB,WAAW,EAAE,sJAAsJ;IACnK,WAAW,EAAE;QACX,IAAI,EAAE,QAAQ;QACd,UAAU,EAAE;YACV,IAAI,EAAE;gBACJ,IAAI,EAAE,QAAQ;gBACd,WAAW,EAAE,4CAA4C;aAC1D;YACD,QAAQ,EAAE;gBACR,IAAI,EAAE,QAAQ;gBACd,IAAI,EAAE,CAAC,YAAY,EAAE,YAAY,EAAE,QAAQ,CAAC;gBAC5C,OAAO,EAAE,YAAY;gBACrB,WAAW,EAAE,sBAAsB;aACpC;YACD,SAAS,EAAE;gBACT,IAAI,EAAE,QAAQ;gBACd,WAAW,EAAE,2CAA2C;aACzD;YACD,cAAc,EAAE;gBACd,IAAI,EAAE,SAAS;gBACf,OAAO,EAAE,IAAI;gBACb,WAAW,EAAE,qCAAqC;aACnD;YACD,kBAAkB,EAAE;gBAClB,IAAI,EAAE,SAAS;gBACf,OAAO,EAAE,IAAI;gBACb,WAAW,EAAE,iCAAiC;aAC/C;SACF;QACD,QAAQ,EAAE,CAAC,MAAM,CAAC;KACnB;CACF,CAAC;AAEF,MAAM,CAAC,KAAK,UAAU,qBAAqB,CAAC,KAAc;IACxD,IAAI,CAAC;QACH,MAAM,MAAM,GAAG,WAAW,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC;QACxC,MAAM,QAAQ,GAAG,IAAI,aAAa,CAAC,MAAM,CAAC,SAAS,IAAI,EAAE,CAAC,CAAC;QAE3D,eAAe;QACf,MAAM,MAAM,GAAG,QAAQ,CAAC,OAAO,CAAC,MAAM,CAAC,IAAI,EAAE,MAAM,CAAC,QAAQ,CAAC,CAAC;QAE9D,kBAAkB;QAClB,MAAM,QAAQ,GAAQ;YACpB,MAAM,EAAE,MAAM,CAAC,OAAO,CAAC,KAAK,GAAG,EAAE,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,MAAM,CAAC,OAAO,CAAC,KAAK,GAAG,EAAE,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,IAAI;YAC5F,OAAO,EAAE;gBACP,WAAW,EAAE,MAAM,CAAC,OAAO,CAAC,WAAW;gBACvC,cAAc,EAAE,MAAM,CAAC,OAAO,CAAC,cAAc;gBAC7C,eAAe,EAAE,MAAM,CAAC,OAAO,CAAC,KAAK;gBACrC,cAAc,EAAE,MAAM,CAAC,OAAO,CAAC,cAAc;aAC9C;SACF,CAAC;QAEF,+BAA+B;QAC/B,IAAI,MAAM,CAAC,cAAc,EAAE,CAAC;YAC1B,QAAQ,CAAC,OAAO,GAAG;gBACjB,oBAAoB,EAAE,MAAM,CAAC,OAAO,CAAC,oBAAoB;gBACzD,mBAAmB,EAAE,MAAM,CAAC,OAAO,CAAC,mBAAmB;gBACvD,YAAY,EAAE,MAAM,CAAC,OAAO,CAAC,YAAY;gBACzC,WAAW,EAAE,MAAM,CAAC,OAAO,CAAC,WAAW;gBACvC,iBAAiB,EAAE,MAAM,CAAC,OAAO,CAAC,iBAAiB;gBACnD,qBAAqB,EAAE,MAAM,CAAC,OAAO,CAAC,qBAAqB;gBAC3D,aAAa,EAAE,MAAM,CAAC,OAAO,CAAC,aAAa;gBAC3C,QAAQ,EAAE,MAAM,CAAC,OAAO,CAAC,QAAQ;aAClC,CAAC;QACJ,CAAC;QAED,iBAAiB;QACjB,IAAI,MAAM,CAAC,MAAM,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YAC7B,QAAQ,CAAC,MAAM,GAAG,MAAM,CAAC,MAAM,CAAC,GAAG,CAAC,KAAK,CAAC,EAAE,CAAC,CAAC;gBAC5C,IAAI,EAAE,KAAK,CAAC,IAAI;gBAChB,QAAQ,EAAE,KAAK,CAAC,QAAQ;gBACxB,IAAI,EAAE,KAAK,CAAC,IAAI;gBAChB,OAAO,EAAE,KAAK,CAAC,OAAO;gBACtB,UAAU,EAAE,KAAK,CAAC,UAAU;gBAC5B,GAAG,CAAC,KAAK,CAAC,OAAO,IAAI,EAAE,OAAO,EAAE,KAAK,CAAC,OAAO,EAAE,CAAC;aACjD,CAAC,CAAC,CAAC;QACN,CAAC;QAED,+BAA+B;QAC/B,IAAI,MAAM,CAAC,kBAAkB,IAAI,MAAM,CAAC,mBAAmB,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YACvE,QAAQ,CAAC,mBAAmB,GAAG,MAAM,CAAC,mBAAmB,CAAC;QAC5D,CAAC;QAED,yBAAyB;QACzB,MAAM,YAAY,GAAG,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,QAAQ,KAAK,MAAM,CAAC,CAAC,MAAM,CAAC;QAC7E,MAAM,cAAc,GAAG,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,QAAQ,KAAK,QAAQ,CAAC,CAAC,MAAM,CAAC;QACjF,MAAM,WAAW,GAAG,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,QAAQ,KAAK,KAAK,CAAC,CAAC,MAAM,CAAC;QAE3E,QAAQ,CAAC,cAAc,GAAG;YACxB,IAAI,EAAE,YAAY;YAClB,MAAM,EAAE,cAAc;YACtB,GAAG,EAAE,WAAW;SACjB,CAAC;QAEF,yBAAyB;QACzB,IAAI,OAAO,GAAG,EAAE,CAAC;QACjB,IAAI,MAAM,CAAC,OAAO,CAAC,WAAW,KAAK,CAAC,EAAE,CAAC;YACrC,OAAO,GAAG,qDAAqD,CAAC;QAClE,CAAC;aAAM,IAAI,MAAM,CAAC,OAAO,CAAC,cAAc,GAAG,CAAC,EAAE,CAAC;YAC7C,OAAO,GAAG,SAAS,MAAM,CAAC,OAAO,CAAC,WAAW,YAAY,MAAM,CAAC,OAAO,CAAC,cAAc,0CAA0C,CAAC;QACnI,CAAC;aAAM,CAAC;YACN,OAAO,GAAG,SAAS,MAAM,CAAC,OAAO,CAAC,WAAW,YAAY,MAAM,CAAC,OAAO,CAAC,cAAc,EAAE,CAAC;QAC3F,CAAC;QAED,QAAQ,CAAC,OAAO,GAAG,OAAO,CAAC;QAE3B,iDAAiD;QACjD,IAAI,MAAM,CAAC,kBAAkB,EAAE,CAAC;YAC9B,QAAQ,CAAC,QAAQ,GAAG;gBAClB,UAAU,EAAE,iBAAiB,CAAC,MAAM,CAAC,OAAO,CAAC;gBAC7C,WAAW,EAAE,mBAAmB,CAAC,MAAM,CAAC,MAAM,CAAC;gBAC/C,WAAW,EAAE,mBAAmB,CAAC,MAAM,CAAC;aACzC,CAAC;QACJ,CAAC;QAED,OAAO,QAAQ,CAAC;IAElB,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QACf,OAAO,CAAC,KAAK,CAAC,6BAA6B,EAAE,KAAK,CAAC,CAAC;QACpD,OAAO;YACL,KAAK,EAAE,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,wBAAwB;YACxE,MAAM,EAAE,OAAO;SAChB,CAAC;IACJ,CAAC;AACH,CAAC;AAED;;GAEG;AACH,SAAS,iBAAiB,CAAC,OAAY;IACrC,MAAM,KAAK,GAAa,EAAE,CAAC;IAE3B,IAAI,OAAO,CAAC,oBAAoB,GAAG,EAAE,EAAE,CAAC;QACtC,KAAK,CAAC,IAAI,CAAC,+BAA+B,OAAO,CAAC,oBAAoB,mCAAmC,CAAC,CAAC;IAC7G,CAAC;SAAM,IAAI,OAAO,CAAC,oBAAoB,GAAG,EAAE,EAAE,CAAC;QAC7C,KAAK,CAAC,IAAI,CAAC,mCAAmC,OAAO,CAAC,oBAAoB,0BAA0B,CAAC,CAAC;IACxG,CAAC;IAED,IAAI,OAAO,CAAC,YAAY,GAAG,CAAC,EAAE,CAAC;QAC7B,KAAK,CAAC,IAAI,CAAC,iBAAiB,OAAO,CAAC,YAAY,mDAAmD,CAAC,CAAC;IACvG,CAAC;SAAM,IAAI,OAAO,CAAC,YAAY,GAAG,CAAC,EAAE,CAAC;QACpC,KAAK,CAAC,IAAI,CAAC,qBAAqB,OAAO,CAAC,YAAY,6BAA6B,CAAC,CAAC;IACrF,CAAC;IAED,IAAI,OAAO,CAAC,iBAAiB,GAAG,GAAG,EAAE,CAAC;QACpC,KAAK,CAAC,IAAI,CAAC,oBAAoB,OAAO,CAAC,iBAAiB,oDAAoD,CAAC,CAAC;IAChH,CAAC;IAED,IAAI,OAAO,CAAC,mBAAmB,GAAG,EAAE,EAAE,CAAC;QACrC,KAAK,CAAC,IAAI,CAAC,8BAA8B,OAAO,CAAC,mBAAmB,0CAA0C,CAAC,CAAC;IAClH,CAAC;IAED,OAAO,KAAK,CAAC,MAAM,GAAG,CAAC;QACrB,CAAC,CAAC,KAAK,CAAC,IAAI,CAAC,GAAG,CAAC;QACjB,CAAC,CAAC,uDAAuD,OAAO,CAAC,KAAK,QAAQ,CAAC;AACnF,CAAC;AAED;;GAEG;AACH,SAAS,mBAAmB,CAAC,MAAa;IACxC,MAAM,QAAQ,GAAa,EAAE,CAAC;IAC9B,MAAM,UAAU,GAAG,IAAI,GAAG,EAAkB,CAAC;IAE7C,KAAK,MAAM,KAAK,IAAI,MAAM,EAAE,CAAC;QAC3B,UAAU,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,EAAE,CAAC,UAAU,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC;IACpE,CAAC;IAED,sBAAsB;IACtB,KAAK,MAAM,CAAC,IAAI,EAAE,KAAK,CAAC,IAAI,UAAU,CAAC,OAAO,EAAE,EAAE,CAAC;QACjD,IAAI,KAAK,KAAK,CAAC,EAAE,CAAC;YAChB,QAAQ,CAAC,IAAI,CAAC,iBAAiB,IAAI,CAAC,OAAO,CAAC,IAAI,EAAE,GAAG,CAAC,EAAE,CAAC,CAAC;QAC5D,CAAC;aAAM,CAAC;YACN,QAAQ,CAAC,IAAI,CAAC,GAAG,KAAK,iBAAiB,IAAI,CAAC,OAAO,CAAC,IAAI,EAAE,GAAG,CAAC,EAAE,CAAC,CAAC;QACpE,CAAC;IACH,CAAC;IAED,OAAO,QAAQ,CAAC;AAClB,CAAC;AAED;;GAEG;AACH,SAAS,mBAAmB,CAAC,MAAW;IACtC,MAAM,OAAO,GAAgE,EAAE,CAAC;IAChF,IAAI,QAAQ,GAAG,CAAC,CAAC;IAEjB,6BAA6B;IAC7B,MAAM,UAAU,GAAG,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAM,EAAE,EAAE,CAAC,CAAC,CAAC,QAAQ,KAAK,MAAM,CAAC,CAAC;IAC3E,KAAK,MAAM,KAAK,IAAI,UAAU,CAAC,KAAK,CAAC,CAAC,EAAE,CAAC,CAAC,EAAE,CAAC;QAC3C,OAAO,CAAC,IAAI,CAAC;YACX,QAAQ;YACR,MAAM,EAAE,OAAO,KAAK,CAAC,IAAI,KAAK,KAAK,CAAC,UAAU,EAAE;YAChD,MAAM,EAAE,gDAAgD;SACzD,CAAC,CAAC;QACH,QAAQ,EAAE,CAAC;IACb,CAAC;IAED,yBAAyB;IACzB,MAAM,YAAY,GAAG,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAM,EAAE,EAAE,CAAC,CAAC,CAAC,QAAQ,KAAK,QAAQ,CAAC,CAAC;IAC/E,KAAK,MAAM,KAAK,IAAI,YAAY,CAAC,KAAK,CAAC,CAAC,EAAE,CAAC,CAAC,EAAE,CAAC;QAC7C,OAAO,CAAC,IAAI,CAAC;YACX,QAAQ;YACR,MAAM,EAAE,WAAW,KAAK,CAAC,IAAI,KAAK,KAAK,CAAC,UAAU,EAAE;YACpD,MAAM,EAAE,sCAAsC;SAC/C,CAAC,CAAC;QACH,QAAQ,EAAE,CAAC;IACb,CAAC;IAED,sCAAsC;IACtC,IAAI,MAAM,CAAC,OAAO,CAAC,oBAAoB,GAAG,EAAE,EAAE,CAAC;QAC7C,OAAO,CAAC,IAAI,CAAC;YACX,QAAQ;YACR,MAAM,EAAE,2DAA2D;YACnE,MAAM,EAAE,wDAAwD;SACjE,CAAC,CAAC;QACH,QAAQ,EAAE,CAAC;IACb,CAAC;IAED,IAAI,MAAM,CAAC,OAAO,CAAC,YAAY,GAAG,CAAC,EAAE,CAAC;QACpC,OAAO,CAAC,IAAI,CAAC;YACX,QAAQ;YACR,MAAM,EAAE,6CAA6C;YACrD,MAAM,EAAE,oCAAoC;SAC7C,CAAC,CAAC;QACH,QAAQ,EAAE,CAAC;IACb,CAAC;IAED,IAAI,MAAM,CAAC,OAAO,CAAC,iBAAiB,GAAG,GAAG,EAAE,CAAC;QAC3C,OAAO,CAAC,IAAI,CAAC;YACX,QAAQ;YACR,MAAM,EAAE,6EAA6E;YACrF,MAAM,EAAE,+CAA+C;SACxD,CAAC,CAAC;IACL,CAAC;IAED,OAAO,OAAO,CAAC;AACjB,CAAC"}
|