muse-crew 0.6.1 → 0.6.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "muse-crew",
-  "version": "0.6.1",
+  "version": "0.6.2",
   "description": "Opinionated orchestration for Muse \u2014 workflows, identities, and tooling for autonomous software development.",
   "license": "UNLICENSED",
   "private": false,

package/workflows/bugfix.js

@@ -323,8 +323,14 @@ function buildVisualCapturePlan(taskTitle, taskDescription, kind, captureTargets
 // opt-in — a missing or garbled line degrades to "unknown", which callers
 // treat as non-experiential. The task is never parked on a garbled line.
 async function resolveExperiential() {
-  if (isExperiential === true) return "yes";
-  if (isExperiential === false) return "no";
+  // Triage notes are immutable within a run: cache the resolved outcome —
+  // yes, no, AND unknown — so the dashboard lookup runs at most once per
+  // run. Without the unknown branch a missing experiential marker would
+  // re-fire the same "resolve-experiential-<taskId>" agent key on every
+  // Capture/Map entry and throw a duplicate replay key error.
+  if (experientialResolved !== null) return experientialResolved;
+  if (isExperiential === true) return (experientialResolved = "yes");
+  if (isExperiential === false) return (experientialResolved = "no");
   var expCheck = null;
   try {
     expCheck = await agent(
@@ -340,12 +346,14 @@ async function resolveExperiential() {
     );
   } catch (e) {
     log("resolveExperiential: agent call failed (" + (e && e.message ? e.message : e) + ") — treating as unknown");
-    return "unknown";
+    experientialResolved = "unknown";
+    return experientialResolved;
   }
   var marker = extractExperiential(expCheck && expCheck.experiential_line ? expCheck.experiential_line : "");
-  if (marker === true) return "yes";
-  if (marker === false) return "no";
-  return "unknown";
+  if (marker === true) experientialResolved = "yes";
+  else if (marker === false) experientialResolved = "no";
+  else experientialResolved = "unknown";
+  return experientialResolved;
 }
 // Baseline evidence status: reads the task's note events for the exact
 // protocol prefixes (explicit state, never English matching). Returns
@@ -456,6 +464,9 @@ let mapperSpec = "";
 // plan, and the Map-gate bounce counter (keeps agent stable-keys unique when
 // a Map-gate bounce re-runs Capture/Map in the same run).
 let isExperiential = null;
+// Caches all three resolveExperiential() outcomes (yes/no/unknown); Triage
+// notes are immutable within a run, so the lookup runs at most once.
+let experientialResolved = null;
 let captureTargets = "";
 let mapGateBounceCount = 0;
 // Merge-time versioning: the release decision is extracted deterministically
@@ -544,7 +555,7 @@ while (i < STEPS.length) {
       "Find the task with id \"" + taskId + "\" in the returned tasks array.\n" +
       "Return exactly { \"project\": \"<the task's project field, or empty string if absent>\" } and nothing else.",
       {
-        key: "project-check-" + step.name + (totalReworkCount > 0 ? "-r" + totalReworkCount : ""),
+        key: "project-check-" + step.name + (totalReworkCount > 0 ? "-r" + totalReworkCount : "") + (mapGateBounceCount > 0 ? "-g" + mapGateBounceCount : ""),
         label: "Checking project before " + step.name,
         schema: { type: "object", properties: { project: { type: "string" } }, required: ["project"] }
       }

package/workflows/chore.js

@@ -321,8 +321,14 @@ function buildVisualCapturePlan(taskTitle, taskDescription, kind, captureTargets
 // opt-in — a missing or garbled line degrades to "unknown", which callers
 // treat as non-experiential. The task is never parked on a garbled line.
 async function resolveExperiential() {
-  if (isExperiential === true) return "yes";
-  if (isExperiential === false) return "no";
+  // Triage notes are immutable within a run: cache the resolved outcome —
+  // yes, no, AND unknown — so the dashboard lookup runs at most once per
+  // run. Without the unknown branch a missing experiential marker would
+  // re-fire the same "resolve-experiential-<taskId>" agent key on every
+  // Capture/Map entry and throw a duplicate replay key error.
+  if (experientialResolved !== null) return experientialResolved;
+  if (isExperiential === true) return (experientialResolved = "yes");
+  if (isExperiential === false) return (experientialResolved = "no");
   var expCheck = null;
   try {
     expCheck = await agent(
@@ -338,12 +344,14 @@ async function resolveExperiential() {
     );
   } catch (e) {
     log("resolveExperiential: agent call failed (" + (e && e.message ? e.message : e) + ") — treating as unknown");
-    return "unknown";
+    experientialResolved = "unknown";
+    return experientialResolved;
   }
   var marker = extractExperiential(expCheck && expCheck.experiential_line ? expCheck.experiential_line : "");
-  if (marker === true) return "yes";
-  if (marker === false) return "no";
-  return "unknown";
+  if (marker === true) experientialResolved = "yes";
+  else if (marker === false) experientialResolved = "no";
+  else experientialResolved = "unknown";
+  return experientialResolved;
 }
 // Baseline evidence status: reads the task's note events for the exact
 // protocol prefixes (explicit state, never English matching). Returns
@@ -414,6 +422,9 @@ let mapperSpec = "";
 // a Map-gate bounce re-runs Capture/Map in the same run). Chore has no QA
 // phase, so there is no visual-verdict function — Capture only.
 let isExperiential = null;
+// Caches all three resolveExperiential() outcomes (yes/no/unknown); Triage
+// notes are immutable within a run, so the lookup runs at most once.
+let experientialResolved = null;
 let captureTargets = "";
 let mapGateBounceCount = 0;
 // Merge-time versioning: the release decision is extracted deterministically
@@ -502,7 +513,7 @@ while (i < STEPS.length) {
       "Find the task with id \"" + taskId + "\" in the returned tasks array.\n" +
       "Return exactly { \"project\": \"<the task's project field, or empty string if absent>\" } and nothing else.",
       {
-        key: "project-check-" + step.name + (reworkCount > 0 ? "-r" + reworkCount : ""),
+        key: "project-check-" + step.name + (reworkCount > 0 ? "-r" + reworkCount : "") + (mapGateBounceCount > 0 ? "-g" + mapGateBounceCount : ""),
         label: "Checking project before " + step.name,
         schema: { type: "object", properties: { project: { type: "string" } }, required: ["project"] }
       }

package/workflows/standard.js

@@ -323,8 +323,14 @@ function buildVisualCapturePlan(taskTitle, taskDescription, kind, captureTargets
 // opt-in — a missing or garbled line degrades to "unknown", which callers
 // treat as non-experiential. The task is never parked on a garbled line.
 async function resolveExperiential() {
-  if (isExperiential === true) return "yes";
-  if (isExperiential === false) return "no";
+  // Triage notes are immutable within a run: cache the resolved outcome —
+  // yes, no, AND unknown — so the dashboard lookup runs at most once per
+  // run. Without the unknown branch a missing experiential marker would
+  // re-fire the same "resolve-experiential-<taskId>" agent key on every
+  // Capture/Map entry and throw a duplicate replay key error.
+  if (experientialResolved !== null) return experientialResolved;
+  if (isExperiential === true) return (experientialResolved = "yes");
+  if (isExperiential === false) return (experientialResolved = "no");
   var expCheck = null;
   try {
     expCheck = await agent(
@@ -340,12 +346,14 @@ async function resolveExperiential() {
     );
   } catch (e) {
     log("resolveExperiential: agent call failed (" + (e && e.message ? e.message : e) + ") — treating as unknown");
-    return "unknown";
+    experientialResolved = "unknown";
+    return experientialResolved;
   }
   var marker = extractExperiential(expCheck && expCheck.experiential_line ? expCheck.experiential_line : "");
-  if (marker === true) return "yes";
-  if (marker === false) return "no";
-  return "unknown";
+  if (marker === true) experientialResolved = "yes";
+  else if (marker === false) experientialResolved = "no";
+  else experientialResolved = "unknown";
+  return experientialResolved;
 }
 // Baseline evidence status: reads the task's note events for the exact
 // protocol prefixes (explicit state, never English matching). Returns
@@ -455,6 +463,9 @@ let mapperSpec = "";
 // plan, and the Map-gate bounce counter (keeps agent stable-keys unique when
 // a Map-gate bounce re-runs Capture/Map in the same run).
 let isExperiential = null;
+// Caches all three resolveExperiential() outcomes (yes/no/unknown); Triage
+// notes are immutable within a run, so the lookup runs at most once.
+let experientialResolved = null;
 let captureTargets = "";
 let mapGateBounceCount = 0;
 // Merge-time versioning: the release decision is extracted deterministically
@@ -543,7 +554,7 @@ while (i < STEPS.length) {
       "Find the task with id \"" + taskId + "\" in the returned tasks array.\n" +
       "Return exactly { \"project\": \"<the task's project field, or empty string if absent>\" } and nothing else.",
       {
-        key: "project-check-" + step.name + (totalReworkCount > 0 ? "-r" + totalReworkCount : ""),
+        key: "project-check-" + step.name + (totalReworkCount > 0 ? "-r" + totalReworkCount : "") + (mapGateBounceCount > 0 ? "-g" + mapGateBounceCount : ""),
         label: "Checking project before " + step.name,
         schema: { type: "object", properties: { project: { type: "string" } }, required: ["project"] }
       }