fix(cline): scan all VS Code variants for task storage

[ozymandiashh]

Refs #874 — secondary finding only. This does not fix #874 and must not close it.

Scope, stated up front

Issue #874 reports two distinct defects. Its headline defect is that the Cline CLI writes ~/.cline/data/sessions/<sessionId>/ and nothing in codeburn reads it, so those sessions report $0.00. That one is not touched here — the issue's author uses the CLI daily, has real sessions to test against, and asked which shape you'd prefer before implementing. It's theirs.

This PR takes only the issue's secondary finding, which is a self-contained pre-existing bug independent of the CLI layout and shouldn't wait on that design discussion.

The bug

cline.ts calls the singular helper and then always passes a concrete overrideDir:

const baseDirs = configuredDirs ?? [
  getVSCodeGlobalStoragePath(EXTENSION_ID),   // returns paths[0] — stable VS Code only
  getClineDataPath(),
]
const sources = await Promise.all(
  baseDirs.map(dir => discoverClineTasks(EXTENSION_ID, 'cline', 'Cline', dir)),
)

Because overrideDir is never undefined, discoverClineTasks' own 3-variant fallback (vscode-cline-parser.ts:48, which calls the plural getVSCodeGlobalStoragePaths) is unreachable for Cline. Its siblings roo-code.ts:20 and kilo-code.ts:38 pass overrideDir straight through, so they do reach it and scan VS Code, VS Code Insiders and VSCodium.

Cline is the lone outlier. Users on Insiders or VSCodium get no globalStorage coverage at all. #230 asked for VSCodium support and getVSCodeGlobalStoragePaths duly covers it — Cline is just the one provider routing around it.

The fix

Spread the plural helper's roots so cline.ts becomes symmetric with its siblings, and hand discoverClineTasks the whole array in one call (it already accepts string | string[]):

const baseDirs = configuredDirs ?? [
  ...getVSCodeGlobalStoragePaths(EXTENSION_ID),
  getClineDataPath(),
]
return dedupeTaskSources(await discoverClineTasks(EXTENSION_ID, 'cline', 'Cline', baseDirs))

vscode-cline-parser.ts is untouched — docs/providers/cline.md asks that the shared Cline-family parser not be destabilized, and Roo Code and KiloCode must keep passing.

No double-counting. dedupeTaskSources already keys by task id (basename) and keeps the newest ui_messages.json by mtime. Absorbing cross-root duplicates is precisely what it exists for, and there is already a test for it across two roots; the new test extends that to three variants.

Tests

npx vitest run tests/providers/cline.test.ts tests/providers/roo-code.test.ts tests/providers/kilo-code.test.ts tests/providers/vscode-cline-parser.test.ts — all pass, siblings unaffected. New cases cover: all three variants discovered, ~/.cline/data still scanned, and cross-variant dedup by task id holding (no inflation).

npx tsc --noEmit clean.

Docs

README.md's provider table explicitly said Insiders/VSCodium coverage was a Roo Code and KiloCode property, which was accurate before this change and wrong after it — corrected. CHANGELOG.md gets an entry under Fixed, worded to describe only the variant scan, since users on those variants will see their totals jump after upgrading.

Credit

The diagnosis, the three call-site citations and the discovery probe are all from the #874 reporter. I only confirmed and applied them.

[iamtoruk]

Reviewed in depth on the merged tree (current main). Refs #874 secondary finding only; correctly does not close the headline CLI-layout defect.

• The cline provider used the singular getVSCodeGlobalStoragePath (stable VS Code only) while its roo-code/kilo-code siblings scan all variants; this switches to the plural helper so Insiders and VSCodium globalStorage are covered too. Real undercount fix for users on those variants.
• Full data-flow trace confirms discoverClineTasks(array) routes through discoverClineTasksInBaseDirs, which scans every root and dedups by path; behaviorally equivalent to the old map+flat, just adding two roots. vscode-cline-parser.ts is untouched, so the siblings are unaffected.
• No double-count: dedupeTaskSources keys by task id (basename) and keeps the newest ui_messages.json by mtime, collapsing cross-variant duplicates. Mutation-checked: reverting to the stable-only helper fails both the variant-coverage and cross-variant-dedup tests.
• tsc clean; cline + 3 sibling suites 29/29; full suite green apart from the two standing parser.test.ts failures and a known parallel-load flake that passes in isolation.

Good to merge.