@solidjs/start 0.5.0 → 0.5.2

package/config/fs-router.js

@@ -24,7 +24,6 @@ export class SolidStartClientFileRouter extends BaseFileSystemRouter {
 
     if (src.endsWith(".md") || src.endsWith(".mdx")) {
       return {
-        type: "page",
         $component: {
           src: src,
           pick: ["$css"]
@@ -40,7 +39,6 @@ export class SolidStartClientFileRouter extends BaseFileSystemRouter {
     const hasRouteConfig = exports.find(e => e.n === "route");
     if (hasDefault) {
       return {
-        type: "page",
         $component: {
           src: src,
           pick: ["default", "$css"]
@@ -58,6 +56,20 @@ export class SolidStartClientFileRouter extends BaseFileSystemRouter {
   }
 }
 
+const HTTP_METHODS = ["GET", "POST", "PUT", "DELETE", "PATCH"];
+function createHTTPHandlers(src, exports) {
+  const handlers = {};
+  for (const exp of exports) {
+    if (HTTP_METHODS.includes(exp.n)) {
+      handlers[`$${exp.n}`] = {
+        src: src,
+        pick: [exp.n]
+      };
+    }
+  }
+  return handlers;
+}
+
 export class SolidStartServerFileRouter extends BaseFileSystemRouter {
   toPath(src) {
     const routePath = cleanPath(src, this.config)
@@ -79,10 +91,8 @@ export class SolidStartServerFileRouter extends BaseFileSystemRouter {
 
   toRoute(src) {
     let path = this.toPath(src);
-
     if (src.endsWith(".md") || src.endsWith(".mdx")) {
       return {
-        type: "page",
         $component: {
           src: src,
           pick: ["$css"]
@@ -95,62 +105,22 @@ export class SolidStartServerFileRouter extends BaseFileSystemRouter {
 
     const [_, exports] = analyzeModule(src);
     const hasRouteConfig = exports.find(e => e.n === "route");
-    if (exports.find(exp => exp.n === "default")) {
+    const hasDefault = exports.find(e => e.n === "default");
+    const hasAPIRoutes = exports.find(exp => HTTP_METHODS.includes(exp.n));
+    if (hasDefault || hasAPIRoutes) {
       return {
-        type: "page",
-        $component: {
-          src: src,
-          pick: ["default", "$css"]
-        },
+        $component: hasDefault
+          ? {
+              src: src,
+              pick: ["default", "$css"]
+            }
+          : undefined,
         $$route: hasRouteConfig
           ? {
               src: src,
               pick: ["route"]
             }
           : undefined,
-        path,
-        filePath: src
-      };
-    }
-  }
-}
-
-const HTTP_METHODS = ["GET", "POST", "PUT", "DELETE", "PATCH"];
-function createHTTPHandlers(src, exports) {
-  const handlers = {};
-  for (const exp of exports) {
-    if (HTTP_METHODS.includes(exp.n)) {
-      handlers[`$${exp.n}`] = {
-        src: src,
-        pick: [exp.n]
-      };
-    }
-  }
-  return handlers;
-}
-
-export class SolidStartAPIFileRouter extends BaseFileSystemRouter {
-  toPath(src) {
-    const routePath = cleanPath(src, this.config)
-      // remove the initial slash
-      .slice(1)
-      .replace(/index$/, "")
-      .replace(/\[([^\/]+)\]/g, (_, m) => {
-        if (m.length > 2 && m.startsWith("...")) {
-          return `**${m.length > 3 ? ":" + m.slice(3) : ""}`;
-        }
-        return `:${m}`;
-      });
-
-    return routePath?.length > 0 ? `/${routePath}` : "/";
-  }
-
-  toRoute(src) {
-    let path = this.toPath(src);
-    const [_, exports] = analyzeModule(src);
-    if (exports.find(exp => HTTP_METHODS.includes(exp.n))) {
-      return {
-        type: "api",
         ...createHTTPHandlers(src, exports),
         path,
         filePath: src

package/config/index.js

@@ -8,7 +8,10 @@ import { createApp, resolve } from "vinxi";
 import { normalize } from "vinxi/lib/path";
 import { config } from "vinxi/plugins/config";
 import solid from "vite-plugin-solid";
-import { SolidStartAPIFileRouter, SolidStartClientFileRouter, SolidStartServerFileRouter } from "./fs-router.js";
+import {
+  SolidStartClientFileRouter,
+  SolidStartServerFileRouter
+} from "./fs-router.js";
 import { serverComponents } from "./server-components.js";
 
 const DEFAULT_EXTENSIONS = ["js", "jsx", "ts", "tsx"];
@@ -37,18 +40,6 @@ function solidStartServerFsRouter(config) {
     );
 }
 
-function solidStartAPIFileRouter(config) {
-  return (router, app) =>
-    new SolidStartAPIFileRouter(
-      {
-        dir: resolve.absolute(config.dir, router.root),
-        extensions: config.extensions ?? DEFAULT_EXTENSIONS
-      },
-      router,
-      app
-    );
-}
-
 export function defineConfig(baseConfig = {}) {
   let { plugins = [], start = {}, ...userConfig } = baseConfig;
   const extensions = [...DEFAULT_EXTENSIONS, ...(start.extensions || [])];
@@ -86,15 +77,6 @@ export function defineConfig(baseConfig = {}) {
         dir: "./public",
         base: "/"
       },
-      {
-        name: "api",
-        type: "http",
-        target: "server",
-        base: "/",
-        handler: normalize(fileURLToPath(new URL("./api-handler.ts", import.meta.url))),
-        routes: solidStartAPIFileRouter({ dir: `${start.appRoot}/routes` }),
-        middleware: start.middleware
-      },
       {
         name: "ssr",
         type: "http",
@@ -209,7 +191,7 @@ export function defineConfig(baseConfig = {}) {
       serverFunctions.router({
         handler: normalize(fileURLToPath(new URL("./server-handler.ts", import.meta.url))),
         runtime: normalize(fileURLToPath(new URL("./server-fns-runtime.ts", import.meta.url))),
-        // routes: solidStartServerFsRouter({ dir: `${start.appRoot}/routes`, extensions }),
+        routes: solidStartServerFsRouter({ dir: `${start.appRoot}/routes`, extensions }),
         plugins: async () => [
           config("user", {
             ...userConfig,

package/config/server-fns-runtime.ts

@@ -1,5 +1,4 @@
 import { getRequestEvent } from "solid-js/web";
-//@ts-ignore
 import { provideRequestEvent } from "solid-js/web/storage";
 import { cloneEvent } from "../server/fetchEvent";
 
@@ -9,7 +8,7 @@ export function createServerReference(fn, id, name) {
   return new Proxy(fn, {
     get(target, prop, receiver) {
       if (prop === "url") {
-        return `${baseURL}/_server?id=${encodeURIComponent(id)}&name=${encodeURIComponent(name)}`;
+        return `${baseURL}/_server/?id=${encodeURIComponent(id)}&name=${encodeURIComponent(name)}`;
       }
       if (prop === "GET") return receiver;
     },

package/config/server-handler.ts

@@ -1,5 +1,6 @@
 /// <reference types="vinxi/types/server" />
 import { crossSerializeStream, fromJSON, getCrossReferenceHeader } from "seroval";
+// @ts-ignore
 import {
   CustomEventPlugin,
   DOMExceptionPlugin,
@@ -13,11 +14,15 @@ import {
   URLSearchParamsPlugin
 } from "seroval-plugins/web";
 import { sharedConfig } from "solid-js";
-/* @ts-ignore */
+import { renderToStringAsync } from "solid-js/web";
 import { provideRequestEvent } from "solid-js/web/storage";
+import { eventHandler, setHeader, setResponseStatus } from "vinxi/http";
 import invariant from "vinxi/lib/invariant";
-import { eventHandler, setHeader } from "vinxi/server";
-import { getFetchEvent, mergeResponseHeaders } from "../server/fetchEvent";
+import { cloneEvent, getFetchEvent, mergeResponseHeaders } from "../server/fetchEvent";
+import { createPageEvent } from "../server/pageEvent";
+// @ts-ignore
+import App from "#start/app";
+import { FetchEvent, PageEvent } from "../server";
 
 function createChunk(data) {
   const bytes = data.length;
@@ -63,8 +68,9 @@ async function handleServerFunction(h3Event) {
   const event = getFetchEvent(h3Event);
   const request = event.request;
 
-  const serverReference = request.headers.get("x-server-id");
-  const instance = request.headers.get("x-server-instance");
+  const serverReference = request.headers.get("X-Server-Id");
+  const instance = request.headers.get("X-Server-Instance");
+  const singleFlight = request.headers.has("X-Single-Flight");
   const url = new URL(request.url);
   let filepath, name;
   if (serverReference) {
@@ -76,7 +82,7 @@ async function handleServerFunction(h3Event) {
     if (!filepath || !name) throw new Error("Invalid request");
   }
 
-  const action = (
+  const serverFunction = (
     await import.meta.env.MANIFEST[import.meta.env.ROUTER_NAME].chunks[filepath].import()
   )[name];
   let parsed = [];
@@ -94,13 +100,15 @@ async function handleServerFunction(h3Event) {
     ) {
       // workaround for https://github.com/unjs/nitro/issues/1721
       // (issue only in edge runtimes)
-      parsed.push(await new Request(request, { ...request, body: h3Event.node.req.body }).formData());
-      // what should work when #1721 is fixed 
+      parsed.push(
+        await new Request(request, { ...request, body: h3Event.node.req.body }).formData()
+      );
+      // what should work when #1721 is fixed
       // parsed.push(await request.formData);
     } else {
       // workaround for https://github.com/unjs/nitro/issues/1721
       // (issue only in edge runtimes)
-      const tmpReq = new Request(request, { ...request, body: h3Event.node.req.body })
+      const tmpReq = new Request(request, { ...request, body: h3Event.node.req.body });
       // what should work when #1721 is fixed
       // just use request.json() here
       parsed = fromJSON(await tmpReq.json(), {
@@ -120,27 +128,23 @@ async function handleServerFunction(h3Event) {
     }
   }
   try {
-    let result = await provideRequestEvent(event, () => {
+    let result = await provideRequestEvent(event, async () => {
       /* @ts-ignore */
       sharedConfig.context = { event };
-      return action(...parsed);
+      return serverFunction(...parsed);
     });
 
+    if (singleFlight && instance) {
+      result = await handleSingleFlight(event, result);
+    }
+
     // handle responses
-    if (result instanceof Response) {
-      if (result.status === 302) {
-        return new Response(null, {
-          status: instance ? 204 : 302,
-          headers: {
-            Location: result.headers.get("Location")
-          }
-        });
-      }
+    if (result instanceof Response && instance) {
       // forward headers
       if (result.headers) mergeResponseHeaders(h3Event, result.headers);
       if ((result as any).customBody) {
         result = await (result as any).customBody();
-      } else if (result.body == undefined) result = undefined;
+      } else if (result.body == undefined) result = null;
     }
 
     // handle no JS success case
@@ -164,27 +168,59 @@ async function handleServerFunction(h3Event) {
         }
       });
     }
-    if (typeof result === "string") return new Response(result);
+    if (!result || typeof result === "string") return result || null;
     setHeader(h3Event, "content-type", "text/javascript");
     return serializeToStream(instance, result);
   } catch (x) {
     if (x instanceof Response) {
-      if (x.status === 302) {
-        return new Response(null, {
-          status: instance ? 204 : 302,
-          headers: {
-            Location: x.headers.get("Location")
-          }
-        });
-      }
+      if (x.status === 302 && !instance) setResponseStatus(h3Event, 302);
       // forward headers
       if (x.headers) mergeResponseHeaders(h3Event, x.headers);
       if ((x as any).customBody) {
         x = await (x as any).customBody();
-      } else if (x.body == undefined) x = undefined;
+      } else if (x.body == undefined) x = null;
     }
     return x;
   }
 }
 
+async function handleSingleFlight(sourceEvent: FetchEvent, result: any) {
+  let revalidate;
+  let url = new URL(sourceEvent.request.headers.get("referer")).toString();
+  if (result instanceof Response) {
+    if (result.headers.has("X-Revalidate"))
+      revalidate = result.headers.get("X-Revalidate").split(",");
+    if (result.headers.has("Location")) url = result.headers.get("Location");
+  }
+  const event = cloneEvent(sourceEvent) as PageEvent;
+  event.request = new Request(url);
+  return await provideRequestEvent(event, async () => {
+    await createPageEvent(event);
+    /* @ts-ignore */
+    sharedConfig.context = { event };
+    /* @ts-ignore */
+    event.router.dataOnly = revalidate || true;
+    /* @ts-ignore */
+    event.router.previousUrl = sourceEvent.request.headers.get("referer");
+    renderToStringAsync(App);
+    /* @ts-ignore */
+    const body = event.router.data;
+    if (body) {
+      let containsKey = false;
+      for (const key in body) {
+        if (body[key] === undefined) delete body[key];
+        else containsKey = true;
+      }
+      if (!containsKey) return result;
+      if (!(result instanceof Response)) {
+        body["_$value"] = result;
+        result = new Response(null, { status: 200 });
+      }
+      result.customBody = () => body;
+      result.headers.set("X-Single-Flight", "true");
+      return result;
+    }
+  });
+}
+
 export default eventHandler(handleServerFunction);

package/config/server-runtime.ts

@@ -109,33 +109,32 @@ async function deserializeStream(id, response) {
 
 let INSTANCE = 0;
 
-function createRequest(base: string, id: string, instance: string, body: any, contentType?: string) {
+function createRequest(base: string, id: string, instance: string, options: RequestInit) {
   return fetch(base, {
     method: "POST",
+    ...options,
     headers: {
-      "x-server-id": id,
-      "x-server-instance": instance,
-      ...(contentType ? { "Content-Type": contentType } : {})
-    },
-    body
+      ...options.headers,
+      "X-Server-Id": id,
+      "X-Server-Instance": instance
+    }
   });
 }
 
-async function fetchServerFunction(base, id, method, args) {
+async function fetchServerFunction(
+  base: string,
+  id: string,
+  options: Omit<RequestInit, "body">,
+  args: any[]
+) {
   const instance = `server-fn:${INSTANCE++}`;
-  const response = await (method === "GET"
-    ? fetch(base + (args.length ? `&args=${JSON.stringify(args)}` : ""), {
-        headers: {
-          "x-server-instance": instance
-        }
-      })
+  const response = await (args.length === 0
+    ? createRequest(base, id, instance, options)
     : args.length === 1 && args[0] instanceof FormData
-    ? createRequest(base, id, instance, args[0])
-    : createRequest(
-        base,
-        id,
-        instance,
-        JSON.stringify(
+    ? createRequest(base, id, instance, { ...options, body: args[0] })
+    : createRequest(base, id, instance, {
+        ...options,
+        body: JSON.stringify(
           await Promise.resolve(
             toJSONAsync(args, {
               plugins: [
@@ -153,17 +152,23 @@ async function fetchServerFunction(base, id, method, args) {
             })
           )
         ),
-        "application/json"
-      ));
-
-  if (response.headers.get("Location")) throw response;
-  if (response.headers.get("X-Revalidate")) {
-    /* @ts-ignore-next-line */
-    response.customBody = () => {
-      return deserializeStream(instance, response);
-    };
-    throw response;
+        headers: { ...options.headers, "Content-Type": "application/json" }
+      }));
+
+  if (
+    response.headers.get("Location") ||
+    response.headers.get("X-Revalidate") ||
+    response.headers.get("X-Single-Flight")
+  ) {
+    if (response.body) {
+      /* @ts-ignore-next-line */
+      response.customBody = () => {
+        return deserializeStream(instance, response);
+      };
+    }
+    return response;
   }
+
   const contentType = response.headers.get("Content-Type");
   let result;
   if (contentType && contentType.startsWith("text/plain")) {
@@ -184,14 +189,31 @@ export function createServerReference(fn, id, name) {
   return new Proxy(fn, {
     get(target, prop, receiver) {
       if (prop === "url") {
-        return `${baseURL}/_server?id=${encodeURIComponent(id)}&name=${encodeURIComponent(name)}`;
+        return `${baseURL}/_server/?id=${encodeURIComponent(id)}&name=${encodeURIComponent(name)}`;
       }
       if (prop === "GET") {
-        return (...args) => fetchServerFunction(`${baseURL}/_server/?id=${encodeURIComponent(id)}&name=${encodeURIComponent(name)}`, `${id}#${name}`, "GET", args);
+        return receiver.withOptions({ method: "GET" });
+      }
+      if (prop === "withOptions") {
+        return (options: RequestInit) => {
+          const fn = (...args) => {
+            const encodeArgs = options.method && options.method.toUpperCase() === "GET";
+            return fetchServerFunction(
+              encodeArgs
+                ? receiver.url + (args.length ? `&args=${JSON.stringify(args)}` : "")
+                : `${baseURL}/_server`,
+              `${id}#${name}`,
+              options,
+              encodeArgs ? [] : args
+            );
+          };
+          fn.url = receiver.url;
+          return fn;
+        };
       }
     },
     apply(target, thisArg, args) {
-      return fetchServerFunction(`${baseURL}/_server`, `${id}#${name}`, "POST", args);
+      return fetchServerFunction(`${baseURL}/_server`, `${id}#${name}`, {}, args);
     }
   });
 }

package/middleware/index.ts

@@ -1,8 +1,8 @@
 import {
-  H3Event,
-  defineMiddleware,
-  sendWebResponse
-} from "vinxi/server";
+    H3Event,
+    defineMiddleware,
+    sendWebResponse
+} from "vinxi/http";
 import { getFetchEvent } from "../server/fetchEvent";
 import type { FetchEvent } from "../server/types";
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@solidjs/start",
-  "version": "0.5.0",
+  "version": "0.5.2",
   "type": "module",
   "author": "Ryan Carniato",
   "publishConfig": {
@@ -42,7 +42,7 @@
     }
   },
   "devDependencies": {
-    "solid-js": "^1.8.12",
+    "solid-js": "^1.8.14",
     "vinxi": "^0.2.1"
   },
   "dependencies": {

package/server/fetchEvent.ts

@@ -12,7 +12,7 @@ import {
   setResponseHeader,
   setResponseStatus,
   toWebRequest
-} from "vinxi/server";
+} from "vinxi/http";
 import type { FetchEvent, ResponseStub } from "./types";
 
 const fetchEventSymbol = Symbol("fetchEvent");
@@ -40,7 +40,6 @@ export function getFetchEvent(h3Event: H3Event): FetchEvent {
   if (!h3Event[fetchEventSymbol]) {
     const fetchEvent = createFetchEvent(h3Event);
     h3Event[fetchEventSymbol] = fetchEvent;
-    // @ts-ignore
   }
 
   return h3Event[fetchEventSymbol];
@@ -75,21 +74,21 @@ class HeaderProxy {
     return Array.isArray(cookies) ? cookies : [cookies];
   }
   forEach(fn: (value: string, key: string, object: Headers) => void) {
-    return this.entries().forEach(([key, value]) => fn(value, key, this));
+    return Object.entries<string | string[]>(getResponseHeaders(this.event)).forEach(([key, value]) => fn(Array.isArray(value) ? value.join(", ") : value, key, this));
   }
   entries() {
-    return getResponseHeaders(this.event).map(([key, value]) => [
+    return Object.entries<string | string[]>(getResponseHeaders(this.event)).map(([key, value]) => [
       key,
       Array.isArray(value) ? value.join(", ") : value
-    ]);
+    ] as [string, string])[Symbol.iterator]();
   }
   keys() {
-    return getResponseHeaders(this.event).map(([key]) => key);
+    return Object.keys(getResponseHeaders(this.event))[Symbol.iterator]();
   }
   values() {
-    return getResponseHeaders(this.event).map(([key, value]) =>
+    return Object.values<string | string[]>(getResponseHeaders(this.event)).map((value) =>
       Array.isArray(value) ? value.join(", ") : value
-    );
+    )[Symbol.iterator]();
   }
   [Symbol.iterator]() {
     return this.entries()[Symbol.iterator]();

package/server/handler.ts

@@ -1,6 +1,5 @@
 import { sharedConfig } from "solid-js";
 import { renderToStream, renderToString } from "solid-js/web";
-/* @ts-ignore */
 import { provideRequestEvent } from "solid-js/web/storage";
 import {
   EventHandlerObject,
@@ -10,10 +9,11 @@ import {
   sendRedirect,
   setHeader,
   setResponseStatus
-} from "vinxi/server";
+} from "vinxi/http";
+import { matchAPIRoute } from "../shared/routes";
 import { getFetchEvent } from "./fetchEvent";
 import { createPageEvent } from "./pageEvent";
-import { FetchEvent, PageEvent } from "./types";
+import type { APIEvent, FetchEvent, PageEvent } from "./types";
 
 export function createBaseHandler(
   fn: (context: PageEvent) => unknown,
@@ -36,6 +36,23 @@ export function createBaseHandler(
       const mode = import.meta.env.START_SSR;
 
       return provideRequestEvent(event, async () => {
+        // api
+        const match = matchAPIRoute(new URL(event.request.url).pathname, event.request.method);
+        if (match) {
+          const mod = await match.handler.import();
+          const fn = mod[event.request.method];
+          (event as APIEvent).params = match.params;
+          // @ts-ignore
+          sharedConfig.context = { event };
+          const res = await fn(event);
+          if (res !== undefined) return res;
+          if (event.request.method !== "GET") {
+            throw new Error(
+              `API handler for ${event.request.method} "${event.request.url}" did not return a response.`
+            );
+          }
+        }
+
         // render
         const context = await createPageEvent(event);
         if (mode === "sync") {
@@ -65,7 +82,7 @@ export function createBaseHandler(
         } else cloned.onCompleteShell = handleShellCompleteRedirect(context, e);
         const stream = renderToStream(() => {
           (sharedConfig.context as any).event = context;
-          return fn(context)
+          return fn(context);
         }, cloned);
         if (context.response && context.response.headers.get("Location")) {
           return sendRedirect(event, context.response.headers.get("Location"));
@@ -86,7 +103,7 @@ function handleShellCompleteRedirect(context: PageEvent, e: H3Event<EventHandler
       setResponseStatus(e, 302);
       setHeader(e, "Location", context.response.headers.get("Location"));
     }
-  }
+  };
 }
 
 function handleStreamCompleteRedirect(context: PageEvent) {
@@ -101,4 +118,4 @@ export function createHandler(
   options: Parameters<typeof createBaseHandler>[1] = {}
 ) {
   return createBaseHandler(fn, { ...options, createPageEvent });
-}
+}

package/server/pageEvent.ts

@@ -1,7 +1,7 @@
 import {
-  getCookie,
-  setCookie
-} from "vinxi/server";
+    getCookie,
+    setCookie
+} from "vinxi/http";
 import { createRoutes } from "../shared/FileRoutes";
 import { FetchEvent, PageEvent } from "./types";
 

package/server/types.ts

@@ -1,5 +1,5 @@
 import type { JSX } from "solid-js";
-import { HTTPEventSymbol, type EventHandlerRequest, type H3Event } from "vinxi/server";
+import { HTTPEventSymbol, type EventHandlerRequest, type H3Event } from "vinxi/http";
 
 // export const FETCH_EVENT = "$FETCH";
 

package/shared/ErrorBoundary.tsx

@@ -1,24 +1,13 @@
-import {
-  ErrorBoundary as DefaultErrorBoundary,
-  ParentProps,
-} from "solid-js";
+import { ErrorBoundary as DefaultErrorBoundary, type ParentProps } from "solid-js";
 import { HttpStatusCode } from "./HttpStatusCode";
 import { DevOverlay } from "./dev-overlay";
 
 export function ErrorBoundary(props: ParentProps) {
   if (import.meta.env.DEV) {
-    return (
-      <DevOverlay>
-        {props.children}
-      </DevOverlay>
-    );
+    return <DevOverlay>{props.children}</DevOverlay>;
   }
   return (
-    <DefaultErrorBoundary
-      fallback={(
-        <HttpStatusCode code={500} />
-      )}
-    >
+    <DefaultErrorBoundary fallback={<HttpStatusCode code={500} />}>
       {props.children}
     </DefaultErrorBoundary>
   );

package/shared/routes.ts

@@ -1,23 +1,25 @@
+import { createRouter } from "radix3";
 import fileRoutes from "vinxi/routes";
 
 interface Route {
   path: string;
   id: string;
-  type: "page";
   children?: Route[];
+  $component?: any;
 }
 
 declare module "vinxi/routes" {
   export interface Register {
     route: {
       path: string;
-      type: "api" | "page";
       children?: Route[];
     };
   }
 }
 
-export const pageRoutes = defineRoutes((fileRoutes as unknown as Route[]).filter(o => o.type === "page"));
+export const pageRoutes = defineRoutes(
+  (fileRoutes as unknown as Route[]).filter(o => o.$component)
+);
 
 function defineRoutes(fileRoutes: Route[]) {
   function processRoute(routes: Route[], route: Route, id: string, full: string) {
@@ -46,3 +48,35 @@ function defineRoutes(fileRoutes: Route[]) {
     }, []);
 }
 
+export function matchAPIRoute(path: string, method: string) {
+  const match = router.lookup(path);
+  if (match) {
+    const handler = match.route[`$${method}`];
+    if (handler === undefined) return;
+    return {
+      handler,
+      params: match.params
+    };
+  }
+}
+
+function containsHTTP(route: Route) {
+  return route["$GET"] || route["$POST"] || route["$PUT"] || route["$PATCH"] || route["$DELETE"];
+}
+
+const router = createRouter({
+  routes: (fileRoutes as unknown as Route[]).reduce((memo, route) => {
+    if (!containsHTTP(route)) return memo;
+    let path = route.path.replace(/\(.*\)\/?/g, "").replace(/\*([^/]*)/g, (_, m) => `**:${m}`);
+    if (/:[^/]*\?/g.test(path)) {
+      throw new Error(`Optional parameters are not supported in API routes: ${path}`);
+    }
+    if (memo[path]) {
+      throw new Error(
+        `Duplicate API routes for "${path}" found at "${memo[path].route.path}" and "${route.path}"`
+      );
+    }
+    memo[path] = { route };
+    return memo;
+  }, {} as Record<string, { route: Route }>)
+});

package/config/api-handler.ts

@@ -1,64 +0,0 @@
-/// <reference types="vinxi/types/server" />
-import { createRouter } from "radix3";
-import { sharedConfig } from "solid-js";
-import fileRoutes from "vinxi/routes";
-
-// @ts-ignore
-import { provideRequestEvent } from "solid-js/web/storage";
-import { eventHandler } from "vinxi/server";
-import { getFetchEvent } from "../server/fetchEvent";
-import { APIEvent } from "../server/types";
-
-export default eventHandler(h3Event => {
-  const event = getFetchEvent(h3Event);
-  const match = matchAPIRoute(new URL(event.request.url).pathname, event.request.method);
-  if (match) {
-    return provideRequestEvent(event, async () => {
-      const mod = await match.handler.import();
-      const fn = mod[event.request.method];
-      (event as APIEvent).params = match.params;
-      // @ts-ignore
-      sharedConfig.context = { event };
-      const res = await fn(event);
-      if (res === undefined && event.request.method !== "GET") {
-        throw new Error(
-          `API handler for ${event.request.method} "${event.request.url}" did not return a response.`
-        );
-      }
-      return res;
-    });
-  }
-});
-
-interface Route {
-  path: string;
-  id: string;
-  type: "api" | "page";
-  children?: Route[];
-}
-
-const router = createRouter({
-  routes: (fileRoutes as unknown as Route[]).reduce((memo, route) => {
-    if (route.type !== "api") return memo;
-    const path = route.path.replace(/\(.*\)\/?/g, "");
-    if (memo[path]) {
-      throw new Error(
-        `Duplicate API routes for "${path}" found at "${memo[path].route.path}" and "${route.path}"`
-      );
-    }
-    memo[path] = { route };
-    return memo;
-  }, {} as Record<string, { route: Route }>)
-});
-
-function matchAPIRoute(path: string, method: string) {
-  const match = router.lookup(path);
-  if (match) {
-    const handler = match.route[`$${method}`];
-    if (handler === "skip" || handler === undefined) return;
-    return {
-      handler,
-      params: match.params
-    };
-  }
-}