`isolatedDeclarations` not giving related errors under `noEmit` about typescript HOT 6 CLOSED

Specifying isolatedDeclarations: true only makes sense if you want the errors (that's all the flag does, after all), so we should be evaluating them here. It's the same as specifying verbatimModuleSyntax and noEmit - the latter doesn't affect whether or not the former should do its checks

from typescript.

Debugging, handleNoEmitOptions skips getting declaration diagnostics when noEmit is set. The editor is different and always grabs declaration errors.

Disabling the noEmit skip there fixes the issue, but then has weird knock-on effects. I haven't been able to solve. My test has been pushed to https://github.com/jakebailey/TypeScript/tree/fix-59024 (it doesn't work in a regular compiler test since we always emit).

@sheetalkamat Do you have any ideas as to how to fix this?

from typescript.

Will take a look.

from typescript.

This is not a regression. We never reported declaration diagnostics under --noEmit through tsc
If we are changing this it will be need to be not a patch.

from typescript.

Not a regression, no, but I feel like we need to do something here because we're offering up these diagnostics in the editor regardless of noEmit, but then you can get a passing result at the CLI or similar which seems really, really weird?

from typescript.

These diagnostics are offered when "emitting". Eg if you ran this code with "--noCheck" you will get that error. So its just where the error gets reported. It was modeled so that we are not doing extra pass .. so dts diagnostics are always reported when emitting dts file. (only excpetion is noEmitOnError)

Reference: https://github.com/microsoft/TypeScript/blob/main/tests/baselines/reference/tsc/noCheck/multiFile/dts-errors.js#L35

from typescript.