synthesisui 0.16.295 → 0.16.297

package/dist/commands/import.js

@@ -47,6 +47,7 @@ import { body, paint, section } from "../output.js";
 import { outsideScope } from "../outside-scope.js";
 import { phase, startProgress } from "../progress.js";
 import { repoStateOf } from "../repo-state.js";
+import { runtimeDeclaredVars, runtimeDeclaredVarsIn } from "../runtime-vars.js";
 import { detectStack, resolveDeps, stackVersions } from "../stack.js";
 import { placeInWorkspace } from "../workspace-place.js";
 import { add } from "./add.js";
@@ -1934,6 +1935,37 @@ export async function takeCensus(root, opts) {
     for (const m of css.matchAll(/(--[a-zA-Z0-9_-]+)\s*:/g)) {
         declaredNames.add(m[1]);
     }
+    /**
+     * E O QUE ELE DECLARA FORA DO CSS - ver `runtimeDeclaredVars`.
+     *
+     * `next/font` gera a variável em tempo de execução, então uma varredura de folhas `.css` não a
+     * enxerga e o relatório acusava as quatro fontes do repositório dele de "não resolvem para nada".
+     * Medido em 24/08: 4 das 6 variáveis reportadas como mortas estavam vivas, e o agente teve que se
+     * desdizer na frente do cliente.
+     */
+    for (const name of runtimeDeclaredVars(sources))
+        declaredNames.add(name);
+    /**
+     * E FORA DO ESCOPO TAMBÉM - senão o conserto acima não alcança um monorepo.
+     *
+     * Com um escopo estreitado este `root` É a pasta do escopo, então `sources` tem o escopo e mais
+     * nada. As fontes deste repositório são declaradas nos apps e CONSUMIDAS dentro da biblioteca - a
+     * divisão que o próprio Next recomenda, e não um acidente de organização.
+     *
+     * MEDIDO: com o leitor acima já publicado, a segunda corrida ainda reportou as quatro fontes como
+     * mortas. A promessa escrita acima é *"a name is only broken if NOTHING declares it, anywhere"*, e
+     * "anywhere" tem que incluir os roots que a pessoa nomeou em uso.
+     */
+    const usageRoots = (opts?.usage ?? []).map((u) => u.path);
+    if (usageRoots.length > 0) {
+        const outside = await runtimeDeclaredVarsIn(usageRoots, {
+            readdir: (p) => readdir(p, { withFileTypes: true, encoding: "utf8" }),
+            readFile: (p) => readFile(p, "utf8"),
+            join,
+        });
+        for (const name of outside)
+            declaredNames.add(name);
+    }
     /**
      * HOW SOMEBODY CALLS THE COMPONENT THEY ALREADY HAVE.
      *

package/dist/runtime-vars.js

@@ -0,0 +1,102 @@
+/**
+ * AS VARIÁVEIS QUE O CÓDIGO DELE DECLARA FORA DO CSS - e sem elas a plataforma acusa o repositório
+ * dele de um defeito que não existe.
+ *
+ * O QUE O CLIENTE VIA, medido numa corrida real em 24/08 contra o `codelevel`:
+ *
+ *     "6 var() que não resolvem para nada: --font-ds-display, --font-ds-body,
+ *      --font-ds-mono, --font-ds-serif, --ring-circ, --ring-target"
+ *
+ * Quatro das seis resolvem. Elas são declaradas por `next/font`, que gera a variável em tempo de
+ * execução e a injeta pelo `className` - o inventário de declarações lia só folhas `.css`, então
+ * para ele aquilo não existia. O agente teve que se desdizer na frente do cliente no MESMO
+ * fechamento, e uma afirmação errada custa mais confiança que a lacuna que ela tentava reportar.
+ *
+ * O ALCANCE, medido no mesmo repositório: 27 arquivos usam `next/font`, declarando 8 variáveis.
+ * Não é caso de borda - é a forma recomendada de carregar fonte em Next, e o nosso próprio
+ * `fonts.ts` a ESCREVE para o cliente. Sabíamos escrever a forma e não sabíamos lê-la.
+ *
+ * POR QUE O IMPORT PRECISA DO ARQUIVO, e não só do padrão: `variable: "--x"` é texto comum o
+ * bastante para aparecer em CSS-in-JS e em objetos de configuração de qualquer biblioteca. Contar
+ * aquilo como declaração em todo lugar trocaria um falso positivo por um falso negativo - a
+ * plataforma deixaria de avisar sobre uma variável genuinamente morta. Então a regra é a origem, e
+ * não o nome: só vale num arquivo que importa de `next/font`, que é o pacote cujo contrato promete
+ * a declaração.
+ */
+/** Um arquivo importa de `next/font`? É a origem que autoriza a leitura abaixo. */
+const IMPORTS_NEXT_FONT = /from\s+["']next\/font(?:\/[a-z]+)?["']/;
+/**
+ * O CAMPO DO CONTRATO. `next/font` aceita `variable` em `Inter({...})` e em `localFont({...})`, e nos
+ * dois casos a promessa é a mesma: a classe devolvida declara essa variável no elemento.
+ */
+const VARIABLE_FIELD = /\bvariable\s*:\s*["'](--[a-zA-Z0-9_-]+)["']/g;
+/**
+ * Os nomes que os arquivos dele declaram em tempo de execução, e que nenhuma folha `.css` carrega.
+ *
+ * Devolve um `Set` porque é assim que o inventário de declarações do import é consultado - a
+ * intenção é somar a ele, nunca substituí-lo: uma variável pode ser declarada nos dois lugares, e
+ * duas fontes que concordam não são um conflito.
+ */
+export function runtimeDeclaredVars(sources) {
+    const out = new Set();
+    for (const { source } of sources) {
+        if (!IMPORTS_NEXT_FONT.test(source))
+            continue;
+        for (const m of source.matchAll(VARIABLE_FIELD))
+            out.add(m[1]);
+    }
+    return out;
+}
+/**
+ * E A DECLARAÇÃO PODE MORAR FORA DO ESCOPO - o caso que o conserto de 24/08 não alcançou.
+ *
+ * MEDIDO no `codelevel` na segunda corrida, já com o leitor acima publicado: as quatro fontes
+ * CONTINUARAM sendo reportadas como mortas. A causa não era o leitor - era o que chega nele.
+ *
+ * `--scope packages/ui` faz o `root` do censo ser a pasta do escopo, então a varredura vê
+ * `packages/ui` e mais nada. As fontes são declaradas em `apps/landing/app/fonts.ts` e
+ * `apps/web/app/fonts.ts`, e USADAS dentro de `packages/ui`. Essa divisão não é acidente de
+ * organização: num monorepo, o app carrega a fonte e a biblioteca a consome, e é a forma
+ * recomendada pelo próprio Next.
+ *
+ * O código já promete o contrário do que fazia - *"a name is only broken if NOTHING declares it,
+ * anywhere"* está escrito no ponto que monta a lista. "Anywhere" tem que incluir os roots de
+ * `--usage`, que a pessoa nomeou justamente para dizer onde o vocabulário é consumido.
+ *
+ * SÓ LEITURA, e só de arquivos de código: nada aqui escreve, e um root de uso já é lido para
+ * evidência - a diferença é que agora ele também pode DECLARAR.
+ */
+export async function runtimeDeclaredVarsIn(roots, read) {
+    const out = new Set();
+    const seen = new Set();
+    const walk = async (dir, depth) => {
+        /**
+         * O TETO DE PROFUNDIDADE é o que mantém isto barato num monorepo grande. Uma declaração de
+         * fonte mora no diretório de app (`app/fonts.ts`, `src/app/fonts.ts`, `styles/fonts.ts`) -
+         * quatro níveis cobrem as três formas com folga, e o custo não cresce com o tamanho do repo.
+         */
+        if (depth < 0 || seen.has(dir))
+            return;
+        seen.add(dir);
+        const entries = await read.readdir(dir).catch(() => []);
+        for (const e of entries) {
+            if (e.name.startsWith(".") || e.name === "node_modules")
+                continue;
+            const full = read.join(dir, e.name);
+            if (e.isDirectory()) {
+                await walk(full, depth - 1);
+                continue;
+            }
+            if (!/\.(ts|tsx|js|jsx|mjs)$/i.test(e.name))
+                continue;
+            const src = await read.readFile(full).catch(() => "");
+            if (!src)
+                continue;
+            for (const name of runtimeDeclaredVars([{ file: full, source: src }]))
+                out.add(name);
+        }
+    };
+    for (const r of roots)
+        await walk(r, 4);
+    return out;
+}

package/package.json

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