@sabaiway/agent-workflow-kit 2.1.0 → 3.1.0

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.
Files changed (107) hide show
  1. package/CHANGELOG.md +94 -1
  2. package/README.md +7 -5
  3. package/SKILL.md +16 -8
  4. package/bin/install.mjs +11 -2
  5. package/bridges/antigravity-cli-bridge/SKILL.md +8 -1
  6. package/bridges/antigravity-cli-bridge/bin/agy-review-honesty.test.mjs +163 -0
  7. package/bridges/antigravity-cli-bridge/bin/agy-review-model-screen.test.mjs +76 -0
  8. package/bridges/antigravity-cli-bridge/bin/agy-review.sh +80 -15
  9. package/bridges/antigravity-cli-bridge/bin/agy-review.test.mjs +171 -22
  10. package/bridges/antigravity-cli-bridge/bin/agy.test.mjs +59 -41
  11. package/bridges/antigravity-cli-bridge/capability.json +3 -2
  12. package/bridges/codex-cli-bridge/SKILL.md +8 -1
  13. package/bridges/codex-cli-bridge/bin/codex-exec.test.mjs +67 -22
  14. package/bridges/codex-cli-bridge/bin/codex-review-honesty.test.mjs +156 -0
  15. package/bridges/codex-cli-bridge/bin/codex-review.sh +71 -10
  16. package/bridges/codex-cli-bridge/bin/codex-review.test.mjs +158 -27
  17. package/bridges/codex-cli-bridge/capability.json +3 -2
  18. package/capability.json +1 -1
  19. package/migrations/3.0.0-hardened-core-loop.md +44 -0
  20. package/migrations/README.md +1 -1
  21. package/package.json +2 -2
  22. package/references/hooks/gate-approve.mjs +1 -1
  23. package/references/modes/bootstrap.md +3 -3
  24. package/references/modes/commit-guard.md +19 -0
  25. package/references/modes/core-evidence.md +15 -0
  26. package/references/modes/coverage-check.md +12 -0
  27. package/references/modes/doc-parity.md +7 -6
  28. package/references/modes/gates.md +7 -6
  29. package/references/modes/grounding.md +1 -2
  30. package/references/modes/hook.md +1 -1
  31. package/references/modes/recommendations.md +1 -0
  32. package/references/modes/review-state.md +3 -3
  33. package/references/modes/upgrade.md +6 -6
  34. package/references/modes/velocity.md +2 -2
  35. package/references/modes/worktrees.md +120 -0
  36. package/references/scripts/archive-decisions.mjs +3 -3
  37. package/references/scripts/archive-decisions.test.mjs +5 -5
  38. package/references/scripts/check-docs-size-cli.test.mjs +41 -0
  39. package/references/scripts/check-docs-size.mjs +46 -29
  40. package/references/scripts/install-git-hooks-repo-exec.test.mjs +82 -0
  41. package/references/scripts/install-git-hooks.mjs +90 -18
  42. package/references/scripts/install-git-hooks.test.mjs +102 -0
  43. package/references/scripts/migrate-gates-branches.test.mjs +157 -0
  44. package/references/scripts/migrate-gates.mjs +395 -0
  45. package/references/scripts/migrate-gates.test.mjs +284 -0
  46. package/references/templates/agent_rules.md +2 -2
  47. package/tools/ack-write.mjs +1 -1
  48. package/tools/atomic-write.mjs +2 -2
  49. package/tools/autonomy-config.mjs +1 -1
  50. package/tools/autonomy-doctor.mjs +2 -2
  51. package/tools/autonomy-write.mjs +1 -1
  52. package/tools/bridge-settings-read.mjs +1 -1
  53. package/tools/bridge-settings.mjs +1 -1
  54. package/tools/changed-surface.mjs +16 -77
  55. package/tools/cheap-agents.mjs +2 -2
  56. package/tools/commands.mjs +26 -10
  57. package/tools/commit-guard.mjs +167 -0
  58. package/tools/core-evidence.mjs +914 -0
  59. package/tools/coverage-check.mjs +260 -0
  60. package/tools/delegation.mjs +1 -1
  61. package/tools/detect-backends.mjs +3 -3
  62. package/tools/doc-parity.mjs +11 -27
  63. package/tools/engine-source.mjs +1 -1
  64. package/tools/family-members.mjs +1 -1
  65. package/tools/family-registry.mjs +1 -1
  66. package/tools/fs-safe.mjs +1 -1
  67. package/tools/gate-hook.mjs +1 -1
  68. package/tools/{seed-gates.mjs → gates-init.mjs} +58 -52
  69. package/tools/grounding.mjs +19 -96
  70. package/tools/hide-footprint.mjs +1 -1
  71. package/tools/inject-methodology.mjs +72 -41
  72. package/tools/known-footprint.mjs +1 -1
  73. package/tools/labels.mjs +1 -1
  74. package/tools/lcov.mjs +6 -10
  75. package/tools/lens-region.mjs +1 -1
  76. package/tools/manifest/validate.mjs +19 -0
  77. package/tools/migrate-adr-store.mjs +5 -5
  78. package/tools/orchestration-config.mjs +1 -1
  79. package/tools/orchestration-write.mjs +1 -1
  80. package/tools/presentation.mjs +1 -1
  81. package/tools/procedures.mjs +3 -3
  82. package/tools/recipes.mjs +62 -11
  83. package/tools/recommendations.mjs +97 -14
  84. package/tools/release-scan.mjs +10 -3
  85. package/tools/renderers.mjs +1 -1
  86. package/tools/review-state.mjs +291 -343
  87. package/tools/run-gates.mjs +216 -92
  88. package/tools/sandbox-masks.mjs +14 -14
  89. package/tools/semver-lite.mjs +1 -1
  90. package/tools/set-autonomy.mjs +1 -1
  91. package/tools/set-recipe.mjs +2 -2
  92. package/tools/setup-backends.mjs +1 -1
  93. package/tools/surface.mjs +1 -1
  94. package/tools/uninstall.mjs +1 -1
  95. package/tools/velocity-profile.mjs +9 -9
  96. package/tools/view-model.mjs +1 -1
  97. package/tools/worktrees.mjs +2292 -0
  98. package/references/modes/fold-completeness.md +0 -30
  99. package/references/modes/review-ledger.md +0 -34
  100. package/references/templates/verification-profile.json +0 -10
  101. package/tools/fold-completeness-run.mjs +0 -1120
  102. package/tools/fold-completeness.mjs +0 -672
  103. package/tools/review-ledger-core.mjs +0 -428
  104. package/tools/review-ledger-write.mjs +0 -647
  105. package/tools/review-ledger.mjs +0 -630
  106. package/tools/sarif.mjs +0 -52
  107. package/tools/verification-profile.mjs +0 -219
package/CHANGELOG.md CHANGED
@@ -4,6 +4,99 @@ Semantically versioned ([semver](https://semver.org)), newest first. The `versio
4
4
  is the current release. `upgrade` mode reads a project's `docs/ai/.workflow-version` and applies
5
5
  every `migrations/<version>-<slug>.md` newer than it, in semver order.
6
6
 
7
+ ## 3.1.0 — parallel feature worktrees v1: provision · list · land --prepare · cleanup (AD-060)
8
+
9
+ New `worktrees` mode + `tools/worktrees.mjs` — several features implemented simultaneously in
10
+ DIFFERENT agent sessions on one machine/repo, zero interference on working-tree files (the ONE
11
+ exception: the default `node_modules` symlink is a shared MUTABLE dependency cache — the printed
12
+ isolated install is the isolation lane), unambiguous ownership. One thin dependency-free tool over git: every verification datum is recomputed live
13
+ from git, never read from stored metadata (the ONE exception: the PREPARED OID in the handoff,
14
+ read back only for recovery). Git >= 2.36 floor (NUL-terminated worktree porcelain); typed STOPs,
15
+ never a guess.
16
+
17
+ - **`provision <slug> --plan <path>`** — visible sibling worktree + `aw/<slug>` branch: the
18
+ registry-derived footprint copy-if-missing (a tracked file is NEVER overwritten), exactly one
19
+ seeded feature plan, the `handoff-<slug>.md` stub, shared-`node_modules` symlink advice
20
+ (`--install` only PRINTS the isolated install — zero spawn, zero write). `--resume` completes a
21
+ half-done provision behind fail-closed identity: at most one handoff, slug AND branch must
22
+ match, the section-required `## Provision record` (a duplicated field or section is a STOP,
23
+ never last-wins), user content preserved byte-exact.
24
+ - **`list`** — read-only and honest: any read failure renders `handoff: (unreadable)` — a silent
25
+ "no" never appears.
26
+ - **`land <slug> --prepare`** — the transient common-git-dir lock (shared with cleanup),
27
+ dirty-main / graph-divergence / docs-ai-drift / red-review-state refusals, the satellite
28
+ working-tree diff versus its base — staged AND unstaged inspected, every unstaged or
29
+ untracked-not-ignored leftover listed and refused (ignored content is outside observation by
30
+ design) — binary-safe transfer excluding exactly `docs/ai` + `docs/plans`, the optional
31
+ porcelain-visible sync adapter, the main gate matrix, and an OID report (main HEAD · TRANSFER ·
32
+ PREPARED · sync delta). The commit is NEVER run by the tool — it stays a dialogue ask. A second
33
+ prepare is reset-only against the recorded PREPARED OID; transfer-apply and post-launch sync
34
+ failures attempt a byte-clean rollback (rollback failures are composed without losing the
35
+ primary error), while a red gate matrix that leaves the snapshot unchanged intentionally KEEPS
36
+ the prepared tree and names both recovery lanes.
37
+ - **`cleanup <slug>`** — live landed-verification against main HEAD (exact land-exclusion
38
+ parity), typed-EXACT ownership of ignored content (file/glob roots own only files), literal
39
+ pathspecs, branch `-d`, prune; foreign content stops it. `--abandon` is the ONE destructive arm
40
+ (`-D`; requires the handoff identity).
41
+ - **Hardening:** every content read and regular-file copy goes through two no-follow descriptor
42
+ doors (identity-bound source · exclusive destination · descriptor-mode update), pinned as the
43
+ only paths by tripwire tests; strict parsers everywhere (NUL porcelain fields,
44
+ scan-before-parse JSON with any-depth duplicate-key refusal, atomic section surgery).
45
+ - **release-scan:** the reviewer-round-identity rung — an `agy` or `codex` name followed by an
46
+ R-number reference is refused; shippable finding IDs use neutral
47
+ `review-<scope>-rNN-<severity>-NN` IDs; the clean line now reads "no AI attribution or
48
+ reviewer-round identity found".
49
+ - **recommendations:** the worktrees parent-dir item — write access stays "not confirmed" without
50
+ a trusted host-capability signal; HAND-APPLY lines are host-qualified.
51
+ - **Docs:** the `references/modes/worktrees.md` contract — the MAIN/SATELLITE ownership matrix
52
+ (shared git state included), satellite forbidden verbs (the v1 docs-only bar), the
53
+ host-specific consent lane, and the other-harnesses PROVEN/ASSUMED split.
54
+
55
+ ## 3.0.0 — strip-the-kit: the hardened computed core replaces the ledger machinery (AD-059)
56
+
57
+ > ### ⚠ BREAKING — the review loop is now computed, not recorded
58
+ >
59
+ > The review-ledger / fold-completeness / verification-profile / sarif / seed-gates tools, their
60
+ > modes, and their git-dir stores are **DELETED**. `run-gates --record` is a usage error (exit 7
61
+ > retired); the loop's ONE receipt is `run-gates --final`. Review receipts without a self-declared
62
+ > `posture` field (any pre-3.0.0/4.0.0 bridge wrapper) **stop satisfying** the review-state gate —
63
+ > fail-closed, like the AD-057 probe marker; re-run the reviews on the refreshed bridges. Node
64
+ > floor is **>= 22** family-wide. The consented `migrate-gates.mjs` migrates an existing
65
+ > `docs/ai/gates.json` (see `migrations/3.0.0-hardened-core-loop.md`).
66
+
67
+ The MAJOR family release (memory **3.0.0** / engine **2.0.0** / bundled codex bridge **3.0.0** /
68
+ bundled antigravity bridge **4.0.0**; deployment-lineage head **3.0.0**). One theme: **every claim
69
+ the loop makes is computed from artifacts it can re-verify — never remembered, never recorded prose.**
70
+
71
+ - **The hardened core (new tools):** `core-evidence.mjs` — the ONE git-dir evidence writer
72
+ (red-proof records with observed-red 3/3 custody + content hashes; explicit per-backend degrade
73
+ records; the stateless end-of-loop `summary`) · `coverage-check.mjs` — the D3(d) final-run
74
+ checker (changed-line lcov coverage + red-proof verification + the `lcov-sha256=` machine line) ·
75
+ `commit-guard.mjs` — the D10 read-only pre-commit (binds a commit to the LATEST green final
76
+ receipt at the exact tree fingerprint; `--no-verify` stays a stated residual).
77
+ - **`run-gates --final`:** canonical realpath-anchored core checks (review-state + coverage-check,
78
+ checker LAST), evidence-store drift teeth, the checker-bound lcov sha with an end re-hash, ONE
79
+ attempt-linked receipt (green/red DERIVED, integrity failures explicit), `AW_GIT_DIR` exported
80
+ to every gate child on every run.
81
+ - **`review-state` D3(b):** ship-class-only on the latest NORMAL receipt; a recognized negative is
82
+ an authoritative VETO; unknown verdicts, probe-marked, unmarked, posture-less, and malformed
83
+ receipts never attest (each with its stated recovery); explicit degrade records are the only
84
+ escape and never all backends.
85
+ - **Consumer lanes:** `gates-init.mjs` (D9 consented fill preview; the coverage-check candidate
86
+ appended LAST so a whole-offer apply is final-run-ready) · `migrate-gates.mjs` (D8, mirrored
87
+ from the memory canon: canonical-anchor matching with move/collision semantics, the full lcov
88
+ reporter flag set, retired-store cleanup, symlink/control-byte STOPs) · the hook installer's
89
+ explicit `--commit-guard <path>` arm with strict carry-forward · a `commit-guard`
90
+ Recommendations item gated on final-run-capability.
91
+ - **Bundled bridges (D4/D5):** a verdict-less review run exits 4 with NO receipt (exact agy
92
+ `### Verdict` parse; structural top-level JSON verdict in codex schema mode); one stderr banner
93
+ states the ACTUAL dispatch posture and every receipt records the same `posture{}`; control-byte
94
+ and unknowable-model postures refuse pre-spend. Manifests carry VALIDATED posture pins; the kit
95
+ renders the CONFIGURED posture (pins + the bridge-settings tier overlay) in the backend status
96
+ line, drift-guarded end to end (`posture-parity`).
97
+ - **Suite economics note:** the supervisory ledger cluster (~98% of the old 94s unit matrix) is
98
+ gone with its machinery; the dedicated speedup pass (plan Phase 5) follows post-release.
99
+
7
100
  ## 2.1.0 — agy code review fails CLOSED pre-spend: grounded facts required, `--ungrounded` is the explicit escape (AD-058)
8
101
 
9
102
  > ### ⚠ the bundled antigravity bridge takes MAJOR 3.0.0 — bare `agy-review code` now refuses
@@ -457,7 +550,7 @@ was blocked on an honest degrade.
457
550
 
458
551
  Contract surfaces moved in lockstep (the tool header, `--help`, the human render, the `--await`
459
552
  comment, `references/modes/review-state.md`). Council-converged both segments (S1 codex + agy SHIP; S2
460
- codex R1 revise → allPresent fold → R2 both SHIP), the two-gate agreement dogfooded live + pinned by a
553
+ review round 1 revise → allPresent fold → review round 2 both SHIP), the two-gate agreement dogfooded live + pinned by a
461
554
  detector-independent `two-gate-agreement.test.mjs` + a 17-case matrix; full suite 2969 green.
462
555
 
463
556
  ## 1.40.0 — Universal verification profile + session-loop economics (a)–(h) (BUGFREE-3, AD-049)
package/README.md CHANGED
@@ -13,7 +13,7 @@ decisions. Works with Claude Code, Codex, Cursor, and any agent that reads `AGEN
13
13
  [![license](https://img.shields.io/npm/l/@sabaiway/agent-workflow-kit)](./LICENSE)
14
14
  [![node](https://img.shields.io/node/v/@sabaiway/agent-workflow-kit)](https://nodejs.org)
15
15
 
16
- `Node ≥ 18` · `dependency-free scripts` · `no telemetry in family code`
16
+ `Node ≥ 22` · `dependency-free scripts` · `no telemetry in family code`
17
17
 
18
18
  **One command to start:**
19
19
 
@@ -229,7 +229,7 @@ file), or run the guarded `/agent-workflow-kit uninstall`.
229
229
  | `/agent-workflow-kit upgrade` | existing deployment | reads `docs/ai/.workflow-version`, shows the changelog diff, preserves your authored memory, applies migrations, re-stamps — then prints a **read-only** one-line backend-status line (what's set up vs missing); refreshes the already-placed bridges from the kit's bundled copies (never installs a new one — set one up with `/agent-workflow-kit setup`) |
230
230
  | `/agent-workflow-kit help` | any time | **read-only command index** — every command, grouped (Inspect / Configure / Orchestrate / Lifecycle) and tagged read-only / writer / guarded. The discoverable entry point, and where any unrecognized invocation lands (always read-only — a garbage invocation never writes). Never writes, never commits, never runs a subscription CLI. |
231
231
  | `/agent-workflow-kit backends` | any time | **read-only** check of the optional execution-backends (the `codex` / `agy` bridges): what's set up vs missing and the next step. Never writes, never commits, never runs a subscription CLI (credentials = marker-file presence, not a live login). |
232
- | `/agent-workflow-kit gates` | any time | **project gate runner** — runs the verification commands **your project itself declares** in `docs/ai/gates.json` (seeded at deploy; hand-editable `{ id, title, cmd }` entries, each `cmd` one bash line) as one batch: a per-gate **PASS/FAIL table** + one machine-readable summary line, exit 0 iff all green; a failing gate's own output is shown verbatim; `--only <id>` re-runs one. Honest distinct outcomes for a missing / empty / malformed declaration — never a silent green. The runner writes nothing **by default** and never commits — opt-in `--record` mints ONE `gate-run` record into the review ledger **via the ledger's sole writer** (the segment's green-baseline receipt: full declaration + what ran + pre/post tree fingerprints; a red run records honestly; a failed record is its own loud exit 7); it executes only your own declared commands (a batching convenience, not a sandbox). The velocity tier auto-approves only the exact no-`--record` form — the recording run stays explicit. |
232
+ | `/agent-workflow-kit gates` | any time | **project gate runner** — runs the verification commands **your project itself declares** in `docs/ai/gates.json` (seeded at deploy; hand-editable `{ id, title, cmd }` entries, each `cmd` one bash line) as one batch: a per-gate **PASS/FAIL table** + one machine-readable summary line, exit 0 iff all green; a failing gate's own output is shown verbatim; `--only <id>` re-runs one. Honest distinct outcomes for a missing / empty / malformed declaration — never a silent green. The runner writes nothing **by default** and never commits — opt-in `--final` runs the FULL declared matrix and mints ONE final-run receipt into the git-dir core-evidence store (status green/red, pre/post tree fingerprints, the declaration, the consumed lcov's sha; the receipt the commit-guard binds); it executes only your own declared commands (a batching convenience, not a sandbox). The velocity tier auto-approves only the exact no-`--final` form — the recording run stays explicit. |
233
233
  | `/agent-workflow-kit setup [backend]` | opt-in, any time | **link-only** auto-setup of a bridge: places the bundled bridge skill (only into an absent / empty / managed dir — never overwrites an unmanaged one) + links its wrappers onto `PATH` via managed symlinks (idempotent; refuses to clobber a non-symlink; try `--dry-run` to preview). The binary install + the one-time subscription login stay **manual**: it prints the exact **login** command and points the binary install at each bridge's `setup/README.md`. POSIX wrappers — on Windows use WSL. Never commits, never runs a subscription CLI. |
234
234
  | `/agent-workflow-kit status` | any time | **read-only** single view of **versions + deployment + settings + bridges**: which members (kit / memory / engine / the two bridges) are installed and at what version (with an honest "installed on this machine" note when one is behind) and — in a project — what's deployed (`docs/ai`, the version stamps, and the **visibility**: visible / hidden / unclear), plus your settings (orchestration recipes, attribution, velocity) and the bridges' readiness. The two version axes (package number vs deployment-structure head) stay decoupled. Never writes, never commits, never runs a subscription CLI. |
235
235
  | `/agent-workflow-kit recipes` | any time | **read-only** orchestration advisor: presents four named recipes for composing the bridges into plan → execute → review — **Solo / Reviewed / Council / Delegated** — plans + recommends one for your environment (degrading with a stated reason when a backend isn't ready), and offers the choice. The orchestrator runs it via the bridge skills and **always commits**; the kit never executes a recipe, never runs a subscription CLI, never commits. |
@@ -237,11 +237,13 @@ file), or run the guarded `/agent-workflow-kit uninstall`.
237
237
  | `/agent-workflow-kit set-recipe` | any time | **config writer** for `docs/ai/orchestration.json`: tell the agent your preference in plain language and it maps it to explicit `--set <activity>.<slot>=<recipe>` / `--unset` ops; the kit validates, **previews by default**, and writes only on `--write` (deployment-gated, atomic, symlink/TOCTOU-safe), resolving the effective recipe vs live readiness. Writes **only** that file — **never runs a backend, never commits**; hand-editing stays fully supported. |
238
238
  | `/agent-workflow-kit review-state` | any time | **read-only review-receipt checker** — makes "reviewed ≠ shipped" detectable: the bridge review wrappers append a receipt per successful review (into a file inside the git dir — never committable); this checks that every backend your configured `plan-execution.review` recipe names holds a **fresh, grounded receipt for the current uncommitted tree** (any later edit moves the fingerprint and stales the receipt; a review continuation never re-attests a folded tree). `--check` gives a gate exit code to declare in `docs/ai/gates.json` **by hand or via the consent-gated seeder** (preview → your explicit yes; never without consent). `--await [--timeout <s>]` (AD-049) BLOCKS until every recipe-named backend has receipted the current tree — the durable completion signal is the receipt, never a process event — so you wait for the bridges instead of hand-polling. Never writes, never commits, never runs a subscription CLI; it spawns read-only `git` queries to fingerprint the tree — and `git commit --no-verify` stays possible (discipline, not a sandbox). |
239
239
  | `/agent-workflow-kit sandbox-masks` | any time | **cosmetic exclude lane for sandbox device masks** — an OS sandbox (Claude Code) injects character-device masks into the work tree as untracked `git status` noise; the review domain already ignores them **by construction** (never-committable untracked classes — char/block devices, FIFOs, sockets — are excluded from the fingerprint, the assembled review payload, and the clean checks). This mode hides them from `git status` too: flagless = read-only probe (derives the CURRENT mask set from the unfiltered walk + lstat — never a frozen list — and revalidates fenced entries, loudly flagging one that became a real path); `--apply` = consent-gated FULL-BLOCK replace of its own fenced block in `git rev-parse --git-path info/exclude` (stale masks drop by construction; `--clear` always means REMOVE the block — it takes precedence over the derivation). Writes ONLY its fence — never `.gitignore`, never global config; symlinked/non-regular exclude paths and malformed fences fail closed. Watch note: a real file at an excluded path is silently skipped by bulk staging (`git add -A`/`git add .`) — delete the stale line first; the probe flags exactly this case. |
240
- | `/agent-workflow-kit grounding` | any time | **grounded-review facts assembler** — mechanizes populating `agy-review --facts @f`: slices your entry-point's **Hard Constraints** section verbatim (exactly one match, else a loud stop) and/or a plan's decision-bearing sections (`## Approach` + `## Verification` required, `## Decisions (locked)` when present; duplicates stop), under the same byte budget the agy wrapper enforces (minus `--reserve-bytes` for the artifact share), with a loud tail-trim on overflow. `--autonomy` (AD-044) appends the COMPUTED effective autonomy policy from the git-top `docs/ai/autonomy.json` (every red-line + per-activity level, stated source line; absent file → the computed defaults ARE the policy, exit 0; a malformed policy fails CLOSED, exit 1). `--ledger-summary` (AD-049) appends a COMPUTED review-ledger digest for the single in-flight segment (rounds · origins · classifications · verdicts · overrides — unrelated loops excluded), so the reviewer sees the loop's own history without hand-copying. Prints to stdout; `--out` writes **one scratch file only** — system-temp outside the repo ($TMPDIR / /tmp, rewritable) or a **fresh** gitignored in-repo path (create-only, exclusive write; an existing in-repo file, even gitignored, is refused — the `.env` clobber class); tracked, not-ignored-in-repo, other outside-repo, and symlink/non-regular destinations are all refused. Never commits, never runs a subscription CLI. |
241
- | `/agent-workflow-kit review-ledger` | any time | **review-round ledger** — turns the review-loop stop rule into a **computed** signal: record each plan-execution review round, its triage, and any recorded **overrides** (`oracle-change` lifts named tampered test files; `red-proof` waives the observed-red proof for exactly one `testId`; v4 adds **`size-cap`** — the exact sanctioned magnitude for a surface over the diff cap, segment-scoped — all loud, durable, auditable waivers, never silent) into a JSONL ledger inside the git dir (never committable), and read the stop decision (`converged > resolved-residual > triage-required > continue`) from the records — never from a remembered rule. **Since v4 every record carries `base` = the HEAD commit and the whole loop is SEGMENT-scoped (AD-048):** round numbering, the caps, and every tooth reset ONLY at a gated commit — a multiphase plan records fully while round 4 within one segment stays refused; `gate-run` records (minted by `gates --record`) make gates-before-review a computed precondition; the triage class **`refuted`** is the honest phantom-finding lane (grounds mandatory); `--telemetry` renders counts-only gate-efficacy data across all loops and both ledgers. `--status` replaces the hand-composed per-round tally; `record --from-receipts` (AD-049) DRAFTS the `backends[]` from the current-fingerprint receipts (verdict per backend, counts from the supplied findings) instead of hand-composing them — a recipe-named backend with no receipt is a loud stop; a **`batch`** verb applies an ordered list of record/classify/override ops in ONE writer call (the prompt-economy lane — the whole envelope is validated structurally first with zero writes, then ops apply sequentially and fail-fast; prior ops stay recorded, append-only); `--check` is a fail-closed gate exit code for `docs/ai/gates.json` (by hand or via the consent-gated seeder). A `fixable-bug` triage **requires its red→green `testId`** (v1..v3 records stay valid on read old records never enter a segment). The writer refuses a round (past the hard-max, while a triage is pending, or lacking a grounded review receipt) and refuses an override outside its single in-flight loop. Honest residual: records are forgeable — self-discipline, not a security boundary. Never commits, never runs a subscription CLI. |
242
- | `/agent-workflow-kit fold-completeness` | any time | **fold-completeness gate** — attests the review loop's folded fixes were proven the HONEST way (no fix theater): the runner executes your suite ONCE under `NODE_V8_COVERAGE`, checks every changed executable line is **executed**, probes every bound `testId` N times (`AW_FOLD_RERUNS`), records content hashes, and scans the test surface for tamper; the `--red` verb observes a test FAILING on the pre-fold tree **BEFORE the fix** and mints an **observed-red receipt**. The read-only checker (`--status` / `--check`) then requires, per bound test: the receipt, receipt-precedes-run order, N/N-green probes, and content **custody** (the green test is byte-identical to the one seen failing) the receipt+custody half waivable for exactly one `testId` by a recorded `red-proof` override; mixed/timed-out probes are QUARANTINE (never converted, no override); **v3 records carry `base` and the proof is segment-scoped (AD-048)**a committed phase's custody obligations close with its commit, and a receipt never crosses a commit boundary; tampered pre-existing test files fail closed unless covered by a recorded `oracle-change` override. Honest residuals: coverage proves **execution, not assertion**, and records are forgeable — self-discipline, not a security boundary. v1 scope is JS/V8 (pluggable via `AW_FOLD_BOUND_CMD`): changed TS/JSX fails closed, docs/config changes are listed but never block, and **no mutation testing ships**. Wire the gate by handthe consent-gated seeder deliberately does not offer it yet (JS-only hold). Never commits, never runs a subscription CLI. |
240
+ | `/agent-workflow-kit grounding` | any time | **grounded-review facts assembler** — mechanizes populating `agy-review --facts @f`: slices your entry-point's **Hard Constraints** section verbatim (exactly one match, else a loud stop) and/or a plan's decision-bearing sections (`## Approach` + `## Verification` required, `## Decisions (locked)` when present; duplicates stop), under the same byte budget the agy wrapper enforces (minus `--reserve-bytes` for the artifact share), with a loud tail-trim on overflow. `--autonomy` (AD-044) appends the COMPUTED effective autonomy policy from the git-top `docs/ai/autonomy.json` (every red-line + per-activity level, stated source line; absent file → the computed defaults ARE the policy, exit 0; a malformed policy fails CLOSED, exit 1). Prints to stdout; `--out` writes **one scratch file only** — system-temp outside the repo ($TMPDIR / /tmp, rewritable) or a **fresh** gitignored in-repo path (create-only, exclusive write; an existing in-repo file, even gitignored, is refused — the `.env` clobber class); tracked, not-ignored-in-repo, other outside-repo, and symlink/non-regular destinations are all refused. Never commits, never runs a subscription CLI. |
241
+ | `/agent-workflow-kit core-evidence` | any time | **the ONE loop-evidence writer** (strip-the-kit) every core evidence record lands in a single append-only JSONL store inside the git dir (never committable; versioned schema, latest-per-key supersession, byte-identical duplicates refused, malformed lines fail every reader closed). `red-proof "<file>#<pattern>"` declares an observed-red **BEFORE a bugfix** (N/N red runs + content custody + base + the pre-fix fingerprint; green/mixed/timeout are DISTINGUISHED refusals nothing written); `degrade --backend --reason` is the ONLY escape for an unavailable review backend (per-tree, never all backends); `summary` renders the whole loop state statelessly (gate result · per-backend verdicts · red-proofs · degrades)no ledger, no rounds, nothing remembered. Honest residual: records are forgeable — self-discipline, not a security boundary. Never commits, never runs a subscription CLI. |
242
+ | `/agent-workflow-kit coverage-check` | any time | **the final-run checker** (D3(c)+(d)) reads the lcov the declared `unit-tests` gate produced at the FIXED git-dir path and fails on any uncovered CHANGED executable Node line (listed `file:line`; a changed file absent from the map is a file-level red; out-of-domain/unsupported files are LISTED — the claim narrowed honestly); VERIFIES every current-base red-proof declaration (bound test exists · custody hash unchanged · green N/N now · pre-fix fingerprint differs); prints `lcov-sha256=<hex|none>` of the exact bytes it consumedthe sha the `--final` receipt binds and re-hashes. An absent lcov is a LOUD `skipped-no-lcov`; a symlinked path is a refusal. `--check` is the gate exit codedeclare it as the LAST gate (`run-gates --final` refuses otherwise). Read-only. |
243
+ | `/agent-workflow-kit commit-guard` | any time | **the read-only pre-commit guard** (D10) — binds the LATEST completed `run-gates --final` receipt to the EXACT current tree: refuses on a missing/red/stale receipt, fingerprint drift under the run, a dangling later attempt, declaration content drift, evidence-hash or lcov drift, or unsatisfied review obligations (the same review-state decision, recomputed over a sanitized env — forged out-of-repo stores never satisfy). Re-runs NO gate or test. Wire it into `.git/hooks/pre-commit` (the installer writes the RESOLVED invocation). `git commit --no-verify` stays the stated residual. |
243
244
  | `/agent-workflow-kit recommendations` | any time (every `upgrade` ends with it) | **read-only deployment advisor** (AD-044) — computes what in THIS deployment is configured sub-optimally (allowlist not seeded, autonomy render drifted, OS sandbox unavailable, gates undeclared, bridge friction, sandbox-mask clutter, an unacknowledged sandbox recipe) and renders **verdict-first**: one composed verdict line (does anything need attention?), then each item as **{severity · what · one-line benefit · an optional `recipe:` line (sandbox-lane only) · the exact consent-gated apply one-liner}**. The agent PRESENTS the section in the user's conversational language — every fact and count, nothing added or dropped; commands, paths, hosts and rule strings byte-exact; raw tool block on request — and runs EXACTLY the rendered one-liners only on your yes, surfacing each item's posture note first. Renders **present-even-when-empty** (`no recommendations — flow optimal.`); a failed probe degrades to a stated skip line. Registry strings are fact-true frozen one-line data (posture/risk notes live in the mode doc at the consent moment); the kit never seeds `sandbox.network.allowedDomains` / `filesystem.allowWrite` (**HAND-APPLY** territory), and the sandbox-lane item's convergence is a neutral fingerprint acknowledgement recorded by a consent-gated ack writer into `docs/ai/acks.json` — never a security key (the recipe is documented per bridge in `capability.json` `networkHosts` + `writableDirs`). `--cwd` is required (the target project is explicit); never writes, never commits, never runs a subscription CLI. |
244
245
  | `/agent-workflow-kit doc-parity` | any time | **read-only doc-parity lint** (AD-049) — kills the doc-drift class where a mode-contract doc silently lags a code constant (a `--check` doc still reading `300` after the diff cap moved to `400`): a **closed, exported registry** binds each live constant (review caps, schema versions, the ledger's own class/scope vocabulary, and the autonomy-doctor EXIT/status/trusted-dir contract) to the exact token its `references/modes/*.md` contract must carry, and asserts the CURRENT value renders into every bound file — a drifted doc, an unreadable file, or an absent token **fails closed**. The values are sourced from the live imports (never re-typed), so the lint can't itself go stale; adding a binding is adding a checked entry (closed-world, edit-safe). `--check` is a gate exit code for `docs/ai/gates.json`. Never writes, never commits, never runs a subscription CLI. |
246
+ | `/agent-workflow-kit worktrees` | any time | **parallel feature worktrees** — run several features in DIFFERENT agent sessions on one repo, zero interference on working-tree files (the ONE exception is the dependency cache, below): `provision <slug> --plan <file>` creates a sibling git worktree on branch `aw/<slug>` and populates it (registry-derived footprint copy-if-missing — a tracked file is never overwritten; EXACTLY ONE seeded feature plan; the `handoff-<slug>.md` record from minute zero; `node_modules` symlinked where the link stays ignored — a shared MUTABLE dependency cache: writes through it hit MAIN's node_modules; for isolation run the printed isolated-install command (`--install` only PRINTS it; on `--resume` run the printed unlink-first recovery first); absolute root-pinned gate commands rebased on untracked copies only, and only while their bytes equal the MAIN source or its rebased form — user-modified copies stay untouched); `list` is read-only (slug, branch, base, dirty, handoff); `land <slug> --prepare` locks the common git dir, fail-closes on divergence or incomplete satellite state, transfers the complete accepted satellite diff onto a CLEAN main, runs sync plus the declared gates, and reports HEAD/TRANSFER/PREPARED OIDs — the commit ALWAYS stays a dialogue ask; `cleanup <slug>` takes the same lock and removes a LANDED worktree only after live landed-verification against main HEAD, while `--abandon` is the ONE destructive arm (destroys unlanded work; **no preview step** on any writer). The parent dir is the `docs/ai/worktrees.json` `parentDir` setting (default: the repo's sibling parent); an unwritable parent degrades to printed maintainer-pasted commands, and the one-time host consent that makes it promptless surfaces via `recommendations`. Never commits, never pushes, never runs a subscription CLI. |
245
247
  | `/agent-workflow-kit uninstall` | opt-in, any time | **guarded teardown** — the inverse of `init` / `setup`. Removes only what's **provably ours** (managed skill dirs + bridge wrappers; in a project, the hidden-mode git-ignore block it added + the pre-commit hook it installed); **never deletes** your `docs/ai` / `AGENTS.md` (prints the exact `rm` to run by hand) or your `.claude/settings.json` (prints an **edit** — remove the attribution key, review any velocity `permissions.*` — never an `rm`). Always `--dry-run` first; preflight-then-mutate; never commits. |
246
248
  | `/agent-workflow-kit velocity` | Claude Code · opt-in | **onboarding velocity profile** — seeds a fixed, audited **read-only** allowlist into `.claude/settings.json` so routine read-only commands stop idling on approval prompts while you're away; opt-in `acceptEdits`; plus a **read-only advisory** of likely project gate commands to add by hand. Writes **only** `.claude/settings.json` — **never** allowlists commit/push/publish, never writes `settings.local.json`, never commits. A seeded entry is a **trust posture, not a sandbox** (a runtime residual remains at the settings level — its guard ships as the opt-in `hook` command); a direct commit/push/publish still asks. `--dry-run` first. |
247
249
  | `/agent-workflow-kit agents` | Claude Code · opt-in | **cheap-lane subagents** — places bundled subagent definitions (`.claude/agents/`) pinned to a **cheap model** (haiku, low effort, read-only tools) for mechanical work: extraction sweeps, changelog fact-skeletons, gate-failure triage. Judgment, review, and real code stay on your main lane — these vehicles only extract and draft, and their output is verified. Preview by default (`--apply` writes); an existing customized file is **preserved, never overwritten**; never touches `settings*.json`, never commits. |
package/SKILL.md CHANGED
@@ -3,7 +3,7 @@ name: agent-workflow-kit
3
3
  description: Deploy or upgrade a portable AI-agent memory-and-workflow system in any project. Use when the user wants to bootstrap `docs/ai/` + an entry-point `AGENTS.md` (+ `CLAUDE.md` alias) + cap/archive/index enforcement in a new or existing repo, set up the Memory Map and session protocols, install the docs-rotation pre-commit hook, or run `/agent-workflow-kit` / `/agent-workflow-kit upgrade`. Triggers on phrases like "set up the memory system", "deploy the AI workflow here", "bootstrap docs/ai", "upgrade the workflow".
4
4
  disable-model-invocation: true
5
5
  metadata:
6
- version: '2.1.0'
6
+ version: '3.1.0'
7
7
  ---
8
8
 
9
9
  # agent-workflow-kit
@@ -71,14 +71,14 @@ Pick the mode from the user's invocation — the mapping is pinned by `tools/com
71
71
  Before acting, read `docs/ai/.workflow-version` (the project's stamp) to decide the route — an **internal** routing decision, **not** a line you print (version display: *Version disclosure* in `${CLAUDE_SKILL_DIR}/references/shared/report-footer.md`). Route:
72
72
 
73
73
  - **absent** → bootstrap (a fresh deployment).
74
- - **stamp < `2.0.0`** (the deployment-lineage head) → `upgrade`.
75
- - **stamp == `2.0.0`** → already current; only the stamp-independent reconciles may run — the FULL set lives in `${CLAUDE_SKILL_DIR}/references/modes/upgrade.md` step 3; run step 3 (never enumerate the reconciles from memory).
74
+ - **stamp < `3.0.0`** (the deployment-lineage head) → `upgrade`.
75
+ - **stamp == `3.0.0`** → already current; only the stamp-independent reconciles may run — the FULL set lives in `${CLAUDE_SKILL_DIR}/references/modes/upgrade.md` step 3; run step 3 (never enumerate the reconciles from memory).
76
76
  - **stamp > head / unparseable** → STOP — never-downgrade gate (see `${CLAUDE_SKILL_DIR}/references/modes/upgrade.md` step 2).
77
77
  - **Exempt:** `autonomy-doctor` provisions the OS, not the deployment — stamp-independent (stated in its mode file).
78
78
 
79
79
  **Two independent version axes — never conflate them:**
80
80
 
81
- 1. **Project deployment** — `docs/ai/.workflow-version` vs the lineage head (`2.0.0`). This is the **only** axis this skill compares.
81
+ 1. **Project deployment** — `docs/ai/.workflow-version` vs the lineage head (`3.0.0`). This is the **only** axis this skill compares.
82
82
  2. **Kit freshness** — this skill's own files vs the published npm package. That is the **npx installer's** job: `npx @sabaiway/agent-workflow-kit@latest init` (it refuses a stale-cache downgrade by comparing the version on disk — **no network**). This skill never checks npm, and the package version is **not** the lineage head.
83
83
 
84
84
  **Refreshed the kit but nothing changed?** After the npx installer updates `~/.claude/skills/agent-workflow-kit/`, **restart the session** so the agent reloads the new skill files.
@@ -171,18 +171,26 @@ read-only — read `${CLAUDE_SKILL_DIR}/references/modes/review-state.md` before
171
171
 
172
172
  writer — read `${CLAUDE_SKILL_DIR}/references/modes/grounding.md` before acting.
173
173
 
174
- ### Mode: review-ledger
174
+ ### Mode: core-evidence
175
175
 
176
- writer — read `${CLAUDE_SKILL_DIR}/references/modes/review-ledger.md` before acting.
176
+ writer — read `${CLAUDE_SKILL_DIR}/references/modes/core-evidence.md` before acting.
177
177
 
178
- ### Mode: fold-completeness
178
+ ### Mode: coverage-check
179
179
 
180
- writer — read `${CLAUDE_SKILL_DIR}/references/modes/fold-completeness.md` before acting.
180
+ project-exec — read `${CLAUDE_SKILL_DIR}/references/modes/coverage-check.md` before acting.
181
+
182
+ ### Mode: commit-guard
183
+
184
+ read-only — read `${CLAUDE_SKILL_DIR}/references/modes/commit-guard.md` before acting.
181
185
 
182
186
  ### Mode: doc-parity
183
187
 
184
188
  read-only — read `${CLAUDE_SKILL_DIR}/references/modes/doc-parity.md` before acting.
185
189
 
190
+ ### Mode: worktrees
191
+
192
+ writer — read `${CLAUDE_SKILL_DIR}/references/modes/worktrees.md` before acting.
193
+
186
194
  ---
187
195
 
188
196
  ## References
package/bin/install.mjs CHANGED
@@ -28,7 +28,7 @@
28
28
  // them from this package's own bundled copies — a purely LOCAL copy, no third server contact —
29
29
  // skippable with `--no-bridges`. No tracking either way.
30
30
  //
31
- // Dependency-free, Node >= 18.
31
+ // Dependency-free, Node >= 22.
32
32
 
33
33
  import { readFile, mkdir, rm } from 'node:fs/promises';
34
34
  import { existsSync, lstatSync, realpathSync } from 'node:fs';
@@ -211,7 +211,16 @@ const spawnMember = ({ command, args, options }) => spawnSync(command, args, opt
211
211
  // npm/network blip (rate-limit, registry hiccup, momentary DNS) — an immediate retry tends to hit the
212
212
  // same blip, so wait briefly first. Atomics.wait is a dependency-free sync sleep (the install flow is
213
213
  // already synchronous here). Injected in tests as a 0ms no-op so the suite never actually sleeps.
214
- const RETRY_DELAY_MS = 1500;
214
+ // AW_INSTALL_RETRY_DELAY_MS is a test seam (the subprocess cascade suite zeroes the wait; the
215
+ // retry POLICY itself stays exercised). Unset, empty, malformed and negative all keep the shipped
216
+ // default — `Number('')` is 0, so the raw value is screened BEFORE coercion (the R1 council bug).
217
+ export const resolveRetryDelayMs = (env) => {
218
+ const raw = env.AW_INSTALL_RETRY_DELAY_MS;
219
+ if (typeof raw !== 'string' || raw.trim() === '') return 1500;
220
+ const parsed = Number(raw);
221
+ return Number.isFinite(parsed) && parsed >= 0 ? parsed : 1500;
222
+ };
223
+ const RETRY_DELAY_MS = resolveRetryDelayMs(process.env);
215
224
  const sleepSync = (ms) => {
216
225
  if (ms > 0) Atomics.wait(new Int32Array(new SharedArrayBuffer(4)), 0, 0, ms);
217
226
  };
@@ -2,7 +2,7 @@
2
2
  name: antigravity-cli-bridge
3
3
  description: Delegate work to Google's Antigravity CLI (`agy`) — the successor to Gemini CLI — to reach Gemini, Claude, and GPT-OSS models under a Google AI Pro/Ultra subscription from the terminal. Use when the user wants to run a headless `agy` prompt, hand a focused task or second-opinion review to `agy`, install or authenticate Antigravity CLI, check or economise its quota/models, bridge project context into `agy`, set up a second delegated-execution backend beside Codex, or troubleshoot `agy` flags, models, auth, conversations, or its no-JSON headless behaviour.
4
4
  metadata:
5
- version: '3.0.0'
5
+ version: '4.0.0'
6
6
  ---
7
7
 
8
8
  # antigravity-cli-bridge
@@ -132,6 +132,13 @@ agy-review diff <diff-file> [--facts @f] … # review a supplied diff
132
132
  agy-review --continue --decided @round1.md --focus "still-open items" # round-2 delta, no re-assembly
133
133
  ```
134
134
 
135
+ **Honesty + posture (D4/D5):** a run with no recognized `### Verdict` section — empty output
136
+ included — **exits 4 with NO receipt**: treat it as a *failed review to re-run*, never a fatal
137
+ session error. One stderr banner states the actual posture (`review posture: model=…`) and the
138
+ receipt records the same `posture {model}`; an attesting review with `AGY_MODEL` explicitly
139
+ emptied refuses pre-spend (`AGY_PROBE=1` exempt), and control bytes in a model string refuse
140
+ pre-spend in every mode.
141
+
135
142
  Frontier default `Gemini 3.1 Pro (High)`; **any** model is allowed (a sub-frontier one earns a
136
143
  silenceable `AGY_PROBE=1` advisory). An oversized `code` review trips the byte ceiling with trim/split
137
144
  guidance; `AGY_REVIEW_ALLOW_ADDDIR=1` offloads only the change set to a private `--add-dir` staging dir
@@ -0,0 +1,163 @@
1
+ // agy-review-honesty.test.mjs — the round-1 hardening pins for the D4/D5 arms (strip Phase 4):
2
+ // the verdict parse is EXACT (an inexact heading or a token buried mid-line never attests —
3
+ // `NOT SHIP` must not read as SHIP), and the empty-AGY_MODEL pre-spend refusal is scoped to the
4
+ // ATTESTING branch only (plan / diff / --ungrounded code never attest, so they run and record
5
+ // posture.model null). Colocated separately from agy-review.test.mjs — that file is
6
+ // red-proof-frozen; this one carries its own minimal standalone harness (the family idiom).
7
+
8
+ import { describe, it, after } from 'node:test';
9
+ import assert from 'node:assert/strict';
10
+ import { mkdtempSync, mkdirSync, writeFileSync, readFileSync, rmSync, chmodSync, existsSync, readdirSync, symlinkSync, cpSync } from 'node:fs';
11
+ import { tmpdir } from 'node:os';
12
+ import { join, dirname, resolve } from 'node:path';
13
+ import { fileURLToPath } from 'node:url';
14
+ import { spawnSync } from 'node:child_process';
15
+
16
+ const HERE = dirname(fileURLToPath(import.meta.url));
17
+ const WRAPPER = join(HERE, 'agy-review.sh');
18
+
19
+ const FAKE_AGY = [
20
+ '#!/usr/bin/env bash',
21
+ 'set -u',
22
+ 'printf invoked > "${AGY_FAKE_SENTINEL:-/dev/null}"',
23
+ 'printf "%s\\n" "${AGY_FAKE_OUTPUT:-### Verdict}"',
24
+ 'exit 0',
25
+ '',
26
+ ].join('\n');
27
+
28
+ const makePathWithout = (root, exclude = []) => {
29
+ const skip = new Set(exclude);
30
+ const dir = mkdtempSync(join(root, 'nobin-'));
31
+ for (const d of (process.env.PATH || '').split(':').filter(Boolean)) {
32
+ let names;
33
+ try { names = readdirSync(d); } catch { continue; }
34
+ for (const name of names) {
35
+ if (skip.has(name)) continue;
36
+ const link = join(dir, name);
37
+ if (existsSync(link)) continue;
38
+ try { symlinkSync(resolve(d, name), link); } catch { /* dup / race — ignore */ }
39
+ }
40
+ }
41
+ return dir;
42
+ };
43
+
44
+ // Farm + sandbox base are READ-ONLY per invocation — built ONCE, shared (a per-run farm rebuild
45
+ // plus a per-test `git init`+commit dominate the wall otherwise).
46
+ const SHARED_ROOT = mkdtempSync(join(tmpdir(), 'agy-honesty-shared-'));
47
+ after(() => rmSync(SHARED_ROOT, { recursive: true, force: true }));
48
+ const FARM = makePathWithout(SHARED_ROOT, ['agy', 'agy-run']);
49
+
50
+ const TEMPLATE_HOME = (() => {
51
+ const home = join(SHARED_ROOT, 'template-home');
52
+ const bin = join(home, '.local', 'bin');
53
+ mkdirSync(bin, { recursive: true });
54
+ writeFileSync(join(bin, 'agy'), FAKE_AGY, { mode: 0o755 });
55
+ const repo = join(home, 'repo');
56
+ mkdirSync(repo);
57
+ const g = (...args) => spawnSync('git', args, { cwd: repo, encoding: 'utf8' });
58
+ g('init', '-q');
59
+ g('config', 'user.email', 'probe@example.com');
60
+ g('config', 'user.name', 'probe');
61
+ writeFileSync(join(repo, 'base.txt'), 'committed base\n');
62
+ g('add', '-A');
63
+ g('commit', '-qm', 'base');
64
+ writeFileSync(join(repo, 'pending.txt'), 'PENDING\n');
65
+ writeFileSync(join(repo, 'plan.md'), '# a plan artifact\n');
66
+ return home;
67
+ })();
68
+
69
+ const makeSandbox = () => {
70
+ const home = mkdtempSync(join(tmpdir(), 'agy-honesty-'));
71
+ cpSync(TEMPLATE_HOME, home, { recursive: true });
72
+ const bin = join(home, '.local', 'bin');
73
+ chmodSync(join(bin, 'agy'), 0o755);
74
+ return { home, bin, repo: join(home, 'repo') };
75
+ };
76
+
77
+ const run = (sb, { args, env = {} } = {}) => {
78
+ const sentinel = join(sb.home, 'cap-sentinel');
79
+ const r = spawnSync('bash', [WRAPPER, ...args], {
80
+ cwd: sb.repo,
81
+ encoding: 'utf8',
82
+ timeout: 30000,
83
+ env: {
84
+ HOME: sb.home,
85
+ PATH: `${sb.bin}:${FARM}`,
86
+ TMPDIR: process.env.TMPDIR ?? '/tmp',
87
+ AGY_FAKE_SENTINEL: sentinel,
88
+ ...env,
89
+ },
90
+ });
91
+ return { ...r, invoked: existsSync(sentinel) };
92
+ };
93
+
94
+ const readReceipts = (repo) => {
95
+ const p = join(repo, '.git', 'agent-workflow-review-receipts.jsonl');
96
+ if (!existsSync(p)) return [];
97
+ return readFileSync(p, 'utf8').trim().split('\n').filter(Boolean).map((l) => JSON.parse(l));
98
+ };
99
+
100
+ describe('agy-review — the verdict parse is EXACT (M1): a buried token never attests', () => {
101
+ it('`NOT SHIP` on the verdict line is a FAILED run — never a SHIP receipt', () => {
102
+ const sb = makeSandbox();
103
+ const r = run(sb, { args: ['code', '--facts', 'a tiny fact'], env: { AGY_FAKE_OUTPUT: '### Verdict\nNOT SHIP — blocked.' } });
104
+ const receipts = readReceipts(sb.repo);
105
+ rmSync(sb.home, { recursive: true, force: true });
106
+ assert.notEqual(r.status, 0, 'NOT SHIP must never read as SHIP');
107
+ assert.equal(receipts.length, 0);
108
+ });
109
+
110
+ it('an INEXACT heading (`### Verdicts`) never parses — the D4 failed-run arm owns it', () => {
111
+ const sb = makeSandbox();
112
+ const r = run(sb, { args: ['code', '--facts', 'a tiny fact'], env: { AGY_FAKE_OUTPUT: '### Verdicts\nSHIP — but under the wrong heading.' } });
113
+ const receipts = readReceipts(sb.repo);
114
+ rmSync(sb.home, { recursive: true, force: true });
115
+ assert.notEqual(r.status, 0);
116
+ assert.equal(receipts.length, 0);
117
+ });
118
+
119
+ it('a WORD-PREFIX (`SHIPPING it`) never matches; punctuation forms (`SHIP.` / `SHIP — …`) do', () => {
120
+ const sb1 = makeSandbox();
121
+ const bad = run(sb1, { args: ['code', '--facts', 'f'], env: { AGY_FAKE_OUTPUT: '### Verdict\nSHIPPING it today.' } });
122
+ const badReceipts = readReceipts(sb1.repo);
123
+ rmSync(sb1.home, { recursive: true, force: true });
124
+ assert.notEqual(bad.status, 0, 'SHIPPING is not in the closed vocabulary');
125
+ assert.equal(badReceipts.length, 0);
126
+ for (const [out, want] of [['### Verdict\nSHIP.', 'SHIP'], ['### Verdict\nSHIP — clean.', 'SHIP'], ['### Verdict\nSHIP WITH NITS — two nits.', 'SHIP WITH NITS']]) {
127
+ const sb = makeSandbox();
128
+ const r = run(sb, { args: ['code', '--facts', 'f'], env: { AGY_FAKE_OUTPUT: out } });
129
+ const receipts = readReceipts(sb.repo);
130
+ rmSync(sb.home, { recursive: true, force: true });
131
+ assert.equal(r.status, 0, r.stderr);
132
+ assert.equal(receipts[0].verdict, want);
133
+ }
134
+ });
135
+ });
136
+
137
+ describe('agy-review — the empty-model refusal is scoped to the ATTESTING branch (M3)', () => {
138
+ it('plan and diff modes with AGY_MODEL= RUN (never attest) and record posture.model null', () => {
139
+ for (const args of [['plan', 'plan.md'], ['diff', 'plan.md']]) {
140
+ const sb = makeSandbox();
141
+ const r = run(sb, { args, env: { AGY_MODEL: '', AGY_FAKE_OUTPUT: '### Verdict\nSHIP — fine.' } });
142
+ const receipts = readReceipts(sb.repo);
143
+ rmSync(sb.home, { recursive: true, force: true });
144
+ assert.equal(r.status, 0, `${args[0]} must run with an emptied model: ${r.stderr}`);
145
+ assert.equal(r.invoked, true);
146
+ assert.deepEqual(receipts[0].posture, { model: null }, 'an unknowable model is recorded null');
147
+ }
148
+ });
149
+
150
+ it('code --ungrounded with AGY_MODEL= RUNS (its receipt never attests); grounded code still refuses', () => {
151
+ const sb = makeSandbox();
152
+ const un = run(sb, { args: ['code', '--ungrounded'], env: { AGY_MODEL: '', AGY_FAKE_OUTPUT: '### Verdict\nSHIP — throwaway.' } });
153
+ const unReceipts = readReceipts(sb.repo);
154
+ rmSync(sb.home, { recursive: true, force: true });
155
+ assert.equal(un.status, 0, un.stderr);
156
+ assert.deepEqual(unReceipts[0].posture, { model: null });
157
+ const sb2 = makeSandbox();
158
+ const grounded = run(sb2, { args: ['code', '--facts', 'a tiny fact'], env: { AGY_MODEL: '' } });
159
+ rmSync(sb2.home, { recursive: true, force: true });
160
+ assert.notEqual(grounded.status, 0, 'the ATTESTING branch still refuses pre-spend');
161
+ assert.equal(grounded.invoked, false);
162
+ });
163
+ });
@@ -0,0 +1,76 @@
1
+ // agy-review-model-screen.test.mjs — the round-2 ordering pin (strip Phase 4, M6): the
2
+ // control-byte screen fires IMMEDIATELY after AGY_MODEL resolution — BEFORE the off-frontier
3
+ // advisory (or any other interpolation) can echo raw control bytes into stderr/the terminal.
4
+ // Colocated separately: both earlier agy spec files are red-proof-frozen. Standalone harness.
5
+
6
+ import { describe, it } from 'node:test';
7
+ import assert from 'node:assert/strict';
8
+ import { mkdtempSync, mkdirSync, writeFileSync, readFileSync, rmSync, chmodSync, existsSync, readdirSync, symlinkSync } from 'node:fs';
9
+ import { tmpdir } from 'node:os';
10
+ import { join, dirname, resolve } from 'node:path';
11
+ import { fileURLToPath } from 'node:url';
12
+ import { spawnSync } from 'node:child_process';
13
+
14
+ const HERE = dirname(fileURLToPath(import.meta.url));
15
+ const WRAPPER = join(HERE, 'agy-review.sh');
16
+
17
+ const FAKE_AGY = '#!/usr/bin/env bash\nset -u\nprintf invoked > "${AGY_FAKE_SENTINEL:-/dev/null}"\nprintf "### Verdict\\nSHIP\\n"\nexit 0\n';
18
+
19
+ const makePathWithout = (root, exclude = []) => {
20
+ const skip = new Set(exclude);
21
+ const dir = mkdtempSync(join(root, 'nobin-'));
22
+ for (const d of (process.env.PATH || '').split(':').filter(Boolean)) {
23
+ let names;
24
+ try { names = readdirSync(d); } catch { continue; }
25
+ for (const name of names) {
26
+ if (skip.has(name)) continue;
27
+ const link = join(dir, name);
28
+ if (existsSync(link)) continue;
29
+ try { symlinkSync(resolve(d, name), link); } catch { /* dup / race — ignore */ }
30
+ }
31
+ }
32
+ return dir;
33
+ };
34
+
35
+ describe('agy-review — the control-byte screen precedes EVERY interpolation of AGY_MODEL (M6)', () => {
36
+ it('a control-byte NON-frontier model refuses pre-spend with NO advisory echo and NO raw byte on stderr', () => {
37
+ const home = mkdtempSync(join(tmpdir(), 'agy-model-screen-'));
38
+ const bin = join(home, '.local', 'bin');
39
+ mkdirSync(bin, { recursive: true });
40
+ writeFileSync(join(bin, 'agy'), FAKE_AGY, { mode: 0o755 });
41
+ chmodSync(join(bin, 'agy'), 0o755);
42
+ const repo = join(home, 'repo');
43
+ mkdirSync(repo);
44
+ const g = (...args) => spawnSync('git', args, { cwd: repo, encoding: 'utf8' });
45
+ g('init', '-q');
46
+ g('config', 'user.email', 'probe@example.com');
47
+ g('config', 'user.name', 'probe');
48
+ writeFileSync(join(repo, 'base.txt'), 'base\n');
49
+ g('add', '-A');
50
+ g('commit', '-qm', 'base');
51
+ writeFileSync(join(repo, 'pending.txt'), 'PENDING\n');
52
+ const sentinel = join(home, 'cap-sentinel');
53
+ const r = spawnSync('bash', [WRAPPER, 'code', '--facts', 'a tiny fact'], {
54
+ cwd: repo,
55
+ encoding: 'utf8',
56
+ timeout: 30000,
57
+ env: {
58
+ HOME: home,
59
+ PATH: `${bin}:${makePathWithout(home, ['agy', 'agy-run'])}`,
60
+ TMPDIR: process.env.TMPDIR ?? '/tmp',
61
+ AGY_FAKE_SENTINEL: sentinel,
62
+ AGY_MODEL: `hostile${String.fromCharCode(27)}model`, // ESC — a terminal-control byte
63
+ },
64
+ });
65
+ const invoked = existsSync(sentinel);
66
+ const receiptsPath = join(repo, '.git', 'agent-workflow-review-receipts.jsonl');
67
+ const receipts = existsSync(receiptsPath) ? readFileSync(receiptsPath, 'utf8').trim() : '';
68
+ rmSync(home, { recursive: true, force: true });
69
+ assert.notEqual(r.status, 0, 'a control-byte model refuses pre-spend');
70
+ assert.equal(invoked, false, 'agy is never invoked');
71
+ assert.equal(receipts, '', 'no receipt is minted');
72
+ assert.match(r.stderr, /control/i, 'named as the control-byte class');
73
+ assert.doesNotMatch(r.stderr, /non-frontier model/, 'the advisory never fires on a value the screen must refuse');
74
+ assert.ok(!r.stderr.includes(String.fromCharCode(27)), 'no raw control byte ever reaches stderr');
75
+ });
76
+ });