Repository navigation
docs: propose service-owned work authority - #70
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 service-owned Work records with immutable scope, shared helper cancellation, finite authority leases, and staged human-approved publication. Example: Alice asks a repository review service to inspect an administrator-approved repository.
Review scores
ProductKind: Feature · Worth it: Needs a maintainer decision Merge readiness⛔ Blocked before merge - 2 items remain Keep this PR open for RFC acceptance: it contributes a distinct Work authority proposal absent from main, with no actionable document defect found. Priority: P3 Decision needed
Before merge
FindingsNone. Agent review detailsHow this fits togetherThe proposed OCC Work boundary receives authenticated invocations and current authorization, then supplies bounded execution authority and durable operation identity to repository mediation and runtime lifecycle owners. flowchart LR
Requester[Authenticated requester] --> OCC[OCC admission]
IAM[Current authorization] --> OCC
OCC --> Work[Root Work record]
Work --> Helpers[Execution and helpers]
Helpers --> Mediation[Trusted repository mediation]
Mediation --> GitHub[GitHub operations]
Mediation --> Receipts[Durable operation receipts]
Technical reviewBest possible solution: Resolve the staged Work contract through RFC discussion, then record acceptance and link an implementation issue if approved. Do we have a high-confidence way to reproduce the issue? This PR proposes future behavior rather than reporting a reproducible runtime failure. Is this the best way to solve the issue? The root-only initial profile coherently separates admission, execution, credential custody, and effects, but its platform policies require RFC acceptance. 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, 4:01 AM ET / 08:01 UTC (Revision 10). |
What Problem This Solves
An Agent’s work can outlast a turn, coordinator process or provider token. Shared Agents need lasting scope and cancellation ownership while keeping requester, service-owner and workload permissions separate.
Why This Change Was Made
Define an initial root work record with requester attribution, immutable scope and any configured horizon, cancellation, execution assignment and effect receipts. Subordinate helpers share its scope, resource limits and stop ownership. This supports managed reads and subsequent approved publication without gating them on a complete public Work API. Separately admitted durable children, broader Work management and public lifecycle controls follow later, with preserved lineage and ancestor constraints.
User Impact
Execution defaults to uncapped under explicit admission policy; authority leases stay finite and renewal cannot extend configured limits, broaden scope or reopen terminal work. Every initial GitHub operation requires current online OCC authority. Durable receipts preserve original work and operation identity across replacement; unknown submission requires reconciliation without blind replay. MVP publication permits any configured, authorized human approver, including the requester, and excludes Agent approval.
This is a documentation proposal extending RFC 0027. It remains draft with a blank implementation issue pending acceptance.
Reading Guide
Evidence