@hailer/mcp 0.1.13 → 0.1.15

package/.claude/agents/agent-lars-code-inspector.md

@@ -0,0 +1,42 @@
+---
+name: agent-lars-code-inspector
+description: LSP-powered code intelligence - finds bugs, dead code, unused imports, and navigates codebases.\n\n<example>\nuser: "Find bugs in src/App.tsx"\nassistant: {"status":"success","result":{"dead_code":[{"line":50,"symbol":"setColorMode"}],"unused_imports":[]},"summary":"Found 1 dead code issue"}\n</example>
+model: haiku
+tools: LSP, Read, Glob
+---
+
+<identity>
+I am Lars. LSP only. Minimal output. JSON only.
+</identity>
+
+<handles>
+- Find unused variables/imports
+- Find dead code
+- Type errors
+- Duplicate declarations
+</handles>
+
+<rules>
+1. **LSP ONLY** - Use LSP operations, never tsc/eslint.
+2. **MINIMAL OUTPUT** - No intermediate results. Final JSON only.
+3. **NO PROSE** - Output JSON, then STOP immediately.
+</rules>
+
+<workflow>
+1. Use `documentSymbol` to list symbols in file
+2. Use `findReferences` on key symbols to check if used
+3. Check LSP diagnostics in response for errors
+4. Return compact JSON
+</workflow>
+
+<protocol>
+Output JSON only. No explanations. No tool output dumps.
+{
+  "status": "success|error",
+  "result": {
+    "dead_code": [{"line":0,"symbol":""}],
+    "type_errors": [{"line":0,"msg":""}]
+  },
+  "summary": "max 30 chars"
+}
+</protocol>

package/CLAUDE.md

@@ -47,6 +47,7 @@ Agents return JSON. You interpret it for the user.
 | Activity CRUD | `agent-dmitri-activity-crud` | haiku |
 | Discussions/chat | `agent-yevgeni-discussions` | haiku |
 | Config audit | `agent-bjorn-config-audit` | haiku |
+| LSP/code bugs | `agent-lars-code-inspector` | haiku |
 | Workflows, fields, phases | `agent-helga-workflow-config` | sonnet |
 | SQL insights | `agent-viktor-sql-insights` | sonnet |
 | Function fields | `agent-alejandro-function-fields` | sonnet |

package/dist/modules/bug-reports/giuseppe-bot.d.ts

@@ -18,6 +18,7 @@ export declare class GiuseppeBot {
     private ai;
     private git;
     private files;
+    private lsp;
     private appsRegistry;
     constructor(userContext: UserContext, config: BugReportsConfig, monitor: BugMonitor);
     /**

package/dist/modules/bug-reports/giuseppe-bot.js

@@ -20,6 +20,7 @@ const app_scaffold_1 = require("../../mcp/tools/app-scaffold");
 const giuseppe_ai_1 = require("./giuseppe-ai");
 const giuseppe_git_1 = require("./giuseppe-git");
 const giuseppe_files_1 = require("./giuseppe-files");
+const giuseppe_lsp_1 = require("./giuseppe-lsp");
 const logger = (0, logger_1.createLogger)({ component: 'giuseppe-bot' });
 // Magic words for approval flow
 const MAGIC_WORD_APPROVED = 'approved';
@@ -44,6 +45,7 @@ class GiuseppeBot {
     ai;
     git;
     files;
+    lsp;
     appsRegistry = new Map();
     constructor(userContext, config, monitor) {
         this.userContext = userContext;
@@ -54,6 +56,7 @@ class GiuseppeBot {
         this.ai = new giuseppe_ai_1.GiuseppeAI(apiKey);
         this.git = new giuseppe_git_1.GiuseppeGit();
         this.files = new giuseppe_files_1.GiuseppeFiles(process.env.DEV_APPS_PATH);
+        this.lsp = new giuseppe_lsp_1.GiuseppeLsp();
         // Load apps registry from config
         if (config.appsRegistry) {
             for (const [appId, entry] of Object.entries(config.appsRegistry)) {
@@ -236,6 +239,12 @@ class GiuseppeBot {
             }
             result.log?.push(`Found app project: ${app.projectPath}`);
             await this.reportProgress(bug, `📁 Found app project: ${app.name}`);
+            // 1.5. Run LSP analysis to understand current codebase state
+            const preAnalysis = await this.lsp.analyzeProject(app.projectPath);
+            if (preAnalysis.issues.length > 0) {
+                result.log?.push(`Pre-fix LSP: ${preAnalysis.summary}`);
+                await this.reportProgress(bug, `🔎 Codebase analysis: ${preAnalysis.summary}`);
+            }
             // 2. Analyze and generate fix
             const allFiles = await this.files.scanSourceFiles(app.projectPath);
             const fixPlan = await this.ai.analyzeAndPlanFix(bug, app, allFiles, (paths) => this.files.readSelectedFiles(app.projectPath, paths));
@@ -283,6 +292,12 @@ class GiuseppeBot {
                 await this.reportProgress(bug, `✏️ Applied fixes to ${applyResult.files.length} file(s)`);
                 const buildResult = await this.buildAndTest(app);
                 if (buildResult.success) {
+                    // Run LSP analysis to check for remaining issues
+                    const lspResult = await this.lsp.validateFix(app.projectPath);
+                    if (!lspResult.valid) {
+                        result.log?.push(`LSP found ${lspResult.errors.length} issue(s)`);
+                        await this.reportProgress(bug, `⚠️ Build passed but LSP found issues:\n${lspResult.errors.slice(0, 3).join('\n')}`);
+                    }
                     fixSuccess = true;
                     result.log?.push(`Build successful (attempt ${attempt})`);
                     await this.reportProgress(bug, '✅ Build successful');

package/dist/modules/bug-reports/giuseppe-lsp.d.ts

@@ -0,0 +1,42 @@
+/**
+ * Giuseppe LSP Module - Code analysis via TypeScript compiler
+ */
+export interface CodeIssue {
+    file: string;
+    line: number;
+    column: number;
+    message: string;
+    code: string;
+    severity: 'error' | 'warning';
+}
+export interface AnalysisResult {
+    success: boolean;
+    issues: CodeIssue[];
+    summary: string;
+}
+export declare class GiuseppeLsp {
+    /**
+     * Analyze a project for TypeScript errors
+     */
+    analyzeProject(projectPath: string): Promise<AnalysisResult>;
+    /**
+     * Analyze a single file
+     */
+    analyzeFile(filePath: string): Promise<AnalysisResult>;
+    /**
+     * Check if code compiles after a fix
+     */
+    validateFix(projectPath: string): Promise<{
+        valid: boolean;
+        errors: string[];
+    }>;
+    /**
+     * Find unused exports/variables in a file
+     */
+    findDeadCode(projectPath: string): Promise<CodeIssue[]>;
+    /**
+     * Parse tsc output into structured issues
+     */
+    private parseTscOutput;
+}
+//# sourceMappingURL=giuseppe-lsp.d.ts.map

package/dist/modules/bug-reports/giuseppe-lsp.js

@@ -0,0 +1,143 @@
+"use strict";
+/**
+ * Giuseppe LSP Module - Code analysis via TypeScript compiler
+ */
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.GiuseppeLsp = void 0;
+const child_process_1 = require("child_process");
+const path = __importStar(require("path"));
+const logger_1 = require("../../lib/logger");
+const logger = (0, logger_1.createLogger)({ component: 'giuseppe-lsp' });
+class GiuseppeLsp {
+    /**
+     * Analyze a project for TypeScript errors
+     */
+    async analyzeProject(projectPath) {
+        try {
+            logger.info(`Analyzing project: ${projectPath}`);
+            const result = (0, child_process_1.execSync)('npx tsc --noEmit --pretty false 2>&1 || true', {
+                cwd: projectPath,
+                encoding: 'utf-8',
+                maxBuffer: 10 * 1024 * 1024
+            });
+            const issues = this.parseTscOutput(result, projectPath);
+            return {
+                success: issues.filter(i => i.severity === 'error').length === 0,
+                issues,
+                summary: issues.length === 0
+                    ? 'No issues found'
+                    : `Found ${issues.length} issue(s)`
+            };
+        }
+        catch (error) {
+            logger.error('Analysis failed', { error });
+            return {
+                success: false,
+                issues: [],
+                summary: `Analysis failed: ${error}`
+            };
+        }
+    }
+    /**
+     * Analyze a single file
+     */
+    async analyzeFile(filePath) {
+        const projectPath = path.dirname(filePath);
+        const result = await this.analyzeProject(projectPath);
+        const fileName = path.basename(filePath);
+        result.issues = result.issues.filter(i => i.file.endsWith(fileName));
+        result.summary = result.issues.length === 0
+            ? `No issues in ${fileName}`
+            : `Found ${result.issues.length} issue(s) in ${fileName}`;
+        return result;
+    }
+    /**
+     * Check if code compiles after a fix
+     */
+    async validateFix(projectPath) {
+        const result = await this.analyzeProject(projectPath);
+        const errors = result.issues
+            .filter(i => i.severity === 'error')
+            .map(i => `${i.file}:${i.line} - ${i.message}`);
+        return {
+            valid: errors.length === 0,
+            errors
+        };
+    }
+    /**
+     * Find unused exports/variables in a file
+     */
+    async findDeadCode(projectPath) {
+        try {
+            const result = (0, child_process_1.execSync)('npx tsc --noEmit --noUnusedLocals --noUnusedParameters --pretty false 2>&1 || true', {
+                cwd: projectPath,
+                encoding: 'utf-8',
+                maxBuffer: 10 * 1024 * 1024
+            });
+            return this.parseTscOutput(result, projectPath)
+                .filter(i => i.message.includes('declared but') || i.message.includes('never used'));
+        }
+        catch (error) {
+            logger.error('Dead code analysis failed', { error });
+            return [];
+        }
+    }
+    /**
+     * Parse tsc output into structured issues
+     */
+    parseTscOutput(output, basePath) {
+        const issues = [];
+        const lines = output.split('\n');
+        // Pattern: src/file.ts(10,5): error TS2304: Cannot find name 'foo'.
+        const pattern = /^(.+?)\((\d+),(\d+)\):\s*(error|warning)\s+(TS\d+):\s*(.+)$/;
+        for (const line of lines) {
+            const match = line.match(pattern);
+            if (match) {
+                issues.push({
+                    file: path.resolve(basePath, match[1]),
+                    line: parseInt(match[2], 10),
+                    column: parseInt(match[3], 10),
+                    severity: match[4],
+                    code: match[5],
+                    message: match[6]
+                });
+            }
+        }
+        return issues;
+    }
+}
+exports.GiuseppeLsp = GiuseppeLsp;
+//# sourceMappingURL=giuseppe-lsp.js.map

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hailer/mcp",
-  "version": "0.1.13",
+  "version": "0.1.15",
   "config": {
     "docker": {
       "registry": "registry.gitlab.com/hailer-repos/hailer-mcp"
@@ -42,6 +42,7 @@
     "form-data": "^4.0.4",
     "openai": "^5.5.1",
     "sharp": "^0.34.5",
+    "typescript-language-server": "^5.1.3",
     "zod": "^3.24.1"
   },
   "devDependencies": {
@@ -54,8 +55,7 @@
     "eslint": "^9.29.0",
     "globals": "^16.2.0",
     "tsx": "^4.19.2",
-    "typescript": "^5",
-    "typescript-language-server": "^5.1.3"
+    "typescript": "^5"
   },
   "bin": {
     "hailer-mcp": "dist/cli.js"