fix(test): replace tautology in playground-api-tab + make test-masking catch it (#6404)

[diegosouzapw]

Recupera o commit a21928e7a6 que ficou solto (detached HEAD) e o aponta para a release atual.

O que muda

tests/unit/ui/playground-api-tab.test.tsx — o teste de SSE sempre caía no ramo do botão desabilitado (o mock de fetch devolvia lista de modelos vazia) e afirmava uma tautologia em vez de exercitar o caminho SSE que diz verificar. Agora o teste seleciona um modelo real para habilitar o Send, afirma que ele está de fato habilitado, e afirma que o conteúdo SSE transmitido chegou ao editor de resposta.

scripts/check/check-test-masking.mjs — a subcheck de tautologia só comparava contagens base-vs-HEAD dentro do diff da própria PR e virava no-op fora de contexto de PR (sem GITHUB_BASE_SHA/REF) — então uma tautologia mergeada uma vez, ou checada com run local puro, ficava invisível para sempre. Adicionado um scan de piso absoluto sempre-ativo (scanBareTautologies/countBareTautologies) sobre todo arquivo de teste rastreado, escopado aos padrões expect(true).toBe(true)/assert.equal(1,1) que têm zero uso legítimo neste código — excluindo deliberadamente assert.ok(true), que tem ~15 usos pré-existentes verificados-legítimos em fallback try/catch e permanece no caminho leniente diff-only.

Testes

• tests/unit/check-test-masking.test.ts — cobre o novo scan de piso absoluto.
• tests/unit/ui/playground-api-tab.test.tsx — reescrito para exercitar o caminho SSE real.

[chatgpt-codex-connector]

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

[gemini-code-assist]

Code Review

This pull request improves CLI server startup diagnostics by piping and buffering both stdout and stderr from the server child process, allowing the CLI to print recent logs and diagnostic advice if a timeout occurs instead of hanging silently. It also updates documentation and quality baselines to reflect recent feature additions, such as the new pipeline routing strategy. The review feedback correctly identifies a risk where verbose stdout output could quickly flush out critical stderr error logs from the limited 50-line buffer, and suggests increasing the buffer size to mitigate this.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

[gemini-code-assist]

Ao direcionar tanto o stdout quanto o stderr para o mesmo buffer this.crashLog (que possui um limite rígido de apenas 50 linhas, definido por CRASH_LOG_LINES), há um risco real de que mensagens informativas ou de depuração mais volumosas vindas do stdout acabem expulsando rapidamente as mensagens de erro críticas e stack traces do stderr.\n\nIsso pode inutilizar o diagnóstico de falhas de inicialização ou travamentos (que é o objetivo principal desta alteração).\n\nSugestão: Aumentar o limite de linhas mantidas no buffer para garantir que os logs de erro do stderr não sejam perdidos devido à verbosidade do stdout.

    const bufferOutput = (data) => {\n      const lines = data.toString().split(\"\\n\").filter(Boolean);\n      this.crashLog.push(...lines);\n      const maxLines = CRASH_LOG_LINES * 4; // Aumenta o limite para acomodar stdout + stderr\n      if (this.crashLog.length > maxLines) {\n        this.crashLog = this.crashLog.slice(-maxLines);\n      }\n    };