docs: rebuild Policy Conformance RFC 0003 - #40
Conversation
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 13455f60-afbb-4a92-b883-5a6ef040ee30
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 13455f60-afbb-4a92-b883-5a6ef040ee30
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 13455f60-afbb-4a92-b883-5a6ef040ee30
|
Codex review: needs real behavior proof before merge. Reviewed August 14, 2026, 3:49 PM ET / 19:49 UTC. ClawSweeper reviewWhat this changesThe PR moves Policy Conformance into numbered RFC 0003, adds a normative Policy 1.0 sidecar, and removes the legacy RFC copy. Merge readinessKeep open: current main still has the legacy Policy RFC under Priority: P2 Review scores
Verification
How this fits togetherPolicy Conformance defines how an operator-authored policy is compared with observed OpenClaw configuration and named evidence. Its findings feed the Policy CLI and Doctor health reporting; this PR documents that contract for implementers and operators. flowchart LR
A[Operator policy] --> B[Policy commands]
B --> C[Configuration evidence]
C --> D[Policy findings]
D --> E[Doctor health reporting]
F[RFC 0003 documentation] --> B
Before merge
Findings
Agent review detailsSecurityNone. Review metrics
Merge-risk optionsMaintainer options:
Technical reviewBest possible solution: Land the numbered RFC and sidecar after linking the canonical Policy implementation issue in the accepted RFC metadata, or correct the lifecycle metadata if no implementation issue exists. Do we have a high-confidence way to reproduce the issue? Not applicable: this PR changes RFC Markdown only, and the relevant verification is metadata and document-contract review. Is this the best way to solve the issue? No: the numbered-RFC migration is appropriate, but it must retain the template-required implementation-issue handoff before merge. Full review comments:
Overall correctness: patch is incorrect 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 (55 earlier review cycles; latest 8 shown)
|
What Problem This Solves
The accepted Policy Conformance RFC remained under
rfcs/needs_refactoring/and no longer described the shipped Policy command, evidence, node-command, or repair contracts.Why This Change Was Made
Moves Policy Conformance back to numbered RFC 0003, rebuilds it in the current repository template, and adds a normative sidecar specification. The update preserves Policy as configuration conformance rather than request-time enforcement while documenting
check/compare/watch, scoped strictness, named artifact evidence, Gateway node-command deny posture, fix classification, guarded automatic narrowers, and review-required previews.User Impact
RFC readers and implementers now have a current, reviewable Policy 1.0 contract in the normal RFC layout. The stale
needs_refactoringcopy is removed.Evidence
openclaw/openclawPolicy CLI, rule metadata, fix metadata, automatic repairs, review-required repairs, and Gateway node-command checks.workspaceRepairsopt-in, publicpolicy.fixRecommendationJSON shape, and current two preview implementations.git diff --check