fix: absolute URLs for the Workers ASSETS binding

env.ASSETS.fetch('/path') throws 'Invalid URL' in the Workers runtime;
the binding requires a full URL (host is ignored, routing is
path-only). Test doubles extract the basename instead of assuming the
path prefix.

Author: ronaldtse

dist/src/index.js

@@ -10,7 +10,7 @@
  */
 import { type CompiledMap, type SystemCode } from "interscript-ts";
 export interface MapAssets {
-    fetch(url: string): Promise<Response>;
+    fetch(input: RequestInfo): Promise<Response>;
 }
 export declare function bundledSystemCodes(): readonly SystemCode[];
 export declare function detectableSystemCodes(): readonly SystemCode[];

dist/src/maps.d.ts

@@ -31,7 +31,9 @@ export async function loadMap(assets, code) {
     const cached = cache.get(code);
     if (cached)
         return cached;
-    const response = await assets.fetch(`/maps/${code}.json`);
+    // The Workers ASSETS binding requires an absolute URL; the host is
+    // ignored (routing is path-only within the binding).
+    const response = await assets.fetch(new Request(`https://assets.internal/maps/${code}.json`));
     if (!response.ok) {
         throw new MapNotFoundError(code);
     }

dist/src/maps.js

@@ -9,7 +9,9 @@ import { app } from "../src/server.js";
 const env = {
     ASSETS: {
         fetch: async (url) => {
-            const code = url.replace("/maps/", "").replace(".json", "");
+            const path = typeof url === "string" ? url : url.url;
+            const name = path.substring(path.lastIndexOf("/") + 1);
+            const code = name.replace(".json", "");
             try {
                 return new Response(readFileSync(`maps/${code}.json`), { status: 200 });
             }

dist/test/parity.test.js

@@ -10,7 +10,9 @@ import { app } from "../src/server.js";
 // deployer ships as static assets, served from the local filesystem.
 const assets = {
     fetch: async (url) => {
-        const code = url.replace("/maps/", "").replace(".json", "");
+        const path = typeof url === "string" ? url : url.url;
+        const name = path.substring(path.lastIndexOf("/") + 1);
+        const code = name.replace(".json", "");
         try {
             const body = readFileSync(`maps/${code}.json`);
             return new Response(body, { status: 200 });

dist/test/server.test.js

@@ -29,7 +29,7 @@ const addressable = systemCodes.filter((code) => !LIBRARIES.has(code))
 const detectable = addressable.filter((code) => !ML_BACKED.has(code))
 
 export interface MapAssets {
-  fetch(url: string): Promise<Response>
+  fetch(input: RequestInfo): Promise<Response>
 }
 
 const cache = new Map<SystemCode, CompiledMap>()
@@ -45,7 +45,11 @@ export function detectableSystemCodes(): readonly SystemCode[] {
 export async function loadMap(assets: MapAssets, code: SystemCode): Promise<CompiledMap> {
   const cached = cache.get(code)
   if (cached) return cached
-  const response = await assets.fetch(`/maps/${code}.json`)
+  // The Workers ASSETS binding requires an absolute URL; the host is
+  // ignored (routing is path-only within the binding).
+  const response = await assets.fetch(
+    new Request(`https://assets.internal/maps/${code}.json`),
+  )
   if (!response.ok) {
     throw new MapNotFoundError(code)
   }

src/maps.ts

@@ -10,8 +10,10 @@ import type { Env } from "../src/resolvers.js"
 
 const env: Env = {
   ASSETS: {
-    fetch: async (url: string): Promise<Response> => {
-      const code = url.replace("/maps/", "").replace(".json", "")
+    fetch: async (url: RequestInfo): Promise<Response> => {
+      const path = typeof url === "string" ? url : url.url
+      const name = path.substring(path.lastIndexOf("/") + 1)
+      const code = name.replace(".json", "")
       try {
         return new Response(readFileSync(`maps/${code}.json`), { status: 200 })
       } catch {

test/parity.test.ts

@@ -10,8 +10,10 @@ import { app } from "../src/server.js"
 // Test double of the Workers ASSETS binding: the same maps directory the
 // deployer ships as static assets, served from the local filesystem.
 const assets = {
-  fetch: async (url: string): Promise<Response> => {
-    const code = url.replace("/maps/", "").replace(".json", "")
+  fetch: async (url: RequestInfo): Promise<Response> => {
+    const path = typeof url === "string" ? url : url.url
+    const name = path.substring(path.lastIndexOf("/") + 1)
+    const code = name.replace(".json", "")
     try {
       const body = readFileSync(`maps/${code}.json`)
       return new Response(body, { status: 200 })