devrites 5.2.1 → 5.2.3

package/CHANGELOG.md

@@ -2,6 +2,18 @@
 
 All notable changes to DevRites are documented here. The format follows [Keep a Changelog](https://keepachangelog.com/en/1.1.0/) and DevRites adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html). Releases are generated automatically by [semantic-release](https://semantic-release.gitbook.io/) from Conventional Commits on `main`.
 
+## [5.2.3](https://github.com/ViktorsBaikers/DevRites/compare/v5.2.2...v5.2.3) (2026-09-12)
+
+### Fixed
+
+* **skills:** chain converge to vet instead of recommending it ([#64](https://github.com/ViktorsBaikers/DevRites/issues/64)) ([9c65e49](https://github.com/ViktorsBaikers/DevRites/commit/9c65e49ebfed3fb297ccef76da314a8728b0d0ba))
+
+## [5.2.2](https://github.com/ViktorsBaikers/DevRites/compare/v5.2.1...v5.2.2) (2026-09-11)
+
+### Fixed
+
+* **skills:** make self-chaining save the caller return cursor ([#63](https://github.com/ViktorsBaikers/DevRites/issues/63)) ([4038577](https://github.com/ViktorsBaikers/DevRites/commit/4038577d5d8d1bbd7fe6e05fb9a0375a3bc07f5e))
+
 ## [5.2.1](https://github.com/ViktorsBaikers/DevRites/compare/v5.2.0...v5.2.1) (2026-09-11)
 
 ### Fixed

package/README.md

@@ -33,7 +33,7 @@ project-conventional push, tag, or PR action, and archive the workspace.
 Unattended runs may create local WIP checkpoint commits along the way, but they
 remain local unless Ship's disclosed plan includes an approved remote action.
 
-**Status:** [`v5.2.1`](https://github.com/ViktorsBaikers/DevRites/releases/tag/v5.2.1): see [`CHANGELOG.md`](CHANGELOG.md) for release notes.
+**Status:** [`v5.2.3`](https://github.com/ViktorsBaikers/DevRites/releases/tag/v5.2.3): see [`CHANGELOG.md`](CHANGELOG.md) for release notes.
 
 This is the latest published release; `main` may contain unreleased work.
 

package/pack/.claude/skills/rite-converge/SKILL.md

@@ -104,9 +104,14 @@ Pull on demand:
    leave the plan/vet verdict untouched and set `Next step: /rite-prove`. Append
    `decisions.md` for any material call.
 7. **STOP.** Report units assessed, built / partial / absent counts, slices appended, and any
-   principle violation found; recommend `/rite-vet` when slices were appended (or
-   `/rite-prove` if the code already
-   converged).
+   principle violation found. A direct invocation that appended slices becomes
+   the caller for the mandated next step: save a return cursor
+   (`return_phase`/`return_next_action`) naming this Converge pass as the
+   caller, invoke `/rite-vet` inline, and consume its results — an agent-owned
+   `NEEDS REPLAN` routes to `/rite-plan` repair internally under the
+   fingerprint caps; its return lands here, never on the user. Stop only at
+   Vet's boundary (READY readback; next step `/rite-build` for the appended
+   slices). Nothing appended → recommend `/rite-prove`.
 
 ## Completion evidence (fail-closed)
 

package/pack/.claude/skills/rite-plan/SKILL.md

@@ -144,12 +144,15 @@ user-facing stop unless a genuine HITL, safety/access, or exhausted-recovery
 condition was recorded.
 
 When invoked directly with no controlling caller and a planning artifact
-changed, become the caller for the mandated next step: record the return
-cursor, invoke `/rite-vet` inline, and consume its results — `NEEDS REPLAN`
-re-enters repair internally under the fingerprint caps; `NEEDS CLARIFICATION`
-and human-owned gates still reach the user. Stop only at Vet's boundary (READY
-readback; next step `/rite-build`). An unchanged plan keeps its recorded next
-step.
+changed, become the caller for the mandated next step: save a return cursor
+(`return_phase`/`return_next_action`) naming this Plan pass as the caller —
+a direct Plan has no caller above it, so without the saved cursor Vet reads
+itself as top-level and its verdict becomes a user-facing command. Invoke
+`/rite-vet` inline and consume its results — `NEEDS REPLAN` re-enters repair
+internally under the fingerprint caps and is never emitted as a user-facing
+command; `NEEDS CLARIFICATION` and human-owned gates still reach the user.
+Stop only at Vet's boundary (READY readback; next step `/rite-build`). An
+unchanged plan keeps its recorded next step.
 
 > **Mid-flight discipline.** Do not change product behavior without confirmation or
 > absorb drift silently. See [`anti-patterns`](reference/anti-patterns.md).

package/pack/.claude/skills/rite-vet/SKILL.md

@@ -53,8 +53,10 @@ plan declares a root-authored executable workflow file, read
   asking and resolve reversible technical choices. Ask only human-owned choices.
 - Preserve a valid technical return cursor. Agent-owned `NEEDS REPLAN` returns
   internally to its caller, not to the human; on a direct invocation there is
-  no caller, so Vet becomes it — record the cursor and invoke `/rite-plan`
-  repair inline under the same fingerprint caps.
+  no caller, so Vet becomes it — save a return cursor
+  (`return_phase`/`return_next_action`) naming this Vet pass as the caller and
+  invoke `/rite-plan` repair inline under the same fingerprint caps. An
+  agent-owned verdict with budget remaining is never a user-facing command.
 - **Recovery recheck is bounded and batched.** A valid cursor plus open
   fingerprints enters Recovery recheck; it does not start another Full Vet or
   repeat unaffected axes/reviewers. One dispatch covers every open fingerprint

package/pack/generated/claude/skills/rite-converge/SKILL.md

@@ -104,9 +104,14 @@ Pull on demand:
    leave the plan/vet verdict untouched and set `Next step: /rite-prove`. Append
    `decisions.md` for any material call.
 7. **STOP.** Report units assessed, built / partial / absent counts, slices appended, and any
-   principle violation found; recommend `/rite-vet` when slices were appended (or
-   `/rite-prove` if the code already
-   converged).
+   principle violation found. A direct invocation that appended slices becomes
+   the caller for the mandated next step: save a return cursor
+   (`return_phase`/`return_next_action`) naming this Converge pass as the
+   caller, invoke `/rite-vet` inline, and consume its results — an agent-owned
+   `NEEDS REPLAN` routes to `/rite-plan` repair internally under the
+   fingerprint caps; its return lands here, never on the user. Stop only at
+   Vet's boundary (READY readback; next step `/rite-build` for the appended
+   slices). Nothing appended → recommend `/rite-prove`.
 
 ## Completion evidence (fail-closed)
 

package/pack/generated/claude/skills/rite-plan/SKILL.md

@@ -144,12 +144,15 @@ user-facing stop unless a genuine HITL, safety/access, or exhausted-recovery
 condition was recorded.
 
 When invoked directly with no controlling caller and a planning artifact
-changed, become the caller for the mandated next step: record the return
-cursor, invoke `/rite-vet` inline, and consume its results — `NEEDS REPLAN`
-re-enters repair internally under the fingerprint caps; `NEEDS CLARIFICATION`
-and human-owned gates still reach the user. Stop only at Vet's boundary (READY
-readback; next step `/rite-build`). An unchanged plan keeps its recorded next
-step.
+changed, become the caller for the mandated next step: save a return cursor
+(`return_phase`/`return_next_action`) naming this Plan pass as the caller —
+a direct Plan has no caller above it, so without the saved cursor Vet reads
+itself as top-level and its verdict becomes a user-facing command. Invoke
+`/rite-vet` inline and consume its results — `NEEDS REPLAN` re-enters repair
+internally under the fingerprint caps and is never emitted as a user-facing
+command; `NEEDS CLARIFICATION` and human-owned gates still reach the user.
+Stop only at Vet's boundary (READY readback; next step `/rite-build`). An
+unchanged plan keeps its recorded next step.
 
 > **Mid-flight discipline.** Do not change product behavior without confirmation or
 > absorb drift silently. See [`anti-patterns`](reference/anti-patterns.md).

package/pack/generated/claude/skills/rite-vet/SKILL.md

@@ -53,8 +53,10 @@ plan declares a root-authored executable workflow file, read
   asking and resolve reversible technical choices. Ask only human-owned choices.
 - Preserve a valid technical return cursor. Agent-owned `NEEDS REPLAN` returns
   internally to its caller, not to the human; on a direct invocation there is
-  no caller, so Vet becomes it — record the cursor and invoke `/rite-plan`
-  repair inline under the same fingerprint caps.
+  no caller, so Vet becomes it — save a return cursor
+  (`return_phase`/`return_next_action`) naming this Vet pass as the caller and
+  invoke `/rite-plan` repair inline under the same fingerprint caps. An
+  agent-owned verdict with budget remaining is never a user-facing command.
 - **Recovery recheck is bounded and batched.** A valid cursor plus open
   fingerprints enters Recovery recheck; it does not start another Full Vet or
   repeat unaffected axes/reviewers. One dispatch covers every open fingerprint

package/pack/generated/codex/skills/rite-converge/SKILL.md

@@ -104,9 +104,14 @@ Pull on demand:
    leave the plan/vet verdict untouched and set `Next step: $rite-prove`. Append
    `decisions.md` for any material call.
 7. **STOP.** Report units assessed, built / partial / absent counts, slices appended, and any
-   principle violation found; recommend `$rite-vet` when slices were appended (or
-   `$rite-prove` if the code already
-   converged).
+   principle violation found. A direct invocation that appended slices becomes
+   the caller for the mandated next step: save a return cursor
+   (`return_phase`/`return_next_action`) naming this Converge pass as the
+   caller, invoke `$rite-vet` inline, and consume its results — an agent-owned
+   `NEEDS REPLAN` routes to `$rite-plan` repair internally under the
+   fingerprint caps; its return lands here, never on the user. Stop only at
+   Vet's boundary (READY readback; next step `$rite-build` for the appended
+   slices). Nothing appended → recommend `$rite-prove`.
 
 ## Completion evidence (fail-closed)
 

package/pack/generated/codex/skills/rite-plan/SKILL.md

@@ -144,12 +144,15 @@ user-facing stop unless a genuine HITL, safety/access, or exhausted-recovery
 condition was recorded.
 
 When invoked directly with no controlling caller and a planning artifact
-changed, become the caller for the mandated next step: record the return
-cursor, invoke `$rite-vet` inline, and consume its results — `NEEDS REPLAN`
-re-enters repair internally under the fingerprint caps; `NEEDS CLARIFICATION`
-and human-owned gates still reach the user. Stop only at Vet's boundary (READY
-readback; next step `$rite-build`). An unchanged plan keeps its recorded next
-step.
+changed, become the caller for the mandated next step: save a return cursor
+(`return_phase`/`return_next_action`) naming this Plan pass as the caller —
+a direct Plan has no caller above it, so without the saved cursor Vet reads
+itself as top-level and its verdict becomes a user-facing command. Invoke
+`$rite-vet` inline and consume its results — `NEEDS REPLAN` re-enters repair
+internally under the fingerprint caps and is never emitted as a user-facing
+command; `NEEDS CLARIFICATION` and human-owned gates still reach the user.
+Stop only at Vet's boundary (READY readback; next step `$rite-build`). An
+unchanged plan keeps its recorded next step.
 
 > **Mid-flight discipline.** Do not change product behavior without confirmation or
 > absorb drift silently. See [`anti-patterns`](reference/anti-patterns.md).

package/pack/generated/codex/skills/rite-vet/SKILL.md

@@ -53,8 +53,10 @@ plan declares a root-authored executable workflow file, read
   asking and resolve reversible technical choices. Ask only human-owned choices.
 - Preserve a valid technical return cursor. Agent-owned `NEEDS REPLAN` returns
   internally to its caller, not to the human; on a direct invocation there is
-  no caller, so Vet becomes it — record the cursor and invoke `$rite-plan`
-  repair inline under the same fingerprint caps.
+  no caller, so Vet becomes it — save a return cursor
+  (`return_phase`/`return_next_action`) naming this Vet pass as the caller and
+  invoke `$rite-plan` repair inline under the same fingerprint caps. An
+  agent-owned verdict with budget remaining is never a user-facing command.
 - **Recovery recheck is bounded and batched.** A valid cursor plus open
   fingerprints enters Recovery recheck; it does not start another Full Vet or
   repeat unaffected axes/reviewers. One dispatch covers every open fingerprint

package/pack/generated/omp/skills/rite-converge/SKILL.md

@@ -104,9 +104,14 @@ Pull on demand:
    leave the plan/vet verdict untouched and set `Next step: /rite-prove`. Append
    `decisions.md` for any material call.
 7. **STOP.** Report units assessed, built / partial / absent counts, slices appended, and any
-   principle violation found; recommend `/rite-vet` when slices were appended (or
-   `/rite-prove` if the code already
-   converged).
+   principle violation found. A direct invocation that appended slices becomes
+   the caller for the mandated next step: save a return cursor
+   (`return_phase`/`return_next_action`) naming this Converge pass as the
+   caller, invoke `/rite-vet` inline, and consume its results — an agent-owned
+   `NEEDS REPLAN` routes to `/rite-plan` repair internally under the
+   fingerprint caps; its return lands here, never on the user. Stop only at
+   Vet's boundary (READY readback; next step `/rite-build` for the appended
+   slices). Nothing appended → recommend `/rite-prove`.
 
 ## Completion evidence (fail-closed)
 

package/pack/generated/omp/skills/rite-plan/SKILL.md

@@ -144,12 +144,15 @@ user-facing stop unless a genuine HITL, safety/access, or exhausted-recovery
 condition was recorded.
 
 When invoked directly with no controlling caller and a planning artifact
-changed, become the caller for the mandated next step: record the return
-cursor, invoke `/rite-vet` inline, and consume its results — `NEEDS REPLAN`
-re-enters repair internally under the fingerprint caps; `NEEDS CLARIFICATION`
-and human-owned gates still reach the user. Stop only at Vet's boundary (READY
-readback; next step `/rite-build`). An unchanged plan keeps its recorded next
-step.
+changed, become the caller for the mandated next step: save a return cursor
+(`return_phase`/`return_next_action`) naming this Plan pass as the caller —
+a direct Plan has no caller above it, so without the saved cursor Vet reads
+itself as top-level and its verdict becomes a user-facing command. Invoke
+`/rite-vet` inline and consume its results — `NEEDS REPLAN` re-enters repair
+internally under the fingerprint caps and is never emitted as a user-facing
+command; `NEEDS CLARIFICATION` and human-owned gates still reach the user.
+Stop only at Vet's boundary (READY readback; next step `/rite-build`). An
+unchanged plan keeps its recorded next step.
 
 > **Mid-flight discipline.** Do not change product behavior without confirmation or
 > absorb drift silently. See [`anti-patterns`](reference/anti-patterns.md).

package/pack/generated/omp/skills/rite-vet/SKILL.md

@@ -53,8 +53,10 @@ plan declares a root-authored executable workflow file, read
   asking and resolve reversible technical choices. Ask only human-owned choices.
 - Preserve a valid technical return cursor. Agent-owned `NEEDS REPLAN` returns
   internally to its caller, not to the human; on a direct invocation there is
-  no caller, so Vet becomes it — record the cursor and invoke `/rite-plan`
-  repair inline under the same fingerprint caps.
+  no caller, so Vet becomes it — save a return cursor
+  (`return_phase`/`return_next_action`) naming this Vet pass as the caller and
+  invoke `/rite-plan` repair inline under the same fingerprint caps. An
+  agent-owned verdict with budget remaining is never a user-facing command.
 - **Recovery recheck is bounded and batched.** A valid cursor plus open
   fingerprints enters Recovery recheck; it does not start another Full Vet or
   repeat unaffected axes/reviewers. One dispatch covers every open fingerprint

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "devrites",
-  "version": "5.2.1",
+  "version": "5.2.3",
   "description": "DevRites: a disciplined senior-engineer workflow pack for Claude Code and Codex",
   "license": "SEE LICENSE IN LICENSE",
   "homepage": "https://github.com/ViktorsBaikers/DevRites#readme",