Skip to content

Record approvals without identity constraints - #34

Merged
b1rdmania merged 1 commit into
mainfrom
codex/record-approval-without-independence
Aug 25, 2026
Merged

Record approvals without identity constraints#34
b1rdmania merged 1 commit into
mainfrom
codex/record-approval-without-independence

Conversation

@b1rdmania

Copy link
Copy Markdown
Owner

Summary

  • retain required, hashed approval and reviewer identities while removing human-prefix and author/reviewer-separation rules
  • make the weaker attribution-only guarantee explicit in gate, ledger, support, and admission-plan docs
  • rewrite the existing identity-policy tests to assert the new contract and keep blank identities fail-closed
  • bump the package to v0.1.0a11

Verification

  • python3.12 -m pytest -q (303 passed)

@coderabbitai

coderabbitai Bot commented Aug 25, 2026

Copy link
Copy Markdown

Important

  • 馃攳 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

鈿欙笍 Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 3b363017-8bda-42c0-8229-fd481d4463e5


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

鉂わ笍 Share

Comment @coderabbitai help to get the list of available commands.

@b1rdmania
b1rdmania merged commit ab4a826 into main Aug 25, 2026
2 checks passed
@b1rdmania
b1rdmania deleted the codex/record-approval-without-independence branch August 25, 2026 20:57
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