Skip to content

Add regression coverage for first-open pull diagnostics - #63933

Draft
Ryan Cavanaugh (RyanCavanaugh) with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-diagnostics-omission
Draft

Add regression coverage for first-open pull diagnostics#63933
Ryan Cavanaugh (RyanCavanaugh) with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-diagnostics-omission

Conversation

Copilot AI commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

The first pull-diagnostics request for the first opened file could omit genuine semantic errors after source-file replacement.

  • Regression coverage
    • Starts a fresh LSP session with only the erroneous file open.
    • Pulls diagnostics immediately.
    • Verifies that TS2322 is reported through the existing path-keyed diagnostic lookup.

@typescript-automation typescript-automation Bot added the For Milestone Bug PRs that fix a bug with a specific milestone label Aug 20, 2026
Co-authored-by: RyanCavanaugh <6685088+RyanCavanaugh@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix diagnostic omission on first opened file in session Add regression coverage for first-open pull diagnostics Aug 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

For Milestone Bug PRs that fix a bug with a specific milestone

Projects

None yet

Development

Successfully merging this pull request may close these issues.

textDocument/diagnostic on the first-opened file in a session can silently omit real errors

2 participants