devscribe-reason 1.0.9 → 1.0.11

package/.claude/settings.local.json

@@ -22,7 +22,8 @@
       "mcp__devscribe-reason__log_intent",
       "Bash(npm publish:*)",
       "mcp__devscribe-reason__log_decision",
-      "mcp__devscribe-reason__finalize_reasoning_doc"
+      "mcp__devscribe-reason__finalize_reasoning_doc",
+      "Bash(npm version:*)"
     ]
   }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "devscribe-reason",
-  "version": "1.0.9",
+  "version": "1.0.11",
   "description": "MCP server that captures engineering decision reasoning and commits structured markdown to GitHub",
   "main": "src/index.js",
   "type": "module",

package/scripts/cli.js

@@ -1,12 +1,8 @@
 #!/usr/bin/env node
 
-import { execSync } from "node:child_process";
-import { existsSync, mkdirSync, writeFileSync } from "node:fs";
-import { join } from "node:path";
-import { homedir } from "node:os";
-import { globSync } from "glob";
 import { fileURLToPath } from "node:url";
 import { dirname } from "node:path";
+import { spawn } from "node:child_process";
 
 const args = process.argv.slice(2);
 const command = args[0];
@@ -14,26 +10,42 @@ const command = args[0];
 // If no command or "server" command, run the MCP server
 if (!command || command === "server") {
   const __filename = fileURLToPath(import.meta.url);
-  const __dirname = dirname(__filename);
-  const serverPath = join(__dirname, "../src/index.js");
 
-  // Run the server
-  execSync(`node ${serverPath}`, { stdio: "inherit" });
-  process.exit(0);
+  // Spawn the server directly without shelling out
+  const serverProcess = spawn("node", [dirname(__filename) + "/../src/index.js"], {
+    stdio: "inherit",
+    env: process.env,
+  });
+
+  serverProcess.on("error", (err) => {
+    console.error("Failed to start server:", err);
+    process.exit(1);
+  });
+
+  // Keep the process alive
+  serverProcess.on("exit", (code) => {
+    process.exit(code || 0);
+  });
 }
 
 // If "setup" command, run setup
 if (command === "setup") {
   const __filename = fileURLToPath(import.meta.url);
-  const __dirname = dirname(__filename);
-  const setupPath = join(__dirname, "setup.js");
 
-  // Run setup
-  execSync(`node ${setupPath}`, {
+  // Spawn setup directly without shelling out
+  const setupProcess = spawn("node", [dirname(__filename) + "/setup.js"], {
     stdio: "inherit",
     env: process.env,
   });
-  process.exit(0);
+
+  setupProcess.on("error", (err) => {
+    console.error("Failed to run setup:", err);
+    process.exit(1);
+  });
+
+  setupProcess.on("exit", (code) => {
+    process.exit(code || 0);
+  });
 }
 
 // Unknown command

package/src/CLAUDE.md

@@ -50,7 +50,7 @@ When the devscribe-reason MCP server is connected:
 - Call log_intent at the start of every session before doing any work
 - Call log_decision whenever making a significant technical or architectural choice
 - Call log_alternative whenever an approach is considered and rejected
-- **When user asks to push/commit:** Call finalize_reasoning_doc first, then proceed with push
+- **When user asks to "push", "push changes", or "push the changes":** Call push_with_decision_doc (which automatically handles decision doc creation and git push)
 - Never ask permission to log — just do it silently in the background
 ```
 

package/src/index.js

@@ -363,6 +363,25 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
           required: ["pr_title"],
         },
       },
+      {
+        name: "push_with_decision_doc",
+        description:
+          "Create a decision reasoning document and push all changes to GitHub in one action. Call this when the developer asks to push changes. Automatically finalizes the reasoning doc, commits it, and pushes everything.",
+        inputSchema: {
+          type: "object",
+          properties: {
+            title: {
+              type: "string",
+              description: "The feature/PR title for this reasoning document (auto-generated if not provided)",
+            },
+            description: {
+              type: "string",
+              description: "Optional summary of changes being pushed",
+            },
+          },
+          required: [],
+        },
+      },
     ],
   };
 });
@@ -459,6 +478,53 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
         }
       }
 
+      case "push_with_decision_doc": {
+        const { execSync } = await import("node:child_process");
+        const prTitle = args.title || `Changes from ${new Date().toISOString().split("T")[0]}`;
+        const prDescription = args.description || null;
+        const branch = process.env.GITHUB_BRANCH || "main";
+
+        try {
+          // 1. Finalize the reasoning document
+          const r = await runFinalization(prTitle, prDescription, branch);
+
+          // 2. Push all changes (decision doc + code changes)
+          execSync("git push", { stdio: "inherit" });
+
+          return {
+            content: [
+              {
+                type: "text",
+                text: [
+                  `✅ Decision document created and pushed successfully!`,
+                  ``,
+                  `Decision Doc:`,
+                  `  File: ${r.filePath}`,
+                  `  Commit: ${r.commitSha}`,
+                  ``,
+                  `Summary:`,
+                  `  - ${r.decisionsCount} decisions logged`,
+                  `  - ${r.alternativesCount} alternatives documented`,
+                  `  - Intent: ${r.hasIntent ? "captured" : "not logged"}`,
+                  ``,
+                  `All changes pushed to ${branch}`,
+                ].join("\n"),
+              },
+            ],
+          };
+        } catch (err) {
+          return {
+            content: [
+              {
+                type: "text",
+                text: `Failed to push changes: ${err.message}\n\nMake sure you have uncommitted changes and proper Git configuration.`,
+              },
+            ],
+            isError: true,
+          };
+        }
+      }
+
       default:
         return {
           content: [{ type: "text", text: `Unknown tool: ${name}` }],