Repository navigation
docs: propose workload identity and bounded enforcement - #69
freeqaz-openai wants to merge 7 commits into
Conversation
|
🦞👀 Pull request received. I will update this pull request when review starts. ClawSweeper review completeClawSweeper finished reviewing this revision. The review result is being finalized. |
|
Codex review: blocked before merge. What this changesThis PR proposes execution-bound workload authentication, finite authority leases, and staged enforcement for Enterprise Agents. Example: Agent execution A is replaced by execution B while A's certificate remains valid.
Review scores
ProductKind: Docs · Worth it: Needs a maintainer decision Merge readiness⛔ Blocked before merge - 1 item remains Keep this PR open for an Enterprise design decision: its proposed authentication and lease contracts are distinct from current main, and the RFC remains unaccepted. Priority: P3 Decision needed
Before merge
FindingsNone. Agent review detailsHow this fits togetherThe proposed boundary sits between Enterprise Agent executions, OCC authorization, and accepting services; it consumes authenticated execution evidence and current policy to permit or deny exact operations. flowchart LR
Compute[Compute evidence] --> Registration[Execution registration]
Registration --> Identity[Execution identity]
Identity --> Service[Accepting service]
OCC[Current OCC authority] --> Lease[Finite authority lease]
Lease --> Service
Service --> Effect[Protected operation]
Technical reviewBest possible solution: Resolve the proposed authentication and lease direction through the repository's RFC acceptance process while preserving the existing baseline until acceptance. Do we have a high-confidence way to reproduce the issue? This PR proposes new contracts rather than repairing a reproduced failure; no target code was run. Is this the best way to solve the issue? The proposal preserves baseline authentication and separates identity, authorization, withdrawal, and termination, but adopting its optional profiles remains an owner decision. AGENTS.md: not found in the target repository. Codex review notes: model internal, reasoning medium; reviewed against 967d9aac7472. SecurityNone. EvidenceWhat I checked:
Likely related people:
LabelsLabel changes: No label changes. Label justifications:
Rating scale6/6 🦀 challenger crab · 5/6 🦞 diamond lobster · 4/6 🐚 platinum hermit · 3/6 🦐 gold shrimp · 2/6 🦪 silver shellfish · 1/6 🧂 unranked krab. Overall follows the weaker of proof and patch quality; ✨ marks media proof (a screenshot, video, or linked artifact) that directly shows the changed behavior. WorkflowClawSweeper edits this one comment on every review. Comment HistoryReview history (9 earlier review cycles; latest 8 shown)
Reviewed October 14, 2026, 5:49 AM ET / 09:49 UTC (Revision 10). |
What Problem This Solves
Certificates and connections can outlive an execution or its permissions. Persistent Agents need stable identity and current operation authority so replacement and renewal cannot restore retired access.
Why This Change Was Made
Map the current Agent-specific
servicePrincipalIdto the proposed stableWorkloadIdentity, with optional execution-bound SPIFFE/SPIRE SVID certificates. The certificate authenticates execution; each operation still needs current authority. Managed repository reads first need a root work record; any qualified subordinate helpers share that work and its scope. Approved publication adds exact-operation approval; a broad Work API, independently continuing children and public Stop/Start controls come later.User Impact
The first GitHub profile keeps provider credentials outside Agent workloads and checks online OCC authority for every operation, including reads and credential maintenance. Both permission increases and decreases require a fresh Pod/gVisor sandbox and execution identity. Work and execution may be uncapped while every lease remains finite. Writer exclusion and physical termination require separate evidence; identity or lease expiry alone establishes neither.
This is a documentation proposal extending RFC 0027. It remains draft with a blank implementation issue pending acceptance.
Reading Guide
Evidence