axstack 0.11.1 → 0.11.3
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
package/package.json
CHANGED
|
@@ -23,9 +23,9 @@ Pair A/B is retired for this contract; its artefacts remain untouched.
|
|
|
23
23
|
- `axstack-monitor` remains an optional read-only observer that never sends.
|
|
24
24
|
|
|
25
25
|
Self is resolved on every run with `gh api user --jq .login`; never hardcode
|
|
26
|
-
it. The review allowlist is `defi-com/monorepo`, `defi-com/mobile`,
|
|
27
|
-
`defi-com/azure-next-hybrid
|
|
28
|
-
`defi-com/mobile`. State both lists verbatim in the driver prompt. An own PR is
|
|
26
|
+
it. The review allowlist is `defi-com/monorepo`, `defi-com/mobile`,
|
|
27
|
+
`defi-com/azure-next-hybrid`, and `defi-com/ci-workflows`. The repair allowlist
|
|
28
|
+
is `defi-com/monorepo` and `defi-com/mobile`. State both lists verbatim in the driver prompt. An own PR is
|
|
29
29
|
open, authored by self, on the repair allowlist, and not a draft. A peer PR is
|
|
30
30
|
open, on the review allowlist, authored by someone else, and either officially
|
|
31
31
|
review-requested from self or has a non-self comment that both mentions self
|
|
@@ -180,11 +180,26 @@ The verdict body ends with this exact marker line:
|
|
|
180
180
|
<!-- axstack-automation verdict head=<sha> -->
|
|
181
181
|
```
|
|
182
182
|
|
|
183
|
+
The verdict body is written for the person reading the PR and reads as one
|
|
184
|
+
reviewer's findings. It never names the reviewer count, the brief, the
|
|
185
|
+
angles, the gate, receipts, or which reviewer found what: "two independent
|
|
186
|
+
reviews", "from secondary", and "the reviewers ran" are pipeline facts, not
|
|
187
|
+
review content. It is self-contained: every validated finding, blocking or
|
|
188
|
+
not, appears in full in the body — evidence and consequence, file and line
|
|
189
|
+
where they exist — so a reader is never told that notes exist without seeing
|
|
190
|
+
them, and a finding is never dropped to keep the body short. It never points
|
|
191
|
+
at the local review file or at anything the reader cannot open. Evidence
|
|
192
|
+
appears as what was checked and observed, not as who ran it: the reviewed
|
|
193
|
+
head and base SHAs, CI status, test counts, and diff size are reader-useful
|
|
194
|
+
facts and belong; "shape verified" and "pinned CI" are pipeline phrasing and
|
|
195
|
+
do not. The marker line is the only pipeline artefact the body carries.
|
|
196
|
+
|
|
183
197
|
Write the local review file to the workspace review directory
|
|
184
198
|
`~/defi/misc/reviews/` under the existing convention:
|
|
185
199
|
`review-PR-<num>.html` with no prefix means `defi-com/monorepo`;
|
|
186
|
-
`review-mobile-PR-<num>.html
|
|
187
|
-
`review-azure-next-hybrid-PR-<num>.html`
|
|
200
|
+
`review-mobile-PR-<num>.html`,
|
|
201
|
+
`review-azure-next-hybrid-PR-<num>.html` and `review-ci-workflows-PR-<num>.html`
|
|
202
|
+
name the other repositories. A write
|
|
188
203
|
failure is recorded but does not withhold the verdict.
|
|
189
204
|
|
|
190
205
|
Authored repair commits a local candidate, obtains one Sol review at that
|
|
@@ -235,7 +250,9 @@ Before opening a `push` token, the repair agent pins its candidate with local
|
|
|
235
250
|
ref `refs/axstack/decisions/<token>` in the project clone, so the candidate
|
|
236
251
|
survives worktree removal. It then sends one
|
|
237
252
|
`hermes send --to telegram` message naming the PR, criterion, every reviewer's
|
|
238
|
-
reason, and the exact replies
|
|
253
|
+
reason, and the exact replies `/axstack-decide approve <token>` and
|
|
254
|
+
`/axstack-decide reject <token>` (the slash form loads the Hermes skill
|
|
255
|
+
deterministically; bare `approve <token>` is best effort). Store the
|
|
239
256
|
send receipt. The driver retries a `failed` send once next tick and reconciles
|
|
240
257
|
an `uncertain` send against Hermes output before any retry.
|
|
241
258
|
|
|
@@ -81,7 +81,8 @@ existing action boundaries before acting; silence never grants permission.
|
|
|
81
81
|
An automation escalation is the narrow exception defined by
|
|
82
82
|
[Automation sessions](../axstack/references/automations.md). The PR agent opens
|
|
83
83
|
an immutable-bound decision token, sends one message with the exact
|
|
84
|
-
|
|
84
|
+
`/axstack-decide approve <token>` and `/axstack-decide reject <token>` replies,
|
|
85
|
+
records the send receipt, and
|
|
85
86
|
exits without waiting. The Hermes script decides by validating the private
|
|
86
87
|
channel and updating only an open token file; Hermes never performs the bound
|
|
87
88
|
GitHub or Git action. The driver consumes the file on a later tick, revalidates
|