Docs: no-misused-promises example for "if promise exists" about typescript-eslint HOT 3 CLOSED

That is, when if (promise) was in fact intended.

This makes sense only when promise is nullable. And that case is currently handled by the rule. In the example below, if (maybePromise) is not reported:

declare const maybePromise: Promise<void> | undefined
if (maybePromise) { }
//  ^?

declare const promise: Promise<void>
if (promise) { }
//  ^?

playground

Could you provide an example where the if (promise) required for non-nullable promise?

from typescript-eslint.

On looser typescript settings, the nullable bit isn't required for it to be nullable. I am ok with adding the nullable type being an example of how to fix the issue, even though that makes the case of Promise<boolen> | undefined unclear on whether the if is meant to be for the value or the promise. The rule works as intended; this suggestion is just for adding explicit examples/guidance to users for how to handle that use case without using eslint-disable.

from typescript-eslint.

On looser typescript settings

In general we don't support "looser" settings and assume you have at least strictNullChecks turned on. Because if you don't have it turned on then your code is provably unsound. Because turning it on is the industry baseline.

Our rules are powered by the types - so if your types are unsound then the rules are in an undefined state. It's usually impossible for us to report "correctly" because things like null can be removed from unions.
How can we possibly understand your code if your types are incorrect?

from typescript-eslint.