@escape-game-over/atlas 0.1.24 → 0.1.26

package/src/warn.ts

@@ -2,22 +2,16 @@
  * The one place lib writes to a console.
  *
  * Everything here is a *warning* by definition: a fact about a file or a string
- * that no type can see, worth saying out loud but never worth failing a build
- * over. Errors throw instead, and say so at the point they are wrong.
+ * that no type can see. Errors throw instead, and say so at the point they are
+ * wrong. `ATLAS_STRICT=1` makes every warning throw too, for CI.
  *
- * Shared so the colouring and the `[lib]` prefix are decided once — a warning
+ * Shared so the colouring and the `[atlas]` prefix are decided once — a warning
  * that looks different depending on which module raised it reads like two
  * different tools.
  */
 
-/**
- * What every line lib writes is labelled with.
- *
- * A constant rather than a literal at each call site: grep for it to find every
- * warning the library can raise, and change it once if this folder is ever
- * published under its real name.
- */
-const PREFIX = "lib";
+/** What every line lib writes is labelled with. */
+const PREFIX = "atlas";
 
 /**
  * The escape character, named rather than embedded. 27 is its code point.
@@ -49,12 +43,16 @@ const RESET = `${ESC}[0m`;
  * `process` and simply gets colour, which is what a browser console renders
  * anyway.
  */
-const colour =
-    (
-        globalThis as {
-            process?: { env?: Readonly<Record<string, string | undefined>> };
-        }
-    ).process?.env?.NO_COLOR === undefined;
+const env = (
+    globalThis as {
+        process?: { env?: Readonly<Record<string, string | undefined>> };
+    }
+).process?.env;
+
+const colour = env?.NO_COLOR === undefined;
+
+/** Read per call rather than once, so a test can switch it. */
+const strict = (): boolean => env?.ATLAS_STRICT === "1";
 
 const paint = (code: string, text: string): string =>
     colour ? `${code}${text}${RESET}` : text;
@@ -66,6 +64,7 @@ const paint = (code: string, text: string): string =>
  * always printed, because a warning nobody can locate is one nobody acts on.
  */
 export function warn(at: string, message: string): void {
+    if (strict()) throw new Error(`${shorten(at)}: ${message}`);
     console.warn(
         `${paint(YELLOW, `[${PREFIX}] [WARN]`)} ${paint(DIM, shorten(at))}\n  ${message}`
     );

package/src/astro/dom.ts

@@ -1,35 +0,0 @@
-/**
- * Scoped element lookups, typed.
- *
- * ```ts
- * const { one, all } = within(root);
- * const dots = all<HTMLButtonElement>("[data-carousel-dot]");
- * ```
- *
- * The root is bound once, so a script cannot reach a second copy of itself
- * elsewhere on the page. `document` belongs in one place per script: finding
- * the roots.
- *
- * **A selector with a combinator still matches against the whole document.**
- * `within(form).one("form p")` can match a `<p>` under a different form; only
- * the final filter is scoped. Use `:scope` if a combinator is needed.
- */
-
-export interface Within {
-    /** The first match inside the root, or `null`. */
-    one<T extends Element = HTMLElement>(selector: string): T | null;
-    /** Every match inside the root, as an array rather than a `NodeList`. */
-    all<T extends Element = HTMLElement>(selector: string): T[];
-}
-
-// `HTMLElement` by default because `hidden`, `dataset` and `focus` live there;
-// the bound stays `Element` so inline `<svg>` lookups are not refused.
-export function within(root: ParentNode): Within {
-    return {
-        one: <T extends Element = HTMLElement>(selector: string) =>
-            root.querySelector<T>(selector),
-        all: <T extends Element = HTMLElement>(selector: string) => [
-            ...root.querySelectorAll<T>(selector),
-        ],
-    };
-}