Skip to content

feat(governance): safely provision missing shared rulesets - #42

Merged
vitormattos merged 2 commits into
mainfrom
feat/safe-default-ruleset-onboarding
Nov 4, 2026
Merged

vitormattos merged 2 commits into
mainfrom
feat/safe-default-ruleset-onboarding

Conversation

@vitormattos

@vitormattos vitormattos commented Nov 4, 2026 •

Copy link
Copy Markdown
Member

Goal

Add safe, explicitly authorized onboarding of common organization rulesets without touching product-specific CI, DCO policies or existing protections.

Behavior

  • --onboard-only --apply creates only a missing organization default ruleset. Caller configuration defines the policy; no organization, product, or bot IDs are hardcoded in the engine.
  • Requires an initialized default branch and a privileged classic branch-protection preflight. Legacy protections are reported for review and never removed.
  • Existing common rulesets are never modified. Any divergent or additional rulesets require review before onboarding; existing repository metadata and product CI remain untouched.
  • Conditional direct-push bypasses (such as nextcloud-bot when appinfo/info.xml is present) are preserved. Additional active rulesets lacking the same direct-push exemption cause a fail-closed review result.
  • Existing rulesets that require signed commits are parsed read-only, preventing the onboarding inventory from failing on MCP repositories with signature rules.
  • Action input onboard-only exposes the safe mode to caller workflows. No credentials are stored in this repository.

Regression and validation

Security and rollout

Privileged writes require explicit --apply with an administration-authorized token. The caller must use a protected environment and scope credentials to the selected repository when practical. No organization-wide apply is performed by this PR. Commit signatures and DCO verified.

…light

Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
@vitormattos
vitormattos merged commit 0789640 into main Nov 4, 2026
8 checks passed
@vitormattos
vitormattos deleted the feat/safe-default-ruleset-onboarding branch November 4, 2026 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant