Maintenance: Prep for ESM

[ndelangen]

What I did

• use node:path over path
• use node:fs over fs
• use named exports of fs and path, where possible.
• replace/split fs-extra use into @ndelangen@fs-extra-unified and node:fs/promise

@ndelangen@fs-extra-unified has an export map in it's package.json, and is explicit about which exports it has.

fs-extra does have ESM support, but to need to explicitly import fs-extra/esm to get it, which is a problem when generating dist code that needs to run in both CJS & EMS environments.
fs-extra/esm only exports a subset of methods, because most can be imported from node:fs/promise.

I'd be great if we cleaned up the mocking situation a bit..

Manual testing

• We should be able to run any script we're used to running in the monorepo during development.
• Creating sandboxes should be unchanged
• Running sandboxes should be unchanged.

I tested the above on my local machine ☝️

These changes should not have any impact on browser code, AFAICS

🦋 Canary release

This PR does not have a canary release associated. You can request a canary release of this pull request by mentioning the @storybookjs/core team here.

core team members can create a canary release here or locally with gh workflow run --repo storybookjs/storybook canary-release-pr.yml --field pr=<PR_NUMBER>

[socket-security]

New dependencies detected. Learn more about Socket for GitHub ↗︎

Package	New capabilities	Transitives	Size	Publisher
npm/@ndelangen/[email protected]	Transitive: environment, filesystem	+8	4.12 MB	ndelangen

View full report↗︎

[JReinhold]

This looks like a great improvement, but I think we want ESLint's help to keep this from rotting.

• https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-node-protocol.md can enforce the node: prefix
• https://eslint.org/docs/latest/rules/no-restricted-imports can be used to disallow the use of default imports from those modules
• https://eslint.org/docs/latest/rules/no-restricted-imports can also be used to disallow importing directly from fs-extra but always use your wrapper instead.

I still see a couple of instances not using 'node:X' yet: