Skip to content

docs: preset facts outside docdrift's scope are stale — wrong default preset in 5 places, stale README pipeline lists #145

Description

@OsherElhadad

The docdrift guard added in #142 covers two files (docs/how-to/choose-a-preset.md,
docs/reference/presets.md). Preset facts stated outside those two files are stale, and
the guard's shape cannot reach them.

1. The default preset is house; five sites say codesmart

cmd/context-guru-proxy/main.go:46:

preset = flag.String("preset", envOr("PRESET", "house"), "... house = the service default ...")

Sites that say otherwise:

Location Says
README.md:111 # --preset codesmart; listens on :4000
README.md:132 Presets: **`codesmart`** (the default — the SWE-bench-winning cache-aware config
README.md:143 flag table: ``
docs/reference/config.md:79 ``
docs/get-started/quickstart-proxy.md:20 # default preset: codesmart

house is [format, dedup, toon, cmdfilter, searchfold, textclean, extract, cachesplit, toolfilter]
— different economics from codesmart, and it still runs toon, which is retired from every
preset. Anyone who runs the binary bare while reading any of these five is measuring a
different config than the one the published SWE-bench numbers describe.

quickstart-proxy.md:20 is the worst of the five: it is step 2 of the first page a new user
runs.

2. README.md:132-136's pipeline lists are both stale

codesmart  doc  [format, toon, dedup, failed_run, cmdfilter, extract_llm, extract, cachesplit]
           code [format, textclean, searchfold, dedup, failed_run, cmdfilter, extract_llm, extract, linecap, cachesplit]
codesafe   doc  [format, dedup, failed_run, cmdfilter, extract, collapse, cachesplit]
           code [format, textclean, searchfold, dedup, failed_run, cmdfilter, extract, collapse, linecap, cachesplit]

The trailing name list also omits agentdiet, house, and housellm.

3. docs/components.md:34-56 claims "verbatim" and is not

12 rows, all currently correct, but it omits house/housellm while claiming to reproduce
Presets (`config/config.go`), **verbatim**.

Why the guard cannot cover these

docRow is anchored ^\|\s* — it matches markdown table rows only. README's and
components.md's lists are prose, and config.md:79's first cell is
`--preset` / `PRESET`, which the pattern does not match either. Widening the guard to
these files matches zero rows.

Suggested fix

Delete the duplicates rather than guard them. config/form.go and dash/ui/index.html:67
already do this correctly — they derive from config.PresetNames() and from runtime data, so
they cannot rot. Replace README's and components.md's bracket lists with a pointer to
docs/reference/presets.md, and have the default-preset claims say "see --help" or be
generated from the flag.

Found during a review sweep of #129/#130/#136/#137/#138/#141/#142 against main e88f5d8.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions