Skip to content

docs: key vendor backend capability matrix, and refuse an unsafe rewrap (CIP-5313) - #1541

Draft
freshtonic wants to merge 8 commits into
james/cip-5312-stash-keyset-create-rewrap-commandsfrom
james/cip-5313-document-key-vendor-backend-trade-offs-capability-matrix
Draft

freshtonic wants to merge 8 commits into
james/cip-5312-stash-keyset-create-rewrap-commandsfrom
james/cip-5313-document-key-vendor-backend-trade-offs-capability-matrix

Conversation

@freshtonic

@freshtonic freshtonic commented Nov 6, 2026 •

Copy link
Copy Markdown
Contributor

Top layer of the CIP-5203 stack. Implements CIP-5313.

  • Capability matrix in the stack-kms-vendor crate docs, linked from stack-kms and every vendor crate. The Go matrix is corrected to match.
  • Docs that had drifted from the code are fixed. This includes ADR-0011's claim that the guest signs the Google JWT, which is false.
  • Updated: the skills/stash-zerokms section, the stash-cli rewrap text, the CLI README stash keyset section, AGENTS.md and SECURITY.md.
  • Fix: a rewrap to a different vendor key under wrapped data keys made every stored value unreadable. It is now refused (RewrapToAnotherKey, CLI code rewrap_refused). A new version of the same key, and any rewrap in derived mode, are still allowed. KeyVendorBackend::same_key decides "same key" for each vendor.

Read first: "## Summary for review" at the end of docs/plans/2026-11-05-key-vendor-backends-decisions.md.

@freshtonic
freshtonic added this pull request to stack #1523 November 6, 2026 02:11
@changeset-bot

changeset-bot Bot commented Nov 6, 2026 •

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 0605831

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 19 packages
Name Type
@cipherstash/auth Minor
stash Minor
@cipherstash/stack Minor
@cipherstash/wizard Minor
@cipherstash/ffi-integration-tests Patch
@cipherstash/basic-example Patch
@cipherstash/e2e Patch
@cipherstash/bench Patch
@cipherstash/stack-drizzle Minor
@cipherstash/stack-prisma Minor
@cipherstash/stack-supabase Minor
@cipherstash/test-kit Patch
@cipherstash/prisma-example Patch
@cipherstash/auth-darwin-arm64 Minor
@cipherstash/auth-darwin-x64 Minor
@cipherstash/auth-linux-arm64-gnu Minor
@cipherstash/auth-linux-x64-gnu Minor
@cipherstash/auth-linux-x64-musl Minor
@cipherstash/auth-win32-x64-msvc Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@freshtonic freshtonic changed the title james/cip 5313 document key vendor backend trade offs capability matrix docs: key vendor backend capability matrix, and refuse an unsafe rewrap (CIP-5313) Nov 6, 2026
@coderabbitai

coderabbitai Bot commented Nov 6, 2026

Copy link
Copy Markdown
Contributor

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

@freshtonic
freshtonic force-pushed the james/cip-5313-document-key-vendor-backend-trade-offs-capability-matrix branch from 7ebb0fd to 5a73cdc Compare November 6, 2026 02:15
@freshtonic
freshtonic force-pushed the james/cip-5313-document-key-vendor-backend-trade-offs-capability-matrix branch from d20b25a to 8367103 Compare November 6, 2026 05:10

This branch has not been deployed

No deployments
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