Something wrong with package exports

[nicksrandall]

Bug report

Packages affected

• sandpack-client
• sandpack-react

Description of the problem

Error from esbuild

✘ [ERROR] Could not resolve "@codesandbox/sandpack-client"

    ../../node_modules/.pnpm/@codesandbox+sandpack-react@2.0.6_biqbaboplfbrettd7655fr4n2y/node_modules/@codesandbox/sandpack-react/dist/esm/index.js:7:95:
      7 │ ...kClient, extractErrorDetails } from '@codesandbox/sandpack-client';
        ╵                                        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

  Importing the directory "./dist" is forbidden by this package:

    ../../node_modules/.pnpm/@codesandbox+sandpack-react@2.0.6_biqbaboplfbrettd7655fr4n2y/node_modules/@codesandbox/sandpack-client/package.json:21:16:
      21 │       "import": "./dist",
         ╵                 ~~~~~~~~

  The presence of "exports" here makes importing a directory forbidden:

    ../../node_modules/.pnpm/@codesandbox+sandpack-react@2.0.6_biqbaboplfbrettd7655fr4n2y/node_modules/@codesandbox/sandpack-client/package.json:15:2:
      15 │   "exports": {
         ╵   ~~~~~~~~~

What were you doing when the problem occurred?

Trying to build an app using esbuild.

Your Environment

Software	Name/Version
Sandpack-client version
Sandpack-react version	2.0.6
Browser
Operating System

[danilowoz]

ohhhh exports...

@DeMoorJasper does it make sense for you? I'm not an expert on exports

[azu]

I noticed that

sandpack/sandpack-client/package.json

Lines 16 to 18 in affdbd5

	"./clients/node": {
	"import": "./dist/clients/node/index.js",
	"require": "./dist/clients/node/index.js"

refer same .js in import/require.

Maybe

    "./clients/node": {
-     "import": "./dist/clients/node/index.js",
+     "import": "./dist/clients/node/index.mjs",
      "require": "./dist/clients/node/index.js"

FYI: publint will be helpful(This ESM and CJS decision seems buggy).
https://publint.dev/@codesandbox/sandpack-client@2.0.7