fix(ci): unbreak estate-rules + A2ML gates introduced by #49 - #50
Merged
Conversation
The tier-2 bundle (#49) left two required gates red on main: - estate-rules "AsciiDoc under docs/" flagged the Documenter source pages (docs/src/*.md). Documenter's native source format is Markdown (it cannot consume .adoc), so docs/src/ is a legitimate carve-out exactly like the docs/wiki/ GitHub-wiki mirror. Exempted docs/src/ in the check + comments. - A2ML validation: .machine_readable/policies/MAINTENANCE-AXES.a2ml and MAINTENANCE-CHECKLIST.a2ml were missing the required identity field. Added name + project to each [metadata] block. Verified: `find docs -name '*.md' -not -path 'docs/wiki/*' -not -path 'docs/src/*'` is empty; both policy files now carry name+project identity. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01UPFC9YQ7g9gc3VnRox42Q1
The docs/ environment Manifest is machine-generated and resolved fresh in the Documentation workflow from docs/Project.toml; it should not be tracked (mirrors the existing root /Manifest.toml ignore, which is root-anchored and so did not catch the nested docs/ one). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01UPFC9YQ7g9gc3VnRox42Q1
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
The tier‑2 bundle (#49) merged with two required gates red; they're now failing on
mainand will fail every subsequent PR's CI. This is a minimal follow‑up that closes both — no functional change to the tier‑2 work.Changes
estate-rules.yml— the "AsciiDoc by default underdocs/" check flagged the Documenter source pages (docs/src/*.md) that Axiom flagship (4) — estate tooling: Aqua/JET/Documenter/test-gate + k9/panic-attack/RSR #49 added for the new docs build. Documenter's native source format is Markdown — it cannot consume.adoc— sodocs/src/is a legitimate carve‑out, exactly like the existingdocs/wiki/GitHub‑wiki mirror. Added-not -path 'docs/src/*'to the check and updated the comments/step name to document the carve‑out..machine_readable/policies/MAINTENANCE-AXES.a2ml+MAINTENANCE-CHECKLIST.a2ml— A2ML validation requires an identity field (agent-id/name/project); these two were missing it. Addedname+projectto each[metadata]block.Verification (local)
find docs -name '*.md' -not -path 'docs/wiki/*' -not -path 'docs/src/*'→ empty (no disallowed.md;docs/legal/*.txtanddocs/governance/*.adocare unaffected).name+projectin[metadata](the exact fields the validator's error named).Why this happened
Two parallel workers built #49 on disjoint file‑sets; the docs worker added
docs/src/*.md(Documenter) while the compliance worker wrote theestate-rulesdocs check — neither saw the other's additions. Caught here on the first real CI run and fixed at the workflow level.🤖 Generated with Claude Code
https://claude.ai/code/session_01UPFC9YQ7g9gc3VnRox42Q1
Generated by Claude Code