feat(integration): add pre-execution plugin delivery revocation fence - #257
Draft
seonghobae wants to merge 17 commits into
Draft
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 8, 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 gap
Advances #130 immediately after #256 with the next LifeOS-owned delivery-lifecycle boundary: an exact per-attempt revocation fence immediately before provider execution. The fence revalidates the worker's active finite claim together with the exact active delivery-origin grant and active owning installation under the same workspace/user scope.
This slice remains Integration-owned. Passing the fence is not outbound-network authorization, does not materialize provider credentials, and does not authorize DNS/IP/redirect/proxy/connect behavior. A later executor must still consume only immutable released/versioned canonical egress authority and otherwise remain fail closed.
RED → causal repair → exact GREEN
Direct parent is #256 exact
0c6221e1fe32bf6594784219a7bd710ad0c1a993. The original regression lineage beginningd2dee04c791dcd31f8ff136fb8686fae302c013aestablishes the missing execution-fence capability. The retained implementation binds exact delivery/workspace/requesting-user identity and claim-token digest, requires an unexpired active claim, and joins the exact active delivery-origin grant plus active owning installation. Real PostgreSQL acceptance proves success while all authority remains active and fail-closed rejection after grant or installation revocation without mutating the accepted claim.Exact
61c179a5886a79e2fa6bbca53b704275bcb3a0ea, run34266240410, job102196214538, completed GREEN on Ubuntu 24.04/PostgreSQL 16 across exact checkout, frozen install, canonical repository formatting, Plugin SDK build, focused execution-fence/PostgreSQL acceptance, Integration typecheck and the complete Integration service suite.Fresh authority review then found a separate opaque-capability defect: the application parsed the raw UUIDv4 claim token with a case-insensitive UUID validator and lowercased it before SHA-256 hashing. A hex-case alias could therefore acquire the same durable claim digest even though the contract calls this an exact raw-claim-token-bound capability. Regression-only
48b823903d14c3ca0c868bf335d023904b2dcf9drequires a case-aliased raw token to fail before the store receives any durable-authority lookup. Hosted run34267014048, job102198804685, acquired a real Ubuntu 24.04/PostgreSQL runner, passed exact checkout, frozen install, formatting and Plugin SDK build, then failed at the focused execution-fence acceptance as intended; typecheck/full suite were skipped after that RED.Minimum causal repair
a40098db19ea9830367b6cabd1bc552a019a990achanges only raw claim-token handling: the token must already be canonical lowercase UUIDv4 text and its exact bytes are hashed without normalization. Other UUID identity normalization and the repository SQL/durable schema remain unchanged. Exact run34267152261, job102199270687, completed GREEN across exact checkout, frozen install, formatting, Plugin SDK build, focused execution-fence/PostgreSQL acceptance, Integration typecheck and the complete Integration suite. Current exact5ac3d2fcdf30e97be006405cf599f3d666cf5946removes only the now-purpose-complete verifier; production/test/repository evidence is unchanged from the proven head.Deliberate boundary
This fence does not consume provider credentials or authorize outbound HTTPS. Durable origin/delivery/claim/retry/outcome/control/fence identity is not DNS/IP/redirect/proxy/connect-time network authority. A later executor must consume an immutable released/versioned canonical egress contract and retain finite I/O, retry/outcome, revocation, recovery and operator evidence.
Keep Draft. #256 must integrate normally first and the unchanged descendant must satisfy current repository/security/central workflow evidence plus independent review/thread authority after prerequisite integration. No self-approval, administrator bypass, force-push, destructive rebase, mutable dependency, source copy, cross-service SQL, stale evidence reuse or gate weakening.
Refs #130, #252, #253, #254, #255, #256; ContextualWisdomLab/EgressWeave.