Skip to content

feat(evidence): record sensitive break-glass audit receipt - #121

Merged
seonghobae merged 16 commits into
feat/sensitive-break-glass-policyfrom
feat/sensitive-break-glass-evidence
Sep 4, 2026
Merged

seonghobae merged 16 commits into
feat/sensitive-break-glass-policyfrom
feat/sensitive-break-glass-evidence

Conversation

@seonghobae

@seonghobae seonghobae commented Aug 12, 2026 •

Copy link
Copy Markdown
Contributor

Partial implementation of #10, stacked on PR #119.

Dependency / live alignment

Current live prerequisite is exact PR #119 head ba16263f58e8c21df909e881b390cc094baa6642. Current #121 head is 6154bebb451e2a166b0ffb2a7ac913fd0d8f8d6c.

This head merges the current prerequisite without rewriting history. Fresh comparison reports ahead 16 / behind 0, with merge base exactly the live prerequisite. The delta remains limited to four #121-owned paths: CHANGELOG.md, crates/originweave-evidence/src/lib.rs, crates/originweave-evidence/src/sensitive_break_glass.rs, and crates/originweave-evidence/tests/sensitive_break_glass_evidence.rs.

Keep Draft while #119/#90 and their prerequisite sensitive-data/model-policy stack remain active. No prerequisite or predecessor check, review, status, coverage, mergeability, or synthetic-merge evidence transfers.

Implemented contract

SensitiveBreakGlassEvidence is an immutable credential-free receipt for one actual exceptional disclosure. Construction validates exact request/decision authority, beneficiary actor binding, reason, approval cardinality, bounded authority interval, decision/disclosure ordering, heightened-monitoring correlation, mandatory review metadata, and retention extending beyond review. Malformed identifiers, field sets, actor or approval mismatch, invalid timing, non-disclosure outcomes, and insufficient retention fail closed.

Current-head verification

On exact head 6154bebb451e2a166b0ffb2a7ac913fd0d8f8d6c against exact live prerequisite ba16263f58e8c21df909e881b390cc094baa6642:

  • cargo fmt --all -- --check: success;
  • cargo test --locked --workspace --all-features: success;
  • cargo clippy --locked --workspace --all-targets --all-features -- -D warnings: success;
  • RUSTDOCFLAGS='-D warnings' cargo doc --locked --workspace --all-features --no-deps: success;
  • GitHub reports this Draft mergeable;
  • formal reviews currently returned: none; and
  • exact-head CI: pending.

The prior successful run 32039944414 belongs to the previous head and is historical only. Security/SAST/central review workflows absent for this stacked Draft are absent, not passing. No predecessor-head, prerequisite-head, queued, pending, skipped, cancelled, stale, synthetic, status-only, or model-only evidence is promoted as current proof.

Truth boundary

This is immutable credential-free evidence metadata only. It does not authenticate actors or approvers, prove policy execution, disclose protected values, execute monitoring or review, durably persist or hash-chain evidence, enforce retention/deletion, or complete issue #10. Trusted runtime and evidence-owner components must perform those responsibilities.

Passing automation is not independent approval. This scheduled actor does not merge, self-approve, alter workflows, add secrets, weaken checks, tag, or publish.

@coderabbitai

coderabbitai Bot commented Aug 12, 2026 •

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

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.

Preserve PR #121's immutable credential-free evidence tree while adopting the current PR #119 break-glass policy head after dependency-root ownership repair.
Merge the current break-glass policy prerequisite while retaining the evidence contribution.

Signed-off-by: Seongho Bae <me@seonghobae.me>
@seonghobae
seonghobae marked this pull request as ready for review September 4, 2026 20:05
@seonghobae
seonghobae merged commit 37f7b5a into feat/sensitive-break-glass-policy Sep 4, 2026
4 checks passed
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