Repository navigation
VSCode client libraries have invalid exports #192144
Description
VSCode has various client libraries that exist in separate repositories. Most of these packages have invalid package exports. As a result, it’s difficult to consume them in a package that is configured correctly. This means some packages can only be used if they are included via bundler hacks. Sometimes it’s the other way around, and they can be used with Node.js, but in a way they are no longer compatible with a bundler.
Currently this is blocking me from publishing a big MDX language server update (mdx-js/mdx-analyzer#340).
I want to help remedy this situation. I have already created some issues and pull requests, but perhaps it’s best to organize this from a central issue, hence this issue. Based on my experience working with language servers, Monaco Editor, and the VSCode list of useful Node.js modules, I have assembled the list below. A checkmark (✓) indicates everything is ok. A cross (❌) indicates there is a problem with a link to an explanation below. If there is no symbol, that indicates there’s no support for that module format, which is fine.
It also appears VSCode itself doesn’t depend on any of these modules. But some are generated from the VSCode repository. As far as I can tell, this means there is no need to support AMD nor UMD. Possibly relevant are #160416 and #166033.
Faux ESM
Faux ESM means a package publishes ESM syntax, but it is not marked as such.
There are two ways to mark Node.js code as ESM:
- The ESM files use the
.mjsextension, and.d.mtsfor file extensions. - If
package.jsoncontains the following,.jsand.d.tsfiles will be treated as ESM.{ "type": "module" }
Dual published faux ESM packages often use the non-standard "module" field in package.json. Many module bundlers respect this field, causing imports to behave different when bundled from when using it with Node.js. Instead, packages should use package "exports" point to the correct location for CJS or ESM.
{
"exports": {
".": {
"import": "./path/to/esm/entrypoint.mjs",
"default": "./path/to/cjs/entrypoint.cjs"
}
}
}Also when using ESM, imports must include the file extension.
UMD
UMD is compatible with CJS. Node.js supports importing CJS as named imports, if the CJS is structured properly.
For example, the following CJS file:
module.exports.a = 'a'
exports.b = 'b'Can be imported as such from ESM:
// 1)
import module from './commonjs.cjs'
// 2)
import { a, b } from './commonjs.cjs'
console.log({ a, b, module })When using UMD, typically only 1) works, not 2). When dual publishing this with faux ESM, often only 2) works in bundlers, not 1).
Also this prevents tree shaking.
Bundled Dependencies
Some packages bundle their dependencies. When bundling dependencies, this means they are included in the package. If another package then depends on the bundled dependency, this means it gets duplicated, increasing the bundle size for the end user.
Browser Field
Some packages use the "browser" field in package.json. The intention is to tell bundlers to resolve a different module when targeting the browser. This is a non-standard field. The correct way to tell bundlers how to resolve imports for a browser, is using package "exports".
{
"exports": {
".": {
"browser": "./path/to/browser/entrypoint.js",
"default": "./path/to/node/entrypoint.js"
}
}
}Likewise, packages can define where packages are imported from conditionally using package "imports":
{
"imports": {
// Beware of https://github.com/nodejs/node/issues/49257 for CJS
"#path": {
"browser": "path-browserify",
"default": "path"
}
}
}ESM Implies Browser
Node.js can use ESM. Module bundlers can resolve imports to CJS, regardless of whether they target Node.js or browsers. Some packages incorrectly assume ESM is for the browser, and ESM is for Node.js. Instead, packages should use "exports" (See Browser Field).
Browser Implies ESM
Some packages resolve the browser condition to an ESM field. This means require() calls may lead to ESM, which is non-standard behaviour and may lead to unexpected error.
File Does Not Exist
This package points to a file that does not exist.
Default Export Compiled to CJS
Although I didn’t find any packages to which this apply, it’s good to add this to a list of pitfalls regarding CJS/ESM interop. When a default export is compiled to CJS, it becomes:
function someExport() {}
exports.default = someExportThis means that from native ESM, usage becomes:
import someExport from './commonjs.cjs'
someExport.default()This is typically underable, so it’s best to avoid compiling default exports to CJS.
Dual Publishing
Various package dual publish ESM and CJS. Many people assume this has benefits, but they don’t truly understand the details. Really this mostly has downsides.
- Dual publishing complicates the build process.
- TypeScript wasn’t meant for dual publishing and becomes less helpful.
- Because of ESM, you can’t use CJS APIs (
require,module,__dirname,__filename). - Because of CJS, you can’t use ESM features (import statements, export statements, top-level await).
- Because of CJS, you can’t use ESM-only dependencies.
- You need to dual-publish TypeScript types too, and make sure they are correct.
Generally people are better off picking one module format. Choose CJS if you want your dependants to be abel to use CJS without using dynamic imports. Choose ESM if you want to help push the JavaScript ecosystem forward to a single module syntax.
Activity
hediet commentedon Sep 12, 2023
Thanks for assembling this list!
What is your suggestion to avoid dual publishing?
Any change would probably be breaking here.
zm-cttae commentedon Sep 14, 2023
Recently an update to unofficial lib
vscode-textmate-languageserviceadded some APIs highly requested in core:Could you assess the library for its export validity?
remcohaszing commentedon Sep 18, 2023
Avoiding dual publishing means to publish either CJS or ESM, not both.
CJS
To publish CJS only, requires the following setup:
When not writing default exports, this works exactly as expected in from both CJS and ESM dependants. The downside is, this means we can’t use ESM dependencies.
This removes support from AMD/UMD. This is technically breaking, but personally I don’t think anyone is affected by this. In fact, this solves an annoying warning that some build tools show:
ESM
To publish ESM only, requires the following setup:
The biggest upside is that you can use ESM dependencies. Also the existence of two module formats in the JavaScript JavaScript ecosystem is hard to deal with for many users. Personally I believe ESM is the future and it’s best to bite the bullet and go ESM only.
This means users of the package also need to go ESM-only.
Currently VSCode extensions don’t support ESM. But I don’t think this is a problem. As far as I know VSCode extensions should bundle their dependencies, so the module format doesn’t really matter.
Dual Publishing
Despite the quirks of dual publishing, it is possible to do so. This means using
"main"is not an option, so we would have to use"exports", marking all other package files as private. This could technically be considered breaking.So, some breaking changes are:
umdtoesmcould break. This is because they are no longer necessary.exportsbreaks imports of internals. The only project of which I’m aware it depends on this, isyaml-language-serverforvscode-json-languageservice. Currently it has an exact version pinned, so it won’t just break. To support their use case, it would be best to export the things they need fromvscode-json-languageserviceas part of the public API.remcohaszing commentedon Sep 4, 2025
Fast forward to 2025, the ecosystem has moved forward.
The supported Node.js versions are 20, 22, and 24. All of these version support
require(esm). ESM support withoutrequire(esm)was introduced in Node.js 12 in 2019.TypeScript 6.0 is upcoming. It’s going to remove support for old module formats:
amd,umd, andsystemjsas well as theoutFileoption.As far as I’m aware, VSCode extensions are mostly bundled. They barely ever use the
dependenciesfield inpackage.json. Bundlers can often handle ESM better than other module formats in regards of scanning, minifying, and tree-shaking. Other module formats require the bundler to inject boilerplate code for ESM ↔ CJS interoperability. For example, see this esbuild playground.I believe it’s time to go ESM-only. There’s no reason to use CJS anymore.
Removing CJS support will is no longer a breaking change for supported Node.js versions. However, it is a breaking change for older unsupported Node.js versions. So this change requires semver-major releases. Since this is breaking, it would also be a good opportunity to think of other breaking changes that can be made.
Two of these packages don’t follow semver:
vscode-languageserver-typesandvscode-languageserver-protocol. They follow the version of the LSP specification. I don’t know how that should be handled.remcohaszing commentedon Nov 28, 2025
Is going ESM-only something the VSCode team wants to pursue? This involves making semver major releases. I would love to help out with this, but I can’t make releases of course. Feel free to reach out and discuss if you would like my help with this.
hediet commentedon Dec 1, 2025
Yes! But we have to be careful to see who still depends on the AMD version.
The current build-setup (which uses vite/rollup) should handle both targets fine though.
KirillTregubov commentedon Jan 29, 2026
Is there any update on this effort? Building upon what Remco Haszing (@remcohaszing) outlined in late 2025, ESM-only seems increasingly standard across the ecosystem. I'm particularly interested in microsoft/node-jsonc-parser#57, as that library has over 1000 downstream dependents on npm.
Are there any major blockers that the broader open source community could help with?
remcohaszing commentedon Feb 13, 2026
Does this comment apply to Monaco editor or all packages? Monaco editor is currently broken. See remcohaszing/monaco-yaml#278 and remcohaszing/monaco-prettier#1.
Also, because Monaco editor is so hard to get to work, packages like
@monaco-editor/loader/@monaco-editor/reactare quite popular. This package actually loads Monaco from the CDN. This of course causes download stats to increase, if you have those number. But really that’s not a choice many people make consiously.Also packages like
monaco-typesshould not have to exist.If the VSCode team is seriously persuing this, I have some suggestions how to improve the situation for Monaco as well as some smaller breaking changes for other packages.