@azerate/claudette-mcp 1.7.1 → 1.7.3

package/dist/index.js

@@ -2,6 +2,7 @@
 import { Server } from "@modelcontextprotocol/sdk/server/index.js";
 import { StdioServerTransport } from "@modelcontextprotocol/sdk/server/stdio.js";
 import { CallToolRequestSchema, ListToolsRequestSchema, } from "@modelcontextprotocol/sdk/types.js";
+import { execSync } from "child_process";
 // Import from extracted modules
 import { getScripts, getScriptOutput, startScript, stopScript } from './scripts.js';
 import { getWorkspaceConfig, saveWorkspaceConfig, clearMemory, deleteMemoryNotes, replaceMemory } from './workspace.js';
@@ -523,7 +524,7 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
             },
             {
                 name: "approve_write_tests",
-                description: "Mark the write tests step as complete after tests have been written for uncovered files.",
+                description: "Mark the write tests step as complete AFTER you have written tests. CRITICAL: You MUST write tests before calling this - NEVER skip this step or ask the user if they want to skip. When workflow reaches ready_write_tests, immediately write tests for uncovered files, then call this tool.",
                 inputSchema: {
                     type: "object",
                     properties: {
@@ -541,7 +542,7 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
             },
             {
                 name: "approve_code_review",
-                description: "Mark the code review step as complete after the PR has been reviewed and approved.",
+                description: "Complete the code review step AND merge the PR. This will merge the PR on GitHub and mark the workflow as complete.",
                 inputSchema: {
                     type: "object",
                     properties: {
@@ -553,6 +554,11 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
                             type: "string",
                             description: "Name of the reviewer (optional)",
                         },
+                        merge_method: {
+                            type: "string",
+                            enum: ["squash", "merge", "rebase"],
+                            description: "How to merge the PR: squash (default), merge, or rebase",
+                        },
                     },
                     required: ["workspace_path"],
                 },
@@ -1384,6 +1390,29 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
                 if (status.error) {
                     output += `\n❌ Error: ${status.error}\n`;
                 }
+                // CRITICAL: Show explicit action required when at ready_write_tests step
+                if (status.currentStep === 'ready_write_tests') {
+                    output += `\n${"═".repeat(50)}\n`;
+                    output += `🚨 ACTION REQUIRED: WRITE TESTS\n`;
+                    output += `${"═".repeat(50)}\n\n`;
+                    output += `Coverage is at ${status.steps.coverage?.percent || 0}%.\n\n`;
+                    output += `DO NOT SKIP THIS STEP. You MUST:\n`;
+                    output += `1. Identify files that need test coverage\n`;
+                    output += `2. Write tests for uncovered code paths\n`;
+                    output += `3. Run tests to verify they pass\n`;
+                    output += `4. Use approve_write_tests when done\n\n`;
+                    if (status.steps.coverage?.uncoveredFiles?.length > 0) {
+                        output += `Files needing tests:\n`;
+                        for (const file of status.steps.coverage.uncoveredFiles.slice(0, 5)) {
+                            output += `  - ${file}\n`;
+                        }
+                        if (status.steps.coverage.uncoveredFiles.length > 5) {
+                            output += `  ... and ${status.steps.coverage.uncoveredFiles.length - 5} more\n`;
+                        }
+                    }
+                    output += `\nNEVER ask the user if they want to skip writing tests.\n`;
+                    output += `ALWAYS write tests before proceeding to commit.\n`;
+                }
                 // Show recommendation if available (auto-populated when changes exist)
                 if (status.recommendation) {
                     const rec = status.recommendation;
@@ -1529,22 +1558,56 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
         case "approve_code_review": {
             const workspacePath = args?.workspace_path;
             const reviewer = args?.reviewer;
+            const mergeMethod = args?.merge_method || 'squash';
             if (!workspacePath) {
                 return { content: [{ type: "text", text: "Error: workspace_path is required" }] };
             }
             try {
+                // First, check if there's an open PR for this branch
+                let prNumber;
+                let prTitle;
+                try {
+                    const prInfo = execSync('gh pr view --json number,title', {
+                        cwd: workspacePath,
+                        encoding: 'utf-8',
+                    });
+                    const parsed = JSON.parse(prInfo);
+                    prNumber = parsed.number?.toString();
+                    prTitle = parsed.title;
+                }
+                catch {
+                    // No PR found
+                }
+                if (!prNumber) {
+                    return { content: [{ type: "text", text: "❌ No open PR found for this branch. Cannot merge." }] };
+                }
+                // Merge the PR using gh CLI
+                const mergeFlag = mergeMethod === 'squash' ? '--squash' : mergeMethod === 'rebase' ? '--rebase' : '--merge';
+                try {
+                    execSync(`gh pr merge ${prNumber} ${mergeFlag} --delete-branch`, {
+                        cwd: workspacePath,
+                        encoding: 'utf-8',
+                        stdio: 'pipe',
+                    });
+                }
+                catch (mergeErr) {
+                    return { content: [{ type: "text", text: `❌ Failed to merge PR #${prNumber}: ${mergeErr.message}` }] };
+                }
+                // Mark workflow as complete
                 const response = await fetch(`${CLAUDETTE_API}/api/workflow/code-review`, {
                     method: "POST",
                     headers: { "Content-Type": "application/json" },
                     body: JSON.stringify({ path: workspacePath, reviewer }),
                 });
                 const result = await response.json();
-                if (result.status === 'passed') {
-                    return { content: [{ type: "text", text: `✅ Code review approved!\n\n${result.reviewer ? `Reviewer: ${result.reviewer}\n` : ''}Workflow complete.` }] };
-                }
-                else {
-                    return { content: [{ type: "text", text: `❌ Failed: ${result.error}` }] };
-                }
+                let output = `✅ PR #${prNumber} merged successfully!\n\n`;
+                output += `Title: ${prTitle}\n`;
+                output += `Method: ${mergeMethod}\n`;
+                output += `Branch deleted: Yes\n`;
+                if (reviewer)
+                    output += `Reviewer: ${reviewer}\n`;
+                output += `\nWorkflow complete.`;
+                return { content: [{ type: "text", text: output }] };
             }
             catch (err) {
                 return { content: [{ type: "text", text: `Error: ${err.message}` }] };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@azerate/claudette-mcp",
-  "version": "1.7.1",
+  "version": "1.7.3",
   "description": "MCP server for Claudette IDE - TypeScript errors, git changes, checkpoints, memory, and script management",
   "type": "module",
   "main": "dist/index.js",