Skip to content

feat(protected-content): Runtime release authority and persisted state - #34

Closed
irzhywau wants to merge 3 commits into
feat/protected-content-wallet-rightsfrom
feat/protected-content-runtime
Closed

irzhywau wants to merge 3 commits into
feat/protected-content-wallet-rightsfrom
feat/protected-content-runtime

Conversation

@irzhywau

@irzhywau irzhywau commented Aug 26, 2026 •

Copy link
Copy Markdown
Collaborator

Note

This is part of a stacked PR chain — review only the commits of this checkpoint; earlier work is in the PRs below it in the stack.

Stack (top → bottom)

Base: feat/protected-content-wallet-rights ← Head: feat/protected-content-runtime (checkpoint 8 of 10)

Commits

  • 7f60911 feat(protected-content): persist Runtime release state
  • b00bfee feat(protected-content): coordinate Runtime release authority

@irzhywau

Copy link
Copy Markdown
Collaborator Author

One commit was added on top of b00bfeeb: a8ac6dc8, the merge of the updated parent feat/protected-content-wallet-rights (checkpoint 7 tip, CI-green). No fixes were needed on this checkpoint itself.

Review notes: the new source-only elastos-protected-content-runtime crate holds together well — RuntimeReleaseCoordinator rejects any Wallet exchange that is not the dedicated rights-signature operation and cross-checks the canonical rights request against the signed operation statement; the journal persists the full operation before any provider effect (crash-safe resume: replayed terminals return as-is, started-but-unfinished effects return a typed nonterminal); provider results are validated against the pinned Runtime issuer, node decisions verified through the authenticated operation (denial = journaled terminal), contributions identity-checked per node, and every provider failure collapses to a journaled fail-closed nonterminal. Zero panics/unwraps in non-test code; the journal reuses the established owner-only/create_new/fsync/O_NOFOLLOW discipline; the crate natively uses the post-checkpoint-6 pinned-issuer contracts API. 19 tests.

Verified locally at the tip: all source gates incl. the provisional-retirement guard, cargo fmt --check, cargo clippy --workspace --all-targets -- -D warnings (cleaned fingerprints), cargo test --workspace — 2003 passed; only the known macOS mDNS-environment room::direct pair, green on this CI run.

@irzhywau irzhywau added this to the 0.7.x milestone Aug 28, 2026
@irzhywau irzhywau closed this Aug 31, 2026
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