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. Reviewed September 14, 2026, 5:59 PM ET / 21:59 UTC (Revision 7). ClawSweeper reviewWhat this changesAdds an illustrated proposal and enforcement specification for execution-specific workload authentication, finite authority leases, and staged Enterprise runtime enforcement. Merge readiness⛔ Blocked before merge - 3 items remain This is a coherent RFC extension absent from current main, with no concrete patch defect found. It remains pending design acceptance under the repository’s RFC lifecycle. Priority: P3 Review scores
Verification
How this fits togetherThe proposed Enterprise controller admits Agent executions and controls their access to protected operations. Execution credentials identify the caller, while current authorization and bounded leases constrain what accepting services may permit. flowchart LR
A[Agent and admitted work] --> B[Enterprise controller]
B --> C[Execution registration]
C --> D[Execution credential]
B --> E[Current authority and finite lease]
D --> F[Accepting service checks]
E --> F
F --> G[Permit or deny operation]
Decision needed
Why: The repository requires explicit RFC acceptance, and selecting these Enterprise contracts requires architectural intent beyond documentation correctness. Before merge
Agent review detailsSecurityNone. Review metrics
Merge-risk optionsMaintainer options:
Technical reviewBest possible solution: Accept a staged design that preserves baseline authentication and keeps optional profiles unavailable until their origin, revocation, and migration guarantees are demonstrated. Do we have a high-confidence way to reproduce the issue? Not applicable: this PR proposes new contracts rather than repairing a reproduced runtime defect; its rendered documentation was inspected. Is this the best way to solve the issue? Yes, as a staged RFC: it separates authentication from authorization and explicitly gates unqualified mechanisms, while final architectural acceptance remains a maintainer decision. AGENTS.md: not found in the target repository. Codex review notes: model internal, reasoning medium; reviewed against 967d9aac7472. LabelsLabel justifications:
EvidenceWhat I checked:
Likely related people:
Rating scale
Overall follows the weaker of proof and patch quality. Workflow
HistoryReview history (6 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