@programinglive/dev-workflow-mcp-server 1.13.13 → 1.13.14

package/git-facade.js

@@ -16,6 +16,7 @@ import {
   getRemoteBranchSha,
   getStagedChanges,
   getStatusOutput,
+  getToplevel,
   hasStagedChanges,
   hasTestChanges,
   hasWorkingChanges,
@@ -34,6 +35,7 @@ export function createGitFacade() {
     getRemoteBranchSha,
     getLastCommitMessage,
     getStatusOutput,
+    getToplevel,
     workingTreeSummary,
   };
 }

package/git-helpers.js

@@ -173,6 +173,18 @@ export async function getStatusOutput() {
   return stdout;
 }
 
+/**
+ * Direktori teratas repo tempat git BENAR-BENAR berjalan.
+ *
+ * `git rev-parse --show-toplevel` menjawab dari repo yang aktif, bukan dari
+ * tebakan kita tentang direktori mana yang dimaksud. Dipakai untuk memastikan
+ * laporan `commit_and_push` menyebut repo yang sama dengan yang di-commit.
+ */
+export async function getToplevel() {
+  const { stdout } = await exec("git rev-parse --show-toplevel");
+  return stdout.trim();
+}
+
 export async function getPrimaryBranch() {
   try {
     // Check if 'main' exists on origin

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@programinglive/dev-workflow-mcp-server",
-  "version": "1.13.13",
+  "version": "1.13.14",
   "description": "MCP server to enforce development workflow discipline",
   "homepage": "https://devworkflow.programinglive.com/",
   "mcpName": "io.github.programinglive/dev-workflow-mcp-server",
@@ -21,7 +21,7 @@
     "start": "node index.js",
     "local": "node index.js",
     "setup": "node scripts/setup-stack.js",
-    "test": "node --no-deprecation --test",
+    "test": "node --no-deprecation --import ./tests/setup-env.js --test",
     "test:security": "node --no-deprecation --test tests/security*.test.js",
     "security:audit": "node index.js call security_audit",
     "lint": "node scripts/lint.mjs",

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

@@ -2,6 +2,7 @@ import { textResponse, isReleaseSatisfied, formatChecklist } from "./shared.js";
 import { handleReadyCheck } from "./status-handlers.js";
 import { containsTestFilesInStatus } from "../../git-helpers.js";
 import { resolveProjectDir, setActiveProjectDir, getActiveProjectDir } from "../../exec.js";
+import path from "path";
 
 /**
  * Label repo yang benar-benar dipakai, untuk ditampilkan di setiap jawaban.
@@ -24,6 +25,18 @@ function repoLabel() {
   }
 }
 
+/**
+ * Apakah dua path menunjuk direktori yang sama.
+ *
+ * Windows tidak membedakan huruf besar/kecil dan kadang memberi garis miring
+ * di akhir, jadi perbandingan string mentah akan salah.
+ */
+function samePath(a, b) {
+  if (!a || !b) return false;
+  const normal = (value) => path.resolve(value).replace(/[\\/]+$/, "").toLowerCase();
+  return normal(a) === normal(b);
+}
+
 /**
  * Terapkan `projectDir` dari argumen tool bila ada.
  * Mengembalikan pesan kesalahan, atau null bila tidak ada masalah.
@@ -127,10 +140,21 @@ export async function handleCommitAndPush(args, context) {
   let primaryBranch;
   let currentBranch;
   let hasWorkingChanges;
+  let gitToplevel = "";
   try {
     primaryBranch = await git.getPrimaryBranch();
     currentBranch = await git.getCurrentBranch();
     hasWorkingChanges = await git.hasWorkingChanges();
+
+    // Tempat git BENAR-BENAR berjalan, menurut git sendiri.
+    //
+    // 31 Oktober 2026 laporan menyebut `portal-beautyworld` untuk commit yang
+    // masuk ke repo lain, karena labelnya dibaca dari variabel global yang bisa
+    // berubah - bukan dari repo tempat git berjalan. Laporan seperti itu membuat
+    // agen percaya commit-nya ada di tempat yang salah.
+    if (typeof git.getToplevel === "function") {
+      gitToplevel = String((await git.getToplevel()) || "").trim();
+    }
   } catch (error) {
     return textResponse(
       `❌ Tidak bisa memeriksa repository Git, jadi tidak ada yang di-commit.\n\n` +
@@ -140,6 +164,22 @@ export async function handleCommitAndPush(args, context) {
     );
   }
 
+  // Kalau git berjalan di repo lain daripada yang diminta, BERHENTI. Commit ke
+  // repo yang salah jauh lebih mahal daripada satu pesan kesalahan.
+  const requestedRepo = getActiveProjectDir();
+  if (gitToplevel && requestedRepo && !samePath(gitToplevel, requestedRepo)) {
+    return textResponse(
+      `❌ Dibatalkan: git berjalan di repo yang BERBEDA dari yang diminta.\n\n` +
+      `Diminta    : ${requestedRepo}\n` +
+      `Dipakai git: ${gitToplevel}\n\n` +
+      `Tidak ada yang di-commit. Sebutkan 'projectDir' yang benar pada 'commit_and_push', ` +
+      `atau jalankan ulang 'start_task' dengan 'projectDir' repo tersebut.`
+    );
+  }
+
+  // Label laporan memakai repo yang benar-benar dipakai git bila diketahui.
+  const repoName = gitToplevel || repoLabel();
+
   const branchForPush = pickPushBranch({ requestedBranch, currentBranch, primaryBranch });
 
   if (!hasWorkingChanges) {
@@ -153,7 +193,7 @@ export async function handleCommitAndPush(args, context) {
     if (!pushed.ok) {
       return textResponse(
         `⚠️ Tidak ada perubahan untuk di-commit, tetapi HEAD BELUM ada di remote.\n` +
-        `Repo: ${repoLabel()}\nBranch: ${branchForPush || "(tidak diketahui)"}\n` +
+        `Repo: ${repoName}\nBranch: ${branchForPush || "(tidak diketahui)"}\n` +
         `Sebab: ${pushed.detail}\n\n` +
         `Jalankan 'commit_and_push' lagi dengan 'branch' yang benar, atau push manual.`
       );
@@ -166,7 +206,7 @@ export async function handleCommitAndPush(args, context) {
     await workflowState.save();
 
     return textResponse(
-      `ℹ️ No changes detected. Assuming commit and push already completed.\nRepo: ${repoLabel()}\nBranch: ${branchForPush} (verified on remote)\nLast commit: ${effectiveCommitMessage || "(unavailable)"
+      `ℹ️ No changes detected. Assuming commit and push already completed.\nRepo: ${repoName}\nBranch: ${branchForPush} (verified on remote)\nLast commit: ${effectiveCommitMessage || "(unavailable)"
       }\n\nNext: Run 'perform_release' to handle the release and push tags.`
     );
   }
@@ -228,7 +268,7 @@ export async function handleCommitAndPush(args, context) {
   if (!pushed.ok) {
     return textResponse(
       `❌ Commit dibuat, tetapi TIDAK ter-push.\n` +
-      `Repo: ${repoLabel()}\nBranch: ${branchForPush || "(tidak diketahui)"}\n` +
+      `Repo: ${repoName}\nBranch: ${branchForPush || "(tidak diketahui)"}\n` +
       `Sebab: ${pushed.detail}\n\n` +
       `Commit lokal: ${generatedSummary}\n\n` +
       `Jalankan 'commit_and_push' lagi dengan 'branch' yang benar, atau push manual.`
@@ -242,7 +282,7 @@ export async function handleCommitAndPush(args, context) {
   await workflowState.save();
 
   return textResponse(
-    `✅ Commit and push completed!\nRepo: ${repoLabel()}\n\nCommit message: ${generatedSummary}\nPushed to: ${branchForPush} (verified at ${pushed.head.slice(0, 8)})\n\nNext: Run 'perform_release' to handle the release and push tags.`
+    `✅ Commit and push completed!\nRepo: ${repoName}\n\nCommit message: ${generatedSummary}\nPushed to: ${branchForPush} (verified at ${pushed.head.slice(0, 8)})\n\nNext: Run 'perform_release' to handle the release and push tags.`
   );
 }
 

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

@@ -1,6 +1,42 @@
 import { textResponse, isReleaseSatisfied, resetToCommitIfWorkingChanges, summarizeHistory } from "./shared.js";
 import { resolveProjectDir, setActiveProjectDir, getActiveProjectDir } from "../../exec.js";
 
+/**
+ * Label repo yang dipakai, untuk ditampilkan di laporan.
+ *
+ * Sama seperti di commit-handlers: laporan harus menyebut repo yang benar-benar
+ * dipakai, bukan tebakan.
+ */
+function repoLabel() {
+  try {
+    const resolved = resolveProjectDir({
+      explicit: getActiveProjectDir(),
+      env: process.env,
+      candidates: [process.cwd(), process.env.INIT_CWD],
+    });
+    const origin = resolved.source === "env" ? " — dari WORKFLOW_PROJECT_DIR (global)" : "";
+    return resolved.dir + origin;
+  } catch {
+    return "(tidak bisa ditentukan)";
+  }
+}
+
+/**
+ * Menjalankan perintah git yang hasilnya hanya dipakai untuk laporan.
+ *
+ * Kegagalan di sini tidak boleh menggagalkan rilis: kalau `git describe`
+ * tidak menemukan tag, itu bukan alasan untuk berhenti. Yang penting adalah
+ * rilis tetap berjalan dan laporannya jujur tentang apa yang tidak diketahui.
+ */
+async function safeExecStdout(exec, command) {
+  try {
+    const { stdout } = await exec(command);
+    return String(stdout || "").trim();
+  } catch {
+    return "";
+  }
+}
+
 export async function handleSkipRelease(args, workflowState) {
   if (workflowState.state.currentPhase === "idle") {
     return textResponse("⚠️ Please start a task first using 'start_task' before skipping release.");
@@ -185,11 +221,45 @@ export async function handlePerformRelease(args, context) {
     );
   }
 
+  // Tipe rilis WAJIB sah sebelum perintah disusun.
+  //
+  // 31 Oktober 2026: `determineReleaseTypeFromCommit` mengembalikan string kosong,
+  // lalu perintahnya menjadi `npm run release -- --release-as ` + "null" ->
+  // standard-version menulis `"version": null` dan membuat tag `vnull`, lalu
+  // keduanya di-push. Yang salah bukan hanya hasilnya: perintah yang mustahil
+  // itu tetap dijalankan tanpa perlawanan.
+  if (!recognizedTypes.has(releaseType)) {
+    return textResponse(
+      `⚠️ Tipe rilis tidak bisa ditentukan dari commit terakhir, jadi rilis TIDAK dijalankan.\n\n` +
+      `Commit terakhir: ${lastCommitMessage || "(kosong)"}\n\n` +
+      `Sebutkan tipe rilisnya secara eksplisit:\n` +
+      `  • 'perform_release' dengan releaseType: "patch" | "minor" | "major"\n` +
+      `  • atau command: "npm run release:patch"\n\n` +
+      `Tanpa itu perintahnya akan menjadi "--release-as null", yang menghasilkan tag vnull ` +
+      `dan package.json berversi null.`
+    );
+  }
+
   const finalCommand =
     releaseCommand.includes("--release-as") || /release:(major|minor|patch)/i.test(releaseCommand)
       ? releaseCommand
       : `${releaseCommand} -- --release-as ${releaseType}`;
 
+  // Jaring pengaman terakhir: apa pun yang terjadi di atas, jangan pernah
+  // menjalankan perintah yang membawa tipe rilis kosong.
+  if (/--release-as\s+(?:null|undefined|)\s*$/i.test(finalCommand) || /\snull\b/i.test(finalCommand)) {
+    return textResponse(
+      `⚠️ Perintah rilis akan berisi tipe rilis yang tidak sah, jadi TIDAK dijalankan:\n\n` +
+      `  ${finalCommand}\n\n` +
+      `Sebutkan tipe rilisnya: releaseType "patch" | "minor" | "major".`
+    );
+  }
+
+  // Catat keadaan SEBELUM rilis supaya, kalau push gagal, kita bisa menyebut
+  // persis apa yang tertinggal di mesin.
+  const headBefore = await safeExecStdout(exec, "git rev-parse HEAD");
+  const tagBefore = await safeExecStdout(exec, "git describe --tags --abbrev=0");
+
   try {
     await exec(finalCommand);
   } catch (error) {
@@ -198,6 +268,9 @@ export async function handlePerformRelease(args, context) {
     );
   }
 
+  const headAfter = await safeExecStdout(exec, "git rev-parse HEAD");
+  const tagAfter = await safeExecStdout(exec, "git describe --tags --abbrev=0");
+
   const branchForPush = workflowState.state.lastPushBranch || (await git.getCurrentBranch()) || "";
   const pushCommand = branchForPush
     ? `git push --follow-tags origin ${utils.shellEscape(branchForPush)}`
@@ -206,8 +279,29 @@ export async function handlePerformRelease(args, context) {
   try {
     await exec(pushCommand);
   } catch (error) {
+    // Rilis SUDAH membuat commit dan tag di mesin; hanya push yang gagal.
+    // Tanpa menyebut yang tertinggal, tidak jelas apa yang harus dibereskan -
+    // dan pada 31 Oktober 2026 hal itu membuat rilis tampak selesai.
+    const leftBehind = [];
+    if (headAfter && headAfter !== headBefore) {
+      leftBehind.push(`  • commit rilis ${headAfter.slice(0, 8)}`);
+    }
+    if (tagAfter && tagAfter !== tagBefore) {
+      leftBehind.push(`  • tag ${tagAfter}`);
+    }
+
     return textResponse(
-      `❌ git push (with tags) failed:\n\n${error.stderr || error.stdout || error.message}`
+      `❌ Push rilis GAGAL. Rilis TIDAK tercatat sebagai selesai.\n\n` +
+      `Repo: ${repoLabel()}\nBranch: ${branchForPush || "(tidak diketahui)"}\n` +
+      `Penyebab: ${error.stderr || error.stdout || error.message}\n\n` +
+      (leftBehind.length > 0
+        ? `Tertinggal di mesin (belum sampai ke remote):\n${leftBehind.join("\n")}\n\n`
+        : "") +
+      `Pemulihan:\n` +
+      `  1. periksa: git log --oneline -3 && git tag --points-at HEAD\n` +
+      `  2. kirim ulang: git push --follow-tags origin ${branchForPush || "<branch>"}\n` +
+      `  3. atau batalkan rilis lokal: git reset --hard HEAD~1 && git tag -d ${tagAfter || "<tag>"}\n\n` +
+      `Jangan tandai task selesai sebelum tag benar-benar ada di remote.`
     );
   }
 

package/workflow-state.js

@@ -400,6 +400,19 @@ async function pathExists(filePath) {
 }
 
 async function ensureCompatibilityLinks(primaryPath) {
+  // Cermin kompatibilitas ditulis ke `<paket>/.state/workflow-state.json`,
+  // BUKAN ke lokasi state yang sebenarnya. Itu benar untuk pemakaian nyata
+  // (CLI dan dasbor web membaca jalur lama), tetapi merusak saat tes: setiap
+  // WorkflowState yang dibuat tes - meski di direktori sementara - menimpa
+  // state repo, dan 31 Oktober 2026 itu menghapus task yang sedang berjalan di
+  // tengah `npm test`.
+  //
+  // Tes mematikan cermin ini lewat DEV_WORKFLOW_NO_COMPAT_LINKS=1 (lihat
+  // tests/setup-env.js). Perilaku produksi tidak berubah.
+  if (process.env.DEV_WORKFLOW_NO_COMPAT_LINKS === "1") {
+    return;
+  }
+
   const resolvedPrimary = path.resolve(primaryPath);
 
   if (!(await pathExists(resolvedPrimary))) {