Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feature request: Enabling typescript isolated modules #212835

Closed
Tracked by #212907
SimonSiefke opened this issue May 15, 2024 · 0 comments · Fixed by #212913
Closed
Tracked by #212907

feature request: Enabling typescript isolated modules #212835

SimonSiefke opened this issue May 15, 2024 · 0 comments · Fixed by #212913
Assignees
Labels
engineering VS Code - Build / issue tracking / etc. feature-request Request for new features or functionality insiders-released Patch has been released in VS Code Insiders verification-needed Verification of issue is requested verified Verification succeeded
Milestone

Comments

@SimonSiefke
Copy link
Contributor

SimonSiefke commented May 15, 2024

Related #160416

For migrating towards ESM in VSCode, it seems the tsconfig isolatedModules option could be useful, ensuring that imports are compatible with ESM.

For example, this typescript code transpiled to javascript would cause a runtime error because of the someType export:

import { someType, someFunction } from "someModule";
 
someFunction();

export { someType, someFunction };

The isolatedModules rule would enforce a type only import for someType:

import { type someType, someFunction } from "someModule";
 
someFunction();

export { type someType, someFunction };

Enabling the isolatedModules tsconfig option in VSCode, it seems these files would need to adjusted:

Found 26 errors in 6 files.

Errors  Files
     9  src/vs/base/common/amd.ts:57
     4  src/vs/base/node/processes.ts:14
     1  src/vs/workbench/contrib/accessibility/browser/accessibilityConfiguration.ts:636
     1  src/vs/workbench/contrib/inlineChat/common/inlineChat.ts:172
     8  src/vs/workbench/contrib/performance/browser/perfviewEditor.ts:331
     3  src/vs/workbench/contrib/tasks/common/taskService.ts:18
@VSCodeTriageBot VSCodeTriageBot added the unreleased Patch has not yet been released in VS Code Insiders label May 22, 2024
@VSCodeTriageBot VSCodeTriageBot added this to the May 2024 milestone May 22, 2024
@jrieken jrieken assigned jrieken and unassigned mjbvz May 22, 2024
@jrieken jrieken added engineering VS Code - Build / issue tracking / etc. feature-request Request for new features or functionality labels May 22, 2024
@VSCodeTriageBot VSCodeTriageBot added insiders-released Patch has been released in VS Code Insiders and removed unreleased Patch has not yet been released in VS Code Insiders labels May 24, 2024
@benibenj benibenj added verified Verification succeeded verification-needed Verification of issue is requested labels May 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
engineering VS Code - Build / issue tracking / etc. feature-request Request for new features or functionality insiders-released Patch has been released in VS Code Insiders verification-needed Verification of issue is requested verified Verification succeeded
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants