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

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.15",
   "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,20 +164,40 @@ 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) {
     const lastCommitMessage = await git.getLastCommitMessage();
     const effectiveCommitMessage = lastCommitMessage || providedCommitMessage;
 
-    // "Tidak ada perubahan" bukan berarti "sudah ter-push". HEAD bisa saja
-    // berupa commit yang belum pernah dikirim. Memeriksanya mencegah tool
-    // menandai selesai padahal remote masih tertinggal.
+    // "Tidak ada perubahan" bukan berarti "sudah ter-push", dan juga bukan
+    // berarti "panggilan ini sudah commit". HEAD bisa saja berupa commit yang
+    // belum pernah dikirim, dan bisa juga sudah terkirim oleh panggilan
+    // SEBELUMNYA. Memeriksanya mencegah tool menandai selesai padahal remote
+    // masih tertinggal - dan kalimat "Assuming commit and push already
+    // completed" (dipakai sampai 31 Oktober 2026) membuat agen percaya
+    // pekerjaannya sudah masuk padahal tidak ada yang dikerjakan sama sekali.
     const pushed = await verifyPushed(git, branchForPush);
     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.`
       );
@@ -165,9 +209,20 @@ export async function handleCommitAndPush(args, context) {
     workflowState.state.currentPhase = isReleaseSatisfied(workflowState.state) ? "ready_to_complete" : "release";
     await workflowState.save();
 
+    const noActiveTask = workflowState.state.currentPhase === "idle" || !workflowState.state.taskDescription;
+
     return textResponse(
-      `ℹ️ No changes detected. Assuming commit and push already completed.\nRepo: ${repoLabel()}\nBranch: ${branchForPush} (verified on remote)\nLast commit: ${effectiveCommitMessage || "(unavailable)"
-      }\n\nNext: Run 'perform_release' to handle the release and push tags.`
+      `ℹ️ No changes detected. No commit was made by this call.\n` +
+      `Repo: ${repoName}\nBranch: ${branchForPush} (verified on remote, HEAD ${pushed.head.slice(0, 8)})\n` +
+      `Commit terakhir: ${effectiveCommitMessage || "(tidak terbaca)"}\n\n` +
+      (noActiveTask
+        ? `Tidak ada task aktif di state workflow (currentPhase "idle"), jadi memang tidak ada ` +
+          `yang diharapkan dari panggilan ini.\n\n`
+        : "") +
+      `Yang terbukti: commit terakhir sudah ada di remote. Yang TIDAK terbukti: commit itu berisi ` +
+      `pekerjaan task ini. Kalau perubahan seharusnya ada, berarti belum ditulis, sudah di-commit ` +
+      `oleh panggilan sebelumnya, atau berada di repo/branch lain.\n\n` +
+      `Next: Run 'perform_release' to handle the release and push tags.`
     );
   }
 
@@ -228,7 +283,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 +297,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/factory-handlers.js

@@ -134,8 +134,19 @@ export async function handleClaimFactoryTask(args = {}, { workflowState } = {})
     // dikerjakan, dan papan terlihat penuh task mati.
     startKeeping(config, session, step);
 
+    // Arsipkan task lokal yang sedang berjalan, jangan hapus diam-diam.
+    //
+    // 31 Oktober 2026 claim_factory_task dipanggil di tengah task lain yang baru
+    // sampai tahap rilis. `reset()` tanpa archive membuat task itu lenyap: tidak
+    // ada entri history, tidak ada peringatan, tidak disebut di jawaban.
+    // `start_task` sudah mengarsipkan sejak awal; claim tidak.
+    let archivedTask = null;
     if (workflowState) {
-      workflowState.reset();
+      const archived = workflowState.reset({
+        archive: true,
+        reason: "Diganti claim_factory_task sebelum task selesai",
+      });
+      archivedTask = archived?.archivedTask || null;
       workflowState.state.currentPhase = "coding";
       workflowState.state.taskDescription = `[${task.taskKey || task.id}] ${task.title}`;
       workflowState.state.taskType = "other";
@@ -145,6 +156,10 @@ export async function handleClaimFactoryTask(args = {}, { workflowState } = {})
     return textResponse(
       [
         `Claimed [${task.taskKey || task.id}] ${task.title}`,
+        archivedTask
+          ? `⚠️ Task lokal yang belum selesai diarsipkan ke history: "${archivedTask}". ` +
+            `Pekerjaannya tidak dihapus, tetapi tidak lagi aktif di sesi ini.`
+          : "",
         session.tookOver && session.previousHolder
           ? `Took over from ${session.previousHolder.deviceName} (${session.previousHolder.agent}). The reason is recorded on the board.`
           : "",

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))) {