RFC 0017: Command Catalog - #32
Conversation
|
Codex review: needs real behavior proof before merge. Reviewed September 9, 2026, 1:25 AM ET / 05:25 UTC (Revision 71). ClawSweeper reviewWhat this changesAdds a draft RFC for a read-only command inventory that combines existing command metadata while preserving source ownership, collection scope, and unknown effects. Merge readiness⛔ Blocked before merge - 3 items remain The proposal remains distinct and absent from main; its four upstream implementation PRs are still open. The earlier discussion requirement remains unresolved, and the repository requires explicit acceptance before merging a draft RFC. Priority: P3 Review scores
Verification
How this fits togetherThis repository stores OpenClaw design proposals rather than executable product code. The proposed catalog would combine CLI, plugin, and selected node metadata into operator-facing inventory and inspection output. flowchart LR
A[CLI metadata] --> D[Proposed command catalog]
B[Opt-in plugin metadata] --> D
C[Selected node observation] --> D
D --> E[Preserve provenance and unknowns]
E --> F[List and inspect output]
E --> G[Future documentation consumers]
Decision needed
Why: The new CLI and metadata contract require product acceptance; neither implementation drafts nor the localization reference establish that acceptance. Before merge
Agent review detailsSecurityNone. Review metrics
Technical reviewBest possible solution: Agree on a bounded operator inventory contract that preserves existing registry authority, then record acceptance and track implementation through the repository's RFC lifecycle. Do we have a high-confidence way to reproduce the issue? Not applicable: the patch proposes a new inventory capability and contains no executable bug fix. Is this the best way to solve the issue? Unclear pending product acceptance: joining source-owned metadata is coherent, and the proposal explicitly distinguishes its operator view from existing Gateway discovery. 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 (70 earlier review cycles; latest 8 shown)
|
|
@clawsweeper re-review |
|
🦞🧹 I asked ClawSweeper to review this item again. |
f3573b5 to
842da7f
Compare
|
Refreshed onto current main at a6cdc07. The RFC remains a one-file documentation-only proposal and git diff --check passes. The required maintainer-discussion link remains the explicit acceptance-process follow-up; this refresh does not claim acceptance. @clawsweeper re-review |
Summary
Proposes a read-only Command Catalog through
openclaw commands listandopenclaw commands inspect. OpenClaw command facts currently live across static CLI descriptors, command routes, routed operations, the runtime Commander tree, opt-in plugin CLI descriptors, and paired-node observations. The RFC joins those existing owners into one machine-readable view without adding a dispatcher, policy engine, runtime hook, or alternate execution path.This is intended to answer operator and compliance inventory questions consistently while also giving later documentation, diagnostics, scoped prompt, drift, and search consumers one normalized input.
Key boundaries
effectProfileandcommandExposure; plugin descriptors may also use existing-stylehiddenmetadata.commands.listas the separate agent/provider-scoped command view.Implementation drafts
Fork PRs #30-#32 are explicitly labeled internal consumer prototypes, not required public surfaces. Historical broader
catalogdrafts remain alternatives, not normative specification.Review requested
openclaw commandsthe right operator/developer surface alongside the narrower Gatewaycommands.listRPC?The required
maintainer-discussionthread still needs to be linked before acceptance.