DanielXMoore / Civet / 24455043415

96%

push

github

auto import (#1956)

* auto import

* ci is much slower for timeouts

* increase completion timeout

* increase diagnostic timeout

* scoping tsconfig to decrease timeouts

* e2e test warmup

* add instrumented warmup to e2e tests for CI debugging

- warmup polls for auto-import readiness before tests run
- timing logs to stderr so we can see exactly what's slow on CI
- avoid opening valid.civet in warmup to not steal diagnostics event
- keep tsconfig types:[] and include to reduce TS program size

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* fix e2e timeouts: proper warmup, increased CI timeouts

Root cause: each `executeCompletionItemProvider` with 1100 resolves takes
~7.5s on Windows CI, meaning only 1 attempt fits in an 8s poll timeout.

- Warmup opens consumer.civet (not valid.civet) to avoid stealing
  diagnostics event, polls cheaply for basic completions
- Poll timeouts based on CI data: 15s Linux, 30s Windows
- Mocha timeouts: 20s Linux, 45s Windows
- 1s poll interval for resolved completion calls
- Keep tsconfig types:[] and include to reduce TS program size
- Timing logs to stderr for CI debugging

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* add auto-import debug logging for Windows CI investigation

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* increase server test timeout for Windows CI

TSService startup can exceed mocha's 2000ms default on Windows CI.
Also keep e2e auto-import debug logging for this run.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* fix auto-import on Windows: canonicalize TS code-action paths

Root cause: TS returns code-action file paths with forward slashes
(e.g. D:/a/Civet/...) but the LSP server uses OS-native backslash
paths on Windows. The === comparison in onCompletionResolve always
failed, so additionalTextEdits were never populated on Windows.

Fix: extract getCanonicalFileName to shared util and use it ... (continued)

4258 of 4641 branches covered (91.75%)

Branch coverage included in aggregate %.

20421 of 21234 relevant lines covered (96.17%)

47414.62 hits per line