Use ESModules instead of module.exports (#786)

[Konakona-chan]

Hello there.

As for 2021, NodeJS versions <12 without ESM support (are deprecated)[https://nodejs.org/en/about/releases/].

This PR switches debug package from CJS to ESM version, so it can be used in modern applications based on asynchronous execution.

Resolves #786.

Implementation notes:

1. This changeset updates dependencies to the latest ones to reduce package fragmentation and introduce ESM-based dependencies
2. It removes debug.destroy() function according to deprecation notice (notably, it updates ms to canary version from ESM version vercel/ms#159)
3. It does not change API for existing CJS users who already used import statement; however require(...) should be replaced to import as described in README.md

[Qix-]

This package is going to be rewritten in the next version, so there's not much I can do with this PR as it stands. Thank you for taking the time, however!

[Konakona-chan]

Cool, I hope it will be released soon!

[milahu]

@Konakona-chan thanks for your work <3
debug-cjs breaks vite, so i need debug-esm

Uncaught ReferenceError: process is not defined

in node.js line 118

export const inspectOpts = Object.keys(process.env).filter(key => {

this happens in browser, so node.js should not be loaded ...

debug is used here

• svelte-jsoneditor/src/lib/components/controls/createAutoScrollHandler.js
• svelte-jsoneditor/src/lib/components/controls/createFocusTracker.js

possible solutions

nope

-import createDebug from 'debug'
+import createDebug from 'debug/src/browser.js'

... or

-import createDebug from 'debug'
+import createDebug from 'debug/browser'

... or in debug/index.js, use dynamic import like

const createDebug = isBrowser
  ? (await import('./browser.js'))
  : (await import('./node.js'))
;

export default createDebug;

debug/package.json

  "exports": {
    ".": {
      "import": {
        "browser": "./src/browser.js",
        "default": "./src/index.js"
      }
    },
    "./package.json": "./package.json"
  },

per npm docs should be

  "exports": {
    ".": {
      "import": {
        "node": "./src/node.js",
        "default": "./src/browser.js"
      }
    },
    "./package.json": "./package.json"
  },

works with rollup : )

fixed in https://github.com/milahu/debug/commits/patch-1

install with

npm install "github:milahu/debug#28e27c8b4249fec2f778cf61f13e18e8c814822a"

to clarify, this is an actual working ESM version of the NPM debug package (what a name ...)

→ no need to wait for

This package is going to be rewritten in the next version

[Konakona-chan]

@milahu , I was able to overcome the issue with Svelte+Vite and unified (esm-only) -> markdown -> debug (cjs-only) with optimizeDeps:
https://gitlab.com/Kona-chan/kona-chan.gitlab.io/-/blob/master/svelte.config.js#L32

[FossPrime]

Please resubmit this.

Deno and UNPKG with ?module are broken because of this.