muse-crew 0.6.8 → 0.7.0

package/lib/crew-api.js

@@ -529,6 +529,12 @@ commands["claim-task"] = (db, args) => {
   const task = requireTask(db, args.task_id);
   const project = requireProject(db, task.project);
   if (project.quiesced) throw conflict("This project is paused. Resume it before claiming tasks.");
+  // Mechanical guard: never claim a task that is already in a terminal state.
+  // The dispatcher may recommend a task based on stale state (e.g. a task that
+  // was marked done between the dispatcher's read and the workflow's claim).
+  // The claim must fail closed here, not launch a duplicate run.
+  if (task.state === "done") throw conflict("Task is already done and cannot be claimed.");
+  if (task.state === "cancelled") throw conflict("Task is cancelled and cannot be claimed.");
 
   const row = {
     id: uuid(), task_id: args.task_id, identity,

package/lib/test-version-write.sh

@@ -30,12 +30,29 @@ TMPBASE="$(mktemp -d)"
 trap 'rm -rf "$TMPBASE"' EXIT
 
 # --- Fixture: temp git repo seeded with the repo's current package.json ---
+# The test injects a \u2014 escape into the description field so the fixture
+# carries the escape this regression is about, regardless of whether the
+# repo's own package.json still has one. We write the JSON manually to ensure
+# the escape appears as literal backslash-u bytes, not the decoded character.
 FIX="$TMPBASE/fixture"
 mkdir -p "$FIX"
 git init -q -b main "$FIX" >/dev/null
 git -C "$FIX" config user.email "test@example.com"
 git -C "$FIX" config user.name "test"
-cp "$REPO_DIR/package.json" "$FIX/package.json"
+node -e '
+const fs = require("fs");
+const pkg = JSON.parse(fs.readFileSync(process.argv[1], "utf8"));
+pkg.version = pkg.version; // keep version from repo
+const lines = JSON.stringify(pkg, null, 2).split("\n");
+// Replace the description line with one carrying a literal \u2014 escape
+const out = lines.map(function (l) {
+  if (l.match(/"description":/)) {
+    return "  \"description\": \"Test description with em-dash escape \\u2014 here\",";
+  }
+  return l;
+});
+fs.writeFileSync(process.argv[2], out.join("\n") + "\n");
+' "$REPO_DIR/package.json" "$FIX/package.json"
 git -C "$FIX" add package.json
 git -C "$FIX" commit -qm "initial"
 

package/package.json

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

package/seed/cron-body-template.md

@@ -1,6 +1,6 @@
 ## Muse Crew Dispatcher
 
-You are the dispatch trigger for Muse Crew. Run the authoritative dispatcher workflow and report its claims. The main chat agent handles the actual task workflow launches.
+You are the dispatch trigger for Muse Crew. Run the authoritative dispatcher workflow and launch its claims autonomously.
 
 ### Steps
 
@@ -12,11 +12,10 @@ You are the dispatch trigger for Muse Crew. Run the authoritative dispatcher wor
 
    Wait for it to complete. It reads the crew's task state, determines eligibility, claims tasks, acknowledges the poll, and returns structured results.
 
-4. **Report claims:** Extract the `claims` array from the dispatcher result. For each claim, emit one line:
-   LAUNCH: script={claim.scriptPath} args={JSON.stringify(claim.args)}
+4. **Launch claims autonomously:** Extract the `claims` array from the dispatcher result. For each claim (up to 3 per tick — if more than 3, launch the first 3 and log the rest as deferred):
+   - Call workflow_launch_async with scriptPath={claim.scriptPath} and args={claim.args}.
+   - The launched workflow self-claims the task as its first action. If the task was already claimed or is done, the claim fails closed and the run stands down quietly — this is the mechanical duplicate protection, not an error.
 
    If the dispatcher returned no claims or the claims array is empty, report: NO_DISPATCH
 
-   Do NOT call workflow_launch_async for any task workflow. The main chat agent does that.
-
 5. Exit silently.

package/workflows/crew-init.js

@@ -1,9 +1,10 @@
 export const meta = {
   name: "crew-init",
-  description: "Initialize Muse Crew: bootstrap release, scaffold orchestration, set up cron jobs. Idempotent — safe to re-run.",
+  description: "Initialize Muse Crew: bootstrap release, scaffold orchestration, register dashboard project, set up cron jobs. Idempotent — safe to re-run.",
   phases: [
     { name: "release", title: "Bootstrap release system" },
     { name: "scaffold", title: "Scaffold orchestration directory" },
+    { name: "project", title: "Register dashboard project" },
     { name: "crons", title: "Create and converge cron jobs" }
   ]
 };
@@ -52,14 +53,17 @@ function gate0Decide(facts) {
 var gateFacts;
 try {
   gateFacts = await agent(
-    "Report raw facts about the requested crew home. Do NOT judge it — just report.\\n\\n" +
+    "Ensure the requested crew home exists as a git repository. Report raw facts.\\n\\n" +
     "Requested crewHome: " + crewHome + "\\n\\n" +
     "Steps (run in shell):\\n" +
     "1. home=$(echo $HOME)\\n" +
     "2. Expand a leading ~/ in the requested crewHome against $HOME (leave other paths untouched).\\n" +
-    "3. Run: git -C <expanded crewHome> rev-parse --git-dir; echo EXIT:$?\\n" +
-    "4. Return JSON { home, crewHomeExpanded, gitExitCode }.\\n\\n" +
-    "Do not create anything. This is a read-only check.",
+    "3. If the expanded path does not exist: mkdir -p <expanded crewHome>\\n" +
+    "4. Run: git -C <expanded crewHome> rev-parse --git-dir; echo EXIT:$?\\n" +
+    "5. If EXIT is not 0: run git -C <expanded crewHome> init -b main (or git init if -b not supported)\\n" +
+    "6. Re-run: git -C <expanded crewHome> rev-parse --git-dir; echo EXIT:$?\\n" +
+    "7. Return JSON { home, crewHomeExpanded, gitExitCode } where gitExitCode is the FINAL exit code after init.\\n\\n" +
+    "This ensures the crew home is a valid git repository. Do not judge — just ensure and report.",
     {
       key: "gate-0",
       label: "Validate crew home",
@@ -91,8 +95,8 @@ if (!gateResult.repoValid) {
   return {
     __hatchWorkflowControl: "blocked",
     result: {
-      blocked_reason: "crewHome is not a git repository",
-      message: "crewHome (" + gateFacts.crewHomeExpanded + ") is not a valid git repository (git exit code " + gateFacts.gitExitCode + "). Initialize it with git init or pick an existing repo."
+      blocked_reason: "crewHome git init failed",
+      message: "crewHome (" + gateFacts.crewHomeExpanded + ") could not be initialized as a git repository (git exit code " + gateFacts.gitExitCode + "). Check permissions and try again."
     }
   };
 }
@@ -109,11 +113,11 @@ try {
     "Release script (in repo): " + crewRepoPath + "/lib/crew-release.sh\n\n" +
     "Steps:\n" +
     "1. Run: test -f " + crewHome + "/crew-release.sh && echo EXISTS || echo MISSING\n" +
-    "2. If EXISTS, run: " + crewHome + "/crew-release.sh current\n" +
+    "2. If EXISTS, run: " + crewHome + "/crew-release.sh current " + crewHome + "\n" +
     "   Return { existed: true, hash: <current hash> }\n" +
     "3. If MISSING, bootstrap:\n" +
     "   a. Run: bash " + crewRepoPath + "/lib/crew-release.sh init " + crewHome + " " + crewRepoPath + "\n" +
-    "   b. Verify: " + crewHome + "/crew-release.sh current\n" +
+    "   b. Verify: " + crewHome + "/crew-release.sh current " + crewHome + "\n" +
     "   c. Return { existed: false, hash: <hash from current> }\n\n" +
     "Return JSON with existed (boolean) and hash (string).",
     {
@@ -179,7 +183,48 @@ var scaffoldCreated = scaffoldResult.created ? scaffoldResult.created.length : 0
 var scaffoldSkipped = scaffoldResult.skipped ? scaffoldResult.skipped.length : 0;
 log("Scaffold: " + scaffoldCreated + " created, " + scaffoldSkipped + " skipped");
 
-// ── Phase 3: Cron jobs (declarative manifest) ──────────────────────────
+// ── Phase 3: Project registration ─────────────────────────────────────
+// Registers the dashboard as the crew's first project in crew-state.db via
+// the Crew API. The dashboard becomes its own first project, so the crew can
+// work on the dashboard itself. Idempotent: if the project already exists,
+// it is left alone.
+phase("project");
+var projectResult;
+var safeDashboardName = dashboardName.split('"').join('\\"');
+try {
+  projectResult = await agent(
+    "Register the dashboard as a project in the crew's task database.\n\n" +
+    "Crew home: " + crewHome + "\n" +
+    "Crew API: " + crewHome + "/current/lib/crew-api.js\n" +
+    "Dashboard slug (project id): " + dashboardSlug + "\n" +
+    "Dashboard name (display_name): " + dashboardName + "\n\n" +
+    "Steps:\n" +
+    "1. Check if the project already exists:\n" +
+    "   node " + crewHome + "/current/lib/crew-api.js --crew-home " + crewHome + " get-project --json '{\"id\": \"" + dashboardSlug + "\"}'\n" +
+    "   If it returns a project (ok: true), return { action: \"exists\", project_id: \"" + dashboardSlug + "\" }.\n" +
+    "2. If not found (error), create it:\n" +
+    "   node " + crewHome + "/current/lib/crew-api.js --crew-home " + crewHome + " create-project --json '{\"id\": \"" + dashboardSlug + "\", \"display_name\": \"" + safeDashboardName + "\", \"repo_path\": \"" + crewHome + "\", \"deploy_type\": \"artifact\", \"deploy_slug\": \"" + dashboardSlug + "\", \"description\": \"The dashboard task service — the crew\\u0027s first project\"}'\n" +
+    "   Return { action: \"created\", project_id: \"" + dashboardSlug + "\" }.\n\n" +
+    "Return JSON with action (\"exists\" or \"created\") and project_id (string).",
+    {
+      key: "project-1",
+      label: "Register dashboard project",
+      schema: {
+        type: "object",
+        properties: {
+          action: { type: "string", enum: ["exists", "created"] },
+          project_id: { type: "string" }
+        },
+        required: ["action", "project_id"]
+      }
+    }
+  );
+} catch (e) {
+  return { __hatchWorkflowControl: "blocked", result: { blocked_reason: "Project registration failed", message: String(e.message || e) } };
+}
+log("Project: " + dashboardSlug + " " + projectResult.action);
+
+// ── Phase 4: Cron jobs (declarative manifest) ──────────────────────────
 // The manifest lives in the repo at seed/crons.json. Body templates live
 // in seed/ with {crewHome} placeholders. Missing jobs are
 // created from the manifest; existing jobs converge to it — except `enabled`,
@@ -199,7 +244,10 @@ try {
     "   or any entry lacks a required field (id, title, enabled, mode, schedule,\n" +
     "   owner, body_template).\n" +
     "2. For each manifest entry, in order:\n" +
-    "   a. The live id is the override for entry.id when present, else entry.id.\n" +
+    "   a. The live id is the override for entry.id when present (inputs.cronIds[entry.id]).\n" +
+    "      Otherwise, the live id is entry.id + '-' + dashboardSlug (instance-safe by\n" +
+    "      construction — two crew instances never share a cron id). The bare manifest\n" +
+    "      id is never used as a live id.\n" +
     "   b. Read the body template at " + crewRepoPath + "/seed/<entry.body_template>.\n" +
     "   c. Replace all occurrences of {crewHome} with: " + crewHome + ".\n" +
     "      Replace all occurrences of {dashboardSlug} with: " + dashboardSlug + ".\n" +
@@ -271,5 +319,6 @@ return {
   dashboardName: dashboardName,
   releaseHash: releaseResult.hash,
   scaffold: { created: scaffoldCreated, skipped: scaffoldSkipped },
+  project: projectResult.action,
   crons: cronsResult.summary.crons
 };