Core: Prebundle node-logger and make it CJS only

[ndelangen]

I was alerted about this stack trace by @integrayshaun :

var stringWidth = require('string-width')
                  ^
Error [ERR_REQUIRE_ESM]: require() of ES Module /Users/shaun/Development/Integrations/storybook-clippy/node_modules/string-width/index.js from /Users/shaun/Development/Integrations/storybook-clippy/node_modules/wide-align/align.js not supported.
Instead change the require of index.js in /Users/shaun/Development/Integrations/storybook-clippy/node_modules/wide-align/align.js to a dynamic import() which is available in all CommonJS modules.
    at Object.<anonymous> (/Users/shaun/Development/Integrations/storybook-clippy/node_modules/wide-align/align.js:2:19)
    at Object.<anonymous> (/Users/shaun/Development/Integrations/storybook-clippy/node_modules/gauge/render-template.js:2:13)
    at Object.<anonymous> (/Users/shaun/Development/Integrations/storybook-clippy/node_modules/gauge/plumbing.js:3:22)
    at Object.<anonymous> (/Users/shaun/Development/Integrations/storybook-clippy/node_modules/gauge/index.js:2:16)
    at Object.<anonymous> (/Users/shaun/Development/Integrations/storybook-clippy/node_modules/npmlog/log.js:3:13)
    at Object.<anonymous> (/Users/shaun/Development/Integrations/storybook-clippy/node_modules/@storybook/node-logger/dist/index.js:1:1141)
    at Object.<anonymous> (/Users/shaun/Development/Integrations/storybook-clippy/node_modules/@storybook/cli/dist/generate.js:11:4254)
    at Object.<anonymous> (/Users/shaun/Development/Integrations/storybook-clippy/node_modules/@storybook/cli/bin/index.js:9:1)
    at Object.<anonymous> (/Users/shaun/Development/Integrations/storybook-clippy/node_modules/storybook/index.js:3:1) {
  code: 'ERR_REQUIRE_ESM'

We traced it to wide-align/align.js indeed calling require('string-width'), which is not allowed by JS law.

I prebundled node-logger, meaning all dependencies are now bundled in.

[ShaunEvening]

Looks okay to me 👍

[ndelangen]

In my experiment to change the bundle script for CJS dist, the NX remote cache was corrupted somehow.
I tried a bunch of things to get it to regenerate, but it seems touching package.json is likely the best way. sigh!

[tjfred35]

I have a setup (nx / vite) where I need the "import" declaration -- in this case, adding back "import": "./dist/index.js" would solve my issue while still using a "prebundled" version of node-logger?

This broke our upgrade from storybook 7.0.24 to 7.4.46