microsoft / microsoft/TypeScript

Suggestion diagnostics show in untyped JS code when regular diagnostics don't

Open
#27,852 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Domain: Error Messages Domain: LS: TSServer In Discussion Suggestion
Dominant language
Go
Stars
111k
Forks
14.3k
Avg merge
2d 4h
Merged PRs (30d)
132

Description

TypeScript Version: 3.2.0-dev.20181011

Code

"".toUperKase();
const b = require("./b");

Expected behavior:

If we're not showing regular diagnostics, we should probably not show any suggestions either.

Actual behavior:

"".toUperKase() is fine. require is a real problem that we need to let the user know about right away!

uperkase

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Reproduce the behavior with the TypeScript snippet in the issue, comparing untyped JavaScript diagnostics with suggestion diagnostics. Trace the compiler's handling of regular and suggestion diagnostics; done means the misspelled method is not reported as a suggestion when regular diagnostics are hidden, while the require problem remains visible.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript, typescript
Domain
compilers
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.