@programinglive/dev-workflow-mcp-server 1.13.15 → 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.15",
+  "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()