synthesisui 0.16.372 → 0.16.374

package/dist/commands/doctor.js

@@ -589,6 +589,55 @@ async function readWiring(root, slug) {
 }
 /** Onde o retrato mora por default: dentro do que a esteira já escreve, e commitável. */
 const DEFAULT_BASELINE = "_synthesisui/drift-baseline.json";
+/**
+ * A PORTA ÚNICA DO `--fix` - e ela existe porque o conserto passou a ter DOIS chamadores.
+ *
+ * O caminho normal chama com o diagnóstico inteiro. O caminho de um projeto sem fiação chama com o
+ * subconjunto que o código DELE já resolve - ver `ownName` e o bloco que decide isso. Duas cópias
+ * desta sequência divergiriam no dia em que uma delas ganhasse um passo: o `writeFix` sem o
+ * `appendEvent`, ou o recibo sem a linha que manda ler o diff.
+ */
+async function runFix(root, d, writing) {
+    const read = await readerFor(root, d.findings.filter((f) => nameToWrite(f)).map((f) => f.file));
+    const { result, next } = planFix(d, read);
+    if (writing)
+        await writeFix(root, next);
+    console.log(section(writing ? "Fixed what had an answer" : "What --fix would do"));
+    for (const line of describeFix(result, !writing))
+        console.log(body(line));
+    if (!writing && result.applied.length > 0) {
+        console.log("");
+        console.log(body("Nothing was written. When the list above is what you want:"));
+        console.log(paint.blue(snippet(["npx synthesisui@latest doctor --fix --write"])));
+    }
+    /**
+     * O CONSERTO DEIXA RASTRO - ver `CheckEvent.kind: "fix"`.
+     *
+     * Sem esta linha, "ele trocou 129 valores por token" existia só no terminal daquele minuto: o
+     * ledger deduz conserto por arquivo que limpou, e um `--fix` que toca 79 arquivos de uma vez não
+     * aparece como um ato.
+     */
+    if (writing && result.applied.length > 0) {
+        await appendEvent(root, {
+            kind: "fix",
+            at: new Date().toISOString(),
+            named: result.applied.length,
+        });
+        console.log("");
+        console.log(body(paint.dim("Read the diff before committing - it is your code, and this replaced literals with the tokens your own system declares.")));
+    }
+}
+/**
+ * O NOME QUE ESTA TROCA ESCREVERIA É DELE? - a pergunta que a recusa do `--fix` nunca fez.
+ *
+ * `nameToWrite` devolve `theirToken ?? token`: quando o repositório dele nomeia aquele valor, é o
+ * nome DELE que vai para o código. Um nome dele resolve no navegador dele HOJE, com ou sem a nossa
+ * folha instalada - a folha só é necessária para os `--ds-*`, que são nossos.
+ */
+const ownName = (f) => {
+    const name = nameToWrite(f);
+    return Boolean(name && !name.startsWith("--ds-"));
+};
 export async function doctor(opts) {
     const root = resolve(opts.dir ?? process.cwd());
     /**
@@ -877,6 +926,21 @@ export async function doctor(opts) {
      * `"yours"` e `"adopted"` são o vocabulário dela, e não há duas linhas para ligar.
      */
     const blocked = table.source === "installed" && (!wiring.imported || !wiring.scoped);
+    /**
+     * QUANTAS DAS DUAS FALTAM - e sem este número três frases mandavam consertar DUAS coisas quando
+     * faltava UMA.
+     *
+     * `blocked` é um OU, e as frases que ele governa foram escritas assumindo o E: *"Until both are
+     * true"*, *"--fix is refused while those two are false"* e *"Wire the two lines first"*. Visto na
+     * máquina do dono em 04/09, num repositório onde `data-ds` já estava lá com um ✓ impresso três
+     * linhas acima: a tela contradizia a própria lista, e a terceira frase é INSTRUÇÃO - a pessoa está
+     * bloqueada, procurando o que escrever, e ela manda escrever duas linhas.
+     *
+     * O caso de faltar UMA é o mais comum: uma instalação que parou no meio erra um dos dois passos,
+     * não os dois. Derivado da mesma leitura que decide `blocked`, para as duas nunca discordarem.
+     */
+    const missingWiring = (wiring.imported ? 0 : 1) + (wiring.scoped ? 0 : 1);
+    const bothMissing = missingWiring === 2;
     const unwired = table.source === "installed" &&
         (!wiring.imported || !wiring.scoped || fontsPending);
     if (unwired) {
@@ -907,7 +971,9 @@ export async function doctor(opts) {
              * enquanto imprimia o número abaixo; agora não há número abaixo, e prometer um seria a mesma
              * incoerência ao contrário.
              */
-            console.log(body(`Until both are true, none of the ${table.byName.size} tokens reach the browser.`));
+            console.log(body(bothMissing
+                ? `Until both are true, none of the ${table.byName.size} tokens reach the browser.`
+                : `Until that one is true, none of the ${table.byName.size} tokens reach the browser.`));
         }
         else {
             console.log(body("Colour and spacing are working. Type is not: the system's faces"));
@@ -965,10 +1031,45 @@ export async function doctor(opts) {
             }).catch(() => { });
         console.log("");
         if (opts.fix) {
-            console.log(body(`--fix is refused while those two are false. ${d.named} of your hand-written values`));
-            console.log(body("do have a name in the system, and swapping them now would point them at"));
-            console.log(body("variables the browser cannot resolve - the declarations would be dropped"));
-            console.log(body("and the page would change. Wire the two lines first."));
+            /**
+             * A RECUSA PASSA A SER POR VALOR, E NÃO PELO COMANDO INTEIRO.
+             *
+             * NASCEU DE UMA PERGUNTA DO DONO (04/09): *"se o design system dele já está funcionando e são
+             * as mesmas referências, faz sentido esse ligamento? combinamos que não seria invasivo"*.
+             *
+             * A MEDIÇÃO DEU RAZÃO A ELE, pelo mesmo caminho que a tela usa (`loadSystem` + `scanSource` +
+             * `diagnose` sobre packages/ui + apps/landing + apps/web do codelevel): dos 216 valores com
+             * nome esperando, **156 (72%) escreveriam o nome DELE** - `--color-brand-violet`,
+             * `--color-tier-gold`, `--duration-base` - e esses resolvem no navegador dele HOJE, sem folha
+             * nenhuma instalada. Só 60 apontam para `--ds-*`, que são nossos e precisam da fiação.
+             *
+             * A recusa olhava `blocked` e nada mais. Ela nunca perguntava de quem era o nome que ia
+             * escrever, embora `nameToWrite` já responda isso (`theirToken ?? token`). O produto recusava
+             * 156 consertos que funcionariam para proteger 60 que quebrariam - e cobrava, em troca, uma
+             * folha inteira dentro do app de alguém que só queria governar o próprio vocabulário.
+             *
+             * ISSO ATINGE O CASO DE USO CENTRAL: quem importa um design system que já existe tem, por
+             * construção, a maioria dos nomes sendo dele.
+             *
+             * O QUE FICA DE FORA CONTINUA SENDO DITO, com o número e o motivo - Lei 8. O que muda é que a
+             * lacuna deixou de fechar a porta inteira.
+             */
+            const theirs = d.findings.filter(ownName).length;
+            const ours = d.findings.filter((f) => nameToWrite(f) && !ownName(f)).length;
+            if (theirs > 0) {
+                console.log(body(`${theirs} of your hand-written values ${theirs === 1 ? "carries" : "carry"} a name your own code declares, and ${theirs === 1 ? "it resolves" : "those resolve"} today - --fix will swap ${theirs === 1 ? "it" : "them"}.`));
+                if (ours > 0)
+                    console.log(body(`${ours} more point at ${table.name ?? table.slug} tokens, which only resolve once the wiring above is done. Left alone.`));
+                console.log("");
+                await runFix(root, { ...d, findings: d.findings.filter(ownName) }, opts.write === true);
+                return;
+            }
+            /** Nenhum nome dele em jogo: aqui a recusa continua sendo a resposta certa, e inteira. */
+            const one = ours === 1;
+            console.log(body(`--fix is refused while ${bothMissing ? "those two are" : "that one is"} false. ${ours} of your hand-written values`));
+            console.log(body(`${one ? "does" : "do"} have a name in the system, and swapping ${one ? "it" : "them"} now would point ${one ? "it" : "them"} at`));
+            console.log(body(`variables the browser cannot resolve - the ${one ? "declaration" : "declarations"} would be dropped`));
+            console.log(body(`and the page would change. Wire ${bothMissing ? "the two lines" : "that line"} first.`));
         }
         else {
             console.log(body(`${d.named} of the ${d.findings.length} hand-written values found have a name waiting in`));
@@ -1859,37 +1960,7 @@ export async function doctor(opts) {
      * 1500 achados e depois dizer "troquei 900" é fazer a pessoa procurar a linha que importa.
      */
     if (opts.fix) {
-        const read = await readerFor(root, d.findings.filter((f) => nameToWrite(f)).map((f) => f.file));
-        const { result, next } = planFix(d, read);
-        const writing = opts.write === true;
-        if (writing)
-            await writeFix(root, next);
-        console.log(section(writing ? "Fixed what had an answer" : "What --fix would do"));
-        for (const line of describeFix(result, !writing))
-            console.log(body(line));
-        if (!writing && result.applied.length > 0) {
-            console.log("");
-            console.log(body("Nothing was written. When the list above is what you want:"));
-            console.log(paint.blue(snippet(["npx synthesisui@latest doctor --fix --write"])));
-        }
-        /**
-         * O CONSERTO DEIXA RASTRO - ver `CheckEvent.kind: "fix"`.
-         *
-         * Sem esta linha, "ele trocou 129 valores por token" existia só no terminal daquele minuto: o
-         * ledger deduz conserto por arquivo que limpou, e um `--fix` que toca 79 arquivos de uma vez não
-         * aparece como um ato.
-         */
-        if (writing && result.applied.length > 0) {
-            await appendEvent(root, {
-                kind: "fix",
-                at: new Date().toISOString(),
-                named: result.applied.length,
-            });
-        }
-        if (writing && result.applied.length > 0) {
-            console.log("");
-            console.log(body(paint.dim("Read the diff before committing - it is your code, and this replaced literals with the tokens your own system declares.")));
-        }
+        await runFix(root, d, opts.write === true);
         return;
     }
     if (opts.writeBaseline) {

package/dist/doctor/gap-triage.js

@@ -235,13 +235,43 @@ values) {
          * VALOR daquilo está no sistema, e não só que não há o que consertar.
          */
         const theirs = group.reduce((n, g) => n + (g.withTheirTokens ?? 0), 0);
-        lines.push("", `${label[verdict]} - ${total} fragment${total === 1 ? "" : "s"}, ${unread > 0 ? Math.round((total / unread) * 100) : 0}% of what was not interpreted${theirs > 0
+        lines.push("", 
+        /**
+         * O DENOMINADOR VAI NA FRASE - e a falta dele era o mesmo defeito consertado pela METADE.
+         *
+         * A linha de cima estabelece decisões (1063, das quais 1007 interpretadas: 56 fora), e este
+         * percentual é sobre FRAGMENTOS (32). Dizer só "of what was not interpreted" faz as duas
+         * grandezas usarem a mesma palavra na mesma tela: quem soma 81 + 9 + 9 conclui que a triagem
+         * cobriu tudo, e ela cobriu 32 de 56.
+         *
+         * Em 03/09 o dono viu exatamente isto e o conserto reescreveu a linha do MEIO ("They live in N
+         * places in your code"). Estes três cabeçalhos ficaram com a expressão antiga - mesma tela,
+         * mesmo defeito, na parte que não foi olhada. Visto de novo em 04/09.
+         */
+        `${label[verdict]} - ${total} fragment${total === 1 ? "" : "s"}, ${unread > 0 ? Math.round((total / unread) * 100) : 0}% of those ${unread} place${unread === 1 ? "" : "s"}${theirs > 0
             ? ` — and ${theirs} of them already wear a token you declare, so their value is in the system`
             : ""}`);
         for (const gap of group) {
             lines.push(`  ${gap.uses} ${gap.shape} in ${gap.files} file${gap.files === 1 ? "" : "s"} (${gap.share}%) - ${gap.work}`);
-            for (const example of gap.examples.slice(0, 2))
+            /**
+             * TODOS OS QUE O CENSO GUARDOU, E O QUE FICOU DE FORA DITO EM VOZ ALTA.
+             *
+             * Isto cortava em dois (`slice(0, 2)`) sobre os até TRÊS que `style-ledger` grava - um por
+             * arquivo -, e não dizia nada. Na tela do dono em 04/09: *"3 class in 3 files"* seguido de dois
+             * arquivos, e o terceiro sumiu sem uma palavra. Ele conserta dois e acha que acabou.
+             *
+             * O GÊMEO JÁ TINHA O AVISO e este não: a tela de admin ganhou *"Not the whole set: N more"* no
+             * #1304. Duas superfícies respondem a mesma pergunta, e só uma admitia o corte - que é a Lei 8
+             * ao contrário, lacuna calada.
+             *
+             * O QUE FALTA SÃO ARQUIVOS, e é isso que a linha conta: o censo guarda um exemplo por arquivo,
+             * então `files` menos os mostrados é literalmente quantos arquivos você não está vendo.
+             */
+            for (const example of gap.examples)
                 lines.push(`      ${example.file}:${example.line}  ${example.text.slice(0, 76)}`);
+            const hidden = gap.files - gap.examples.length;
+            if (hidden > 0)
+                lines.push(`      not the whole set: ${hidden} more file${hidden === 1 ? "" : "s"} with this shape, not listed here`);
         }
     }
     return lines;

package/package.json

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