mcp-ai-agent-guidelines 0.7.2 → 0.7.4

package/dist/tools/clean-code-scorer.js

@@ -0,0 +1,359 @@
+import { z } from "zod";
+import { buildReferencesSection } from "./shared/prompt-utils.js";
+const CleanCodeScorerSchema = z.object({
+    projectPath: z.string().optional(),
+    codeContent: z.string().optional(),
+    language: z.string().optional(),
+    framework: z.string().optional(),
+    coverageMetrics: z
+        .object({
+        statements: z.number().min(0).max(100).optional(),
+        branches: z.number().min(0).max(100).optional(),
+        functions: z.number().min(0).max(100).optional(),
+        lines: z.number().min(0).max(100).optional(),
+    })
+        .optional(),
+    includeReferences: z.boolean().optional().default(true),
+    includeMetadata: z.boolean().optional().default(true),
+    inputFile: z.string().optional(),
+});
+export async function cleanCodeScorer(args) {
+    const input = CleanCodeScorerSchema.parse(args);
+    const scoreResult = calculateCleanCodeScore(input);
+    const references = input.includeReferences
+        ? buildReferencesSection([
+            "Clean Code principles: https://www.freecodecamp.org/news/clean-coding-for-beginners/",
+            "Code quality metrics: https://docs.sonarqube.org/latest/user-guide/metric-definitions/",
+            "Test coverage best practices: https://martinfowler.com/bliki/TestCoverage.html",
+            "TypeScript best practices: https://www.typescriptlang.org/docs/handbook/declaration-files/do-s-and-don-ts.html",
+        ])
+        : undefined;
+    const metadata = input.includeMetadata
+        ? [
+            "### Metadata",
+            `- Updated: ${new Date().toISOString().slice(0, 10)}`,
+            "- Source tool: mcp_ai-agent-guid_clean-code-scorer",
+            input.inputFile ? `- Input file: ${input.inputFile}` : undefined,
+        ]
+            .filter(Boolean)
+            .join("\n")
+        : undefined;
+    return {
+        content: [
+            {
+                type: "text",
+                text: `## 🏆 Clean Code Score Report
+
+${metadata ? `${metadata}\n` : ""}
+### 📊 Overall Score
+**${scoreResult.overallScore}/100** - ${scoreResult.scoreDescription}
+
+${generateScoreBar(scoreResult.overallScore)}
+
+### 📈 Category Breakdown
+
+${Object.entries(scoreResult.categories)
+                    .map(([category, data]) => {
+                    const categoryName = category
+                        .replace(/([A-Z])/g, " $1")
+                        .replace(/^./, (str) => str.toUpperCase());
+                    const percentage = (data.score / data.weight) * 100;
+                    const status = getScoreStatus(percentage);
+                    return `#### ${status} ${categoryName}
+- Score: ${data.score}/${data.weight} (${percentage.toFixed(1)}%)
+${data.issues.length > 0 ? `- Issues:\n${data.issues.map((issue) => `  - ${issue}`).join("\n")}` : "- ✅ No issues found"}`;
+                })
+                    .join("\n\n")}
+
+${scoreResult.achievements.length > 0 ? `### 🎉 Achievements\n${scoreResult.achievements.map((a, i) => `${i + 1}. ${a}`).join("\n")}\n` : ""}
+
+### 💡 Recommendations
+${scoreResult.recommendations.map((rec, index) => `${index + 1}. ${rec}`).join("\n")}
+
+### 🚀 Next Steps
+${scoreResult.nextSteps.map((step, index) => `${index + 1}. ${step}`).join("\n")}
+
+### 📊 Score Distribution
+\`\`\`
+Code Hygiene:   ${generateMiniBar(scoreResult.categories.codeHygiene.score, scoreResult.categories.codeHygiene.weight)}
+Test Coverage:  ${generateMiniBar(scoreResult.categories.testCoverage.score, scoreResult.categories.testCoverage.weight)}
+TypeScript:     ${generateMiniBar(scoreResult.categories.typeScript.score, scoreResult.categories.typeScript.weight)}
+Linting:        ${generateMiniBar(scoreResult.categories.linting.score, scoreResult.categories.linting.weight)}
+Documentation:  ${generateMiniBar(scoreResult.categories.documentation.score, scoreResult.categories.documentation.weight)}
+Security:       ${generateMiniBar(scoreResult.categories.security.score, scoreResult.categories.security.weight)}
+\`\`\`
+
+${references ? `\n${references}\n` : ""}
+### ⚠️ Disclaimer
+- This score is calculated based on multiple quality metrics and best practices
+- Achieving 100/100 requires excellence across all categories
+- Regular monitoring and improvement is recommended
+`,
+            },
+        ],
+    };
+}
+function calculateCleanCodeScore(input) {
+    const categories = {
+        codeHygiene: { score: 0, weight: 20, issues: [] },
+        testCoverage: { score: 0, weight: 25, issues: [] },
+        typeScript: { score: 0, weight: 20, issues: [] },
+        linting: { score: 0, weight: 15, issues: [] },
+        documentation: { score: 0, weight: 10, issues: [] },
+        security: { score: 0, weight: 10, issues: [] },
+    };
+    const recommendations = [];
+    const nextSteps = [];
+    const achievements = [];
+    // Code Hygiene Scoring
+    if (input.codeContent) {
+        const hygieneResult = analyzeCodeHygiene(input.codeContent, input.language || "javascript");
+        categories.codeHygiene.score = Math.round((hygieneResult.score / 100) * categories.codeHygiene.weight);
+        categories.codeHygiene.issues = hygieneResult.issues;
+        if (hygieneResult.score >= 85) {
+            achievements.push("Excellent code hygiene maintained");
+        }
+        else if (hygieneResult.score < 70) {
+            recommendations.push("Improve code hygiene by addressing identified issues");
+            nextSteps.push("Run code cleanup and remove dead code");
+        }
+    }
+    else {
+        // If no code provided, assume good hygiene
+        categories.codeHygiene.score = categories.codeHygiene.weight;
+    }
+    // Test Coverage Scoring
+    if (input.coverageMetrics) {
+        const coverage = input.coverageMetrics;
+        const avgCoverage = ((coverage.statements || 0) +
+            (coverage.branches || 0) +
+            (coverage.functions || 0) +
+            (coverage.lines || 0)) /
+            4;
+        categories.testCoverage.score = Math.round((avgCoverage / 100) * categories.testCoverage.weight);
+        if (avgCoverage < 80) {
+            categories.testCoverage.issues.push(`Average coverage ${avgCoverage.toFixed(1)}% is below 80% target`);
+        }
+        if ((coverage.statements || 0) < 80) {
+            categories.testCoverage.issues.push(`Statement coverage ${coverage.statements}% is below 80%`);
+        }
+        if ((coverage.branches || 0) < 80) {
+            categories.testCoverage.issues.push(`Branch coverage ${coverage.branches}% is below 80%`);
+        }
+        if ((coverage.functions || 0) < 80) {
+            categories.testCoverage.issues.push(`Function coverage ${coverage.functions}% is below 80%`);
+        }
+        if (avgCoverage >= 90) {
+            achievements.push("Excellent test coverage achieved (≥90%)");
+        }
+        else if (avgCoverage < 70) {
+            recommendations.push("Increase test coverage to at least 80% across all metrics");
+            nextSteps.push("Identify and test uncovered code paths");
+        }
+    }
+    else {
+        // Default to assume 80% coverage
+        categories.testCoverage.score = Math.round(0.8 * categories.testCoverage.weight);
+    }
+    // TypeScript Scoring (assume passing if no errors)
+    categories.typeScript.score = categories.typeScript.weight;
+    achievements.push("TypeScript strict mode enabled and passing");
+    // Linting Scoring (assume Biome is configured and passing)
+    categories.linting.score = categories.linting.weight;
+    achievements.push("Biome linting and formatting configured");
+    // Documentation Scoring
+    if (input.codeContent) {
+        const docScore = analyzeDocumentation(input.codeContent);
+        categories.documentation.score = Math.round((docScore / 100) * categories.documentation.weight);
+        if (docScore < 70) {
+            categories.documentation.issues.push("Insufficient code documentation and comments");
+            recommendations.push("Add comprehensive documentation and comments");
+        }
+        if (docScore >= 90) {
+            achievements.push("Well-documented codebase");
+        }
+    }
+    else {
+        categories.documentation.score = Math.round(0.8 * categories.documentation.weight);
+    }
+    // Security Scoring
+    if (input.codeContent) {
+        const securityScore = analyzeSecurityIssues(input.codeContent);
+        categories.security.score = Math.round((securityScore.score / 100) * categories.security.weight);
+        categories.security.issues = securityScore.issues;
+        if (securityScore.score < 80) {
+            recommendations.push("Address security vulnerabilities immediately");
+            nextSteps.push("Run security audit and fix identified issues");
+        }
+        if (securityScore.score === 100) {
+            achievements.push("No security vulnerabilities detected");
+        }
+    }
+    else {
+        categories.security.score = categories.security.weight;
+        achievements.push("Security checks passed");
+    }
+    // Calculate overall score
+    const overallScore = Math.round(Object.values(categories).reduce((sum, cat) => sum + cat.score, 0));
+    let scoreDescription = "";
+    if (overallScore >= 95) {
+        scoreDescription = "🏆 Perfect - Clean Code Excellence";
+    }
+    else if (overallScore >= 90) {
+        scoreDescription = "✨ Excellent - Near Perfect Quality";
+    }
+    else if (overallScore >= 80) {
+        scoreDescription = "✅ Very Good - High Quality Code";
+    }
+    else if (overallScore >= 70) {
+        scoreDescription = "👍 Good - Quality Standards Met";
+    }
+    else if (overallScore >= 60) {
+        scoreDescription = "⚠️ Fair - Improvements Needed";
+    }
+    else {
+        scoreDescription = "❌ Poor - Significant Issues";
+    }
+    // Add general recommendations
+    if (overallScore < 100) {
+        if (recommendations.length === 0) {
+            recommendations.push("Continue maintaining current quality standards");
+        }
+        recommendations.push("Regular code reviews and pair programming sessions");
+        recommendations.push("Automated quality gates in CI/CD pipeline");
+    }
+    if (nextSteps.length === 0) {
+        if (overallScore === 100) {
+            nextSteps.push("Maintain this excellent quality level");
+            nextSteps.push("Share best practices with the team");
+        }
+        else {
+            nextSteps.push("Focus on lowest scoring categories first");
+            nextSteps.push("Set up automated quality monitoring");
+        }
+    }
+    return {
+        overallScore,
+        scoreDescription,
+        categories,
+        recommendations,
+        nextSteps,
+        achievements,
+    };
+}
+function analyzeCodeHygiene(code, language) {
+    let score = 100;
+    const issues = [];
+    // Check for TODOs/FIXMEs
+    if (code.includes("TODO") || code.includes("FIXME")) {
+        score -= 5;
+        issues.push("TODO or FIXME comments found");
+    }
+    // Check for debug statements
+    if ((code.includes("console.log") || code.includes("print(")) &&
+        (language === "javascript" ||
+            language === "typescript" ||
+            language === "python")) {
+        score -= 10;
+        issues.push("Debug statements found");
+    }
+    // Check for hardcoded credentials
+    if (/(apiKey|api_key|password|secret|token)\s*=\s*['"][^'"]+['"]/i.test(code)) {
+        score -= 20;
+        issues.push("Potential hardcoded credentials detected");
+    }
+    // Check for commented code
+    const commentedLines = (code.match(/^\s*(\/\/|#)\s*(const|let|var|def|function)/gm) || []).length;
+    if (commentedLines > 3) {
+        score -= 5;
+        issues.push(`${commentedLines} lines of commented code found`);
+    }
+    // Check for complex functions (>50 lines)
+    const functionMatches = code.match(/function\s+\w+\s*\([^)]*\)\s*{[\s\S]*?}/g) || [];
+    for (const func of functionMatches) {
+        const lines = func.split("\n").length;
+        if (lines > 50) {
+            score -= 10;
+            issues.push("Complex function detected (>50 lines)");
+            break;
+        }
+    }
+    return { score: Math.max(0, score), issues };
+}
+function analyzeDocumentation(code) {
+    let score = 70; // Base score
+    // Check for JSDoc/docstring comments
+    const docComments = (code.match(/\/\*\*[\s\S]*?\*\/|'''[\s\S]*?'''|"""[\s\S]*?"""/g) || []).length;
+    if (docComments > 0) {
+        score += 15;
+    }
+    // Check for inline comments
+    const inlineComments = (code.match(/\/\/.*|#.*/g) || []).length;
+    if (inlineComments > 5) {
+        score += 15;
+    }
+    // Check for README or documentation keywords
+    if (/README|CONTRIBUTING|CHANGELOG/i.test(code)) {
+        score += 10;
+    }
+    return Math.min(100, score);
+}
+function analyzeSecurityIssues(code) {
+    let score = 100;
+    const issues = [];
+    // Check for eval() or exec()
+    if (/\b(eval|exec)\s*\(/i.test(code)) {
+        score -= 30;
+        issues.push("Use of eval() or exec() detected - security risk");
+    }
+    // Check for SQL injection risks
+    if (/(SELECT|INSERT|UPDATE|DELETE)\s+.*\+.*/.test(code)) {
+        score -= 25;
+        issues.push("Potential SQL injection vulnerability");
+    }
+    // Check for XSS risks
+    if (/innerHTML\s*=|dangerouslySetInnerHTML/i.test(code)) {
+        score -= 20;
+        issues.push("Potential XSS vulnerability with innerHTML");
+    }
+    // Check for hardcoded secrets
+    if (/(secret|password|api_key|apiKey|token|auth|credential)\s*=\s*['"][^'"]+['"]/i.test(code)) {
+        score -= 25;
+        issues.push("Hardcoded secrets or credentials found");
+    }
+    return { score: Math.max(0, score), issues };
+}
+function generateScoreBar(score) {
+    const barLength = 50;
+    const filledLength = Math.round((score / 100) * barLength);
+    const emptyLength = barLength - filledLength;
+    const filled = "█".repeat(filledLength);
+    const empty = "░".repeat(emptyLength);
+    let color = "🔴";
+    if (score >= 90)
+        color = "🟢";
+    else if (score >= 70)
+        color = "🟡";
+    else if (score >= 50)
+        color = "🟠";
+    return `${color} [${filled}${empty}] ${score}%`;
+}
+function generateMiniBar(score, weight) {
+    const percentage = (score / weight) * 100;
+    const barLength = 20;
+    const filledLength = Math.round((percentage / 100) * barLength);
+    const emptyLength = barLength - filledLength;
+    const filled = "█".repeat(filledLength);
+    const empty = "░".repeat(emptyLength);
+    return `[${filled}${empty}] ${score}/${weight}`;
+}
+function getScoreStatus(percentage) {
+    if (percentage >= 90)
+        return "🟢";
+    if (percentage >= 70)
+        return "🟡";
+    if (percentage >= 50)
+        return "🟠";
+    return "🔴";
+}
+//# sourceMappingURL=clean-code-scorer.js.map

package/dist/tools/clean-code-scorer.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"clean-code-scorer.js","sourceRoot":"","sources":["../../src/tools/clean-code-scorer.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAC;AACxB,OAAO,EAAE,sBAAsB,EAAE,MAAM,0BAA0B,CAAC;AAElE,MAAM,qBAAqB,GAAG,CAAC,CAAC,MAAM,CAAC;IACtC,WAAW,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;IAClC,WAAW,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;IAClC,QAAQ,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;IAC/B,SAAS,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;IAChC,eAAe,EAAE,CAAC;SAChB,MAAM,CAAC;QACP,UAAU,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,GAAG,CAAC,CAAC,QAAQ,EAAE;QACjD,QAAQ,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,GAAG,CAAC,CAAC,QAAQ,EAAE;QAC/C,SAAS,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,GAAG,CAAC,CAAC,QAAQ,EAAE;QAChD,KAAK,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,GAAG,CAAC,CAAC,QAAQ,EAAE;KAC5C,CAAC;SACD,QAAQ,EAAE;IACZ,iBAAiB,EAAE,CAAC,CAAC,OAAO,EAAE,CAAC,QAAQ,EAAE,CAAC,OAAO,CAAC,IAAI,CAAC;IACvD,eAAe,EAAE,CAAC,CAAC,OAAO,EAAE,CAAC,QAAQ,EAAE,CAAC,OAAO,CAAC,IAAI,CAAC;IACrD,SAAS,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;CAChC,CAAC,CAAC;AAoBH,MAAM,CAAC,KAAK,UAAU,eAAe,CAAC,IAAa;IAClD,MAAM,KAAK,GAAG,qBAAqB,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IAEhD,MAAM,WAAW,GAAG,uBAAuB,CAAC,KAAK,CAAC,CAAC;IACnD,MAAM,UAAU,GAAG,KAAK,CAAC,iBAAiB;QACzC,CAAC,CAAC,sBAAsB,CAAC;YACvB,sFAAsF;YACtF,wFAAwF;YACxF,gFAAgF;YAChF,gHAAgH;SAChH,CAAC;QACH,CAAC,CAAC,SAAS,CAAC;IAEb,MAAM,QAAQ,GAAG,KAAK,CAAC,eAAe;QACrC,CAAC,CAAC;YACA,cAAc;YACd,cAAc,IAAI,IAAI,EAAE,CAAC,WAAW,EAAE,CAAC,KAAK,CAAC,CAAC,EAAE,EAAE,CAAC,EAAE;YACrD,oDAAoD;YACpD,KAAK,CAAC,SAAS,CAAC,CAAC,CAAC,iBAAiB,KAAK,CAAC,SAAS,EAAE,CAAC,CAAC,CAAC,SAAS;SAChE;aACC,MAAM,CAAC,OAAO,CAAC;aACf,IAAI,CAAC,IAAI,CAAC;QACb,CAAC,CAAC,SAAS,CAAC;IAEb,OAAO;QACN,OAAO,EAAE;YACR;gBACC,IAAI,EAAE,MAAe;gBACrB,IAAI,EAAE;;EAER,QAAQ,CAAC,CAAC,CAAC,GAAG,QAAQ,IAAI,CAAC,CAAC,CAAC,EAAE;;IAE7B,WAAW,CAAC,YAAY,YAAY,WAAW,CAAC,gBAAgB;;EAElE,gBAAgB,CAAC,WAAW,CAAC,YAAY,CAAC;;;;EAI1C,MAAM,CAAC,OAAO,CAAC,WAAW,CAAC,UAAU,CAAC;qBACtC,GAAG,CAAC,CAAC,CAAC,QAAQ,EAAE,IAAI,CAAC,EAAE,EAAE;oBACzB,MAAM,YAAY,GAAG,QAAQ;yBAC3B,OAAO,CAAC,UAAU,EAAE,KAAK,CAAC;yBAC1B,OAAO,CAAC,IAAI,EAAE,CAAC,GAAG,EAAE,EAAE,CAAC,GAAG,CAAC,WAAW,EAAE,CAAC,CAAC;oBAC5C,MAAM,UAAU,GAAG,CAAC,IAAI,CAAC,KAAK,GAAG,IAAI,CAAC,MAAM,CAAC,GAAG,GAAG,CAAC;oBACpD,MAAM,MAAM,GAAG,cAAc,CAAC,UAAU,CAAC,CAAC;oBAE1C,OAAO,QAAQ,MAAM,IAAI,YAAY;WAC5B,IAAI,CAAC,KAAK,IAAI,IAAI,CAAC,MAAM,KAAK,UAAU,CAAC,OAAO,CAAC,CAAC,CAAC;EAC5D,IAAI,CAAC,MAAM,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,cAAc,IAAI,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,OAAO,KAAK,EAAE,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC,CAAC,qBAAqB,EAAE,CAAC;gBAC1H,CAAC,CAAC;qBACD,IAAI,CAAC,MAAM,CAAC;;EAEZ,WAAW,CAAC,YAAY,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,wBAAwB,WAAW,CAAC,YAAY,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,GAAG,CAAC,GAAG,CAAC,KAAK,CAAC,EAAE,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE;;;EAG1I,WAAW,CAAC,eAAe,CAAC,GAAG,CAAC,CAAC,GAAG,EAAE,KAAK,EAAE,EAAE,CAAC,GAAG,KAAK,GAAG,CAAC,KAAK,GAAG,EAAE,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC;;;EAGlF,WAAW,CAAC,SAAS,CAAC,GAAG,CAAC,CAAC,IAAI,EAAE,KAAK,EAAE,EAAE,CAAC,GAAG,KAAK,GAAG,CAAC,KAAK,IAAI,EAAE,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC;;;;kBAI9D,eAAe,CAAC,WAAW,CAAC,UAAU,CAAC,WAAW,CAAC,KAAK,EAAE,WAAW,CAAC,UAAU,CAAC,WAAW,CAAC,MAAM,CAAC;kBACpG,eAAe,CAAC,WAAW,CAAC,UAAU,CAAC,YAAY,CAAC,KAAK,EAAE,WAAW,CAAC,UAAU,CAAC,YAAY,CAAC,MAAM,CAAC;kBACtG,eAAe,CAAC,WAAW,CAAC,UAAU,CAAC,UAAU,CAAC,KAAK,EAAE,WAAW,CAAC,UAAU,CAAC,UAAU,CAAC,MAAM,CAAC;kBAClG,eAAe,CAAC,WAAW,CAAC,UAAU,CAAC,OAAO,CAAC,KAAK,EAAE,WAAW,CAAC,UAAU,CAAC,OAAO,CAAC,MAAM,CAAC;kBAC5F,eAAe,CAAC,WAAW,CAAC,UAAU,CAAC,aAAa,CAAC,KAAK,EAAE,WAAW,CAAC,UAAU,CAAC,aAAa,CAAC,MAAM,CAAC;kBACxG,eAAe,CAAC,WAAW,CAAC,UAAU,CAAC,QAAQ,CAAC,KAAK,EAAE,WAAW,CAAC,UAAU,CAAC,QAAQ,CAAC,MAAM,CAAC;;;EAG9G,UAAU,CAAC,CAAC,CAAC,KAAK,UAAU,IAAI,CAAC,CAAC,CAAC,EAAE;;;;;CAKtC;aACG;SACD;KACD,CAAC;AACH,CAAC;AAED,SAAS,uBAAuB,CAAC,KAA2B;IAC3D,MAAM,UAAU,GAAG;QAClB,WAAW,EAAE,EAAE,KAAK,EAAE,CAAC,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,EAAE,EAAc,EAAE;QAC7D,YAAY,EAAE,EAAE,KAAK,EAAE,CAAC,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,EAAE,EAAc,EAAE;QAC9D,UAAU,EAAE,EAAE,KAAK,EAAE,CAAC,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,EAAE,EAAc,EAAE;QAC5D,OAAO,EAAE,EAAE,KAAK,EAAE,CAAC,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,EAAE,EAAc,EAAE;QACzD,aAAa,EAAE,EAAE,KAAK,EAAE,CAAC,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,EAAE,EAAc,EAAE;QAC/D,QAAQ,EAAE,EAAE,KAAK,EAAE,CAAC,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,EAAE,EAAc,EAAE;KAC1D,CAAC;IAEF,MAAM,eAAe,GAAa,EAAE,CAAC;IACrC,MAAM,SAAS,GAAa,EAAE,CAAC;IAC/B,MAAM,YAAY,GAAa,EAAE,CAAC;IAElC,uBAAuB;IACvB,IAAI,KAAK,CAAC,WAAW,EAAE,CAAC;QACvB,MAAM,aAAa,GAAG,kBAAkB,CACvC,KAAK,CAAC,WAAW,EACjB,KAAK,CAAC,QAAQ,IAAI,YAAY,CAC9B,CAAC;QACF,UAAU,CAAC,WAAW,CAAC,KAAK,GAAG,IAAI,CAAC,KAAK,CACxC,CAAC,aAAa,CAAC,KAAK,GAAG,GAAG,CAAC,GAAG,UAAU,CAAC,WAAW,CAAC,MAAM,CAC3D,CAAC;QACF,UAAU,CAAC,WAAW,CAAC,MAAM,GAAG,aAAa,CAAC,MAAM,CAAC;QAErD,IAAI,aAAa,CAAC,KAAK,IAAI,EAAE,EAAE,CAAC;YAC/B,YAAY,CAAC,IAAI,CAAC,mCAAmC,CAAC,CAAC;QACxD,CAAC;aAAM,IAAI,aAAa,CAAC,KAAK,GAAG,EAAE,EAAE,CAAC;YACrC,eAAe,CAAC,IAAI,CACnB,sDAAsD,CACtD,CAAC;YACF,SAAS,CAAC,IAAI,CAAC,uCAAuC,CAAC,CAAC;QACzD,CAAC;IACF,CAAC;SAAM,CAAC;QACP,2CAA2C;QAC3C,UAAU,CAAC,WAAW,CAAC,KAAK,GAAG,UAAU,CAAC,WAAW,CAAC,MAAM,CAAC;IAC9D,CAAC;IAED,wBAAwB;IACxB,IAAI,KAAK,CAAC,eAAe,EAAE,CAAC;QAC3B,MAAM,QAAQ,GAAG,KAAK,CAAC,eAAe,CAAC;QACvC,MAAM,WAAW,GAChB,CAAC,CAAC,QAAQ,CAAC,UAAU,IAAI,CAAC,CAAC;YAC1B,CAAC,QAAQ,CAAC,QAAQ,IAAI,CAAC,CAAC;YACxB,CAAC,QAAQ,CAAC,SAAS,IAAI,CAAC,CAAC;YACzB,CAAC,QAAQ,CAAC,KAAK,IAAI,CAAC,CAAC,CAAC;YACvB,CAAC,CAAC;QAEH,UAAU,CAAC,YAAY,CAAC,KAAK,GAAG,IAAI,CAAC,KAAK,CACzC,CAAC,WAAW,GAAG,GAAG,CAAC,GAAG,UAAU,CAAC,YAAY,CAAC,MAAM,CACpD,CAAC;QAEF,IAAI,WAAW,GAAG,EAAE,EAAE,CAAC;YACtB,UAAU,CAAC,YAAY,CAAC,MAAM,CAAC,IAAI,CAClC,oBAAoB,WAAW,CAAC,OAAO,CAAC,CAAC,CAAC,uBAAuB,CACjE,CAAC;QACH,CAAC;QACD,IAAI,CAAC,QAAQ,CAAC,UAAU,IAAI,CAAC,CAAC,GAAG,EAAE,EAAE,CAAC;YACrC,UAAU,CAAC,YAAY,CAAC,MAAM,CAAC,IAAI,CAClC,sBAAsB,QAAQ,CAAC,UAAU,gBAAgB,CACzD,CAAC;QACH,CAAC;QACD,IAAI,CAAC,QAAQ,CAAC,QAAQ,IAAI,CAAC,CAAC,GAAG,EAAE,EAAE,CAAC;YACnC,UAAU,CAAC,YAAY,CAAC,MAAM,CAAC,IAAI,CAClC,mBAAmB,QAAQ,CAAC,QAAQ,gBAAgB,CACpD,CAAC;QACH,CAAC;QACD,IAAI,CAAC,QAAQ,CAAC,SAAS,IAAI,CAAC,CAAC,GAAG,EAAE,EAAE,CAAC;YACpC,UAAU,CAAC,YAAY,CAAC,MAAM,CAAC,IAAI,CAClC,qBAAqB,QAAQ,CAAC,SAAS,gBAAgB,CACvD,CAAC;QACH,CAAC;QAED,IAAI,WAAW,IAAI,EAAE,EAAE,CAAC;YACvB,YAAY,CAAC,IAAI,CAAC,yCAAyC,CAAC,CAAC;QAC9D,CAAC;aAAM,IAAI,WAAW,GAAG,EAAE,EAAE,CAAC;YAC7B,eAAe,CAAC,IAAI,CACnB,2DAA2D,CAC3D,CAAC;YACF,SAAS,CAAC,IAAI,CAAC,wCAAwC,CAAC,CAAC;QAC1D,CAAC;IACF,CAAC;SAAM,CAAC;QACP,iCAAiC;QACjC,UAAU,CAAC,YAAY,CAAC,KAAK,GAAG,IAAI,CAAC,KAAK,CACzC,GAAG,GAAG,UAAU,CAAC,YAAY,CAAC,MAAM,CACpC,CAAC;IACH,CAAC;IAED,mDAAmD;IACnD,UAAU,CAAC,UAAU,CAAC,KAAK,GAAG,UAAU,CAAC,UAAU,CAAC,MAAM,CAAC;IAC3D,YAAY,CAAC,IAAI,CAAC,4CAA4C,CAAC,CAAC;IAEhE,2DAA2D;IAC3D,UAAU,CAAC,OAAO,CAAC,KAAK,GAAG,UAAU,CAAC,OAAO,CAAC,MAAM,CAAC;IACrD,YAAY,CAAC,IAAI,CAAC,yCAAyC,CAAC,CAAC;IAE7D,wBAAwB;IACxB,IAAI,KAAK,CAAC,WAAW,EAAE,CAAC;QACvB,MAAM,QAAQ,GAAG,oBAAoB,CAAC,KAAK,CAAC,WAAW,CAAC,CAAC;QACzD,UAAU,CAAC,aAAa,CAAC,KAAK,GAAG,IAAI,CAAC,KAAK,CAC1C,CAAC,QAAQ,GAAG,GAAG,CAAC,GAAG,UAAU,CAAC,aAAa,CAAC,MAAM,CAClD,CAAC;QAEF,IAAI,QAAQ,GAAG,EAAE,EAAE,CAAC;YACnB,UAAU,CAAC,aAAa,CAAC,MAAM,CAAC,IAAI,CACnC,8CAA8C,CAC9C,CAAC;YACF,eAAe,CAAC,IAAI,CAAC,8CAA8C,CAAC,CAAC;QACtE,CAAC;QACD,IAAI,QAAQ,IAAI,EAAE,EAAE,CAAC;YACpB,YAAY,CAAC,IAAI,CAAC,0BAA0B,CAAC,CAAC;QAC/C,CAAC;IACF,CAAC;SAAM,CAAC;QACP,UAAU,CAAC,aAAa,CAAC,KAAK,GAAG,IAAI,CAAC,KAAK,CAC1C,GAAG,GAAG,UAAU,CAAC,aAAa,CAAC,MAAM,CACrC,CAAC;IACH,CAAC;IAED,mBAAmB;IACnB,IAAI,KAAK,CAAC,WAAW,EAAE,CAAC;QACvB,MAAM,aAAa,GAAG,qBAAqB,CAAC,KAAK,CAAC,WAAW,CAAC,CAAC;QAC/D,UAAU,CAAC,QAAQ,CAAC,KAAK,GAAG,IAAI,CAAC,KAAK,CACrC,CAAC,aAAa,CAAC,KAAK,GAAG,GAAG,CAAC,GAAG,UAAU,CAAC,QAAQ,CAAC,MAAM,CACxD,CAAC;QACF,UAAU,CAAC,QAAQ,CAAC,MAAM,GAAG,aAAa,CAAC,MAAM,CAAC;QAElD,IAAI,aAAa,CAAC,KAAK,GAAG,EAAE,EAAE,CAAC;YAC9B,eAAe,CAAC,IAAI,CAAC,8CAA8C,CAAC,CAAC;YACrE,SAAS,CAAC,IAAI,CAAC,8CAA8C,CAAC,CAAC;QAChE,CAAC;QACD,IAAI,aAAa,CAAC,KAAK,KAAK,GAAG,EAAE,CAAC;YACjC,YAAY,CAAC,IAAI,CAAC,sCAAsC,CAAC,CAAC;QAC3D,CAAC;IACF,CAAC;SAAM,CAAC;QACP,UAAU,CAAC,QAAQ,CAAC,KAAK,GAAG,UAAU,CAAC,QAAQ,CAAC,MAAM,CAAC;QACvD,YAAY,CAAC,IAAI,CAAC,wBAAwB,CAAC,CAAC;IAC7C,CAAC;IAED,0BAA0B;IAC1B,MAAM,YAAY,GAAG,IAAI,CAAC,KAAK,CAC9B,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC,MAAM,CAAC,CAAC,GAAG,EAAE,GAAG,EAAE,EAAE,CAAC,GAAG,GAAG,GAAG,CAAC,KAAK,EAAE,CAAC,CAAC,CAClE,CAAC;IAEF,IAAI,gBAAgB,GAAG,EAAE,CAAC;IAC1B,IAAI,YAAY,IAAI,EAAE,EAAE,CAAC;QACxB,gBAAgB,GAAG,oCAAoC,CAAC;IACzD,CAAC;SAAM,IAAI,YAAY,IAAI,EAAE,EAAE,CAAC;QAC/B,gBAAgB,GAAG,oCAAoC,CAAC;IACzD,CAAC;SAAM,IAAI,YAAY,IAAI,EAAE,EAAE,CAAC;QAC/B,gBAAgB,GAAG,iCAAiC,CAAC;IACtD,CAAC;SAAM,IAAI,YAAY,IAAI,EAAE,EAAE,CAAC;QAC/B,gBAAgB,GAAG,iCAAiC,CAAC;IACtD,CAAC;SAAM,IAAI,YAAY,IAAI,EAAE,EAAE,CAAC;QAC/B,gBAAgB,GAAG,+BAA+B,CAAC;IACpD,CAAC;SAAM,CAAC;QACP,gBAAgB,GAAG,6BAA6B,CAAC;IAClD,CAAC;IAED,8BAA8B;IAC9B,IAAI,YAAY,GAAG,GAAG,EAAE,CAAC;QACxB,IAAI,eAAe,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;YAClC,eAAe,CAAC,IAAI,CAAC,gDAAgD,CAAC,CAAC;QACxE,CAAC;QACD,eAAe,CAAC,IAAI,CAAC,oDAAoD,CAAC,CAAC;QAC3E,eAAe,CAAC,IAAI,CAAC,2CAA2C,CAAC,CAAC;IACnE,CAAC;IAED,IAAI,SAAS,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QAC5B,IAAI,YAAY,KAAK,GAAG,EAAE,CAAC;YAC1B,SAAS,CAAC,IAAI,CAAC,uCAAuC,CAAC,CAAC;YACxD,SAAS,CAAC,IAAI,CAAC,oCAAoC,CAAC,CAAC;QACtD,CAAC;aAAM,CAAC;YACP,SAAS,CAAC,IAAI,CAAC,0CAA0C,CAAC,CAAC;YAC3D,SAAS,CAAC,IAAI,CAAC,qCAAqC,CAAC,CAAC;QACvD,CAAC;IACF,CAAC;IAED,OAAO;QACN,YAAY;QACZ,gBAAgB;QAChB,UAAU;QACV,eAAe;QACf,SAAS;QACT,YAAY;KACZ,CAAC;AACH,CAAC;AAED,SAAS,kBAAkB,CAC1B,IAAY,EACZ,QAAgB;IAEhB,IAAI,KAAK,GAAG,GAAG,CAAC;IAChB,MAAM,MAAM,GAAa,EAAE,CAAC;IAE5B,yBAAyB;IACzB,IAAI,IAAI,CAAC,QAAQ,CAAC,MAAM,CAAC,IAAI,IAAI,CAAC,QAAQ,CAAC,OAAO,CAAC,EAAE,CAAC;QACrD,KAAK,IAAI,CAAC,CAAC;QACX,MAAM,CAAC,IAAI,CAAC,8BAA8B,CAAC,CAAC;IAC7C,CAAC;IAED,6BAA6B;IAC7B,IACC,CAAC,IAAI,CAAC,QAAQ,CAAC,aAAa,CAAC,IAAI,IAAI,CAAC,QAAQ,CAAC,QAAQ,CAAC,CAAC;QACzD,CAAC,QAAQ,KAAK,YAAY;YACzB,QAAQ,KAAK,YAAY;YACzB,QAAQ,KAAK,QAAQ,CAAC,EACtB,CAAC;QACF,KAAK,IAAI,EAAE,CAAC;QACZ,MAAM,CAAC,IAAI,CAAC,wBAAwB,CAAC,CAAC;IACvC,CAAC;IAED,kCAAkC;IAClC,IACC,8DAA8D,CAAC,IAAI,CAAC,IAAI,CAAC,EACxE,CAAC;QACF,KAAK,IAAI,EAAE,CAAC;QACZ,MAAM,CAAC,IAAI,CAAC,0CAA0C,CAAC,CAAC;IACzD,CAAC;IAED,2BAA2B;IAC3B,MAAM,cAAc,GAAG,CACtB,IAAI,CAAC,KAAK,CAAC,+CAA+C,CAAC,IAAI,EAAE,CACjE,CAAC,MAAM,CAAC;IACT,IAAI,cAAc,GAAG,CAAC,EAAE,CAAC;QACxB,KAAK,IAAI,CAAC,CAAC;QACX,MAAM,CAAC,IAAI,CAAC,GAAG,cAAc,gCAAgC,CAAC,CAAC;IAChE,CAAC;IAED,0CAA0C;IAC1C,MAAM,eAAe,GACpB,IAAI,CAAC,KAAK,CAAC,0CAA0C,CAAC,IAAI,EAAE,CAAC;IAC9D,KAAK,MAAM,IAAI,IAAI,eAAe,EAAE,CAAC;QACpC,MAAM,KAAK,GAAG,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,MAAM,CAAC;QACtC,IAAI,KAAK,GAAG,EAAE,EAAE,CAAC;YAChB,KAAK,IAAI,EAAE,CAAC;YACZ,MAAM,CAAC,IAAI,CAAC,uCAAuC,CAAC,CAAC;YACrD,MAAM;QACP,CAAC;IACF,CAAC;IAED,OAAO,EAAE,KAAK,EAAE,IAAI,CAAC,GAAG,CAAC,CAAC,EAAE,KAAK,CAAC,EAAE,MAAM,EAAE,CAAC;AAC9C,CAAC;AAED,SAAS,oBAAoB,CAAC,IAAY;IACzC,IAAI,KAAK,GAAG,EAAE,CAAC,CAAC,aAAa;IAE7B,qCAAqC;IACrC,MAAM,WAAW,GAAG,CACnB,IAAI,CAAC,KAAK,CAAC,mDAAmD,CAAC,IAAI,EAAE,CACrE,CAAC,MAAM,CAAC;IACT,IAAI,WAAW,GAAG,CAAC,EAAE,CAAC;QACrB,KAAK,IAAI,EAAE,CAAC;IACb,CAAC;IAED,4BAA4B;IAC5B,MAAM,cAAc,GAAG,CAAC,IAAI,CAAC,KAAK,CAAC,aAAa,CAAC,IAAI,EAAE,CAAC,CAAC,MAAM,CAAC;IAChE,IAAI,cAAc,GAAG,CAAC,EAAE,CAAC;QACxB,KAAK,IAAI,EAAE,CAAC;IACb,CAAC;IAED,6CAA6C;IAC7C,IAAI,gCAAgC,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC;QACjD,KAAK,IAAI,EAAE,CAAC;IACb,CAAC;IAED,OAAO,IAAI,CAAC,GAAG,CAAC,GAAG,EAAE,KAAK,CAAC,CAAC;AAC7B,CAAC;AAED,SAAS,qBAAqB,CAAC,IAAY;IAI1C,IAAI,KAAK,GAAG,GAAG,CAAC;IAChB,MAAM,MAAM,GAAa,EAAE,CAAC;IAE5B,6BAA6B;IAC7B,IAAI,qBAAqB,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC;QACtC,KAAK,IAAI,EAAE,CAAC;QACZ,MAAM,CAAC,IAAI,CAAC,kDAAkD,CAAC,CAAC;IACjE,CAAC;IAED,gCAAgC;IAChC,IAAI,wCAAwC,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC;QACzD,KAAK,IAAI,EAAE,CAAC;QACZ,MAAM,CAAC,IAAI,CAAC,uCAAuC,CAAC,CAAC;IACtD,CAAC;IAED,sBAAsB;IACtB,IAAI,wCAAwC,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC;QACzD,KAAK,IAAI,EAAE,CAAC;QACZ,MAAM,CAAC,IAAI,CAAC,4CAA4C,CAAC,CAAC;IAC3D,CAAC;IAED,8BAA8B;IAC9B,IACC,8EAA8E,CAAC,IAAI,CAClF,IAAI,CACJ,EACA,CAAC;QACF,KAAK,IAAI,EAAE,CAAC;QACZ,MAAM,CAAC,IAAI,CAAC,wCAAwC,CAAC,CAAC;IACvD,CAAC;IAED,OAAO,EAAE,KAAK,EAAE,IAAI,CAAC,GAAG,CAAC,CAAC,EAAE,KAAK,CAAC,EAAE,MAAM,EAAE,CAAC;AAC9C,CAAC;AAED,SAAS,gBAAgB,CAAC,KAAa;IACtC,MAAM,SAAS,GAAG,EAAE,CAAC;IACrB,MAAM,YAAY,GAAG,IAAI,CAAC,KAAK,CAAC,CAAC,KAAK,GAAG,GAAG,CAAC,GAAG,SAAS,CAAC,CAAC;IAC3D,MAAM,WAAW,GAAG,SAAS,GAAG,YAAY,CAAC;IAE7C,MAAM,MAAM,GAAG,GAAG,CAAC,MAAM,CAAC,YAAY,CAAC,CAAC;IACxC,MAAM,KAAK,GAAG,GAAG,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC;IAEtC,IAAI,KAAK,GAAG,IAAI,CAAC;IACjB,IAAI,KAAK,IAAI,EAAE;QAAE,KAAK,GAAG,IAAI,CAAC;SACzB,IAAI,KAAK,IAAI,EAAE;QAAE,KAAK,GAAG,IAAI,CAAC;SAC9B,IAAI,KAAK,IAAI,EAAE;QAAE,KAAK,GAAG,IAAI,CAAC;IAEnC,OAAO,GAAG,KAAK,KAAK,MAAM,GAAG,KAAK,KAAK,KAAK,GAAG,CAAC;AACjD,CAAC;AAED,SAAS,eAAe,CAAC,KAAa,EAAE,MAAc;IACrD,MAAM,UAAU,GAAG,CAAC,KAAK,GAAG,MAAM,CAAC,GAAG,GAAG,CAAC;IAC1C,MAAM,SAAS,GAAG,EAAE,CAAC;IACrB,MAAM,YAAY,GAAG,IAAI,CAAC,KAAK,CAAC,CAAC,UAAU,GAAG,GAAG,CAAC,GAAG,SAAS,CAAC,CAAC;IAChE,MAAM,WAAW,GAAG,SAAS,GAAG,YAAY,CAAC;IAE7C,MAAM,MAAM,GAAG,GAAG,CAAC,MAAM,CAAC,YAAY,CAAC,CAAC;IACxC,MAAM,KAAK,GAAG,GAAG,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC;IAEtC,OAAO,IAAI,MAAM,GAAG,KAAK,KAAK,KAAK,IAAI,MAAM,EAAE,CAAC;AACjD,CAAC;AAED,SAAS,cAAc,CAAC,UAAkB;IACzC,IAAI,UAAU,IAAI,EAAE;QAAE,OAAO,IAAI,CAAC;IAClC,IAAI,UAAU,IAAI,EAAE;QAAE,OAAO,IAAI,CAAC;IAClC,IAAI,UAAU,IAAI,EAAE;QAAE,OAAO,IAAI,CAAC;IAClC,OAAO,IAAI,CAAC;AACb,CAAC"}

package/dist/tools/code-hygiene-analyzer.js

@@ -74,11 +74,81 @@ function analyzeCodeHygiene(input) {
     const nextSteps = [];
     const code = input.codeContent;
     const language = input.language.toLowerCase();
+    // Check for empty or minimal code
+    const trimmedCode = code.trim();
+    if (trimmedCode.length === 0) {
+        issues.push({
+            type: "Empty Code",
+            description: "No code provided for analysis",
+            severity: "critical",
+        });
+        recommendations.push("Provide code content for meaningful analysis");
+    }
+    else if (trimmedCode.length < 20) {
+        issues.push({
+            type: "Insufficient Code",
+            description: "Code too short for comprehensive analysis (less than 20 characters)",
+            severity: "minor",
+        });
+        recommendations.push("Provide more substantial code for better analysis");
+    }
+    // Check for code complexity metrics
+    const lines = code.split("\n");
+    const nonEmptyLines = lines.filter((line) => line.trim().length > 0).length;
+    const codeOnlyLines = lines.filter((line) => {
+        const trimmed = line.trim();
+        // Count lines that are not comments
+        return (trimmed.length > 0 &&
+            !trimmed.startsWith("//") &&
+            !trimmed.startsWith("#") &&
+            !trimmed.startsWith("*") &&
+            !trimmed.startsWith("/*"));
+    }).length;
+    // Check for lack of documentation
+    const hasDocComments = /\/\*\*[\s\S]*?\*\/|\/\/\/|#\s*@|"""[\s\S]*?"""|'''[\s\S]*?'''/g.test(code);
+    if (codeOnlyLines > 15 && !hasDocComments) {
+        issues.push({
+            type: "Documentation",
+            description: "No documentation comments found in substantial codebase",
+            severity: "minor",
+        });
+        recommendations.push("Add documentation comments for functions and classes");
+    }
+    // Check for magic numbers (simple heuristic - may have false positives for HTTP codes, ports, etc.)
+    const magicNumberPattern = /(?:^|[^\w.])([2-9]\d{2,}|[1-9]\d{4,})(?:[^\w.]|$)/g;
+    const magicNumbers = code.match(magicNumberPattern);
+    if (magicNumbers && magicNumbers.length > 2) {
+        issues.push({
+            type: "Code Quality",
+            description: `Found ${magicNumbers.length} potential magic numbers - consider using named constants`,
+            severity: "minor",
+        });
+        recommendations.push("Replace magic numbers with named constants for clarity");
+    }
+    // Check for deep nesting (heuristic based on indentation - may vary with tab/space preferences)
+    let maxIndentation = 0;
+    for (const line of lines) {
+        const leadingSpaces = line.match(/^(\s*)/)?.[1].length || 0;
+        // Estimate indent level (assuming 2-4 space indentation is common)
+        const indentLevel = Math.floor(leadingSpaces / 2);
+        if (indentLevel > maxIndentation) {
+            maxIndentation = indentLevel;
+        }
+    }
+    if (maxIndentation > 4) {
+        issues.push({
+            type: "Code Complexity",
+            description: `Deep nesting detected (${maxIndentation} levels) - consider refactoring`,
+            severity: "major",
+        });
+        recommendations.push("Reduce nesting depth by extracting functions or using early returns");
+    }
     // Common code hygiene checks
     if (code.includes("TODO") || code.includes("FIXME")) {
         issues.push({
             type: "Technical Debt",
             description: "Found TODO or FIXME comments indicating incomplete work",
+            severity: "minor",
         });
         recommendations.push("Address pending TODO and FIXME items before production");
     }
@@ -87,6 +157,7 @@ function analyzeCodeHygiene(input) {
         issues.push({
             type: "Debug Code",
             description: "Found console.log statements that should be removed in production",
+            severity: "major",
         });
         recommendations.push("Remove debug console.log statements");
     }
@@ -94,11 +165,11 @@ function analyzeCodeHygiene(input) {
         issues.push({
             type: "Debug Code",
             description: "Found print statements that should use proper logging",
+            severity: "major",
         });
         recommendations.push("Replace print statements with proper logging");
     }
     // Check for long functions (simple heuristic)
-    const lines = code.split("\n");
     let functionLines = 0;
     let inFunction = false;
     for (const line of lines) {
@@ -116,6 +187,7 @@ function analyzeCodeHygiene(input) {
                     issues.push({
                         type: "Code Complexity",
                         description: `Found function with ${functionLines} lines - consider breaking into smaller functions`,
+                        severity: "major",
                     });
                 }
                 inFunction = false;
@@ -128,9 +200,35 @@ function analyzeCodeHygiene(input) {
             issues.push({
                 type: "Outdated Pattern",
                 description: "Using var instead of let/const",
+                severity: "minor",
             });
             recommendations.push("Replace var declarations with let or const");
         }
+        // Check for potential callback hell
+        const nestedCallbacks = (code.match(/\)\s*{\s*[^}]*\([^)]*\)\s*{/g) || [])
+            .length;
+        if (nestedCallbacks > 2) {
+            issues.push({
+                type: "Code Complexity",
+                description: `Potential callback hell detected (${nestedCallbacks} nested callbacks)`,
+                severity: "major",
+            });
+            recommendations.push("Consider using async/await or Promise chains instead of nested callbacks");
+        }
+        // Check for missing type annotations in TypeScript
+        if (language === "typescript" && codeOnlyLines > 10) {
+            const functionDeclarations = (code.match(/function\s+\w+\s*\([^)]*\)\s*{/g) || []).length;
+            const typedFunctions = (code.match(/function\s+\w+\s*\([^)]*\)\s*:\s*\w+/g) || []).length;
+            if (functionDeclarations > 0 &&
+                typedFunctions / functionDeclarations < 0.5) {
+                issues.push({
+                    type: "Type Safety",
+                    description: "Many functions lack return type annotations",
+                    severity: "minor",
+                });
+                recommendations.push("Add explicit return type annotations to functions");
+            }
+        }
     }
     // Check for missing error handling
     if (code.includes("await ") &&
@@ -139,21 +237,87 @@ function analyzeCodeHygiene(input) {
         issues.push({
             type: "Error Handling",
             description: "Async code without proper error handling",
+            severity: "critical",
         });
         recommendations.push("Add try-catch blocks around async operations");
     }
-    // Calculate score
+    // Check for hardcoded credentials or sensitive data
+    if (/password\s*=\s*["']|api[_-]?key\s*=\s*["']|secret\s*=\s*["']/i.test(code)) {
+        issues.push({
+            type: "Security Risk",
+            description: "Potential hardcoded credentials or API keys detected",
+            severity: "critical",
+        });
+        recommendations.push("Move sensitive data to environment variables or secure configuration");
+    }
+    // Check for commented out code
+    const commentedLines = lines.filter((line) => {
+        const trimmed = line.trim();
+        return ((trimmed.startsWith("//") && /[a-zA-Z0-9()]/.test(trimmed.slice(2))) ||
+            (trimmed.startsWith("#") &&
+                language === "python" &&
+                /[a-zA-Z0-9()]/.test(trimmed.slice(1))));
+    }).length;
+    if (commentedLines > 3) {
+        issues.push({
+            type: "Dead Code",
+            description: `Found ${commentedLines} lines of commented code - consider removing`,
+            severity: "minor",
+        });
+        recommendations.push("Remove commented out code or move to version control history");
+    }
+    // Calculate score with severity-based penalties
     let score = 100;
-    score -= issues.length * 10;
+    for (const issue of issues) {
+        if (issue.severity === "critical") {
+            score -= 20;
+        }
+        else if (issue.severity === "major") {
+            score -= 12;
+        }
+        else {
+            score -= 5;
+        }
+    }
     score = Math.max(0, score);
+    // Apply additional scoring adjustments for overall code quality
+    if (trimmedCode.length > 0 && codeOnlyLines > 0) {
+        // Reward code with good practices
+        let qualityBonus = 0;
+        // Check for good practices
+        if (hasDocComments && codeOnlyLines > 10) {
+            qualityBonus += 0; // Neutral - documentation is expected, not bonus
+        }
+        // Penalize very short code that might not be representative
+        if (codeOnlyLines < 5 && trimmedCode.length < 100) {
+            score = Math.max(70, score - 10); // Reduce score for minimal code samples
+        }
+        // Penalize code with low comment ratio (if substantial)
+        if (codeOnlyLines > 15) {
+            const commentLines = nonEmptyLines - codeOnlyLines;
+            const commentRatio = commentLines / codeOnlyLines;
+            if (commentRatio < 0.1) {
+                score -= 5;
+                if (!issues.some((i) => i.type === "Documentation")) {
+                    issues.push({
+                        type: "Documentation",
+                        description: "Low comment-to-code ratio (< 10%)",
+                        severity: "minor",
+                    });
+                    recommendations.push("Consider adding more explanatory comments");
+                }
+            }
+        }
+        score = Math.max(0, Math.min(100, score + qualityBonus));
+    }
     let scoreDescription = "";
-    if (score >= 90)
+    if (score >= 85)
         scoreDescription = "Excellent";
-    else if (score >= 80)
-        scoreDescription = "Good";
     else if (score >= 70)
+        scoreDescription = "Good";
+    else if (score >= 50)
         scoreDescription = "Fair";
-    else if (score >= 60)
+    else if (score >= 30)
         scoreDescription = "Needs Improvement";
     else
         scoreDescription = "Poor";
@@ -162,12 +326,20 @@ function analyzeCodeHygiene(input) {
         nextSteps.push("Code hygiene looks good! Consider adding automated linting if not already present.");
     }
     else {
-        nextSteps.push("Address the identified issues in order of priority");
+        const criticalIssues = issues.filter((i) => i.severity === "critical").length;
+        const majorIssues = issues.filter((i) => i.severity === "major").length;
+        if (criticalIssues > 0) {
+            nextSteps.push(`Address ${criticalIssues} critical issue(s) immediately`);
+        }
+        if (majorIssues > 0) {
+            nextSteps.push(`Fix ${majorIssues} major issue(s) before merging`);
+        }
+        nextSteps.push("Address the identified issues in order of priority (critical > major > minor)");
         nextSteps.push("Set up automated code quality checks (ESLint, Prettier, Biome, etc.)");
         nextSteps.push("Consider implementing pre-commit hooks");
     }
     return {
-        issues,
+        issues: issues.map(({ severity, ...rest }) => rest),
         recommendations,
         score,
         scoreDescription,