feat(integration): compose authenticated Vault plugin operator authority - #243
Draft
seonghobae wants to merge 10 commits into
Draft
feat(integration): compose authenticated Vault plugin operator authority#243seonghobae wants to merge 10 commits into
seonghobae wants to merge 10 commits into
Conversation
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: trueThanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
This was referenced Sep 3, 2026
This was referenced Sep 4, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Buyer/security outcome
Advances #130 beyond a standalone Vault adapter by composing it with authenticated Plugin operator authority. The composition reads verifier/Vault configuration only from Integration-owned configuration, combines signed workspace+user authority with durable replay consumption, and routes credential plaintext only through
PluginVaultSecretStore; durable LifeOS metadata retains only opaque secret references.Current stack
Parent #242 is now
e753de422be91d0962a8fd260c5ce366994acb54. Current exact headd03b886f67552e6bb2b53352087d2041dedba07cis a non-force two-parent descendant of prior #243 heada2a8b5395c843fee6438f173659b9409d240a545and current #242. Fresh compare from the parent is ahead-only (behind_by=0) with the same six Integration/ADR/architecture paths. No predecessor check/review evidence transfers.This slice preserves its authenticated Vault operator composition,
INTEGRATION_PLUGIN_VAULT_*-only configuration authority and Proposed ADR 0004. Hosted PostgreSQL bootstrap remains a successor rather than being pulled backward into this bounded slice.Current hosted successor #244 is
f0b91869f306b1b7d0d5759307acf347e413aa06; concrete PostgreSQL/default-entrypoint successor #245 is45b5ad55b1575a677c9edf65fc350e55e3c68f8f. #245 now preserves both one-time query authority and one-time shutdown authority. Latest RED95c23186b09114e4ae01b6a6eaf1c8e4aec126ba→ repair0c6065378a57c99574694e203463b5111847238ccloses the statefulendaccessor gap; exact successor coverage also rejects throwing/non-callable shutdown accessors without reflecting native detail.The
apps/integration-servicefrozen-lock importer still lacks directpg/@types/pgevidence, so REDa0b5653426f0d55e50f6baff6ad66543a57d932fremains open. Earlier whole-lock churnf3bc29112e691e1c30008c4e4132b486f20567ffwas preserved and reverted byb6c1befd8fea6b9943d95461b9f19b1d9f4f5061; it is not accepted repair evidence.Keep Draft until prerequisites integrate normally and exact-head package/coverage/security/review plus real PostgreSQL/Vault/TLS acceptance are reacquired. No source copy, cross-service SQL, mutable dependency, self-approval, bypass or force-push.
Refs #130, #205, #235, #241, #242, #244, #245.