RFC 0029: clarify Control Model capability and additive scope - #62
giodl73-repo wants to merge 34 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 9, 2026, 1:29 AM ET / 05:29 UTC (Revision 29). ClawSweeper reviewWhat this changesAdds a framework-neutral conversation-model proposal, companion contracts and adoption plans, and a runnable example of renderer selection and fallback. Merge readiness⛔ Blocked before merge - 3 items remain Keep open for RFC acceptance. The contributor’s new terminal evidence resolves the prototype proof gap; the required maintainer discussion and scope decision remain outstanding. No same-repository merged replacement was established. Priority: P3 Review scores
Verification
How this fits togetherThis repository records OpenClaw design proposals. The proposed Control Model would turn Gateway events and history into shared conversation snapshots and commands for Control UI and independently owned product interfaces. flowchart LR
A[Gateway events and history] --> B[Existing Gateway client]
B --> C[Proposed Control Model]
C --> D[Conversation snapshots]
D --> E[OpenClaw Control UI]
D --> F[Independent product interface]
F --> G[Typed commands]
G --> B
Decision needed
Why: The repository requires a maintainer discussion and acceptance before merge, and the proposal cannot establish that acceptance through implementation evidence or owner nominations. Before merge
Findings
Agent review detailsSecurityNone. Review metrics
Technical reviewBest possible solution: An explicitly accepted, bounded conversation-model contract with independently gated sibling proposals, implementation reviews, and publication ownership. Do we have a high-confidence way to reproduce the issue? Not applicable to the proposed capability; the contributor has supplied a direct run demonstrating the standalone example’s selection and fallback behavior. Is this the best way to solve the issue? Unclear pending scope acceptance: the proposal separates shared conversation behavior from product presentation and preserves independent rollout gates, but maintainers must choose the supported contract. Full review comments:
Overall correctness: patch is correct 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:
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 (28 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
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. |
Updates RFC 0029 so the upstream review ask is explicit and bounded:
Sidecar specs now linked from the RFC:
rfcs/0029/control-model-v1-spec.mdrfcs/0029/ui-artifact-v1-spec.mdrfcs/0029/hosted-control-ui-policy-v1-spec.mdImplementation drafts linked from the RFC:
Hosted policy siblings linked from the RFC:
Scope guard: the hosted-policy sidecar is related and belongs in the one-shot RFC family, but it is not a Control Model v1 dependency. Each sidecar has an independent conformance and acceptance gate.
Validation:
git diff --checkinC:\src\openclaw-rfcs-control-model.Renderer-path clarification (2026-08-28)
The latest revision makes the presentation paths explicit:
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.