@sabaiway/agent-workflow-kit 14.26.0 → 14.28.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.
- package/CHANGELOG.md +69 -0
- package/README.md +1 -1
- package/SKILL.md +1 -1
- package/bridges/codex-cli-bridge/SKILL.md +1 -1
- package/bridges/codex-cli-bridge/bin/codex-exec.sh +7 -7
- package/bridges/codex-cli-bridge/bin/codex-review.sh +1 -1
- package/bridges/codex-cli-bridge/capability.json +1 -1
- package/capability.json +1 -1
- package/package.json +1 -1
- package/references/modes/recipes.md +2 -0
- package/references/modes/recommendations.md +9 -10
- package/references/modes/set-recipe.md +1 -1
- package/references/modes/velocity.md +5 -6
- package/references/shared/report-footer.md +29 -32
- package/tools/ack-write.mjs +11 -11
- package/tools/apply-danger-check.mjs +19 -14
- package/tools/bridge-sandbox-recipe.mjs +102 -15
- package/tools/bridge-state-dirs.mjs +94 -0
- package/tools/bridge-wiring.mjs +196 -0
- package/tools/manifest/schema.md +18 -15
- package/tools/manifest/validate.mjs +11 -11
- package/tools/recipes.mjs +75 -15
- package/tools/recommendations.mjs +64 -118
- package/tools/velocity-profile.mjs +195 -119
- package/tools/write-lanes.mjs +0 -1
package/CHANGELOG.md
CHANGED
|
@@ -4,6 +4,75 @@ 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
|
+
## 14.28.0 — `ready` means wired: the status line and `/agent-workflow-kit recipes` say when the project sandbox settings lack a bridge's entries (AD-178)
|
|
8
|
+
|
|
9
|
+
**A bridge reads `ready` on the status line only when the project settings cover its sandbox entries, where they
|
|
10
|
+
enable the sandbox.** Before, `ready` meant only that the bridge was installed and signed in, so a bridge whose
|
|
11
|
+
`<wrapper> *` exclusion, host or state dir was missing from `.claude/settings.json` read `ready` and then failed or
|
|
12
|
+
prompted under the sandbox; and the Recommendations `sandbox-lane` item settled once you acknowledged it, whether or
|
|
13
|
+
not any entry was written.
|
|
14
|
+
|
|
15
|
+
- **The status line.** A ready bridge whose entries are missing reads `✗ not wired — <n> sandbox entr(ies) missing`
|
|
16
|
+
(`✗ not wired (unused) — …` when no slot uses it); one that cannot be judged (an unreadable settings file or config,
|
|
17
|
+
a missing bridge manifest, a settings state the bridge tier refuses) reads `✗ unchecked — <reason>`. A new
|
|
18
|
+
` · sandbox: <condition>` segment says whether `ready` means wired (the sandbox enabled, where the host honors the
|
|
19
|
+
settings sandbox keys) or installed (not enabled), or reads `unchecked — <reason>` when it cannot be judged. A
|
|
20
|
+
failed load of the kit's own wiring module fails the `recipes` run with its error. The recommended recipe is unchanged; its clause adds each ready
|
|
21
|
+
bridge whose review entries are missing, with the route, or cannot be judged. Readiness, dispatch and every roster
|
|
22
|
+
read as before.
|
|
23
|
+
- **`/agent-workflow-kit recipes`** lists, per bridge that is not wired, the missing `excludedCommands`,
|
|
24
|
+
`allowedDomains` and `allowWrite` entries, the condition and the route: the chat yes of
|
|
25
|
+
`/agent-workflow-kit upgrade` (for a bridge no slot uses, once a slot uses it for review). The welcome mat of the
|
|
26
|
+
upgrade and bootstrap reports points a `✗ not wired —` or `✗ unchecked` cell there, never a `✗ not wired (unused)`
|
|
27
|
+
one.
|
|
28
|
+
- **The Recommendations `bridge-tier` item** keeps its text and count and gains a `recipe:` line naming each missing
|
|
29
|
+
entry (allow rules, exclusions, hosts, state dirs), what `codex-exec` needs where an execute slot delegates to it,
|
|
30
|
+
and, where the project settings enable the sandbox, that the bridge is wired once they are added.
|
|
31
|
+
- **The `sandbox-lane` item is retired**: `bridge-tier` already offers the writer for the same entries. An
|
|
32
|
+
acknowledgement already in `docs/ai/acks.json` or a settings file stays where it is and is no longer read.
|
|
33
|
+
- **One comparison.** The new `tools/bridge-wiring.mjs` compares the bridge surfaces against the project `.claude/settings.json`
|
|
34
|
+
(entries only in `settings.local.json` do not count);
|
|
35
|
+
`velocity-profile --bridge-tier` takes what it adds from the same function, so the tier and the status line agree.
|
|
36
|
+
- No shipped text tells you to add a bridge's sandbox keys by hand or says the kit never writes them: the
|
|
37
|
+
`recommendations`, `velocity` and `set-recipe` mode docs, the README advisor row, the manifest schema page and the
|
|
38
|
+
comments of `tools/manifest/validate.mjs` and `tools/ack-write.mjs` were corrected.
|
|
39
|
+
`/agent-workflow-kit set-recipe`'s advisory after a reviewed, council or delegated write says the upgrade's
|
|
40
|
+
`bridge-tier` item wires the used bridges on its chat yes.
|
|
41
|
+
|
|
42
|
+
## 14.27.0 — the bridge tier wires the bridges your recipes use: `<wrapper> *` exclusions, their hosts and their state dirs (AD-177)
|
|
43
|
+
|
|
44
|
+
**`velocity-profile --bridge-tier` now wires every used bridge's exclusions, hosts and state dirs.** Before, it
|
|
45
|
+
seeded each bridge whose review wrapper was placed on PATH, wrote the bare review wrapper name into
|
|
46
|
+
`sandbox.excludedCommands`, gave `codex-exec` no entry, and left the bridges' state dirs for you to add by hand. A bare
|
|
47
|
+
name matches only the command run with no arguments, so a real `codex-review code` call did not match it; and a bridge
|
|
48
|
+
that cannot write its state dir cannot run.
|
|
49
|
+
|
|
50
|
+
- **The used bridges.** The tier reads `docs/ai/orchestration.json` and the host's readiness and takes every bridge a
|
|
51
|
+
review or execute slot resolves to (the computed default when the file is absent). A placed bridge no slot uses gets
|
|
52
|
+
nothing.
|
|
53
|
+
- **Per used bridge:** for a review role, the wrapper's `code`-mode allow rule (and the quoted grounding rule with
|
|
54
|
+
`agy-review`) and `<wrapper> *` in `sandbox.excludedCommands`; for every used bridge, the manifest's `networkHosts`
|
|
55
|
+
in `sandbox.network.allowedDomains` and its `writableDirs` in `sandbox.filesystem.allowWrite`, home-relative
|
|
56
|
+
(`~/.codex`, `~/.gemini/antigravity-cli`), an env override honoured. Where an execute slot delegates to codex,
|
|
57
|
+
`codex-exec *` is excluded, with no allow rule: a delegated run keeps its prompt, as the review `plan` and `diff`
|
|
58
|
+
modes keep theirs.
|
|
59
|
+
- **Safe by construction.** Foreign entries and sibling keys stay, a second apply writes the same bytes, an entry an
|
|
60
|
+
earlier version wrote stays, and an entry already covering a dir (`~/.gemini` covers `~/.gemini/antigravity-cli`)
|
|
61
|
+
counts as present. A malformed `sandbox.filesystem` or `allowWrite` stops the run with nothing written, as does a
|
|
62
|
+
state dir that resolves to `/`, your home or an ancestor of your home or the project (the message names the env
|
|
63
|
+
variable, or the declared default, to fix).
|
|
64
|
+
- **One derivation everywhere.** The new `tools/bridge-state-dirs.mjs` resolves and spells the dirs; the tier, the
|
|
65
|
+
Recommendations `bridge-tier` item, the danger check and the `--autonomy` preview read the same list. The item now
|
|
66
|
+
counts missing `*` exclusions and state dirs, so a project wired before 14.27.0 is offered the one re-run in
|
|
67
|
+
`/agent-workflow-kit upgrade`. The danger check admits only this list, additions only. The `--autonomy` preview
|
|
68
|
+
names each entry the tier derives for a used bridge's role whose consent rule is in the project file (a review role's
|
|
69
|
+
`code`-mode allow rule, or `codex-exec *` for a delegated codex) as part of your bridge consent and what it widens, instead of asking
|
|
70
|
+
you to remove it.
|
|
71
|
+
- The `velocity` and `recommendations` mode docs, the README advisor row and the manifest schema page say so, and
|
|
72
|
+
none of them says the state dirs stay hand-apply or that the kit withholds the grant for safety; two tool comments
|
|
73
|
+
(`tools/manifest/validate.mjs`, `tools/ack-write.mjs`) still say the kit never seeds filesystem allowances and are
|
|
74
|
+
queued as debt.
|
|
75
|
+
|
|
7
76
|
## 14.26.0 — the agent applies the harness's own configuration on your chat "yes" again, after a danger check of the concrete change (AD-176)
|
|
8
77
|
|
|
9
78
|
**`/agent-workflow-kit upgrade` applies the harness's own configuration again.** In 14.25.0 an item that writes
|
package/README.md
CHANGED
|
@@ -256,7 +256,7 @@ file), or run the guarded `/agent-workflow-kit uninstall`.
|
|
|
256
256
|
| `/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. |
|
|
257
257
|
| `/agent-workflow-kit coverage-check` | any time | **the final-run checker** (D3(c)+(d)) — **certifies coverage ONLY inside the `--final` run that owns the lcov** (ownership is exclusive by CONVENTION over the fixed path, not enforced — a concurrent writer to it is a stated residual, queued as LCOV-EXCLUSIVE-OWNERSHIP): an artifact on disk proves nothing about the tree it came from, so a standalone run prints its findings and states `attested=no` / `NO VERDICT` rather than a PASS (an lcov that predates an edit would otherwise certify a line the suite never executed). `attested=` claims a verdict was ISSUED, pass or fail — a run over uncovered lines still reads `attested=yes` and still exits 1, and a run whose coverage arm never executed (no lcov) reads `attested=no` even inside `--final`, because nothing was read and nothing is certified. The runner passes a nonce whose one-way commitment over `{nonce, fingerprint, base}` is the `final-start.attempt` it recorded; a context describing another tree, or matching no recorded attempt, is a REFUSAL, never a verdict. Findings are unchanged — 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 consumed — the 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 code — declare it as the LAST gate (`run-gates --final` refuses otherwise). Read-only. |
|
|
258
258
|
| `/agent-workflow-kit commit-guard` | any time | **the read-only pre-commit guard** (D10) — makes the commit capture the whole current working tree, so «verified» and «about to be committed» are the same bytes (the receipt itself has a stated residual — see the mode doc). FIRST it refuses an **INDEX that lags the verified working tree** (the gates and the fingerprint describe the WORKING tree while `git commit` takes the INDEX alone, and the fingerprint cannot tell them apart — so a lagging index used to ship a strict subset of what was verified): unstaged tracked paths or reviewable untracked-not-ignored paths, named up to a bounded cap with the remainder stated, a dirty tracked **submodule** named separately with its own recovery, and fail-closed on an undecidable probe. This deliberately blocks a partial commit. Then it 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. |
|
|
259
|
-
| `/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,
|
|
259
|
+
| `/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, a feature-spec layer never adopted) 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 (the bridge-tier item's missing-entries line — each missing entry by kind: allow rules, exclusions, hosts, state dirs, then what the execution wrapper codex-exec needs, then the host condition; the worktrees-dir hand-apply-first grant advice, the agents hidden-mode reconcile follow-up, the spec-adoption decline preview, or another offer's decline preview) · 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); `velocity --bridge-tier` seeds the used bridges' review allow rules, `<wrapper> *` exclusions (`codex-exec *` where an execute slot delegates, keeping its permission prompt), declared hosts into `sandbox.network.allowedDomains`, and home-relative state dirs into `sandbox.filesystem.allowWrite`; hosts and dirs are declared per bridge in `capability.json` `networkHosts` + `writableDirs`, and the advisor's `bridge-tier` item counts and names each missing one. `--cwd` is required (the target project is explicit); writes nothing, never commits, never runs a subscription CLI. |
|
|
260
260
|
| `/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 the gate exit code a kit maintainer runs: the lint checks the kit's own package. Never writes, never commits, never runs a subscription CLI. |
|
|
261
261
|
| `/agent-workflow-kit dispatch` | any time | **the delegation engine** — makes "how much does delegating a sub-task actually buy?" a measured number instead of a feeling. `check <file>` validates a sub-task brief's contract block and exits 0/1 naming the first violated field — **form only**: whether the task is genuinely bounded, its design decided and its acceptance adequate stays your judgment, and a well-formed absurdity passes here by construction. `advise --step-class <c>` answers the question that comes BEFORE that one — which vehicle should carry this kind of sub-task, whether it is even present on this machine, and what the ledger has already recorded for threads of that class (finished, failed, closed without a fold, and still open — counted separately) — and it decides nothing: it refuses no dispatch, gates no verb, and a choice that diverges from the advice is noted rather than blocked. It reads the filesystem only: nothing is launched to find out what is installed, an absent ledger prints "no recorded history", an unreadable one prints the store's own words, and either way the advice still prints. The same block appears as a footer under a form-valid `check` — never under a refusal, so it can never hide one. `register` pre-registers an acceptance wave (step classes, pairing key, minimum observations per class, the mean and first-pass thresholds) so the bar can never be chosen after the results it judges — immutable per wave. `observe` records ONE hand-written observation (`solo-construction`, the baseline whose ratio is 1 by construction — except over a scope measuring zero bytes, which has no ratio at all and is recorded ineligible by name — or `self-reported`) — delegated numbers are never hand-written, they are derived. The writer verbs are where that derivation happens: `open` puts a delegated thread on the record with every mint-time field copied from the brief's own header, refusing a deadline the wrapper's cap plus its kill grace would not fit inside, and recording whether the tree it started from was clean against the thread's base — HEAD by default, or the tree `--checkpoint <oid>` names, so the second task of a story is measured over the first one's uncommitted work instead of opening dirty (a dirty one makes the result honestly unmeasurable rather than quietly counted), while `--task <brief>` also records which task brief the thread carries out and the files that brief lists, refusing, since the last commit, a task thread whose files overlap by path prefix those of an open task thread or of a folded thread of another brief (`files-overlap`), and a mix of the two kinds — an untasked open beside an open task thread or, for a `code` open of backend `codex`, after a task fold, and a task open beside an open untasked thread (`mixed-open`), refusing at open a Files path the snapshot scope can never hold (`task-files-scope`), and measuring a task thread over its files alone, so `return` and `fold` refuse a changed path that no open or folded task thread on the same base claims (`out-of-files`); `await` waits for that one dispatch to answer and writes nothing — only the finished receipt satisfies it, a run still holding the nonce means keep waiting, and a wait that runs out says so with its own exit code, names whether the deadline or your own timeout ended it, and authorizes nothing: a wait that ended without an answer is a question for you, never permission to dispatch again; `return` absorbs the receipt the wrapper minted — only a finished one, checked against the contract it actually ran, its deadline and its own report, and refused outright when the tree hides changes from git or moves while the return is being computed — and derives the bytes from git rather than from anyone's claim; `fold` is the acceptance, and it refuses if the tree moved since the return — precisely, it re-confirms **equality of the visible canonical payload**, which is a change detector rather than a cryptographic identity of the tree: under the default head base that payload's untracked section is unframed and carries no file mode, so a content or symlink target that imitates the marker opening the next entry can alias two trees, and making an untracked file executable moves nothing (both named as residuals in the mode doc, with the fix queued); under a checkpoint base it is one indexed diff that carries modes. A change set carrying a binary, a non-regular path, a submodule, or a symlink whose target is not valid UTF-8 is **refused** at `return` and `fold` — CONSERVATIVELY, rather than promise what the default head payload may not be able to follow — and the same refusals hold for a checkpoint thread. `degrade` closes a thread that never earned a fold, on the record and with its reason. `aggregate` prints one wave: the registered thresholds, every observation as context, and per step class the finished threads — a folded success with git-provable bytes contributes its ratio, one whose bytes are unprovable is excluded from the mean but still counts in the first-pass rate, and a failed thread, a degrade-closed one, or one folded after failing acceptance each count as a real zero. Below the registered minimum nothing is computed (insufficient); at or above it is computed and labeled PILOT evidence. It REFUSES rather than guessing: no pre-registration, an unfinished thread in scope, a recorded refusal-to-delegate that never opened a thread, or an ambiguous wave each stop the computation by name. `handoff-return --slug <s>` closes the worktree loop after `land --prepare`: it re-attests that main still holds exactly the prepared tree under the same HEAD (both recorded in the handoff at prepare time), prints the satellite handoff's user-owned content byte for byte with its boundaries, byte lengths and the main-owned destinations each part folds into, prints the handoff digest and both OIDs as its proof, states the after-the-fold order (a fold landed after the gates leaves them stale), and records ONE self-reported worktree-stream observation only when the whole prepared change set is measurable — a deletion, a rename, a symlink, a submodule or a mode-only change ends with a named NOT RECORDED instead, never a partial number. Appends only to its own ledger inside the git dir (never committable); it also writes `handoff-return`'s `write-tree` tree object and, for a checkpoint measurement, the leaf's OWN temporary index file and the blobs its forced adds store — never a ref, never the real index; never commits, never runs a subscription CLI. |
|
|
262
262
|
| `/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 — printed only for a link whose RAW TARGET is MAIN's `node_modules`, since a foreign or unreadable link is reported as such with no removal advised); 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); `prompt <slug>` is read-only too and re-prints the satellite's cold-start prompt — the same text `provision` ends its report with: where the worktree is, its ONE seeded plan, the handoff as the one channel back, MAIN's orientation (shared series index, landing) and, under its own heading because it is probed on the satellite, that checkout's install posture — every value derived LIVE rather than replayed from the frozen provision record, a recorded value that no longer matches NAMED as a divergence (with the cause its source makes likely) instead of printed as the runnable one, every offered command marked with WHO runs it (`MAIN $ …` for the landing, `HERE $ …` for this checkout's own install, a posture with nothing to run staying prose), and a control character in any rendered value a typed STOP rather than a forged prompt line; `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. |
|
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". New to the epic, story and task tier? Run `/agent-workflow-kit tier`. A typed decision with a confidence from Jev? Run `/agent-workflow-kit jev`.
|
|
4
4
|
disable-model-invocation: true
|
|
5
5
|
metadata:
|
|
6
|
-
version: '14.
|
|
6
|
+
version: '14.28.0'
|
|
7
7
|
---
|
|
8
8
|
|
|
9
9
|
# agent-workflow-kit
|
|
@@ -2,7 +2,7 @@
|
|
|
2
2
|
name: codex-cli-bridge
|
|
3
3
|
description: Delegate work to the OpenAI Codex CLI (`codex`) under a ChatGPT subscription — run plan/instruction EXECUTION in a sandboxed workspace, or get a read-only ADVISORY review of a plan or working-tree diff — as a second delegated-execution backend beside Antigravity. Use when the user wants to hand a bounded coding task or plan to `codex exec`, get a second-opinion review from codex, install or authenticate Codex CLI, understand its sandbox/network/approval policy, drive codex efficiently from the main agent (exec vs review, resume, the commit boundary), bridge project context (`AGENTS.md`) into codex, or troubleshoot codex flags, models, auth, or its no-TTY headless behaviour.
|
|
4
4
|
metadata:
|
|
5
|
-
version: '5.0.
|
|
5
|
+
version: '5.0.1'
|
|
6
6
|
---
|
|
7
7
|
|
|
8
8
|
# codex-cli-bridge
|
|
@@ -430,7 +430,7 @@ CHATGPT_LOGIN_GUARD="Logged in using ChatGPT"
|
|
|
430
430
|
# receipt this wrapper mints; scripts/release/version-sync.mjs bumps it under the one-anchor-per-file
|
|
431
431
|
# rule, so a release can never leave it behind (the AD-053 drift class).
|
|
432
432
|
AW_RECEIPT_BACKEND="codex"
|
|
433
|
-
AW_BRIDGE_VERSION="5.0.
|
|
433
|
+
AW_BRIDGE_VERSION="5.0.1" # aw-version-anchor
|
|
434
434
|
# The kill grace handed to timeout(1) as --kill-after, and recorded in the receipt as killGraceS:
|
|
435
435
|
# ONE constant, so the number the ledger checks against the dispatch deadline is the number the run
|
|
436
436
|
# actually applied.
|
|
@@ -1495,9 +1495,9 @@ aw_scan_nested_sandbox() { # $1 = rc, $2 = trace path
|
|
|
1495
1495
|
if [[ "$1" -ne 0 ]]; then
|
|
1496
1496
|
if grep -qiE "$AW_NS_MECHANISM" "$2" 2>/dev/null && grep -qiE "$AW_NS_FAILURE" "$2" 2>/dev/null; then
|
|
1497
1497
|
echo "hint: this looks like a NESTED-SANDBOX failure — codex-exec ships its own OS sandbox (bwrap)," >&2
|
|
1498
|
-
echo " which cannot run nested inside a harness sandbox (the FS is read-only). Route codex-exec" >&2
|
|
1499
|
-
echo "
|
|
1500
|
-
echo "
|
|
1498
|
+
echo " which cannot run nested inside a harness sandbox (the FS is read-only). Route codex-exec OUTSIDE it:" >&2
|
|
1499
|
+
echo " the sandbox excludedCommands entry 'codex-exec *' (velocity-profile --bridge-tier adds it where an execute" >&2
|
|
1500
|
+
echo " slot delegates to codex), or a per-run consented bypass for this one run. Do NOT blanket-disable the sandbox." >&2
|
|
1501
1501
|
fi
|
|
1502
1502
|
return 0
|
|
1503
1503
|
fi
|
|
@@ -1508,9 +1508,9 @@ aw_scan_nested_sandbox() { # $1 = rc, $2 = trace path
|
|
|
1508
1508
|
echo " signature. codex-exec ships its own OS sandbox (bwrap), which cannot run nested inside a" >&2
|
|
1509
1509
|
echo " harness sandbox (the FS turns read-only), so the backend most likely could not read what" >&2
|
|
1510
1510
|
echo " it was asked to check: the answer above may be UNGROUNDED — treat it as such rather than" >&2
|
|
1511
|
-
echo " banking it. Re-dispatch OUTSIDE the harness sandbox:
|
|
1512
|
-
echo "
|
|
1513
|
-
echo " sandbox. The exit status stays 0 on purpose — this is a warning, never a gate." >&2
|
|
1511
|
+
echo " banking it. Re-dispatch OUTSIDE the harness sandbox: the sandbox excludedCommands entry 'codex-exec *'" >&2
|
|
1512
|
+
echo " (velocity-profile --bridge-tier adds it where an execute slot delegates to codex), or a per-run consented" >&2
|
|
1513
|
+
echo " bypass. Do NOT blanket-disable the sandbox. The exit status stays 0 on purpose — this is a warning, never a gate." >&2
|
|
1514
1514
|
return 0
|
|
1515
1515
|
fi
|
|
1516
1516
|
done <"$2"
|
|
@@ -342,7 +342,7 @@ aw_scrub_billing_keys() {
|
|
|
342
342
|
# Review-receipt identity (AD-038). AW_BRIDGE_VERSION mirrors this bridge's SKILL.md/capability.json
|
|
343
343
|
# version (drift-guarded by codex-review.test.mjs against capability.json).
|
|
344
344
|
AW_RECEIPT_BACKEND="codex"
|
|
345
|
-
AW_BRIDGE_VERSION="5.0.
|
|
345
|
+
AW_BRIDGE_VERSION="5.0.1" # aw-version-anchor
|
|
346
346
|
# Generous hard cap for a slow review at the host posture's effort (subscription latency varies).
|
|
347
347
|
CODEX_HARD_TIMEOUT="${CODEX_HARD_TIMEOUT:-1800}"
|
|
348
348
|
# Above this assembled-payload size (bytes), the diff goes via a git-dir-local temp
|
package/capability.json
CHANGED
package/package.json
CHANGED
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "@sabaiway/agent-workflow-kit",
|
|
3
|
-
"version": "14.
|
|
3
|
+
"version": "14.28.0",
|
|
4
4
|
"description": "Portable, cross-agent memory & workflow for AI coding agents — Claude Code, Codex, Cursor, Devin Desktop. One command deploys an AGENTS.md entry point + docs/ai context with cap/archive/index enforcement into any repo.",
|
|
5
5
|
"keywords": [
|
|
6
6
|
"ai-agents",
|
|
@@ -20,4 +20,6 @@ The five recipes (defined over each bridge's `provides` roles — `codex`: execu
|
|
|
20
20
|
|
|
21
21
|
**The configured-recipe line (`--active-line`, read-only).** `node ${CLAUDE_SKILL_DIR}/tools/recipes.mjs --active-line` prints exactly **one** machine-composed line: the **CONFIGURED** recipe of every activity/slot, resolved from the target project's `docs/ai/orchestration.json` (read from the current directory) + live readiness — each slot with its source (configured vs computed default), its degradation stated, and its dispatched wrapper set — explicitly contrasted with the readiness **recommendation** (which is informational; the configured recipes are what runs). Paste it verbatim: it fills the session-start discovery step (the deployed `agent_rules.md` §1.1) and the handover "Active recipes:" slot; `set-recipe` echoes the same line after every successful write. A malformed config fails loud (exit 1), never a silent fallback.
|
|
22
22
|
|
|
23
|
+
On the status line, `ready` for a bridge means wired where the project settings enable the sandbox: the segment ` · sandbox: <condition>` states `ready means wired` where the host honors the settings sandbox keys when they enable it, `ready means installed` where they do not, or `unchecked — <reason>` when it cannot be judged. A ready bridge whose sandbox entries are missing shows `✗ not wired — <n> sandbox entr(ies) missing` (`✗ not wired (unused) — <n> sandbox entr(ies) missing` when no slot uses it) or `✗ unchecked — <reason>`. The recommended recipe stays the readiness one, its clause naming each ready bridge whose review surfaces are not wired (`not wired: <alias> (review)`) or unchecked; a bridge missing only its execution wrapper's entries shows in its cell and its not-wired line. After the recommended line, the recipes output lists one `not wired: <alias> — missing …; <condition>; <route>` line per such bridge and one `unchecked: <alias> — <reason>` line per unchecked bridge; the route is the chat yes of `/agent-workflow-kit upgrade` (once a slot uses an unused bridge for review).
|
|
24
|
+
|
|
23
25
|
**Invariants:** read-only · never runs a subscription CLI · never commits · the orchestrator executes the recipe via the bridge skills or the executor vehicle, not the kit.
|
|
@@ -4,7 +4,7 @@
|
|
|
4
4
|
|
|
5
5
|
Requires: ${CLAUDE_SKILL_DIR}/references/shared/report-footer.md
|
|
6
6
|
|
|
7
|
-
The **read-only deployment advisor** — the deterministic section every `upgrade` run ends with, also invocable on its own. It computes what in THIS deployment is configured sub-optimally (allowlist not seeded, autonomy render drifted, sandbox unavailable, gates undeclared, a declared gate matrix that verifies nothing, bridge friction, sandbox-mask clutter,
|
|
7
|
+
The **read-only deployment advisor** — the deterministic section every `upgrade` run ends with, also invocable on its own. It computes what in THIS deployment is configured sub-optimally (allowlist not seeded, autonomy render drifted, sandbox unavailable, gates undeclared, a declared gate matrix that verifies nothing, bridge friction, sandbox-mask clutter, a configured `subagent` carrier whose executor vehicle cannot carry it, a difference from the reference profile the project has not declined, Jev not connected on this host, the Jev skill not installed for every agent) and renders **verdict-first**: one composed verdict line, then each item as **{severity · what is sub-optimal · the benefit in ONE plain line · `lane: chat`, `lane: harness` or `lane: user` · an optional `recipe:` line (the `bridge-tier` missing-entries line, the `worktrees-dir` hand-apply-first grant advice, or the `agents` / `executor-vehicle` preconditions and hidden-mode reconcile follow-up) · a harness item's `check:` line before its apply slot · the apply slot: a chat one-liner, a harness item's apply line or a user item's own step}**. The tool computes deterministic English DATA; **you PRESENT the section in the user's conversational language** — every fact, count and item from the tool, nothing added or dropped; commands, paths, hosts and rule strings stay **byte-exact**; show the raw tool block on request (the AD-032 report-contract lane — the tool cannot know the dialogue language, so the language rendering is your presentation layer).
|
|
8
8
|
|
|
9
9
|
**Live host/session facts are tool-composed only.** Every fact this section states about the current
|
|
10
10
|
host or session — prompts fired, sandbox scope, whether a bypass was needed, network reachability,
|
|
@@ -17,7 +17,7 @@ Run `node ${CLAUDE_SKILL_DIR}/tools/recommendations.mjs --cwd <project-root> [--
|
|
|
17
17
|
|
|
18
18
|
1. **`--cwd` is REQUIRED** — the target project is explicit, never inferred from the shell's current directory (a subdirectory invocation still advises on the named root).
|
|
19
19
|
2. **Verdict first (present-even-when-empty, opening at the `## Recommendations (agent-workflow)` header).** With everything optimal the body is exactly `no recommendations — flow optimal.` — the empty-state line ALONE is the verdict, zero added lines. Every other state opens the body with ONE verdict line composed from the frozen templates: `{K} item(s) need attention` leads when any item's severity is `attention` (a CONFIGURED declaration that is broken, drifted, degrading or invalid — the deployment needs review); `{N} optional recommendation(s), apply any you want` covers the `optional` class (offers to enable an unconfigured capability), led by `nothing is broken` ONLY when no item needs attention AND no probe check was skipped (a skipped probe could hide an attention-class problem — the claim never overreaches); `optimality NOT attested — {M} probe check(s) skipped` is appended last whenever probe checks were skipped. (The `(s)` invariant form IS the pinned pluralization — no singular/plural branching.) Items render attention-first, each tagged `needs attention:` / `optional:`. A failed probe renders as a stated `⚠ skipped item …` line — never a crash, never a fabricated item.
|
|
20
|
-
3. **The apply-through-agent lane — an explicit informed-consent checkpoint:** the agent applies chat and harness items on the chat yes, harness items by the consent route below. A chat item is never retried outside the sandbox and every failure of a chat step is a STOP. A user item, and any apply slot that opens with HAND-APPLY whatever its lane, is the user's own step, handed over as worded. After presenting, OFFER to apply; the user selects items in plain language; for each selected item you **surface its posture note inline** where one exists (the per-item notes below cover exactly the risk-marked keys
|
|
20
|
+
3. **The apply-through-agent lane — an explicit informed-consent checkpoint:** the agent applies chat and harness items on the chat yes, harness items by the consent route below. A chat item is never retried outside the sandbox and every failure of a chat step is a STOP. A user item, and any apply slot that opens with HAND-APPLY whatever its lane, is the user's own step, handed over as worded. After presenting, OFFER to apply; the user selects items in plain language; for each selected item you **surface its posture note inline** where one exists (the per-item notes below cover exactly the risk-marked keys). For a chat item: the user **explicitly confirms**, and **only then** — **no command runs before confirmation** — do you run the rendered command. Do NOT infer safety from the presence or absence of an `--apply` flag: most chat items' rendered command IS the mutation and completes on that one run, **including a no-`--apply` mutation** such as `spec-adoption`'s `ensure-configs --reconcile --only specs`. Some items instead render a **dry-run preview** that changes nothing and prints an explicit follow-up `--apply`/mutating command to run NEXT (e.g. the `gates-declaration` seeder): after the SAME confirmation you run that printed follow-up command — no second ask, no improvisation (each writer keeps its own consent semantics: previews, `--apply` flags, refusals). An item marked **HAND-APPLY** (e.g. `enforcement` or `read-lane.stale`) is **the user's own step** — hand the user its rendered line together with its posture note; HAND-APPLY is **maintainer territory wherever it renders** — the apply slot or the `recipe:` line. For `worktrees-dir` the two classes COMPOSE in one item and the ORDER is part of the contract: its `recipe:` line is the HAND-APPLY grant advice, its apply one-liner is the dry-run ack preview, and you **wait for the maintainer to confirm the grant is applied (or the terminal fallback chosen)** before running that preview and the exact `--apply` command it prints — the ack RECORDS a choice that must already have been made, and against a trusted host NO the item renders no ack at all.
|
|
21
21
|
4. Registry strings (benefits + item texts) are frozen tool data, fact-true, ONE line under the pinned shape cap — posture/risk detail lives in the notes below at the consent moment, never inline in the overview. The dual velocity+security wording (`safer — blast radius bounded by the OS sandbox, not human attention`) rides ONLY the items with a real security delta (the autonomy render, the sandbox provisioning); the bridge-wrappers item claims **velocity only**.
|
|
22
22
|
|
|
23
23
|
The **consent route** for a harness item: run its `check:` line, show its result, take the user's yes on that result and run its `apply:` line with that result's digest, in the sandbox first. On exactly a write the OS refuses on a protected file, a writer's report that the sandbox masks its target and nothing was written (the mcp masked report), or the check's `masked:` line, say so and run that same line once more outside the sandbox, so the harness asks at the point of action and the check re-judges in the apply's own environment; never another line, never unseen. After a `masked:` check, the yes is taken on the outside result. A check that refuses (exit 1) leaves the item unapplied and pending, its result shown; a changed digest (exit 5) applies nothing, shows the new result and asks a new yes. A no, a harness refusal and any other failure of a harness apply leave the item pending with its output shown, and the agent continues. `init`, run in a console from the project's folder, is the route the user may take instead.
|
|
@@ -28,11 +28,10 @@ The **consent route** for a harness item: run its `check:` line, show its result
|
|
|
28
28
|
|
|
29
29
|
A variant's posture note, where its key has one, is printed by `init` from this installed mode doc (the one source); a variant without a note shows its preview, and a harness variant also the check's lines.
|
|
30
30
|
|
|
31
|
-
- `bridge-tier` — the lane is harness and env-dependent: the agent applies the item on the chat yes after the check, writing the project `.claude/settings.json`; `init` previews `velocity-profile --bridge-tier` and applies it on the terminal y as the route the user may take instead. It
|
|
32
|
-
- `sandbox-lane` — surface this note TOGETHER with the sandbox-lanes ladder below (the ladder IS the practical half of the note — inline, never a pointer). Pure DISCOVERABILITY: it surfaces the manifest-declared observed session-sandbox recipe (egress hosts ∪ resolved writable state dirs — `networkHosts` ∪ `writableDirs` of the wired bridges' `capability.json`, the single documentation source) and converges on a NEUTRAL fingerprint acknowledgement recorded by the consent-gated **ack writer** into the family-owned `docs/ai/acks.json` (`sandboxLaneAck`; a changed recipe re-fires the item). The store is family-owned so no host settings validator guards it (AD-055 relocated the ack off the Claude Code settings schema, which rejected the unknown key); the legacy `"agentWorkflow": { "sandboxLaneAck": … }` settings-scope key is still READ for one deprecation window (until the next kit MAJOR). It never claims the settings security keys take effect on any host class; the tier seeds `sandbox.network.allowedDomains` with the placed bridges' declared hosts only through the consented `--bridge-tier` (AD-172, which reverses the bridge council of 2026-07-11 for those hosts alone; the egress they widen for every sandboxed command is stated at that consent), and `sandbox.filesystem.allowWrite` stays unseeded (that council: a write allowance on CLI state dirs would expose credential dirs). Posture history: an IDE-managed session sandbox was live-observed (2026-07-11/12) ignoring hand-applied settings security keys in BOTH scopes, and codex needs a writable HOME (EROFS `~/.codex` in-sandbox); whether a session's sandbox honors the settings keys is runtime-unknowable from the advisor (a denial-only signal) — which is exactly why the item states only detectable facts and no zero-prompt promise on any host class.
|
|
31
|
+
- `bridge-tier` — the lane is harness and env-dependent: the agent applies the item on the chat yes after the check, writing the project `.claude/settings.json`; `init` previews `velocity-profile --bridge-tier` and applies it on the terminal y as the route the user may take instead. It wires the USED bridges from the project's effective recipes and host readiness: each used review wrapper's `code`-mode allow rule (a wrapper then runs UNATTENDED: it reads any repo file it is pointed at and sends the assembled payload to its subscription backend), the quoted grounding pre-step rule when `agy-review` is used, `codex-review *` / `agy-review *` in `sandbox.excludedCommands`, and `codex-exec *` where an execute slot delegates (no execution allow rule: it keeps its prompt). It also seeds the used bridges' declared hosts (`networkHosts`) in `sandbox.network.allowedDomains` and resolved state dirs (`writableDirs`), home-relative where under home, in `sandbox.filesystem.allowWrite`. Where the host honours the settings sandbox keys, an excluded wrapper runs outside the sandbox, every sandboxed command of the project can reach the seeded hosts, and every sandboxed command can write the seeded state dirs. The item fires while an allow rule, a `*` exclusion, a host or a state dir of a used bridge is missing, so a project wired before this revision is offered the one re-run. Its `recipe:` line names each missing entry by kind (allow rules, exclusions, hosts, state dirs), then what the execution wrapper `codex-exec` needs where an execute slot delegates, then, when the project settings enable the sandbox, "once added, wired where the host honors the settings sandbox keys", or "host condition unchecked" with the reason.
|
|
33
32
|
|
|
34
33
|
- `read-lane` — enabling the opt-in read-only compound lane auto-approves *compounds* (and singles) of the seeded read-only core that carry ZERO shell metaprogramming: an UNATTENDED trust extension, bounded by the audited read-only core (never a command outside it; prompt-bypass only, never a sandbox bypass) and applied regardless of which of those core commands you seeded as individual settings rules. It is a PROJECT-PERSISTENT declaration in `docs/ai/lanes.json` — every future session, subagents' Bash too where the host fires hooks on subagent Bash, and (committed) every checkout. The apply depends on state: when the lane is OFF, it is the `gate-hook --read-lane` preview (whose own currency check refuses a stale hook — a pre-1.48 hook never reads `lanes.json`); when the placed hook is STALE (an enabled lane over an old hook) or MISSING, the item instead surfaces a **delete-to-reseed** / re-place recovery (a destructive `rm` + `--apply`, an attention item — never the safe preview). Risk profile: a bounded read-only trust-posture extension — no write/exec exposure beyond the audited core.
|
|
35
|
-
- `worktrees-dir` — on a settings-native host that honors the key, the HAND-APPLY line widens the OS-sandbox WRITE surface to the whole worktrees parent dir: every sibling path under it (other repositories included) becomes agent-writable, and the widening persists for every later session. A harness-managed host may ignore that project setting; grant the narrow parent through host/session controls or use the provision terminal fallback instead. When that scope is wider than you want, narrow it FIRST: create a dedicated dir yourself (outside the agent's write surface), point `docs/ai/worktrees.json` `parentDir` at it, then re-run recommendations — the item re-renders with the narrowed dir. The kit
|
|
34
|
+
- `worktrees-dir` — on a settings-native host that honors the key, the HAND-APPLY line widens the OS-sandbox WRITE surface to the whole worktrees parent dir: every sibling path under it (other repositories included) becomes agent-writable, and the widening persists for every later session. A harness-managed host may ignore that project setting; grant the narrow parent through host/session controls or use the provision terminal fallback instead. When that scope is wider than you want, narrow it FIRST: create a dedicated dir yourself (outside the agent's write surface), point `docs/ai/worktrees.json` `parentDir` at it, then re-run recommendations — the item re-renders with the narrowed dir. The kit does not write this worktrees grant itself; the line is always yours to paste. **Convergence** is two-path: a declared `sandbox.filesystem.allowWrite` entry covering the probed dir (either settings scope — `~` and `~/…` resolve against home, and coverage is path-segment-aware, so a grant on a sibling or on a child never counts), or — for a host that ignores that key — the neutral dir-bound acknowledgement recorded by this item's consent-gated apply one-liner (`ack-write --lane worktrees-dir` — a dry-run preview that prints the exact `--apply`; recorded as `worktreesDirAck` in the family-owned `docs/ai/acks.json`, never a security key), while the grant advice itself rides the `recipe:` line as the labeled FIRST step; against a trusted host NO the apply stays the HAND-APPLY grant advice and no ack is offered. Neither is proof of write CAPABILITY: the provision preflight's real create+delete probe stays the runtime truth, and the fingerprint is bound to the **resolved probe dir**, so the item re-fires only when that resolved dir changes (two absent `parentDir` values sharing an existing ancestor resolve to the same dir and keep the same ack). Risk profile: a real write-surface widening where honored — scope it deliberately.
|
|
36
35
|
|
|
37
36
|
- `gates-inert` — the gate matrix is DECLARED but verifies nothing, and the two causes have different remedies, so the item renders a different apply for each. **Cause A** — a canonical `coverage-check` gate with no producer gate declared BEFORE it (none at all, or one declared after it, which writes the lcov too late): the run certifies no coverage of its own — `coverage=not-run` when nothing wrote an lcov, or a verdict over STALE bytes an earlier run left in the git dir, which is worse because it reads as `coverage=certified`. This item is what surfaces that state at upgrade instead of leaving it to be noticed. The remedy needs the producer to run BEFORE the checker — declared or MOVED there — and what the fill can do about that is exactly what the apply says. When the checker is the LAST declared gate and the project's own scripts yield an offerable producer, the fill PLACES that producer before it, so the apply is the ordinary `gates-init` dry-run preview. Otherwise — no offerable producer, or a checker that is not last — this arm stays **HAND-APPLY**: the maintainer's own step is to edit `docs/ai/gates.json` (the exact producer form lives in `${CLAUDE_SKILL_DIR}/references/modes/gates.md`), because the fill never reorders entries it did not write. **Cause B** — every declared gate is one of the kit's own canonical checkers, so the matrix runs no project-verification command at all: the apply is the `gates-init` dry-run preview, the same consent-gated seeder the `gates-declaration` item renders, and after the SAME confirmation you run the `--apply` line it prints. Neither cause changes any gate result, exit code or receipt — the item adds no enforcement, only the offer. Risk profile: `docs/ai/gates.json` is a PRIVILEGED file (a declared gate is auto-approvable by the wired hook), so both causes land in the same place — where the fill can help, it declares a project-controlled command on your explicit yes; where it cannot, the edit stays maintainer-only.
|
|
38
37
|
|
|
@@ -47,16 +46,16 @@ A variant's posture note, where its key has one, is printed by `init` from this
|
|
|
47
46
|
- `jev-skill` — the lane is harness; both `jev-skill` and `jev-skill.earlier` are env-dependent: the agent applies the item on the chat yes after the check, showing the files it would write; it changes no harness setting. `init` applies it on the batch terminal yes as the route the user may take instead. When the recipe line opens with `run the apply in <place>`, that place is where a user who prefers their own terminal runs it. The skill is third-party text — the vendor's, MIT — that the kit ships pinned and installs as a verified copy into three agent skill roots: `~/.agents/skills`, the Claude Code skills dir (`CLAUDE_CONFIG_DIR` or `~/.claude`) and `~/.gemini/config/skills`. It is never updated silently: a copy at an earlier pin is replaced only by that command, and a foreign copy is left untouched and named on the recipe line. A copy the vendor's `skills` CLI wrote with the same bytes reads current, and a later pin move replaces it like a kit copy; a vendor plugin install is invisible to the file check, so the Claude Code root may read absent while the plugin serves it. Only when the user asks to see the plan does the agent run the command without `--apply` (a read-only dry run). The decline is the HAND-APPLY alternative on the recipe line, asked apart BEFORE the confirmation: on "install" run the check line and, on the yes given on its result, the apply line by the consent route, on "decline" run nothing and hand over the recipe line; the decline is recorded per project as `jevSkillAck` in `docs/ai/acks.json`. Risk profile: third-party text written into the user's agent skill roots by the kit's skill command; no network.
|
|
48
47
|
- `adr-store-migration` — other items write project files too; what is unique here is that the crossing **overwrites and deletes files the project already has**: it replaces the deployed enforcement scripts in `scripts/` (the directional subset — only basenames the project already has; a locally-edited copy is snapshotted first, never silently clobbered) and, where a retired archive file exists, DELETES it once conservation has been proven. That is why it is **HAND-APPLY** and why the command shown in the apply slot is a **`--dry-run`** — it writes nothing and prints the whole plan. `--apply` is a SEPARATE step, run only after that plan has been shown and **fresh consent** obtained for it; this item is the user's own step, with its preview and apply confirmed separately. Every write is idempotent and the run is re-runnable to completion after any interruption, so a re-run repairs rather than double-applies. It never commits. Risk profile: overwrite + delete of existing project files, gated on a preview you have actually read.
|
|
49
48
|
|
|
50
|
-
**Sandbox lanes (what to DO with the
|
|
49
|
+
**Sandbox lanes (what to DO with the bridge surfaces, per host class):**
|
|
51
50
|
|
|
52
51
|
The sandbox items state: on native Windows Claude Code runs no sandbox: this item takes no effect there. `sandbox-provision.installable` is env-dependent and carries the user's own step for a sudo install; the bare `sandbox-provision` keeps its current text.
|
|
53
52
|
Only Claude Code on Linux/WSL2 is measured; Codex, Devin and agy as hosts and macOS are unmeasured.
|
|
54
53
|
A first session may meet ordinary prompts until the autonomy render lands; a chat apply is promptless only where the host auto-allows sandboxed commands.
|
|
55
54
|
|
|
56
|
-
- **Settings-native sandbox** (the harness reads the `.claude/settings.json` sandbox keys): where the harness honours the `--bridge-tier` wiring's `excludedCommands
|
|
57
|
-
- **Harness-managed sandbox** (an IDE/session-imposed sandbox that ignores the settings keys) — a NARROWEST-SCOPE ladder: (1) prefer wrapper/command-SCOPED sandbox rules where the harness offers them (only the review wrappers gain the egress + state-dir writes); (2) a SESSION-scoped allowance (the manifest hosts + the resolved state dirs for the whole session) is an INFORMED WIDENING — it carries the same blast-radius class the
|
|
58
|
-
- Either way the recipe is **observed-minimal, honestly incomplete** — a blocked host names itself at run time;
|
|
55
|
+
- **Settings-native sandbox** (the harness reads the `.claude/settings.json` sandbox keys): where the harness honours the `--bridge-tier` wiring's `excludedCommands` entries (`codex-review *` / `agy-review *`, and `codex-exec *` where an execute slot delegates, keeping its prompt), the used bridges' wrappers run OUTSIDE the sandbox and never consult `allowedDomains`/`allowWrite` — nothing beyond the tier. A bare name matches only the argument-free command: observed 2026-10-06, Claude Code 2.1.291 ran an agent's plain `codex-review` inside its sandbox beside the bare entry; a harness may keep a `*`-excluded wrapper sandboxed too (unmeasured). There a recipe host outside `allowedDomains` is refused by the harness proxy or held for a network prompt (a refusal reads `Tunnel connection failed: 403` or `CONNECT tunnel failed, response 403` in the run). The lane for the hosts AND the state dirs is the `--bridge-tier` re-run: it seeds the used bridges' declared hosts into `sandbox.network.allowedDomains` and resolved state dirs into `sandbox.filesystem.allowWrite`, and the `bridge-tier` item offers it while any used-bridge surface is missing. Where the host honours the settings sandbox keys, every sandboxed command of the project can reach those hosts and every sandboxed command can write those state dirs.
|
|
56
|
+
- **Harness-managed sandbox** (an IDE/session-imposed sandbox that ignores the settings keys) — a NARROWEST-SCOPE ladder: (1) prefer wrapper/command-SCOPED sandbox rules where the harness offers them (only the review wrappers gain the egress + state-dir writes); (2) a SESSION-scoped allowance (the manifest hosts + the resolved state dirs for the whole session) is an INFORMED WIDENING — it carries the same blast-radius class the `--bridge-tier` keys carry where a host honours them (every sandboxed command in the session gains that egress and write access to those state dirs), acceptable only as the maintainer's deliberate choice for a review-heavy session. (3) Else the per-run consented bypass. The kit writes no shape for this sandbox — the harness's own controls (its scoped rules, a session allowance, its consent at the point of action) govern there; an IDE-managed session sandbox was live-observed (2026-07-11/12) ignoring the settings security keys in BOTH scopes, and whether a session's sandbox honours them is not knowable from the advisor.
|
|
57
|
+
- Either way the recipe is **observed-minimal, honestly incomplete** — a blocked host names itself at run time; the manifests record what was observed.
|
|
59
58
|
|
|
60
|
-
**Invariants:** read-only (never writes, never commits, never runs a subscription CLI) · `--cwd` required · present-even-when-empty · verdict-first from frozen templates · probe failures degrade to stated skip lines · chat apply one-liners are cwd-independent (absolute tool paths + a pinned `--cwd`; the ONE exception is the `set-autonomy` item, a conversational skill invocation explicitly labeled *run IN the target project*) · the bare `sandbox-provision` item pins via a `cd <root> &&` prefix — the doctor reads its cwd · harness items carry a check and an apply bound to its digest; user items, including `family-freshness`, carry the user's own step, and no-writer variants keep their text · `sandbox.filesystem.allowWrite`
|
|
59
|
+
**Invariants:** read-only (never writes, never commits, never runs a subscription CLI) · `--cwd` required · present-even-when-empty · verdict-first from frozen templates · probe failures degrade to stated skip lines · chat apply one-liners are cwd-independent (absolute tool paths + a pinned `--cwd`; the ONE exception is the `set-autonomy` item, a conversational skill invocation explicitly labeled *run IN the target project*) · the bare `sandbox-provision` item pins via a `cd <root> &&` prefix — the doctor reads its cwd · harness items carry a check and an apply bound to its digest; user items, including `family-freshness`, carry the user's own step, and no-writer variants keep their text · the used bridges' `sandbox.filesystem.allowWrite` and `sandbox.network.allowedDomains` are seeded only by the consented `velocity-profile --bridge-tier`; the worktrees grant keeps its hand-apply step · the `worktrees-dir` item converges on a declared `allowWrite` entry covering its probed dir, or on its own `worktreesDirAck` lane in `docs/ai/acks.json` (recorded by its apply one-liner, never by the `recipe:` line).
|
|
61
60
|
|
|
62
61
|
**Exit codes:** `0` report rendered (items or the empty state); `1` error (e.g. `--cwd` is not a directory); `2` usage.
|
|
@@ -42,7 +42,7 @@ Run **`node ${CLAUDE_SKILL_DIR}/tools/set-recipe.mjs [--set <activity>.<slot>=<v
|
|
|
42
42
|
`parallel` is a **flag**, not a recipe: it resolves outside the recipe lattice and is never degraded. Examples: `--set routine.carrier=subagent`, `--set routine.parallel=off`, `--set plan-authoring.author=subagent`, `--set plan-authoring.fold=subagent`. A value outside its slot's list is a **usage** error (`2`) naming the accepted values, never a coercion to a neighbour.
|
|
43
43
|
2. **Preview by default** — prints `current → proposed` for the **changed** slots only, plus the **effective recipe resolved against live carrier readiness** (degradation stated honestly, e.g. *council requested, 1 ready reviewer → runs reviewed until a 2nd backend is ready*; *subagent requested, the executor vehicle is `missing` → runs solo until the vehicle is placed, naming the apply command*). A `parallel` value is reported as requested — a flag has nothing to degrade against. It writes **nothing**. Re-run with **`--write`** to apply (same effective/degradation note — a direct `--write` is never quieter than the preview). `--unset` returns a slot to its computed default (reverting needs no hand-edit either). A no-op `--set` (slot already equals) writes nothing and never re-seeds the onboarding note.
|
|
44
44
|
3. **`--write`** applies via a hardened, atomic write (deployment-gated — refuses to scatter a config into a repo with no `docs/ai`; exclusive-create temp + rename; symlink/TOCTOU-safe; last-writer-wins). It preserves the onboarding note + every untouched slot, normalizing to canonical 2-space JSON.
|
|
45
|
-
**Standing-consent advisory (after a successful `--write` only, and only when the written config names a `reviewed
|
|
45
|
+
**Standing-consent advisory (after a successful `--write` only, and only when the written config names a `reviewed`, `council` or `delegated` recipe — a solo recipe gets NO advisory):** the upgrade's Recommendations `bridge-tier` item wires the used bridges on its chat yes — the used review wrappers' code-mode allow rules (and agy-review's quoted grounding pre-step rule), the `<wrapper> *` exclusions (`codex-exec *` where an execute slot delegates; it keeps its prompt), the hosts and the state dirs. Plan and diff review modes keep their prompt. State plainly: auto-approving a review wrapper **spends subscription quota without a per-run prompt** — that is exactly what standing consent means here, so it is the maintainer's call. The grounding pre-step writes one scratch facts file, covered by that consent; its rule matches the double-quoted path the procedures advisor renders.
|
|
46
46
|
4. **Exit codes:** `0` success (an explicit recipe that gracefully degrades is still `0`); `2` usage (a bare/duplicate op, or `--write` with no ops); `1` config error (malformed/unreadable config — the file is left **untouched**, never clobbered) or a write STOP (no deployment / a symlinked config). A `1`/`2` failure is loud; on a malformed config, offer to show the parse error so you can help the user fix the JSON.
|
|
47
47
|
|
|
48
48
|
Output is **English/structured** — **localize it to the user's conversational language** when you narrate. Surface the effective-recipe/degradation note plainly.
|
|
@@ -4,7 +4,6 @@
|
|
|
4
4
|
<!-- opt-in-capability: kit-tools-tier -->
|
|
5
5
|
<!-- opt-in-capability: bridge-tier -->
|
|
6
6
|
<!-- opt-in-capability: autonomy-render -->
|
|
7
|
-
<!-- opt-in-capability: sandbox-lane -->
|
|
8
7
|
<!-- opt-in-capability: accept-edits -->
|
|
9
8
|
|
|
10
9
|
Requires: ${CLAUDE_SKILL_DIR}/references/shared/command-shapes.md
|
|
@@ -46,15 +45,15 @@ Honesty notes: tier entries get **NO PreToolUse-hook residual coverage — with
|
|
|
46
45
|
|
|
47
46
|
**Invariants:** creates `.claude/` if absent and writes **only** `.claude/settings.json` (no other file); **never** allowlists commit/push/publish; **never** writes `settings.local.json`; never commits; opt-in `acceptEdits`, never silent.
|
|
48
47
|
|
|
49
|
-
**Every settings-derived RUNTIME claim on this mode's surfaces is host-conditional.** Whether a host applies the `sandbox.*` settings keys is **not knowable from here** — a settings-native host honors them; an IDE/session-imposed sandbox was observed ignoring
|
|
48
|
+
**Every settings-derived RUNTIME claim on this mode's surfaces is host-conditional.** Whether a host applies the `sandbox.*` settings keys is **not knowable from here** — a settings-native host honors them; an IDE/session-imposed sandbox was observed ignoring `sandbox.network.allowedDomains` / `sandbox.filesystem.allowWrite` in BOTH scopes. So every statement about what a settings key DOES at run time is stated **`where the host honors the settings sandbox keys`**, never promised — the bridge tier's exclusion routing; the autonomy render's `allowedDomains` / `allowWrite` / `allowUnsandboxedCommands` / `excludedCommands` degrade lines over FOREIGN keys; and equally the render's claims about the keys it writes ITSELF (the `sandbox:` line, the `fs_outside_repo` confinement note, the network prompt-on-egress and credentials-coverage degrades), which is where a clean deployment reads a promise. The qualifier rides every such line, and one notice naming the unknown closes **every** `--autonomy` render, clean settings included. A malformed `allowWrite` (not an array, or an entry that is not a non-empty string) is reported as unreadable rather than assumed empty — as UNVERIFIABLE, never as a weakening: nothing may be claimed about what a value the tool cannot read would widen.
|
|
50
49
|
|
|
51
50
|
**The boundary of that qualifier, stated so it is not guessed.** It covers claims about `sandbox.*` keys — the block whose honouring was observed to vary by host. It deliberately does NOT cover `permissions.*` semantics (an allow rule auto-approving a command, `local > project` precedence for a non-sandbox key): that is the harness's own documented permission model, the layer this whole mode rests on, and hedging it would say nothing true. **Stated residual:** which sentences count as settings-derived runtime claims is decided by review, not by a checker — deciding whether a sentence asserts a runtime effect is a question about prose meaning, and this family deliberately has no mechanism that judges that (the same limit that retired the byte-scanning deny rung). A new claim-bearing line therefore has to carry the qualifier by authorship; the acceptance criterion is that every line stating what a `sandbox.*` key DOES carries it, and the tests pin the surfaces that exist today. What does NOT change is the CLASSIFICATION: which red-line a key would weaken, and that a proven bridge-tier exclusion is a note rather than a weakening, are properties of the declaration and stay flat statements.
|
|
52
51
|
|
|
53
|
-
**The `--bridge-tier` (own opt-in, AD-044).** Seeds
|
|
52
|
+
**The `--bridge-tier` (own opt-in, AD-044).** Seeds the USED bridges from every recipe's review and execute slots in `docs/ai/orchestration.json`, resolved over host readiness to the effective recipe: a degraded recipe uses its resolved providers; an absent config takes the computed default. A bridge no slot uses gets nothing. Per used review role it seeds `Bash(<wrapper> code:*)` into `permissions.allow` and, with agy-review used, the quoted grounding pre-step rule. Per used role it seeds `<wrapper> *` into `sandbox.excludedCommands`, the manifest's `networkHosts` into `sandbox.network.allowedDomains`, and its resolved `writableDirs` into `sandbox.filesystem.allowWrite`, home-relative under the home and absolute outside it, with env overrides honoured. All entries land in the **project** `.claude/settings.json`, preserving existing entries and sibling keys. **Where the host honors the settings sandbox keys**, an exclusion means the wrapper runs outside the sandbox; a host means every sandboxed command of the project can reach it; a dir means every sandboxed command can write it — each widening is part of this consent. A session-imposed sandbox may ignore those keys; its lane is in `${CLAUDE_SKILL_DIR}/references/modes/recommendations.md`. An exclusion placed only in `settings.local.json` was live-observed NOT to route the command outside the sandbox (2026-07-11: the wrapper then starts sandboxed and dies on a read-only HOME), so hand-wiring the local file is no substitute for this tier. Under a harness-managed session sandbox (live-observed 2026-07-11/12) an IDE-managed sandbox ignores `sandbox.network.allowedDomains` and `sandbox.filesystem.allowWrite` in BOTH settings scopes — its own per-host consents govern egress, and the durable lanes there are the session sandbox config (the hosts plus the CLI state-dir writes) or the per-run consented bypass. Each bridge's observed egress hosts are declared in its `capability.json` `networkHosts` — the single documentation source (this doc retypes no host list); an apex domain is NOT covered by its `*.`-wildcard form, so the manifests carry both forms where observed. The upgrade Recommendations advisor's `bridge-tier` item offers this tier while any used-bridge surface is missing, its `recipe:` line naming each missing entry; an acknowledgement the retired session-sandbox-recipe item recorded in `docs/ai/acks.json` is never read, relabelled or removed — it keeps the meaning "seen". **Bare-entry reading (measured 2026-10-06):** an exclusion uses Bash rule syntax; a bare name matches only the argument-free command, so the tier writes the form with ` *`; an earlier bare entry stays and does not satisfy that surface. Where an execute slot delegates, codex-exec gets its exclusion, hosts and state dirs, but no allow rule, so a delegated run keeps its prompt. Only the code review mode is auto-allowed; plan/diff keep their prompt. The grounding rule uses the double-quoted absolute kit path the procedures advisor renders; an unseedable path is a stated skip. Its writer guard accepts only system-temp scratch or a fresh gitignored in-repo destination, refusing tracked, non-ignored, existing in-repo, outside-repo non-temp and symlink/non-regular destinations. The seeded byte-forms (this list IS the documented-invocation source for the bridge tier):
|
|
54
53
|
|
|
55
|
-
- `Bash(codex-review code:*)` — the code-mode
|
|
56
|
-
- `Bash(agy-review code:*)` — the code-mode
|
|
57
|
-
- `
|
|
54
|
+
- `Bash(codex-review code:*)` — the review code-mode allow rule; `codex-review *` — its sandbox.excludedCommands entry.
|
|
55
|
+
- `Bash(agy-review code:*)` — the review code-mode allow rule; `agy-review *` — its sandbox.excludedCommands entry; `Bash(node "${CLAUDE_SKILL_DIR}/tools/grounding.mjs":*)` — its quoted grounding pre-step rule.
|
|
56
|
+
- `codex-exec *` — the sandbox.excludedCommands entry for a used execute role; no allow rule.
|
|
58
57
|
|
|
59
58
|
**Consented posture (printed on every `--bridge-tier` run, stated not pretended away):** an auto-allowed review wrapper runs **UNATTENDED** — it reads any repo file it is pointed at and sends the assembled payload to its **subscription backend**; prefix rules cannot inspect arguments, so a code-mode argument that names a readable file (agy's `--facts`/`--decided`) rides the same consent. That is the tier's purpose (unattended council review runs) and its residual. Tier entries get no PreToolUse-hook coverage — the same documented residual class as the autonomy red-line rules. The grounding entry's writer surface is bounded by `grounding.mjs`'s own scratch-destination guard (a tracked or in-repo-not-ignored `--out` is refused by the tool itself).
|
|
60
59
|
|
|
@@ -1,8 +1,7 @@
|
|
|
1
1
|
### The one-line backend-status line (shared by bootstrap + upgrade)
|
|
2
2
|
|
|
3
|
-
Bootstrap (step 11) and **every** successful `upgrade` exit (steps 4 + 8)
|
|
4
|
-
|
|
5
|
-
run the status-line composer and **paste its single emitted line verbatim**:
|
|
3
|
+
Bootstrap (step 11) and **every** successful `upgrade` exit (steps 4 + 8) share a read-only,
|
|
4
|
+
**machine-composed** backend summary — **paste its single emitted line verbatim**:
|
|
6
5
|
`node ${CLAUDE_SKILL_DIR}/tools/recipes.mjs --status-line`
|
|
7
6
|
The tool runs the backend detector and appends the recipe recommendation itself
|
|
8
7
|
(`composeStatusLine`); the agent **composes nothing factual** — no readiness token, no glyph, no
|
|
@@ -10,6 +9,11 @@ recipe clause of its own, ever.
|
|
|
10
9
|
|
|
11
10
|
- **Placeholder template (structure only — never copy this example; paste the tool's line):**
|
|
12
11
|
`backends: <alias> <✓|✗> <readiness> · <alias> <✓|✗> <readiness> — run /agent-workflow-kit backends · recipes: <recommendation clause> — see /agent-workflow-kit recipes · autonomy: <per-activity levels> (<policy state>)`
|
|
12
|
+
- **Ready bridge wiring cells (placeholders only):** `<alias> ✗ not wired — <n> sandbox entr(ies) missing`,
|
|
13
|
+
`<alias> ✗ not wired (unused) — <n> sandbox entr(ies) missing`, `<alias> ✗ unchecked — <reason>`.
|
|
14
|
+
The ` · sandbox: <condition>` segment follows `— see /agent-workflow-kit recipes`: enabled in the project
|
|
15
|
+
settings — `ready means wired` where the host honors the settings sandbox keys; not enabled in the project
|
|
16
|
+
settings — `ready means installed`; `unchecked — <reason>`.
|
|
13
17
|
- The **`autonomy:` segment is appended by the composer itself** (`composeAutonomyFacts`, AD-044) —
|
|
14
18
|
per-activity levels + the policy state (computed defaults / declared / MALFORMED, loud) — the
|
|
15
19
|
agent never types it; the composer resolves the policy from the project root, so the line is
|
|
@@ -19,18 +23,15 @@ recipe clause of its own, ever.
|
|
|
19
23
|
**never blank**: both backends ready → *"Council available, Reviewed the everyday default"*; one
|
|
20
24
|
ready → *"Reviewed available (via …)"*; **none installed → *"Solo — run /agent-workflow-kit setup to
|
|
21
25
|
add a backend"***; a backend present-but-not-ready → Solo with that backend's specific remedy.
|
|
22
|
-
(
|
|
26
|
+
(`recommendWiredRecipe` appends ` · not wired: <alias> (review) — <route>` and
|
|
27
|
+
` · unchecked: <aliases>` to the readiness clause in the same tool run.)
|
|
23
28
|
- **Invariants:** **read-only · never blocks the commit gate · never runs a subscription CLI · the
|
|
24
29
|
pointer is the in-agent `backends` mode, never a network fetch · the appended `recipes:` clause
|
|
25
30
|
routes to the in-agent `recipes` mode the same way · `init`/npx never *places* bridges (it
|
|
26
31
|
refreshes only what `setup` already placed, AD-011 §5).**
|
|
27
|
-
- **Composer unavailable → skip with a stated reason, never silently.**
|
|
28
|
-
|
|
29
|
-
|
|
30
|
-
project has no Node runtime". On that condition, skip the line and say so **with the concrete
|
|
31
|
-
reason**, e.g. *"Couldn't run the backend status-line composer here (node is not on the agent host
|
|
32
|
-
PATH), so I'm skipping the backend-status line."* — never a silent skip (Hard Constraint — no
|
|
33
|
-
silent failures).
|
|
32
|
+
- **Composer unavailable → skip with a stated reason, never silently.** This Node script runs on the
|
|
33
|
+
**agent host**: skip only when the host lacks `node` on PATH or the tool errors — **not** "the
|
|
34
|
+
project has no Node runtime". Skip the line and state the concrete reason — never a silent skip.
|
|
34
35
|
|
|
35
36
|
### The version block + welcome mat (shared by bootstrap + upgrade)
|
|
36
37
|
|
|
@@ -46,13 +47,11 @@ line), not a fresh helper call. **Beside the backend-status line**, when the tar
|
|
|
46
47
|
documents it; same agent-host skip-with-reason contract as the status line). Present everything in the
|
|
47
48
|
user's conversational language; never paste the JSON or any internal field name.
|
|
48
49
|
|
|
49
|
-
**Success state — the happy path never leads with a structure number.**
|
|
50
|
-
|
|
51
|
-
vocabulary — that number is inert here and only confuses; it belongs to *Version disclosure* (below).
|
|
50
|
+
**Success state — the happy path never leads with a structure number.** Omit the internal `docs/ai`
|
|
51
|
+
structure version, stamp filename and versioning vocabulary; see *Version disclosure* below.
|
|
52
52
|
Frame the success itself plainly, in the **user's conversational language** (never hardcode a phrase):
|
|
53
53
|
- a **zero-diff no-op `upgrade`** (step 4) → **settings already current — no update is required**
|
|
54
|
-
(
|
|
55
|
-
language, in your own words);
|
|
54
|
+
(convey this meaning in the user's conversational language, in your own words);
|
|
56
55
|
- a **fresh `bootstrap`** → its normal "deployed and ready" success, minus the number.
|
|
57
56
|
|
|
58
57
|
**Version block — the installed package versions, fed from `--json`** (the `docs/ai` structure version
|
|
@@ -69,12 +68,9 @@ that skill slot" · `placeholder` → "a placeholder, not a working install" ·
|
|
|
69
68
|
but its manifest didn't validate" · `unsupported` → "installed but its manifest schema is too new for
|
|
70
69
|
this kit" · `uncheckable` → "couldn't be checked (a permission error)".
|
|
71
70
|
|
|
72
|
-
**Helper-failure contract (mirror the backend-status line).** The
|
|
73
|
-
|
|
74
|
-
|
|
75
|
-
e.g. *"Couldn't run the family-registry helper here (node is not on the agent host PATH), so I'm
|
|
76
|
-
skipping the installed-versions block."* — never a silent skip (Hard Constraint). It is non-essential:
|
|
77
|
-
the rest of the report — and the commit gate — proceeds.
|
|
71
|
+
**Helper-failure contract (mirror the backend-status line).** The **agent host** runs family-registry.
|
|
72
|
+
If it lacks `node` on PATH or the helper errors, **skip the version block and say so with the concrete
|
|
73
|
+
reason** — never a silent skip. The rest of the report — and the commit gate — proceeds.
|
|
78
74
|
|
|
79
75
|
**Welcome mat — the last line(s) of the footer.** After the version block and the backend-status
|
|
80
76
|
line, print *"Run `/agent-workflow-kit help` to see every command."* and *"New to the epic, story and task tier? Run `/agent-workflow-kit tier`."* then **one** recommended next
|
|
@@ -87,17 +83,20 @@ priority order:
|
|
|
87
83
|
note carries `/agent-workflow-kit setup`). An **uncheckable** member ("couldn't be checked" — an
|
|
88
84
|
unknown-freshness note) is **never** a refresh trigger: only a behind note fires this step — the
|
|
89
85
|
uncheckable note already appears in the version block, add nothing more;
|
|
90
|
-
2. else **no
|
|
86
|
+
2. else **no bridge is ready by the detector's readiness** (not the cell word) → *set one up with
|
|
91
87
|
`/agent-workflow-kit setup`*;
|
|
92
|
-
3. else **a
|
|
88
|
+
3. else **a ready bridge's cell reads `✗ not wired —` or `✗ unchecked`** → *see what is missing with
|
|
89
|
+
`/agent-workflow-kit recipes`* (its lines name each bridge's missing entries and route, or its reason);
|
|
90
|
+
never on a `✗ not wired (unused)` cell;
|
|
91
|
+
4. else **a backend is ready but the orchestration config is still all-Solo** (no `reviewed` /
|
|
93
92
|
`council` / `delegated` slot anywhere — inspect `docs/ai/orchestration.json`, or read the
|
|
94
93
|
procedures advisor's resolved recipes) → *put it to work with `/agent-workflow-kit recipes`*;
|
|
95
|
-
|
|
94
|
+
5. else **the velocity allowlist is not yet seeded** (the envelope's velocity settings show zero
|
|
96
95
|
allow entries) → the optional *`/agent-workflow-kit velocity`* opt-in (never run it without a
|
|
97
96
|
yes);
|
|
98
|
-
|
|
97
|
+
6. else **the cheap-lane agent vehicles are not placed** (the envelope's agents settings show zero
|
|
99
98
|
placed) → the optional *`/agent-workflow-kit agents`* opt-in (never run it without a yes);
|
|
100
|
-
|
|
99
|
+
7. else **gates are declared but the approval hook is not wired** (the envelope's hook settings:
|
|
101
100
|
at least one declared gate, wired = no) → the optional *`/agent-workflow-kit hook`* opt-in
|
|
102
101
|
(never run it without a yes). If no rung applies, the two lines above stand alone.
|
|
103
102
|
|
|
@@ -105,11 +104,9 @@ Keep it compact — a few short lines, plain language, no kit-internal terms.
|
|
|
105
104
|
|
|
106
105
|
### Version disclosure — the `docs/ai` structure version, on demand only
|
|
107
106
|
|
|
108
|
-
The
|
|
109
|
-
|
|
110
|
-
|
|
111
|
-
than the published package version, it reads as *"why is this smaller than what GitHub shows?"* Surface
|
|
112
|
-
it in exactly **three** places, and nowhere else:
|
|
107
|
+
The internal **`docs/ai` structure version** (`deploymentHead`) is what `upgrade` compares the stamp
|
|
108
|
+
against to decide whether a migration is due. Hide it on the happy path; surface it in exactly
|
|
109
|
+
**three** places:
|
|
113
110
|
1. the **never-downgrade STOP** (`${CLAUDE_SKILL_DIR}/references/modes/upgrade.md` step 2) — the stamp is ahead of what this kit knows,
|
|
114
111
|
so the number IS the message;
|
|
115
112
|
2. the **explicit version-status view** (`${CLAUDE_SKILL_DIR}/references/modes/status.md`) the user deliberately opens;
|