synthesisui 0.16.103 → 0.16.104

package/dist/commands/import.js

@@ -636,6 +636,21 @@ export async function takeCensus(root, opts) {
             }
             usageSeen.add(rel);
             usageProgress?.step(rel);
+            /**
+             * THE THEME IS SWITCHED WHERE THE APP IS, and this loop was walking right
+             * past the file that says so. Measured on a real monorepo (dono, 02/08):
+             * `packages/ui` declares 346 dark utilities across 58 files and NONE of the
+             * four switch fields; `apps/web-dashboard` declares `data-theme`,
+             * `defaultTheme="dark"`, `enableSystem={false}` and an `<html>` carrying both
+             * - so "which face does it open in" was unanswerable from the census and cost
+             * a hand read of a file this loop already had open.
+             *
+             * The scope is read first and wins on every field, so this only fills what a
+             * library legitimately does not have - and the label is recorded, because the
+             * system being measured here and the theme being switched there are two facts,
+             * not one average.
+             */
+            readSignalsInto(signals, importTally, rel, src, u.label);
             scanComponentsInto(usageTally, rel, src, [...shared, ...uInternal]);
             // WHERE THE PAIRS LIVE. A library composes almost none of itself inside
             // itself - `Button` goes inside `Modal` in the APP, not in the library

package/dist/doctor/signals.js

@@ -75,6 +75,7 @@ export function emptySignals() {
             enableSystem: null,
             prefersColorScheme: 0,
             htmlCarries: [],
+            switchedIn: null,
             page: [],
         },
         libraries: [],
@@ -113,7 +114,13 @@ export function libraryOf(specifier) {
  * Fold one file into the signals. Called from the walk the census already does, so nothing
  * here costs a second pass over the disk.
  */
-export function readSignalsInto(into, imports, file, source) {
+export function readSignalsInto(into, imports, file, source, 
+/**
+ * WHERE THIS FILE IS, when it is not the system's own folder - the `--usage`
+ * label. Present means "read for evidence", and the only thing it changes is
+ * that a switch found here is RECORDED AS FOUND HERE. See `switchedIn`.
+ */
+from) {
     const isCode = /\.(tsx|jsx|ts|js|mjs|cjs|vue|svelte)$/i.test(file);
     const isStyle = /\.(css|scss|sass|less)$/i.test(file);
     if (isStyle) {
@@ -146,6 +153,16 @@ export function readSignalsInto(into, imports, file, source) {
             }
         }
     }
+    /**
+     * THE SCOPE WINS, ALWAYS - `??=` is the whole rule. The system's own folder is
+     * read first, so a switch it declares is never overwritten by an app's; an app
+     * only fills what was null, which is the four fields a library legitimately does
+     * not have.
+     */
+    const before = into.theme.attribute === null &&
+        into.theme.defaultTheme === null &&
+        into.theme.enableSystem === null &&
+        into.theme.htmlCarries.length === 0;
     into.theme.attribute ??= THEME_ATTRIBUTE.exec(source)?.[1] ?? null;
     into.theme.defaultTheme ??= DEFAULT_THEME.exec(source)?.[1] ?? null;
     if (into.theme.enableSystem === null) {
@@ -160,12 +177,42 @@ export function readSignalsInto(into, imports, file, source) {
      */
     const html = HTML_ATTRS.exec(source);
     if (html) {
-        for (const m of html[1].matchAll(/([a-zA-Z-]+)\s*=\s*(?:["']([^"']*)["']|\{["']([^"']*)["']\})/g)) {
-            const attr = `${m[1]}="${m[2] ?? m[3] ?? ""}"`;
+        const carry = (attr) => {
             if (!into.theme.htmlCarries.includes(attr)) {
                 into.theme.htmlCarries.push(attr);
             }
+        };
+        for (const m of html[1].matchAll(/([a-zA-Z-]+)\s*=\s*(?:["']([^"']*)["']|\{["']([^"']*)["']\})/g)) {
+            carry(`${m[1]}="${m[2] ?? m[3] ?? ""}"`);
         }
+        /**
+         * `style={{ colorScheme: "dark" }}` - the SECOND thing that decides what a dark
+         * rule matches, and the attribute pass above cannot see it: its value is not a
+         * quoted string sitting after an `=`, it is a property inside an object.
+         *
+         * It is on the owner's own root layout, right beside the `data-theme` the pass
+         * DID read, and it is what tells the browser which face to paint its own
+         * widgets - a scrollbar and a form control follow `color-scheme`, not a class
+         * (dono, 02/08).
+         */
+        const scheme = /\bcolorScheme\s*:\s*["']([^"']+)["']/.exec(html[1]);
+        if (scheme)
+            carry(`colorScheme="${scheme[1]}"`);
+    }
+    /**
+     * WHOSE FILE ANSWERED IT. Stamped only when the switch was null before this file
+     * and is not null after, so the label names the file's ROOT that actually
+     * supplied it - and only for a `--usage` root, because the scope needs no
+     * attribution: it is the system by definition.
+     */
+    if (from &&
+        into.theme.switchedIn === null &&
+        before &&
+        (into.theme.attribute !== null ||
+            into.theme.defaultTheme !== null ||
+            into.theme.enableSystem !== null ||
+            into.theme.htmlCarries.length > 0)) {
+        into.theme.switchedIn = from;
     }
     /**
      * THE PAGE PAINTS, from the markup. A `min-h-screen` container with a background is
@@ -310,6 +357,15 @@ export function describeSignals(s) {
             bits.push("with the OS allowed a vote");
         out.push(`${bits.join(", ")}.`);
     }
+    /**
+     * TWO PLACES, TWO FACTS. A library that declares 346 dark utilities and no
+     * switch is not a library missing something - the switch belongs to the app that
+     * mounts it. Saying WHERE keeps the sentence true and stops the reader from
+     * concluding the system declares a provider it does not have.
+     */
+    if (t.switchedIn) {
+        out.push(`the system is measured where you scoped it; the theme is switched in \`${t.switchedIn}\` - that is where the attribute, the default and the \`<html>\` came from.`);
+    }
     if (t.htmlCarries.length > 0) {
         out.push(`\`<html>\` carries ${t.htmlCarries.map((a) => `\`${a}\``).join(" ")} - which is the ancestor every dark rule of theirs matches, and therefore the one ours must match.`);
     }

package/dist/skill-import.js

@@ -331,6 +331,13 @@ census carries is the failure this pipeline keeps paying for. The sketch now run
 so a big component arrives WHOLE: \`TextEditor\` used to lose its editable region at node 60 and
 that single truncation is what forced a hand-read of the file (dono, 01/08).
 
+\`signals.theme\` includes the SWITCH - \`attribute\`, \`defaultTheme\`, \`enableSystem\` and what
+\`<html>\` carries, \`colorScheme\` included - read from the \`--usage\` roots as well as the scope,
+because a component library declares the dark utilities and the APP declares the switch. When it
+came from an app, \`switchedIn\` names which one: say "the system is measured in \`packages/ui\`, the
+theme is switched in \`apps/web-dashboard\`" rather than reporting that the library declares a
+provider it does not have. Answering 2d never needs a layout file opened.
+
 **Validate in ONE call, at the end.** Build every recipe in memory first, then send them all to
 \`validate_recipes\` - one round trip for the whole reading instead of one per component. A reader
 with 35 anatomies spent 35 turns asking the same question, and none of that was thinking. Only

package/package.json

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