synthesisui 0.10.0 → 0.10.1

package/dist/commands/doctor.js

@@ -401,7 +401,10 @@ export async function doctor(opts) {
             }
             return false;
         };
-        const offSystem = drifting.filter((o) => !onSystem(o)).length;
+        // Three acts, not two. `transform: none` does not leave the system - it
+        // REMOVES what the recipe sets, and no token can hold "no transform".
+        // Marking it as a raw value asked for something that cannot exist.
+        const offSystem = drifting.filter((o) => !onSystem(o) && !isReset(o.wrote)).length;
         console.log(section("Overruled"));
         console.log(body(`${drifting.length} place${drifting.length === 1 ? "" : "s"} where the code takes a component`));
         console.log(body("the system defines, and then overrules it locally."));
@@ -434,8 +437,10 @@ export async function doctor(opts) {
                     console.log(`            the recipe binds ${o.recipe}`);
                 else if (o.where)
                     console.log(`            the recipe binds it ${o.where}`);
-                if (!onSystem(o) && lawKept(o) === null) {
-                    console.log("            ↑ a raw value, not a token");
+                if (lawKept(o) === null && !onSystem(o)) {
+                    console.log(isReset(o.wrote)
+                        ? "            ↑ removes it rather than replacing it"
+                        : "            ↑ a raw value, not a token");
                 }
                 const kept = lawKept(o);
                 if (kept) {

package/dist/doctor/scan.js

@@ -40,6 +40,21 @@ const SPACING = /(?:\b[pmg](?:[trblxy])?-\[|gap-\[|(?:padding|margin|gap)\s*:\s*
 const FONT = /font-family\s*:\s*([^;}\n]+)/g;
 /** Uses of the system. Coverage is meaningless without them. */
 const TOKEN_USE = /var\(\s*--ds-[a-z0-9-]+/gi;
+/**
+ * `var(--ds-color-semantic-primary, #5266eb)` - the literal is the TOKEN'S OWN
+ * fallback, written for safety, and reporting it as drift told an author to
+ * tokenize something they had already tokenized (investidorez, 25/07). The
+ * spans below are the fallback arguments on a line.
+ */
+const VAR_FALLBACK = /var\(\s*--ds-[a-z0-9-]+\s*,([^()]*)\)/gi;
+function fallbackSpans(line) {
+    const out = [];
+    for (const m of line.matchAll(VAR_FALLBACK)) {
+        const start = (m.index ?? 0) + m[0].indexOf(",") + 1;
+        out.push([start, start + m[1].length]);
+    }
+    return out;
+}
 /** Under this, a radius or spacing value is idiom rather than a decision. */
 const IDIOM = new Set(["0", "0px", "1px", "9999px", "100%", "50%"]);
 export function scanSource(file, source, table) {
@@ -77,10 +92,18 @@ export function scanSource(file, source, table) {
         // written by hand - flagging it would be telling someone to tokenize a
         // variable. And the same literal twice in one declaration (a two-stop
         // shadow) is one decision, so it is reported once.
+        const spans = fallbackSpans(line);
+        const inFallback = (at) => spans.some(([a, b]) => at >= a && at <= b);
         const seen = new Set();
-        const push = (kind, literal) => {
+        // NOT named `at`: that is the line number in this scope, and shadowing it
+        // put column positions into the report as line numbers.
+        const push = (kind, literal, col = -1) => {
             if (literal.includes("$") || literal.includes("{"))
                 return;
+            if (col >= 0 && inFallback(col)) {
+                aside++;
+                return;
+            }
             const key = `${kind}:${literal}`;
             if (seen.has(key))
                 return;
@@ -103,17 +126,17 @@ export function scanSource(file, source, table) {
                 aside++;
                 continue;
             }
-            push("color", m[0]);
+            push("color", m[0], m.index ?? -1);
         }
         for (const m of line.matchAll(RADIUS)) {
             const value = `${m[1]}${m[2]}`;
             if (!IDIOM.has(value))
-                push("radius", value);
+                push("radius", value, m.index ?? -1);
         }
         for (const m of line.matchAll(SPACING)) {
             const value = `${m[1]}${m[2]}`;
             if (!IDIOM.has(value))
-                push("spacing", value);
+                push("spacing", value, m.index ?? -1);
         }
         for (const m of line.matchAll(FONT)) {
             const stack = m[1].trim();
@@ -128,7 +151,12 @@ export function scanSource(file, source, table) {
         findings,
         tokenUses,
         ...(aside > 0
-            ? { setAside: { reason: "SVG artwork", count: aside } }
+            ? {
+                setAside: {
+                    reason: "SVG artwork or a token's own fallback",
+                    count: aside,
+                },
+            }
             : null),
     };
 }

package/package.json

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