Skip to content

[policy] feat(maintainers): add risk-based PR evidence gate#33

Merged
heimanba merged 2 commits into
mainfrom
codex/maintainer-friendly-gates
Jul 19, 2026
Merged

[policy] feat(maintainers): add risk-based PR evidence gate#33
heimanba merged 2 commits into
mainfrom
codex/maintainer-friendly-gates

Conversation

@heimanba

Copy link
Copy Markdown
Contributor

Summary

  • Add a lightweight evidence check for changes to provider contracts, configuration/schema parsing, release/package files, dependencies, and GitHub workflows.
  • Keep ordinary documentation, examples, and small isolated changes free of this extra merge barrier.
  • Add a maintainer review guide and make the check part of the existing Gate.

Why

AI-assisted contributions can produce a large, plausible diff quickly. This gives maintainers a concise statement of the behavior and validation evidence before reviewing high-blast-radius changes, without requiring AI disclosure, a design document, or an Issue for routine contributions.

Behavior / risk

High-risk PRs without a non-empty ## Behavior / risk and ## Validation section now fail the aggregate Gate. Low-risk paths continue to pass without those sections. The evidence check itself is included in the existing policy-surface protection.

Validation

  • bun run verify:push
  • bun test scripts/pr-evidence.test.ts
  • bun test --timeout 20000 scripts/open-source.test.ts
  • bun run typecheck:scripts
  • Parsed .github/workflows/ci.yml with Ruby YAML and ran git diff --check.

Change-Id: I6fca63510c1953053a0feddd77b4bdff29c64191
Comment thread scripts/pr-evidence.ts Fixed
Change-Id: I8a21b384a5d0a18f8069becce1464cefaa1250ae
@heimanba
heimanba marked this pull request as ready for review July 19, 2026 07:19
@heimanba
heimanba merged commit 9ee0f7f into main Jul 19, 2026
16 checks passed
@heimanba
heimanba deleted the codex/maintainer-friendly-gates branch July 19, 2026 07:20
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.

2 participants