feat(config-schema): mirror experimental swe_pruner keys from CLI config - #4416
feat(config-schema): mirror experimental swe_pruner keys from CLI config#4416Drilmo wants to merge 1 commit into
Conversation
Mirrors the new experimental.swe_pruner and experimental.swe_pruner_model keys added in Kilo-Org/kilocode#11980, following the sync convention documented in extras.ts, with the matching schema-test assertions.
Code Review SummaryStatus: No Issues Found | Recommendation: Merge Executive SummaryMechanical schema mirror adding two experimental config keys ( Files Reviewed (2 files)
Reviewed by claude-sonnet-5-20260630 · Input: 14 · Output: 4.2K · Cached: 273K Review guidance: REVIEW.md from base branch |
|
@Drilmo right now the swe_pruner doesn't perform very well cost wise on our internal evals. I am actively researching why, but until then we should put any further changes on hold. |
|
Let's merge this anyway already, since it's already in the client. |
|
This Pull Request has been inactive for 30 days and will be closed in 7 days if no further activity occurs. Please update or close this PR if it is no longer relevant. |
Summary
Mirrors two new experimental CLI config keys in the config JSON Schema overlay, per the sync convention documented at the top of
apps/web/src/app/config.json/extras.ts:experimental.swe_pruner(boolean) — enables SWE-Pruner, task-aware pruning of large read/grep tool outputsexperimental.swe_pruner_model(string) — optionalprovider/modeloverride for the skimming modelBoth keys are added in Kilo-Org/kilocode#11980 (marked
kilocode_changeinpackages/opencode/src/config/config.ts). Without this mirror, users with$schema: https://app.kilo.ai/config.jsonget "unknown property" warnings when setting these keys. Descriptions are copied verbatim from the zod schema, and the matching assertions were added toapps/web/src/tests/cli-config-schema.test.ts(step 4 of the documented procedure).Verification
codebase_search/native_notebook_toolsentries. The local environment lacks Node >=24 required by this repo's engines field, sopnpm test/pnpm formatcould not be run locally — relying on CI for the jest suite and format check. The edits copy the existing file style (quotes, indentation, trailing commas) so oxfmt should be a no-op.Visual Changes
N/A
Reviewer Notes
Companion to Kilo-Org/kilocode#11980 — best merged once that PR lands (the keys are harmless in the schema overlay before then; the CLI simply ignores unknown experimental keys). Note that
sandbox/sandbox_restrict_network/speech_to_text_modelfrom the samekilocode_changeexperimental block are currently absent from this overlay too; left untouched here to keep this PR scoped, but may be worth a follow-up sweep.