should allow use `export = ` in `.cts` when module is esnext

[bluelovers]

Bug Report

🔎 Search Terms

🕗 Version & Regression Information

• This is a crash
• This changed between versions ______ and _______
• This is the behavior in every version I tried, and I reviewed the FAQ for entries about _________
• I was unable to test this on prior versions because _______

⏯ Playground Link

Playground link with relevant code

💻 Code

index.cts

import { globToTree } from './index';

export = globToTree

index.ts

export function globToTree(): any {}
Object.defineProperty(globToTree, "__esModule", { value: true });
Object.defineProperty(globToTree, "globToTree", { value: globToTree });
Object.defineProperty(globToTree, "default", { value: globToTree });
export default globToTree

{
  "compilerOptions": {
    "module": "esnext",
    "target": "es2019",
    "jsx": "preserve",
    "moduleResolution": "node",
    "allowJs": false,
    "allowUmdGlobalAccess": false,
    "sourceMap": true,
    "inlineSourceMap": false,
    "inlineSources": true,
    "declaration": false,
    "experimentalDecorators": true,
    "emitDecoratorMetadata": true,
    "allowSyntheticDefaultImports": true,
    "resolveJsonModule": true,
    "strict": true,
    "strictBindCallApply": true,
    "strictNullChecks": false,
    "strictFunctionTypes": true,
    "noImplicitThis": true,
    "noImplicitOverride": true,
    "noUncheckedIndexedAccess": true,
    "noPropertyAccessFromIndexSignature": true,
    "preserveConstEnums": true,
    "forceConsistentCasingInFileNames": true,
    "incremental": true,
    "noErrorTruncation": true,
    "removeComments": false,
    "traceResolution": false,
    "newLine": "lf",
    "esModuleInterop": true,
    "importHelpers": true,
    "noFallthroughCasesInSwitch": true,
    "noUnusedLocals": true,
    "noUnusedParameters": true,
    "skipLibCheck": true,
    "noEmit": true
  }
}

🙁 Actual behavior

show error

or use @ts-ignore avoid it

import { globToTree } from './index';

// @ts-ignore
export = globToTree

🙂 Expected behavior

no need @ts-ignore

[RyanCavanaugh]

What code is supposed to be generated in this case?

[bluelovers]

input
https://github.com/bluelovers/ws-glob/blob/master/packages/glob-tree-list/src/index.cts
https://github.com/bluelovers/ws-glob/blob/master/packages/glob-tree-list/src/index.ts

output
https://github.com/bluelovers/ws-glob/blob/master/packages/glob-tree-list/dist/index.cjs.development.cjs
https://github.com/bluelovers/ws-glob/blob/master/packages/glob-tree-list/dist/index.esm.mjs

[RyanCavanaugh]

https://github.com/bluelovers/ws-glob/blob/master/packages/glob-tree-list/dist/index.esm.mjs - this is just a default export, like it's telling you to do. export = (a function doesn't make sense in an ES Module output format. What's the motivation for not writing export default globToTree ?

[fatcerberus]

@RyanCavanaugh I guess they’re expecting the .cts file to compile to CommonJS instead of ESM?

[andrewbranch]

This was implemented as a new feature in 5.6 (#58825)