docs: slim AGENTS.md to per-session context only

Why: AGENTS.md is preloaded into every session via CLAUDE.md's @AGENTS.md
import, so every word is a tax paid whether or not it is relevant. It had
grown to 3,371 words (~4,550 tokens), 75% of it a single "Setup and testing"
section that was mostly neither: roughly 1,600 words were gate forensics —
why a hook is shaped the way it is, what was tried and rejected, verification
numbers — and only ~390 words were genuine one-time setup.

Implementation notes: split three ways by audience rather than by heading.
One-time setup, prerequisites and test commands move to a new root README.md,
which did not previously exist. The gate forensics move to a new
docs/spec/gates.md, preserving every constant, hook ID, error string and
tried-and-rejected note (policy.fetch_failure_default, the removed fallback
frontmatter reader, extending skill-size-check's files: pattern to agents,
tidying .claude/settings.json out of the pretty-format-json exclude list) —
these exist to stop settled decisions being re-litigated, so none were
dropped. AGENTS.md keeps only what applies every session, at one clause of
rationale per rule. The .apm/-mirror section collapses to a pointer because
docs/spec/architecture.md already covers it in full. The hot ADR-0020 gate
counts stay in AGENTS.md deliberately: they block any skill edit today.

Also corrects docs/spec/architecture.md's stale claim of 19 ADRs; ADR-0020
landed without updating the count.

Impact: per-session preload drops from ~4,550 to ~1,116 tokens, a 75% cut.
agentsmd-audit passes: secrets clean, drift clean, 4 informational findings
for sections deliberately relocated to the README.

AGENTS.md

@@ -1,29 +1,25 @@
 # Working in this repo
 
-This repo is the global AI development configuration repository — the authoritative source for agent definitions, skills, workflows, and prompts across all projects. Built as a homelab tool intended to scale to professional environments.
+The global AI development configuration repository — the authoritative source for agent definitions, skills, workflows, and prompts across all projects.
+
+This file carries only what applies to **every** session. Setup, prerequisites, and test commands are in `README.md`; the reasoning behind each enforcement gate is in `docs/spec/gates.md`.
 
 ## Structure
 
-- `plugins/` — installable plugin units; each is an apm package (`apm.yml` + `.apm/`) carrying skills, agents, hooks, MCP servers, and bundled assets. This repo consumes them through **apm**, not Claude Code's native plugin install: root `apm.yml` declares all six as `dependencies.apm` git+path entries against the holocron remote, and `apm install` deploys them into `.claude/skills/` and `.claude/agents/` (both gitignored). External consumers can still install natively via `claude plugin install <name>@holocron` — the marketplace manifests are unchanged
+- `plugins/` — six installable plugin units, each an apm package (`apm.yml` + `.apm/`). Root `apm.yml` declares all six as `dependencies.apm`; `apm install` deploys them into `.claude/skills/` and `.claude/agents/`, both gitignored install output.
-- `providers/claude-code/` — Claude Code adapter (deployed to `~/.claude/` via `install.sh`)
+- `providers/claude-code/` — Claude Code adapter, deployed to `~/.claude/` via `scripts/install.sh`.
 
 ## Edit `.apm/`, never the flat mirror
 
-Inside a plugin, `plugins/<name>/.apm/` is the **only** hand-edited source for **plugin content** — the skills, agents, commands, instructions, extensions and hooks a host discovers. Everything in a plugin root that mirrors an `.apm/` primitive, plus both `plugin.json` manifests, is generated:
+`plugins/<name>/.apm/` is the only hand-edited source for plugin content. The flat `plugins/<name>/{skills,agents,commands,instructions,extensions}/` directories, the merged `plugins/<name>/hooks/hooks.json`, and both `plugin.json` manifests are generated — nothing marks them as generated, so check the path before you edit. An edit to the mirror is discarded by the next sync and reported as drift by the `check-plugin-content-sync` pre-push hook.
 
-- `scripts/sync-plugin-content.sh` generates the flat `plugins/<name>/{skills,agents,commands,instructions,extensions}/` directories and the merged `plugins/<name>/hooks/hooks.json` (ADR-0017)
+Not everything in a plugin root is generated. `README.md`, `docs/`, `bin/`, `sources.md`, `.mcp.json` and per-plugin extras are hand-authored there with no `.apm/` source — edit those in place. The rule is per-path, not per-directory. But a file placed *inside* a mirrored directory is deleted on the next sync (`sync_dir` runs `rm -rf` before every copy), so plugin-root documentation goes in `docs/`, never in `hooks/` or `skills/`.
-- `apm pack` generates both per-plugin manifests — `plugins/<name>/.claude-plugin/plugin.json` and `plugins/<name>/.github/plugin/plugin.json` — and **two of the three** root marketplace manifests: `.claude-plugin/marketplace.json` (apm's `claude` output profile) and `.agents/plugins/marketplace.json` (its `codex` profile, a differently-shaped file) (ADR-0015)
-- `scripts/sync-marketplace-mirror.sh` generates the third, `.github/plugin/marketplace.json` — Copilot CLI's legacy manifest path. **No apm output profile targets it**: apm ships exactly two marketplace output profiles, `claude` and `codex` (documented in `plugins/kyberforge/.apm/skills/apm-workflow/references/marketplace.md`). The mirror is a byte-identical copy of `.claude-plugin/marketplace.json`, gated by the `check-marketplace-mirror-sync` pre-push hook. Do not expect `apm pack` to refresh it — that assumption is exactly the drift this pair exists to prevent
 
-**A plugin root is not wholly generated.** Material that is not an `.apm/` primitive is hand-authored there and no compiler touches it: `README.md`, `docs/`, `bin/`, `sources.md`, `.mcp.json`, plus per-plugin extras like `plugins/git/config.example.json`, `plugins/gitea/references/` and `plugins/bin/evals/`. Edit those in place — they have no `.apm/` source, and looking for one wastes a search. The rule is per-path, not per-directory: `plugins/<name>/skills/` is generated, `plugins/<name>/docs/` is not. `docs/spec/architecture.md` carries the same carve-out.
+Full model: `docs/spec/architecture.md`.
-
-One qualification: "hand-authored, untouched" holds only at the plugin *root*. A file placed **inside** a mirrored directory is destroyed — `sync_dir` runs `rm -rf "$dst"` before every copy, so a `README.md` under `plugins/<name>/hooks/` or `plugins/<name>/skills/` is deleted on the next sync whether or not `.apm/` has a counterpart. Put root-level plugin documentation in `docs/`, never in a mirrored directory.
-
-Nothing labels a generated file as generated — `plugins/kyberforge/skills/forge/SKILL.md` is byte-identical to its `.apm/` original, with no marker in either. Check the path before you edit. An edit to the mirror is discarded by the next sync and is reported as drift by the `check-plugin-content-sync` pre-push hook, which is the earliest anyone finds out. Details in `docs/spec/architecture.md`.
 
 ## Prefer plugin skills over raw shell
 
-This repo dogfoods its own plugins. Before shelling out to git, gitea, or lint tooling directly, check whether an installed skill already owns the operation — it usually does:
+This repo dogfoods its own plugins. Before shelling out, check whether a skill already owns the operation — it usually does:
 
 - Commits, branches, history, worktrees, remotes → `git-commits`, `git-branches`, `git-history`, `git-worktrees`, `git-remotes`
 - Pre-commit hook install/config/troubleshooting → `pc-run` / `pc-author`
@@ -31,40 +27,30 @@ This repo dogfoods its own plugins. Before shelling out to git, gitea, or lint t
 - Vale prose linting → `vale-config` / `vale-run`
 - This repo's own AGENTS.md → `agentsmd-author` / `agentsmd-audit`
 
-Use the bare, **unnamespaced** names above. Under the old `claude plugin install` these were `git:git-commits`, `kyberforge:skill-audit`, and so on; `apm install` deploys each skill to `.claude/skills/<name>/` as a plain project skill, which has no plugin prefix to carry. The `<plugin>:` form has not stopped resolving here, though — `~/.claude.json` still enables `core`, `git`, `gitea`, `kyberforge`, and `lint` at **user** scope, and ADR-0018 left those native installs in place on purpose, converting them being a separate decision with a blast radius beyond this repo. Every skill is therefore live under both names right now, and a working `gitea:gitea-prs` is the user-scope copy answering — not evidence that the apm install or this file is broken, and not something to "fix". Prefer the bare name anyway: apm deploys it, an external consumer installing holocron through apm gets it, and it is the form that survives those user-scope installs eventually being converted. The namespaced form also still resolves in any project that installs holocron natively, so a skill body written for both audiences should name the bare skill. Same for agents: `git-orchestrate`, not `git:git-orchestrate`.
+Use the bare, **unnamespaced** names. The `<plugin>:` form (`gitea:gitea-prs`) also still resolves, because user-scope native installs were left enabled on purpose (ADR-0018) — a working namespaced call is not evidence that anything is broken and is not something to "fix". Prefer the bare name anyway: it is what `apm install` deploys, and what survives those user-scope installs eventually being converted.
 
 Fall back to raw shell only when no skill covers it.
 
-## Setup and testing
+## Session rules
 
-- Run `apm install` to deploy this repo's own skills and agents into `.claude/skills/` and `.claude/agents/`. Both are gitignored install output, not authoring source — `plugins/<name>/.apm/` remains the only place to edit. The six dependencies in root `apm.yml` resolve from the holocron **remote**, unpinned against the default branch, so a `.apm/` edit is not visible to the running session until it is pushed and `apm update` re-runs (`apm install` deploys from `apm.lock.yaml` and does not re-resolve refs). Needs the network, and needs `apm_modules/` (which it materializes) left gitignored. `apm install` also configures the `obsidian` MCP server into the repo's `.mcp.json`, carried over from `plugins/bin/.mcp.json`.
+- **Do not add repo-owned keys to `.claude/settings.json`.** apm treats it as its own deployed artifact and `apm audit --ci` replays the install and diffs, so anything apm would not have written is permanent drift that fails the `apm-audit-ci` pre-push hook. A hook you want here is authored in `plugins/<name>/.apm/hooks/` and deployed by apm, never hand-written into that file. Machine-specific settings go in the gitignored `.claude/settings.local.json`; shared enforcement goes in `.pre-commit-config.yaml`.
-- Do not add repo-owned keys to `.claude/settings.json`. apm treats that file as its own deployed artifact: `apm audit --ci` replays the install into a scratch tree and diffs, so anything apm would not have written there — an `enabledPlugins` block, a real `hooks` entry — is permanent drift that fails the `apm-audit-ci` pre-push hook. Its committed content is whatever apm last wrote, which today is the merged `SessionStart` entry for kyberforge's `check-apm-current.sh` — apm's own output, and it belongs in the commit (ADR-0019). What does not change is that nothing repo-authored goes in the file. A hook you want in this repo is authored in `plugins/<name>/.apm/hooks/` and deployed by apm, never hand-written here. The file is also **excluded from `pretty-format-json`** in `.pre-commit-config.yaml` — the sixth and last alternation in that `exclude:` pattern, and the only one there for a reason other than "generated manifest". Mind which number you are quoting: six alternations, expanding to sixteen real files (3 root marketplace manifests, 2 per plugin × 6 plugins, plus this one). `pretty-format-json --autofix` sorts object keys while apm emits insertion order, so leaving the file in that hook's scope rewrites apm's output on the way into every commit and `apm audit --ci` then reports permanent drift on a file with an empty `git diff`. Do not tidy it out of that list; it is load-bearing (see `LESSONS.md`, 2026-08-14). Machine-specific settings go in the gitignored `.claude/settings.local.json`, which apm does not deploy and the replay does not compare; shared enforcement belongs in `.pre-commit-config.yaml`.
+- **`apm.lock.yaml` turning up modified is expected, not a bug.** kyberforge's `SessionStart` hook runs `apm outdated` at startup and `apm update --yes` when something is behind, which rewrites the lock. Commit or discard it deliberately.
-- Keeping the install current is automatic but not free. Because the six dependencies are unpinned, deployed skills go stale whenever anyone merges. kyberforge ships a `SessionStart` hook that runs `apm outdated` at startup (~0.7s) and, when something is behind, runs `apm update --yes` and asks the host to re-scan skills (~10.4s). That rewrites `apm.lock.yaml`, so an unexplained modification to it after opening a session is expected, not a bug — commit or discard it deliberately. Note `apm install` alone will **not** pick up remote changes; it deploys from the lock. `apm update` is the command that re-resolves refs.
+- **A `.apm/` edit is not live in this session until it is pushed.** The six dependencies resolve from the holocron remote, unpinned against the default branch. `apm install` deploys from the lock; `apm update` is what re-resolves refs.
-- Install git hooks via `pc-run`, wiring all three stages — this repo's `.pre-commit-config.yaml` has no `default_install_hook_types`, so a plain install silently skips `commit-msg` (Conventional Commits) and `pre-push` (the 14-hook gate described below).
+- **The ADR-0020 skill gates ship hot, with no baseline.** 26 of 39 descriptions and 9 of 39 bodies exceed their FAIL tier, and the `Kyberforge.CompositionNote` Vale rule fires 10 errors across `gitea-issues`, `gitea-labels-milestones`, `gitea-prs` and `gitea-workflow`. Editing any of those skills *for any reason* means retrofitting it to the contract first — a one-line fix cannot be committed until the skill complies. Deliberate; tracked as Gitea issue #99. `skill-size-check` will not warn you about the Vale half, so check both: `pre-commit run --all-files`.
-- Install the `apm` CLI — four pre-push hooks shell out to it: `apm-marketplace-check`, `apm-audit-ci`, `apm-pack-check-clean`, and `check-plugin-content-sync` (via `scripts/sync-plugin-content.sh`, which wraps `apm pack`). `apm-marketplace-check` and `apm-pack-check-clean` are bare `apm …` hook entries and `apm-audit-ci` is a `bash -c` loop calling `apm` once per package, so without it the push dies with an unhelpful "command not found". Use `apm-install`, or `curl -sSL https://aka.ms/apm-unix | sh`; verify with `apm --version`.
+- **Run `bash tests/run-tests.sh` before considering any change done.**
-- Install `jq` — required by `scripts/check-manifests.sh` and `scripts/sync-plugin-content.sh`, both pre-push. These at least fail loudly (`Error: jq is required but not installed`).
+- **Before pushing, run the whole gate locally:** `pre-commit run --hook-stage pre-push --all-files`. Pushing runs 14 repo-defined hooks, not just the test suite.
-- Install `python3` — required by `scripts/skill-size-check.sh`, the `skill-size-check` pre-commit hook. It measures the *folded* `description` value: most descriptions here are `>`-block scalars, so a regex over the raw lines measures indentation and newlines instead of the value. Missing it fails the hook with an install pointer rather than skipping the ADR-0020 checks, which would be a vacuous green. In practice it is already present — pre-commit is itself a Python application. **PyYAML is a hard requirement too**, not an optional accelerator: the hand-rolled fallback frontmatter reader has been removed, because a reader that mis-parses an unfamiliar scalar shape reports a clean pass on a file it never measured, which is the exact vacuous-green failure the `python3` check exists to avoid. `pip install pyyaml` if the hook reports it missing.
+- **Pushing without a network** needs `SKIP=apm-marketplace-check,apm-pack-check-clean git push` — those two resolve a remote marketplace entry via `git ls-remote`. Skip only those two; the rest are real local checks, and adding one to `SKIP` disarms it silently.
-- That hook enforces **two independent gate families** over `plugins/*/.apm/skills/*/SKILL.md`, and neither replaced the other. The agentskills.io spec backstop is unchanged: 500 lines and 2,770 words, counted over the **whole file including frontmatter**. ADR-0020 adds a context budget measured differently — `description` 250 chars SUGGESTION / 400 FAIL (it is preloaded into every session whether the skill fires or not), **body-only** word count 600 SUGGESTION / 900 FAIL (everything after the frontmatter's closing `---`), a missing, valueless or `null` `description:` (a hard FAIL, not a skip — a gate that declines to measure the one preloaded field reports green), every boundary-clause routing target resolving to a real skill or agent, and every `references/<file>.md` a body names actually existing. Target resolution walks up **from the file being checked** to an authoring root — the nearest ancestor holding `plugins/*/.apm/{skills,agents}`, falling back to the nearest `.git`, in two passes so a nested `.git` cannot beat a real monorepo root. The universe is then every skill and agent under `<root>/plugins/*/`, plus the checked file's own apm package and whatever that package declares in its own `apm.yml` `dependencies.apm`; the **root** manifest's `dependencies:` block is not read, and no plugin here declares a cross-plugin apm dependency. Deployed `.claude/`/`.agents/` trees are consulted only when the walk found no plugin monorepo root — whether it landed on a bare `.git` ancestor or on nothing at all (the consumer case). The gate keys on which of the two passes matched, not on whether the root contributed any new name: a single-plugin monorepo re-collects its own package and adds nothing, so a name-count test reads zero there and would drag the deployed trees back into the universe. That matters because those trees are gitignored `apm install` output: resolution used to reach the four cross-plugin `gitea-*` → `git-*` targets through `.claude/skills/` alone, so the same commit measured 2 dangling targets on a developer machine and 6 on a fresh clone. It no longer does — verified by running the hook over a tree holding only `plugins/` and the root `apm.yml`, which reports findings identical to the working tree (26 description / 9 body / 2 dangling / 0 missing references / 58 SUGGESTIONs). Three further checks are SUGGESTION-only: a description with no boundary clause at all, a `## Gotchas` section with more than five entries, and a `## Gotchas` section over 25% of the body. A file can sit well inside one family and fail the other. The hook is `verbose: true` so the SUGGESTION tier is audible — pre-commit prints nothing at all for a passing hook, and a SUGGESTION deliberately does not fail. `skill-audit`'s `validate.sh` holds a second copy of the four ADR-0020 constants; `tests/test-skill-size-check.sh` asserts the copies agree.
+- **Author commits with `git-commits`** — it validates Conventional Commits, which `commit-msg` enforces.
-- **Those ADR-0020 gates ship hot, with no baseline file.** 26 of 39 descriptions and 9 of 39 bodies currently exceed their FAIL tier, so editing one of those skills *for any reason* means retrofitting it to the contract first — a one-line fix to `gitea-prs` cannot be committed until that skill complies. This is deliberate, and the retrofit is tracked as Gitea issue #99. Check where a skill stands before starting: `pre-commit run skill-size-check --all-files`.
-- **A second gate ships hot alongside it, and `skill-size-check` will not warn you about it.** `Kyberforge.CompositionNote` — the ADR-0020 Vale rule banning composition and architecture prose from a description — currently fires **10 errors across four skills**: `gitea-issues`, `gitea-labels-milestones`, `gitea-prs` and `gitea-workflow`. Every Vale rule here is `level: error` with no ignorable tier, so touching any of those four means fixing its prose findings as well as its size findings. Scoping a retrofit off `skill-size-check` output alone will leave you blocked at the second gate. Check both: `pre-commit run --all-files`.
-- Install the `vale` binary — required by the `vale-audit-prefilter-skill`/`-agent` pre-commit hooks. Their `files:` patterns are `.apm/`-scoped: `^plugins/[^/]+/\.apm/skills/[^/]+/SKILL\.md$` and `^plugins/[^/]+/\.apm/agents/[^/]+\.agent\.md$`. Only the authoring source triggers them — a `SKILL.md` in the generated mirror matches neither pattern, so prose findings surface only when you edit the file you are supposed to be editing. Without the binary the hooks fail with a bare "command not found" and no install pointer. `brew install vale` (macOS), `snap install vale` (Linux), `choco install vale` (Windows), or see https://vale.sh/docs/vale-cli/installation/. No `vale sync` needed — the `Kyberforge` styles are committed under `plugins/kyberforge/.apm/skills/{skill-audit,agent-audit}/assets/vale/styles/`, not downloaded packages (see ADR-0014).
-- `vale` is also a **pre-push** dependency, not only pre-commit. `check-vale-style-sync` runs six glob-coverage probes by invoking `vale --config` — they are the only assertions in it that catch a `.vale.ini` glob typo, the failure mode where every text-level check stays clean while vale lints zero files. Missing `vale` is therefore a hard failure there. The opt-out is `CHECK_VALE_STYLE_SYNC_ALLOW_MISSING_VALE=1`, and it is **not** `SKIP=`: the hook still runs and still asserts everything verifiable from file text, but the six probes do not, and its summary says so explicitly — `Vale style sync check passed (text-level only, vale unavailable): … 0 glob probe(s) verified`. Use it only on a machine that genuinely cannot install `vale`, and read that summary line as "the glob axis was not checked", not as a pass.
-- Run `bash tests/run-tests.sh` before considering any change done — it runs every `test-*.sh` script in the repo plus the bats suite (`--bats-only` for just bats). First run auto-initializes the bats submodules; no manual `git submodule update` needed.
-- A suite that exits 77 because a dependency is missing is reported as SKIPPED, and does **not** fail an ad-hoc run. The pre-push hook invokes the same script as `--strict` (`RUN_TESTS_STRICT=1` is equivalent), where a skip **does** fail the push: at pre-push a skip means one of the dependencies above is absent on this machine, so the gate would otherwise report success having run fewer suites than it appears to. Without vale, for instance, three suites skip (`test-check-vale-style-sync.sh`, `test-vale-hooks-consumer.sh`, `test-vale-wrap.sh`) and the strict failure names each one and what to install.
-- `tests/run-bats.sh` derives the set of `.bats` files it expects from `git ls-files`, so a `.bats` file deleted from the worktree but still tracked in the index fails the run rather than silently shrinking the suite. Remove one with `git rm` (or stage the deletion) when the removal is intentional; an untracked new `.bats` file is picked up and needs no ceremony. Both discovery walks (`tests/run-bats.sh` and `tests/run-tests.sh`) exclude `apm_modules/`: `apm install` materializes a full copy of every plugin there, and running a dependency's copy of a `.bats` file breaks its relative path to the bats helpers — 167 spurious failures before the exclusion landed.
-- Pushing runs 14 repo-defined pre-push hooks, not just the test suite — `run-tests` and `check-manifests`, plus generated-content drift gates (`check-plugin-content-sync`, `check-marketplace-mirror-sync`, `check-vale-style-sync`, `check-scope-walkup-sync`, `check-executables-allow-sync`), artifact validators (`check-apm-agents-valid`, which runs agent-audit's `validate.sh` over every real `plugins/*/.apm/agents/*.agent.md`), apm's own gates (`apm-marketplace-check`, `apm-audit-ci`, `apm-pack-check-clean`), host validators (`validate-plugins`, `validate-marketplace`, both needing the `claude` CLI), and `check-release-needed`. `check-executables-allow-sync` is the odd one in that first group — it guards a silent failure rather than drift in generated text. apm gates a package's `hooks/` and `bin/` on an exact `<package>#<version>` lookup in root `apm.yml`'s `executables.allow`, with no wildcard and no version-less form, so bumping `plugins/kyberforge/apm.yml`'s `version:` without bumping the key errors nowhere: the entry simply stops matching, kyberforge's `SessionStart` hook stops deploying, and the install goes quietly stale — the failure ADR-0019 records as live. Run `pre-commit run --hook-stage pre-push --all-files` locally — one command, the whole gate. That command reports **16**, not 14: pre-commit's own `meta` hooks, `check-hooks-apply` and `check-useless-excludes`, declare no `stages:` and so run at every stage including this one.
-- `apm-audit-ci` runs `apm audit --ci` once per manifest — the root one and each of the six plugin packages — because the root-only invocation audits the marketplace manifest and **nothing else**, and `apm-pack-check-clean` does not parse plugin `dependencies:` blocks either (verified: a malformed one passes `apm pack --check-versions --check-clean --dry-run` and fails `apm audit --ci` in that package's directory). It verifies two things and claims no more: each `apm.yml` parses as a valid APM manifest, and any package declaring dependencies has a consistent `apm.lock.yaml`. It does **not** enforce an org policy — apm discovers one from the git remote and only understands github.com and Azure DevOps, so against this repo's self-hosted Gitea remote it prints `No org policy found at unknown; enforcement skipped`. Do **not** "fix" that with `policy.fetch_failure_default: block` in `apm.yml`: it was tested and rejected, because with no reachable policy source it makes the hook exit 1 on every push forever.
-- `check-apm-agents-valid` derives its expected agent-file set from `git ls-files` (same pattern as `tests/run-bats.sh`), so an agent file deleted from the worktree but still tracked fails the run, and discovering zero agent files is an error rather than a pass. An untracked new agent file is still validated — the derivation is one-directional on purpose, so uncommitted work is not blocked but also cannot bypass the gate. Agents take the ADR-0020 description gates (`agent-audit`'s `validate.sh` holds its own copy of those two constants) and, deliberately, **no** body word gate: an agent body becomes the system prompt of a fresh context rather than competing with the caller's live conversation, so the 900-word FAIL does not transfer. A bats test pins that absence in `agent-audit`'s validator — adding a body gate there contradicts the ADR rather than fixing an inconsistency. Be precise about the scope of that guarantee, though: it holds for the **validator**, not for the shared script. `scripts/skill-size-check.sh` applies its body gate to whatever path it is handed, and `bash scripts/skill-size-check.sh plugins/*/.apm/agents/*.agent.md` exits 1 today with 900-word body FAILs on `git-orchestrate` (933), `gitea-orchestrate` (1,199) and `apm-orchestrate` (1,080). Agent files escape only because the hook definitions filter on `SKILL.md` — a file-pattern accident that happens to implement the design, not the design itself. Do not "extend" that hook's `files:` pattern to cover agents on the assumption that the script already knows the difference.
-- **Two** pre-push hooks need the network, for one shared reason: root `apm.yml`'s `marketplace.packages[]` contains exactly one remote entry (`mattpocock-skills`, `source: mattpocock/skills`), and resolving it needs a `git ls-remote`. `apm-marketplace-check` resolves every entry and is `always_run`, so it fails with `No cached refs (offline)`. `apm-pack-check-clean` (`apm pack --check-versions --check-clean --dry-run`) re-resolves the same entry and fails with `Error: Git network timeout during ls-remote`. Pinning the entry to an exact version does **not** remove the call — an exact pin still ls-remotes. `--offline` rescues neither. To push without a network, skip both using pre-commit's own mechanism: `SKIP=apm-marketplace-check,apm-pack-check-clean git push`. Skip those two alone — verified under `unshare -rn`, the other twelve pre-push hooks pass offline because they are real local checks (`check-executables-allow-sync` landed after that run, but reads two local manifests and makes no network call), and adding one of them to `SKIP` disarms it silently. `apm-audit-ci` calls `apm` too but stays local: its org-policy discovery resolves nothing on this remote before any network call, so it does not join the pair above.
-- Author commits with `git-commits` — it validates Conventional Commits (enforced at `commit-msg`) for you.
 
 ## Key documents
 
-Read CONTEXT.md at the start of every session in this repo.
+Read `CONTEXT.md` at the start of every session.
 
 Read these on demand:
 
-- `docs/spec/architecture.md` — current directory structure, install pipeline, provider model
+- `README.md` — prerequisites, install, and test commands
+- `docs/spec/gates.md` — what each pre-commit and pre-push hook enforces and why; read when a gate fails or before changing hook config
+- `docs/spec/architecture.md` — directory structure, install pipeline, provider model
 - `docs/adr/` — architectural decisions; read before answering design questions or proposing structural changes
 - `docs/ai-constitution.md` — full governance evidence base; read when a governance decision needs justification
 - `docs/research/ai-coding-factory/ai-coding-factory-principles.md` — factory design rationale; read when implementing, auditing, or reviewing skills or factory structure

README.md

@@ -0,0 +1,126 @@
+# holocron
+
+The global AI development configuration repository — the authoritative source for agent definitions, skills, workflows, and prompts across all projects. Built as a homelab tool intended to scale to professional environments.
+
+Content ships as six installable plugins, each an apm (Agent Package Manager) package. This repo consumes its own plugins through apm, so the working copy runs the same released content every other consumer gets.
+
+## Repo layout
+
+| Path | What it holds |
+| --- | --- |
+| `plugins/` | Six apm packages — `bin`, `core`, `git`, `gitea`, `kyberforge`, `lint` — each carrying skills, agents, hooks, MCP servers, and bundled assets |
+| `providers/claude-code/` | Claude Code adapter, deployed to `~/.claude/` via `scripts/install.sh` |
+| `core/` | Provider-agnostic always-on content — `core/AGENTS.md` and `core/instructions/` |
+| `docs/` | Specs (`docs/spec/`), architectural decisions (`docs/adr/`), governance, research, and notes |
+| `scripts/` | Install, sync, and check scripts used by the git hooks |
+| `tests/` | `run-tests.sh`, `run-bats.sh`, the `test-*.sh` suites, and the bats submodules |
+
+The six plugins:
+
+- **kyberforge** — skills and agents for creating, maintaining, and managing a Claude Code / Copilot CLI plugin marketplace
+- **git** — conventional commits, branch management, pull requests, feature flow
+- **gitea** — issues, pull requests, milestones, releases, wikis
+- **core** — authoring and auditing a repo's `AGENTS.md` and the provider adapter files that defer to it
+- **lint** — configuring and running linters
+- **bin** — a place for things to be binned
+
+## Prerequisites
+
+Install all of these before setting up. Each one is a hard dependency of a git hook or a script — several fail with an unhelpful "command not found" if missing.
+
+| Tool | Why | Install |
+| --- | --- | --- |
+| `apm` CLI | Four pre-push hooks shell out to it (`apm-marketplace-check`, `apm-audit-ci`, `apm-pack-check-clean`, and `check-plugin-content-sync` via `scripts/sync-plugin-content.sh`) | The `apm-install` skill, or `curl -sSL https://aka.ms/apm-unix \| sh`. Verify with `apm --version` |
+| `jq` | Required by `scripts/check-manifests.sh` and `scripts/sync-plugin-content.sh`, both pre-push | Your package manager |
+| `python3` + PyYAML | Required by `scripts/skill-size-check.sh` (the `skill-size-check` pre-commit hook), which reads folded YAML frontmatter | `python3` is usually present — pre-commit is itself a Python application. `pip install pyyaml` if the hook reports PyYAML missing |
+| `vale` | Required by the `vale-audit-prefilter-skill` / `-agent` pre-commit hooks and the `check-vale-style-sync` pre-push hook | `brew install vale` (macOS), `snap install vale` (Linux), `choco install vale` (Windows), or https://vale.sh/docs/vale-cli/installation/ |
+| `claude` CLI | Required by the `validate-plugins` and `validate-marketplace` pre-push hooks | Claude Code |
+
+Two notes worth reading before you skip one:
+
+- **PyYAML is a hard requirement, not an optional accelerator.** The hand-rolled fallback frontmatter reader was removed deliberately: a reader that mis-parses an unfamiliar scalar shape reports a clean pass on a file it never measured.
+- **No `vale sync` is needed.** The `Kyberforge` styles are committed under `plugins/kyberforge/.apm/skills/{skill-audit,agent-audit}/assets/vale/styles/`, not downloaded packages (ADR-0014).
+
+## Setup
+
+Run these in order, from the repo root.
+
+```bash
+# 1. Deploy this repo's own skills and agents
+apm install
+
+# 2. Install the git hooks — all three stages
+#    (use the `pc-run` skill, which knows the stage wiring)
+```
+
+**`apm install`** deploys the six plugins into `.claude/skills/` and `.claude/agents/`. Both are gitignored install output, *not* authoring source — `plugins/<name>/.apm/` remains the only place to edit. It needs the network, materializes `apm_modules/` (which stays gitignored), and also configures the `obsidian` MCP server into the repo's `.mcp.json`.
+
+**Git hooks** go in via the `pc-run` skill, wiring **all three stages**. This repo's `.pre-commit-config.yaml` has no `default_install_hook_types`, so a plain `pre-commit install` silently skips `commit-msg` (Conventional Commits) and `pre-push` (the full gate).
+
+## Keeping the install current
+
+The six dependencies in root `apm.yml` are unpinned against the default branch, so deployed skills go stale whenever anyone merges. kyberforge ships a `SessionStart` hook that runs `apm outdated` at startup (~0.7s) and, when something is behind, runs `apm update --yes` and asks the host to re-scan skills (~10.4s).
+
+That rewrites `apm.lock.yaml` — an unexplained modification to it after opening a session is expected, not a bug. Commit or discard it deliberately.
+
+Note the difference between the two commands:
+
+- `apm install` deploys from `apm.lock.yaml`. It does **not** pick up remote changes.
+- `apm update` re-resolves refs. This is the command that pulls in a merged `.apm/` edit.
+
+## Running tests
+
+```bash
+bash tests/run-tests.sh              # every test-*.sh script plus the bats suite
+bash tests/run-tests.sh --bats-only  # just bats
+```
+
+The first run auto-initializes the bats submodules; no manual `git submodule update` needed.
+
+A suite that exits 77 because a dependency is missing is reported as SKIPPED and does **not** fail an ad-hoc run. It *does* fail under `--strict` (equivalently `RUN_TESTS_STRICT=1`), which is how the pre-push hook invokes it — at pre-push, a skip means one of the prerequisites above is absent on this machine, and the gate would otherwise report success having run fewer suites than it appears to. The strict failure names each skipped suite and what to install.
+
+## Before pushing
+
+Run the whole pre-push gate locally in one command:
+
+```bash
+pre-commit run --hook-stage pre-push --all-files
+```
+
+See [`docs/spec/gates.md`](docs/spec/gates.md) for what each hook enforces and why.
+
+**Offline?** Exactly two pre-push hooks need the network, because root `apm.yml`'s marketplace contains one remote package entry that must be resolved with `git ls-remote`:
+
+```bash
+SKIP=apm-marketplace-check,apm-pack-check-clean git push
+```
+
+Skip **only** those two. The remaining pre-push hooks are real local checks and pass offline; adding one of them to `SKIP` disarms it silently.
+
+## Editing plugin content
+
+`plugins/<name>/.apm/` is the only hand-edited source for plugin content — skills, agents, commands, instructions, extensions, and hooks. The flat `plugins/<name>/{skills,agents,commands,instructions,extensions}/` directories, the merged `hooks/hooks.json`, and every `plugin.json` / `marketplace.json` manifest are generated. Nothing labels a generated file as generated, so check the path before you edit; an edit to the mirror is discarded by the next sync and reported as drift by the `check-plugin-content-sync` pre-push hook.
+
+Hand-authored material that is *not* an `.apm/` primitive — `README.md`, `docs/`, `bin/`, `sources.md`, `.mcp.json` — lives at the plugin **root** and is untouched. Never place such a file inside a mirrored directory: the sync removes the destination before every copy, so it is deleted with no drift report.
+
+Full detail in [`docs/spec/architecture.md`](docs/spec/architecture.md).
+
+## For external consumers
+
+Install a plugin natively from the marketplace manifests:
+
+```bash
+claude plugin install <name>@holocron
+```
+
+Or consume the packages through apm, the way this repo does — declare them as `dependencies.apm` git+path entries against the holocron remote and run `apm install`.
+
+## Where to go next
+
+- [`AGENTS.md`](AGENTS.md) — the rules for AI agents working in this repo
+- [`CONTEXT.md`](CONTEXT.md) — domain language; read at the start of every session here
+- [`docs/spec/architecture.md`](docs/spec/architecture.md) — directory structure, install pipeline, provider model
+- [`docs/spec/gates.md`](docs/spec/gates.md) — the enforcement gates in depth
+- [`docs/adr/`](docs/adr/) — architectural decisions; read before proposing structural changes
+- [`docs/VISION.md`](docs/VISION.md) — where this is going
+- [`LESSONS.md`](LESSONS.md) — things that went wrong once and should not again

docs/spec/architecture.md

@@ -63,4 +63,4 @@ This repo also has a `CLAUDE.md` at its root — the Claude Code entry point for
 
 ## Architectural decisions
 
-Key hard-to-reverse decisions are recorded as ADRs in `docs/adr/`. There is no index file — the directory holds 19 numbered ADRs whose filenames state their decision, so `ls docs/adr/` is the index. Read a superseding ADR before the one it supersedes: ADR-0015 (apm as the authoring source of truth) supersedes ADR-0001 and moots ADR-0006, ADR-0017 corrects ADR-0015's host-discovery gap, and ADR-0019 supersedes one claim in ADR-0018 (that `.claude/settings.json`'s committed content is exactly `{"hooks": {}}`) while keeping the rule behind it. Entry points for the structure described on this page: ADR-0002 (two-tier CLAUDE.md), ADR-0003 (AGENTS.md as the provider-agnostic entry point), ADR-0015 and ADR-0017 (the two compilers behind the plugin roots).
+Key hard-to-reverse decisions are recorded as ADRs in `docs/adr/`. There is no index file — the directory holds 20 numbered ADRs whose filenames state their decision, so `ls docs/adr/` is the index. Read a superseding ADR before the one it supersedes: ADR-0015 (apm as the authoring source of truth) supersedes ADR-0001 and moots ADR-0006, ADR-0017 corrects ADR-0015's host-discovery gap, and ADR-0019 supersedes one claim in ADR-0018 (that `.claude/settings.json`'s committed content is exactly `{"hooks": {}}`) while keeping the rule behind it. Entry points for the structure described on this page: ADR-0002 (two-tier CLAUDE.md), ADR-0003 (AGENTS.md as the provider-agnostic entry point), ADR-0015 and ADR-0017 (the two compilers behind the plugin roots).

docs/spec/gates.md

@@ -0,0 +1,471 @@
+# Enforcement gates
+
+Reference for this repo's pre-commit and pre-push hooks: what each one guards, what its numbers
+mean, and which shapes were tried and rejected. Read it when a gate fails, before changing anything
+in `.pre-commit-config.yaml`, or before "fixing" something that looks like an inconsistency — several
+of the oddities documented here are load-bearing and have already been re-litigated once.
+
+`AGENTS.md` carries only the operative rules an agent needs in the moment. The reasoning lives here.
+
+---
+
+## Running the gates
+
+| Command | Scope |
+|---|---|
+| `pre-commit run --all-files` | the commit-stage hooks |
+| `pre-commit run --hook-stage pre-push --all-files` | the whole push gate, one command |
+| `pre-commit run skill-size-check --all-files` | just the ADR-0020 size/context gates |
+
+Install hooks via `pc-run`, wiring **all three stages**. This repo's `.pre-commit-config.yaml` has no
+`default_install_hook_types`, so a plain install silently skips `commit-msg` (Conventional Commits)
+and `pre-push` (everything below).
+
+The pre-push command reports **16** hooks, not 14. The extra two are pre-commit's own `meta` hooks,
+`check-hooks-apply` and `check-useless-excludes`: they declare no `stages:`, so they run at every
+stage including this one. Fourteen is the count of repo-defined pre-push hooks.
+
+## The pre-push gate
+
+Fourteen hooks, in config order.
+
+**Core checks**
+
+| Hook | Guards |
+|---|---|
+| `run-tests` | `bash tests/run-tests.sh --strict` — the whole suite, skips fatal (see [Tests](#tests)) |
+| `check-manifests` | `marketplace.json` and `plugin.json` paths resolve (needs `jq`) |
+
+**Generated-content drift gates**
+
+| Hook | Guards |
+|---|---|
+| `check-plugin-content-sync` | each plugin's flat `skills/agents/commands/hooks` mirror matches `.apm/` (issue #90) |
+| `check-marketplace-mirror-sync` | `.github/plugin/marketplace.json` is byte-identical to `.claude-plugin/marketplace.json` — no apm output profile targets that path |
+| `check-vale-style-sync` | skill-audit's Vale copy matches agent-audit's canonical copy, plus six glob-coverage probes (see [Vale](#vale)) |
+| `check-scope-walkup-sync` | `validate.sh`, `validate-provenance.sh`, `new-agent.sh` and `new-skill.sh`'s four independent `$HOME`/`.git`/`apm.yml` walk-up ports still agree behaviorally |
+| `check-executables-allow-sync` | root `apm.yml`'s `executables.allow` key names kyberforge's actual version (see [apm gates](#apm-gates)) |
+
+`check-executables-allow-sync` is the odd one in this group: it guards a *silent failure* rather than
+drift in generated text.
+
+**Artifact validators**
+
+| Hook | Guards |
+|---|---|
+| `check-apm-agents-valid` | runs agent-audit's `validate.sh` over every real `plugins/*/.apm/agents/*.agent.md` (see [Agent files](#agent-files-take-the-description-gates-not-the-body-gate)) |
+
+**apm's own gates**
+
+| Hook | Guards |
+|---|---|
+| `apm-marketplace-check` | every `marketplace.packages[]` entry resolves, including network reachability of remote refs |
+| `apm-audit-ci` | `apm audit --ci` once per manifest — root plus each of the six plugin packages |
+| `apm-pack-check-clean` | `apm pack --check-versions --check-clean --dry-run` — the compiled marketplace still matches what `apm.yml` + `.apm/` would generate, and per-package versions agree with the `per_package` strategy |
+
+**Host validators** (both need the `claude` CLI on PATH)
+
+| Hook | Guards |
+|---|---|
+| `validate-plugins` | `claude plugin validate --strict` on every plugin directory |
+| `validate-marketplace` | `claude plugin validate --strict` on the root marketplace manifest |
+
+**Release**
+
+| Hook | Guards |
+|---|---|
+| `check-release-needed` | on push to `main` only — fails if files exposed via `.pre-commit-hooks.yaml` changed since the last tag |
+
+Four of these shell out to `apm`: `apm-marketplace-check`, `apm-audit-ci`, `apm-pack-check-clean`,
+and `check-plugin-content-sync` (via `scripts/sync-plugin-content.sh`, which wraps `apm pack`). The
+first and third are bare `apm …` entries and the second is a `bash -c` loop calling `apm` once per
+package, so without the CLI the push dies with an unhelpful "command not found". Install with
+`apm-install`, or `curl -sSL https://aka.ms/apm-unix | sh`; verify with `apm --version`. `jq` is
+needed by `scripts/check-manifests.sh` and `scripts/sync-plugin-content.sh` — those at least fail
+loudly (`Error: jq is required but not installed`).
+
+## Skill and agent context gates (ADR-0020)
+
+The `skill-size-check` pre-commit hook, scoped to `^plugins/[^/]+/\.apm/skills/[^/]+/SKILL\.md$`,
+runs `scripts/skill-size-check.sh`.
+
+### Two independent gate families, neither replaced the other
+
+**Family 1 — agentskills.io spec backstop** (unchanged, conformance not quality):
+
+| Constant | Value | Measured over |
+|---|---|---|
+| `MAX_LINES` | 500 | whole file, **frontmatter included** |
+| `MAX_WORDS` | 2,770 | whole file, **frontmatter included** |
+
+**Family 2 — ADR-0020 context budget** (measured differently, on purpose):
+
+| Check | SUGGESTION | FAIL | Measured over |
+|---|---|---|---|
+| `description` characters | 250 | 400 | the YAML-**folded** value |
+| body words | 600 | 900 | **body only** — everything after the frontmatter's closing `---` |
+
+Plus three hard FAILs with no suggestion tier:
+
+- **A missing, valueless or `null` `description:`.** Not a skip. The description is the one field
+  preloaded into every session, so a gate that declines to measure it reports green. (This is not
+  hypothetical: `description:` with no value followed by `model: sonnet` let a line regex capture the
+  *next* key, which looked non-empty, so the "missing or empty" branch never fired and every gate
+  below early-returned on the genuinely empty folded value — exit 0, zero output, on a blocking gate.)
+- **Every boundary-clause routing target must resolve** to a real skill or agent.
+- **Every `references/<file>.md` a body names must exist** on disk. A dispatch table pointing at a
+  file that was never written is a silently dead branch, and nothing else in the gate/audit/vale
+  stack notices it.
+
+A file can sit well inside one family and fail the other. 2,770 whole-file words is a conformance
+backstop; 900 body-only words is a quality gate. Conflating them is what produced the current state.
+
+### Target resolution walk
+
+Resolution walks up **from the file being checked** — never from the script's own location. Deriving
+it from `${BASH_SOURCE}` leaked holocron's 39-skill universe into every consumer repo running the
+hook through pre-commit, so a consumer skill routing to `skill-audit` resolved against a plugin it
+had never installed.
+
+The walk finds an **authoring root**: the nearest ancestor holding `plugins/*/.apm/skills` or
+`plugins/*/.apm/agents`, falling back to the nearest ancestor holding `.git`. **Two passes, not one
+interleaved walk**, so a nested `.git` (a submodule, a sub-package worktree) cannot beat a real
+monorepo root further up.
+
+The universe is then:
+
+1. every skill and agent under `<root>/plugins/*/` — sibling plugins resolve, which is what a
+   monorepo means;
+2. the checked file's own apm package;
+3. the packages that package declares in **its own** `apm.yml` `dependencies.apm`.
+
+The **root** manifest's `dependencies:` block is not read, and no plugin here declares a cross-plugin
+apm dependency — none needs to.
+
+Deployed `.claude/` / `.agents/` trees are consulted **only** when the walk found no plugin monorepo
+root, whether it landed on a bare `.git` ancestor or on nothing at all. That is the consumer case.
+
+**The gate keys on which of the two passes matched, never on whether the root contributed a new
+name.** A name-count delta looks equivalent and is not: `_collect_authoring_root()` re-collects the
+checked file's own plugin, whose names the earlier steps already added, so a single-plugin monorepo
+shows a delta of zero and would wrongly reach for the deployed trees — including the user's global
+`~/.claude/skills`, making the verdict depend on what happens to be installed.
+
+Why it matters: those trees are gitignored `apm install` output, present only on a machine that has
+run it. Four cross-plugin targets here (`gitea-branches` → `git-branches`, `gitea-branches` →
+`git-history`, `gitea-issues` → `git-branches`, `gitea-workflow` → `git-workflow`) once resolved
+through `.claude/skills/` alone, so **the same commit measured 2 dangling targets on a developer
+machine and 6 on a fresh clone**. A gate shipping hot with no baseline cannot give two answers.
+
+Verified fixed: running the hook over a tree holding only `plugins/` and the root `apm.yml`, with no
+`.claude/` or `.agents/` anywhere, produces findings identical to the working tree — **26 description
+FAILs, 9 body FAILs, 2 dangling targets, 0 missing references, 58 SUGGESTIONs**.
+
+### SUGGESTION-only checks
+
+Three more, deterministic to measure but judgment to act on:
+
+- a description with **no boundary clause at all**;
+- a `## Gotchas` section with **more than five entries**;
+- a `## Gotchas` section over **25% of the body**.
+
+### `verbose: true` is load-bearing
+
+The hook is declared `verbose: true` so the SUGGESTION tier is audible. pre-commit prints nothing at
+all for a passing hook, and a SUGGESTION deliberately does not fail — without verbose every
+suggestion is swallowed, which is exactly the invisibility ADR-0013 records for Vale warnings.
+ADR-0020's arithmetic depends on it: writing to the 400-char FAIL lands the preload at 39 × 400 =
+15,600 chars (a 33% cut off 23,427); writing to the 250-char SUGGESTION lands at 9,750 (58%). The
+halving depends entirely on that tier being visible. It costs nothing on a clean file — the script
+prints only findings.
+
+### Duplicated constants
+
+`skill-audit`'s `validate.sh` holds a second copy of the four ADR-0020 constants
+(`DESC_SUGGEST_CHARS` / `DESC_MAX_CHARS` / `BODY_SUGGEST_WORDS` / `BODY_MAX_WORDS`), and
+`agent-audit`'s `validate.sh` holds a third copy of the two description constants. They are copied
+rather than imported because a cache-installed plugin's scripts cannot read files outside their own
+plugin directory. `tests/test-skill-size-check.sh` asserts the copies agree, so drift fails CI rather
+than silently letting an audit bless a skill the commit hook then rejects. The shared boundary
+resolver block is embedded verbatim in all three scripts between `BEGIN`/`END ADR-0020 SHARED
+BOUNDARY RESOLVER` markers and must stay byte-identical.
+
+### `python3` and PyYAML are hard requirements
+
+Both, and neither is a best-effort accelerator.
+
+`python3` because the script measures the **folded** `description` value. Most descriptions here are
+`>`-block scalars, so a regex over the raw lines measures indentation and newlines instead of the
+value. Missing it fails the hook with an install pointer rather than skipping the ADR-0020 checks,
+which would be a vacuous green. In practice it is already present — pre-commit is itself a Python
+application.
+
+**PyYAML** because the hand-rolled fallback frontmatter reader has been **removed deliberately**. It
+disagreed with a real parser across the FAIL boundary — one corpus description measured 270
+characters parsed and 412 unparsed — and a quoted `"description"` key or an explicit
+`description: null` returned empty from it, silently skipping the description *and* routing checks. A
+reader that mis-parses an unfamiliar scalar shape reports a clean pass on a file it never measured,
+which is the exact vacuous-green failure the `python3` check exists to avoid. `pip install pyyaml`
+(or `python3 -m pip install PyYAML`, or the distro's `python3-yaml`) if the hook reports it missing.
+
+## Agent files take the description gates, not the body gate
+
+`check-apm-agents-valid` runs agent-audit's `validate.sh` over every real
+`plugins/*/.apm/agents/*.agent.md`. It derives its expected file set from `git ls-files` — the pattern
+`tests/run-bats.sh` established — so an agent file deleted from the worktree but still tracked fails
+the run, and **discovering zero agent files is an error, not a pass**. An untracked *new* agent file
+is still validated: the derivation is one-directional on purpose, so uncommitted work is not blocked
+but also cannot bypass the gate.
+
+The hook exists because `validate.sh` was previously exercised only by `check-scope-walkup-sync`,
+against synthetic `mktemp` fixtures — it had never run against the agent files it governs. That is
+how ADR-0016 could be amended to bless a `disallowedTools` frontmatter field while `validate.sh`'s
+allowlist still rejected it: spec and enforcer disagreed and every gate stayed green.
+
+Agents take the ADR-0020 **description** gates (agent-audit's `validate.sh` holds its own copy of
+those two constants) and, deliberately, **no body word gate**. A skill body is loaded into the
+caller's context and competes with the live conversation; an agent body becomes the system prompt of
+a *fresh* context. The rationale for the 900-word FAIL does not transfer. A bats test pins that
+absence in agent-audit's validator — adding a body gate there contradicts the ADR rather than fixing
+an inconsistency.
+
+**Be precise about the scope of that guarantee: it holds for the *validator*, not for the shared
+script.** `scripts/skill-size-check.sh` applies its body gate to whatever path it is handed, and
+
+```
+bash scripts/skill-size-check.sh plugins/*/.apm/agents/*.agent.md
+```
+
+exits 1 today with 900-word body FAILs on `git-orchestrate` (933), `gitea-orchestrate` (1,199) and
+`apm-orchestrate` (1,080). Agent files escape only because the hook definitions filter on `SKILL.md`
+— a file-pattern accident that happens to implement the design, not the design itself. **Do not
+"extend" that hook's `files:` pattern to cover agents** on the assumption that the script already
+knows the difference; doing so silently enforces a gate ADR-0020 declines to set.
+
+## Current retrofit status
+
+**The ADR-0020 gates ship hot, with no baseline file.** A shrinking baseline recording each
+non-compliant skill's current numbers was considered and rejected in favour of hot gates.
+
+Two independent hot gates are currently red, and the first will not warn you about the second.
+
+| Gate | Current findings |
+|---|---|
+| `skill-size-check` | **26 of 39** descriptions and **9 of 39** bodies exceed their FAIL tier; 2 dangling targets; 58 SUGGESTIONs |
+| `Kyberforge.CompositionNote` (Vale) | **10 errors across four skills**: `gitea-issues`, `gitea-labels-milestones`, `gitea-prs`, `gitea-workflow` |
+
+`Kyberforge.CompositionNote` is the ADR-0020 Vale rule banning composition and architecture prose
+from a description. Every Vale rule here is `level: error` with no ignorable tier, so touching any of
+those four skills means fixing its prose findings as well as its size findings.
+
+Consequence: editing a non-compliant skill *for any reason* means retrofitting it to the contract
+first — a one-line fix to `gitea-prs` cannot be committed until that skill complies. This is
+deliberate; it guarantees convergence and avoids a half-state. Tracked as Gitea issue **#99**.
+
+Check where a skill stands before starting, and check **both** gates:
+
+```
+pre-commit run skill-size-check --all-files   # size/context only
+pre-commit run --all-files                    # size AND Vale
+```
+
+Scoping a retrofit off `skill-size-check` output alone leaves you blocked at the second gate.
+
+## Vale
+
+Install the `vale` binary — `brew install vale` (macOS), `snap install vale` (Linux),
+`choco install vale` (Windows), or see <https://vale.sh/docs/vale-cli/installation/>. No `vale sync`
+is needed: the `Kyberforge` styles are **committed** under
+`plugins/kyberforge/.apm/skills/{skill-audit,agent-audit}/assets/vale/styles/`, not downloaded
+packages (ADR-0014).
+
+### Pre-commit
+
+Two prefilter hooks, with `.apm/`-scoped `files:` patterns:
+
+| Hook | Pattern |
+|---|---|
+| `vale-audit-prefilter-skill` | `^plugins/[^/]+/\.apm/skills/[^/]+/SKILL\.md$` |
+| `vale-audit-prefilter-agent` | `^plugins/[^/]+/\.apm/agents/[^/]+\.agent\.md$` |
+
+Only the **authoring source** triggers them. A `SKILL.md` in the generated flat mirror matches
+neither pattern, so prose findings surface only when you edit the file you are supposed to be
+editing. Without the binary the hooks fail with a bare "command not found" and no install pointer.
+
+### Pre-push
+
+`vale` is a **pre-push** dependency too, not only pre-commit. `check-vale-style-sync` runs **six
+glob-coverage probes** by invoking `vale --config` — one representative path per file shape the
+prefilter is supposed to cover. They are the only assertions in the script that catch a `.vale.ini`
+glob typo (`[**/SKILL.md]` → `[**/SKILLS.md]`), the failure mode where every text-level check stays
+clean while vale lints zero files. As a warning this self-disabled on exactly that mutation and
+exited 0, and since pre-commit swallows a passing hook's output the stderr line was never seen — the
+hook reported `Passed`. Missing `vale` is therefore a hard failure here.
+
+The opt-out is `CHECK_VALE_STYLE_SYNC_ALLOW_MISSING_VALE=1`, and **it is not `SKIP=`**: the hook
+still runs and still asserts everything verifiable from file text, but the six probes do not, and its
+summary says so explicitly —
+
+```
+Vale style sync check passed (text-level only, vale unavailable): … 0 glob probe(s) verified.
+```
+
+Use it only on a machine that genuinely cannot install `vale`, and read that line as "the glob axis
+was not checked", not as a pass. The hook is `verbose: true` for exactly that reason — its clean
+output is a single line, so it costs one line per push.
+
+## Tests
+
+```
+bash tests/run-tests.sh              # every test-*.sh plus the bats suite
+bash tests/run-tests.sh --bats-only  # just bats
+```
+
+First run auto-initializes the bats submodules; no manual `git submodule update` needed.
+
+**Exit 77 = SKIPPED.** A suite that skips because a dependency is missing does **not** fail an ad-hoc
+run. The pre-push hook invokes the same script as `--strict` (`RUN_TESTS_STRICT=1` is equivalent),
+where a skip **does** fail the push: at pre-push a skip means one of the documented dependencies is
+absent on this machine, so the gate would otherwise report success having run fewer suites than it
+appears to. Without `--strict` the gate once went green having verified 15 of 17 suites on a
+vale-less PATH, with the skip list swallowed. Without vale, three suites skip —
+`test-check-vale-style-sync.sh`, `test-vale-hooks-consumer.sh`, `test-vale-wrap.sh` — and the strict
+failure names each one and what to install.
+
+`tests/run-bats.sh` derives the set of `.bats` files it expects from `git ls-files`, so a `.bats`
+file deleted from the worktree but still tracked in the index fails the run rather than silently
+shrinking the suite. Remove one with `git rm` (or stage the deletion) when intentional; an untracked
+new `.bats` file is picked up and needs no ceremony.
+
+Both discovery walks (`tests/run-bats.sh` and `tests/run-tests.sh`) exclude `apm_modules/`:
+`apm install` materializes a full copy of every plugin there, and running a dependency's copy of a
+`.bats` file breaks its relative path to the bats helpers — **167 spurious failures** before the
+exclusion landed.
+
+## apm gates
+
+### `apm-audit-ci`
+
+Runs `apm audit --ci` **once per manifest** — the root one and each of the six plugin packages —
+because the root-only invocation audits the marketplace manifest and **nothing else**, and
+`apm-pack-check-clean` does not parse plugin `dependencies:` blocks either. Verified: a malformed
+dependency entry passes `apm pack --check-versions --check-clean --dry-run` and fails
+`apm audit --ci` in that package's directory. Costs ~0.5s per package.
+
+It verifies **exactly two things** per manifest and claims no more:
+
+- **manifest-parse** — each `apm.yml` parses as a valid APM manifest. Unconditional; verified to fire
+  on a dependency entry missing its `git`/`path`/`registry` field (`Cannot parse apm.yml`).
+- **lockfile-exists** — any package declaring dependencies has a consistent `apm.lock.yaml`.
+  Conditional, and vacuous while every plugin `apm.yml` declares `dependencies: {apm: [], mcp: []}`;
+  it arms itself the moment one does not (verified by adding a git dependency to
+  `plugins/lint/apm.yml`).
+
+It does **not** enforce an org policy. apm discovers one from the git remote and only understands
+github.com and Azure DevOps, so against this repo's self-hosted Gitea remote it prints:
+
+```
+No org policy found at unknown; enforcement skipped
+```
+
+**Do not "fix" that with `policy.fetch_failure_default: block` in `apm.yml`.** apm's own message
+suggests it; it was tried on a scratch copy and **rejected**. With no reachable policy source it does
+not make the check meaningful, it makes it permanently red — `apm audit --ci` exits 1 with
+`No org policy found at unknown (policy.fetch_failure_default=block)` on every push, forever. A gate
+that can never go green is not a gate. Revisit only if this repo gains a policy source apm can reach.
+
+It also does not scan for hidden Unicode: that scan is plain `apm audit`, a different mode (`--ci`
+refuses to combine with `--file`/`--strip`/`--dry-run`/`PACKAGE`), and plain `apm audit` here reports
+`No apm.lock.yaml found -- nothing to scan` and exits 0. Adding it would buy a second vacuous check.
+
+### `check-executables-allow-sync`
+
+apm gates a package's `hooks/` and `bin/` on an **exact `<package>#<version>` dictionary lookup** in
+root `apm.yml`'s `executables.allow` (`apm_cli/security/executables.py`, `is_package_approved`).
+There is no wildcard and no version-less form.
+
+So bumping `plugins/kyberforge/apm.yml`'s `version:` without bumping the key **errors nowhere**: the
+entry simply stops matching, the gate blocks the hook, kyberforge's `SessionStart` hook stops
+deploying, and the apm install goes quietly stale — the exact failure ADR-0019 exists to end,
+reintroduced through the mechanism meant to secure it. ADR-0019 records this as a live failure mode;
+the release that shipped the hook hit it immediately.
+
+`scripts/check-executables-allow-sync.sh` parses `version:` out of `plugins/kyberforge/apm.yml` and
+asserts root `apm.yml` carries the matching `kyberforge#<version>` key. A comment in the
+`executables:` block stays as the human-facing pointer; the hook is what actually holds. It parses
+with PyYAML where importable and falls back to a two-shape scan otherwise, so a missing pip package
+cannot become the thing that blocks every push.
+
+## `.claude/settings.json`
+
+**apm owns this file. Nothing repo-authored goes in it.**
+
+`apm audit --ci` replays the install into a scratch tree and diffs the result byte-for-byte, so
+anything apm would not have written there — an `enabledPlugins` block, a real `hooks` entry — is
+permanent drift that fails `apm-audit-ci`. A hook you want in this repo is authored in
+`plugins/<name>/.apm/hooks/` and deployed by apm, never hand-written here.
+
+Its committed content is whatever apm last wrote, which today is the merged `SessionStart` entry for
+kyberforge's `check-apm-current.sh`. That is apm's own output and it belongs in the commit (ADR-0019;
+ADR-0018's statement that the committed content is exactly `{"hooks": {}}` is superseded on that
+point only). Machine-specific settings go in the gitignored `.claude/settings.local.json`, which apm
+does not deploy and the replay does not compare; shared enforcement belongs in
+`.pre-commit-config.yaml`.
+
+### Why it is excluded from `pretty-format-json`
+
+It is the **sixth and last alternation** in that hook's `exclude:` pattern, and the only one there
+for a reason other than "generated manifest". Mind which number you are quoting: **six alternations,
+expanding to sixteen real files** — 3 root marketplace manifests, 2 per plugin × 6 plugins, plus this
+one.
+
+`pretty-format-json --autofix` sorts object keys unless `--no-sort-keys` is passed, while apm's hook
+integrator emits insertion order (`matcher` before `hooks`, `type` before `command`). Leaving the
+file in that hook's scope therefore rewrites apm's output into a form apm would never produce on the
+way into **every** commit, and `apm-audit-ci` then reports permanent drift on a file with an empty
+`git diff` — exactly what happened when the `SessionStart` hook first landed in `2e395a4`. Re-running
+`apm install` fixes the file; leaving it in scope would re-break it on the very commit carrying the
+fix.
+
+**Load-bearing. Do not tidy it out of that list** (see `LESSONS.md`, 2026-08-14).
+
+## Pushing without a network
+
+Exactly **two** pre-push hooks need the network, for one shared reason: root `apm.yml`'s
+`marketplace.packages[]` contains exactly one remote entry — `mattpocock-skills`,
+`source: mattpocock/skills` — and resolving it needs a `git ls-remote`.
+
+| Hook | Offline failure |
+|---|---|
+| `apm-marketplace-check` (`always_run`, resolves every entry) | `No cached refs (offline)` |
+| `apm-pack-check-clean` (re-resolves the same entry) | `Error: Git network timeout during ls-remote` |
+
+Pinning the entry to an exact version does **not** remove the call — an exact pin still ls-remotes.
+`--offline` rescues neither.
+
+To push without a network, skip both using pre-commit's own mechanism:
+
+```
+SKIP=apm-marketplace-check,apm-pack-check-clean git push
+```
+
+**Skip those two alone.** Verified under `unshare -rn`: the other twelve pre-push hooks pass offline
+because they are real local checks. (`check-executables-allow-sync` landed after that run, but reads
+two local manifests and makes no network call.) Adding any other hook to `SKIP` disarms it silently.
+
+`apm-audit-ci` calls `apm` too but stays local: its org-policy discovery resolves nothing on this
+remote *before* any network call, so it does not join the pair above.
+
+---
+
+## See also
+
+- `docs/adr/0020-skill-description-and-body-context-contract.md` — the context contract, its
+  enforcement table (deterministic vs. auditor judgment), and every rejected alternative
+- `docs/adr/0019-session-start-hook-keeps-the-apm-install-current.md` — the `SessionStart` hook, the
+  executable-trust gate, and the version-pinned allow key
+- `docs/adr/0017` / `0015` / `0014` — plugin content sync, apm-generated manifests, committed Vale
+  styles
+- `docs/spec/architecture.md` — directory structure, install pipeline, what is generated and what is
+  hand-authored
+- `.pre-commit-config.yaml` — the hooks themselves, with inline rationale comments