@theme-registry/refract 0.1.1 → 0.1.2

package/dist/cli.js

@@ -6118,6 +6118,33 @@ const resolveReferences = (value, tokensByPath, visited = new Set()) => {
     return value;
 };
 
+/**
+ * Fail-loud shape guard for a value that is *supposed* to be a {@link RawTheme}.
+ *
+ * `createTheme` accepts a bare object, and an empty `{}` is a valid (empty) theme — so a value of the
+ * *wrong* shape doesn't necessarily throw during a build. That's the trap `refract diff` fell into: hand
+ * it a `defineConfig({ raw, targets })` where it wanted the raw theme and it built the config as an
+ * (effectively empty) theme, then reported a nonsense "every token removed" diff and exited 0. A
+ * governance tool must not quietly mis-report. This guard is the loud, coded gate for that class of
+ * mistake — used by `diff` (and available to `validate` / the MCP server) before anything is compared.
+ *
+ * It is deliberately narrow: it rejects only values that cannot be a RawTheme (non-objects, arrays,
+ * `null`) or that are affirmatively something else (a `defineConfig`, spotted by its `targets` array).
+ * A bare `{}` still passes — an empty theme is legitimately empty.
+ */
+function assertRawTheme(value, source) {
+    const at = source ? ` (${source})` : "";
+    if (value === null || typeof value !== "object" || Array.isArray(value)) {
+        const got = Array.isArray(value) ? "an array" : value === null ? "null" : typeof value;
+        throw new RefractError("REFRACT_E_RAW_SHAPE", `Expected a RawTheme object${at}, got ${got}. Pass the raw theme (a module's default export, or a .json raw theme).`);
+    }
+    // The documented mistake: a defineConfig({ raw, targets }) passed where the bare RawTheme was wanted.
+    // A RawTheme never has a top-level `targets`; a config always does.
+    if (Array.isArray(value.targets)) {
+        throw new RefractError("REFRACT_E_RAW_SHAPE", `Expected a RawTheme object${at}, but got what looks like a defineConfig({ raw, targets }). Pass its \`raw\` (or a module that default-exports the raw theme), not the whole config.`);
+    }
+}
+
 /**
  * `defineAdapter` — turns an `AdapterSpec` (identity + a `bind` returning the four
  * required primitives) into a full `ThemeAdapter` whose `bind` yields a complete
@@ -7171,8 +7198,10 @@ export default defineConfig({
   raw: {
     breakpoints: { sm: 576, md: 768, lg: 1024, xl: 1280 },
     colors: {
-      primary: { base: "#4dabf7", text: "#fff", variants: { dark: "#1c7ed6", light: "#a5d8ff" } },
-      neutral: { base: "#868e96", text: "#fff", variants: { light: "#f1f3f5", dark: "#343a40" } },
+      // Starter palette passes its own contrast audit (WCAG AA): white text clears 4.5:1 on both
+      // bases. Keep that when you retune — the auditor is a shipped feature; the default should model it.
+      primary: { base: "#1864ab", text: "#fff", variants: { dark: "#0b4a86", light: "#a5d8ff" } },
+      neutral: { base: "#495057", text: "#fff", variants: { light: "#f1f3f5", dark: "#212529" } },
       recipes: {
         solid: {
           primary: { background: "primary", color: "primary.text" },
@@ -7745,26 +7774,35 @@ function diffThemes(base, candidate) {
  */
 const WCAG_RANK = { fail: 0, "AA-large": 1, AA: 2, AAA: 3 };
 let counter = 0;
-/** Load a candidate raw theme from a module (default export) or a `.json` file. */
+/**
+ * Load a candidate raw theme from a module (default export) or a `.json` file, and assert it's actually
+ * theme-shaped before returning — a mis-shaped candidate (e.g. a `defineConfig`) must fail loud here,
+ * not silently diff as "everything removed" (see {@link assertRawTheme}).
+ */
 async function loadCandidate(path) {
     var _a, _b, _c, _d;
     if (!node_fs.existsSync(path))
         throw new Error(`candidate theme not found at "${path}".`);
+    let candidate;
     if (node_path.extname(path) === ".json") {
-        return JSON.parse(node_fs.readFileSync(path, "utf8"));
+        candidate = JSON.parse(node_fs.readFileSync(path, "utf8"));
     }
-    if (node_path.extname(path) === ".ts") {
+    else if (node_path.extname(path) === ".ts") {
         const { entry, cleanup } = await compileTsConfigGraph(path);
         try {
             const mod = (await import(node_url.pathToFileURL(entry).href));
-            return ((_b = (_a = mod.default) !== null && _a !== void 0 ? _a : mod.raw) !== null && _b !== void 0 ? _b : mod);
+            candidate = (_b = (_a = mod.default) !== null && _a !== void 0 ? _a : mod.raw) !== null && _b !== void 0 ? _b : mod;
         }
         finally {
             cleanup();
         }
     }
-    const mod = (await import(`${node_url.pathToFileURL(path).href}?v=${++counter}`));
-    return ((_d = (_c = mod.default) !== null && _c !== void 0 ? _c : mod.raw) !== null && _d !== void 0 ? _d : mod);
+    else {
+        const mod = (await import(`${node_url.pathToFileURL(path).href}?v=${++counter}`));
+        candidate = (_d = (_c = mod.default) !== null && _c !== void 0 ? _c : mod.raw) !== null && _d !== void 0 ? _d : mod;
+    }
+    assertRawTheme(candidate, path);
+    return candidate;
 }
 /** Does this built theme expose the class-emitting surface (CSS-style adapter)? */
 const emitsClasses = (theme) => typeof theme.renderRecipe === "function";
@@ -7868,6 +7906,21 @@ Commands:
   audit    Score colour pairings for WCAG-2 contrast (+ advisory APCA). Reports; --strict fails.
   skills   Install the bundled AI skills into your agent CLI(s) (claude/codex/…).
 `;
+/**
+ * Uniform error reporter for a command's catch block. Surfaces a `RefractError`'s stable `code`
+ * (`[REFRACT_E_…] message`) so an agent or CI log sees the machine-readable code, and lists each
+ * collect-all `failure` — a plain `Error` just prints its message. Returns the exit code (1).
+ */
+function reportError(cmd, err) {
+    var _a;
+    const e = err;
+    const code = typeof e.code === "string" && e.code.startsWith("REFRACT_E_") ? `[${e.code}] ` : "";
+    process.stderr.write(`refract ${cmd}: ${code}${(_a = e.message) !== null && _a !== void 0 ? _a : String(err)}\n`);
+    if (e.failures)
+        for (const f of e.failures)
+            process.stderr.write(`  - ${f}\n`);
+    return 1;
+}
 async function cmdInit(argv) {
     const { values } = node_util.parseArgs({
         args: argv,
@@ -7890,8 +7943,7 @@ async function cmdInit(argv) {
         return 0;
     }
     catch (err) {
-        process.stderr.write(`refract init: ${err.message}\n`);
-        return 1;
+        return reportError("init", err);
     }
 }
 async function cmdImport(argv) {
@@ -7935,8 +7987,7 @@ async function cmdImport(argv) {
         return 0;
     }
     catch (err) {
-        process.stderr.write(`refract import: ${err.message}\n`);
-        return 1;
+        return reportError("import", err);
     }
 }
 async function cmdBuild(argv) {
@@ -7965,8 +8016,7 @@ async function cmdBuild(argv) {
         return 0;
     }
     catch (err) {
-        process.stderr.write(`refract build: ${err.message}\n`);
-        return 1;
+        return reportError("build", err);
     }
 }
 async function cmdTokens(argv) {
@@ -7988,8 +8038,7 @@ async function cmdTokens(argv) {
         return 0;
     }
     catch (err) {
-        process.stderr.write(`refract tokens: ${err.message}\n`);
-        return 1;
+        return reportError("tokens", err);
     }
 }
 const WCAG_LEVELS = new Set(["AAA", "AA", "AA-large"]);
@@ -8046,8 +8095,7 @@ async function cmdDiff(argv) {
         return 0;
     }
     catch (err) {
-        process.stderr.write(`refract diff: ${err.message}\n`);
-        return 1;
+        return reportError("diff", err);
     }
 }
 async function cmdAudit(argv) {
@@ -8089,8 +8137,7 @@ async function cmdAudit(argv) {
         return 0;
     }
     catch (err) {
-        process.stderr.write(`${err.message}\n`);
-        return 1;
+        return reportError("audit", err);
     }
 }
 /** Parse an `--agent` value (`"all"` or a comma list) into validated targets. */
@@ -8142,8 +8189,7 @@ async function cmdSkills(argv) {
             return 0;
         }
         catch (err) {
-            process.stderr.write(`refract skills list: ${err.message}\n`);
-            return 1;
+            return reportError("skills list", err);
         }
     }
     const { values } = node_util.parseArgs({
@@ -8202,8 +8248,7 @@ async function cmdSkills(argv) {
         return 1;
     }
     catch (err) {
-        process.stderr.write(`refract skills: ${err.message}\n`);
-        return 1;
+        return reportError("skills", err);
     }
 }
 /** Dispatch a parsed argv (without the node/script prefix). Returns a process exit code. */