synthesisui 0.16.4 → 0.16.6

package/dist/commands/doctor.js

@@ -264,6 +264,53 @@ function verdict(d, hasSystem, overruled, conflicts) {
     }
     return lines;
 }
+/**
+ * Wiring is a property of the PROJECT, never of the scope being read.
+ *
+ * This ran inside the scoped walk, so `doctor app/login` read only that folder
+ * - and the import lives in `app/globals.css`. It concluded "installed, but not
+ * wired up yet" about a project that was perfectly wired (my-test4, 27/07).
+ *
+ * Which would be a mild annoyance, except the managed block now tells the agent
+ * to run `doctor <the file you just wrote>` after every edit. The false alarm
+ * fires on every loop, and an agent that believes the system is unwired starts
+ * repairing wiring that was already correct - editing the one file we most need
+ * it to leave alone.
+ *
+ * The system itself was always resolved from the root. So is this now.
+ */
+async function readWiring(root, slug) {
+    const w = {
+        imported: false,
+        scoped: false,
+        /** `init` wrote a fonts file for this project (Next targets only). */
+        fontsWritten: false,
+        /** …and the stylesheet actually maps it onto the system's type tokens. */
+        fontsMapped: false,
+    };
+    if (!slug)
+        return w;
+    for await (const file of walk(root)) {
+        const src = await readFile(file, "utf8").catch(() => "");
+        if (!src)
+            continue;
+        if (src.includes(`_synthesisui/ds/${slug}/tokens.css`))
+            w.imported = true;
+        if (src.includes(`data-ds="${slug}"`))
+            w.scoped = true;
+        // The requirement that stayed invisible. `init` writes a fonts file and
+        // asks for two more edits; an agent told to check only the first two did
+        // exactly that, stopped, and left the project rendering in the framework's
+        // default face (my-test2, 27/07). What the checker checks is what gets done.
+        if (src.includes("--font-ds-") && /next\/font/.test(src))
+            w.fontsWritten = true;
+        if (/--ds-typography-families-\w+\s*:\s*var\(\s*--font-ds-/.test(src))
+            w.fontsMapped = true;
+        if (w.imported && w.scoped && w.fontsMapped)
+            break;
+    }
+    return w;
+}
 export async function doctor(opts) {
     const root = resolve(opts.dir ?? process.cwd());
     const scopes = (opts.scopes ?? []).map((s) => resolve(root, s));
@@ -292,20 +339,12 @@ export async function doctor(opts) {
      *
      * Nobody debugs from 0%. They conclude the product does not work.
      */
-    const wiring = { imported: false, scoped: false };
+    const wiring = await readWiring(root, table.slug);
     for await (const file of scopes.length > 0 ? walkAll(scopes) : walk(root)) {
         const src = await readFile(file, "utf8").catch(() => "");
         if (!src)
             continue;
         const rel = relative(root, file);
-        // The two facts that decide whether an installed system reaches the
-        // browser at all. Free: these files are already open and in memory.
-        if (table.slug) {
-            if (src.includes(`_synthesisui/ds/${table.slug}/tokens.css`))
-                wiring.imported = true;
-            if (src.includes(`data-ds="${table.slug}"`))
-                wiring.scoped = true;
-        }
         reports.push(scanSource(rel, src, table));
         if (recipes.size > 0) {
             for (const o of findOverrides(src, recipes))
@@ -378,7 +417,11 @@ export async function doctor(opts) {
     // Before the number, because the number is the thing that misleads. An
     // installed-but-unwired system reads 0%, and 0% reads as "broken product"
     // rather than "one import missing".
-    const unwired = table.source === "installed" && (!wiring.imported || !wiring.scoped);
+    // Fonts only count as missing when `init` actually wrote the file - a
+    // non-Next project has no fonts.ts and must not be nagged about one.
+    const fontsPending = wiring.fontsWritten && !wiring.fontsMapped;
+    const unwired = table.source === "installed" &&
+        (!wiring.imported || !wiring.scoped || fontsPending);
     if (unwired) {
         console.log("");
         console.log(body(`${table.name ?? table.slug} is installed - but not wired up yet.`));
@@ -389,9 +432,28 @@ export async function doctor(opts) {
         console.log(body(wiring.scoped
             ? `  ✓ data-ds="${table.slug}" found`
             : `  ✗ no element carries data-ds="${table.slug}"`));
+        if (wiring.fontsWritten) {
+            console.log(body(wiring.fontsMapped
+                ? "  ✓ the type from fonts.ts is mapped onto the system"
+                : "  ✗ fonts.ts exists but nothing maps it - the system's type is not being used"));
+        }
         console.log("");
-        console.log(body(`Until both are true, none of the ${table.byName.size} tokens reach the browser`));
-        console.log(body("and the number below cannot mean anything."));
+        // Three requirements now, and they fail differently. Missing the import or
+        // the scope means NOTHING reaches the browser and the number below is
+        // noise. Missing only the type mapping means colour and spacing are
+        // working and the faces are not - saying "nothing reaches the browser"
+        // there would be false, and a report that overstates is one nobody trusts
+        // the next time.
+        const blocked = !wiring.imported || !wiring.scoped;
+        if (blocked) {
+            console.log(body(`Until the first two are true, none of the ${table.byName.size} tokens reach the`));
+            console.log(body("browser and the number below cannot mean anything."));
+        }
+        else {
+            console.log(body("Colour and spacing are working. Type is not: the system's faces"));
+            console.log(body("are declared and nothing points at them, so the page renders in"));
+            console.log(body("whatever the framework picked."));
+        }
         console.log(body("The exact snippets are in the output of `init`."));
     }
     if (hasSystem && measurable) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "synthesisui",
-  "version": "0.16.4",
+  "version": "0.16.6",
   "description": "Bring SynthesisUI design systems into any project - tokens, typed components, whole pages and an agent-ready CLAUDE.md manifest.",
   "type": "module",
   "bin": {