synthesisui 0.16.1 → 0.16.4

package/dist/claude-md.js

@@ -31,31 +31,26 @@ async function readInstalled(projectRoot) {
     }
     return locks;
 }
-/** First sentence of a description, capped - the manifest must stay lean. */
-function summarize(desc) {
-    if (typeof desc !== "string" || !desc.trim())
-        return "";
-    const first = desc.trim().split(/(?<=\.)\s/)[0] ?? desc.trim();
-    return first.length > 90 ? `${first.slice(0, 87)}…` : first;
-}
 /** One manifest line per recipe: name, what it is, and its variant axes. */
-function catalogLines(recipes) {
-    return Object.entries(recipes)
-        .sort(([a], [b]) => a.localeCompare(b))
-        .map(([name, recipe]) => {
-        const r = recipe;
-        const axes = Object.entries(r.variants ?? {})
-            .map(([axis, options]) => {
-            const keys = Object.keys(options);
-            return keys.length <= 4 ? `${axis}: ${keys.join("|")}` : axis;
-        })
-            .join("; ");
-        const desc = summarize(r.description);
-        return {
-            name,
-            line: `  - \`ds-${name}\`${desc ? ` - ${desc}` : ""}${axes ? ` [${axes}]` : ""}`,
-        };
-    });
+/**
+ * NAMES ONLY, and the 87% that buys.
+ *
+ * The manifest was name + description + variant axes for every component -
+ * 4059 of this block's 5837 bytes on a 48-component system, carried into every
+ * session including the ones that never touch UI, and growing linearly with
+ * each system installed.
+ *
+ * But the question the agent asks here is binary: "is there already something
+ * for this?" A name answers it. Description and variants only matter AFTER
+ * that decision, and by then the agent is opening the GUIDE or running
+ * `component <slug> <name>`, which hands it the real typed props anyway.
+ *
+ * Names stay INLINE rather than moving to a file, deliberately. A lookup that
+ * costs a file read is a lookup an agent skips when it is in a hurry, and then
+ * writes the fourth button. Cheap to consult is the whole point.
+ */
+function catalogNames(recipes) {
+    return Object.keys(recipes).sort((a, b) => a.localeCompare(b));
 }
 /**
  * The COMPONENT MANIFEST for one installed system, read from its versioned
@@ -67,19 +62,20 @@ async function readManifest(projectRoot, ds) {
     try {
         const raw = await readFile(join(projectRoot, "_synthesisui", "ds", ds.slug, `v${ds.version}`, "design-system.json"), "utf8");
         const doc = JSON.parse(raw);
-        const components = catalogLines(doc.components ?? {});
-        const blocks = catalogLines(doc.blocks ?? {});
+        const components = catalogNames(doc.components ?? {});
+        const blocks = catalogNames(doc.blocks ?? {});
         if (components.length === 0 && blocks.length === 0)
             return null;
         const lines = [];
         if (components.length > 0) {
-            lines.push(`  Components (${components.length}) - USE these before creating new ones:`);
-            lines.push(...components.map((c) => c.line));
+            lines.push(`  Components (${components.length}) - look here BEFORE writing anything new:`);
+            lines.push(`  ${components.map((n) => `ds-${n}`).join(" ")}`);
         }
         if (blocks.length > 0) {
             lines.push(`  Engagement blocks (${blocks.length}):`);
-            lines.push(...blocks.map((c) => c.line));
+            lines.push(`  ${blocks.map((n) => `ds-${n}`).join(" ")}`);
         }
+        lines.push("  What each one does, its variants and states: the GUIDE above.");
         return lines.join("\n");
     }
     catch {

package/dist/commands/adopt.js

@@ -1,4 +1,4 @@
-import { mkdir, readFile, writeFile } from "node:fs/promises";
+import { mkdir, readdir, readFile, writeFile } from "node:fs/promises";
 import { join, relative, resolve } from "node:path";
 import { syncClaudeMd } from "../claude-md.js";
 import { parseRootTokens } from "../doctor/tokens.js";
@@ -75,6 +75,34 @@ function guessSlug(tokens, pkgName) {
         .replace(/^-|-$/g, "")
         .toLowerCase() || "my-system");
 }
+/**
+ * A system WE installed, if there is one.
+ *
+ * Run inside a repo already carrying Moments - 158 tokens, 93% coverage - adopt
+ * announced "found no custom properties … if you do not have a system yet,
+ * start from one" (27/07). Telling somebody with a design system that they have
+ * none is the exact dead end this command was written to avoid: our own tokens
+ * live under `_synthesisui/`, which the walk skips, so it looked and found
+ * nothing and drew the wrong conclusion from it.
+ */
+async function installedSystems(root) {
+    const dir = join(root, "_synthesisui", "ds");
+    const slugs = [];
+    for (const e of await readdir(dir, { withFileTypes: true }).catch(() => [])) {
+        if (!e.isDirectory())
+            continue;
+        const raw = await readFile(join(dir, e.name, ".lock"), "utf8").catch(() => "");
+        try {
+            const lock = JSON.parse(raw);
+            if (!lock.adopted)
+                slugs.push(lock.name ?? e.name);
+        }
+        catch {
+            // no readable lock - not something we put there
+        }
+    }
+    return slugs;
+}
 async function findTokens(root, only) {
     let files = 0;
     const from = [];
@@ -146,6 +174,14 @@ export async function adopt(opts) {
     const found = await findTokens(root, opts.tokens);
     console.log(section("Adopt"));
     if (found.tokens.size === 0) {
+        const already = await installedSystems(root);
+        if (already.length > 0) {
+            console.log(body(`${already.join(", ")} ${already.length === 1 ? "is" : "are"} already installed here, and adopt is for a system we did not install.`));
+            console.log("");
+            console.log(body("Nothing to do. To see how the code is holding up:"));
+            console.log(snippet(["npx synthesisui@latest doctor"]));
+            return;
+        }
         // Never "failed to adopt". Always: what was looked for, where, and the
         // next thing to type. Being stuck with no move is the worst outcome.
         console.log(body(`Looked in ${SCOPE_HINT} across ${found.files} stylesheet${found.files === 1 ? "" : "s"} and found no custom properties.`));

package/dist/doctor/tokens.js

@@ -189,24 +189,69 @@ export function parseTokens(css) {
  */
 export function parseRootTokens(css) {
     const out = new Map();
-    for (const block of css.matchAll(/([^{}]*)\{([^{}]*)\}/g)) {
-        const selector = block[1].trim();
-        const isRoot = /^@theme\b/i.test(selector) ||
-            /(^|[\s,>+~])(:root|html|:host)\b/i.test(selector);
-        if (!isRoot)
+    const isRoot = (sel) => /^@theme\b/i.test(sel) || /(^|[\s,>+~])(:root|html|:host)\b/i.test(sel);
+    // Every `<selector> {` in the file. Nested ones show up too and are filtered
+    // by isRoot, so an @keyframes inside @theme is skipped rather than mined.
+    const opens = /([^{}]*)\{/g;
+    let m = opens.exec(css);
+    while (m !== null) {
+        // The capture runs back to the previous brace, so it carries imports and
+        // comments with it. The SELECTOR is only what follows the last `;` or
+        // comment - and `@theme` is anchored to the start, so without this it
+        // matched only when the block happened to be the first thing in the file.
+        // Every fixture had it first. This repo's own globals.css does not, and
+        // reported 0 of its 48 tokens (27/07).
+        const sel = (m[1]
+            .replace(/\/\*[\s\S]*?\*\//g, "")
+            .split(";")
+            .pop() ?? "").trim();
+        if (!isRoot(sel)) {
+            m = opens.exec(css);
             continue;
-        for (const m of block[2].matchAll(/(--[a-z0-9_-]+)\s*:\s*([^;}]+)/gi)) {
-            const name = m[1].toLowerCase();
+        }
+        // Walk to the MATCHING close, counting depth. The flat regex this replaces
+        // required a body with no braces at all, so it silently skipped any
+        // `@theme` containing `@keyframes` - which is the documented Tailwind v4
+        // layout. Measured against this repo's own globals.css: 48 tokens present,
+        // 2 found, and the 2 came from unrelated test fixtures (27/07).
+        const start = m.index + m[0].length;
+        let depth = 1;
+        let i = start;
+        while (i < css.length && depth > 0) {
+            const c = css[i];
+            if (c === "{")
+                depth++;
+            else if (c === "}")
+                depth--;
+            i++;
+        }
+        // Only declarations that are DIRECT children count. A custom property
+        // inside a keyframe step is animation state, not a design token.
+        let flat = "";
+        let d = 0;
+        for (let j = start; j < i - 1; j++) {
+            const c = css[j];
+            if (c === "{")
+                d++;
+            else if (c === "}")
+                d--;
+            else if (d === 0)
+                flat += c;
+        }
+        for (const t of flat.matchAll(/(--[a-z0-9_-]+)\s*:\s*([^;}]+)/gi)) {
+            const name = t[1].toLowerCase();
             // Belt and braces: v4 emits some `--tw-*` bookkeeping into @theme, and
             // it is machinery, not somebody's design vocabulary.
             if (name.startsWith("--tw-"))
                 continue;
-            const value = m[2].trim();
+            const value = t[2].trim();
             if (!value || value.startsWith("var("))
                 continue;
             if (!out.has(name))
                 out.set(name, value);
         }
+        opens.lastIndex = i;
+        m = opens.exec(css);
     }
     return out;
 }

package/package.json

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