@infinitedusky/indusk-mcp 1.19.0 → 1.20.0

package/hooks/eval-trigger.js

@@ -226,15 +226,48 @@ syslog(
 );
 
 const syslogPath = resolve(projectRoot, ".indusk", "eval", "system.log");
+// NOTE: this inline script runs with --input-type=module (see spawn below).
+// ESM scope — use static imports from node: specifiers only. CJS module
+// resolution throws ReferenceError in ESM scope at parse, and stdio:"ignore"
+// on the detached spawn would swallow the error. For the full history see
+// .indusk/planning/archive/bug-fix-eval-agent/diagnosis.md
 const evaluatorScript = `
-const fs = require("fs");
-const path = require("path");
+import { mkdirSync, appendFileSync } from "node:fs";
+import { dirname, join } from "node:path";
 function syslog(msg) {
   try {
-    fs.mkdirSync(path.dirname("${syslogPath}"), { recursive: true });
-    fs.appendFileSync("${syslogPath}", new Date().toISOString() + " " + msg + "\\n");
+    mkdirSync(dirname("${syslogPath}"), { recursive: true });
+    appendFileSync("${syslogPath}", new Date().toISOString() + " " + msg + "\\n");
   } catch {}
 }
+// Belt-and-suspenders: if the evaluator crashes with an unhandled exception
+// or rejection, write a loud error entry to results.log before exit so the
+// failure is never silent again.
+function writeErrorResult(message) {
+  try {
+    const logPath = join(${JSON.stringify(projectRoot)}, ".indusk", "eval", "results.log");
+    mkdirSync(dirname(logPath), { recursive: true });
+    const entry = JSON.stringify({
+      version: 1,
+      timestamp: new Date().toISOString(),
+      mode: "eval",
+      changeId: ${JSON.stringify(changeId)},
+      error: true,
+      message,
+    });
+    appendFileSync(logPath, entry + "\\n", "utf8");
+  } catch {}
+}
+process.on("uncaughtException", (err) => {
+  syslog("evaluator uncaughtException — " + (err && err.message ? err.message : String(err)));
+  writeErrorResult("uncaughtException: " + (err && err.message ? err.message : String(err)));
+  process.exit(1);
+});
+process.on("unhandledRejection", (reason) => {
+  syslog("evaluator unhandledRejection — " + (reason && reason.message ? reason.message : String(reason)));
+  writeErrorResult("unhandledRejection: " + (reason && reason.message ? reason.message : String(reason)));
+  process.exit(1);
+});
 syslog("evaluator process started — changeId: ${changeId}");
 import("${useModule}")
   .then(m => {
@@ -254,17 +287,7 @@ import("${useModule}")
   })
   .catch(err => {
     syslog("evaluator crashed — " + (err.message || String(err)));
-    const logPath = path.join(${JSON.stringify(projectRoot)}, ".indusk", "eval", "results.log");
-    fs.mkdirSync(path.dirname(logPath), { recursive: true });
-    const entry = JSON.stringify({
-      version: 1,
-      timestamp: new Date().toISOString(),
-      mode: "eval",
-      changeId: ${JSON.stringify(changeId)},
-      error: true,
-      message: err.message || String(err),
-    });
-    fs.appendFileSync(logPath, entry + "\\n", "utf8");
+    writeErrorResult(err.message || String(err));
     process.exit(1);
   });
 `;

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@infinitedusky/indusk-mcp",
-	"version": "1.19.0",
+	"version": "1.20.0",
 	"description": "InDusk development system — skills, MCP tools, and CLI for structured AI-assisted development",
 	"type": "module",
 	"files": [

package/skills/planner.md

@@ -202,6 +202,12 @@ status: proposed | accepted | deprecated | superseded | abandoned
 
 # {Title}
 
+## Goal
+
+**{One sentence. The headline outcome, in plain language. What will be true when this ADR's decisions ship that isn't true today.}**
+
+{One short paragraph — 2-4 sentences — grounding the goal in concrete user-visible terms. Name at least one specific current failure this fixes, so a reader arriving cold can tell what problem the rest of the ADR is solving. The Y-statement below formalizes the decision; this section lets a reader skim the headline without hunting through seven clauses first.}
+
 ## Y-Statement
 
 **In the context of:**