RFC 0016: consolidate the Claws contract - #48
Conversation
cb19a7d to
88ff067
Compare
|
This addendum finalizes portable per-agent tool, filesystem, and memory-search fields in experimental Claw schema v1 and extends the normative lifecycle/consent contract. These are the right documents to own that contract, but the proposed shape currently conflicts with canonical OpenClaw configuration and with its own security and disclosure guarantees. LOC: +201/-19 (3 files) Findings
Best-fix verdict: correct owner and timing, but internally inconsistent as written. Align the memory and policy contracts first, then keep detailed Control UI implementation mechanics in a non-normative plan or linked issue after its disclosure contract is resolved. Specs/contracts read: full RFC 0016 and both normative sidecars; RFC repository guidance and #27 discussion; linked implementation PR #112773; canonical OpenClaw memory config, tool-policy, filesystem-policy, and security-audit owners. Remaining uncertainty: this is a docs/spec review. I did not run contributor-controlled implementation code, and I did not inspect every line of the separate draft Control UI implementation PRs. |
|
Added a non-blocking post-merge roadmap at signed head @clawsweeper re-review |
|
🦞🧹 I asked ClawSweeper to review this item again. Re-review progress:
|
|
Portability feedback is incorporated at signed head Implementation is at openclaw/openclaw#112773. Final Codex review found no actionable correctness issues. |
|
Aligned the RFC and sidecar spec with the reviewed implementation at signed head |
|
Codex review: needs real behavior proof before merge. Reviewed August 14, 2026, 9:29 AM ET / 13:29 UTC. ClawSweeper reviewWhat this changesConsolidates RFC 0016 into five Markdown specifications covering the portable package, prompt, OpenClaw profile, native bootstrap, and project authoring lifecycle. Merge readinessKeep open for an explicit maintainer product-direction decision: this is a coherent, documentation-only consolidation of an experimental Claws contract, but it establishes compatibility- and security-relevant normative guidance beyond what the current RFC repository has accepted. Priority: P2 Review scores
Verification
How this fits togetherClaws package one purpose-built agent with its workspace, dependencies, MCP servers, and scheduled work. The RFC documents how authored package inputs are validated and handed to OpenClaw's existing lifecycle owners for planning, consent, and application. flowchart LR
A[Claw project sources] --> B[Validation and deterministic build]
B --> C[Immutable package artifact]
C --> D[OpenClaw inspection and plan]
D --> E[Consent and owner delegation]
E --> F[Agent workspace and provenance]
C --> G[Registry publication and discovery]
Decision needed
Why: The patch is a large normative product contract with compatibility and consent implications; source review can establish coherence, but cannot choose whether this is the intended stable direction for the project. Before merge
Agent review detailsSecurityNone. Review metrics
Root-cause clusterRelationship: Members:
Proposal only: this assessment does not dispatch repair, suppress jobs, mutate sibling items, close, or merge anything. Merge-risk optionsMaintainer options:
Technical reviewBest possible solution: Have the responsible maintainer confirm that the unified draft is the intended experimental contract and that its named OpenClaw behavior is the canonical reference; keep later graduation behind the repository's accepted-RFC and implementation-tracker process. Do we have a high-confidence way to reproduce the issue? Not applicable: this PR changes RFC Markdown rather than reporting a runtime failure; the relevant review path is cross-document and implementation-contract verification. Is this the best way to solve the issue? Unclear until a maintainer confirms product direction. Consolidation is a maintainable way to remove the closed overlapping drafts, but publishing it as the canonical experimental contract requires explicit ownership approval. AGENTS.md: not found in the target repository. Codex review notes: model internal, reasoning high; reviewed against af708f0ddb6f. 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 (25 earlier review cycles; latest 8 shown)
|
|
The implementation evidence requested by the latest ClawSweeper review is now available: openclaw/openclaw#112773 merged as The final RFC head retains host authority and exact consent disclosure as normative constraints; it does not make the browser projection the authority or permit a Claw profile to bypass host enforcement. @clawsweeper re-review |
|
🦞🧹 I asked ClawSweeper to review this item again. Re-review progress:
|
|
@clawsweeper re-review |
|
🦞🧹 I asked ClawSweeper to review this item again. Re-review progress:
|
|
@clawsweeper re-review |
|
🦞🧹 I asked ClawSweeper to review this item again. Re-review progress:
|
|
Exact head @clawsweeper re-review |
|
🦞🧹 I asked ClawSweeper to review this item again. Re-review progress:
|
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 34377cd6-beac-4e49-8ab0-22cfaf8b54a5
19fe123 to
cbfbf73
Compare
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 34377cd6-beac-4e49-8ab0-22cfaf8b54a5
|
Final implementation evidence is now linked in the RFC body:
This proves both sides of the draft contract on their published heads. @clawsweeper re-review |
|
🦞🧹 I asked ClawSweeper to review this item again. |
Supersedes #52 and #56 by folding their still-relevant contracts into RFC 0016.
What Problem This Solves
The Claws contract was split across overlapping RFC drafts while package, profile, bootstrap, extension, heartbeat, memory, metadata, and project behavior continued to converge.
Why This Change Was Made
This revision makes draft RFC 0016 and its normative sidecars the single schema-v1 contract. It incorporates the application-project and OpenClaw-profile specifications while preserving host ownership of runtime configuration, secrets, shared plugins, and lifecycle state.
User Impact
Implementers and reviewers get one coherent draft specification instead of reconciling conflicting requirements across RFC #48, #52, and #56.
Evidence
agents.entriesgit diff --checkFinal implementation proof
acf99d988d90b8d106a4c1d22f66b139bfb02deff69dcea5cf28697720593113092d4e323fa583e2: 87 exact-head checks passed, plus the full localsrc/clawssuite passed 452 tests with 1 skip.9955d795c7e997d0fbedf7df23b29651757ad0de: full CI passed.f8c0e1b8325b1fc36e039cf357a2c4602f76d5aa.