synthesisui 0.16.374 → 0.16.376

package/dist/doctor/apply-fix.js

@@ -51,6 +51,8 @@ export function planFix(d, read) {
     const next = new Map();
     /** Linhas por arquivo, mutadas em memória: dois achados no mesmo arquivo compõem. */
     const lines = new Map();
+    /** A mesma chave, congelada antes da primeira troca - ver o skip de `not-written`. */
+    const originals = new Map();
     /**
      * DE BAIXO PARA CIMA no arquivo não é necessário - a troca não muda a contagem de linhas - mas a
      * ORDEM por arquivo é, para dois achados na mesma linha não se atropelarem: o segundo procura o seu
@@ -119,20 +121,84 @@ export function planFix(d, read) {
                 continue;
             }
             lines.set(f.file, raw.split("\n"));
+            /**
+             * A LINHA COMO ELA ESTAVA ANTES DE QUALQUER TROCA DESTA RODADA - e é ela que separa dois
+             * skips que hoje se dizem com a mesma frase. Ver o bloco de `swapped === null`.
+             */
+            originals.set(f.file, raw.split("\n"));
         }
         const body = lines.get(f.file);
         if (!body)
             continue;
         const idx = f.line - 1;
         const current = body[idx];
+        /**
+         * A DECLARAÇÃO DE UM TOKEN NÃO É UM USO DELE - e trocar ali APAGA o token do app inteiro.
+         *
+         * O QUE ACONTECIA: `--color-brand-blue: #4f46e5;` é onde o token NASCE. O scan vê o literal, o
+         * sistema tem um nome para aquele valor - o próprio `--color-brand-blue` -, e a troca escrevia
+         * `--color-brand-blue: var(--color-brand-blue)`. Isso é um ciclo de custom property, e a regra do
+         * CSS é dura: a propriedade fica inválida no elemento E em todos os descendentes. Como esses
+         * tokens moram em `:root`, o app inteiro perde a cor.
+         *
+         * MEDIDO em 05/09, sobre uma cópia do repositório do dono: **26 declarações** viraram
+         * auto-referência numa passada. E cada uma delas é o destino de dezenas de trocas que o mesmo
+         * comando acabou de fazer - o conserto apagava aquilo que ele mesmo tinha acabado de apontar.
+         *
+         * É A MESMA FAMÍLIA DO CICLO DOS EASINGS que o dono encontrou em 04/09, e desta vez produzida
+         * pelo nosso comando de conserto. Sem erro no console: o navegador simplesmente descarta.
+         *
+         * A REGRA É ESTREITA DE PROPÓSITO. Apontar um token DELE para outro token DELE
+         * (`--brand-hover: #4f46e5` -> `var(--color-brand-blue)`) continua valendo e é bom. Só a
+         * auto-referência é fatal, e só ela é recusada.
+         *
+         * É GERAL: toda folha declara token com valor literal - 149 no repositório dele, 180 no
+         * frontend-hub.
+         */
+        const declaring = /^\s*(--[a-z0-9-]+)\s*:/i.exec(current ?? "");
+        if (declaring && declaring[1] === name) {
+            skipped.push({
+                file: f.file,
+                line: f.line,
+                literal: f.literal,
+                because: "self-reference",
+            });
+            continue;
+        }
         const swapped = current === undefined ? null : swap(current, f.literal, name);
         if (swapped === null) {
-            /** O arquivo mudou desde a medição, ou o literal já foi trocado por outro achado. */
+            /**
+             * DOIS MOTIVOS DIFERENTES, E A TELA DIZIA UM SÓ - e o que ela dizia era falso quase sempre.
+             *
+             * O QUE O DONO VIU em 05/09: *"13 lines changed since the scan and were left alone - run it
+             * again"*, num repositório onde ele não tinha editado nada. Medido: **13 de 13** eram classe
+             * utilitária. A linha era `"transition-all duration-300 ease-[var(--ease-out-soft)]"`, o scan
+             * derivou `300ms` de `duration-300` - corretamente -, e o `swap` procura o texto `300ms`, que
+             * nunca esteve escrito ali. `grep 300ms` no arquivo devolve zero.
+             *
+             * O comentário que estava aqui já sabia disso: *"o arquivo mudou desde a medição, OU o literal
+             * já foi trocado por outro achado"*. A frase impressa contava só o primeiro caso, e mandava
+             * rodar de novo - o que não muda nada, porque nada mudou.
+             *
+             * A LINHA ORIGINAL DESEMPATA. Sem o literal nela, o valor veio de forma abreviada e trocá-lo é
+             * outra reescrita (`duration-[var(--duration-base)]`), não uma troca de literal. Com o literal
+             * nela, alguma coisa mexeu na linha depois - e aí rodar de novo é o conselho certo.
+             *
+             * É GERAL: `duration-*` é Tailwind puro - 15 arquivos no repositório dele, 37 no frontend-hub.
+             */
+            /**
+             * A LINHA SUMIU É OUTRA COISA - e é `moved` de verdade: um arquivo que encolheu desde a
+             * medição mudou, e rodar de novo é o conselho certo. `not-written` só quando a linha ESTÁ lá
+             * e o valor não está escrito nela.
+             */
+            const before = originals.get(f.file)?.[idx];
             skipped.push({
                 file: f.file,
                 line: f.line,
                 literal: f.literal,
-                because: "moved",
+                because: before === undefined || before.includes(f.literal)
+                    ? "moved"
+                    : "not-written",
             });
             continue;
         }
@@ -176,6 +242,10 @@ export async function readerFor(root, files) {
 export function describeFix(result, dry) {
     const { applied, skipped } = result;
     const moved = skipped.filter((s) => s.because === "moved").length;
+    /** O valor veio de forma abreviada - ver `not-written` em `planFix`. */
+    const notWritten = skipped.filter((s) => s.because === "not-written").length;
+    /** A linha declara o próprio token - ver `self-reference` em `planFix`. */
+    const selfRef = skipped.filter((s) => s.because === "self-reference").length;
     const coincidence = skipped.filter((s) => s.because === "cross-family").length;
     const unread = skipped.filter((s) => s.because === "unreadable").length;
     const decisions = skipped.filter((s) => s.because === "no-token").length;
@@ -187,23 +257,67 @@ export function describeFix(result, dry) {
             ? `Nothing to apply. All ${decisions} findings are values your system has no name for - those are design decisions, not fixes.`
             : relative > 0
                 ? `Nothing to apply. All ${relative} findings are lengths in \`em\`, which follow the element's font size - swapping them can move the layout, so that call is yours.`
-                : "Nothing to apply.");
+                : /**
+                   * E QUANDO NADA É APLICÁVEL, O MOTIVO SAI AQUI TAMBÉM - senão a saída inteira é
+                   * "Nothing to apply." e a pessoa não tem como saber que o valor existe, tem nome, e só
+                   * não está escrito como texto.
+                   */
+                    notWritten > 0
+                        ? `Nothing to apply. All ${notWritten} findings are written as a shorthand - a utility class like \`duration-300\` carries the value in its name, so there is no literal on the line to replace. Changing them is a different rewrite.`
+                        : /**
+                           * E quando tudo o que havia eram as linhas onde os tokens nascem, "Nothing to apply."
+                           * sozinho manda a pessoa procurar um defeito. O comando passou por ali de propósito.
+                           */
+                            selfRef > 0
+                                ? `Nothing to apply. ${selfRef === 1 ? "The one finding is" : `All ${selfRef} findings are`} on the line where the token itself is declared - replacing there would point it at itself, and the browser would drop it.`
+                                : "Nothing to apply.");
         return lines;
     }
     lines.push(`${dry ? "Would replace" : "Replaced"} ${applied.length} hand-written value${applied.length === 1 ? "" : "s"} with the token your system already has, across ${result.files} file${result.files === 1 ? "" : "s"}.`);
     const byToken = new Map();
     for (const a of applied)
         byToken.set(a.token, (byToken.get(a.token) ?? 0) + 1);
-    for (const [token, n] of [...byToken.entries()]
-        .sort((a, b) => b[1] - a[1])
-        .slice(0, 6))
+    const ranked = [...byToken.entries()].sort((a, b) => b[1] - a[1]);
+    const SHOWN = 6;
+    for (const [token, n] of ranked.slice(0, SHOWN))
         lines.push(`  var(${token}) · ${n} time${n === 1 ? "" : "s"}`);
+    /**
+     * O QUE O CORTE ESCONDEU, DITO - e é a mesma lei que a linha abaixo já obedece.
+     *
+     * O QUE O DONO VIU em 05/09: `Would replace 143 ... across 20 files` e seis linhas somando 81. As
+     * outras 62 ocorrências não apareciam em lugar nenhum, e ele aprovaria a escrita sem saber que
+     * existiam.
+     *
+     * O comentário três linhas abaixo já enuncia a regra - *"DITO SEMPRE QUE ACONTECE, senão o número
+     * some da conta sem explicação"* -, aplicada ao `relative` e ao `moved` e não ao próprio corte
+     * desta lista. Terceira vez nesta família num dia: o `gaps` (#1327) e a tela de admin (#1304).
+     */
+    const rest = ranked.slice(SHOWN);
+    if (rest.length > 0) {
+        const restUses = rest.reduce((n, [, uses]) => n + uses, 0);
+        lines.push(`  and ${restUses} more across ${rest.length} other token${rest.length === 1 ? "" : "s"}, not listed here`);
+    }
     /**
      * DITO SEMPRE QUE ACONTECE - senão o número de "trocado" some da conta sem explicação, e a pessoa
      * conclui que o comando falhou onde ele se recusou de propósito.
      */
     if (relative > 0)
         lines.push(`  ${relative} more ${relative === 1 ? "is a length" : "are lengths"} in \`em\`, which follow the element's font size - your system names the same number, but the pixel may differ. Left for you to decide.`);
+    /**
+     * O VALOR QUE NÃO ESTÁ ESCRITO - a frase que substituiu uma afirmação falsa sobre o repositório.
+     *
+     * Isto saía como *"N lines changed since the scan - run it again"*, e na tela do dono nada tinha
+     * mudado: 13 de 13 eram `duration-300` e afins. Mandar rodar de novo não muda nada, e mandar
+     * alguém procurar uma edição que ela não fez é pior que calar.
+     */
+    /**
+     * DITO, e não calado: é a linha onde o token NASCE, e a pessoa precisa saber que o comando passou
+     * por ali de propósito - senão ela procura por que aquele valor não foi trocado.
+     */
+    if (selfRef > 0)
+        lines.push(`  ${selfRef} ${selfRef === 1 ? "is" : "are"} the line where the token itself is declared - replacing there would point it at itself and the browser would drop it. Left alone.`);
+    if (notWritten > 0)
+        lines.push(`  ${notWritten} more ${notWritten === 1 ? "is" : "are"} written as a shorthand - a utility class like \`duration-300\` carries the value in its name, so there is no literal on the line to replace. Changing ${notWritten === 1 ? "it" : "those"} is a different rewrite. Left alone.`);
     if (moved > 0)
         lines.push(`${moved} line${moved === 1 ? "" : "s"} changed since the scan and ${moved === 1 ? "was" : "were"} left alone - run it again.`);
     if (coincidence > 0)

package/package.json

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