@wular/pnext 0.0.15 → 0.0.16

package/bin/pnext

@@ -1,7 +1,9 @@
 #!/bin/sh
 
+# Presence only. The version floor is enforced in src/cli/index.ts, where
+# Bun.version costs nothing — asking `bun --version` here would fork per command.
 if ! command -v bun >/dev/null 2>&1; then
-  echo "pnext requires Bun. Install it from https://bun.sh/get" >&2
+  echo "pnext requires Bun >=1.3.6. Install it from https://bun.sh/get" >&2
   exit 1
 fi
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wular/pnext",
-  "version": "0.0.15",
+  "version": "0.0.16",
   "description": "A fast little framework for server-first React apps, fully compatible with Next.js",
   "type": "module",
   "bin": {
@@ -33,6 +33,9 @@
   "publishConfig": {
     "access": "public"
   },
+  "engines": {
+    "bun": ">=1.3.6"
+  },
   "exports": {
     ".": "./src/index.ts",
     "./internal": "./src/internal.ts",

package/reference/getting-started.md

@@ -2,7 +2,7 @@
 
 pnext is a Preact framework for file-routed apps. It renders on the server by default and ships client JavaScript only where a component opts in.
 
-> pnext requires [Bun](https://bun.sh/get) - make sure it is installed first.
+> pnext requires [Bun](https://bun.sh/get) 1.3.6 or newer - make sure it is installed first.
 
 <!-- tabs:start -->
 

package/src/cli/adapters/vercel.ts

@@ -381,9 +381,12 @@ export default Object.assign(handler, { fetch: handler });
     `${JSON.stringify(
       {
         // The Bun function runtime (project-level `bunVersion` does not apply
-        // to prebuilt Build Output API functions). Note: bun1.x functions have
-        // a 150 MiB uncompressed size limit vs 250 MiB for nodejs.
-        runtime: 'bun1.x',
+        // to prebuilt Build Output API functions, so the version is pinned
+        // here). Not `bun1.x`: that resolves to Bun 1.3.14, which segfaults a
+        // second into serving and takes the whole function down with SIGABRT.
+        // Note: bun functions have a 150 MiB uncompressed size limit vs 250
+        // MiB for nodejs.
+        runtime: 'bun1.4.x',
         handler: 'index.mjs',
         launcherType: 'Nodejs',
         architecture: FUNCTION_PLATFORM.architecture,

package/src/cli/index.ts

@@ -4,9 +4,29 @@ import path from 'node:path'
 import { markBoot } from './boot/trace'
 import { commandBinaryName, nameEsbuildProcess, namedBunBinary } from './boot/named-bin'
 
+const MIN_BUN = '1.3.6'
+// Not a floor but a hole in one: 1.3.14 segfaults about a second into serving,
+// so a pnext server dies before its first response with no error of its own to
+// show. 1.3.13 and 1.4.x are both fine.
+const BROKEN_BUN = new Set(['1.3.14'])
+
 // Reached only if something bypassed bin/pnext (which guards the same way).
 if (typeof Bun === 'undefined') {
-  console.error('pnext requires Bun. Install it from https://bun.sh/get')
+  console.error(`pnext requires Bun >=${MIN_BUN}. Install it from https://bun.sh/get`)
+  process.exit(1)
+}
+// A prerelease sorts below its own release, so compare the release: a 1.4.0-canary is a 1.4.0 here.
+const bunVersion = Bun.version.split('-')[0] ?? Bun.version
+if (BROKEN_BUN.has(bunVersion)) {
+  console.error(
+    `pnext does not support Bun ${Bun.version}: it crashes the process at runtime. ` +
+      'Upgrade to latest Bun with `bun upgrade`.',
+  )
+  process.exit(1)
+} else if (!Bun.semver.satisfies(bunVersion, `>=${MIN_BUN}`)) {
+  console.error(
+    `pnext requires Bun >=${MIN_BUN}, found ${Bun.version}. Upgrade with \`bun upgrade\`.`,
+  )
   process.exit(1)
 }
 

package/src/cli/serve/pipeline.ts

@@ -14,6 +14,7 @@ import { loadConfig, pathToFileHref } from '../../config'
 import type { ResolvedConfig } from '../../config'
 import { bootstrapCompat } from '../../compat-bootstrap'
 import { registerServerRuntime, serverBundleTargetForRuntime } from '../../runtime/loader'
+import { importModuleOnce } from '../../runtime/modules'
 import { applyProxyResponse, createProxyRunner } from '../../routing/proxy'
 import {
   isPprShellUpgradeEligible,
@@ -146,7 +147,7 @@ export async function createRequestHandler(
           const href = compatModuleMode(config)
             ? await moduleHrefForRoute(config, route)
             : pathToFileHref(route.file)
-          const module = (await import(href)) as Parameters<typeof handleRouteModule>[0]
+          const module = await importModuleOnce<Parameters<typeof handleRouteModule>[0]>(href)
           const rendered = await getRenderExtensions().collectRenderMeta(
             () =>
               withRouteRuntime(route.segmentConfig?.runtime, () =>
@@ -980,8 +981,9 @@ async function handleRoute(
       const href = compatModuleMode(config)
         ? await moduleHrefForRoute(config, route)
         : new URL(`file://${route.file}`).href
-      const imported = (await import(href)) as RouteHandlerModule &
-        Parameters<typeof metadataRouteHandlerModule>[0]
+      const imported = await importModuleOnce<
+        RouteHandlerModule & Parameters<typeof metadataRouteHandlerModule>[0]
+      >(href)
       return (metadataRouteHandlerModule(imported, route) ?? imported) as RouteHandlerModule
     })
     return spans.withRouteHandlerSpan(route.route, () =>

package/src/render/renderer.ts

@@ -26,7 +26,12 @@ import {
 import { clientEntryName } from '../client/chunk-name'
 import { pathToFileHref, type ResolvedConfig } from '../config'
 import { markErrorLogged } from '../utils/error-log'
-import { devClientModuleHref, devServerModuleHref, importDevModule } from '../runtime/modules'
+import {
+  devClientModuleHref,
+  devServerModuleHref,
+  importDevModule,
+  importModuleOnce,
+} from '../runtime/modules'
 import { serverBundleTargetForRuntime } from '../runtime/loader'
 import {
   getActionModuleExtensions,
@@ -663,8 +668,8 @@ function clientModuleHref(file: string, options: ModuleLoadOptions) {
 }
 
 async function importModule(file: string, options: ModuleLoadOptions) {
-  // In dev, re-throw a module's FIRST evaluation error on every later request
-  // rather than the downstream symptom a half-evaluated module produces.
+  // Re-throw a module's FIRST evaluation error on every later request rather
+  // than the downstream symptom a half-evaluated module produces.
   let module: Record<string, unknown>
   if (options.moduleLoader) module = await options.moduleLoader(file)
   else if (options.dev) {
@@ -672,7 +677,7 @@ async function importModule(file: string, options: ModuleLoadOptions) {
   } else {
     const href = await moduleHref(file, options)
     await drainPreplanBuilds()
-    module = (await import(href)) as Record<string, unknown>
+    module = await importModuleOnce<Record<string, unknown>>(href)
   }
   // Tag any 'use server' action module's exports with their wire id so
   // <form action={fn}> can progressively enhance. Compat-owned; a no-op for a
@@ -687,7 +692,7 @@ async function importClientModule(file: string, options: ModuleLoadOptions) {
   const href = await clientModuleHref(file, options)
   if (options.dev) return importDevModule<Record<string, unknown>>(href)
   await drainPreplanBuilds()
-  return import(href) as Promise<Record<string, unknown>>
+  return importModuleOnce<Record<string, unknown>>(href)
 }
 
 function clientScriptPath(route: RouteManifestEntry, dev?: boolean) {

package/src/runtime/modules.ts

@@ -185,21 +185,34 @@ const routeModuleLoaders = new Map<string, Promise<DevRouteModuleLoaders>>()
 // and the entry is never consulted again.
 const moduleEvalErrors = new Map<string, unknown>()
 
-/** Import a compiled dev module, re-throwing its first evaluation error on later imports. */
-export async function importDevModule<T>(href: string): Promise<T> {
+/**
+ * Import a compiled module, re-throwing its first evaluation error on later imports.
+ *
+ * Production needs this as much as dev: a re-import resolves with a half-evaluated namespace whose
+ * `export const` bindings are still in TDZ, so the render's next read (`module.metadata`) throws
+ * `Cannot access 'metadata' before initialization` and that replaces the real error in the response.
+ * Prod hrefs are content-addressed too, so the entry keys the same way dev's does.
+ */
+export async function importModuleOnce<T>(href: string): Promise<T> {
   if (moduleEvalErrors.has(href)) throw moduleEvalErrors.get(href)
-  // Pre-planned vendor builds are enqueued without an awaiter; nothing
-  // evaluates until they all settle (no-op when the pipeline is empty).
-  await drainPreplanBuilds()
   try {
-    const module = (await import(href)) as T
-    noteModuleImported(href)
-    return module
+    return (await import(href)) as T
   } catch (error) {
     moduleEvalErrors.set(href, error)
     throw error
   }
 }
+
+/** `importModuleOnce` plus dev's registry-generation accounting. */
+export async function importDevModule<T>(href: string): Promise<T> {
+  if (moduleEvalErrors.has(href)) throw moduleEvalErrors.get(href)
+  // Pre-planned vendor builds are enqueued without an awaiter; nothing
+  // evaluates until they all settle (no-op when the pipeline is empty).
+  await drainPreplanBuilds()
+  const module = await importModuleOnce<T>(href)
+  noteModuleImported(href)
+  return module
+}
 const extensionlessDynamicImportExtensions = new Set(['.ts', '.tsx', '.js', '.jsx', '.mts', '.cts'])
 const externalLoadNamespaces = {
   server: 'pnext-route-external-server',