claude-code-session-manager 0.42.0 → 0.43.0

package/dist/index.html

@@ -7,7 +7,7 @@
     <link rel="preconnect" href="https://fonts.googleapis.com">
     <link rel="preconnect" href="https://fonts.gstatic.com" crossorigin>
     <link href="https://fonts.googleapis.com/css2?family=Newsreader:ital,opsz,wght@0,6..72,400;0,6..72,500;0,6..72,600;0,6..72,700;1,6..72,400&family=Geist:wght@300;400;500;600;700&family=IBM+Plex+Mono:wght@400;500;600&display=swap" rel="stylesheet">
-    <script type="module" crossorigin src="./assets/index-D4ga1iZ5.js"></script>
+    <script type="module" crossorigin src="./assets/index-6w1gIeyB.js"></script>
     <link rel="modulepreload" crossorigin href="./assets/monaco-editor-BW5C4Iv1.js">
     <link rel="stylesheet" crossorigin href="./assets/monaco-editor-BTnBOi8r.css">
     <link rel="stylesheet" crossorigin href="./assets/index-CxGY-V5G.css">

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-code-session-manager",
-  "version": "0.42.0",
+  "version": "0.43.0",
   "description": "Local cockpit for the Claude Code CLI — multi-tab terminal, full config surface, scheduler, voice dictation, and live observability.",
   "type": "module",
   "main": "src/main/index.cjs",

package/src/main/runVerify.cjs

@@ -377,16 +377,43 @@ function parsePrdBodyDepFragments(body) {
  */
 function checkDeps(queueEntry, allJobs, prdBody) {
   const bySlug = new Map(allJobs.map((j) => [j.slug, j]));
+  // A PRD's filename carries the allocator's `NN-` prefix, but `dependsOn:` is
+  // authored with the bare name (the author cannot know the number that will
+  // be issued). Resolve exact slug first, then bare-name — identical rule to
+  // schedulerBatch.pickForProject's rowsForDep, which MUST agree with this
+  // function (see the missing-row note below).
+  const rowsByBareSlug = new Map();
+  for (const j of allJobs) {
+    const bare = String(j.slug ?? '').replace(/^\d+-/, '');
+    if (!rowsByBareSlug.has(bare)) rowsByBareSlug.set(bare, []);
+    rowsByBareSlug.get(bare).push(j);
+  }
+  const rowsForDep = (slug) => {
+    const exact = bySlug.get(slug);
+    if (exact) return [exact];
+    return rowsByBareSlug.get(String(slug ?? '').replace(/^\d+-/, '')) ?? [];
+  };
 
   // 1. Queue-level dependsOn (explicit).
   const queueDeps = Array.isArray(queueEntry.dependsOn) ? queueEntry.dependsOn : [];
   for (const depSlug of queueDeps) {
-    const dep = bySlug.get(depSlug);
-    if (!dep) {
-      return { ok: false, reason: `dependsOn: "${depSlug}" not found in queue` };
-    }
-    if (dep.status !== 'completed') {
-      return { ok: false, reason: `dependsOn: "${depSlug}" is ${dep.status} (need completed)` };
+    const deps = rowsForDep(depSlug);
+    // A dep with NO row is SATISFIED, not unmet. Completed rows are retired to
+    // history shards and their PRD is archived, so absence is the normal
+    // end-state of a finished dependency — exactly how pickForProject reads it.
+    //
+    // This used to return deps_unmet, which put the two halves of the
+    // scheduler in direct contradiction: the picker treated a missing dep as
+    // done and fired the job, then this check treated the same missing dep as
+    // unmet and downgraded the finished run back to pending — an infinite
+    // re-run, one full billed claude -p per cycle. It also silently dropped
+    // real work: PRDs 854 and 857 each ran to exit 0, were downgraded on a
+    // bare dep name that never matched any row, and ended up on disk with no
+    // queue row at all.
+    if (deps.length === 0) continue;
+    const unfinished = deps.find((d) => d.status !== 'completed');
+    if (unfinished) {
+      return { ok: false, reason: `dependsOn: "${depSlug}" is ${unfinished.status} (need completed)` };
     }
   }