Skip to content

fix(auth): publish verified proofs at the commit boundary#192

Open
marcus-pousette wants to merge 1 commit into
fix/atomic-local-auth-proof-handofffrom
fix/auth-stage-verified-proofs
Open

fix(auth): publish verified proofs at the commit boundary#192
marcus-pousette wants to merge 1 commit into
fix/atomic-local-auth-proof-handofffrom
fix/auth-stage-verified-proofs

Conversation

@marcus-pousette

@marcus-pousette marcus-pousette commented Jul 11, 2026

Copy link
Copy Markdown
Collaborator

Problem

Local authorization happens before a database write commits. verifyOps previously published proof material immediately, so a failed or retried local proposal could leave durable proof state for an operation that never committed.

What changes

  • verifyOps validates operations without publishing proof material.
  • onVerifiedOps publishes accepted proof material to the in-memory cache and optional store.
  • Remote sync keeps its proof-first order: verify, publish proof, then apply the batch.
  • Local backends can use fix(auth): expose local proofs for atomic backend commits #187's proof handoff and commit the operation and proof in one database transaction.
  • The local signing cache remains available for forwarding committed operations without signing again.

Fast paths

  • Configurations without an external proof store perform no new durable work.
  • Remote sync still performs one proof publication per verified batch.
  • Local signing retains its in-memory cache.

Tests

  • repeated local proposal validation has no proof-store side effect
  • onVerifiedOps publishes the accepted proof exactly once
  • remote verified-batch publication remains supported

Stack

Stacked on #187. SQLite and PostgreSQL transaction implementations follow from this commit boundary.

@marcus-pousette marcus-pousette added bug Something isn't working area:auth Authorization, capabilities, proofs, signatures, crypto, and privacy labels Jul 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:auth Authorization, capabilities, proofs, signatures, crypto, and privacy bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant