Saída colorida (FORCE_COLOR/CLICOLOR_FORCE) derruba a classificação JS do gate

[CRangelP]

Achado numa revisão do PR #33, reproduzido com pnpm, yarn e bun reais.

Com FORCE_COLOR=1 / CLICOLOR_FORCE=1 no ambiente — comum em CI e em alguns
terminais — os managers emitem o epílogo com escapes ANSI:

\e[41m[ELIFECYCLE]\e[0m Test failed. See above for more details.
\e[2K\e[1Gerror Command failed with exit code 1.

O filtro que remove o epílogo (grep -vE '^[[:space:]]*(…ELIFECYCLE…)' em
scripts/gate.sh) não casa mais, o epílogo permanece em $ev, e uma suíte
legitimamente vazia é lida como falha encadeada → RED.

A direção é segura (nunca vira GREEN), mas anula nesses ambientes o fix de
epílogo que o PR #33 introduziu.

Fix sugerido

Normalizar antes de classificar, com o perl que já é dependência aceita:

out=$(printf '%s\n' "$out" | perl -pe 's/\e\[[0-9;?]*[ -\/]*[@-~]//g')

e/ou rodar o script filho com NO_COLOR=1 FORCE_COLOR=0.

Teste

Fixtures dos três managers com o epílogo colorido, provando que a suíte vazia
continua contando como cap e não como RED.

[CRangelP]

Correção: a direção NÃO é segura — o corpo desta issue está errado nesse ponto

O corpo afirma que "a direção é segura (nunca vira GREEN)". É falso, e a
consequência é a mais grave que este repositório admite: um repo com zero
arquivo de teste é classificado GREEN, e GREEN libera deleção autônoma de
exports mais as fases 2 e 3.

A análise original só olhou o ramo rc != 0 (o epílogo do package manager). O
mesmo ANSI quebra o ramo rc == 0, que é onde mora a única barreira entre
"repo sem suíte" e GREEN.

Reprodução isolada dos dois detectores de gate.sh:js_script()

node:test — regex '(^|[^[:alnum:]])tests[[:space:]]+0[[:space:]]*$'
  'ℹ tests 0'                    -> detectou      -> uncounted_suite -> YELLOW cap
  '\e[34mℹ tests 0\e[39m'        -> NÃO detectou  -> ran_test=1      -> GREEN

passWithNoTests — regex '^[[:space:]]*(No test files found|...)'
  'No test files found...'        -> detectou      -> uncounted_suite -> YELLOW cap
  '\e[33mNo test files found...' -> NÃO detectou  -> ran_test=1      -> GREEN

A causa é a âncora: o detector do node:test ancora em $ e o reset de cor
(\e[39m) vem depois do 0; o do --passWithNoTests ancora em ^ e o
escape vem antes do texto. Cada um é derrotado por uma ponta diferente da
mesma linha colorida.

Por que isto não é borda de CI

gate.sh não tem nenhuma normalização de ANSI (grep -c por
NO_COLOR|FORCE_COLOR|\033|\e[ = 0) e gate_test.sh não tem nenhuma
fixture colorida (= 0). O caso nunca foi exercitado.

E FORCE_COLOR=1 / CLICOLOR_FORCE=1 não vêm de shell rc nem de login shell:
são injetados por harness de agente. Verificado no ambiente desta análise —
ambas presentes. Ou seja, é plausivelmente a condição default de onde esta
skill roda, não exceção.

Isto derruba a frase que sustenta o cap de suíte vazia no SKILL.md: "uma
suíte que não existe não pode passar". Hoje ela pode.

Consequência para o encaminhamento

Sai da fila normal: é patch imediato, não item de milestone que espera lote.
O milestone existe para amarrar a #36 e a infraestrutura de fixture que falta,
não para adiar a #35.

_{Levantado por análise multiagente das 9 issues sem milestone; reproduzido de forma independente antes de comentar.}

[CRangelP]

Plano de ataque

Reproduzido e ampliado antes de planejar. A issue descreve um sintoma; a causa raiz é
maior do que ela, e o plano muda por causa disso.

1. Causa raiz, em uma frase

gate.sh classifica a saída do runner JS com regex ancorada (^ ou $) sobre texto que
nunca é normalizado — grep -c 'NO_COLOR|FORCE_COLOR|\033|\e\[' em gate.sh retorna
0, e gate_test.sh não tem uma única fixture colorida.

2. Raio de alcance — o que mais a mesma causa quebra

Não são dois detectores, são cinco pontos de decisão mais o filtro de epílogo, todos
derivados da mesma variável $out (captura única em out=$(cat "$outfile")):

ponto	âncora	efeito sob cor
passWithNoTests (rc==0)	^	não casa → ran_test=1 → GREEN
node:test (rc==0)	$	não casa → ran_test=1 → GREEN
filtro de epílogo ($ev)	^	epílogo do package manager sobrevive e polui $ev
passWithNoTests (rc!=0)	^	não casa → não cai em uncounted_suite
guarda de falha real	^|[[:space:]]	não casa → falha real fica invisível
inline_rc (awk)	^[[:space:]]*	não casa → desempate de exit code perdido

A guarda de falha real é o achado que a issue não tem. Sob coloração mista — linha de
suíte vazia sem cor, marcador de falha colorido, que é exatamente o que um runner que
colore só o erro produz — verifiquei:

1) casa "suite vazia": SIM
2) guarda de falha real: CEGA  =>  suíte que FALHOU vira YELLOW (uncounted), não RED

Ou seja, a issue documenta a direção "sem teste vira GREEN"; existe também "teste
quebrado vira YELLOW". As duas mentem sobre a suíte, em direções diferentes.

3. A decisão de projeto: normalizar na fronteira, não regex por regex

Consertar seis regex é consertar o mesmo bug seis vezes e deixar a sétima nascer com o
próximo detector. Há um ponto de captura, e todos os seis derivam dele. Normalize ali,
uma vez, num valor usado só para classificar — a saída que o usuário vê continua
colorida, porque é a saída dele:

printf '%s\n' "$out"                       # exibição: intacta
out_plain=$(printf '%s\n' "$out" | strip_ansi)   # classificação: normalizada
ev=$(printf '%s\n' "$out_plain" | grep -vE '…')

Validado no /bin/bash 3.2 de fábrica do macOS, sed POSIX, sem perl (o gate já
degrada quando perl falta; a normalização não pode herdar essa dependência):

ESC=$(printf '\033')
strip_ansi() { sed "s/${ESC}\[[0-9;?]*[a-zA-Z]//g"; }

Com isso, os três casos que hoje falham voltam a casar, e o filtro de epílogo volta a ver
npm ERR!.

4. O que foi considerado e recusado

Forçar NO_COLOR=1 / desligar FORCE_COLOR no ambiente do filho. Recusado: muda o
comportamento do comando do usuário, e o gate observa, não altera. Suíte que troca de
caminho sob CI/NO_COLOR passaria a ser medida diferente do que roda de verdade. Além
disso não resolve: um script com --color explícito no package.json colore assim mesmo.
Vale no máximo como defesa em profundidade depois da normalização, nunca no lugar dela.

Ancorar as regex com (\x1b\[[0-9;]*m)* em cada ponto. Recusado: seis lugares para
manter em sincronia, e o próximo detector nasce sem.

5. Sequência de execução

Passo 0 — teste vermelho primeiro. Fixtures em gate_test.sh sob
FORCE_COLOR=1/CLICOLOR_FORCE=1 para: passWithNoTests colorido, node:test colorido
(\e[34mℹ tests 0\e[39m), epílogo do pnpm/yarn colorido, e o caso misto do item 2.
Elas têm de reprovar no código de hoje — é o critério que a #37 fixou como cultura do
repo e que a seção 14 do coherence usou como método.

Passo 1 — strip_ansi + out_plain, um ponto só. Verde nas fixtures novas.

Passo 2 — varredura dos outros stacks. Já verifiquei: Go, Rust, Python, JVM, .NET e
Ruby decidem suíte vazia por evidência de sistema de arquivos (*_test.go,
tests/*.rs, #[test], tests/, marcadores em csproj), não por saída de runtime — são
imunes por construção. Registrar isso no comentário do código evita que alguém "conserte"
o que não está quebrado, e avisa quem adicionar um stack novo.

Passo 3 — invariante que morde. O coherence_test.sh deve travar a regra nova: nenhum
ponto de classificação lê $out direto. Forma sugerida — derivar do próprio gate.sh que
todo grep/awk de classificação dentro de js_script consome $out_plain ou $ev, e
falhar se $out aparecer em posição de classificação. Provar por mutação, trocando um
$out_plain por $out e exigindo vermelho.

Passo 4 — contagem publicada nos dois READMEs (127/127 é verificado por máquina;
sobe com as fixtures novas), CHANGELOG, bump de versão.

6. Relação com a #36 (mesmo milestone)

Fazer #35 primeiro, na ordem literal. A #36 questiona a regra da falha encadeada
silenciosa, que roda sobre $ev — e $ev muda de conteúdo no passo 1, porque o filtro de
epílogo volta a funcionar. Decidir a #36 sobre o $ev de hoje é decidir sobre uma entrada
que está prestes a mudar. As duas também tocam o mesmo bloco (js_script, a maior função
do repo com 139 linhas, medida pelo metrics.sh).

7. Pronto quando

• Repo com zero arquivo de teste não atinge GREEN sob nenhuma combinação de
  NO_COLOR/FORCE_COLOR/CLICOLOR_FORCE.
• Suíte que falhou com marcador colorido classifica RED, nunca uncounted.
• As fixtures novas reprovam no gate.sh anterior (provado, não afirmado).
• Invariante impede classificar saída não normalizada, e reprova sob mutação.
• scripts/test.sh verde no macOS e no container Linux; contagens publicadas
  batendo.
• O corpo desta issue corrigido: "a direção é segura" é falso, e agora há uma segunda
  direção documentada.

8. Risco de não fazer

FORCE_COLOR=1 e CLICOLOR_FORCE=1 estão setados por harness de agente — verifiquei no
ambiente onde esta skill roda. Não é borda de CI, é a condição provável de execução. E
GREEN é o nível que libera deleção autônoma de exports e as fases 2, 3 e 4. A frase que
sustenta o cap no SKILL.md — "uma suíte que não existe não pode passar" — é falsa hoje.

[CRangelP]

Fechada pelo PR #46, mergeado em main (f4d59fb).

Correção do corpo desta issue

O corpo afirmava que "a direção é segura (nunca vira GREEN)". Era falso, e a correção está
no comentário acima com a reprodução. Além disso, a issue descrevia um defeito e existiam
dois, em direções opostas:

cenário	antes	depois
repo com zero teste, saída colorida	GREEN — libera deleção autônoma	'test' not counted, YELLOW cap
suíte que falhou, FAIL colorido	YELLOW cap	RED

O segundo apareceu ao investigar o primeiro: a guarda de falha real é ancorada em
(^|[[:space:]]) e sob cor o byte antes de FAIL é o m do escape.

O que foi entregue

Normalização única na fronteira de captura (strip_ansi → out_plain), não sete regexes
remendadas — há um só ponto de captura e todos os consumidores derivam dele. A saída que o
usuário vê continua colorida: o gate observa, não reescreve, e há um teste que trava essa
decisão.

Oito casos novos no gate_test.sh, quatro deles reprovando o gate.sh anterior (129/133
contra a cópia pristina de HEAD). Seção 15 do coherence_test.sh impedindo que qualquer
classificação futura leia a captura crua.

Três defeitos que a própria revisão do PR encontrou depois

Vale registrar porque nenhum estava no plano original:

• sed abortando sob UTF-8 (illegal byte sequence) esvaziava out_plain e fazia o GREEN
  reentrar pela porta do conserto. LC_ALL=C resolve, e o default do macOS era o lado inseguro.
• OSC 8 (hyperlink) reproduzia o mesmo defeito por outro escape, e não precisa de TTY.
• \r de spinner mata a âncora ^ do mesmo jeito.

E um defeito pré-existente adjacente: o filtro de epílogo conhecia só npm ERR!, e o npm
renomeou para npm error na v10 — inerte em todo npm atual.

135/135 gate · 42/42 guarda · 5/5 rollback · 35/35 métricas · 296/296 invariantes

Restam a #36 neste milestone. A ordem recomendada continua valendo, e agora com um dado a mais:
$ev mudou de conteúdo com este merge — linhas antes escondidas por escapes agora são texto
visível — e a regra da falha encadeada tem sete consumidores, não seis. A #36 deve ser
decidida sobre esse $ev, não sobre o anterior.