synthesisui 0.16.33 → 0.16.34

package/dist/commands/doctor.js

@@ -8,7 +8,7 @@ import { checkableName, readRequests, verifyAndCloseRequests, } from "../doctor/
 import { diagnose, scanSource, siblingTokens, } from "../doctor/scan.js";
 import { findSelfConflicts, forbiddenProps, isReset, propMatchesLabel, } from "../doctor/self-conflict.js";
 import { buildTable, EMPTY_TABLE, nearestToken, } from "../doctor/tokens.js";
-import { body, section, snippet } from "../output.js";
+import { body, paint, section, snippet } from "../output.js";
 /**
  * `synthesisui doctor` - the check nobody else ships.
  *
@@ -43,7 +43,11 @@ const SKIP = new Set([
     // Our own installed artifacts are the answer, not the problem.
     "_synthesisui",
 ]);
-export async function* walk(dir) {
+export async function* walk(dir, 
+/** Filled with nested projects that were SKIPPED - each speaks its own
+ *  system and must not be billed against this one (30/07: the monorepo
+ *  run charged aluna-landing's tokens against the root's harvest). */
+skippedProjects) {
     // `readdir`'s overloads infer a Buffer-named Dirent without an explicit
     // encoding; naming it keeps `e.name` a string.
     const entries = await readdir(dir, {
@@ -57,7 +61,14 @@ export async function* walk(dir) {
         if (e.isDirectory()) {
             if (SKIP.has(e.name))
                 continue;
-            yield* walk(full);
+            // A CHILD directory carrying its own _synthesisui is a separately
+            // governed project - its record, its doctor, its numbers.
+            const governed = await readdir(join(full, "_synthesisui")).then(() => true, () => false);
+            if (governed) {
+                skippedProjects?.push(full);
+                continue;
+            }
+            yield* walk(full, skippedProjects);
         }
         else if (EXTS.some((x) => e.name.endsWith(x))) {
             yield full;
@@ -199,7 +210,7 @@ const plural = (n, one, many = `${one}s`) => `${n} ${n === 1 ? one : many}`;
 /** A bar you can read at a glance, in a font that is always monospace. */
 function meter(pct, width = 24) {
     const filled = Math.round((pct / 100) * width);
-    return `${"█".repeat(filled)}${"░".repeat(width - filled)}`;
+    return `${paint.blue("█".repeat(filled))}${paint.faint("░".repeat(width - filled))}`;
 }
 function verdict(d, hasSystem, overruled, conflicts) {
     // A system at war with itself outranks every other number here. The closing
@@ -368,7 +379,10 @@ export async function doctor(opts) {
      * Nobody debugs from 0%. They conclude the product does not work.
      */
     const wiring = await readWiring(root, table.slug);
-    for await (const file of scopes.length > 0 ? walkAll(scopes) : walk(root)) {
+    const skippedProjects = [];
+    for await (const file of scopes.length > 0
+        ? walkAll(scopes)
+        : walk(root, skippedProjects)) {
         const src = await readFile(file, "utf8").catch(() => "");
         if (!src)
             continue;
@@ -442,7 +456,11 @@ export async function doctor(opts) {
         // `var(--ds-color-semantic-knob, #ffffff)` - is set aside for the opposite
         // reason: it is already tokenized. The summary said the false half out
         // loud and hid the true half behind a flag.
-        console.log(body(`set aside ${plural(asideTotal, "value")} that are not drift (--verbose for why)`));
+        console.log(body(`set aside ${plural(asideTotal, "value")} that are not drift` +
+            (skippedProjects.length > 0
+                ? ` and ${plural(skippedProjects.length, "nested project")} that speak their own systems`
+                : "") +
+            " (--verbose for why)"));
     }
     // 0 of 0 is not a perfect score, it is an empty measurement - printing a
     // full bar there would be the report's first lie.
@@ -522,8 +540,19 @@ export async function doctor(opts) {
     }
     if (hasSystem && measurable) {
         console.log("");
-        console.log(body(`Token coverage  ${meter(d.coverage)}  ${String(d.coverage).padStart(3)}%`));
-        console.log(body(`                ${d.tokenUses} from the system, ${d.findings.length} by hand${d.phantomUses > 0 ? `, ${d.phantomUses} naming nothing` : ""}`));
+        console.log(body(`Token coverage  ${meter(d.coverage)}  ${paint.strong(`${String(d.coverage).padStart(3)}%`)}`));
+        console.log(body(paint.dim(`                ${d.tokenUses} from the system, ${d.findings.length} by hand${d.phantomUses > 0 ? `, ${d.phantomUses} naming nothing` : ""}`)));
+        // The strongest number leads, not trails: it used to sit two screens
+        // down, after the phantom list (30/07). For the reader who already owns
+        // a system - the ICP - THIS line is the report.
+        if (table.source === "yours" && d.findings.length > 0) {
+            const named = d.findings.filter((f) => f.token).length;
+            if (named > 0) {
+                console.log("");
+                console.log(body(`${paint.strong(String(named))} of the ${d.findings.length} hand-written values already have a name in YOUR system.`));
+                console.log(body(paint.dim("Your agent has no way to know: the tokens exist, the contract does not.")));
+            }
+        }
     }
     /**
      * THE RECORD - the layer that remembers, printed where the moment is shown.
@@ -635,13 +664,21 @@ export async function doctor(opts) {
         console.log(body(`${plural(phantoms.size, "name")} written as tokens that your system never declares.`));
         console.log(body("An undeclared custom property applies nothing at all."));
         console.log("");
-        for (const [name, at] of [...phantoms].sort()) {
+        // Capped like every other section: 99 lines is a report nobody reads,
+        // and it buried the strongest number two screens down (30/07). The full
+        // list keeps living behind --verbose.
+        const PHANTOM_CAP = 8;
+        const sorted = [...phantoms].sort();
+        for (const [name, at] of sorted.slice(0, PHANTOM_CAP)) {
             const where = [...at.files][0];
-            console.log(body(`${name}   ${where}:${at.line}${at.files.size > 1 ? ` +${at.files.size - 1} more` : ""}`));
+            console.log(body(`${name}   ${paint.dim(`${where}:${at.line}${at.files.size > 1 ? ` +${at.files.size - 1} more` : ""}`)}`));
             const kin = siblingTokens(name, table);
             if (kin.length > 0)
                 console.log(snippet([`your system has ${kin.join(", ")}`]));
         }
+        if (sorted.length > PHANTOM_CAP) {
+            console.log(body(paint.dim(`and ${sorted.length - PHANTOM_CAP} more · synthesisui doctor --verbose`)));
+        }
         console.log("");
         console.log(body("Add the name to the system, or use one it has. Do not leave it."));
     }
@@ -1009,11 +1046,16 @@ export async function doctor(opts) {
         }
         if (d.findings.length > 0 || overrides.length > 0) {
             console.log("");
-            console.log(body(`${plural(d.findings.length, "value")} by hand · ${d.named} already ${d.named === 1 ? "has" : "have"} a name`));
-            console.log(body(`${plural(overrides.length - lawKeepingCount, "override")} · ${offSystemCount} left the system` +
-                (lawKeepingCount > 0
-                    ? ` · ${lawKeepingCount} more kept a law`
-                    : "")));
+            if (table.source !== "yours") {
+                console.log(body(`${plural(d.findings.length, "value")} by hand · ${d.named} already ${d.named === 1 ? "has" : "have"} a name`));
+            }
+            // An all-zero status line is filler, not information.
+            if (overrides.length > 0 || offSystemCount > 0) {
+                console.log(body(`${plural(overrides.length - lawKeepingCount, "override")} · ${offSystemCount} left the system` +
+                    (lawKeepingCount > 0
+                        ? ` · ${lawKeepingCount} more kept a law`
+                        : "")));
+            }
         }
         if (plan.length > 0) {
             console.log("");
@@ -1021,7 +1063,7 @@ export async function doctor(opts) {
             const w = Math.min(52, Math.max(...plan.slice(0, 5).map((j) => j.what.length)));
             plan.slice(0, 5).forEach((j, i) => {
                 const what = j.what.length > w ? `${j.what.slice(0, w - 1)}…` : j.what.padEnd(w);
-                console.log(`      ${i + 1}. ${what}  ${j.size}${j.cheap ? "  (cheap)" : ""}`);
+                console.log(`      ${i + 1}. ${what}  ${paint.dim(`${j.size}${j.cheap ? "  (cheap)" : ""}`)}`);
             });
         }
         console.log("");
@@ -1040,15 +1082,6 @@ export async function doctor(opts) {
          * thread, which is the other job this paragraph does.
          */
         if (table.source === "yours") {
-            const named = d.findings.filter((f) => f.token).length;
-            const anonymous = d.findings.length - named;
-            console.log("");
-            console.log(body(named > 0
-                ? `${named} of these already have a name in your own system.`
-                : "None of these have a name in your system yet."));
-            if (anonymous > 0) {
-                console.log(body(`The other ${anonymous} do not - and your coding agent has no way to know they should.`));
-            }
             console.log("");
             // Careful about what is being offered. This reader ALREADY has a design
             // system, so "install ours" would mean replacing theirs, and a CTA that
@@ -1058,9 +1091,9 @@ export async function doctor(opts) {
             console.log(body("Your tokens exist. What your agent is missing is a contract:"));
             console.log(body("rules it reads BEFORE writing UI, and a manifest of what exists."));
             console.log("");
-            console.log(snippet(["npx synthesisui@latest init --ds <slug>"]));
-            console.log(body("starts you on a system that ships one - browse them at"));
-            console.log(body("https://www.synthesisui.com/gallery"));
+            console.log(paint.blue(snippet(["npx synthesisui@latest init --ds <slug>"])));
+            console.log(body(paint.dim("starts you on a system that ships one - browse them at")));
+            console.log(body(paint.blue("https://www.synthesisui.com/gallery")));
         }
         console.log("");
     }

package/dist/doctor/scan.js

@@ -231,7 +231,35 @@ function fallbackSpans(line, table) {
 const DECLARES_TOKEN = /^\s*(--[a-z0-9_-]+)\s*:/i;
 /** Under this, a radius or spacing value is idiom rather than a decision. */
 const IDIOM = new Set(["0", "0px", "1px", "9999px", "100%", "50%"]);
+/** Test fixtures speak in example names on purpose - `--ds-nope` inside a
+ *  spec is the TEST, not drift. Every real project has these; accusing them
+ *  buried the report's real findings under 99 phantom lines the first time
+ *  the doctor ran on a repo with its own test suite (30/07). */
+const IS_FIXTURE = /(\.(spec|test)\.[a-z]+$|__tests__\/|\/fixtures?\/)/;
 export function scanSource(file, source, table) {
+    if (IS_FIXTURE.test(file)) {
+        const real = scanCore(file, source, table);
+        const aside = real.findings.length +
+            (real.phantoms?.length ?? 0) +
+            (real.setAside?.reduce((n, a) => n + a.count, 0) ?? 0);
+        return {
+            file,
+            findings: [],
+            // Neither accused nor credited: fixture var() uses would inflate
+            // coverage exactly as unfairly as fixture hexes would deflate it.
+            tokenUses: 0,
+            ...(aside > 0
+                ? {
+                    setAside: [
+                        { reason: "test fixtures speak in example names", count: aside },
+                    ],
+                }
+                : null),
+        };
+    }
+    return scanCore(file, source, table);
+}
+function scanCore(file, source, table) {
     const findings = [];
     const phantoms = [];
     let tokenUses = 0;

package/dist/output.js

@@ -10,7 +10,7 @@ const WIDTH = 66;
 export function section(title) {
     const head = `── ${title} `;
     const rest = Math.max(4, WIDTH - head.length);
-    return `\n${head}${"─".repeat(rest)}\n`;
+    return `\n${head}${paint.faint("─".repeat(rest))}\n`;
 }
 /** Indented, copy-pasteable code block (empty lines stay truly empty). */
 export function snippet(lines) {
@@ -20,3 +20,27 @@ export function snippet(lines) {
 export function body(line) {
     return `  ${line}`;
 }
+/**
+ * Colour, used the way the product uses motion: a quiet base and a few
+ * deliberate accents (dono, 30/07: "um cinza mais escuro e um mais claro,
+ * e o coverage com um azul bem discreto"). Two grays carry hierarchy -
+ * chrome/meta in the darker one, secondary prose in the lighter - and one
+ * muted steel blue marks what is ALIVE: the coverage bar, commands, links.
+ *
+ * Off out of respect wherever colour is not wanted: piped output (no TTY),
+ * NO_COLOR, or a dumb terminal - assertions in tests never see a code.
+ */
+const COLOR_ON = Boolean(process.stdout.isTTY) &&
+    !process.env.NO_COLOR &&
+    process.env.TERM !== "dumb";
+const wrap = (code) => (s) => COLOR_ON ? `\x1b[${code}m${s}\x1b[0m` : s;
+export const paint = {
+    /** Lighter gray - secondary prose, explanations, counts. */
+    dim: wrap("38;5;246"),
+    /** Darker gray - chrome: rules, meters' empty track, set-aside notes. */
+    faint: wrap("38;5;240"),
+    /** Discreet steel blue - the coverage fill, commands, links. */
+    blue: wrap("38;5;67"),
+    /** Bold - the one or two numbers that carry the report. */
+    strong: wrap("1"),
+};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "synthesisui",
-  "version": "0.16.33",
+  "version": "0.16.34",
   "description": "Bring SynthesisUI design systems into any project - tokens, typed components, whole pages and an agent-ready CLAUDE.md manifest.",
   "type": "module",
   "bin": {
@@ -29,7 +29,7 @@
     "node": ">=18"
   },
   "scripts": {
-    "build": "tsc -p tsconfig.json",
+    "build": "tsc -p tsconfig.json && chmod +x dist/index.js",
     "dev": "tsx src/index.ts",
     "prepublishOnly": "npm run build",
     "test": "vitest run"