Don't include full paths in module names in error messages

TypeScript Version: 4.0.5 (also tested with typescript@next)

Search Terms: namespace has no exported member error

Description

Currently certain error messages shows full file paths. For example:

Namespace ‘“y:/projects/opf/portal/OPF.Portal.Web/Content/Scripts/DinSide/App/Store/index”’ has no exported member ‘All’. ts(2694)

Could not find a declaration file for module ‘shortid’. ‘y:/projects/opf/portal/OPF.Portal.Web/node_modules/shortid/index.js’ implicitly has an ‘any’ type. Try npm install @types/shortid if it exists or add a new declaration (.d.ts) file containing declare module 'shortid'; ts(7016)

Our project was started without strict: true (and few @typescript-eslint rules), so the code base includes several thousand errors. To be able to gradually fix them, we have started to use betterer, which includes committing a snapshot of all the current errors. Snapshot is then used to report report when we fix errors, and fail our build if we add new errors. But because of certain Typescript errors using full file paths, we’re getting unnecessary diffs in this snapshot.

Would it be possible to use file paths relative to the project/tsconfig.json root in the error messages instead?

So, with tsconfig.json in y:/projects/opf/portal/OPF.Portal.Web/, the error message would instead be:

Namespace ‘“Content/Scripts/DinSide/App/Store/index”’ has no exported member ‘All’. ts(2694)

An alternative solution could be to change betterer instead, make it auto-replace file paths with relative paths, or something like that. But figured I could at least ask her, especially since it could maybe be an issue for other tools as well.

Related Issues: Couldn’t find any

[RyanCavanaugh]

It’s any time we reference the name of a module, which are internally represented by their full disk path. During error message construction we should convert them back to tsconfig-relative paths

[RyanCavanaugh]

👍