[BUG] Wrong identifier name with strict esm

[Mister-Hope]

• I have searched for existing issues that already reported this problem and found none

Describe the bug

The following are dist file of markmap-toolbar/dist/index.mjs, which is supposed to use under esm env.

/*! markmap-toolbar v0.15.4 | MIT License */
import { mountDom } from '@gera2ld/jsx-dom';
import { jsx, jsxs, Fragment } from '@gera2ld/jsx-dom/jsx-runtime';
// ...

However, the '@gera2ld/jsx-dom/jsx-runtime' is not resolvable under strict esm env, as every file requires externsion, and @gera2ld/jsx-dom has no export field that declares /jsx-runtime export.

So I am getting:

demo/md-enhance demo:webpack-build:   message: "Module not found: Error: Can't resolve '@gera2ld/jsx-dom/jsx-runtime' in '/home/runner/work/vuepress-theme-hope/vuepress-theme-hope/node_modules/.pnpm/markmap-toolbar@0.15.4_markmap-common@0.15.3/node_modules/markmap-toolbar/dist'\n" +
demo/md-enhance demo:webpack-build:     "Did you mean 'jsx-runtime.js'?\n" +
demo/md-enhance demo:webpack-build:     "BREAKING CHANGE: The request '@gera2ld/jsx-dom/jsx-runtime' failed to resolve only because it was resolved as fully specified\n" +
demo/md-enhance demo:webpack-build:     `(probably because the origin is strict EcmaScript Module, e. g. a module with javascript mimetype, a '*.mjs' file, or a '*.js' file where the package.json contains '"type": "module"').\n` +
demo/md-enhance demo:webpack-build:     'The extension in the request is mandatory for it to be fully specified.\n' +
demo/md-enhance demo:webpack-build:     'Add the extension to the request.',

The problem should belong to your bundling logic, to make things right, it should be :

- import { jsx, jsxs, Fragment } from '@gera2ld/jsx-dom/jsx-runtime';
+ import { jsx, jsxs, Fragment } from '@gera2ld/jsx-dom/jsx-runtime.js';

[Mister-Hope]

Currently I will try to patch this dep to avoid problem, but I am creating a tool which depends on mindmap while I can not apply my patch downstream.

It would be great if you can managed to fix this critical issue ASAP @gera2ld

[Mister-Hope]

Update: to speed up this apply, I try to make a PR to fix it, but it seems that you are creating wrapper package about rollup of your own:

    "@gera2ld/plaid": "~2.5.9",
    "@gera2ld/plaid-common-ts": "~2.5.9",
    "@gera2ld/plaid-rollup": "~2.5.9",
    "@gera2ld/plaid-test": "~2.5.9",

Which confuses me, and stop me from raising a valid PR. Maybe you should check that yourself?

[gera2ld]

Could you help to verify in markmap-toolbar@0.15.5-alpha.1? Thanks.

[Mister-Hope]

It works, thanks

[gera2ld]

FYI 0.15.5 has been released

[yimuc]

Hello, can you export umd packages? @gera2ld