Repository navigation
fix(governance): refresh zi exception pins, skip opensource.org links - #777
Merged
Merged
Conversation
Zi moved to protected main in z-shell/zi#631, which changed its pull-request template and contributing guide and deleted the promotion template, so the daily Community Health Audit reported drift. Refresh both blob IDs, restate their reasons for the ADR-0039 model and drop the promotion exception. opensource.org answers 403 to automated clients, which fails Lychee on the charter link; exclude the domain.
ss-o
commented
Oct 9, 2026
ss-o
left a comment
Member
Author
There was a problem hiding this comment.
Fallback review under ADR-0026: Copilot request not registered on 430e3e7
Findings
No actionable findings.
Scope: the complete diff of 430e3e7 against main at 6f37298.
Note
This is a self-review by the pull request's author, not an independent review or approval to merge.
Validation
- Passed: Community Health Audit (no drift in any repository, z-shell/zi included), Repo Settings Audit Tests, Validate Agent Instructions, Org Routing, CodeQL and the four Policy checks on 430e3e7.
- Failed, expected:
links-checkreports the same two opensource.org 403 errors asmain. Lychee runs onpull_request_target, so this check reads.github/lychee.tomlfrommain, not from this pull request (its run excluded 7 links, the count for themainconfig). Lychee v0.18.1, the version the workflow uses, run locally with this branch's config on the same 1432 links: 0 errors, 9 excluded (the 7 plus the two opensource.org links). The check onmainafter merge is the hosted proof. - Copilot: requested at 2026-10-09T19:09:20Z; no
review_requestedevent for Copilot followed (only the CODEOWNERS team request at 19:09:08Z).
Checklist verdicts and supporting evidence
- Blob IDs: match zi
main..github/PULL_REQUEST_TEMPLATE.mdis 3c4dc44 anddocs/CONTRIBUTING.mdis 5b318bf at the contents API on z-shell/zimainafter z-shell/zi#631;.github/PULL_REQUEST_TEMPLATE/promotion.mdreturns 404 there. - Reasons: accurate. The zi template states the protected
mainmodel and keeps the Migration plan section; the zi guide covers the branch model, Zsh verification and signed milestone tags, which the organization guide does not. - Exclusion: narrow. Only
^https://opensource\.org/is excluded; the charter text is unchanged and the domain answers 403 to curl with and without a browser user agent.
Limits
- The opensource.org exclusion stops checking that domain at all; a link that really breaks there will not be reported.
This was referenced Oct 9, 2026
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
mainin chore: cut over Zi integration to protected main zi#631. That changed.github/PULL_REQUEST_TEMPLATE.mdanddocs/CONTRIBUTING.mdand deleted.github/PULL_REQUEST_TEMPLATE/promotion.md, so the daily Community Health Audit reported drift for z-shell/zi. This refreshes both blob IDs to the files on zimain, restates their reasons for the ADR-0039 branch model, and removes the promotion template exception..github/lychee.tomlnow excludes that domain; the charter text is unchanged.Verification
ruby automation/governance/repo-settings-audit.rb --all-repos --community-health --workflow-pins --skip-settings --fail-on-drift: exit 0, no drift in any repository.lychee --config .github/lychee.toml './**/*.md*' './**/*.html': 1383 links, 0 errors.ruby automation/governance/test-repo-settings-audit.rb,knowledge-delivery.py --checkandknowledge-coverage.pypass.No handoff needed.