@escape-game-over/atlas 0.1.93 → 0.1.94

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@escape-game-over/atlas",
-    "version": "0.1.93",
+    "version": "0.1.94",
     "type": "module",
     "description": "Typed, data-driven machinery for static multi-locale, multi-deployment Astro sites.",
     "private": false,

package/src/analytics/google.ts

@@ -275,13 +275,19 @@ const CATEGORY_SIGNALS: Readonly<
     marketing: ["adStorage", "adUserData", "adPersonalization"],
 };
 
-/** The signals no answer grants — see `CATEGORY_SIGNALS`. */
-const UNASKED = SIGNAL_KEYS.filter(
-    (key) =>
-        !CONSENT_CATEGORIES.some((category) =>
-            CATEGORY_SIGNALS[category].includes(key)
-        )
-);
+/**
+ * The signals no answer grants — see `CATEGORY_SIGNALS`. A function, not a
+ * value worked out at import: browser bundles import this file for its
+ * consent constants and would otherwise compute it on every page.
+ */
+function unasked(): readonly (keyof ConsentDefaults)[] {
+    return SIGNAL_KEYS.filter(
+        (key) =>
+            !CONSENT_CATEGORIES.some((category) =>
+                CATEGORY_SIGNALS[category].includes(key)
+            )
+    );
+}
 
 /**
  * What `google-tag.ts` is handed: the settings, already in Google's own names.
@@ -435,7 +441,7 @@ export function googleScripts(
     }
 
     const consent = google.consent ?? DENIED_BY_DEFAULT;
-    for (const key of UNASKED) {
+    for (const key of unasked()) {
         if (consent.some((given) => given[key] === "denied")) {
             warn(
                 at,

package/src/warn.ts

@@ -43,19 +43,22 @@ const RESET = `${ESC}[0m`;
  * `process` and simply gets colour, which is what a browser console renders
  * anyway.
  */
-const env = (
-    globalThis as {
-        process?: { env?: Readonly<Record<string, string | undefined>> };
-    }
-).process?.env;
+const env = () =>
+    (
+        globalThis as {
+            process?: { env?: Readonly<Record<string, string | undefined>> };
+        }
+    ).process?.env;
 
-const colour = env?.NO_COLOR === undefined;
+// Functions, not values worked out at import: a browser bundle that imports
+// anything near this file would otherwise run the reads on every page.
+const colour = (): boolean => env()?.NO_COLOR === undefined;
 
 /** Read per call rather than once, so a test can switch it. */
-const strict = (): boolean => env?.ATLAS_STRICT === "1";
+const strict = (): boolean => env()?.ATLAS_STRICT === "1";
 
 const paint = (code: string, text: string): string =>
-    colour ? `${code}${text}${RESET}` : text;
+    colour() ? `${code}${text}${RESET}` : text;
 
 /**
  * Warns about one page, or one file.