synthesisui 0.16.380 → 0.16.381

package/dist/doctor/fragments.js

@@ -22,8 +22,28 @@ const CLASS_LITERAL = /className=(?:"([^"]{0,400})"|\{\s*"([^"]{0,400})"\s*\})/g
 const CLASS_TEMPLATE = /className=\{\s*`([^`]{0,400})`\s*\}/g;
 /** `className={algo}` sem literal nenhum: não existe na fonte. */
 const CLASS_RUNTIME = /className=\{\s*([A-Za-z_$][\w$.[\]]{0,60})\s*\}/g;
-/** Um literal dentro de `cn("…", …)` e amigos. */
-const CALL_LITERAL = /\b(?:cn|clsx|classNames|cx|twMerge|twJoin|classcat)\(([^)]{0,600})\)/g;
+/**
+ * A ABERTURA de `cn("…", …)` e amigos - o FIM sai por parêntese balanceado, e não por regex.
+ *
+ * O QUE ACONTECIA: o corpo era `([^)]{0,600})`, que para no PRIMEIRO `)`. Basta uma chamada aninhada
+ * - `cn("flex", buttonVariants({ size }), "p-4")` - para tudo o que vem depois dela desaparecer da
+ * conta. Não é o teto de 600: é o parêntese.
+ *
+ * MEDIDO EM TRÊS POPULAÇÕES (05/09, classes que o coletor não via por causa do corte):
+ *
+ *     codelevel/packages/ui             21   de 33 chamadas truncadas de 83
+ *     frontend-hub/packages/ui          41   de 17 chamadas truncadas de 78
+ *     frontend-hub/apps/web-dashboard    1   de  1 chamada  truncada  de 47
+ *
+ * E o que sumia não era resto: `shadow-[0_10px_30px_-10px_rgba(…)]`, `data-[starting-style]:…`,
+ * `bg-[var(--color-lightgray-300)]` - as classes mais ricas do repositório, que são exatamente as
+ * que o cliente quer ver contadas. Num ledger cuja promessa é que nada é descartado em silêncio,
+ * elas não eram lidas E não eram "não lidas": não existiam.
+ *
+ * É O MESMO PRECEDENTE do `sx` e do `styled` três blocos abaixo, e pela mesma razão: um regex com
+ * teto escolhe a formatação do arquivo de outra pessoa.
+ */
+const CALL_OPEN = /\b(?:cn|clsx|classNames|cx|twMerge|twJoin|classcat)\(/g;
 const STRING_IN_CALL = /"([^"]{0,300})"|'([^']{0,300})'/g;
 /** `style={{ … }}`. */
 const INLINE = /style=\{\{([^}]{0,400})\}\}/g;
@@ -72,6 +92,7 @@ const MODULE_REF = /\$\{\s*[A-Za-z_$][\w$]*\.[A-Za-z_$][\w$]*\s*\}/g;
  *
  *     status === "success" ? "bg-success-500" : ...     "success" é OPERANDO da comparação
  *     // ... to prevent the "short width" issue        "short width" está num COMENTÁRIO
+ *     cn(iconVariants({ animation: … : "none" }), …)   "none" é ARGUMENTO de outra chamada
  *
  * MEDIDO EM 11/08 no repo real, sobre as 26 declarações que o censo listava como `value-not-read`:
  *
@@ -93,6 +114,38 @@ function blankComments(body) {
 function isComparedTo(body, index) {
     return /(?:===|!==|==|!=)\s*$/.test(body.slice(0, index));
 }
+/** Um utilitário de classe abrindo parêntese, para saber QUEM abriu a chamada mais interna. */
+const OPENS_A_CLASS_CALL = /\b(?:cn|clsx|classNames|cx|twMerge|twJoin|classcat)\($/;
+/**
+ * O LITERAL É ARGUMENTO DE OUTRA CHAMADA, e argumento de outra chamada não é classe.
+ *
+ * `cn(iconVariants({ animation: shouldAnimate ? animation : "none" }), className)` põe `"none"`
+ * dentro do `cn`, mas ele não veste nada: é o valor que a função de variantes recebe para escolher a
+ * classe. A esteira listava `none` como uma classe que não conseguiu ler, e o cliente ia procurar no
+ * código dele um estilo que ele nunca escreveu - o mesmo custo do operando de `===` logo acima.
+ *
+ * MEDIDO EM TRÊS POPULAÇÕES (05/09, sobre os literais que já passam pelos dois guards anteriores):
+ *
+ *     codelevel/packages/ui             3 de  83   `"none"` em Card, Icon, Typography
+ *     frontend-hub/packages/ui          0 de 123
+ *     frontend-hub/apps/web-dashboard   0 de  68
+ *
+ * QUEM ABRIU A CHAMADA DECIDE, e não o simples fato de haver uma. Um utilitário de classe aninhado
+ * (`cn(clsx("flex"))`) continua vestindo, e o que está dentro dele É classe - sem esta metade, o
+ * conserto de um falso positivo criaria um falso negativo. Nas três populações essa forma dá N=0
+ * hoje; ela está aqui porque aninhar utilitário é forma de qualquer projeto, e o `CALL_LITERAL` só
+ * casa a chamada de fora.
+ */
+function isArgumentOfAnotherCall(body, index) {
+    const open = [];
+    for (let i = 0; i < index; i++) {
+        if (body[i] === "(")
+            open.push(!OPENS_A_CLASS_CALL.test(body.slice(0, i + 1)));
+        else if (body[i] === ")")
+            open.pop();
+    }
+    return open[open.length - 1] === true;
+}
 export function fragmentsOfSource(file, source) {
     const out = [];
     const at = (index) => source.slice(0, index).split("\n").length;
@@ -127,15 +180,28 @@ export function fragmentsOfSource(file, source) {
      * OS LITERAIS DENTRO DE UMA CHAMADA. `cn("flex p-4", cond && "gap-2")` tem dois, e os dois são
      * classe escrita à mão - a forma mais comum do repo dele (113 arquivos).
      */
-    for (const call of source.matchAll(CALL_LITERAL)) {
-        const body = blankComments(call[1] ?? "");
-        const base = (call.index ?? 0) + (call[0].length - body.length - 1);
+    /**
+     * SÓ A CHAMADA MAIS EXTERNA - senão um utilitário aninhado conta os literais dele duas vezes.
+     *
+     * Enquanto o corpo parava no primeiro `)`, um `clsx(` dentro de um `cn(` era uma segunda match e
+     * ninguém percebia. Com o corpo inteiro, o de fora já cobre o de dentro.
+     */
+    let until = 0;
+    for (const call of source.matchAll(CALL_OPEN)) {
+        const from = (call.index ?? 0) + call[0].length;
+        if (from < until)
+            continue;
+        until = closingParen(source, from);
+        const body = blankComments(source.slice(from, until));
+        const base = from;
         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;
+            if (isArgumentOfAnotherCall(body, s.index ?? 0))
+                continue;
             push("class", base + (s.index ?? 0), text);
         }
     }
@@ -151,6 +217,19 @@ export function fragmentsOfSource(file, source) {
     }
     return out;
 }
+/** O fim da chamada aberta em `from`, contando parênteses. Sem fechamento, o fim do arquivo. */
+function closingParen(source, from) {
+    let depth = 1;
+    let i = from;
+    while (i < source.length && depth > 0) {
+        if (source[i] === "(")
+            depth += 1;
+        else if (source[i] === ")")
+            depth -= 1;
+        i += 1;
+    }
+    return depth === 0 ? i - 1 : source.length;
+}
 /** O fim de um objeto aberto com duas chaves - `sx={{ … }}` fecha em duas. */
 function balanced(source, from, close) {
     let depth = 2;

package/dist/install-marks.js

@@ -680,7 +680,7 @@ export const CHECKER_SINCE = "0.16.378";
  * posição de conteúdo. Um censo medido antes desta versão conta como deriva o que a folha do app
  * nem alcança - e o número do cliente é a promessa.
  */
-export const READER_SINCE = "0.16.378";
+export const READER_SINCE = "0.16.381";
 /**
  * O QUE ESTÁ INSTALADO AQUI FICOU PARA TRÁS - e as DUAS condições que fazem isso ser verdade.
  *

package/package.json

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