refactor(config): config-surface reduction tranche 3 β product consolidations (review request)#111527
Conversation
|
Final-head follow-up for a37e2a4: the branch was rebased onto f199384, and the newly added Claw lifecycle path was ported fully to canonical agents.entries ownership. Focused proof is green (51 Claw unit tests and 7 CLI lifecycle E2E scenarios), with config baseline 2351/3627/3556, env 530/530, SDK surface, formatting, lint, and fresh autoreview clean. The Rank-up move resolutions and explicit rescue-opt-out skip reason in the earlier maintainer comment still apply to this final head. The latest completed ClawSweeper review remains stale and the later review is only the queue-outage placeholder, so there are no additional final-head Rank-up moves to apply. |
|
ClawSweeper status: review started. I am starting a fresh review of this pull request: refactor(config): config-surface reduction tranche 3 β product consolidations (review request) This is item 1/1 in the current shard. Shard 0/1. This placeholder means the worker is alive and reading the current context. I will edit this same comment with the actual review when the claws are done clicking. Crustacean status: shell secured, claws on keyboard, evidence pebbles being sorted. |
|
Merged via squash.
|
Scope and merge status
This is the single review vehicle for the complete config-surface reduction program. PR #111584 has been folded into this branch. Do not merge this PR without maintainer approval.
Product decisions requested
surfaces.*.silentReplyversusagents.defaults.silentReply: choose delivery-surface ownership or agent-behavior ownership.ui.prefs.*server-synced, or move it client-side once.talk.providersintotts.providerswhile Talk retains session/VAD settings.registerCliBackend, and modelcompat.*flags to provider/catalog hooks.What this consolidates
Product consolidations
agents.listarrays to keyedagents.entrieswhile preserving runtime agent IDs and bindings.Fresh-eyes layout
Environment cleanup folded from #111584
Tier-eval tranche 6a
doctor-tier-eval-tranche, through 2026-09-18.Tier-eval tranche 6b
prepareSynthesis; Doctor strips old prompt blocks and docs point users to provider-specific config or a custom speech provider.summarize()and prompt hooks throughbefore_prompt_build.postCompactionSectionswas kept because it owns a dedicated AGENTS.md extraction/injection pipeline and exceeded the small-hookify seam.systemPromptwas kept because it is the primary documented persona surface across channel plugins; removing it would delete supported group personas, not a redundant echo.Kept because reader evidence contradicted the retirement claim
humanDelay.*: active runtime reader.meta.lastTouchedVersionand the migration marker: required file-scoped downgrade/migration compatibility; onlylastTouchedAtmoved to SQLite.threadBindings: documented runtime-read account/channel overrides.messages.responsePrefix: retained for implicit and custom channels; explicit supported channels receive copied canonical values.security/ask: representable policies move tomode; malformed, partial,ask=always, andfull+on-misspolicies remain for explicit repair.proxy.enabled: retained as the advanced explicit-off escape hatch when an environment URL is present.tts.prefsPath: retained because agents can own distinct preference stores; only the global path moved to machine state.memory.search.experimental.sessionMemoryandstore.vector.enabled: retained because root/agent inheritance can invert behavior if stripped.Rejected with reasons
skipBootstrap,broadcast,plugins.deny, global loop-detection enablement, account names, health-monitor enablement, and strict-JSON annotation fields: each has an active setup, policy, recovery, or usability contract.Follow-ups
registerCliBackend.compat.*flags to provider/model-catalog hooks.postCompactionSectionsonly with a designed replacement for its file extraction/injection behavior.Compatibility
openclaw doctor --fix.Whole-program surface metrics
OPENCLAW_*namesVerification
config:docs:genandconfig:docs:check: clean; final counts 2351/3627/3556.tsgo:allandcheck:test-types: clean.git diff --check: clean.Checklist