fix(rest): Bundle path-to-regexp into ESM output for StackBlitz compat

StackBlitz WebContainers fundamentally cannot resolve CJS named exports
from path-to-regexp, regardless of import style (named, namespace, etc).

Add a rollup ESM build (dist/esm/index.js) that bundles path-to-regexp
directly, eliminating the CJS boundary. Point the "module" and "browser"
export conditions to this bundled ESM output. CJS and React Native paths
are unchanged.

Reverts the namespace import workaround since it's no longer needed.

Made-with: Cursor

Author: ntucker

.changeset/bundle-path-to-regexp-esm.md

@@ -0,0 +1,10 @@
+---
+'@data-client/rest': patch
+---
+
+Fix StackBlitz WebContainers compatibility with path-to-regexp
+
+Bundle `path-to-regexp` into the ESM output (`dist/esm/index.js`) via rollup,
+eliminating the CJS/ESM boundary that StackBlitz WebContainers cannot resolve.
+The `"module"` and `"browser"` export conditions now point to this bundled ESM
+build, while CJS (`dist/index.js`) and React Native (`lib/`) paths are unchanged.

packages/rest/package.json

@@ -84,10 +84,11 @@
   },
   "exports": {
     ".": {
-      "module": "./lib/index.js",
+      "types": "./lib/index.d.ts",
+      "module": "./dist/esm/index.js",
       "import": "./node.mjs",
       "require": "./dist/index.js",
-      "browser": "./lib/index.js",
+      "browser": "./dist/esm/index.js",
       "react-native": "./lib/index.js",
       "default": "./lib/index.js"
     },

packages/rest/rollup.config.mjs

@@ -26,6 +26,12 @@ function isExternal(id) {
   return dependencies.some(dep => dep === id || id.startsWith(dep));
 }
 
+// Bundles path-to-regexp but externalizes other deps
+function isExternalBundlePTR(id) {
+  if (id === 'path-to-regexp' || id.startsWith('path-to-regexp/')) return false;
+  return isExternal(id);
+}
+
 const configs = [];
 if (process.env.BROWSERSLIST_ENV !== 'node12') {
   //TODO: this needs to use src/index.ts so it doesn't include the wrong polyfills
@@ -54,6 +60,28 @@ if (process.env.BROWSERSLIST_ENV !== 'node12') {
       filesize({ showBrotliSize: true }),
     ],
   });
+  // ESM build that bundles path-to-regexp (CJS) into ESM output.
+  // StackBlitz WebContainers cannot resolve CJS named exports, so we must
+  // eliminate the CJS boundary by inlining path-to-regexp here.
+  configs.push({
+    input: 'src/index.ts',
+    external: isExternalBundlePTR,
+    output: [{ file: 'dist/esm/index.js', format: 'esm' }],
+    onwarn,
+    plugins: [
+      babel({
+        exclude: ['node_modules/**', '/**__tests__/**'],
+        extensions,
+        babelHelpers: 'runtime',
+        rootMode: 'upward',
+        caller: { polyfillMethod: false },
+      }),
+      resolve({ extensions }),
+      resolveTsAsJs,
+      commonjs({ extensions }),
+      json(),
+    ],
+  });
   configs.push(typeConfig);
   configs.push(typeConfigNext);
 } else {

packages/rest/src/RestHelpers.ts

@@ -1,11 +1,11 @@
-// Namespace import avoids webpack's per-export CJS validation, which fails in
-// StackBlitz WebContainers. Named imports (`import { compile }`) trigger
-// `importExportsPresence` checks that break there. `import *` defers property
-// access to runtime. No tree-shaking loss: CJS modules are included whole
-// regardless of import style, and webpack 5 traces static access on namespaces.
-import type { PathFunction, Token, ParamData } from 'path-to-regexp';
-import * as pathToRegexpModule from 'path-to-regexp';
-const { compile, parse, pathToRegexp } = pathToRegexpModule;
+import {
+  compile,
+  PathFunction,
+  parse,
+  pathToRegexp,
+  Token,
+  ParamData,
+} from 'path-to-regexp';
 
 import { ShortenPath } from './pathTypes.js';