@gtrabanco/pi-agentic-workflow 0.35.0 → 0.35.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gtrabanco/pi-agentic-workflow",
-  "version": "0.35.0",
+  "version": "0.35.1",
   "description": "Pi package: canonical agentic-workflow skills, friendly slash commands, and per-command model routing.",
   "keywords": [
     "pi-package",

package/skills/execute-phase/SKILL.md

@@ -1,7 +1,7 @@
 ---
 name: execute-phase
 user-invocable: true
-version: 6.2.2
+version: 6.3.1
 argument-hint: <NN-slug> [P<k>|step] [--max-attempts N] [--force]
 allowed-tools: [Bash, Read, Edit, Write, MultiEdit]
 author: "Gabriel Trabanco <1969593+gtrabanco@users.noreply.github.com>"
@@ -182,7 +182,13 @@ lane hand-off is a defect; re-read the hand-off before looping. After each
 step: gates, execute, diff guard (if implement), evidence/progress update,
 commit. Skip intermediate review checkpoints; the mandatory end review covers
 the full accumulated diff. On gate red at any step: repair that step (up to
-`--max-attempts`), then continue the loop. Terminal:
+`--max-attempts`), then continue the loop. The close-out is part of the last
+step's completion: open the PR on `main` with `Closes #N`, write `- PR: <url>`
+into References, and settle the index row with `agentic-workflow roadmap
+<index> rowUpsert '<json>'` (`docs/features/ROADMAP.md`; a fix unit edits its
+`docs/fix/README.md` status cell — no `NN` column, no `rowUpsert`).
+`bun scripts/roadmap-settle.mjs` exits 0 (#316).
+Terminal:
 
 ```text
 UNIT LOOP — <unit> COMPLETE

package/skills/unit-lane/SKILL.md

@@ -1,7 +1,7 @@
 ---
 name: unit-lane
 user-invocable: true
-version: 2.4.3
+version: 2.5.1
 argument-hint: <NN-slug | "<idea>"> [--retriage] | --fix <issue-number> | --from-issue <issue-number>
 description: >
   One-shot lane conductor for a delivery unit: triage, implement, evidence, and

package/skills/unit-lane/references/PLAN.md

@@ -100,6 +100,11 @@ P1 — <task description> (validator: <backticked command> <expected outcome>)
 P2 — <task description> (validator: <backticked command> <expected outcome>)
 …
 Pn — last task is verification when the unit has behavior
+P<n+1> — release settle, when the unit opens a PR: after the PR exists, settle
+  the roadmap row via the crate's writer (`agentic-workflow roadmap rowUpsert`,
+  status `done · [#N](<pr-url>)`, every other cell preserved) — validator:
+  `bun scripts/roadmap-settle.mjs` exits 0 (#316: the settle is a task the
+  executor owns, never a missing courtesy)
 ```
 
 Both phase grammars are accepted (owner: `phase-contract`): these bullets, or
@@ -164,7 +169,8 @@ name those files instead.
 - [ ] Every AC maps to at least one task, and no task re-decides an AC
 - [ ] Tasks are ordered smallest first
 - [ ] Each task names its validator (a backticked command plus its expected outcome)
-- [ ] Final task is verification (proves the whole unit works)
+- [ ] Final task is verification (proves the whole unit works); a unit that
+      opens a PR ends with the release-settle task (#316)
 - [ ] No task reaches into a future-phase concern (docs, release, migrations)
 - [ ] Phases labeled P1, P2, … — `#308`: `S#` is the requirement id (a spec),
       `P#` is the phase label (a task); the two vocabularies never swap roles
@@ -190,7 +196,8 @@ name those files instead.
 - Do not decide at the ask stage, and do not re-ask an answered question
 - Do not bundle across phase boundaries — one phase = one commit
 - Do not plan work beyond the unit's acceptance criteria
-- Do not include docs/release tasks here — those are separate catalog steps
+- Do not include docs/release tasks here — those are separate catalog steps;
+  the release-settle task above is the sole exception (#316)
 - Do not invent tasks that the ACs do not demand
 - Do not turn the sweep into a gate, stage verdict or blocking check — it routes,
   it never blocks a miss, and it adds no authority

package/skills/unit-lane/references/RELEASE.md

@@ -30,7 +30,13 @@ Write into the unit doc's **References** section:
 - [ ] Branch follows the project's Workflow conventions
 - [ ] Commit messages follow conventional format (`type(scope): description`)
 - [ ] PR body includes the Evidence section as verification record
-- [ ] Roadmap row updated to `done` (or `merged` per conventions)
+- [ ] Index row settled via the crate's writer — feature: re-emit the row with
+      `agentic-workflow roadmap docs/features/ROADMAP.md rowUpsert
+      '{"nn":"<NN>","slug":"<slug>","status":"done · [#<pr>](<pr-url>)"}'`,
+      every other cell as it is; fix: the `docs/fix/README.md` status cell — that
+      index has no `NN` column, so `rowUpsert` (the roadmap's writer) does not
+      apply to it. `bun scripts/roadmap-settle.mjs` exits 0 afterwards (issue
+      #316: the settle is code, never prose)
 - [ ] `git status --porcelain` is empty
 
 ## Allowed