📝 Check off the LSP shutdown bug

Record the cause (Session.updateWatches raced against exit) and the
stdin-EOF fix now that npm run verify is green with no context-canceled
output.
This commit is contained in:
tmu committed 2026-09-18 07:31:10 +00:00
1 parent 8b73143740
commit b278a153d3
1 file changed
+3 -1
+3 -1
View File
@@ -48,7 +48,9 @@ Matcher:
→ `{ a, b, _ }` for `T = "a" | "b"` is accepted today; the redundant `_` should be rejected → `{ a, b, _ }` for `T = "a" | "b"` is accepted today; the redundant `_` should be rejected
Bugs: Bugs:
✔ TS 7 LSP server logs `context canceled` on stderr at shutdown @done
→ `handleExit` returns `io.EOF`, cancelling the background context while `Session.updateWatches` is still in flight; the bare error is flushed to stderr and the server exits 1
→ close stdin after `shutdown` instead of sending `exit`; the server exits cleanly (code 0, no output), kill kept as a fallback
Enhancements: Enhancements:
☐ Allow boolean literals in primitive union patterns (e.g. `true: () => "yes"`) @medium ☐ Allow boolean literals in primitive union patterns (e.g. `true: () => "yes"`) @medium
☐ Are there other primitive types that should be supported in union patterns? (e.g. `bigint`, `symbol`) @medium ☐ Are there other primitive types that should be supported in union patterns? (e.g. `bigint`, `symbol`) @medium