RFC 0019: Managed Configuration - #34
Conversation
c30594e to
9d72be9
Compare
|
Closing because this RFC is still being developed on the fork and was opened upstream prematurely. |
|
Codex review: needs real behavior proof before merge. Reviewed August 1, 2026, 4:35 PM ET / 20:35 UTC. ClawSweeper reviewWhat this changesThis branch adds an RFC and normative sidecar specifying opt-in, startup-only Gateway configuration layers that compose ordinary config files while rejecting conflicting or policy-weakening values. Merge readinessThis is a well-scoped draft RFC, not an implementation already present on this repository’s current main branch. It should remain open for an explicit maintainer product and security-boundary decision on whether ordered, authoritative Gateway configuration layers belong in core; the linked implementation PR remains open and is not evidence of a shipped solution. Priority: P2 Review scores
Verification
How this fits togetherThe RFC repository records OpenClaw design decisions before implementation. This proposal would let a host pass ordered configuration sources into Gateway startup, where configuration bootstrap would compose and validate them before Gateway publishes one runtime snapshot. flowchart LR
A[Host or operator] --> B[Ordered config files]
B --> C[Gateway startup]
C --> D[Prepare each source]
D --> E[Authority and policy checks]
E --> F[Schema and plugin validation]
F --> G[Immutable Gateway snapshot]
G --> H[Runtime consumers]
Decision needed
Why: This choice adds a user-visible CLI/configuration surface and defines a long-lived compatibility and security boundary; source review cannot determine whether maintainers want that contract in core. Before merge
Agent review detailsSecurityNone. Review metrics
Merge-risk optionsMaintainer options:
Technical reviewBest possible solution: Keep the RFC draft open, record a maintainer decision on the narrow V1 core contract, and review the RFC alongside openclaw/openclaw#107026 before accepting the design or scheduling implementation. Do we have a high-confidence way to reproduce the issue? Not applicable: this PR is an RFC-only design proposal, not a report of broken current behavior. The implementation claims should be evaluated on the separate OpenClaw implementation PR, not by executing this Markdown branch. Is this the best way to solve the issue? Unclear. The proposal is internally narrow and explicitly preserves ordinary no-flag behavior, but whether OpenClaw should own this new configuration authority and lifecycle contract is a maintainer product decision. 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 (34 earlier review cycles; latest 8 shown)
|
48a424c to
43e3256
Compare
|
Closing for now while we finish the implementation plan, role review, and supporting prototype proof on the fork. We will revisit upstream submission when the RFC and delivery slices are ready for maintainer review. |
43e3256 to
a189eb9
Compare
823cf9f to
eaeeb56
Compare
eaeeb56 to
481116b
Compare
|
Rebased RFC 0019 onto current The rebase does not expand or change the V1 contract. The paired implementation has also been rebased onto current OpenClaw main. Its historical 63-test and live-Gateway evidence remains documented, while exact-head post-rebase validation is explicitly pending rather than overstated. ClawSweeper reports no automated repair item here. The remaining question is the expected maintainer decision on the core ownership/security boundary and RFC-to-implementation acceptance. |
|
@clawsweeper re-review |
|
🦞🧹 I asked ClawSweeper to review this item again. Re-review progress:
|
Decision requested
Should OpenClaw provide an opt-in, startup-only way to compose independently
authored configuration sources while preserving field authority?
Why
Hosts often combine deployment defaults, tenant connectivity, and
operator-local settings. Lobster currently demonstrates the cost of flattening
those sources outside OpenClaw: source boundaries disappear, launchers must
approximate OpenClaw config behavior, and a later overlay can silently replace
an earlier security boundary.
RFC 0019 keeps the values in the ordinary OpenClaw schema and adds one generic
composition seam around them. It does not create a hosted-config schema,
control plane, policy language, or set of built-in host/tenant/operator roles.
Ownership model
validation, runtime behavior, and the two schema-specific bounded rules.
order, rollout, and process replacement.
authority admission, and fail-closed startup.
The first declaring layer has authority over a path only within the evaluated
candidate and accepted process activation. Layer ids are diagnostic labels,
not principals or durable owner identities. A full process restart recomputes
authority from current sources; an in-process Gateway restart reuses the
accepted snapshot.
This gives OCC, Fleet, Lobster, or another host a clean caller boundary: they
may compile desired state and select sources without taking ownership of
OpenClaw config semantics.
Normative contract
The Managed Configuration V1 sidecar
is normative. The RFC supplies motivation, examples, evidence, and migration
guidance.
V1 is deliberately narrow:
--config-layer <id=path>startup inputs;tools.allownarrowing andtools.denybroadening;No flag preserves ordinary OpenClaw config behavior.
Example
A later exact-path replacement rejects with
ControlledByEarlierLayer. Alater bounded tool-policy declaration is accepted only when OpenClaw can prove
that it tightens the earlier value.
Implementation and evidence
and QA checks on the current implementation review head
rejection, and no canonical config creation or modification
Delivery
onto current main.
rollout and rollback evidence.
The RFC remains draft while maintainers review the ownership boundary and
bounded V1 contract.