@gobing-ai/spur 0.3.33 → 0.3.35

package/config/config.example.yaml

@@ -74,7 +74,7 @@ agent:
     #   agent: pi
     #   tier: capable-1
 
-  # `default-by-phase` is a deprecated backward-compat shim (ADR-033). Prefer
+  # `default-by-phase` is a REMOVED (0452). Prefer
   # executor `tier` + stage-registry model_policy. When present, a matching
   # phase mapping is still authoritative (fail-fast; no fall back to default)
   # and shadows stage routing for that phase.

package/config/corpus-baseline.json

@@ -0,0 +1,61 @@
+{
+    "note": "Accepted `spur task check` / `spur feature check` errors. This is a POLICY file, not a status dump: it is an INPUT to `bun run corpus-check`, so it must be committed — an untracked baseline means the gate behaves differently on every clone and CI, and the first person to meet 8 unexplained failures disables it. Do not move it under `.spur/` (gitignored, ephemeral run artifacts). The gate fails on any error NOT listed here, and ALSO on any entry that no longer reproduces, so this cannot rot into a silent suppression list. Every entry needs a real diagnosis, a reason, and a date — 'pre-existing, not investigated' is not a reason.",
+    "entries": [
+        {
+            "kind": "task",
+            "id": "0368",
+            "code": "L3.required-section-placeholder",
+            "reason": "RATCHET DRIFT, not a bypass. Closed 2026-07-28; the rule that flags it landed 2026-08-01 (f373e90b), four days later. It was legally closed under the rules of its day and became non-compliant retroactively. Backfill Testing/Solution or exempt permanently when someone next touches it.",
+            "since": "2026-08-07"
+        },
+        {
+            "kind": "task",
+            "id": "0454",
+            "code": "L3.solution-file-line",
+            "reason": "GENUINE PRE-FIX BYPASS. Closed 2026-08-06, well after the rule existed (2026-07-24), via the `--no-lifecycle` hole closed on 2026-08-07. Its Solution needs one file:line citation. The only entry here representing real unpaid debt.",
+            "since": "2026-08-07"
+        },
+        {
+            "kind": "feature",
+            "id": "M",
+            "code": "L3.ac-bdd-error",
+            "reason": "CHECKER GAP, not corpus debt. M is a wayfinder MAP; its Acceptance Criteria section carries an explicit prose disclaimer ('This feature is a wayfinder MAP... It has no acceptance criteria — its target is the ## Goal'). The BDD validator requires a `Feature:` declaration and has no notion of the map feature type. Correct fix is to teach `feature check` about wayfinder maps, not to bolt fake Gherkin onto a map.",
+            "since": "2026-08-07"
+        },
+        {
+            "kind": "feature",
+            "id": "M",
+            "code": "L3.ac-bdd-invalid",
+            "reason": "Downstream of the M ac-bdd-error entry — same missing-Feature-declaration cause, reported twice by the validator.",
+            "since": "2026-08-07"
+        },
+        {
+            "kind": "feature",
+            "id": "F82",
+            "code": "L3.ac-bdd-error",
+            "reason": "CHECKER GAP, same class as M: wayfinder map with a deliberate no-AC prose disclaimer (plus a noted exception for two implementation-ready child tickets). Not a syntax error to fix.",
+            "since": "2026-08-07"
+        },
+        {
+            "kind": "feature",
+            "id": "F82",
+            "code": "L3.ac-bdd-invalid",
+            "reason": "Downstream of the F82 ac-bdd-error entry — same cause, reported twice.",
+            "since": "2026-08-07"
+        },
+        {
+            "kind": "feature",
+            "id": "F821",
+            "code": "L3.ac-bdd-error",
+            "reason": "LEGACY AC FORMAT, not a map. F821's Acceptance Criteria are written as plain `R1 @core — …` prose rows rather than Gherkin, an older convention predating the BDD validator. Converting them is a real (small) job with traceability risk: AC titles are identity keys, so renaming breaks child-task scenario references.",
+            "since": "2026-08-07"
+        },
+        {
+            "kind": "feature",
+            "id": "F821",
+            "code": "L3.ac-bdd-invalid",
+            "reason": "Downstream of the F821 ac-bdd-error entry — same cause, reported twice.",
+            "since": "2026-08-07"
+        }
+    ]
+}

package/config/templates/task/brainstorm.md

@@ -10,6 +10,7 @@ parent_wbs: null
 priority: P2
 tags: []
 dependencies: []
+ac_numbering: task-local
 created_at: "{{ CREATED_AT }}"
 updated_at: "{{ CREATED_AT }}"
 ---

package/config/templates/task/feature-impl.md

@@ -10,6 +10,7 @@ parent_wbs: null
 priority: P2
 tags: []
 dependencies: []
+ac_numbering: task-local
 created_at: "{{ CREATED_AT }}"
 updated_at: "{{ CREATED_AT }}"
 ---
@@ -30,7 +31,9 @@ updated_at: "{{ CREATED_AT }}"
 
 ### Q&A
 
-<!-- Clarifications and decisions made during refinement. Keep empty if none. -->
+<!-- CLOSED decisions from refinement: what was chosen and why, what was deferred and on what
+     condition. Not a parking lot for open questions — an unanswered question here means the task
+     is not ready to hand off. Keep empty if none. -->
 
 ### Design
 

package/config/templates/task/issue.md

@@ -10,6 +10,7 @@ parent_wbs: null
 priority: P2
 tags: ["bug"]
 dependencies: []
+ac_numbering: task-local
 created_at: "{{ CREATED_AT }}"
 updated_at: "{{ CREATED_AT }}"
 ---

package/config/templates/task/meta.md

@@ -10,6 +10,7 @@ parent_wbs: null
 priority: P2
 tags: ["meta"]
 dependencies: []
+ac_numbering: task-local
 created_at: "{{ CREATED_AT }}"
 updated_at: "{{ CREATED_AT }}"
 ---
@@ -30,7 +31,9 @@ updated_at: "{{ CREATED_AT }}"
 
 ### Q&A
 
-<!-- Clarifications and decisions made during refinement. Keep empty if none. -->
+<!-- CLOSED decisions from refinement: what was chosen and why, what was deferred and on what
+     condition. Not a parking lot for open questions — an unanswered question here means the task
+     is not ready to hand off. Keep empty if none. -->
 
 ### Design
 

package/config/templates/task/review.md

@@ -10,6 +10,7 @@ parent_wbs: null
 priority: P2
 tags: ["review"]
 dependencies: []
+ac_numbering: task-local
 created_at: "{{ CREATED_AT }}"
 updated_at: "{{ CREATED_AT }}"
 ---

package/config/templates/task/standard.md

@@ -10,6 +10,7 @@ parent_wbs: null
 priority: P2
 tags: []
 dependencies: []
+ac_numbering: task-local
 created_at: "{{ CREATED_AT }}"
 updated_at: "{{ CREATED_AT }}"
 ---
@@ -30,7 +31,9 @@ updated_at: "{{ CREATED_AT }}"
 
 ### Q&A
 
-<!-- Clarifications and decisions made during refinement. Keep empty if none. -->
+<!-- CLOSED decisions from refinement: what was chosen and why, what was deferred and on what
+     condition. Not a parking lot for open questions — an unanswered question here means the task
+     is not ready to hand off. Keep empty if none. -->
 
 ### Design
 

package/config/workflows/feature-lifecycle.yaml

@@ -74,6 +74,16 @@ transitions:
     guard:
       kind: always
 
+  # Reopen: done → active (design §7.5 — done is re-enterable; cancelled is not)
+  - from: done
+    to: active
+    description: >
+      Reopen completed feature for follow-up work (mandatory History entry).
+      Mirrors task-lifecycle done→wip. Prefer spur feature sync --force when
+      non-terminal tasks are already linked.
+    guard:
+      kind: always
+
   # Blocked ↔ active — bidirectional
   - from: active
     to: blocked

package/config/workflows/idea-pipeline.yaml

@@ -247,7 +247,7 @@ states:
       - kind: agent.run
         options:
           agent: ${vars.agent}
-          input: "Run sp:spec-decomposition for feature ${vars.featureId}. Read the brainstorm artifact, feature AC, and design doc. Produce a task-batch JSON array at .spur/run/${vars.__runId}-idea-task-batch.json, validated against apps/cli/schemas/task-batch.schema.json. Schema-permitted fields per entry: `name`, `background`, `requirements`, `feature_id`, `parent_wbs`, `priority`, `tags`, `template` — schema validation rejects anything else. Acceptance Criteria, Design, and Plan sections are filled in by the per-task refine step after batch-create, NOT at decompose time. Validate locally against the schema before emitting."
+          input: "Run sp:spec-decomposition for feature ${vars.featureId}. Read the brainstorm artifact, feature AC, and design doc. SIZING FIRST, before any JSON: apply the skill's `Default to NOT decomposing` rubric to the whole unit of work — if it scores 0-2 the correct output is a ONE-entry batch, not many. Scenario count is not task count: merge scenarios that one task delivers (same file surface, same subsystem, or unreadable apart in review), and list every scenario a task covers in its background. Merging never costs AC coverage — one task may carry several scenarios. Do not emit one entry per scenario or per requirement by reflex. Then produce a task-batch JSON array at .spur/run/${vars.__runId}-idea-task-batch.json, validated against apps/cli/schemas/task-batch.schema.json. Schema-permitted fields per entry: `name`, `background`, `requirements`, `feature_id`, `parent_wbs`, `priority`, `tags`, `template` — schema validation rejects anything else. Acceptance Criteria, Design, and Plan sections are filled in by the per-task refine step after batch-create, NOT at decompose time. Validate locally against the schema before emitting."
           expectFile: .spur/run/${vars.__runId}-idea-task-batch.json
           timeoutMs: ${vars.stepTimeoutMs}
 

package/config/workflows/task-pipeline.yaml

@@ -57,6 +57,10 @@ vars:
   # `--vars '{"agent":"claude"}'`. Pinned (not left to the AiRunner's <default>
   # selection) so a broken/misconfigured agent on the box can't silently capture the run.
   agent: "omp"
+  # Implement-only executor override (R1, task 0454). Defaults to same as agent.
+  # Override with `--vars '{"implementAgent":"omp-zai"}'` to pin implement to a
+  # specific executor while other hops keep the default agent.
+  implementAgent: "omp"
   # Step-level timeout for agentic hops (review / verify / test-fix) in ms.
   # Soft quality-gate shells are unbounded by this var (host shell only).
   # Raised 600s → 1800s (task 0398 R4 / H6 dogfood). Override:
@@ -86,6 +90,12 @@ vars:
   # project can point it at its own formatter; invoked best-effort (a missing or
   # failing formatter must never abort a run — the quality gate is the real gate).
   formatCmd: "bun run format"
+  # Max R-items in ## Requirements before size precheck fails (R2, task 0454).
+  # Override with `--vars '{"maxImplementReqs":"12"}'`.
+  maxImplementReqs: "5"
+  # Max checklist items under ## Plan before size precheck fails (R2, task 0454).
+  # Override with `--vars '{"maxImplementPlanItems":"15"}'`.
+  maxImplementPlanItems: "8"
 
 states:
   - id: precheck
@@ -113,6 +123,32 @@ states:
           # hook at run start — see the 0062 follow-up note; there is no link-writing
           # CLI verb to call from a shell step, so it cannot live in pure YAML.
           message: "Pipeline start for task ${vars.wbs}."
+      # R1 (0453): auto-profile precheck reopens a done feature before task check.
+      # Under profile=auto, resolve feature_id, sync (preferred) or update to active.
+      # Under non-auto, leave R4 message to guide the operator.
+      - kind: shell
+        options:
+          command: >-
+            if [ "$profile" = auto ]; then
+              FID=$($spurBin task show $wbs --json 2>/dev/null |
+              jq -r '.feature_id // .frontmatter.feature_id // empty' 2>/dev/null);
+              if [ -n "$FID" ]; then
+                $spurBin feature sync "$FID" --force --json 2>/dev/null ||
+                $spurBin feature update "$FID" active 2>/dev/null || true;
+              fi;
+            fi;
+            exit 0
+      # R2 (0454): task size precheck — evaluate R-item and Plan-item counts.
+      # Writes PASS/FAIL to .spur/run/<wbs>-precheck-size.status. Always exit 0
+      # (soft check, like doctor). The precheck→implement guard reads the file.
+      - kind: shell
+        options:
+          command: >-
+            SIZE_FILE=".spur/run/$wbs-precheck-size.status" &&
+            mkdir -p .spur/run &&
+            bun plugins/sp/scripts/task-size-precheck.ts "$wbs"
+            --max-reqs "$maxImplementReqs" --max-plan-items "$maxImplementPlanItems" &&
+            exit 0
 
   - id: implement
     description: >
@@ -130,7 +166,7 @@ states:
     onEnter:
       - kind: agent.run
         options:
-          agent: ${vars.agent}
+          agent: ${vars.implementAgent}
           # Pure slash command only (ADR-043). Anti-recursion / implement discipline
           # lives in /sp:dev-run --mode implement → sp:code-implementation, not YAML prose.
           input: /sp:dev-run --mode implement ${vars.wbs} --auto
@@ -181,8 +217,6 @@ states:
             ATTEMPT_FILE=".spur/run/$wbs-test-fix-attempt" &&
             echo 0 > "$ATTEMPT_FILE" &&
             set +e &&
-            # sh -c re-parses multi-command gates (&&, ;); do NOT use bare `$qualityGateCmd`
-            # word-split (breaks on &&). qualityGateCmd is pipeline-owned trusted config only.
             ( sh -c "$qualityGateCmd" ); gate_rc=$?; set -e &&
             if [ "$gate_rc" -eq 0 ]; then
               printf 'PASS\n' > "$STATUS_FILE";
@@ -223,7 +257,6 @@ states:
             mkdir -p .spur/run &&
             STATUS_FILE=".spur/run/$wbs-test-gate.status" &&
             set +e &&
-            # See test probe: sh -c for multi-command; qualityGateCmd is trusted config only.
             ( sh -c "$qualityGateCmd" ); gate_rc=$?; set -e &&
             if [ "$gate_rc" -eq 0 ]; then
               printf 'PASS\n' > "$STATUS_FILE";
@@ -340,7 +373,7 @@ transitions:
     guard:
       kind: shell
       options:
-        command: 'test "$(cat .spur/run/$wbs-precheck-doctor.status 2>/dev/null)" = PASS && $spurBin task check $wbs'
+        command: 'test "$(cat .spur/run/$wbs-precheck-doctor.status 2>/dev/null)" = PASS && test "$(cat .spur/run/$wbs-precheck-size.status 2>/dev/null)" = PASS && $spurBin task check $wbs'
   - from: precheck
     to: failed
     description: Doctor FAIL and/or task check failed — stop before implement.

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@gobing-ai/spur",
-    "version": "0.3.33",
+    "version": "0.3.35",
     "description": "Spur CLI — local-first harness for mainstream coding agents: constraint checking, workflow orchestration, agent health, and history analytics. Bun-native; exposes the `spur` command.",
     "keywords": [
         "spur",
@@ -50,14 +50,14 @@
     },
     "devDependencies": {
         "@commander-js/extra-typings": "^14.0.0",
-        "@gobing-ai/ts-db": "^0.4.19",
-        "@gobing-ai/ts-ai-runner": "^0.4.19",
-        "@gobing-ai/ts-dual-workflow-engine": "^0.4.19",
-        "@gobing-ai/ts-infra": "^0.4.19",
-        "@gobing-ai/ts-llm-jsonl-importer": "^0.4.19",
-        "@gobing-ai/ts-rule-engine": "^0.4.19",
-        "@gobing-ai/ts-runtime": "^0.4.19",
-        "@gobing-ai/ts-utils": "^0.4.19",
+        "@gobing-ai/ts-db": "^0.4.22",
+        "@gobing-ai/ts-ai-runner": "^0.4.22",
+        "@gobing-ai/ts-dual-workflow-engine": "^0.4.22",
+        "@gobing-ai/ts-infra": "^0.4.22",
+        "@gobing-ai/ts-llm-jsonl-importer": "^0.4.22",
+        "@gobing-ai/ts-rule-engine": "^0.4.22",
+        "@gobing-ai/ts-runtime": "^0.4.22",
+        "@gobing-ai/ts-utils": "^0.4.22",
         "@types/bun": "1.3.14",
         "@types/figlet": "^1.7.0",
         "@types/node-notifier": "8.0.5",

package/schemas/spur-config.schema.json

@@ -2,7 +2,7 @@
     "$schema": "https://json-schema.org/draft/2020-12/schema",
     "$id": "https://github.com/gobing-ai/spur/schemas/spur-config.schema.json",
     "title": "Spur Project Configuration",
-    "description": "Schema for .spur/config.yaml — the single project configuration surface consumed by spur and (future) spur-server.",
+    "description": "Schema for .spur/config.yaml \u2014 the single project configuration surface consumed by spur and (future) spur-server.",
     "type": "object",
     "required": ["version", "name"],
     "properties": {
@@ -107,7 +107,7 @@
                 },
                 "executors": {
                     "type": "array",
-                    "description": "Named executor profiles: each pairs a canonical agent with an optional model override. Referenced by name from `default` and `default-by-phase`.",
+                    "description": "Named executor profiles: each pairs a canonical agent with an optional model override. Referenced by name from `agent.default` and stage model_policy (default-by-phase removed 0452).",
                     "items": {
                         "type": "object",
                         "required": ["name", "agent"],
@@ -130,18 +130,11 @@
                             "tier": {
                                 "type": "string",
                                 "enum": ["cheap", "standard", "capable-1", "capable-2", "capable-3"],
-                                "description": "Capability tier for stage-registry adaptive model routing (ADR-033, 0343). Live values: cheap | standard | capable-1 | capable-2 | capable-3 (1=low output quality, 3=high within the capable band). A stage starts on the cheapest eligible executor meeting its model_policy min_tier. Bare legacy `capable` is accepted at runtime (zod preprocess → capable-1) during the deprecation window but is not part of this editor enum."
+                                "description": "Capability tier for stage-registry adaptive model routing (ADR-033, 0343). Live values: cheap | standard | capable-1 | capable-2 | capable-3 (1=low output quality, 3=high within the capable band). A stage starts on the cheapest eligible executor meeting its model_policy min_tier. Bare legacy `capable` is accepted at runtime (zod preprocess \u2192 capable-1) during the deprecation window but is not part of this editor enum."
                             }
                         }
                     }
                 },
-                "default-by-phase": {
-                    "type": "object",
-                    "description": "DEPRECATED (ADR-033). Backward-compat map of dev phase (e.g. dev-run, dev-review) to executor selector. Prefer executor `tier` + stage-registry model_policy. When present, a matching phase mapping is still authoritative (fail-fast; no fall back to `default`) and shadows stage routing for that phase.",
-                    "additionalProperties": {
-                        "type": "string"
-                    }
-                },
                 "team": {
                     "type": "object",
                     "description": "Declarative agent teams keyed by team id (the map key is the teamId). Each team materializes a roster of agent specs (feature M). Mirrors @gobing-ai/spur-config TeamConfigSchema (Zod is SSOT).",
@@ -169,7 +162,10 @@
                                 "description": "Roster of member references (>= 1). A bare string is shorthand for { executor: <string> }; the composed agent id is <teamId>-<id ?? executor>.",
                                 "items": {
                                     "oneOf": [
-                                        { "type": "string", "minLength": 1 },
+                                        {
+                                            "type": "string",
+                                            "minLength": 1
+                                        },
                                         {
                                             "type": "object",
                                             "required": ["executor"],
@@ -177,27 +173,41 @@
                                                 "executor": {
                                                     "type": "string",
                                                     "minLength": 1,
-                                                    "description": "Executor name — resolved against agent.executors first, else a raw canonical agent type."
+                                                    "description": "Executor name \u2014 resolved against agent.executors first, else a raw canonical agent type."
                                                 },
                                                 "id": {
                                                     "type": "string",
                                                     "minLength": 1,
                                                     "description": "Local member id; composed id is <teamId>-<id ?? executor> (must match ^[a-z][a-z0-9_-]{1,63}$)."
                                                 },
-                                                "purpose": { "type": "string" },
+                                                "purpose": {
+                                                    "type": "string"
+                                                },
                                                 "workspace": {
                                                     "type": "string",
                                                     "minLength": 1,
                                                     "description": "Per-member workspace override. A leading `~` is tilde-expanded at load."
                                                 },
-                                                "model": { "type": "string", "minLength": 1 },
-                                                "autonomy": { "type": "string" },
-                                                "systemPrompt": { "type": "string" },
+                                                "model": {
+                                                    "type": "string",
+                                                    "minLength": 1
+                                                },
+                                                "autonomy": {
+                                                    "type": "string"
+                                                },
+                                                "systemPrompt": {
+                                                    "type": "string"
+                                                },
                                                 "command": {
                                                     "type": "array",
-                                                    "items": { "type": "string", "minLength": 1 }
+                                                    "items": {
+                                                        "type": "string",
+                                                        "minLength": 1
+                                                    }
                                                 },
-                                                "autostart": { "type": "boolean" }
+                                                "autostart": {
+                                                    "type": "boolean"
+                                                }
                                             }
                                         }
                                     ]
@@ -213,7 +223,9 @@
             "properties": {
                 "paths": {
                     "type": "array",
-                    "items": { "type": "string" },
+                    "items": {
+                        "type": "string"
+                    },
                     "description": "Glob paths to rule files."
                 }
             }
@@ -223,7 +235,9 @@
             "properties": {
                 "paths": {
                     "type": "array",
-                    "items": { "type": "string" },
+                    "items": {
+                        "type": "string"
+                    },
                     "description": "Glob paths to workflow directories."
                 }
             }
@@ -239,11 +253,11 @@
         },
         "tasks": {
             "type": "object",
-            "description": "Task-folder registration (design §9). Absorbs the legacy docs/.tasks/config.json folders + baseCounter concepts. Mirrors @gobing-ai/spur-config tasksConfigSchema (Zod is SSOT).",
+            "description": "Task-folder registration (design \u00a79). Absorbs the legacy docs/.tasks/config.json folders + baseCounter concepts. Mirrors @gobing-ai/spur-config tasksConfigSchema (Zod is SSOT).",
             "properties": {
                 "folders": {
                     "type": "object",
-                    "description": "Map of task-folder path → folder config.",
+                    "description": "Map of task-folder path \u2192 folder config.",
                     "additionalProperties": {
                         "type": "object",
                         "properties": {
@@ -275,7 +289,7 @@
         },
         "features": {
             "type": "object",
-            "description": "Feature directory location (design §9). Mirrors @gobing-ai/spur-config featuresConfigSchema (Zod is SSOT).",
+            "description": "Feature directory location (design \u00a79). Mirrors @gobing-ai/spur-config featuresConfigSchema (Zod is SSOT).",
             "properties": {
                 "dir": {
                     "type": "string",