@marko/run 0.0.1-beta6 → 0.0.1-beta7

package/dist/runtime/internal.js

@@ -1,15 +1,18 @@
 // src/runtime/internal.ts
-globalThis.MarkoRun ?? (globalThis.MarkoRun = {});
-globalThis.MarkoRun.route = (handler) => handler;
-var RequestNotHandled = Symbol();
-var RequestNotMatched = Symbol();
+var NotHandled = Symbol();
+var NotMatched = Symbol();
+globalThis.MarkoRun ?? (globalThis.MarkoRun = {
+  NotHandled,
+  NotMatched,
+  route(handler) {
+    return handler;
+  }
+});
 function createInput(context) {
   let existing;
   return (data) => {
     existing ?? (existing = {
-      $global: {
-        context
-      }
+      $global: context
     });
     return data ? Object.assign(existing, data) : existing;
   };
@@ -46,7 +49,7 @@ async function call(handler, next, context) {
       response = await handler(context, next);
     } catch (error) {
       if (error == null) {
-        throw RequestNotHandled;
+        throw NotHandled;
       } else if (error instanceof Response) {
         return error;
       }
@@ -54,7 +57,7 @@ async function call(handler, next, context) {
     }
   }
   if (response === null) {
-    throw RequestNotMatched;
+    throw NotMatched;
   }
   return response || next();
 }
@@ -103,8 +106,8 @@ function notMatched() {
   return null;
 }
 export {
-  RequestNotHandled,
-  RequestNotMatched,
+  NotHandled,
+  NotMatched,
   call,
   compose,
   createInput,

package/dist/runtime/router.cjs

@@ -20,9 +20,9 @@ var __toCommonJS = (mod) => __copyProps(__defProp({}, "__esModule", { value: tru
 // src/runtime/router.ts
 var router_exports = {};
 __export(router_exports, {
-  invokeRoute: () => invokeRoute,
-  matchRoute: () => matchRoute,
-  router: () => router
+  fetch: () => fetch,
+  invoke: () => invoke,
+  match: () => match
 });
 module.exports = __toCommonJS(router_exports);
 function notImplemented() {
@@ -30,12 +30,12 @@ function notImplemented() {
     "This should have been replaced by the @marko/run plugin at build/dev time"
   );
 }
-var router = notImplemented;
-var matchRoute = notImplemented;
-var invokeRoute = notImplemented;
+var fetch = notImplemented;
+var match = notImplemented;
+var invoke = notImplemented;
 // Annotate the CommonJS export names for ESM import in node:
 0 && (module.exports = {
-  invokeRoute,
-  matchRoute,
-  router
+  fetch,
+  invoke,
+  match
 });

package/dist/runtime/router.d.ts

@@ -1,4 +1,3 @@
-import type { MatchRoute, RequestContext, Route } from "./types";
-export declare const router: <T>(context: RequestContext<T>) => Promise<Response | void>;
-export declare const matchRoute: MatchRoute;
-export declare const invokeRoute: <T>(route: Route | null, context: RequestContext<T>) => Promise<Response | void>;
+export declare const fetch: <Platform = unknown>(request: Request, platform: Platform) => Promise<void | Response>;
+export declare const match: (method: string, pathname: string) => import("./types").RouteWithHandler<{}, unknown, string> | null;
+export declare const invoke: <Platform = unknown>(route: import("./types").RouteWithHandler<{}, unknown, string> | null, request: Request, platform: Platform) => Promise<void | Response>;

package/dist/runtime/router.js

@@ -4,11 +4,11 @@ function notImplemented() {
     "This should have been replaced by the @marko/run plugin at build/dev time"
   );
 }
-var router = notImplemented;
-var matchRoute = notImplemented;
-var invokeRoute = notImplemented;
+var fetch = notImplemented;
+var match = notImplemented;
+var invoke = notImplemented;
 export {
-  invokeRoute,
-  matchRoute,
-  router
+  fetch,
+  invoke,
+  match
 };

package/dist/runtime/types.d.ts

@@ -7,20 +7,17 @@ export interface RouteContextExtensions {
 }
 export declare type ParamsObject = Record<string, string>;
 export declare type InputObject = Record<PropertyKey, any>;
-export interface RequestContext<T = unknown> {
+export declare type RouteContext<Platform = unknown, TRoute extends Route = Route> = TRoute extends any ? Combine<RouteContextExtensions & Readonly<{
     url: URL;
-    method: string;
     request: Request;
-    platform: T;
-}
-export declare type RouteContext<TRoute extends Route = Route> = TRoute extends any ? Combine<RouteContextExtensions & Readonly<RequestContext & {
     route: TRoute["path"];
     params: TRoute["params"];
     meta: TRoute["meta"];
+    platform: Platform;
 }>> : never;
 export declare type NextFunction = () => Awaitable<Response>;
 export declare type HandlerLike<TRoute extends Route = Route> = Awaitable<OneOrMany<RouteHandler<TRoute>>>;
-export declare type RouteHandler<TRoute extends Route = Route> = (context: RouteContext<TRoute>, next: NextFunction) => Awaitable<Response | null | void>;
+export declare type RouteHandler<TRoute extends Route = Route> = (context: RouteContext<unknown, TRoute>, next: NextFunction) => Awaitable<Response | null | void>;
 export interface Route<Params extends ParamsObject = {}, Meta = unknown, Path extends string = string> {
     path: Path;
     params: Params;
@@ -29,9 +26,14 @@ export interface Route<Params extends ParamsObject = {}, Meta = unknown, Path ex
 export interface RouteWithHandler<Params extends ParamsObject = {}, Meta = unknown, Path extends string = string> extends Route<Params, Meta, Path> {
     handler: RouteHandler<this>;
 }
-export declare type MatchRoute = (method: string, pathname: string) => RouteWithHandler | null;
-export declare type Router<T = unknown> = (context: RequestContext<T>) => Promise<Response | void>;
-export declare type InvokeRoute<T = unknown> = (route: Route | null, context: RequestContext<T>) => Promise<Response | void>;
+export declare type Fetch<Platform = unknown> = (request: Request, platform: Platform) => Promise<Response | void>;
+export declare type Match = (method: string, pathname: string) => RouteWithHandler | null;
+export declare type Invoke = <Platform = unknown>(route: RouteWithHandler | null, request: Request, platform: Platform) => Promise<Response | void>;
+export interface RuntimeModule {
+    fetch: <Platform = unknown>(request: Request, platform: Platform) => Promise<Response | void>;
+    match: (method: string, pathname: string) => RouteWithHandler | null;
+    invoke: <Platform = unknown>(route: RouteWithHandler | null, request: Request, platform: Platform) => Promise<Response | void>;
+}
 declare type Member<T, U> = T extends T ? (U extends T ? T : never) : never;
 declare type Segments<T extends string, Acc extends string[] = []> = T extends "" ? Acc : T extends `${infer Left}/${infer Rest}` ? Segments<Rest, [...Acc, Left]> : [...Acc, T];
 declare type GTE<A extends any[], B extends any[]> = A["length"] extends B["length"] ? 1 : A extends [infer _Ha, ...infer Ta] ? B extends [infer _Hb, ...infer Tb] ? GTE<Ta, Tb> : 1 : 0;

package/dist/vite/codegen/index.d.ts

@@ -3,4 +3,4 @@ export declare function renderRouteTemplate(route: Route): string;
 export declare function renderRouteEntry(route: Route): string;
 export declare function renderRouter(routes: BuiltRoutes, options?: RouterOptions): string;
 export declare function renderMiddleware(middleware: RoutableFile[]): string;
-export declare function renderRouteTypeInfo(routes: BuiltRoutes, pathPrefix?: string): string;
+export declare function renderRouteTypeInfo(routes: BuiltRoutes, pathPrefix?: string, adapterTypes?: string): string;

package/dist/vite/index.cjs

@@ -160,7 +160,14 @@ async function buildRoutes(walk, basePath) {
       if (handler || page) {
         const key = path3.replace(/(\$\$?)[^\/]*/g, "$1").replace(/^\/+/, "").replace(/[^a-z0-9_$\/]+/gi, "").replace(/\//g, "__") || "index";
         if (routes.has(key)) {
-          console.warn(`Duplicate route for path ${path3} -- ignoring`, current);
+          const existing = routes.get(key);
+          const existingFiles = [existing.handler, existing.page].filter(Boolean).map((f) => f.filePath);
+          const currentFiles = [handler, page].filter(Boolean).map((f) => f.filePath);
+          throw new Error(`Duplicate routes for path ${path3} were defined. A route established by:
+  ${existingFiles}
+    collides with
+  ${currentFiles.join(" and ")}
+  `);
         } else {
           const index = nextRouteIndex++;
           routes.set(key, {
@@ -208,14 +215,16 @@ async function buildRoutes(walk, basePath) {
       const prevChildren = children;
       const prevCurrent = current;
       const prevIsRoot = isRoot;
-      if (name.charCodeAt(0) === 95 || name.charCodeAt(0) === 40 && name.charCodeAt(name.length - 1) === 41 || name.toLowerCase() === "index") {
+      if (name.charCodeAt(0) === 95) {
       } else {
         if (name.charCodeAt(0) === 36) {
           if (name.charCodeAt(1) === 36) {
-            paramStack.push({
-              name: name.slice(2) || "rest",
-              index: -1
-            });
+            if (name.length > 2) {
+              paramStack.push({
+                name: name.slice(2),
+                index: -1
+              });
+            }
           } else if (name.length > 1) {
             paramStack.push({
               name: name.slice(1),
@@ -534,7 +543,7 @@ function renderRouteEntry(route) {
     );
   }
   if (middleware.length) {
-    const names = middleware.map((m) => `mware$${m.id}`);
+    const names = middleware.map((m) => `mware${m.id}`);
     imports.writeLines(
       `import { ${names.join(
         ", "
@@ -545,9 +554,9 @@ function renderRouteEntry(route) {
     writer.writeLines("");
     const names = [];
     for (const verb of handler.verbs) {
-      const name = verb === "delete" ? "del" : verb;
-      names.push(name);
-      writer.writeLines(`const handler$${name} = normalize(${name});`);
+      const importName = verb.toUpperCase();
+      names.push(importName);
+      writer.writeLines(`const ${verb}Handler = normalize(${importName});`);
     }
     imports.writeLines(
       `import { ${names.join(", ")} } from './${handler.importPath}';`
@@ -560,7 +569,7 @@ function renderRouteEntry(route) {
   }
   if (meta) {
     imports.writeLines(
-      `export { default as meta$${index} } from './${meta.importPath}';`
+      `export { default as meta${index} } from './${meta.importPath}';`
     );
   }
   for (const verb of verbs) {
@@ -591,14 +600,19 @@ function writeRouteEntryHandler(writer, route, verb) {
   let nextName;
   let currentName;
   let hasBody = false;
-  writer.writeLines("").writeBlockStart(
-    `export async function ${verb}$${index}(context, buildInput) {`
-  );
+  writer.writeLines("");
+  if (page) {
+    writer.writeBlockStart(
+      `export async function ${verb}${index}(context, buildInput) {`
+    );
+  } else {
+    writer.writeBlockStart(`export async function ${verb}${index}(context) {`);
+  }
   const continuations = writer.branch("cont");
   if (page && verb === "get") {
     currentName = "__page";
     if ((_a = handler == null ? void 0 : handler.verbs) == null ? void 0 : _a.includes(verb)) {
-      const name = `handler$${verb}`;
+      const name = `${verb}Handler`;
       writePageResponse(continuations, currentName);
       if (len) {
         nextName = currentName;
@@ -616,7 +630,7 @@ function writeRouteEntryHandler(writer, route, verb) {
       hasBody = true;
     }
   } else if (handler) {
-    const name = `handler$${verb}`;
+    const name = `${verb}Handler`;
     currentName = `__${name}`;
     nextName = "noContent";
     if (len) {
@@ -632,7 +646,7 @@ function writeRouteEntryHandler(writer, route, verb) {
     let i = len;
     while (i--) {
       const { id } = middleware[i];
-      const name = `mware$${id}`;
+      const name = `mware${id}`;
       nextName = currentName;
       currentName = i ? `__${name}` : "";
       writeMiddleware(continuations, name, nextName, currentName);
@@ -648,12 +662,12 @@ function renderRouter(routes, options = {
   writer.writeLines(`// @marko/run/router`);
   const imports = writer.branch("imports");
   imports.writeLines(
-    `import { RequestNotHandled, RequestNotMatched, createInput } from 'virtual:marko-run/internal';`
+    `import { NotHandled, NotMatched, createInput } from 'virtual:marko-run/internal';`
   );
   for (const route of routes.list) {
     const verbs = getVerbs(route);
-    const names = verbs.map((verb) => `${verb}$${route.index}`);
-    route.meta && names.push(`meta$${route.index}`);
+    const names = verbs.map((verb) => `${verb}${route.index}`);
+    route.meta && names.push(`meta${route.index}`);
     imports.writeLines(
       `import { ${names.join(
         ", "
@@ -662,10 +676,16 @@ function renderRouter(routes, options = {
   }
   for (const { key } of Object.values(routes.special)) {
     imports.writeLines(
-      `import page$${key} from '${virtualFilePrefix}/${markoRunFilePrefix}special__${key}.marko${serverEntryQuery}';`
+      `import page${key} from '${virtualFilePrefix}/${markoRunFilePrefix}special__${key}.marko${serverEntryQuery}';`
     );
   }
-  writer.writeLines(``).writeBlockStart(`function findRoute(method, pathname) {`).writeBlockStart(`switch (method.toLowerCase()) {`);
+  writer.writeLines(``).writeBlockStart(`export function match(method, pathname) {`).writeLines(
+    `if (!pathname) {
+    pathname = '/';
+  } else if (pathname.charAt(0) !== '/') {
+    pathname = '/' + pathname;
+  }`
+  ).writeBlockStart(`switch (method.toLowerCase()) {`);
   for (const verb of httpVerbs) {
     const filteredRoutes = routes.list.filter((route) => hasVerb(route, verb));
     if (filteredRoutes.length) {
@@ -677,16 +697,12 @@ function renderRouter(routes, options = {
   }
   writer.writeBlockEnd("}").writeLines("return null;").writeBlockEnd("}");
   writer.write(`
-export function matchRoute(method, pathname) {
-  if (!pathname) {
-    pathname = '/';
-  } else if (pathname.charAt(0) !== '/') {
-    pathname = '/' + pathname;
-  }
-  return findRoute(method, pathname);
-}
-
-export async function invokeRoute(route, context) {
+export async function invoke(route, request, platform, url = new URL(request.url)) {
+  const context = {
+    url,
+    request,
+    platform
+  };
 	try {
     const buildInput = createInput(context);
 		if (route) {
@@ -696,9 +712,9 @@ export async function invokeRoute(route, context) {
 				const response = await route.handler(context, buildInput);
 				if (response) return response;
 			} catch (error) {
-				if (error === RequestNotHandled) {
+				if (error === NotHandled) {
 					return;
-				} else if (error !== RequestNotMatched) {
+				} else if (error !== NotMatched) {
 					throw error;
 				}
 			}
@@ -710,7 +726,7 @@ export async function invokeRoute(route, context) {
       context.meta = {};
     }
     if (context.request.headers.get('Accept')?.includes('text/html')) {
-      return new Response(page$404.stream(buildInput()), {
+      return new Response(page404.stream(buildInput()), {
         status: 404,
         headers: { "content-type": "text/html;charset=UTF-8" },
       });
@@ -726,7 +742,7 @@ export async function invokeRoute(route, context) {
     writer.writeBlockStart(
       `if (context.request.headers.get('Accept')?.includes('text/html')) {`
     ).writeBlock(
-      `return new Response(page$500.stream(buildInput({ error })), {`,
+      `return new Response(page500.stream(buildInput({ error })), {`,
       [
         `status: 500,`,
         `headers: { "content-type": "text/html;charset=UTF-8" },`
@@ -735,9 +751,9 @@ export async function invokeRoute(route, context) {
     ).writeBlockEnd("}");
   }
   writer.writeLines(`throw error;`).writeBlockEnd("}").writeBlockEnd("}").write(`
-export async function router(context) {
+export async function fetch(request, platform) {
   try {
-    const { url, method } = context;
+    const url = new URL(request.url);
     let { pathname } = url;`);
   switch (options.trailingSlashes) {
     case "RedirectWithout":
@@ -769,8 +785,8 @@ export async function router(context) {
   }
   writer.write(`   
 
-    const route = matchRoute(method, pathname);
-    return await invokeRoute(route, context);
+    const route = match(request.method, pathname);
+    return await invoke(route, request, platform, url);
   } catch (error) {
     const message = import.meta.env.DEV
       ? \`Internal Server Error (\${error.message})\`
@@ -948,9 +964,9 @@ function renderParamsInfoType(params) {
 }
 function renderMatch(verb, route, pathIndex) {
   var _a;
-  const handler = `${verb}$${route.index}`;
+  const handler = `${verb}${route.index}`;
   const params = ((_a = route.params) == null ? void 0 : _a.length) ? renderParamsInfo(route.params, pathIndex) : "{}";
-  const meta = route.meta ? `meta$${route.index}` : "{}";
+  const meta = route.meta ? `meta${route.index}` : "{}";
   return `{ handler: ${handler}, params: ${params}, meta: ${meta} }`;
 }
 function renderMiddleware(middleware) {
@@ -962,9 +978,9 @@ function renderMiddleware(middleware) {
   imports.writeLines(`import { normalize } from 'virtual:marko-run/internal';`);
   writer.writeLines("");
   for (const { id, importPath } of middleware) {
-    const importName = `mware${id}`;
+    const importName = `middleware${id}`;
     imports.writeLines(`import ${importName} from './${importPath}';`);
-    writer.writeLines(`export const mware$${id} = normalize(${importName});`);
+    writer.writeLines(`export const mware${id} = normalize(${importName});`);
   }
   imports.join();
   return writer.end();
@@ -979,7 +995,7 @@ function stripTsExtension(path3) {
   }
   return path3;
 }
-function renderRouteTypeInfo(routes, pathPrefix = ".") {
+function renderRouteTypeInfo(routes, pathPrefix = ".", adapterTypes = "") {
   var _a, _b;
   const writer = createStringWriter();
   writer.writeLines(
@@ -987,8 +1003,10 @@ function renderRouteTypeInfo(routes, pathPrefix = ".") {
   WARNING: This file is automatically generated and any changes made to it will be overwritten without warning.
   Do NOT manually edit this file or your changes will be lost.
 */
-  `,
-    `import type { HandlerLike, Route, RouteContext, ValidatePath, ValidateHref } from "@marko/run";
+`,
+    `import type { HandlerLike, Route, RouteContext, ValidatePath, ValidateHref } from "@marko/run";`,
+    adapterTypes,
+    `
 
 declare global {
   namespace MarkoRun {`
@@ -1058,7 +1076,7 @@ declare module '${pathPrefix}/${page.relativePath}' {
 
   namespace MarkoRun {
     type CurrentRoute = ${routeType};
-    type CurrentContext = RouteContext<CurrentRoute>;
+    type CurrentContext = RouteContext<RouteContext['platform'], CurrentRoute>;
   }
 }`);
     }
@@ -1123,7 +1141,7 @@ declare module '${pathPrefix}/${file.relativePath}' {
 
   namespace MarkoRun {
     type CurrentRoute = ${routeTypes.join(" | ")};
-    type CurrentContext = RouteContext<CurrentRoute>;
+    type CurrentContext = RouteContext<RouteContext['platform'], CurrentRoute>;
   }
 }`);
   }
@@ -1141,7 +1159,7 @@ declare module '${pathPrefix}/${route.page.relativePath}' {
 
   namespace MarkoRun {
     type CurrentRoute = Route;
-    type CurrentContext = RouteContext<CurrentRoute>;
+    type CurrentContext = RouteContext<RouteContext['platform'], CurrentRoute>;
   }
 }`);
     }
@@ -1154,7 +1172,7 @@ function writeRouteTypeModule(writer, pathPrefix, path3, routeType) {
 declare module '${pathPrefix}/${stripTsExtension(path3)}' {
   namespace MarkoRun {
     type CurrentRoute = ${routeType};
-    type CurrentContext = RouteContext<CurrentRoute>;
+    type CurrentContext = RouteContext<RouteContext['platform'], CurrentRoute>;
     type Handler<_Params = CurrentRoute['params'], _Meta = CurrentRoute['meta']> = HandlerLike<CurrentRoute>;
     function route(handler: Handler): typeof handler;
     function route<_Params = CurrentRoute['params'], _Meta = CurrentRoute['meta']>(handler: Handler): typeof handler;
@@ -1385,7 +1403,7 @@ function markoServe(opts = {}) {
     ))) {
       const filepath = import_path2.default.join(typesDir, "routes.d.ts");
       const adapterTypeInfo = (adapter == null ? void 0 : adapter.writeTypeInfo) && await (adapter == null ? void 0 : adapter.writeTypeInfo());
-      const data = renderRouteTypeInfo(routes, import_path2.default.relative(typesDir, routesDir)) + adapterTypeInfo;
+      const data = renderRouteTypeInfo(routes, import_path2.default.relative(typesDir, routesDir), adapterTypeInfo);
       if (data !== typesFile || !import_fs2.default.existsSync(filepath)) {
         await ensureDir(typesDir);
         await import_fs2.default.promises.writeFile(filepath, typesFile = data);
@@ -1652,8 +1670,8 @@ async function getVerbsFromFileBuild(context, filePath) {
   if (result) {
     const exportIds = getExportIdentifiers(result.ast);
     for (const id of exportIds) {
-      const verb = id === "del" ? "delete" : id;
-      if (httpVerbs.includes(verb)) {
+      const verb = id.toLowerCase();
+      if (id === verb.toUpperCase() && httpVerbs.includes(verb)) {
         verbs.push(verb);
       }
     }
@@ -1664,12 +1682,19 @@ async function getVerbsFromFileDev(devServer, filePath) {
   const verbs = [];
   const result = await devServer.transformRequest(filePath, { ssr: true });
   if (result && result.code) {
-    const verbMatchReg = /__vite_ssr_exports__,\s+["'](get|post|put|del)["']/g;
+    const verbMatchReg = /__vite_ssr_exports__,\s+["'](GET|POST|PUT|DELETE)["']/gi;
     let match = verbMatchReg.exec(result.code);
     while (match) {
-      const verb = match[1] === "del" ? "delete" : match[1];
+      const id = match[1];
+      const verb = id.toLowerCase();
       if (httpVerbs.includes(verb)) {
-        verbs.push(verb);
+        if (id === verb.toUpperCase()) {
+          verbs.push(verb);
+        } else {
+          console.warn(
+            `Found export '${id}' in handler ${filePath} which is close to '${verb.toUpperCase()}'. Exported handlers need to be uppercase: GET, POST, PUT or DELETE.`
+          );
+        }
       }
       match = verbMatchReg.exec(result.code);
     }