@programinglive/dev-workflow-mcp-server 1.13.1 → 1.13.2

package/git-helpers.js

@@ -1,27 +1,37 @@
 import { exec } from "./exec.js";
 
 export async function hasStagedChanges() {
-  try {
-    const { stdout } = await exec("git status --porcelain");
-    return stdout
-      .split("\n")
-      .filter(Boolean)
-      .some((line) => {
-        const indexStatus = line[0];
-        return indexStatus && indexStatus !== " " && indexStatus !== "?";
-      });
-  } catch (error) {
-    return false;
-  }
+  const { stdout } = await exec("git status --porcelain");
+  return stdout
+    .split("\n")
+    .filter(Boolean)
+    .some((line) => {
+      const indexStatus = line[0];
+      return indexStatus && indexStatus !== " " && indexStatus !== "?";
+    });
 }
 
+/**
+ * Apakah ada perubahan yang belum di-commit?
+ *
+ * MENGAPA TIDAK `catch { return false }`
+ * --------------------------------------
+ * Dulu fungsi ini menelan setiap error git dan menjawab `false`. Pemanggil
+ * membacanya sebagai "tidak ada perubahan", lalu `commit_and_push` menjawab
+ * "No changes detected. Assuming commit and push already completed" - sukses
+ * palsu, padahal tidak ada commit sama sekali.
+ *
+ * Yang terjadi 30 Oktober 2026: server MCP dijalankan dengan cwd direktori sesi
+ * OpenCode (`Documents\Default Project`) yang bukan repo git. `git status`
+ * gagal, error ditelan, delapan berkas tidak pernah ter-commit, dan tool tetap
+ * melaporkan berhasil.
+ *
+ * "Tidak tahu" bukan "tidak ada perubahan". Sekarang error diteruskan supaya
+ * pemanggil bisa melaporkan masalah sebenarnya.
+ */
 export async function hasWorkingChanges() {
-  try {
-    const { stdout } = await exec("git status --porcelain");
-    return stdout.split("\n").some((line) => line && line.trim().length > 0);
-  } catch (error) {
-    return false;
-  }
+  const { stdout } = await exec("git status --porcelain");
+  return stdout.split("\n").some((line) => line && line.trim().length > 0);
 }
 
 export function workingTreeSummary(statusOutput) {
@@ -113,51 +123,31 @@ function parseNameStatusLine(line) {
 }
 
 export async function getStagedChanges() {
-  try {
-    const { stdout } = await exec("git diff --cached --name-status");
-    return stdout
-      .split("\n")
-      .map((line) => parseNameStatusLine(line))
-      .filter(Boolean);
-  } catch (error) {
-    return [];
-  }
+  const { stdout } = await exec("git diff --cached --name-status");
+  return stdout
+    .split("\n")
+    .map((line) => parseNameStatusLine(line))
+    .filter(Boolean);
 }
 
 export async function hasTestChanges() {
-  try {
-    const { stdout } = await exec("git status --porcelain");
-    return containsTestFilesInStatus(stdout);
-  } catch (error) {
-    return false;
-  }
+  const { stdout } = await exec("git status --porcelain");
+  return containsTestFilesInStatus(stdout);
 }
 
 export async function getLastCommitMessage() {
-  try {
-    const { stdout } = await exec("git log -1 --pretty=%B");
-    return stdout.trim();
-  } catch (error) {
-    return "";
-  }
+  const { stdout } = await exec("git log -1 --pretty=%B");
+  return stdout.trim();
 }
 
 export async function getCurrentBranch() {
-  try {
-    const { stdout } = await exec("git rev-parse --abbrev-ref HEAD");
-    return stdout.trim();
-  } catch (error) {
-    return "";
-  }
+  const { stdout } = await exec("git rev-parse --abbrev-ref HEAD");
+  return stdout.trim();
 }
 
 export async function getStatusOutput() {
-  try {
-    const { stdout } = await exec("git status --porcelain");
-    return stdout;
-  } catch (error) {
-    return "";
-  }
+  const { stdout } = await exec("git status --porcelain");
+  return stdout;
 }
 
 export async function getPrimaryBranch() {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@programinglive/dev-workflow-mcp-server",
-  "version": "1.13.1",
+  "version": "1.13.2",
   "description": "MCP server to enforce development workflow discipline",
   "homepage": "https://devworkflow.programinglive.com/",
   "mcpName": "io.github.programinglive/dev-workflow-mcp-server",

package/tools/handler-modules/commit-handlers.js

@@ -15,11 +15,26 @@ export async function handleCommitAndPush(args, context) {
     typeof args.commitMessage === "string" ? args.commitMessage.trim() : "";
 
   const requestedBranch = typeof args.branch === "string" ? args.branch.trim() : "";
-  const primaryBranch = await git.getPrimaryBranch();
-  const currentBranch = await git.getCurrentBranch();
-  const branchForPush = requestedBranch || primaryBranch || currentBranch || "";
 
-  const hasWorkingChanges = await git.hasWorkingChanges();
+  // Preflight: kalau git tidak bisa dijalankan, jangan menebak. Dulu rantai ini
+  // gagal, error ditelan helper, lalu tool menjawab "No changes detected" dan
+  // menandai commit selesai padahal tidak ada commit sama sekali.
+  let primaryBranch;
+  let currentBranch;
+  let hasWorkingChanges;
+  try {
+    primaryBranch = await git.getPrimaryBranch();
+    currentBranch = await git.getCurrentBranch();
+    hasWorkingChanges = await git.hasWorkingChanges();
+  } catch (error) {
+    return textResponse(
+      `❌ Tidak bisa memeriksa repository Git, jadi tidak ada yang di-commit.\n\n` +
+      `Penyebab: ${error.message}\n\n` +
+      `Set WORKFLOW_PROJECT_DIR ke direktori proyek, atau jalankan server MCP dari dalam direktori proyek.`
+    );
+  }
+
+  const branchForPush = requestedBranch || primaryBranch || currentBranch || "";
 
   if (!hasWorkingChanges) {
     const lastCommitMessage = await git.getLastCommitMessage();

package/tools/handler-modules/release-handlers.js

@@ -50,7 +50,18 @@ export async function handlePerformRelease(args, context) {
     return textResponse("⚠️ Please start a task first using 'start_task'!");
   }
 
-  if (await resetToCommitIfWorkingChanges(workflowState, git)) {
+  let hasNewChanges;
+  try {
+    hasNewChanges = await resetToCommitIfWorkingChanges(workflowState, git);
+  } catch (error) {
+    return textResponse(
+      `❌ Tidak bisa memeriksa repository Git, jadi status rilis tidak dapat diverifikasi.\n\n` +
+      `Penyebab: ${error.message}\n\n` +
+      `Set WORKFLOW_PROJECT_DIR ke direktori proyek, atau jalankan server MCP dari dalam direktori proyek.`
+    );
+  }
+
+  if (hasNewChanges) {
     return textResponse(
       "⚠️ Detected new changes after the last commit. Please run 'commit_and_push' again before recording a release."
     );
@@ -63,7 +74,16 @@ export async function handlePerformRelease(args, context) {
   }
 
   if (!workflowState.state.commitAndPushCompleted) {
-    const hasWorkingChanges = await git.hasWorkingChanges();
+    let hasWorkingChanges;
+    try {
+      hasWorkingChanges = await git.hasWorkingChanges();
+    } catch (error) {
+      return textResponse(
+        `❌ Tidak bisa memeriksa repository Git, jadi status rilis tidak dapat diverifikasi.\n\n` +
+        `Penyebab: ${error.message}\n\n` +
+        `Set WORKFLOW_PROJECT_DIR ke direktori proyek, atau jalankan server MCP dari dalam direktori proyek.`
+      );
+    }
 
     if (hasWorkingChanges) {
       return textResponse(