docs: propose workload identity and bounded enforcement - #69
docs: propose workload identity and bounded enforcement#69freeqaz-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. Reviewed September 11, 2026, 3:10 PM ET / 19:10 UTC (Revision 6). ClawSweeper reviewWhat this changesAdds an illustrated RFC and enforcement specification proposing execution-specific authentication, finite authority leases, and staged Enterprise runtime enforcement. Merge readiness⛔ Blocked before merge - 3 items remain Keep open: this is a coherent extension of the accepted Enterprise RFC, absent from main, with no concrete patch defect found. The related drafts cover complementary work rather than superseding it. Priority: P3 Review scores
Verification
How this fits togetherThe proposed Enterprise controller assigns Agent executions and authorizes their operations. Execution certificates establish identity, while current permission checks and bounded leases govern access to protected services. flowchart LR
A[Agent execution] --> B[Execution identity verification]
C[Controller assignment] --> B
D[Current permissions and work scope] --> E[Operation authorization]
B --> E
F[Finite authority lease] --> E
E --> G[Protected service operation]
Decision needed
Why: The repository explicitly reserves RFC acceptance for its discussion process; technical consistency and rendered documentation cannot establish that product decision. Before merge
Agent review detailsSecurityNone. Review metrics
Merge-risk optionsMaintainer options:
Technical reviewBest possible solution: Accept a staged identity-and-authority contract that preserves the token baseline and keeps every optional runtime profile unavailable until separately qualified. Do we have a high-confidence way to reproduce the issue? Not applicable: this PR proposes a future authentication and authority model rather than reporting a reproducible defect in this repository. Is this the best way to solve the issue? Yes, as a design proposal: it preserves the accepted identity boundary, stages delivery, and explicitly distinguishes authentication, authorization withdrawal and physical termination. AGENTS.md: not found in the target repository. Codex review notes: model internal, reasoning medium; reviewed against 967d9aac7472. LabelsLabel changes:
Label justifications:
EvidenceWhat I checked:
Likely related people:
Rating scale
Overall follows the weaker of proof and patch quality. Workflow
HistoryReview history (5 earlier review cycles)
|
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