@sproutboat/runtime 0.13.3 → 0.13.4

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # @sproutboat/runtime
 
+## 0.13.4
+
+### Patch Changes
+
+- 5bf4ea6: The handler bundle runs in its own function scope inside the generated module (part of baronunread/sproutboat#238). It used to be spliced into the prelude's top level, and Porffor resolves top-level names module-wide, so a bundle with its own top-level `const URL = ...` (uuid exports one) replaced the `URL` class the prelude extends and the binary died at startup with `Cannot get property of null`. Static imports from a direct-ESM handler stay at module top level.
+
 ## 0.13.3
 
 ### Patch Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sproutboat/runtime",
-  "version": "0.13.3",
+  "version": "0.13.4",
   "license": "MIT",
   "type": "module",
   "repository": {

package/src/wrap.ts

@@ -228,11 +228,21 @@ export function wrapNativeFetchHandler(
     ? `__sbStartLocalTriggers(__sbHandlers, ${JSON.stringify(bindings)});\n`
     : "";
 
+  // baronunread/sproutboat#238: the handler runs in its own function scope.
+  // Spliced in at the prelude's top level, a bundle's own top-level
+  // `const URL = ...` (uuid exports one) replaced the URL class for the whole
+  // module, prelude included, and the binary died at startup. Porffor resolves
+  // a top-level name module-wide, so a block is not enough; a function is.
+  // Static imports (a direct-ESM handler, no bundler in front) are only legal
+  // at module top level, so they stay outside the function.
+  const importPattern = /^import\s(?:[\s\S]*?\sfrom\s*)?["'][^"'\n]+["'];?[ \t]*$/gm;
+  const imports = (neutralised.match(importPattern) ?? []).join("\n");
+  const body = neutralised.replace(importPattern, "");
   return (
     `${prelude}\n${nativeTransferAbi}${compat}${env}${versionMetadataLine}${wire}` +
-    `${neutralised}\n` +
-    `${registerDO}${triggers}` +
-    `export default {\n  port: ${port},\n  fetch(request) { return __sbEntry(__sbHandlers, request); }\n};\n`
+    `${imports ? `${imports}\n` : ""}const __sbUserHandlers = (() => {\n${body}\n` +
+    `${registerDO}${triggers}return __sbHandlers;\n})();\n` +
+    `export default {\n  port: ${port},\n  fetch(request) { return __sbEntry(__sbUserHandlers, request); }\n};\n`
   );
 }
 

package/test/native-utf8.ts

@@ -12,7 +12,10 @@ import { EMPTY_BINDINGS, preludePath, TRANSPORT_MARKER, transportPath, wrapNativ
 // two hot paths (KV, D1) and a non-ASCII SQLite error message.
 const text = "/café/日本/😀/x";
 
+// #238: the handler's own top-level \`URL\` (uuid exports one) must not replace
+// the URL class the prelude extends; the handler runs in its own scope.
 const handler = `
+const URL = "shadowed";
 export default {
   async fetch() {
     await env.KV.put("k", ${JSON.stringify(text)});
@@ -22,7 +25,7 @@ export default {
     const d1 = (await env.DB.prepare("SELECT v FROM t").first()).v;
     let error = "";
     try { await env.DB.prepare("SELECT * FROM tabellà").first(); } catch (e) { error = String(e.message); }
-    return Response.json({ kv, kvLength: kv.length, d1, d1Length: d1.length, error });
+    return Response.json({ kv, kvLength: kv.length, d1, d1Length: d1.length, error, url: URL });
   },
 };
 `;
@@ -96,13 +99,21 @@ try {
   assert.equal(response.status, 200);
   // SAFETY: the handler above is the only producer of this body and always
   // returns exactly these five fields; deepEqual below checks every one.
-  const body = (await response.json()) as { kv: string; kvLength: number; d1: string; d1Length: number; error: string };
+  const body = (await response.json()) as {
+    kv: string;
+    kvLength: number;
+    d1: string;
+    d1Length: number;
+    error: string;
+    url: string;
+  };
   assert.deepEqual(
     { kv: body.kv, kvLength: body.kvLength, d1: body.d1, d1Length: body.d1Length },
     { kv: text, kvLength: text.length, d1: text, d1Length: text.length },
   );
   assert.match(body.error, /tabellà/);
-  console.log("Non-ASCII KV, D1 and error text round-trip through the broker");
+  assert.equal(body.url, "shadowed");
+  console.log("Non-ASCII KV, D1 and error text round-trip through the broker; a handler-level URL stays scoped");
 } finally {
   if (server) {
     server.kill(9);