Add '@rollup/plugin-esm-shim'

[Peeterush]

A new attempt at getting both ESM and UMD builds to play well!

[Peeterush]

Ah, Deno is like that one dentist.

I made a PR to @rollup/plugin-esm-shim to add the prefix that Deno is asking for.

[Peeterush]

Edit: added a test for dist files like suggested by iambumblehead.

[dlemstra]

We could also use the features here instead so we don't need a regex?

[Peeterush]

Good idea!

[dlemstra]

Can we avoid using that module if magick-native would provide a magick.js and magick.mjs file? There is a build option in emscripten sdk to create a ES6 module. How should I package those files in a package.json file? I am now using this file: https://github.com/dlemstra/Magick.Native/blob/main/publish/wasm/package.json.

[Peeterush]

ESM would be better yes! Vite would only use the ESM if it's available, so no CJS version needed. Package should look something like this:

{
    "name": "@dlemstra/magick-native",
    "version": "",
    "type": "module",
    "exports": {
        ".": {
            "types": "./index.d.ts",
            "default": "./index.js",
        },
    },
    "module": "./index.js",
    "types": "./index.d.ts",
    "repository": {
        "type": "git",
        "url": "git+https://github.com/dlemstra/Magick.Native.git",
    },
    "author": "Dirk Lemstra",
    "license": "Apache-2.0",
    "files": [
        "./index.d.ts",
        "./index.js",
        "magick.wasm",
        "NOTICE",
    ],
}

Note: because of "type": "module", any .js is assumed to be ESM, so no .mjs needed.

[Peeterush]

Also; my PR to @rollup/plugin-esm-shim has been accepted and published, so Deno is happy now!

[dlemstra]

Do we still need the @rollup/plugin-esm-shim once Magick.Native has switched to ESM?

[dlemstra]

I have made some changes to the Magick.Native build but I did not rename the files. Can you check if that works? You can use version 0.202311.80706 for this.