docs: propose persistent Agent runtime lifecycle - #71
freeqaz-openai wants to merge 8 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 8). ClawSweeper reviewWhat this changesAdds an illustrated proposal for safe Agent replacement, persistent stop intent, and staged recovery and completed-result delivery. Merge readiness⛔ Blocked before merge - 3 items remain Keep open: the proposal contains substantive lifecycle contracts beyond accepted RFC 0027. No concrete patch defect was found, but repository policy requires design acceptance before merging a draft RFC. Priority: P3 Review scores
Verification
How this fits togetherThe proposed lifecycle belongs to the OpenClaw Enterprise controller, which turns authorized Agent configuration and control requests into runtime actions. Compute and sandbox drivers report execution and termination observations that govern replacement and recovery. flowchart LR
A[Authorized Agent intent] --> B[Enterprise controller]
B --> C[Isolated candidate]
B --> D[Retire predecessor]
D --> E{Writers confirmed stopped?}
E -->|No| F[Retain data and block replacement]
E -->|Yes| G[Activate candidate and enable routing]
C --> G
Decision needed
Why: The repository explicitly reserves RFC acceptance for human design review; technical coherence and presentation proof cannot establish acceptance. Before merge
Agent review detailsSecurityNone. Review metrics
Merge-risk optionsMaintainer options:
Technical reviewBest possible solution: Adopt a staged lifecycle contract that preserves observed writer exclusion while leaving public Stop defaults and optional recovery profiles to explicit later qualification. Do we have a high-confidence way to reproduce the issue? Not applicable: this PR proposes lifecycle contracts rather than repairing executable behavior; its presentation was inspected directly. Is this the best way to solve the issue? Yes, the staged RFC is a coherent extension of the accepted deployment contract, with runtime qualification explicitly separated from document acceptance. 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 (7 earlier review cycles)
|
What Problem This Solves
A restart or accepted stop request does not establish safe continuation or physical termination. Operators need clear outcomes when retained files, helper processes or remote effects remain unresolved.
Why This Change Was Made
The current ordinary Kubernetes controller selects and requests activation of the successor before requesting predecessor retirement. That path does not yet establish observed predecessor termination and writer exclusion before successor writes. This proposal requires that contract and accepts the resulting availability gap. Managed reads, then approved publication, need this replacement discipline, root work and complete ownership of any supported helpers. Public Stop task / Stop Agent / Start Agent controls, completed-state recovery and broader durable Work and child coordination are later delivery stages.
User Impact
The proposed controls distinguish accepted intent, authority withdrawal, physical termination and credential cleanup. Recovery begins with compatible same-build, same-cluster retained storage; unknown creates or termination block writable replacement. Fresh execution rechecks current authority and preserves effect receipts without replaying uncertain operations. Execution may be uncapped; leases, operation deadlines and any selected drain or delivery remain finite. Public Stop defaults remain open.
This is a documentation proposal extending RFC 0027. It remains draft with a blank implementation issue pending acceptance.
Reading Guide
Evidence