synthesisui 0.16.374 → 0.16.375

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,6 +121,11 @@ 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)
@@ -127,12 +134,38 @@ export function planFix(d, read) {
         const current = body[idx];
         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 +209,8 @@ 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;
     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 +222,55 @@ 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.`
+                        : "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.
+     */
+    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.375",
   "description": "Bring SynthesisUI design systems into any project - tokens, typed components, whole pages and an agent-ready CLAUDE.md manifest.",
   "type": "module",
   "bin": {