Skip to content

Require mirroring policy changes in Axiom RuleSpec - #1877

Merged
MaxGhenis merged 2 commits into
mainfrom
axiom-parity-guidelines
Oct 4, 2026
Merged

MaxGhenis merged 2 commits into
mainfrom
axiom-parity-guidelines

Conversation

@MaxGhenis

@MaxGhenis MaxGhenis commented Sep 26, 2026 •

Copy link
Copy Markdown
Collaborator

Adds the contributor rule that any PR changing policy in a country model must also leave the same provision correct in the matching Axiom rulespec repo, stated with one axiom: line in the PR description (encoded-correct / encoded / queued / n/a). queued is allowed only when the signed encoder is blocked (the blocker is recorded in the issue; each billed encoder run needs separate approval), and a queued rulespec issue must be dispatch-ready (module path and corpus citation, verbatim law, outputs, companion tests from the same external source as the PE tests) and labelled pe-parity. Example: TheAxiomFoundation/rulespec-us#1416. External contributors who cannot complete the Axiom work may write axiom: needed; a maintainer follows up.

axiom: n/a: contributor documentation only

🤖 Generated with Claude Code

MaxGhenis and others added 2 commits September 26, 2026 09:49
Adds the axiom: PR-description line (encoded-correct | encoded | queued | n/a)
and the dispatch-ready standard for queued rulespec issues.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Per the independent review: use queued only when the signed encoder is
blocked, record the blocker in the issue, and require separate approval
for each billed encoder run. The PR template and the github-prs skill
also allow external contributors to write `axiom: needed`, and the
template's YAML-test checkbox applies only to policy changes.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@MaxGhenis

Copy link
Copy Markdown
Collaborator Author

Merge audit: head af92247.

  • Checks: gh pr checks exits 0 on the latest run (all 7 pass). mergeable is MERGEABLE, merge state CLEAN, not a draft, no CHANGES_REQUESTED review.
  • Review: round 1 (Codex) requested changes; commit af92247 applies them (queued only when the signed encoder is blocked, blocker recorded, separate approval for each billed encoder run; axiom: needed for external contributors in the PR template and docs/engineering/skills/github-prs.md; YAML-test checkbox scoped to policy changes). Round 2 is an independent hard-tier Subfleet review (job 20261003-222611-axiom-parity-r2) with verdict VERDICT PolicyEngine/policyengine-uk#1877 af92247 APPROVE.
  • Dependency: Require mirroring policy changes in Axiom RuleSpec .github#6 merged first (a8e190d), so the #mirror-policy-changes-in-axiom link here now resolves.
  • Not a duplicate: Require published numbers to come from model runs .github#8 is the separate published-numbers rule; this PR is the UK half of .github#6.

@MaxGhenis
MaxGhenis merged commit 8997f3a into main Oct 4, 2026
7 checks passed
@MaxGhenis
MaxGhenis deleted the axiom-parity-guidelines branch October 4, 2026 10:13
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