docs: propose 0025 CI routing gaps for docs, governance, and tooling - #12
Merged
Merged
Conversation
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.
Proposes decision record 0025: two CI-routing gaps and the narrow fixes.
Why
PR #11 measured the status quo: 39m33s, Full profile, success, for two new docs files (validation run 35787826906). Two findings explain the price:
Get-HumCiChangeProfilesends every non-ordinary-modification to Full; both docs: accept 0024 performance north star; seed perf-debt ledger #11 files were additions.docs/decisions/,docs/research/,workorders/,.github/, most ofdocs/— and all oftools/— are unregistered. Every CI-tooling PR, which was most of today, pays Full.What the record proposes
Invoke-HumFixedProfileadds the hygiene group unconditionally;hum-dev evidence <profile>dispatches totools/check_all.ps1 -EvidenceTier <Profile>) and records the split it produces: docs/ and tools/ consumers are already covered at language rank; workorders/ boundary classifier is not, so its implementation must wire it in or keep workorders/ at a higher rank.Review
Claude pre-issuance review passed at 06a4f85: all four required fixes landed, branch rebased onto current main (f586f7e), diff is exactly the one new file with no deletions. Researcher-owned; no builder-lane files.
Expected CI
Full (~40 min) — by the very rule this record documents. If accepted, the implementation goes to the builder lane, and docs/tooling PRs stop paying Full.