Provide an ES Module entrypoint for @elastic/elasticsearch

[adamyeats]

Related to #1281 and #1742.

🚀 Feature Proposal

@elastic/elasticsearch is currently a CommonJS module. While CommonJS isn't going away any time soon, ECMAScript Modules are very much "the future" of the JavaScript ecosystem. As the community moves in this direction, it may be wise for us to consider moving this library in the same way.

To accommodate both CJS and ESM users, we might want to create an ES module wrapper that exports both a CJS and ESM version of the library.

At the very least, as suggested in #1281, it would be good to make the type of the module explicit in package.json if we decide to remain with CJS for the time being.

Motivation

This will allow the library to work in Deno, and enable asynchronous module loading in Deno and Node.js.

[github-actions]

This issue is stale because it has been open 90 days with no activity. Remove the stale label, or leave a comment, or this will be closed in 14 days.

[JoshMock]

Still relevant. We've got Deno/Bun.js support on the long-term roadmap. I recently merged #1281 to explicitly declare the library as a CommonJS package, but improving to include ESM support will be an important step toward multi-runtime compatibility.

[karfau]

FYI: I'm able to use import * as elasticsearch from "npm:@elasticsearch/elasticsearch"; in Deno already without any issues.

[github-actions]

This issue is stale because it has been open 90 days with no activity. Remove the stale label, or leave a comment, or this will be closed in 14 days.

[karfau]

4 months ago it was said it is still relevant

[JoshMock]

thanks for your patience, @karfau. I was on leave for 2 months, just getting back to addressing open issues now.

[github-actions]

This issue is stale because it has been open 90 days with no activity. Remove the stale label, or leave a comment, or this will be closed in 14 days.

[github-actions]

This issue is stale because it has been open 90 days with no activity. Remove the stale label, or leave a comment, or this will be closed in 14 days.

[karfau]

At least for more recent deno versions this might have been "resolved" by improved nodejs compatibility...

[JoshMock]

It would still be ideal to have good ESM support soon, since that's the unified standard everyone is moving toward. I managed to improve things a bit for @elastic/transport recently, so it should hopefully be relatively straightforward by adding exports metadata to package.json.

[harshulj]

#2348 This seems to have broken import in ECMAScript modules.

https://github.com/elastic/elasticsearch-js/pull/2348/files#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519R8

[JoshMock]

Fixed in #2475