synthesisui 0.16.205 → 0.16.208

package/dist/commands/align.js

@@ -5,22 +5,27 @@ import { pinnedHookVersion } from "../agent-wiring.js";
 import { isOlderCli } from "../cli-version.js";
 import { readToken, resolveRegistry } from "../config.js";
 import { unsentEvents } from "../doctor/ledger.js";
-import { CHECKER_SINCE, MATERIALISER_SINCE } from "../install-marks.js";
+import { CHECKER_SINCE, MATERIALISER_SINCE, READER_SINCE, } from "../install-marks.js";
 import { measuredScope } from "../measured-scope.js";
-import { READER } from "../reader-version.js";
 /**
- * Qual leitor mediu o censo em disco, quando ele o registra.
+ * QUAL CLI MEDIU O CENSO EM DISCO - e era o `reader`, um inteiro, até 11/08.
  *
- * `null` para censo anterior a esta marca: um censo que não diz quem o leu não prova nada, e supor
- * que ele é velho faria toda pessoa que ainda não re-mediu ver o aviso para sempre.
+ * O carimbo `ledger.reader` existe e continua sendo gravado, mas ele não serve para responder esta
+ * pergunta: ficou parado em 1 enquanto os leitores mudavam, então censos de julho e de ontem levam
+ * o mesmo número e nada os separa. A versão que mediu separa - ver `READER_SINCE`.
+ *
+ * `null` para censo que não diz quem o mediu: um censo assim não prova nada, e supor que ele é
+ * velho faria toda pessoa que ainda não re-mediu ver o aviso para sempre.
  */
-async function censusReader(root) {
+async function censusMeasuredBy(root) {
     const raw = await readFile(join(root, "_synthesisui", "census.json"), "utf8").catch(() => null);
     if (!raw)
         return null;
     try {
         const c = JSON.parse(raw);
-        return typeof c.ledger?.reader === "number" ? c.ledger.reader : null;
+        return typeof c.ledger?.cli === "string" && c.ledger.cli
+            ? c.ledger.cli
+            : null;
     }
     catch {
         return null;
@@ -121,11 +126,13 @@ opts = {}) {
      * leitura de antes e NENHUMA re-interpretação nossa alcança isso. Só uma medição nova alcança, e
      * ela é um comando que ninguém tinha motivo para rodar.
      *
-     * Compara `READER`, não a string do CLI: em 07/08 o CLI subiu quatro vezes e nenhuma delas mudou um
-     * leitor - pedir re-medição em cada bump faria a pessoa parar de ler estas linhas.
+     * Compara contra `READER_SINCE`, e não contra a string do CLI: em 07/08 o CLI subiu quatro vezes e
+     * nenhuma delas mudou um leitor - pedir re-medição em cada bump faria a pessoa parar de ler estas
+     * linhas. E não contra o carimbo `reader`, que ficou parado em 1 enquanto os leitores andavam:
+     * medido em 11/08, 12 de 36 componentes têm árvore diferente entre o leitor de 07/08 e o de hoje.
      */
-    const reader = await censusReader(root);
-    if (reader != null && reader !== READER)
+    const measuredBy = await censusMeasuredBy(root);
+    if (behind(measuredBy, cli, READER_SINCE))
         out.push({
             says: `the measurement stored in this repo was read by an older reader, so what the platform knows about your components is what that reader could see. A re-measure is the only thing that reaches it - the fix lives on this machine, not on the server.`,
             run: "npx synthesisui sync",

package/dist/commands/doctor.js

@@ -758,6 +758,26 @@ export async function doctor(opts) {
         console.log("");
         console.log(body(`Token coverage  ${meter(d.coverage)}  ${paint.strong(`${String(d.coverage).padStart(3)}%`)}`));
         console.log(body(paint.dim(`                ${d.tokenUses} from the system, ${d.findings.length} by hand${d.phantomUses > 0 ? `, ${d.phantomUses} naming nothing` : ""}`)));
+        /**
+         * ZERO NUM REPO QUE ORIGINOU O SISTEMA É O ESTADO CERTO, e sem esta linha ele lê como falha
+         * NOSSA.
+         *
+         * "Token coverage 0% - 1 from the system, 4433 by hand" é literalmente verdade e ainda assim
+         * engana: quando o sistema foi medido A PARTIR daqui, esses valores escritos à mão são a FONTE
+         * de onde os tokens saíram, não um desvio deles. Não há nada consertado ainda porque não havia
+         * nada quebrado - o passo seguinte é o código passar a apontar para os nomes que agora existem.
+         *
+         * O sinal é `measured.system`: a pasta de onde o import mediu. Ele só existe quando este
+         * repositório é a origem, então a frase não aparece para quem instalou um sistema de fora - ali
+         * um zero significa outra coisa e merece outra frase.
+         *
+         * A lei 14 em uma linha: um zero pelado lê como falha nossa, um zero com motivo lê como fato.
+         */
+        if (d.tokenUses === 0 && measured.system) {
+            console.log(body(paint.dim(`                zero is the expected start here - this system was measured FROM`)));
+            console.log(body(paint.dim(`                \`${measured.system}\`, so these values are its source, not a drift`)));
+            console.log(body(paint.dim(`                from it. They count once the code points at the names they became.`)));
+        }
         // The strongest number leads, not trails: it used to sit two screens
         // down, after the phantom list (30/07). For the reader who already owns
         // a system - the ICP - THIS line is the report.

package/dist/doctor/fragments.js

@@ -58,6 +58,36 @@ const MODULE_REF = /\$\{\s*[A-Za-z_$][\w$]*\.[A-Za-z_$][\w$]*\s*\}/g;
  * que nenhum leitor cobre (`sx`, `styled`, `runtime`) já saem com o motivo, porque nesses casos o
  * motivo é a própria forma e não depende de leitura.
  */
+/**
+ * O QUE ESTÁ DENTRO DE UM `cn(...)` E NÃO É CLASSE - e por que isto vira número na tela dele.
+ *
+ * O corpo da chamada é varrido por literal, e todo literal virava um fragmento de estilo. Dois deles
+ * nunca foram estilo, e a esteira estava dizendo ao cliente que não conseguia ler o que ele nem
+ * escreveu:
+ *
+ *     status === "success" ? "bg-success-500" : ...     "success" é OPERANDO da comparação
+ *     // ... to prevent the "short width" issue        "short width" está num COMENTÁRIO
+ *
+ * MEDIDO EM 11/08 no repo real, sobre as 26 declarações que o censo listava como `value-not-read`:
+ *
+ *      8 (31%)  operando de ===   Card, RadioCard x4, SelectionCard x2
+ *      2  (8%)  comentário        Input, Tag
+ *     16 (61%)  classe de verdade  sr-only, flex flex-col, data-[side=top]:top-full, ...
+ *
+ * Ou seja: 10 das 26 eram acusação NOSSA. E a lei 8 é explícita - lacuna declarada é confiança,
+ * mas lacuna FALSA custa o mesmo que a escondida: ela manda a pessoa procurar um problema que não
+ * existe no código dela, e gasta a credibilidade do número que está certo.
+ */
+/** Apaga comentários preservando o COMPRIMENTO, senão todo índice depois deles aponta para a linha errada. */
+function blankComments(body) {
+    return body
+        .replace(/\/\*[\s\S]*?\*\//g, (m) => " ".repeat(m.length))
+        .replace(/\/\/[^\n]*/g, (m) => " ".repeat(m.length));
+}
+/** `size === "md"`: o literal é o que está sendo COMPARADO, e comparar não é vestir. */
+function isComparedTo(body, index) {
+    return /(?:===|!==|==|!=)\s*$/.test(body.slice(0, index));
+}
 export function fragmentsOfSource(file, source) {
     const out = [];
     const at = (index) => source.slice(0, index).split("\n").length;
@@ -93,12 +123,14 @@ export function fragmentsOfSource(file, source) {
      * classe escrita à mão - a forma mais comum do repo dele (113 arquivos).
      */
     for (const call of source.matchAll(CALL_LITERAL)) {
-        const body = call[1] ?? "";
+        const body = blankComments(call[1] ?? "");
         const base = (call.index ?? 0) + (call[0].length - body.length - 1);
         for (const s of body.matchAll(STRING_IN_CALL)) {
             const text = s[1] ?? s[2] ?? "";
             if (!LOOKS_LIKE_CLASS.test(text))
                 continue;
+            if (isComparedTo(body, s.index ?? 0))
+                continue;
             push("class", base + (s.index ?? 0), text);
         }
     }

package/dist/doctor/transcribe.js

@@ -1325,12 +1325,44 @@ function layerFor(out, when, at) {
  * attempt.
  */
 const INLINE_STYLE = /style=\{\{([^{}]*(?:\{[^{}]*\}[^{}]*)*)\}\}/;
+/**
+ * AS PROPRIEDADES, SEPARADAS PELA VÍRGULA QUE SEPARA - e não pela que está dentro de um valor.
+ *
+ * `split(",")` partia `var(--color-track, var(--color-darkgray-300))` ao meio e entregava duas
+ * metades que não são nada. Vírgula dentro de parêntese ou de aspas pertence ao valor.
+ */
+function splitProperties(body) {
+    const out = [];
+    let depth = 0;
+    let quote = null;
+    let start = 0;
+    for (let i = 0; i < body.length; i += 1) {
+        const c = body[i];
+        if (quote) {
+            if (c === quote && body[i - 1] !== "\\")
+                quote = null;
+            continue;
+        }
+        if (c === '"' || c === "'" || c === "`")
+            quote = c;
+        else if (c === "(" || c === "[")
+            depth += 1;
+        else if (c === ")" || c === "]")
+            depth -= 1;
+        else if (c === "," && depth === 0) {
+            out.push(body.slice(start, i));
+            start = i + 1;
+        }
+    }
+    out.push(body.slice(start));
+    return out;
+}
 export function readInlineStyle(source) {
     const m = INLINE_STYLE.exec(source);
     if (!m)
         return {};
     const out = {};
-    for (const piece of m[1].split(",")) {
+    for (const piece of splitProperties(m[1])) {
         const colon = piece.indexOf(":");
         if (colon === -1)
             continue;
@@ -1339,10 +1371,24 @@ export function readInlineStyle(source) {
             .trim()
             .replace(/^["']|["']$/g, "");
         let value = piece.slice(colon + 1).trim();
+        /**
+         * UMA STRING ENTRE ASPAS ESTÁ NO FONTE, e o que ela tem dentro não muda isso.
+         *
+         * O guarda descartava todo valor com parêntese, e com isso todo `var()`, `linear-gradient()`,
+         * `calc()` e `rgb()` escrito em estilo inline - que são exatamente os valores que carregam
+         * decisão de design. Medido em 11/08 no repo dele: 2 dos 6 fragmentos que o censo chamava de
+         * `computed` eram literais, entre eles
+         * `style={{ stroke: "var(--color-track, var(--color-darkgray-300))" }}`.
+         *
+         * A distinção certa não é "tem parêntese", é "está no fonte": literal entre aspas se lê inteiro,
+         * e só o que NÃO está entre aspas passa pelo guarda de expressão.
+         */
+        const quoted = /^(["'])([\s\S]*)\1$/.exec(value);
+        if (quoted)
+            value = quoted[2];
         // A value computed at runtime is not in the source, so there is nothing to read.
-        if (!value || /[`$]|\?|=>|\(/.test(value))
+        else if (!value || /[`$]|\?|=>|\(/.test(value))
             continue;
-        value = value.replace(/^["']|["']$/g, "");
         if (!/^[a-zA-Z][\w]*$/.test(prop))
             continue;
         if (/[<>{}@;]/.test(value))

package/dist/install-marks.js

@@ -64,3 +64,36 @@ export const MATERIALISER_SINCE = "0.16.204";
  * Os arquivos que decidem: `doctor/` inteiro e `commands/hook.ts`.
  */
 export const CHECKER_SINCE = "0.16.202";
+/**
+ * A ÚLTIMA VERSÃO EM QUE OS LEITORES PASSARAM A PRODUZIR UM CENSO DIFERENTE.
+ *
+ * A terceira do mesmo bolo, e a que tinha o desenho MAIS antigo e mais errado. O `READER` era um
+ * inteiro carimbado no censo na hora da medição, e a comparação era `carimbo !== READER`. O defeito
+ * disso não é ruído, é PERDA DE INFORMAÇÃO: com o número parado em 1 desde 07/08, um censo medido
+ * ontem e um medido em julho carregam o mesmo `1` e nada os separa. Subir o número para 2 chamaria
+ * os dois para re-medir - inclusive quem já está em dia.
+ *
+ * Uma versão não perde nada, porque o censo já grava QUEM mediu (`StyleLedger.cli`).
+ *
+ * MEDIDO EM 11/08, rodando o leitor de 07/08 e o de hoje sobre os mesmos 35 componentes do repo
+ * real, e comparando o censo canônico:
+ *
+ *     12 de 36  componentes com ÁRVORE diferente - 10 lendo mais (LineChart +556 bytes)
+ *      8 de 36  com FRONTEIRA diferente - Tooltip, Menu, Modal, Select, ColorPicker, ...
+ *      2        custom properties que eram descartadas: --loader-color, --color-track
+ *
+ * Ou seja, o `READER` estava errado e ninguém tinha sido avisado. A marca nasceu em `0.16.202`,
+ * bisseccionada: o censo de 0.16.201 difere do de então, o de 0.16.202 é byte a byte igual.
+ *
+ * ANDOU DUAS VEZES no mesmo dia, e nas duas foi o portão que mandou:
+ *
+ *     0.16.207  `fragmentsOfSource` parou de tomar operando de `===` e texto de comentário por
+ *               classe - 11 das 26 declarações que o censo dele listava como ilegíveis eram
+ *               acusação NOSSA
+ *     0.16.208  `readInlineStyle` parou de descartar valor com parêntese - `var()` e
+ *               `linear-gradient()` em `style={{}}` chegam na receita, e 2 dos 6 fragmentos
+ *               chamados de `computed` eram literais no fonte
+ *
+ * Um censo medido antes disto carrega as duas leituras erradas.
+ */
+export const READER_SINCE = "0.16.208";

package/dist/reader-version.js

@@ -20,5 +20,21 @@
  *     READER          a medição na máquina dela   ->  npx synthesisui sync
  *     COMPILER        o CSS de uma versão já instalada  ->  npx synthesisui connect
  *     INTERPRETATION  a nossa metade, sobre censos já guardados  ->  roda no servidor, sem comando
+ *
+ * A DECISÃO SAIU DAQUI EM 11/08, e este número virou só um carimbo.
+ *
+ * O desenho acima está certo na intenção e errado no formato: um inteiro gravado no censo NO
+ * MOMENTO DA MEDIÇÃO só separa duas medições se ele tiver subido entre elas. Ele não subiu - ficou
+ * em 1 desde 07/08 - enquanto os leitores andavam. Medido rodando o leitor daquele dia e o de hoje
+ * sobre os mesmos 35 componentes do repo real: 12 de 36 componentes com ÁRVORE diferente, 8 com
+ * FRONTEIRA diferente, e 2 custom properties que eram descartadas. Todos esses censos carregam `1`,
+ * e a informação para separá-los não existe mais.
+ *
+ * Subir para 2 agora chamaria para re-medir também quem já está em dia. Então quem responde
+ * *"este censo está velho?"* passou a ser `READER_SINCE`, em `install-marks.ts`, comparado contra a
+ * VERSÃO que mediu (`StyleLedger.cli`) - que o censo sempre gravou e nunca se perde.
+ *
+ * O carimbo continua sendo escrito porque ele é barato e é registro; ele só não decide mais nada.
+ * Sobe para 2 aqui por honestidade com o que foi medido.
  */
-export const READER = 1;
+export const READER = 2;

package/package.json

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