microsoft / microsoft/TypeScript
Don't include full paths in module names in error messages
Nobody has claimed this yet.
- Dominant language
- Go
- Stars
- 111k
- Forks
- 14.3k
- Avg merge
- 2d 4h
- Merged PRs (30d)
- 132
Description
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.
Trynpm install @types/shortidif it exists or add a new declaration (.d.ts) file containingdeclare 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 here, especially since it could maybe be an issue for other tools as well.
Related Issues: Couldn't find any
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Reproduce diagnostic codes 2694 and 7016 using the examples and a tsconfig.json root, then trace where module names and declaration-file paths are formatted. Done means these messages use paths relative to the project root without breaking existing diagnostics or the suggested npm declaration text.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- compilers
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100