@gobing-ai/spur 0.4.14 → 0.4.15

package/.claude-plugin/marketplace.json

@@ -7,7 +7,7 @@
     "plugins": [
         {
             "name": "sp",
-            "version": "0.4.14",
+            "version": "0.4.15",
             "source": "./plugins/sp"
         }
     ]

package/config/pipeline-budgets.json

@@ -28,26 +28,12 @@
             "source": "Post-merge count (0607 R2: doc-sync + learning-capture consolidated, 2 -> 1). Real-run reading (R5): median 253s, max-sane 2319s (n=44). Budget = 60 min ceiling.",
             "decision": null
         },
-        "pr-review": {
-            "modelQueries": 0,
-            "wallClockMs": null,
-            "tokenCostUsd": null,
-            "source": "Zero-query pipeline (the live definition declares no model queries); query budget trivially 0 — the gate handles an empty list (0607 R3). No real full run recorded; wall-clock unenforced.",
-            "decision": null
-        },
         "history-anatomy": {
             "modelQueries": 3,
             "wallClockMs": null,
             "tokenCostUsd": null,
             "source": "Coverage entry (0826 R2): the live definition's model-query count (extractResolvedWorkflowFacts, 2026-09-11). No real-run measurement yet: wall-clock and cost are unenforced until measured, never 0. 0920 recount 2026-09-23: 4 -> 3 — the resolve-scope model hop was removed; scope/window validation is deterministic in the `spur history anatomy paths` op (history-anatomy-cache before script-absorption PR-6).",
             "decision": null
-        },
-        "wayfinder-resolution": {
-            "modelQueries": 2,
-            "wallClockMs": null,
-            "tokenCostUsd": null,
-            "source": "Coverage entry (0826 R2): the live definition's model-query count (extractResolvedWorkflowFacts, 2026-09-11). No real-run measurement yet: wall-clock and cost are unenforced until measured, never 0.",
-            "decision": null
         }
     }
 }

package/config/workflow-candidates.json

@@ -68,6 +68,24 @@
         }
     ],
     "retirements": [
+        {
+            "name": "feature-verification",
+            "recordedBy": "1246/1252 (D66, ADR-142)",
+            "date": "2026-10-18",
+            "rationale": "The wrapper was retired by the D66 consolidation decision; feature-lifecycle's verifying entry calls spur feature verify directly. Existing run history is retained."
+        },
+        {
+            "name": "pr-review",
+            "recordedBy": "1251/1252 (D66, ADR-142)",
+            "date": "2026-10-18",
+            "rationale": "The sequencing workflow was retired by the D66 consolidation decision; the PR review skill calls the flat spur builder pr-* verbs directly. Existing run history is retained."
+        },
+        {
+            "name": "wayfinder-resolution",
+            "recordedBy": "1251/1252 (D66, ADR-142)",
+            "date": "2026-10-18",
+            "rationale": "The workflow was retired by the D66 consolidation decision; investigation remains a wayfinder skill procedure. Existing run history is retained."
+        },
         {
             "name": "planning-pipeline",
             "recordedBy": "0872 (ADR-072/075 composition-baseline pass 2dc86579a)",

package/config/workflows/feature-lifecycle.yaml

@@ -40,24 +40,17 @@ states:
     onEnter:
       # Caller wiring (task 0880, ADR-119): entering verification RUNS the
       # feature-scoped pass instead of only passively reading its status file.
-      # 0948 R1: pass BOTH vars the inner workflow needs. `--vars` was documented as
-      # replace-not-merge and a partial override silently blanked `spurBin`; the engine
-      # merges declared defaults today (0948 R2), but the caller stays explicit so the
-      # nested run is self-sufficient under either semantic.
+      # 1246 R1: the pass is the verb itself — no nested `spur workflow run`, no
+      # recursion-marker clearing. Without `--run-id` the verb opens and finalizes
+      # its own run row bound to this definition (1246 R2), so the done gate's
+      # receipt-integrity rule (recording run terminal `done`) holds.
+      # `env -u __runId`: the verb's `--run-id` falls back to the `__runId` env name, and a hop
+      # driven from inside another workflow run (a pipeline step, the quality gate) inherits that
+      # run's id. Without the unset the pass silently binds its receipt to the OUTER run instead of
+      # opening its own — which is still `running` at verifying→done, so the done gate fails closed.
       - kind: shell
         options:
-          # Pass featureId and spurBin together. A partial --vars map used to be
-          # applied as a replacement, so the inner run lost spurBin and loadModule
-          # fell into source mode (0948 R1). The nested JSON is shell-expanded:
-          # $spurBin inside the double quotes keeps a multi-word invocation intact.
-          # 1147 R4: clear the inherited nested-run marker for this ONE child. The
-          # bounded, definition-pinned feature-verification pass (no agent.run, no
-          # further nested run) is a legitimate child, but the outer run's
-          # SPUR_WORKFLOW_RUN_ACTIVE marker rides every child env — so the CLI's
-          # nested-run refusal (0610 R4) rejected it and the same pass that passes
-          # standalone failed here with a bare exit 1. The recursion guard stays
-          # binding for every other nested invocation.
-          command: 'env -u SPUR_WORKFLOW_RUN_ACTIVE $spurBin workflow run feature-verification.yaml --vars "{\"featureId\":\"$featureId\",\"spurBin\":\"$spurBin\"}"'
+          command: 'env -u __runId $spurBin feature verify $featureId --no-logo'
   - id: blocked
     description: Impediment; work suspended.
   - id: done