fix(cli): register 'reindex' in CLI_ONLY so v0.32.7 sweep is reachable

[vinsew]

Summary

gbrain reindex --markdown (introduced in v0.32.7 #898) exits with "Unknown command: reindex" on every invocation. The dispatcher in src/cli.ts only reaches case 'reindex' (inside handleCliOnly) when CLI_ONLY.has(command) is true at line 90, but the set lists 'reindex-code' and 'reindex-frontmatter' without bare 'reindex'. Adding the missing token wires the case back into the dispatch.

Reproduce on unmodified origin/master

$ gbrain reindex --markdown --dry-run --json
Unknown command: reindex
Run gbrain --help for available commands.
EXIT=1

Same input after this patch (gbrain reindex --markdown --dry-run --json) returns the expected {"pending":N,"would_reindex":N,"dry_run":true,"chunker_version":2}.

Byte-level evidence the case is reachable only via CLI_ONLY

• src/cli.ts:30 — CLI_ONLY set, currently lists 'reindex-code', 'reindex-frontmatter', not 'reindex'.
• src/cli.ts:90 — if (CLI_ONLY.has(command)) { await handleCliOnly(...); return; } is the only path into handleCliOnly.
• src/cli.ts:96-100 — fallthrough is cliOps.get(command) → undefined for 'reindex' (zero name: 'reindex' entry in src/core/operations.ts) → console.error('Unknown command: ${command}'); process.exit(1).
• src/cli.ts:1075 — case 'reindex' lives inside async function handleCliOnly (line 707). Identical pattern to case 'orphans' (1069) and case 'salience' (1081), both of which are registered in CLI_ONLY.

Why CI didn't catch this in #898

test/reindex.test.ts exercises runReindex(engine, args) directly, bypassing the CLI dispatcher. Two new cases in this PR close that gap:

1. Structural — read src/cli.ts, locate the CLI_ONLY = new Set(...) line, require it match /'reindex',/.
2. Runtime — spawn bun run src/cli.ts reindex --markdown --help in a subprocess and assert stderr+stdout does NOT contain Unknown command: reindex.

The structural test is the same shape as test/book-mirror.test.ts's "book-mirror is in CLI_ONLY (does not get 'Unknown command')" guard from PR #564.

Context on the #898 review note

#898 body recorded Codex outside-voice finding #3 as a false positive ("CLI_ONLY is the set that doesn't need an engine; reindex correctly belongs to the engine-backed dispatch"). The premise that CLI_ONLY excludes engine-backed commands does not match the current codebase: handleCliOnly orchestrates engine connect inside per-case branches for many commands (orphans, salience, anomalies, sources, mounts, dream, etc.). All currently-working engine-backed commands inside handleCliOnly are in CLI_ONLY; reindex is the only one that isn't, which is why it's the only one that fails this way.

Verified end-to-end

Production brain (Postgres, 279 pages, all chunker_version=1 before fix) with this patch applied:

$ gbrain reindex --markdown --dry-run --json
{"pending":279,"would_reindex":279,"dry_run":true,"chunker_version":2}

$ gbrain reindex --markdown --json
[reindex.markdown] 279/279 (100%) done
{"pending":279,"reindexed":279,"skipped":0,"failed":0,"chunker_version":2}

279/279 reindexed, 0 failures, ~3:28, ~$0.08 in OpenAI text-embedding-3-large. CJK sync (品牌圣经.md) round-trips cleanly through the new chunker on the post-sweep brain.

Test plan

• bun run typecheck — clean
• bun test test/reindex.test.ts — 8 pass / 0 fail (was 6; 2 new cases for CLI_ONLY registration)
• Subprocess reproduction (before fix): gbrain reindex --markdown --dry-run → "Unknown command: reindex", EXIT=1
• Subprocess reproduction (after fix): gbrain reindex --markdown --dry-run --json → valid JSON, EXIT=0
• End-to-end sweep against production Postgres brain (279 pages)

🤖 Generated with Claude Code

---

^{Need help on this PR? Tag @codesmith with what you need.}

• Let Codesmith autofix CI failures and bot reviews