synthesisui 0.16.367 → 0.16.368

package/dist/commands/align.js

@@ -10,7 +10,6 @@ import { readRequests } from "../doctor/requests.js";
 import { declaredReference } from "../group-role.js";
 import { CHECKER_SINCE, installedBehind, MATERIALISER_SINCE, READER_SINCE, } from "../install-marks.js";
 import { fmt, say } from "../lang.js";
-import { mcpAwaitingApproval } from "../mcp-approval.js";
 import { measuredScope } from "../measured-scope.js";
 import { body, bodyWrapped, paint, section, snippet } from "../output.js";
 import { SKILLS } from "../skills.js";
@@ -97,33 +96,6 @@ export async function localMisalignments(root,
 opts = {}) {
     const { cli, home = homedir() } = opts;
     const out = [];
-    /**
-     * O PORTÃO QUE MATOU UM IMPORT INTEIRO, dito na sessão em que ele está fechado.
-     *
-     * Medido em 03/09: o agente mediu 262 arquivos, leu 78 componentes, descreveu a paleta dele - e
-     * parou, porque o playbook do import é servido pelo MCP e a aprovação estava pendente. Isto vem
-     * ANTES de qualquer outro desalinho porque é o único que impede a jornada principal de começar:
-     * um repo sem sistema não está desalinhado, e um repo cujas ferramentas ninguém aprovou não pode
-     * nem medir de volta.
-     *
-     * `/mcp` e não um comando de terminal: quem aprova é o agente, e um `npx` aqui mandaria a pessoa
-     * ao lugar errado. Ver `mcp-approval.ts` para de onde o estado é lido, e para o silêncio quando
-     * não há como saber.
-     */
-    if ((await mcpAwaitingApproval(root, home).catch(() => null)) === true)
-        out.push({
-            says: "The system's tools are declared here and still waiting for your approval - until you give it, I cannot reach the import playbook, only measure your files.",
-            run: "/mcp",
-            /**
-             * O VALOR NÃO TRAZ O PRÓPRIO RÓTULO - quem rotula é quem desenha.
-             *
-             * Ele dizia `then say: import my design system`, e o render põe "then, to pick up where you
-             * left off:" acima: a tela saía com dois rótulos e dois "then" para UM passo. É o defeito que
-             * o #1314 consertou, reintroduzido por um valor que carregava a moldura dentro.
-             */
-            then: "import my design system",
-            where: "agent",
-        });
     const locks = (await locksIn(root)).filter((l) => l.slug && !l.adopted);
     /** Sem sistema instalado não há alinho a cobrar - não é desalinho, é um repo sem DS. */
     if (locks.length === 0)
@@ -543,7 +515,7 @@ from = "session") {
             : from === "shell"
                 ? "This repo is out of alignment with the design system that governs it:"
                 : "Before this session starts, this environment is out of alignment with the design system that governs it:",
-        ...grouped(items, from),
+        ...grouped(items),
     ].join("\n");
 }
 /**
@@ -562,13 +534,7 @@ from = "session") {
  *
  * A ordem é a da primeira aparição: o primeiro achado decide onde o grupo dele fica.
  */
-function grouped(items, 
-/**
- * QUEM ESTÁ LENDO decide duas coisas aqui: se um remédio do agente precisa dizer onde mora, e se
- * a linha ganha cor. Numa sessão a pessoa JÁ está no agente, e o rótulo seria a tela falando de
- * si mesma.
- */
-from = "session") {
+function grouped(items) {
     const order = [];
     const byRun = new Map();
     for (const m of items) {
@@ -585,7 +551,7 @@ from = "session") {
         const group = byRun.get(key) ?? [];
         for (const m of group)
             out.push(`  - ${m.says}`);
-        const { run, then, where } = group[0];
+        const { run, then } = group[0];
         /**
          * A HIERARQUIA, e ela faltava inteira - *"o texto está sem hierarquia de cor, está muito ruim"*
          * (dono, na 0.16.365). Este bloco saía cru enquanto a ação recomendada, quinze linhas acima,
@@ -595,16 +561,8 @@ from = "session") {
          * `paint` se apaga sozinho sem TTY, então o `SessionStart` - que entrega texto a um modelo -
          * continua recebendo o mesmo texto de sempre, sem um único código de escape.
          */
-        if (run) {
-            /**
-             * UM REMÉDIO DO AGENTE DIZ ONDE MORA, na superfície onde ele não é digitável. `/mcp` no
-             * prompt do zsh é um comando que não existe; com o rótulo, a linha continua colável no lugar
-             * certo em vez de mandar a pessoa ao lugar errado.
-             */
-            if (where === "agent" && from !== "session")
-                out.push(paint.dim(`      ${say("inside your agent session:")}`));
+        if (run)
             out.push(paint.blue(`      ${run}`));
-        }
         /** O `then` sai numa linha própria e rotulada: um segundo comando solto lê como parte do primeiro. */
         if (then)
             out.push(`${paint.dim(`      ${say("then, to pick up where you left off:")}`)}\n${paint.blue(`      ${then}`)}`);
@@ -668,14 +626,10 @@ opts = {}) {
  * repositório inteiro, e isto roda em TODA abertura de sessão. Um aviso que custa um walk completo
  * a cada sessão é um aviso que alguém desliga.
  */
-export async function nextStepFor(root, 
-/** `home` porque a decisão sobre as ferramentas mora nele - ver `mcpAwaitingApproval`. */
-opts = {}) {
+export async function nextStepFor(root) {
     const locks = (await locksIn(root)).filter((l) => l.slug && !l.adopted);
     if (locks.length > 0)
         return null;
-    /** `true` só quando declarado e sem decisão; `null` (não sei) e `false` calam do mesmo jeito. */
-    const awaiting = (await mcpAwaitingApproval(root, opts.home ?? homedir()).catch(() => null)) === true;
     const measured = await readFile(join(root, "_synthesisui", "census.json"), "utf8").then(() => true, () => false);
     /**
      * MEDIDO E SEM SISTEMA INSTALADO SÃO DUAS HISTÓRIAS, e a frase antiga contava a errada.
@@ -692,17 +646,14 @@ opts = {}) {
      * `list --mine` primeiro porque o slug nasce no servidor: ele não está em nenhum arquivo daqui, e
      * chutá-lo seria pior que pedir para olhar.
      */
-    const awaits = awaiting ? { toolsAwaiting: true } : {};
     return measured
         ? {
-            ...awaits,
             sentence: "This repo has been measured and has no system installed here. If you already imported, the system is in your account: `synthesisui list --mine`, then `synthesisui add <slug>`.",
             headline: "Your system is already in your account. Bring it in here:",
             run: ["synthesisui list --mine", "synthesisui add <slug>"],
             why: "Re-measuring would cost minutes and change nothing the platform already knows.",
         }
         : {
-            ...awaits,
             sentence: 'This repo has no design system contract yet. Ask me: "import my design system."',
             headline: "Open a new agent session in this repo",
             open: await agentOpenCommands(root).catch(() => []),
@@ -767,15 +718,17 @@ export function renderNextStep(next, freshSession) {
                 : "This run moved {what}, and a new session is what reads them.", { what: said })).map(paint.dim));
     }
     /**
-     * A LINHA DA APROVAÇÃO, UMA VEZ, PELO FATO OU PELO QUE ACABOU DE MOVER.
+     * A LINHA DAS FERRAMENTAS SÓ NA RODADA QUE ACABOU DE DECLARÁ-LAS - e o segundo caso foi REMOVIDO.
+     *
+     * Havia aqui um `next.toolsAwaiting`, lido do `~/.claude.json`, para dizer que a aprovação estava
+     * pendente. Ele saiu em 04/09 porque afirmava um estado que nada observa - a medição inteira está
+     * na mensagem do PR e na lição do estado: as duas listas daquele arquivo estão vazias em SETE de
+     * sete projetos do dono, inclusive onde o MCP serve o playbook sem problema.
      *
-     * Ela estava dentro do `if (freshSession)` e dependia de o MCP ter mudado NESTA rodada. O caso do
-     * dono é o contrário e é o mais comum a partir da segunda vez: a fiação já estava lá ("nothing to
-     * change - this environment is already current") e a aprovação continuava pendente. `next.toolsAwaiting`
-     * é o fato lido do estado; `freshSession.mcp` continua valendo para a rodada que acabou de
-     * declarar as ferramentas, quando ainda não há decisão nenhuma para ler.
+     * `freshSession.mcp` continua e é honesto por construção: ele não infere estado nenhum, ele diz
+     * que ESTA rodada acabou de declarar o servidor - um fato sobre o que o comando fez.
      */
-    if (next.toolsAwaiting || freshSession?.mcp)
+    if (freshSession?.mcp)
         lines.push(...bodyWrapped(say("The project's tools ask for approval once - say yes.")).map(paint.dim));
     return lines.join("\n");
 }
@@ -791,25 +744,22 @@ export function renderNextStep(next, freshSession) {
  */
 export async function reportWhatIsLeft(root, opts = {}) {
     const items = await misalignments(root, opts).catch(() => []);
-    const next = await nextStepFor(root, {
-        ...(opts.home ? { home: opts.home } : {}),
-    }).catch(() => null);
+    const next = await nextStepFor(root).catch(() => null);
     /**
-     * A AÇÃO RECOMENDADA GANHA DO QUE SÓ SE RESOLVE DENTRO DELA - e a falta desta regra é a
-     * regressão que o dono leu na 0.16.365.
+     * A AÇÃO RECOMENDADA GANHA DA LISTA - e a falta desta regra foi a regressão que o dono leu na
+     * 0.16.365.
      *
      * Isto era um OU EXCLUSIVO: um desalinho qualquer e o bloco "Do this next" não saía. O aviso da
-     * aprovação pendente nasceu em 03/09 e é verdadeiro justamente no repo que acabou de rodar
-     * `connect` - foi ele que declarou as ferramentas. E num repo sem sistema instalado aquele é o
-     * ÚNICO desalinho possível: todos os outros vêm depois de um `.lock`, e o remoto devolve `null`
-     * sem lock. Então a instrução de abrir o agente não desapareceu às vezes; ela desapareceu SEMPRE,
-     * no único momento em que ela é a coisa que a pessoa procura na tela.
-     *
-     * A separação não esconde nada: o que se resolve no terminal continua saindo, embaixo da ação. O
-     * que se resolve dentro do agente vira a segunda metade da ação, que é o que ele é - a aprovação
-     * acontece na sessão que o passo 1 manda abrir.
+     * aprovação pendente, que nasceu em 03/09, era verdadeiro justamente no repo que acabou de rodar
+     * `connect` - foi ele que declarou as ferramentas -, e num repo sem sistema aquele era o ÚNICO
+     * desalinho possível. Então a instrução de abrir o agente não desaparecia às vezes: desaparecia
+     * SEMPRE, no único momento em que ela é a coisa que a pessoa procura na tela.
+     *
+     * AQUELE AVISO SAIU EM 04/09 - ele afirmava um estado que nada observa -, e a ordem FICA. Ela é o
+     * desenho certo independentemente dele: uma lista de manutenção nunca deve substituir a única
+     * ação que a tela está pedindo. O que sobra para o terminal continua saindo, embaixo da ação.
      */
-    const onTheTerminal = items.filter((m) => m.where !== "agent");
+    const onTheTerminal = items;
     if (next) {
         console.log(renderNextStep(next, opts.freshSession));
         if (onTheTerminal.length > 0) {

package/dist/mcp-approval.js

@@ -75,5 +75,37 @@ export async function mcpAwaitingApproval(root, home) {
     /** Formato diferente do que a gente conhece: silêncio, nunca um palpite. */
     if (!Array.isArray(enabled) || !Array.isArray(disabled))
         return null;
-    return !enabled.includes(SERVER) && !disabled.includes(SERVER);
+    /** Recusado explicitamente é um estado REAL, e é o único que este arquivo registra com certeza. */
+    if (disabled.includes(SERVER))
+        return true;
+    if (enabled.includes(SERVER))
+        return false;
+    /**
+     * AUSENTE DAS DUAS LISTAS É "NÃO SEI", E TRATÁ-LO COMO "PENDENTE" FOI O DEFEITO.
+     *
+     * A premissa desta função era: *"um servidor declarado que não está em nenhuma das duas listas
+     * está PENDENTE - medido, as duas listas vazias para aquele projeto"*. A medição existiu e estava
+     * certa sobre aquele projeto; a INFERÊNCIA não, porque ninguém mediu a outra metade.
+     *
+     * REMEDIDO em 04/09, na máquina do dono, nos SETE projetos que declaram este servidor: as duas
+     * listas estão vazias em TODOS - incluindo o `codelevel-monorepo`, onde o import inteiro rodou no
+     * dia anterior com o MCP servindo o playbook. Estado byte a byte idêntico entre "funciona" e
+     * "está pendente": a leitura não distinguia nada, era uma constante disfarçada de sinal.
+     *
+     * E procurei o registro em todas as casas antes de concluir - `~/.claude.json`,
+     * `.claude/settings.json`, `.claude/settings.local.json`, `.mcp.json` e `~/.claude/`. A aprovação
+     * de um servidor do `.mcp.json` não é gravada em nenhuma delas. `hasTrustDialogAccepted` é o
+     * diálogo do PROJETO, e `mcp-needs-auth-cache.json` é dos servidores do claude.ai.
+     *
+     * O CUSTO DE AFIRMAR SEM SABER, e ele não foi teórico: esta linha chega ao contexto do agente em
+     * toda abertura de sessão (`align` no `SessionStart`). Em 04/09 o dono pediu *"import my design
+     * system"*, o agente leu o aviso, acreditou nele e RECUSOU a jornada principal - mandou rodar
+     * `/mcp` num projeto cujas ferramentas estavam disponíveis. O defeito que o aviso existia para
+     * evitar era o agente parar sem saber por quê; o aviso passou a ser a causa dele.
+     *
+     * Então volta a valer a regra que o cabeçalho deste módulo sempre declarou - *"cala quando não
+     * sabe"* -, e quem observa se as ferramentas chegaram é o AGENTE, que sabe quais tem. A instrução
+     * mora no playbook, endereçada a ele.
+     */
+    return null;
 }

package/dist/skill-import.js

@@ -12,4 +12,4 @@
  * invocada, e ela não é segredo - a esteira é.
  */
 export const IMPORT_SKILL_PATH = ".claude/skills/sui-import-ds/SKILL.md";
-export const IMPORT_SKILL = '---\nname: sui-import-ds\ndescription: Turn a codebase the user ALREADY has into a SynthesisUI design system. Use when someone points at an existing repo, app or component library and asks to import it, adopt it, bring it in, or "make a design system from this" (e.g. "/sui-import-ds", "importa o meu packages/ui", "turn this app into a design system"). Drives the full pipeline - measurement \u2192 your reading of the app \u2192 import \u2192 upgrade proposal - and answers the questions arithmetic cannot.\n---\n\n# Import Design System - served live\n\nThis playbook is served from the platform, not shipped in this file - it is\nalways current, and your context only carries the step you are on.\n\n1. Call the `playbook` tool on the `synthesisui` MCP server with\n   { "skill": "import" } - you get the framing and a table of contents.\n2. Fetch ONLY the chapter for the step you are on:\n   { "skill": "import", "section": "<id from the toc>" }. Never fetch more\n   than the current step needs.\n3. Follow it exactly. When the step is done, fetch the next chapter.\n\n## If something is missing, they skipped a step - do not stall them\n\nThe supported order is `login`, then `connect`, then open this session. When\nit was followed, neither case below happens.\n\nNOT SIGNED IN. Do not stop and wait. Measure first, so they see their own\nrepo before being asked for anything:\n\n1. Run `npx synthesisui@latest import --dry` - it is local, free, needs no\n   account, and writes `_synthesisui/census.json`.\n2. Read that file and tell them what is in THEIR repo, in THEIR names: how\n   many declarations, how many components, how much already runs on tokens.\n3. Then: "to turn this into a system on your account, run\n   `npx synthesisui@latest login` and tell me when it is done."\n\nNO `synthesisui` MCP SERVER AT ALL. Tell them to run\n`npx synthesisui@latest connect` and then START A NEW SESSION - tools are read\nwhen a session opens, so this one cannot pick it up. Say it plainly and stop\nthere; carrying on in a session that cannot reach the playbook is guessing.\n';
+export const IMPORT_SKILL = '---\nname: sui-import-ds\ndescription: Turn a codebase the user ALREADY has into a SynthesisUI design system. Use when someone points at an existing repo, app or component library and asks to import it, adopt it, bring it in, or "make a design system from this" (e.g. "/sui-import-ds", "importa o meu packages/ui", "turn this app into a design system"). Drives the full pipeline - measurement \u2192 your reading of the app \u2192 import \u2192 upgrade proposal - and answers the questions arithmetic cannot.\n---\n\n# Import Design System - served live\n\nThis playbook is served from the platform, not shipped in this file - it is\nalways current, and your context only carries the step you are on.\n\n1. Call the `playbook` tool on the `synthesisui` MCP server with\n   { "skill": "import" } - you get the framing and a table of contents.\n2. Fetch ONLY the chapter for the step you are on:\n   { "skill": "import", "section": "<id from the toc>" }. Never fetch more\n   than the current step needs.\n3. Follow it exactly. When the step is done, fetch the next chapter.\n\n## If something is missing, they skipped a step - do not stall them\n\nThe supported order is `login`, then `connect`, then open this session. When\nit was followed, neither case below happens.\n\nNOT SIGNED IN. Do not stop and wait. Measure first, so they see their own\nrepo before being asked for anything:\n\n1. Run `npx synthesisui@latest import --dry` - it is local, free, needs no\n   account, and writes `_synthesisui/census.json`.\n2. Read that file and tell them what is in THEIR repo, in THEIR names: how\n   many declarations, how many components, how much already runs on tokens.\n3. Then: "to turn this into a system on your account, run\n   `npx synthesisui@latest login` and tell me when it is done."\n\nTOOLS DECLARED, TOOLS NOT THERE. If `.mcp.json` names `synthesisui` and you\ncannot see its tools, the approval is pending - and only YOU can see that, so\nnobody can tell it to you. Measure anyway before you ask: `npx\nsynthesisui@latest import --dry` is local and needs no tool. Show them what is\nin their repo, THEN ask for `/mcp`. Arriving empty-handed helps nobody.\n\nNO `synthesisui` MCP SERVER AT ALL. Tell them to run\n`npx synthesisui@latest connect` and then START A NEW SESSION - tools are read\nwhen a session opens, so this one cannot pick it up. Say it plainly and stop\nthere; carrying on in a session that cannot reach the playbook is guessing.\n';

package/package.json

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