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

package/index.js

@@ -14,7 +14,7 @@ import {
 } from "@modelcontextprotocol/sdk/types.js";
 import { fileURLToPath } from "url";
 import path from "path";
-import { acquireProcessLock, releaseProcessLock, lockWarning } from "./process-lock.js";
+import { acquireProcessLock, releaseProcessLock, lockWarning, projectLockDir } from "./process-lock.js";
 import { exec, setActiveProjectDir } from "./exec.js";
 import { normalizeRequestArgs, shellEscape } from "./utils.js";
 import { createGitFacade } from "./git-facade.js";
@@ -199,26 +199,31 @@ export async function main() {
     // Select request metadata before the first disk write. Creating an idle
     // PID fallback here leaves a phantom session alongside the real sessions.
     safeLog('error', `[${new Date().toISOString()}] Workflow state initialization complete.`);
-
-    // Penanda proses: 31 Oktober 2026 dua proses MCP hidup bersamaan dan
-    // memegang kredensial Factory yang berbeda, sehingga tool yang sama
-    // menjawab berbeda tergantung proses mana yang melayaninya. Proses lain
-    // TIDAK dibunuh (itu berbahaya); cukup dilaporkan supaya pengguna tahu
-    // harus menutup OpenCode dan membukanya lagi.
-    try {
-      const stateDir = path.dirname(path.resolve(workflowState.stateFile));
-      const warning = lockWarning(acquireProcessLock({ dir: stateDir }));
-      if (warning) safeLog('error', warning);
-      process.once('exit', () => releaseProcessLock({ dir: stateDir }));
-    } catch (lockError) {
-      safeLog('error', 'Warning: could not record process lock:', lockError.message);
-    }
   } catch (error) {
     safeLog('error', `[${new Date().toISOString()}] Warning: Failed to initialize workflow state:`, error.message);
     // Continue anyway - workflow state is optional for MCP to function
     workflowState = new WorkflowState();
   }
 
+  // Penanda proses - DI LUAR try di atas, dengan sengaja.
+  //
+  // 31 Oktober 2026: blok ini semula berada di dalam try yang sama dengan
+  // inisialisasi state. Saat dua proses MCP start serempak, satu di antaranya
+  // gagal menginisialisasi state (berebut berkas yang sama), masuk ke catch, dan
+  // penanda prosesnya tidak pernah ditulis. Hasilnya dua proses hidup dengan
+  // satu lock file: proses ganda tidak pernah terdeteksi.
+  //
+  // Lokasinya juga harus di `.state` PROYEK, bukan di direktori user: direktori
+  // user bisa berbeda antar proses, sehingga keduanya tidak saling melihat.
+  try {
+    const lockDir = projectLockDir(workflowState.stateFile);
+    const warning = lockWarning(acquireProcessLock({ dir: lockDir }));
+    if (warning) safeLog('error', warning);
+    process.once('exit', () => releaseProcessLock({ dir: lockDir }));
+  } catch (lockError) {
+    safeLog('error', 'Warning: could not record process lock:', lockError.message);
+  }
+
   const transport = new StdioServerTransport();
   // Cover SDK-generated protocol errors as well as tool-handler responses.
   const send = transport.send.bind(transport);

package/package.json

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

package/process-lock.js

@@ -24,6 +24,41 @@ export function processLockPath({ dir }) {
   return path.join(dir, LOCK_FILENAME);
 }
 
+/**
+ * Direktori `.state` milik PROYEK untuk sebuah berkas state.
+ *
+ * Kenapa bukan `path.dirname(stateFile)` saja: berkas state sesi berada di
+ * `.state/users/<user>/`, dan direktori user bisa BERBEDA antar proses (id user
+ * dibuat per direktori state). Pada 31 Oktober 2026 dua proses MCP hidup
+ * bersamaan tetapi hanya satu lock file yang tercipta, karena masing-masing
+ * menulis ke direktori user-nya sendiri - jadi keduanya tidak pernah saling
+ * melihat dan peringatan proses ganda tidak muncul.
+ *
+ * Lock harus satu per PROYEK, bukan per user. Fungsi ini naik sampai menemukan
+ * direktori bernama `.state` dan memakai direktori itu. Kalau tidak ada (state
+ * di luar tata letak itu), pakai direktori berkasnya sendiri.
+ */
+export function projectLockDir(stateFilePath) {
+  const resolved = path.resolve(stateFilePath);
+  let current = path.dirname(resolved);
+  const { root } = path.parse(current);
+
+  while (true) {
+    if (path.basename(current) === ".state") {
+      return current;
+    }
+
+    const parent = path.dirname(current);
+    if (parent === current || current === root) {
+      break;
+    }
+
+    current = parent;
+  }
+
+  return path.dirname(resolved);
+}
+
 /**
  * Membaca isi lock, atau null bila tidak ada / tidak terbaca.
  *
@@ -66,7 +101,13 @@ function defaultIsAlive(pid) {
  *   { acquired: true, tookOverFrom: <pid> }   -> pemegang lama sudah mati
  *   { acquired: false, holder: {...} }        -> ada proses lain yang hidup
  */
-export function acquireProcessLock({ dir, pid = process.pid, now = Date.now(), isAlive = defaultIsAlive }) {
+export function acquireProcessLock({
+  dir,
+  pid = process.pid,
+  now = Date.now(),
+  isAlive = defaultIsAlive,
+  writeFileImpl = writeFileSync,
+}) {
   const existing = readProcessLock({ dir });
 
   if (existing && existing.pid !== pid && isAlive(existing.pid)) {
@@ -79,10 +120,17 @@ export function acquireProcessLock({ dir, pid = process.pid, now = Date.now(), i
 
   try {
     mkdirSync(dir, { recursive: true });
-    writeFileSync(processLockPath({ dir }), JSON.stringify(payload), "utf8");
-  } catch {
-    // Tidak bisa menulis penanda bukan alasan mematikan server.
-    return { acquired: true, wrote: false };
+    writeFileImpl(processLockPath({ dir }), JSON.stringify(payload), "utf8");
+  } catch (error) {
+    // Tidak bisa menulis penanda bukan alasan mematikan server, TETAPI juga
+    // tidak boleh ditelan: sampai 31 Oktober 2026 kegagalan ini hanya
+    // mengembalikan { acquired: true } sehingga sistem tampak sehat padahal
+    // tidak ada penanda apa pun di disk. Simpan sebabnya supaya bisa dilaporkan.
+    return {
+      acquired: true,
+      wrote: false,
+      error: (error && (error.code || error.message)) || "unknown",
+    };
   }
 
   return tookOverFrom ? { acquired: true, tookOverFrom } : { acquired: true };
@@ -118,7 +166,16 @@ function safeHostname() {
  * nyata, dan supaya pemanggil bebas memilih tidak menampilkan apa pun.
  */
 export function lockWarning(result) {
-  if (!result || result.acquired !== false || !result.holder) return null;
+  if (!result) return null;
+
+  if (result.wrote === false) {
+    return (
+      `Tidak bisa menulis penanda proses ke ${LOCK_FILENAME} (${result.error}). ` +
+      "Deteksi proses MCP ganda tidak akan bekerja sampai ini diperbaiki."
+    );
+  }
+
+  if (result.acquired !== false || !result.holder) return null;
 
   const started = Number.isFinite(result.holder.startedAt)
     ? new Date(result.holder.startedAt).toISOString()

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