RFC 0029: clarify Control Model capability and additive scope - #62
giodl73-repo wants to merge 35 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 17, 2026, 12:06 PM ET / 16:06 UTC (Revision 30). ClawSweeper reviewWhat this changesAdds a framework-neutral conversation-model proposal, artifact and hosted-policy specifications, adoption and ownership plans, and a runnable renderer-selection example. Merge readiness⛔ Blocked before merge - 3 items remain Keep open for RFC acceptance. The previous date finding is fixed, the prototype proof remains applicable, and no blocking patch defect was found. Current main does not contain this proposal. Priority: P3 Review scores
Verification
How this fits togetherThe proposed Control Model sits above OpenClaw’s Gateway client, turning events and history into conversation snapshots and typed commands for independent interfaces. Hosts retain presentation ownership, while the Gateway retains authorization. flowchart TD
G[Gateway events and history] --> T[Gateway client]
T --> M[Proposed Control Model]
M --> C[Control UI]
M --> H[Independent host]
H --> R{Trusted renderer available}
R --> N[Native presentation]
R --> F[Structured or sandboxed fallback]
Decision needed
Why: The proposal supplies a coherent design and bounded demonstration, but only maintainers can accept the long-term public contract and ownership obligations. Before merge
Agent review detailsSecurityNone. Review metrics
Merge-risk optionsMaintainer options:
Technical reviewBest possible solution: Accept a bounded conversation-model and artifact contract only with explicit scope disposition, preserving independent acceptance and release gates for the sibling proposals. Do we have a high-confidence way to reproduce the issue? Not applicable: this PR proposes a new contract rather than reporting broken existing behavior; the prototype’s declared scenarios have contributor-supplied execution output. Is this the best way to solve the issue? Yes as a design-review direction: extracting shared conversation behavior while retaining host presentation and Gateway authorization is coherent; permanent API acceptance remains a maintainer choice. 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:
Rank-up movesOptional improvements that raise the rating; they are not merge blockers.
Rating scale
Overall follows the weaker of proof and patch quality. Workflow
HistoryReview history (29 earlier review cycles; latest 8 shown)
|
|
@clawsweeper re-review Addressed the submission-status findings at
Validation:
|
|
🦞🧹 I asked ClawSweeper to review this item again. Re-review progress:
|
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: e5715557-1677-47e0-9651-88e96e996584
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: e5715557-1677-47e0-9651-88e96e996584
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: e5715557-1677-47e0-9651-88e96e996584
Document the fork-only selected-question and safe Canvas/MCP artifact adoption slice, its ownership boundaries, and the remaining CU6 observation and deletion gate. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: e5715557-1677-47e0-9651-88e96e996584
Add the fork-only OC5 run, tool, question, artifact, and retained-bounds conformance continuation while keeping performance, compatibility, security, and publication as separate remaining gates. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: e5715557-1677-47e0-9651-88e96e996584
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: e5715557-1677-47e0-9651-88e96e996584
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: e5715557-1677-47e0-9651-88e96e996584
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: e5715557-1677-47e0-9651-88e96e996584
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: e5715557-1677-47e0-9651-88e96e996584
Define explicit package, protocol, UI, security, release, and RFC ownership acceptance gates, and refresh the completed OC5 evidence. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: e5715557-1677-47e0-9651-88e96e996584
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: e5715557-1677-47e0-9651-88e96e996584
…iltering guidance Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…e and streaming notes\n\nCo-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
|
Refreshed patch-equivalently onto current main at $head. Direct Node execution of the changed prototype now provides the requested after-change evidence: Command: @clawsweeper re-review |
daab871 to
cd60778
Compare
|
🦞🧹 I asked ClawSweeper to review this item again. |
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 2c44506f-c357-47e5-a867-91775df2093d
|
Updated the RFC at
The required maintainer discussion and acceptance decision remain open. @clawsweeper re-review |
|
🦞🧹 I asked ClawSweeper to review this item again. |
Updates RFC 0029 so the upstream review ask is explicit and bounded:
a2ui/uiDetailsappendix includes a concrete mapping example and keeps actions non-executable.Current review status (2026-09-17)
The latest revision:
operator.questionsout of the browser and notes that the adoption remains behind a default-off flight; andThe remaining decision is maintainer acceptance of the bounded Control Model v1 and UI-artifact contract. Hosted policy, Board Model, and Config Model retain separate acceptance gates.
Sidecar specs
rfcs/0029/control-model-v1-spec.mdrfcs/0029/ui-artifact-v1-spec.mdrfcs/0029/hosted-control-ui-policy-v1-spec.mdImplementation drafts
Renderer paths
A runnable protocol-shape prototype covers exact dialect/catalog selection, composed A2UI messages, explicit MCP App fallback, structured fallback, and fail-closed version handling. It is not claimed as renderer conformance.