@noverax/courier-build 0.4.0 → 0.5.0

package/lib/engine/worker.js

@@ -1,10 +1,10 @@
 import { t as RoutesReadError } from "../routes-read-error-DQhKIE2F.js";
 import { n as isRecord } from "../errno-code-CNx_E80R.js";
 import { t as workerMessages } from "../worker-messages-BT2vvNQJ.js";
-import { a as projectTransformCache, c as UntrustedConfigError, f as configCandidates, i as loadCourierConfigFile, t as findConfigFile } from "../load-config-BeedbpkN.js";
+import { T as configCandidates, a as projectTransformCache, c as UntrustedConfigError, d as configLoadError, h as localPath, i as loadCourierConfigFile, p as maskedError, t as findConfigFile } from "../load-config-barNPfXN.js";
 import * as nodeModule from "node:module";
 import { createRequire, syncBuiltinESMExports } from "node:module";
-import { basename, dirname, isAbsolute, resolve, sep } from "node:path";
+import { basename, dirname, isAbsolute, sep } from "node:path";
 import { existsSync, readFileSync } from "node:fs";
 import { fileURLToPath, pathToFileURL } from "node:url";
 import { parseEnv } from "node:util";
@@ -98,10 +98,12 @@ async function loadProjectConfig(configFile, request, options) {
 			transformCache
 		});
 	} catch (error) {
-		throw new RoutesReadError("CONFIG_NOT_LOADED", `${basename(configFile)} did not load, so the routes a page takes were not read: ${error instanceof Error ? error.message : String(error)}`, {
+		const failure = configLoadError(error, { entry: configFile });
+		const message = `${basename(configFile)} did not load, so the routes a page takes were not read: ${failure.message} ${failure.hint}`;
+		throw new RoutesReadError("CONFIG_NOT_LOADED", message, {
 			root: request.root,
 			configFile
-		}, { stack: error instanceof Error ? error.stack : void 0 });
+		}, { stack: failure.cause instanceof Error ? failure.cause.stack : void 0 });
 	}
 }
 // src/engine/read-env-files.ts
@@ -132,27 +134,39 @@ function parsedFrom(files) {
 	for (const file of files) for (const [key, value] of Object.entries(parseEnv(readFileSync(file, "utf8")))) if (value !== void 0 && !(key in parsed)) parsed[key] = value;
 	return parsed;
 }
-// src/util/local-path.ts
-const NAMESPACE = "\\\\?\\";
-const SHARE_NAMESPACE = "\\\\?\\UNC\\";
+// src/engine/reading-failure.ts
 /**
-* A path as this system names it: absolute, written with its own
-* separator, `.` and `..` resolved, and on Windows out of the namespace
-* `path.toNamespacedPath` writes it in (`\\?\D:\app` is `D:\app`). The code
-* a reading runs may name one file in any of these ways; a watcher reports
-* it in this one.
+* Why a reading of the routes failed, as a `RoutesReadError` whatever it
+* threw. The engine's own failure is kept as it is, its text the engine's
+* words and what the loader quoted of the project, masked already. Anything
+* else came from the project's code, a getter of its config or a source the
+* core read, and is `CONFIG_NOT_LOADED`, its text and stack with no secret
+* left in them: a bundler prints them, shows them on the page, and a CI
+* log, which may be public, keeps them.
 *
-* @param path - The path, absolute or from the working folder
-* @returns The path
+* @param thrown - What the reading threw
+* @param context - The project's root, and the config file once found
+* @param inputs - What the reading had read, for a failure that names none yet
+* @returns The failure
 * @example
 * ```ts
-* localPath("/app/src/../flags.json"); // "/app/flags.json"
+* readingFailure(new Error(`bad ${process.env.API_KEY}`), { root: "/app" }).message;
+* // → "The routes a page takes were not read: bad [redacted]"
 * ```
 */
-function localPath(path) {
-	if (sep === "\\" && path.startsWith(SHARE_NAMESPACE)) return resolve(`\\\\${path.slice(8)}`);
-	if (sep === "\\" && path.startsWith(NAMESPACE)) return resolve(path.slice(4));
-	return resolve(path);
+function readingFailure(thrown, context, inputs) {
+	if (thrown instanceof RoutesReadError) {
+		if (thrown.inputs !== void 0 || inputs === void 0) return thrown;
+		return new RoutesReadError(thrown.code, thrown.message, thrown.context, {
+			stack: thrown.stack,
+			inputs
+		});
+	}
+	const masked = maskedError(thrown);
+	return new RoutesReadError("CONFIG_NOT_LOADED", `The routes a page takes were not read: ${masked.message}`, context, {
+		stack: thrown instanceof Error ? masked.stack : void 0,
+		inputs
+	});
 }
 // src/engine/track-env-reads.ts
 /**
@@ -468,7 +482,11 @@ async function readProjectRoutes(request) {
 			routes
 		};
 	} catch (error) {
-		throw withInputs(error, await reading.inputs(), request);
+		const inputs = await reading.inputs();
+		throw readingFailure(error, {
+			root: request.root,
+			configFile: inputs.configFile
+		}, inputs);
 	}
 }
 async function routesOfSources(loaded, root) {
@@ -495,21 +513,6 @@ async function projectRoutesOf(configFile, root) {
 		configFile
 	});
 }
-function withInputs(error, inputs, request) {
-	const stack = error instanceof Error ? error.stack : void 0;
-	if (!(error instanceof RoutesReadError)) return new RoutesReadError("CONFIG_NOT_LOADED", `The routes a page takes were not read: ${error instanceof Error ? error.message : String(error)}`, {
-		root: request.root,
-		configFile: inputs.configFile
-	}, {
-		stack,
-		inputs
-	});
-	if (error.inputs !== void 0) return error;
-	return new RoutesReadError(error.code, error.message, error.context, {
-		stack,
-		inputs
-	});
-}
 function byCodeUnits(a, b) {
 	return a < b ? -1 : a > b ? 1 : 0;
 }
@@ -529,23 +532,17 @@ async function answer(request) {
 	process.send?.(workerMessages.reply(reply), () => process.exit(0));
 }
 function failure(error, request) {
-	if (error instanceof RoutesReadError) return {
-		ok: false,
-		code: error.code,
-		message: error.message,
-		context: error.context,
-		stack: error.stack,
-		inputs: error.inputs
-	};
+	const failed = readingFailure(error, {
+		root: request.root,
+		configFile: request.configFile
+	});
 	return {
 		ok: false,
-		code: "CONFIG_NOT_LOADED",
-		message: `The routes a page takes were not read: ${error instanceof Error ? error.message : String(error)}`,
-		context: {
-			root: request.root,
-			configFile: request.configFile
-		},
-		stack: error instanceof Error ? error.stack : void 0
+		code: failed.code,
+		message: failed.message,
+		context: failed.context,
+		stack: failed.stack,
+		inputs: failed.inputs
 	};
 }
 export {};

package/lib/esbuild/index.js

@@ -1,11 +1,11 @@
 import { t as RoutesReadError } from "../routes-read-error-DQhKIE2F.js";
-import { n as routesModuleCode } from "../routes-module-code-DO3aII6v.js";
+import { n as routesModuleCode } from "../routes-module-code-DYtP7uj1.js";
 import { t as configInPageMessage } from "../config-in-page-message-DLd3f0Jd.js";
-import { d as CONFIG_BASENAMES } from "../load-config-BeedbpkN.js";
-import { t as sharedEngine } from "../shared-engine-CCu7vP_0.js";
-import { t as configFileOf } from "../config-file-of-Zkwyi9zf.js";
+import { w as CONFIG_BASENAMES } from "../load-config-barNPfXN.js";
+import { t as sharedEngine } from "../shared-engine-D0j7uXg8.js";
+import { t as configFileOf } from "../config-file-of-Clf_dWUQ.js";
 import { n as ROUTES_PATTERN, r as ROUTES_SPECIFIER } from "../routes-module-id-CnUjQjYF.js";
-import { t as routesFile } from "../routes-file-DcpgxYb1.js";
+import { t as routesFile } from "../routes-file-DRoqUZ0S.js";
 import { basename, extname } from "node:path";
 // src/esbuild/config-in-page.ts
 const RESOLVING = "@noverax/courier-build:resolving-config";

package/lib/index.js

@@ -1,4 +1,4 @@
 import { t as RoutesReadError } from "./routes-read-error-DQhKIE2F.js";
-import { i as createRoutesEngine, n as routesModuleCode, r as unreadRoutesModuleCode, t as routesCommonJsCode } from "./routes-module-code-DO3aII6v.js";
+import { i as createRoutesEngine, n as routesModuleCode, r as unreadRoutesModuleCode, t as routesCommonJsCode } from "./routes-module-code-DYtP7uj1.js";
 import { t as configInPageMessage } from "./config-in-page-message-DLd3f0Jd.js";
 export { RoutesReadError, configInPageMessage, createRoutesEngine, routesCommonJsCode, routesModuleCode, unreadRoutesModuleCode };

package/lib/jest/routes-thread.js

@@ -1,6 +1,6 @@
-import { n as routesModuleCode, t as routesCommonJsCode } from "../routes-module-code-DO3aII6v.js";
+import { n as routesModuleCode, t as routesCommonJsCode } from "../routes-module-code-DYtP7uj1.js";
 import { n as isRecord } from "../errno-code-CNx_E80R.js";
-import { t as sharedEngine } from "../shared-engine-CCu7vP_0.js";
+import { t as sharedEngine } from "../shared-engine-D0j7uXg8.js";
 import { parentPort } from "node:worker_threads";
 // src/jest/routes-thread.ts
 parentPort?.on("message", (message) => {