Repository navigation
MIP-0086: ruleset-sync: every manifest in .github/rulesets/ - #114
Merged
Merged
Conversation
brunogbv
commented
Oct 14, 2026
brunogbv
commented
Collaborator
- ruleset-sync: every manifest in .github/rulesets/, and ci-rule
- Describe main-rule and ci-rule in the docs and CHANGELOG
- ruleset-sync: a response without bypass_actors is an error; when to apply ci-rule
ruleset-sync checks and applies each manifest in .github/rulesets/ (bypass-extras.json excepted), matched on GitHub by name, listing a repo's rulesets once. Output names the ruleset on each line. Repeated --manifest replaces the default set; two manifests with one name are refused. bypass-extras.json, @Consumers included, applies to main-rule alone. ci-rule.json requires ci-ok from GitHub Actions (integration 15368), non-strict, with an explicit empty bypass list so check reports a hand-added actor and apply removes it. main-rule needs 1 approval. The normaliser treats an absent do_not_enforce_on_create as false, drops a null integration_id and sorts the required checks: GitHub's OpenAPI schema makes both fields optional, so a live response may omit them. Tested: scripts/ruleset-sync.sh --self-test (20 failures before the change, ok after); devkit-quality.sh rc=0 Co-Authored-By: Claude <[email protected]>
The README, docs/1-design.md, the ruleset-sync row in docs/4-reference_tools.md and the devkit.just recipe comments name both rulesets. CHANGELOG opens v0.10.0 with what rulesets-apply changes after the release. Closes #112 Tested: devkit-quality.sh (ruff, shellcheck, actionlint, self-tests, agents-check, docs-lint, skills check, release --check, merge-key-check) rc=0 Co-Authored-By: Claude <[email protected]>
…pply ci-rule Tested: ruleset-sync.sh --self-test (new case red, then green); devkit quality gate Co-Authored-By: Claude <[email protected]>
github-actions
Bot
changed the title
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
1 participant
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.