wrangler 0.0.0-e7cf0caa → 0.0.0-e8053554

package/src/pages/errors.ts

@@ -1,67 +0,0 @@
-import {
-	MAX_FUNCTIONS_ROUTES_RULES,
-	MAX_FUNCTIONS_ROUTES_RULE_LENGTH,
-	ROUTES_SPEC_VERSION,
-} from "./constants";
-import { RoutesValidationError } from "./functions/routes-validation";
-
-/**
- * Exit code for `pages functions build` when no routes are found.
- */
-export const EXIT_CODE_FUNCTIONS_NO_ROUTES_ERROR = 156;
-export const EXIT_CODE_FUNCTIONS_NOTHING_TO_BUILD_ERROR = 157;
-
-/**
- * Pages error when no routes are found in the functions directory
- */
-export class FunctionsNoRoutesError extends Error {
-	constructor(message: string) {
-		super(message);
-	}
-}
-
-/**
- * Warning message for when buildFunctions throws FunctionsNoRoutesError
- */
-export function getFunctionsNoRoutesWarning(
-	functionsDirectory: string,
-	suffix?: string
-) {
-	return `No routes found when building Functions directory: ${functionsDirectory}${
-		suffix ? " - " + suffix : ""
-	}`;
-}
-
-export function getRoutesValidationErrorMessage(
-	errorCode: RoutesValidationError,
-	routesPath: string
-): string {
-	switch (errorCode) {
-		case RoutesValidationError.NO_INCLUDE_RULES:
-			return `Invalid _routes.json file found at: ${routesPath}
-Routes must have at least 1 include rule, but no include rules were detected.`;
-		case RoutesValidationError.TOO_MANY_RULES:
-			return `Invalid _routes.json file found at: ${routesPath}
-Detected rules that are over the ${MAX_FUNCTIONS_ROUTES_RULES} rule limit. Please make sure you have a total of ${MAX_FUNCTIONS_ROUTES_RULES} include and exclude rules combined.`;
-		case RoutesValidationError.RULE_TOO_LONG:
-			return `Invalid _routes.json file found at: ${routesPath}
-Detected rules the are over the ${MAX_FUNCTIONS_ROUTES_RULE_LENGTH} character limit. Please make sure that each include and exclude routing rule is at most ${MAX_FUNCTIONS_ROUTES_RULE_LENGTH} characters long.`;
-		case RoutesValidationError.INVALID_RULES:
-			return `Invalid _routes.json file found at: ${routesPath}
-All rules must start with '/'.`;
-		case RoutesValidationError.OVERLAPPING_RULES:
-			return `Invalid _routes.json file found at: ${routesPath}
-Overlapping rules found. Please make sure that rules ending with a splat (eg. "/api/*") don't overlap any other rules (eg. "/api/foo"). This applies to both include and exclude rules individually.`;
-		case RoutesValidationError.INVALID_JSON_SPEC:
-		default:
-			return `Invalid _routes.json file found at: ${routesPath}
-Please make sure the JSON object has the following format:
-{
-	version: ${ROUTES_SPEC_VERSION};
-	include: string[];
-	exclude: string[];
-}
-and that at least one include rule is provided.
-			`;
-	}
-}

package/src/pages/functions/buildPlugin.ts

@@ -1,113 +0,0 @@
-import { access, lstat } from "node:fs/promises";
-import { relative, resolve } from "node:path";
-import { bundleWorker } from "../../bundle";
-import { getBasePath } from "../../paths";
-import { D1_BETA_PREFIX } from "../../worker";
-import { buildNotifierPlugin } from "./buildWorker";
-import type { Options as WorkerOptions } from "./buildWorker";
-
-type Options = Omit<
-	WorkerOptions,
-	"outfile" | "fallbackService" | "buildOutputDirectory" | "nodejsCompat"
-> & { outdir: string };
-
-export function buildPlugin({
-	routesModule,
-	outdir,
-	minify = false,
-	sourcemap = false,
-	watch = false,
-	onEnd = () => {},
-	legacyNodeCompat,
-	functionsDirectory,
-	local,
-	betaD1Shims,
-}: Options) {
-	return bundleWorker(
-		{
-			file: resolve(getBasePath(), "templates/pages-template-plugin.ts"),
-			directory: functionsDirectory,
-			format: "modules",
-			moduleRoot: functionsDirectory,
-		},
-		resolve(outdir),
-		{
-			inject: [routesModule],
-			entryName: "index",
-			minify,
-			sourcemap,
-			watch,
-			legacyNodeCompat,
-			// We don't currently have a mechanism for Plugins 'requiring' a specific compat date/flag,
-			// but if someone wants to publish a Plugin which does require this new `nodejs_compat` flag
-			// and they document that on their README.md, we should let them.
-			nodejsCompat: true,
-			define: {},
-			betaD1Shims: (betaD1Shims || []).map(
-				(binding) => `${D1_BETA_PREFIX}${binding}`
-			),
-			doBindings: [], // Pages functions don't support internal Durable Objects
-			plugins: [
-				buildNotifierPlugin(onEnd),
-				{
-					name: "Assets",
-					setup(pluginBuild) {
-						pluginBuild.onResolve({ filter: /^assets:/ }, async (args) => {
-							const directory = resolve(
-								args.resolveDir,
-								args.path.slice("assets:".length)
-							);
-
-							const exists = await access(directory)
-								.then(() => true)
-								.catch(() => false);
-
-							const isDirectory =
-								exists && (await lstat(directory)).isDirectory();
-
-							if (!isDirectory) {
-								return {
-									errors: [
-										{
-											text: `'${directory}' does not exist or is not a directory.`,
-										},
-									],
-								};
-							}
-
-							const path = `assets:./${relative(outdir, directory)}`;
-
-							return { path, external: true, namespace: "assets" };
-						});
-					},
-				},
-				// TODO: Replace this with a proper outdir solution for Plugins
-				// But for now, let's just mark all wasm/bin files as external
-				{
-					name: "Mark externals",
-					setup(pluginBuild) {
-						pluginBuild.onResolve(
-							{ filter: /.*\.(wasm|bin)$/ },
-							async (args) => {
-								return {
-									external: true,
-									path: `./${relative(
-										outdir,
-										resolve(args.resolveDir, args.path)
-									)}`,
-								};
-							}
-						);
-					},
-				},
-			],
-			serveAssetsFromWorker: false,
-			disableModuleCollection: false,
-			rules: [],
-			checkFetch: local,
-			targetConsumer: local ? "dev" : "publish",
-			local,
-			experimentalLocal: false,
-		}
-	);
-}

package/src/pages/functions/buildWorker.ts

@@ -1,291 +0,0 @@
-import { access, cp, lstat, rm } from "node:fs/promises";
-import { tmpdir } from "node:os";
-import { join, resolve } from "node:path";
-import { build as esBuild } from "esbuild";
-import { nanoid } from "nanoid";
-import { bundleWorker } from "../../bundle";
-import { FatalError } from "../../errors";
-import { logger } from "../../logger";
-import { getBasePath } from "../../paths";
-import { D1_BETA_PREFIX } from "../../worker";
-import type { Plugin } from "esbuild";
-
-export type Options = {
-	routesModule: string;
-	outfile?: string;
-	outdir?: string;
-	minify?: boolean;
-	sourcemap?: boolean;
-	fallbackService?: string;
-	watch?: boolean;
-	onEnd?: () => void;
-	buildOutputDirectory?: string;
-	legacyNodeCompat?: boolean;
-	nodejsCompat?: boolean;
-	functionsDirectory: string;
-	local: boolean;
-	betaD1Shims?: string[];
-};
-
-export function buildWorker({
-	routesModule,
-	outfile = join(tmpdir(), `./functionsWorker-${Math.random()}.js`),
-	outdir,
-	minify = false,
-	sourcemap = false,
-	fallbackService = "ASSETS",
-	watch = false,
-	onEnd = () => {},
-	buildOutputDirectory,
-	legacyNodeCompat,
-	nodejsCompat,
-	functionsDirectory,
-	local,
-	betaD1Shims,
-}: Options) {
-	return bundleWorker(
-		{
-			file: resolve(getBasePath(), "templates/pages-template-worker.ts"),
-			directory: functionsDirectory,
-			format: "modules",
-			moduleRoot: functionsDirectory,
-		},
-		outdir ? resolve(outdir) : resolve(outfile),
-		{
-			inject: [routesModule],
-			...(outdir ? { entryName: "index" } : {}),
-			minify,
-			sourcemap,
-			watch,
-			legacyNodeCompat,
-			nodejsCompat,
-			define: {
-				__FALLBACK_SERVICE__: JSON.stringify(fallbackService),
-			},
-			betaD1Shims: (betaD1Shims || []).map(
-				(binding) => `${D1_BETA_PREFIX}${binding}`
-			),
-			doBindings: [], // Pages functions don't support internal Durable Objects
-			plugins: [
-				buildNotifierPlugin(onEnd),
-				{
-					name: "Assets",
-					setup(pluginBuild) {
-						const identifiers = new Map<string, string>();
-
-						pluginBuild.onResolve({ filter: /^assets:/ }, async (args) => {
-							const directory = resolve(
-								args.resolveDir,
-								args.path.slice("assets:".length)
-							);
-
-							const exists = await access(directory)
-								.then(() => true)
-								.catch(() => false);
-
-							const isDirectory =
-								exists && (await lstat(directory)).isDirectory();
-
-							if (!isDirectory) {
-								return {
-									errors: [
-										{
-											text: `'${directory}' does not exist or is not a directory.`,
-										},
-									],
-								};
-							}
-
-							// TODO: Consider hashing the contents rather than using a unique identifier every time?
-							identifiers.set(directory, nanoid());
-							if (!buildOutputDirectory) {
-								console.warn(
-									"You're attempting to import static assets as part of your Pages Functions, but have not specified a directory in which to put them. You must use 'wrangler pages dev <directory>' rather than 'wrangler pages dev -- <command>' to import static assets in Functions."
-								);
-							}
-							return { path: directory, namespace: "assets" };
-						});
-
-						pluginBuild.onLoad(
-							{ filter: /.*/, namespace: "assets" },
-							async (args) => {
-								const identifier = identifiers.get(args.path);
-
-								if (buildOutputDirectory) {
-									const staticAssetsOutputDirectory = join(
-										buildOutputDirectory,
-										"cdn-cgi",
-										"pages-plugins",
-										identifier as string
-									);
-									await rm(staticAssetsOutputDirectory, {
-										force: true,
-										recursive: true,
-									});
-									await cp(args.path, staticAssetsOutputDirectory, {
-										force: true,
-										recursive: true,
-									});
-
-									return {
-										// TODO: Watch args.path for changes and re-copy when updated
-										contents: `export const onRequest = ({ request, env, functionPath }) => {
-                    const url = new URL(request.url)
-                    const relativePathname = \`/\${url.pathname.replace(functionPath, "") || ""}\`.replace(/^\\/\\//, '/');
-                    url.pathname = '/cdn-cgi/pages-plugins/${identifier}' + relativePathname
-                    request = new Request(url.toString(), request)
-                    return env.ASSETS.fetch(request)
-                  }`,
-									};
-								}
-							}
-						);
-					},
-				},
-			],
-			isOutfile: !outdir,
-			serveAssetsFromWorker: false,
-			disableModuleCollection: false,
-			rules: [],
-			checkFetch: local,
-			targetConsumer: local ? "dev" : "publish",
-			local,
-			experimentalLocal: false,
-		}
-	);
-}
-
-export type RawOptions = {
-	workerScriptPath: string;
-	outfile?: string;
-	outdir?: string;
-	directory: string;
-	minify?: boolean;
-	sourcemap?: boolean;
-	watch?: boolean;
-	plugins?: Plugin[];
-	onEnd?: () => void;
-	buildOutputDirectory?: string;
-	legacyNodeCompat?: boolean;
-	nodejsCompat?: boolean;
-	local: boolean;
-	betaD1Shims?: string[];
-};
-
-/**
- * This function bundles a raw `_worker.js` Pages file
- * before it gets deployed.
- *
- * This allows Wrangler to add shims and other wrappers
- * around the handlers, which is useful to support beta features.
- */
-export function buildRawWorker({
-	workerScriptPath,
-	outfile = join(tmpdir(), `./functionsWorker-${Math.random()}.js`),
-	outdir,
-	directory,
-	minify = false,
-	sourcemap = false,
-	watch = false,
-	plugins = [],
-	onEnd = () => {},
-	legacyNodeCompat,
-	nodejsCompat,
-	local,
-	betaD1Shims,
-}: RawOptions) {
-	return bundleWorker(
-		{
-			file: workerScriptPath,
-			directory: resolve(directory),
-			format: "modules",
-			moduleRoot: resolve(directory),
-		},
-		outdir ? resolve(outdir) : resolve(outfile),
-		{
-			minify,
-			sourcemap,
-			watch,
-			legacyNodeCompat,
-			nodejsCompat,
-			define: {},
-			betaD1Shims: (betaD1Shims || []).map(
-				(binding) => `${D1_BETA_PREFIX}${binding}`
-			),
-			doBindings: [], // Pages functions don't support internal Durable Objects
-			plugins: [...plugins, buildNotifierPlugin(onEnd)],
-			isOutfile: !outdir,
-			serveAssetsFromWorker: false,
-			disableModuleCollection: false,
-			rules: [],
-			checkFetch: local,
-			targetConsumer: local ? "dev" : "publish",
-			local,
-			experimentalLocal: false,
-		}
-	);
-}
-
-/**
- * Creates an esbuild plugin that can notify Wrangler (via the `onEnd()`)
- * when the build completes.
- */
-export function buildNotifierPlugin(onEnd: () => void): Plugin {
-	return {
-		name: "wrangler notifier and monitor",
-		setup(pluginBuild) {
-			pluginBuild.onEnd((result) => {
-				if (result.errors.length > 0) {
-					logger.error(
-						`${result.errors.length} error(s) and ${result.warnings.length} warning(s) when compiling Worker.`
-					);
-				} else if (result.warnings.length > 0) {
-					logger.warn(
-						`${result.warnings.length} warning(s) when compiling Worker.`
-					);
-					onEnd();
-				} else {
-					logger.log("✨ Compiled Worker successfully");
-					onEnd();
-				}
-			});
-		},
-	};
-}
-
-/**
- * Runs the script through a simple esbuild bundle step to check for unwanted imports.
- *
- * This is useful when the user chooses not to bundle the `_worker.js` file by setting
- * `--no-bundle` at the command line.
- */
-export async function checkRawWorker(scriptPath: string, onEnd: () => void) {
-	await esBuild({
-		entryPoints: [scriptPath],
-		write: false,
-		// we need it to be bundled so that any imports that are used are affected by the blocker plugin
-		bundle: true,
-		plugins: [blockWorkerJsImports, buildNotifierPlugin(onEnd)],
-	});
-}
-
-const blockWorkerJsImports: Plugin = {
-	name: "block-worker-js-imports",
-	setup(build) {
-		build.onResolve({ filter: /.*/g }, (args) => {
-			// If it's the entrypoint, let it be as is
-			if (args.kind === "entry-point") {
-				return {
-					path: args.path,
-				};
-			}
-			// Otherwise, block any imports that the file is requesting
-			throw new FatalError(
-				"_worker.js is not being bundled by Wrangler but it is importing from another file.\n" +
-					"This will throw an error if deployed.\n" +
-					"You should bundle the Worker in a pre-build step, remove the import if it is unused, or ask Wrangler to bundle it by setting `--bundle`.",
-				1
-			);
-		});
-	},
-};

package/src/pages/functions/filepath-routing.test.ts

@@ -1,234 +0,0 @@
-import { writeFileSync, mkdirSync } from "fs";
-import { runInTempDir } from "../../__tests__/helpers/run-in-tmp";
-import { toUrlPath } from "../../paths";
-import { compareRoutes, generateConfigFromFileTree } from "./filepath-routing";
-import type { UrlPath } from "../../paths";
-import type { HTTPMethod, RouteConfig } from "./routes";
-
-describe("filepath-routing", () => {
-	describe("compareRoutes()", () => {
-		test("routes / last", () => {
-			expect(
-				compareRoutes(routeConfig("/"), routeConfig("/foo"))
-			).toBeGreaterThanOrEqual(1);
-			expect(
-				compareRoutes(routeConfig("/"), routeConfig("/:foo"))
-			).toBeGreaterThanOrEqual(1);
-			expect(
-				compareRoutes(routeConfig("/"), routeConfig("/:foo*"))
-			).toBeGreaterThanOrEqual(1);
-		});
-
-		test("routes with fewer segments come after those with more segments", () => {
-			expect(
-				compareRoutes(routeConfig("/foo"), routeConfig("/foo/bar"))
-			).toBeGreaterThanOrEqual(1);
-			expect(
-				compareRoutes(routeConfig("/foo"), routeConfig("/foo/bar/cat"))
-			).toBeGreaterThanOrEqual(1);
-		});
-
-		test("routes with wildcard segments come after those without", () => {
-			expect(compareRoutes(routeConfig("/:foo*"), routeConfig("/foo"))).toBe(1);
-			expect(compareRoutes(routeConfig("/:foo*"), routeConfig("/:foo"))).toBe(
-				1
-			);
-		});
-
-		test("routes with dynamic segments come after those without", () => {
-			expect(compareRoutes(routeConfig("/:foo"), routeConfig("/foo"))).toBe(1);
-		});
-
-		test("routes with dynamic segments occurring earlier come after those with dynamic segments in later positions", () => {
-			expect(
-				compareRoutes(routeConfig("/foo/:id/bar"), routeConfig("/foo/bar/:id"))
-			).toBe(1);
-		});
-
-		test("routes with no HTTP method come after those specifying a method", () => {
-			expect(
-				compareRoutes(routeConfig("/foo"), routeConfig("/foo", "GET"))
-			).toBe(1);
-		});
-
-		test("two equal routes are sorted according to their original position in the list", () => {
-			expect(
-				compareRoutes(routeConfig("/foo", "GET"), routeConfig("/foo", "GET"))
-			).toBe(0);
-		});
-
-		test("it returns -1 if the first argument should appear first in the list", () => {
-			expect(
-				compareRoutes(routeConfig("/foo", "GET"), routeConfig("/foo"))
-			).toBe(-1);
-		});
-	});
-
-	describe("generateConfigFromFileTree", () => {
-		runInTempDir();
-
-		it("should generate a route entry for each file in the tree", async () => {
-			writeFileSync(
-				"foo.ts",
-				`
-      export function onRequestGet() {}
-      export function onRequestPost() {}
-      `
-			);
-			writeFileSync(
-				"bar.ts",
-				`
-      export function onRequestPut() {}
-      export function onRequestDelete() {}
-      `
-			);
-
-			mkdirSync("todos");
-
-			writeFileSync(
-				"todos/[id].ts",
-				`
-      export function onRequestPost() {}
-      export function onRequestDelete() {}
-      `
-			);
-
-			mkdirSync("authors");
-			mkdirSync("authors/[authorId]");
-			mkdirSync("authors/[authorId]/todos");
-
-			writeFileSync(
-				"authors/[authorId]/todos/[todoId].ts",
-				`
-      export function onRequestPost() {}
-      `
-			);
-
-			mkdirSync("books");
-
-			writeFileSync(
-				"books/[[title]].ts",
-				`
-      export function onRequestPost() {}
-      `
-			);
-
-			mkdirSync("cats");
-			mkdirSync("cats/[[breed]]");
-
-			writeFileSync(
-				"cats/[[breed]]/blah.ts",
-				`
-      export function onRequestPost() {}
-      `
-			);
-
-			// This won't actually work at runtime.
-			writeFileSync(
-				"cats/[[breed]]/[[name]].ts",
-				`
-      export function onRequestPost() {}
-      `
-			);
-
-			const entries = await generateConfigFromFileTree({
-				baseDir: ".",
-				baseURL: "/base" as UrlPath,
-			});
-			expect(entries).toMatchInlineSnapshot(`
-        Object {
-          "routes": Array [
-            Object {
-              "method": "POST",
-              "module": Array [
-                "authors/[authorId]/todos/[todoId].ts:onRequestPost",
-              ],
-              "mountPath": "/base/authors/:authorId/todos",
-              "routePath": "/base/authors/:authorId/todos/:todoId",
-            },
-            Object {
-              "method": "POST",
-              "module": Array [
-                "cats/[[breed]]/blah.ts:onRequestPost",
-              ],
-              "mountPath": "/base/cats/:breed*",
-              "routePath": "/base/cats/:breed*/blah",
-            },
-            Object {
-              "method": "POST",
-              "module": Array [
-                "cats/[[breed]]/[[name]].ts:onRequestPost",
-              ],
-              "mountPath": "/base/cats/:breed*",
-              "routePath": "/base/cats/:breed*/:name*",
-            },
-            Object {
-              "method": "DELETE",
-              "module": Array [
-                "todos/[id].ts:onRequestDelete",
-              ],
-              "mountPath": "/base/todos",
-              "routePath": "/base/todos/:id",
-            },
-            Object {
-              "method": "POST",
-              "module": Array [
-                "todos/[id].ts:onRequestPost",
-              ],
-              "mountPath": "/base/todos",
-              "routePath": "/base/todos/:id",
-            },
-            Object {
-              "method": "POST",
-              "module": Array [
-                "books/[[title]].ts:onRequestPost",
-              ],
-              "mountPath": "/base/books",
-              "routePath": "/base/books/:title*",
-            },
-            Object {
-              "method": "DELETE",
-              "module": Array [
-                "bar.ts:onRequestDelete",
-              ],
-              "mountPath": "/base/",
-              "routePath": "/base/bar",
-            },
-            Object {
-              "method": "PUT",
-              "module": Array [
-                "bar.ts:onRequestPut",
-              ],
-              "mountPath": "/base/",
-              "routePath": "/base/bar",
-            },
-            Object {
-              "method": "GET",
-              "module": Array [
-                "foo.ts:onRequestGet",
-              ],
-              "mountPath": "/base/",
-              "routePath": "/base/foo",
-            },
-            Object {
-              "method": "POST",
-              "module": Array [
-                "foo.ts:onRequestPost",
-              ],
-              "mountPath": "/base/",
-              "routePath": "/base/foo",
-            },
-          ],
-        }
-      `);
-		});
-	});
-});
-
-function routeConfig(routePath: string, method?: string): RouteConfig {
-	return {
-		routePath: toUrlPath(routePath),
-		mountPath: toUrlPath("/"),
-		method: method as HTTPMethod,
-	};
-}