Charles Wiltgen 7482a0273d fix(codex): ship the suites Codex skills already point at, and fix the pointers ...

Three suites (axiom-tools, axiom-shipping, axiom-apple-docs) were excluded from
the Codex build. That was not a decision — the original list (2b5d6adb) was a
blanket "Router skills — Codex has native progressive disclosure, so these are
unnecessary". The premise was abandoned and the list curated, but these three
survived: two still annotated "Codex fit not yet assessed", and axiom-tools
carried a reason written when it was named axiom-using-axiom and held ONLY
discipline text. It then absorbed six sub-skills, five of them harness-neutral
tool references, all inheriting an exclusion that predated them.

Measured cost: 41 dangling "see X" pointers plus ~94 name citations across ~40
emitted Codex skills, aimed at files the build never wrote.

- CODEX_EXCLUDED_SUITES is now empty; per-FILE exclusion replaces it, holding one
  entry — getting-started.md, which drives AskUserQuestion and cites 15 /axiom:
  commands, and has zero inbound references.
- dropExcludedSubSkillRows strips a router's table row for a withheld sub-skill,
  so per-file exclusion cannot recreate the defect it exists to avoid.
- rewriteInlineAuditorBlockForCodex translates auditor file paths to Codex skill
  names; that block previously told Codex users to read files this build skips.
- rewriteAgentInvokesForCodex fixes the layer under that: routers said "Launch
  `build-fixer` agent or `/axiom:fix-build`" while `axiom-fix-build` — the thing
  that exists there — appeared zero times. Neither agents nor /axiom: commands
  exist on Codex, and the mapping is not derivable (spm-conflict-resolver →
  axiom-resolve-spm). Applied to routers, agent-skills, and sub-skills, since a
  verbatim copy reintroduces it. Two commands have no Codex equivalent at all
  (console, compare-traces) and are labelled Claude Code-only rather than
  redirected somewhere that does a different job.
- agentToSkillName moved into the shared module so the pre-deploy gate derives
  the agent-skill set instead of guessing from names — a name heuristic
  misclassifies both ways (axiom-health and axiom-testing are routers that read
  verb-first; axiom-swift-simplifier is an agent-skill that does not).
- The count test now derives the agent total from the agents directory; the
  hardcoded 41/68 had drifted from a real 42/69 and passed anyway by being
  self-consistent.

2026-08-15 12:30:00 -07:00

7.3 KiB

Raw Permalink Blame History

View Raw