-
Notifications
You must be signed in to change notification settings - Fork 13.3k
Open
Labels
Needs ProposalThis issue needs a plan that clarifies the finer details of how it could be implemented.This issue needs a plan that clarifies the finer details of how it could be implemented.SuggestionAn idea for TypeScriptAn idea for TypeScript
Description
🔎 Search Terms
nodenext, ts2351, construct
🕗 Version & Regression Information
- This is the behavior in every version I tried, and I reviewed the FAQ for entries about "A default import of a commonjs module with a default in a esm file doesn't seem to be the default export of that module when module is node16 or nodenext".
⏯ Playground Link
N/A
💻 Code
git clone https://github.com/monsanto/typescript-nodenext-bug
cd typescript-nodenext-bug
npm install
npm run tsc
🙁 Actual behavior
foo/index.ts:6:5 - error TS2351: This expression is not constructable.
Type 'typeof import("<dir>/typescript-nodenext-bug/.cache/foolib/class")' has no construct signatures.
6 new CBad();
~~~~
Found 1 error.
🙂 Expected behavior
It succeeds.
Additional information about the issue
- This issue requires a composite project,
NodeNext, a package boundary, a default export, and a parent directory with a file package.{json, json5, yaml} withouttype:module.- Please note that import default inside
foo's package boundary works. - One can delete the provided package.json after installing and run
node_modules/typescript/bin/tsc --build tsconfig-base.jsonto test withoutpackage.jsonpresent. - One can see other package extensions trigger the bug by
mv package.json package.yamlandpnpm run tsc.
- Please note that import default inside
- This is not a CommonJS project. It doesn't matter that there is a parent
package.jsonwithout it. Node doesn't even readpackage.yaml.- This is from a real-world monorepo where there is a root pnpm
package.json5for cross-package scripts.
- This is from a real-world monorepo where there is a root pnpm
Cross-ref for #62200
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
Needs ProposalThis issue needs a plan that clarifies the finer details of how it could be implemented.This issue needs a plan that clarifies the finer details of how it could be implemented.SuggestionAn idea for TypeScriptAn idea for TypeScript