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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@programinglive/dev-workflow-mcp-server",
-  "version": "1.13.14",
+  "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",

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

@@ -186,9 +186,13 @@ export async function handleCommitAndPush(args, context) {
     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(
@@ -205,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: ${repoName}\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.`
     );
   }
 

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.`
           : "",