@woodsportal/hubspot-kit 1.0.36 → 1.0.38

package/CHANGELOG.md

@@ -19,6 +19,16 @@
 - **CSS entries** — shared `tailwind.css.config.css`; documented never-merge rules on `main.dev.css` / `main.prod.css`
 - **`wp doctor`** — CSS pipeline checks (prefix, `.temp` @source, `skipRuntimeCss`, `module.css` dw utilities, CDN `require_css` warning)
 
+## 1.0.38 — 2026-06-08
+
+### Dev server
+- **Module-config overlay** — stop force-resolving `react` / `react-dom` to CJS `index.js` for kit UI under `.wphs/`; fixes `createContext` / named export errors in the browser on `yarn local`
+
+## 1.0.37 — 2026-06-08
+
+### Dev server
+- **Consumer dep resolution** — replace `createRequire` / `require.resolve` with a filesystem `package.json` exports resolver for Vite, React plugins, and JSX runtime aliases; fixes `getStatus` crash on Node 24+ when starting `yarn local`
+
 ## 1.0.36 — 2026-06-08
 
 ### Dev server

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@woodsportal/hubspot-kit",
-  "version": "1.0.36",
+  "version": "1.0.38",
   "description": "WoodsCLI — HubSpot CMS dev kit by WoodsPortal. wp CLI, Vite presets, module-config overlay, and theme/module build pipelines.",
   "type": "module",
   "main": "./dist/index.js",
@@ -51,7 +51,7 @@
     "build": "tsup",
     "dev": "tsup --watch",
     "typecheck": "tsc --noEmit",
-    "test": "node --import tsx --test 'src/lib/**/*.test.ts' 'src/schema/**/*.test.ts' 'src/vite/**/*.test.ts' 'scripts/portal-build-id.test.mjs' 'scripts/vite/ensure-dev-kit-shims.test.mjs' 'scripts/vite/consumer-react-aliases.test.mjs' && vitest run src/dev-module-config/__tests__/normalize-repeater-items.test.ts src/dev-module-config/__tests__/module-config-transform.test.ts src/dev-module-config/__tests__/flatten-visible-fields.test.ts",
+    "test": "node --import tsx --test 'src/lib/**/*.test.ts' 'src/schema/**/*.test.ts' 'src/vite/**/*.test.ts' 'scripts/portal-build-id.test.mjs' 'scripts/resolve-consumer-dep.test.mjs' 'scripts/vite/ensure-dev-kit-shims.test.mjs' 'scripts/vite/consumer-react-aliases.test.mjs' 'scripts/vite/resolve-from-project-root.test.mjs' && vitest run src/dev-module-config/__tests__/normalize-repeater-items.test.ts src/dev-module-config/__tests__/module-config-transform.test.ts src/dev-module-config/__tests__/flatten-visible-fields.test.ts",
     "test:examples": "node examples/smoke/run-matrix.mjs --tier static",
     "lint": "eslint src",
     "verify": "npm run typecheck && npm run test && npm run test:examples && npm run build",

package/scripts/resolve-consumer-dep.mjs

@@ -1,25 +1,90 @@
 /**
  * Resolve and import packages from the consumer project (woodscli.json root).
- * Kit scripts live under @woodsportal/hubspot-kit; Vite/React plugins are consumer deps.
+ * Pure filesystem resolution — avoids createRequire (Node 24+ getStatus) and
+ * import.meta.resolve from kit context (wrong node_modules).
  */
-import { createRequire } from 'node:module'
-import { resolve } from 'node:path'
+import { existsSync, readFileSync } from 'node:fs'
+import { resolve, sep } from 'node:path'
 import { pathToFileURL } from 'node:url'
+
 import { findProjectRoot } from './project-root.mjs'
 
 /**
- * @param {string} specifier - package name or subpath export
+ * @param {string} specifier
+ * @param {string} projectRoot
+ */
+function readPkgJson(projectRoot, pkgName) {
+  const pkgJsonPath = resolve(projectRoot, 'node_modules', ...pkgName.split('/'), 'package.json')
+  if (!existsSync(pkgJsonPath)) {
+    throw new Error(`Cannot find package '${pkgName}' under ${projectRoot}`)
+  }
+  return { pkgJsonPath, pkg: JSON.parse(readFileSync(pkgJsonPath, 'utf8')) }
+}
+
+/**
+ * @param {string} pkgRoot
+ * @param {string} target
+ */
+function resolveTargetFile(pkgRoot, target) {
+  const rel = target.replace(/^\.\//, '')
+  const abs = resolve(pkgRoot, rel)
+  if (existsSync(abs)) return abs
+  if (existsSync(`${abs}.js`)) return `${abs}.js`
+  if (existsSync(`${abs}.mjs`)) return `${abs}.mjs`
+  if (existsSync(resolve(abs, 'index.js'))) return resolve(abs, 'index.js')
+  return abs
+}
+
+/**
+ * @param {unknown} exportEntry
+ * @returns {string | null}
  */
-export function resolveConsumerDep(specifier) {
-  const root = findProjectRoot()
-  const require = createRequire(resolve(root, 'package.json'))
-  return require.resolve(specifier)
+function pickExportTarget(exportEntry) {
+  if (typeof exportEntry === 'string') return exportEntry
+  if (!exportEntry || typeof exportEntry !== 'object') return null
+
+  const record = /** @type {Record<string, unknown>} */ (exportEntry)
+  for (const key of ['import', 'default', 'node', 'require', 'module-sync']) {
+    const val = record[key]
+    if (typeof val === 'string') return val
+    const nested = pickExportTarget(val)
+    if (nested) return nested
+  }
+  return null
+}
+
+/**
+ * @param {string} specifier e.g. vite, @vitejs/plugin-react, react/jsx-dev-runtime
+ * @param {string} projectRoot
+ */
+export function resolveConsumerDep(specifier, projectRoot = findProjectRoot()) {
+  const segments = specifier.split('/')
+  const pkgName = segments[0].startsWith('@')
+    ? `${segments[0]}/${segments[1]}`
+    : segments[0]
+  const subpath = segments[0].startsWith('@') ? segments.slice(2).join('/') : segments.slice(1).join('/')
+  const pkgRoot = resolve(projectRoot, 'node_modules', ...pkgName.split('/'))
+  const { pkg } = readPkgJson(projectRoot, pkgName)
+
+  if (!subpath) {
+    const rootExport = pickExportTarget(pkg.exports?.['.']) ?? pkg.module ?? pkg.main ?? 'index.js'
+    return resolveTargetFile(pkgRoot, rootExport)
+  }
+
+  const exportKey = `./${subpath}`
+  const exportTarget = pickExportTarget(pkg.exports?.[exportKey])
+  if (exportTarget) {
+    return resolveTargetFile(pkgRoot, exportTarget)
+  }
+
+  return resolveTargetFile(pkgRoot, subpath)
 }
 
 /**
  * @param {string} specifier
+ * @param {string} [projectRoot]
  */
-export async function importConsumerDep(specifier) {
-  const resolved = resolveConsumerDep(specifier)
-  return import(pathToFileURL(resolved).href)
+export async function importConsumerDep(specifier, projectRoot = findProjectRoot()) {
+  const abs = resolveConsumerDep(specifier, projectRoot)
+  return import(pathToFileURL(abs).href)
 }

package/scripts/resolve-consumer-dep.test.mjs

@@ -0,0 +1,22 @@
+import assert from 'node:assert/strict'
+import { describe, it } from 'node:test'
+
+import { importConsumerDep, resolveConsumerDep } from './resolve-consumer-dep.mjs'
+import { consumerDepFixtureRoot } from './test-fixtures/consumer-dep-fixture-root.mjs'
+
+describe('resolveConsumerDep', () => {
+  it('resolves vite from consumer node_modules via filesystem resolver', () => {
+    const vitePath = resolveConsumerDep('vite', consumerDepFixtureRoot)
+    assert.match(vitePath, /node_modules[/\\]vite[/\\]/)
+  })
+
+  it('imports vite without createRequire', async () => {
+    const vite = await importConsumerDep('vite', consumerDepFixtureRoot)
+    assert.equal(typeof vite.createServer, 'function')
+  })
+
+  it('resolves nested package exports (TanStack router vite subpath)', () => {
+    const path = resolveConsumerDep('@tanstack/router-plugin/vite', consumerDepFixtureRoot)
+    assert.match(path, /node_modules[/\\]@tanstack[/\\]router-plugin[/\\]/)
+  })
+})

package/scripts/test-fixtures/consumer-dep/node_modules/@tanstack/router-plugin/dist/esm/vite.js

@@ -0,0 +1,3 @@
+export function TanStackRouterVite() {
+  return { name: 'tanstack-router-vite-fixture' }
+}

package/scripts/test-fixtures/consumer-dep/node_modules/@tanstack/router-plugin/package.json

@@ -0,0 +1,16 @@
+{
+  "name": "@tanstack/router-plugin",
+  "type": "module",
+  "exports": {
+    "./vite": {
+      "import": {
+        "types": "./dist/esm/vite.d.ts",
+        "default": "./dist/esm/vite.js"
+      },
+      "require": {
+        "types": "./dist/cjs/vite.d.cts",
+        "default": "./dist/cjs/vite.cjs"
+      }
+    }
+  }
+}

package/scripts/test-fixtures/consumer-dep/node_modules/react/jsx-dev-runtime.js

@@ -0,0 +1 @@
+export function jsxDEV() {}

package/scripts/test-fixtures/consumer-dep/node_modules/react/jsx-runtime.js

@@ -0,0 +1 @@
+export function jsx() {}

package/scripts/test-fixtures/consumer-dep/node_modules/react/package.json

@@ -0,0 +1,8 @@
+{
+  "name": "react",
+  "type": "module",
+  "exports": {
+    "./jsx-dev-runtime": "./jsx-dev-runtime.js",
+    "./jsx-runtime": "./jsx-runtime.js"
+  }
+}

package/scripts/test-fixtures/consumer-dep/node_modules/vite/dist/node/index.js

@@ -0,0 +1 @@
+export function createServer() {}

package/scripts/test-fixtures/consumer-dep/node_modules/vite/package.json

@@ -0,0 +1,10 @@
+{
+  "name": "vite",
+  "type": "module",
+  "exports": {
+    ".": {
+      "import": "./dist/node/index.js",
+      "require": "./index.cjs"
+    }
+  }
+}

package/scripts/test-fixtures/consumer-dep-fixture-root.mjs

@@ -0,0 +1,8 @@
+import { dirname, resolve } from 'node:path'
+import { fileURLToPath } from 'node:url'
+
+/** Portable fake consumer root — no sibling monorepo checkout required. */
+export const consumerDepFixtureRoot = resolve(
+  dirname(fileURLToPath(import.meta.url)),
+  'consumer-dep',
+)

package/scripts/vite/consumer-react-aliases.js

@@ -1,6 +1,4 @@
-import { createRequire } from 'node:module'
-import { resolve } from 'node:path'
-import { pathToFileURL } from 'node:url'
+import { resolveConsumerDep } from '../resolve-consumer-dep.mjs'
 
 /** Subpath runtimes only — do not alias top-level `react` / `react-dom` (breaks `react/jsx-runtime`). */
 const REACT_RUNTIME_SPECS = [
@@ -15,13 +13,12 @@ const REACT_RUNTIME_SPECS = [
  * `node_modules/@woodsportal/hubspot-kit` never loads a mismatched CJS copy in the browser.
  */
 export function consumerReactAliases(projectRoot) {
-  const require = createRequire(pathToFileURL(resolve(projectRoot, 'package.json')))
   /** @type {Record<string, string>} */
   const alias = {}
 
   for (const spec of REACT_RUNTIME_SPECS) {
     try {
-      alias[spec] = require.resolve(spec)
+      alias[spec] = resolveConsumerDep(spec, projectRoot)
     } catch {
       // consumer may omit optional subpaths until install completes
     }

package/scripts/vite/consumer-react-aliases.test.mjs

@@ -1,32 +1,12 @@
 import assert from 'node:assert/strict'
-import { mkdirSync, mkdtempSync, rmSync, writeFileSync } from 'node:fs'
-import { tmpdir } from 'node:os'
-import { join, resolve } from 'node:path'
 import { describe, it } from 'node:test'
 
+import { consumerDepFixtureRoot } from '../test-fixtures/consumer-dep-fixture-root.mjs'
 import { consumerReactAliases } from './consumer-react-aliases.js'
 
 describe('consumerReactAliases', () => {
   it('resolves react jsx runtimes from the consumer project root', () => {
-    const root = mkdtempSync(join(tmpdir(), 'wp-react-alias-'))
-    try {
-      mkdirSync(join(root, 'node_modules', 'react'), { recursive: true })
-      writeFileSync(
-        join(root, 'node_modules', 'react', 'package.json'),
-        JSON.stringify({
-          name: 'react',
-          exports: {
-            './jsx-dev-runtime': './jsx-dev-runtime.js',
-          },
-        }),
-      )
-      writeFileSync(join(root, 'node_modules', 'react', 'jsx-dev-runtime.js'), 'export {}')
-      writeFileSync(join(root, 'package.json'), JSON.stringify({ name: 'fixture', type: 'module' }))
-
-      const alias = consumerReactAliases(root)
-      assert.match(alias['react/jsx-dev-runtime'], /node_modules[/\\]react[/\\]jsx-dev-runtime\.js$/)
-    } finally {
-      rmSync(root, { recursive: true, force: true })
-    }
+    const alias = consumerReactAliases(consumerDepFixtureRoot)
+    assert.match(alias['react/jsx-dev-runtime'], /node_modules[/\\]react[/\\]jsx-dev-runtime/)
   })
 })

package/scripts/vite/resolve-from-project-root.js

@@ -1,15 +1,29 @@
-import { createRequire } from 'node:module'
 import { resolve, sep } from 'node:path'
-import { pathToFileURL } from 'node:url'
+
+import { resolveConsumerDep } from '../resolve-consumer-dep.mjs'
+
+/** Let Vite dedupe + optimizeDeps handle React; forced absolute paths serve CJS in the browser. */
+export const SKIP_CONSUMER_RESOLVE_SPECS = new Set([
+  'react',
+  'react-dom',
+  'react/jsx-runtime',
+  'react/jsx-dev-runtime',
+  'react-dom/client',
+  'react/compiler-runtime',
+])
+
+function shouldSkipConsumerResolve(source) {
+  if (SKIP_CONSUMER_RESOLVE_SPECS.has(source)) return true
+  if (source.startsWith('react/') || source.startsWith('react-dom/')) return true
+  return false
+}
 
 /**
  * When Vite transforms kit dev UI (`.wphs/`, package src, or dev-module-config),
  * bare imports must resolve from the consumer project's node_modules.
  */
 export default function resolveFromProjectRoot(projectRoot, kitRoot) {
-  const require = createRequire(pathToFileURL(resolve(projectRoot, 'package.json')))
   const kitRootNorm = resolve(kitRoot)
-  const wphsNorm = resolve(projectRoot, '.wphs')
 
   function isKitUiImporter(importer) {
     const norm = resolve(importer)
@@ -31,8 +45,11 @@ export default function resolveFromProjectRoot(projectRoot, kitRoot) {
       if (!isKitUiImporter(importer)) {
         return null
       }
+      if (shouldSkipConsumerResolve(source)) {
+        return null
+      }
       try {
-        return require.resolve(source)
+        return resolveConsumerDep(source, projectRoot)
       } catch {
         return null
       }

package/scripts/vite/resolve-from-project-root.test.mjs

@@ -0,0 +1,37 @@
+import assert from 'node:assert/strict'
+import { resolve } from 'node:path'
+import { describe, it } from 'node:test'
+
+import { consumerDepFixtureRoot } from '../test-fixtures/consumer-dep-fixture-root.mjs'
+import resolveFromProjectRoot, {
+  SKIP_CONSUMER_RESOLVE_SPECS,
+} from './resolve-from-project-root.js'
+
+describe('resolveFromProjectRoot', () => {
+  const kitRoot = resolve(consumerDepFixtureRoot, 'kit')
+  const kitUiFile = resolve(consumerDepFixtureRoot, '.wphs/.kit-module-config/ui/ModuleConfigNavContext.tsx')
+  const appFile = resolve(consumerDepFixtureRoot, 'src/app.tsx')
+
+  const plugin = resolveFromProjectRoot(consumerDepFixtureRoot, kitRoot)
+  const resolveId = /** @type {(source: string, importer: string) => Promise<string | null>} */ (
+    plugin.resolveId.bind(plugin)
+  )
+
+  it('does not force-resolve react for kit UI (avoids CJS index.js in browser)', async () => {
+    assert.ok(SKIP_CONSUMER_RESOLVE_SPECS.has('react'))
+    assert.equal(await resolveId('react', kitUiFile), null)
+  })
+
+  it('does not force-resolve react/jsx-dev-runtime for kit UI (alias + optimizeDeps)', async () => {
+    assert.equal(await resolveId('react/jsx-dev-runtime', kitUiFile), null)
+  })
+
+  it('still resolves other deps from the consumer for kit UI', async () => {
+    const resolved = await resolveId('vite', kitUiFile)
+    assert.match(resolved ?? '', /node_modules[/\\]vite[/\\]/)
+  })
+
+  it('ignores non-kit importers', async () => {
+    assert.equal(await resolveId('vite', appFile), null)
+  })
+})