LearnNewsExamplesServices
Frontmatter
titlefix(skills): correct relative section-ref paths in audit files (#12588)
authorneo-opus-grace
stateClosed
createdAtJun 5, 2026, 8:31 PM
updatedAtJun 5, 2026, 8:43 PM
closedAtJun 5, 2026, 8:43 PM
mergedAt
branchesdevclaude/12588-skill-audits-ref-paths
urlhttps://github.com/neomjs/neo/pull/12589
Closed
neo-opus-grace
neo-opus-grace commented on Jun 5, 2026, 8:31 PM

Resolves #12588

Authored by Claude Opus 4.8 (Claude Code). Session 0f6d0fa0-327f-42ec-b970-e32f388699b4.

Corrects relative section-ref paths in three skill audits/-resident files. They referenced the sibling references/ directory as references/<file>.md §N (skill-root-relative), but from audits/ the resolvable path is ../references/<file>.md §N. ~13 refs across 3 files; a pure relative-path correction — no anchor or prose-content change. This is the legitimate path-fix PR #12581 originally attempted (dropped pre-merge when its bundled §-ref restyle was flagged in review).

Evidence: L1 (static lint audit — lint-skill-manifest --base HEAD~1 → OK; every rewritten target + numeric anchor resolves) → L1 required (doc-path correctness, no runtime ACs). No residuals.

Micro-change exemption (§6.1): pure documentation, no runtime impact — cross-family review welcome but not merge-gating.

Deltas from ticket

None — implements #12588 one-to-one.

Slot-Rationale (Substrate-Mutation Pre-Flight — AGENTS.md §13)

Touches .agents/skills/ideation-sandbox/audits/consensus-mandate.md, .agents/skills/ideation-sandbox/audits/tier-2-revalidation.md, .agents/skills/pull-request/audits/consensus-gate-mirror.md — conditionally-loaded World-Atlas audit files (read-on-demand, not always-loaded turn substrate).

  • Modified (all 3 files): disposition rewrite — corrected the relative path of existing section-refs (references/../references/). No section added or removed; no prose/anchor change; zero always-loaded substrate growth (+3 chars ../ per ref). Reason for the shift: the paths were file-relative-incorrect from audits/; the #12584 named-section-ref lint flags them on next edit, so this pre-empts CI breakage.

Test Evidence

  • node ai/scripts/lint/lint-skill-manifest.mjs --base HEAD~1[lint-skill-manifest] OK (exit 0).
  • Seam grep: 0 broken references/…§ refs remain in the three audits/ files; the 13 refs now use ../references/…§, all targets + numeric anchors resolving.
  • Pure docs — no unit/e2e tests apply.

Post-Merge Validation

  • lint-skill-manifest + lint-pr-body green on the PR head.

Commits

  • 535772e3d — fix(skills): correct relative section-ref paths in audit files (#12588)
neo-opus-grace
neo-opus-grace commented on Jun 5, 2026, 8:43 PM

Closing — stale-base authoring error on my part. My local dev was at session-start 1cdda8feb vs current origin/dev 216cea943.

On current dev, consensus-mandate.md already carries the ../references/ fix via #12581 (named-section form) — I'd earlier mis-concluded #12581 dropped those changes, but that was a stale read. This PR's consensus-mandate.md edits would conflict-or-revert #12581.

The valid remaindertier-2-revalidation.md + consensus-gate-mirror.md (still broken references/… on dev; #12581 missed them) — I'll redo on a fresh base under #12588. Apologies for the churn; reinforces the verify-against-fresh-dev discipline.