axstack 0.47.0 → 0.48.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "axstack",
-  "version": "0.47.0",
+  "version": "0.48.0",
   "description": "Axstack installer and setup CLI: installs owned chat skills and role data, configures supported harness settings, and checks T3 Code capabilities.",
   "keywords": [
     "claude-code",

package/skills/axstack/references/run-record.md

@@ -81,6 +81,29 @@ owner stops.
 deleted.
 The driver records `paused` on a user request or a hold; idle alone is neither.
 
+## Review-round budgets
+
+Each PR membership entry carries `rounds: <n>` and `step-back: unused | used r<n>`.
+The field `rounds` counts failing review rounds, not `repairs`.
+Ticketless PRs (small-change intents) and adopted PRs have entries too.
+A new planned PR, small-change PR or adopted PR starts at `rounds: 0` and `step-back: unused`
+when it enters the run, unless a prior run entry exists.
+When a prior run entry exists for the PR, copy that entry, including adopted PRs.
+A prior record predates the field only when its `Runtime` line names an Axstack version below 0.48.0.
+Such a pre-field entry starts with `rounds: 0` and `step-back: unused` only if the field is absent.
+Except for that absent-field initialization, an inherited entry with missing or unclear `rounds` or `step-back` values
+counts as `step-back: used`, including a newer record missing the field or an unknown `Runtime` version.
+Distinct PRs that share one ticket or issue keep separate budgets.
+Write `step-back: used r<n>` before dispatching any repair from the step-back,
+including a user-picked path from the advised step-back card.
+A replacement or split PR copies its predecessor's entry.
+The entry persists across rebases, resumes and owner transfer.
+A `step-back: used` entry permits no further extension.
+Any counted failing review round after the extension records `held` with an advised card,
+including a re-review after a base update and a round after a passing round 4.
+Follow [Advised holds](contracts.md#advised-holds) for that card.
+An interrupted or failed extra-round attempt never earns a second extension and follows the existing failure rules.
+
 ## Handoff and resumption
 
 Keep handoff state in this same record, never a second wrapper record. Before an
@@ -190,7 +213,7 @@ Deploying bases: <base -> integration | deploying; docs/workflow evidence>
 PR digest watermarks: <repo -> absolute path of its per-repository JSON file inside the private run directory> | none
 Release: <AGENTS.md file:line + tag-triggered workflow path + named install hosts> | not applicable (<reason>)
 Source base: <exact revision or source identity>
-PR membership: <PR -> ordinary-chain | legacy-native-stack; direct dependencies incl unpublished; active authors; head/base branch/base SHA>
+PR membership: <PR -> ordinary-chain | legacy-native-stack; direct dependencies incl unpublished; active authors; head/base branch/base SHA; rounds: <n>; step-back: unused | used r<n>>
 IDs: <projectId + driver threadId/runId + dispatch identity receipt pointers>
 Runtime: <host + T3 version + installed Axstack SHA + installed Axstack version/check time + last reference read time + capabilities JSON path>
 Schedules: <scheduledTaskIds of every watch and manager schedule>

package/skills/axstack-implement/SKILL.md

@@ -250,9 +250,8 @@ For each PR:
    polish.
    `REQUEST_CHANGES`, a failed required check, or post-readiness feedback returns
    findings to the same author for a new revision, increments `repairs`, and
-   returns to step 1. `INCOMPLETE`, a provenance gap, unavailable model, serious
-   risk, or the third review round with `REQUEST_CHANGES` and/or diligence
-   `FINDINGS` on one PR records `held`. A changed parent immediately invalidates affected child readiness; refresh
+   returns to step 1. `INCOMPLETE`, a provenance gap, unavailable model or serious
+   risk records `held`. A changed parent immediately invalidates affected child readiness; refresh
    the next ready children as needed under PR chains before step 1.
    Follow [Advised holds](../axstack/references/contracts.md#advised-holds) for eligible `held` causes.
    Merge-ready also requires a current diligence `PASS` at that head; diligence
@@ -263,7 +262,15 @@ For each PR:
    rule as a finding on the owning PR and hold merge-ready until the contradiction is resolved.
    An unrelated run-record `Learning` leaves merge-ready eligibility unchanged.
    A round with reviewer `REQUEST_CHANGES` and/or diligence `FINDINGS` increments
-   `repairs` once and counts once toward the third-round hold.
+   `repairs` once and counts once toward the round-3 step-back.
+   Failed checks and post-readiness feedback increment `repairs` but never count
+   as failing review rounds.
+   On any counted failing review round that leaves `rounds` at 3 or more while
+   `step-back: unused`, the driver steps back with both `axstack-advisor-astra`
+   and `axstack-advisor-opus` to assess exactly one extra repair round (round 4
+   in the normal case).
+   Load and follow [Round-3 step-back](references/step-back.md).
+   Load and follow [Review-round budgets](../axstack/references/run-record.md#review-round-budgets) before dispatching any repair from the step-back.
 
 The T3 run watch reconciles every unsettled dispatch attempt; the bounded
 forge check wait is the only other implementation wait.

package/skills/axstack-implement/references/step-back.md

@@ -0,0 +1,31 @@
+# Round-3 step-back
+
+An unavailable step-back seat counts as no agreement and falls to the user with the advised step-back card.
+Follow [Advised holds](../../axstack/references/contracts.md#advised-holds) for
+seat attempts, waits, gaps and cards, including the round-4 card's delta-only adviser reuse.
+
+Each adviser independently gets the open findings and the author's replies.
+Each adviser independently gets the per-round deltas.
+Each adviser independently gets blame showing the round in which each flagged line first appeared.
+Each adviser independently gets the approved scope, sketch and approach.
+
+Send one advised repair brief to the same author for the extra round only when all conditions below hold:
+
+- The driver accepts the path.
+- Both advisers agree on the path.
+- The path stays inside the approved scope.
+  The path must not break a sketch `Binding` line.
+- No independent hold applies.
+
+The extra round requires fresh independent reviewer and diligence receipts, with a fresh session of the same mapped reviewer role.
+Do not add an extra reviewer.
+Advisers must never lower a rating or clear a finding.
+Advisers can propose `Pre-existing` with evidence, which the extra-round reviewer validates under
+[review's Pre-existing rules](../../axstack-review/SKILL.md#finding-severity).
+
+When any condition fails, record `held` and fall to the user with the advised step-back card.
+Failed conditions include an adviser split, an unavailable seat, the driver rejecting the agreed path,
+or the only path leaving approved scope or breaking a `Binding` line.
+A scope change follows the existing material-revision route.
+An independent hold keeps its own card and rules, with received step-back advice attached as context.
+After an independent hold clears, recheck all extra-round conditions and reuse unchanged advice.