From e323a7b051efb05e92a0b29000021cb421ccfb1c Mon Sep 17 00:00:00 2001
From: Francisco Solla <fsolla@pm.me>
Date: Wed, 16 Sep 2026 18:47:43 -0300
Subject: [PATCH] =?UTF-8?q?docs(C178):=20corrige=20o=20diagn=C3=B3stico=20?=
 =?UTF-8?q?do=20d=C3=A9bito=20OPS124=20(knip=20sai=200,=20an=C3=A1lise=20d?=
 =?UTF-8?q?egradada=20em=20CI=20e=20local)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 docs/changelog/2026-09-16-c178.md             |  2 +-
 ...layer-youtube-sem-parede-de-signin-impl.md |  2 +-
 docs/plans/ops-knip-importmap-artifact.md     | 20 ++++++++++---------
 3 files changed, 13 insertions(+), 11 deletions(-)

diff --git a/docs/changelog/2026-09-16-c178.md b/docs/changelog/2026-09-16-c178.md
index 7328e1f3..f45c7176 100644
--- a/docs/changelog/2026-09-16-c178.md
+++ b/docs/changelog/2026-09-16-c178.md
@@ -1 +1 @@
-**C178 — Acervo: o player do YouTube deixa de ser uma parede de signin (2026-09-16):** o defeito continuava após o C171/#1047: abrir uma fala com vídeo ainda caía primeiro no embed do YouTube, que responde "faça login para exibir o vídeo" — e o app não vê o erro interno do iframe (sem `onError`/`enablejsapi`), então a saída não podia depender de detectar o bloqueio. A resolução é remover a porta de entrada embutida: no `SpeechDetailPlayer`, o `<iframe>` do YouTube sai de cena e a superfície passa a ser derivada do dado — com trecho da Câmara (`vodResolvable`) o caminho padrão é a própria Câmara (painel "Assistir o trecho" resolvido sob clique, reuso integral do C162/C169/C171, sem POST no render), sem trecho a fala ganha uma **capa clicável** (thumbnail real do YouTube sobre gradiente, `data-slot="speech-youtube-facade"`) que abre `watch?v=<id>&t=<segundos>` em nova aba, e sem nenhum dos dois permanece o estado honesto "Vídeo indisponível neste momento". O bloco de saída "Abrir no YouTube" (`buildSpeechExcerptYoutubeUrl`, `t=` omitido quando o offset é desconhecido) fica **sempre visível**, inclusive em resolvendo/gerando/falha, e vira a ação primária assim que a Câmara sai do estado ocioso (o download recua para outline, como no design); os links externos usam `target="_blank" rel="noreferrer"`. A transcrição passa a posicionar apenas a superfície que o app controla (`seekable = Boolean(playbackUrl)`; sem `<video>` as frases ficam inertes fora do modo de seleção), e seleção/share/corte/download (C166/C167) seguem intactos. Código morto removido no mesmo delivery: `surface`, `youtubeStart`, `buildYoutubeSrc`, `watchVod`, `inlineNotice` e `pendingSeekRef`. Sem `Consent`, collection, migration, URL pública nova ou escrita; gate do acervo e crédito "Fonte: Câmara dos Deputados · CC BY 4.0" intocados. Testes: unit RTL do player reescrito nos quadrantes (nunca iframe, Câmara default com ambos, capa + link externo no ponto, saída visível em loading/falha, deep link) e e2e browserless do acervo ajustado (both-sources sem `youtube.com/embed`, caso só-YouTube novo, VOD-only e sem-vídeo). Design: crítica do `designer` (tier `openai/gpt-5.6-sol`) não certificou a paridade — divergências materiais aceitas pelo humano: a transcrição permanece no estado só-YouTube e o aviso C166 "Compartilhar por link exige o vídeo no YouTube" permanece no só-VOD (superfícies pré-existentes fora da cobertura das cenas do artefato). Débitos registrados: **C179 (#1093)** helper único do thumbnail `hqdefault` (4 módulos; destrava com o C172) e **OPS124 (#1094)** knip local quebra com o `importMap` gitignored no working tree.
+**C178 — Acervo: o player do YouTube deixa de ser uma parede de signin (2026-09-16):** o defeito continuava após o C171/#1047: abrir uma fala com vídeo ainda caía primeiro no embed do YouTube, que responde "faça login para exibir o vídeo" — e o app não vê o erro interno do iframe (sem `onError`/`enablejsapi`), então a saída não podia depender de detectar o bloqueio. A resolução é remover a porta de entrada embutida: no `SpeechDetailPlayer`, o `<iframe>` do YouTube sai de cena e a superfície passa a ser derivada do dado — com trecho da Câmara (`vodResolvable`) o caminho padrão é a própria Câmara (painel "Assistir o trecho" resolvido sob clique, reuso integral do C162/C169/C171, sem POST no render), sem trecho a fala ganha uma **capa clicável** (thumbnail real do YouTube sobre gradiente, `data-slot="speech-youtube-facade"`) que abre `watch?v=<id>&t=<segundos>` em nova aba, e sem nenhum dos dois permanece o estado honesto "Vídeo indisponível neste momento". O bloco de saída "Abrir no YouTube" (`buildSpeechExcerptYoutubeUrl`, `t=` omitido quando o offset é desconhecido) fica **sempre visível**, inclusive em resolvendo/gerando/falha, e vira a ação primária assim que a Câmara sai do estado ocioso (o download recua para outline, como no design); os links externos usam `target="_blank" rel="noreferrer"`. A transcrição passa a posicionar apenas a superfície que o app controla (`seekable = Boolean(playbackUrl)`; sem `<video>` as frases ficam inertes fora do modo de seleção), e seleção/share/corte/download (C166/C167) seguem intactos. Código morto removido no mesmo delivery: `surface`, `youtubeStart`, `buildYoutubeSrc`, `watchVod`, `inlineNotice` e `pendingSeekRef`. Sem `Consent`, collection, migration, URL pública nova ou escrita; gate do acervo e crédito "Fonte: Câmara dos Deputados · CC BY 4.0" intocados. Testes: unit RTL do player reescrito nos quadrantes (nunca iframe, Câmara default com ambos, capa + link externo no ponto, saída visível em loading/falha, deep link) e e2e browserless do acervo ajustado (both-sources sem `youtube.com/embed`, caso só-YouTube novo, VOD-only e sem-vídeo). Design: crítica do `designer` (tier `openai/gpt-5.6-sol`) não certificou a paridade — divergências materiais aceitas pelo humano: a transcrição permanece no estado só-YouTube e o aviso C166 "Compartilhar por link exige o vídeo no YouTube" permanece no só-VOD (superfícies pré-existentes fora da cobertura das cenas do artefato). Débitos registrados: **C179 (#1093)** helper único do thumbnail `hqdefault` (4 módulos; destrava com o C172) e **OPS124 (#1094)** knip carrega `payload.config.ts` com erro (`importMap.js` commitado) e roda com análise degradada em CI e local (exit 0).
diff --git a/docs/plans/acervo-player-youtube-sem-parede-de-signin-impl.md b/docs/plans/acervo-player-youtube-sem-parede-de-signin-impl.md
index 425053b9..14b2b911 100644
--- a/docs/plans/acervo-player-youtube-sem-parede-de-signin-impl.md
+++ b/docs/plans/acervo-player-youtube-sem-parede-de-signin-impl.md
@@ -117,7 +117,7 @@ flowchart LR
 
 ## Triage pós-simplify + crítica de design (não reabrir)
 
-**Registrado:** C179 (#1093) — helper único da URL do thumbnail `hqdefault` (4 módulos; `depends: C172`); OPS124 (#1094) — `pnpm knip` local quebra com o `importMap` gitignored no working tree (pré-existente, não é regressão desta entrega).
+**Registrado:** C179 (#1093) — helper único da URL do thumbnail `hqdefault` (4 módulos; `depends: C172`); OPS124 (#1094) — knip carrega `payload.config.ts` com erro (`importMap.js` commitado) e roda com análise degradada em CI e local, mas **sai com código 0** (a hipótese inicial de que quebrava o `gate:ci` local foi corrigida na própria Issue; pré-existente, não é regressão desta entrega).
 
 **Defer com gatilho:** duplicação do markup do bloco de saída (`renderYoutubeExit`) — revisitar com uma 3ª variante/superfície de saída ou reuso fora do `SpeechDetailPlayer`; testes unit quase-duplicados (VOD-only vs both-sources) — descartado (legibilidade dos quadrantes).
 
diff --git a/docs/plans/ops-knip-importmap-artifact.md b/docs/plans/ops-knip-importmap-artifact.md
index ae329f12..920717c9 100644
--- a/docs/plans/ops-knip-importmap-artifact.md
+++ b/docs/plans/ops-knip-importmap-artifact.md
@@ -1,4 +1,4 @@
-# OPS: knip local quebra com o importMap (artefato gitignored) no working tree
+# OPS: knip carrega o payload.config.ts com erro (importMap commitado) e roda com análise degradada
 
 Status: rascunho
 Atualizado em: 2026-09-16
@@ -9,23 +9,25 @@ Appetite: ~30 min eng; sem schema, sem deploy
 
 ## Intenção
 
-`pnpm knip` falha localmente com `ERROR: Error loading src/payload.config.ts (This module cannot be imported from a Client Component module. It should only be used from a Server Component.)` quando `src/app/(payload)/admin/importMap.js` existe no working tree — artefato gitignored gerado por `pnpm predev`/build (OPS99), que `knip.json` ainda lista em `entry`. Sem o artefato, o knip completa (com os 2 unresolved imports esperados de `[[...segments]]`). No CI o knip roda antes do build (sem o artefato) e passa — confirmado no run `35099887216`; reproduzido idêntico no worktree `main` (`/home/fsolla/Code/teqo`), então **não é regressão de entrega nenhuma**. Efeito: `pnpm push`/`gate:ci` local quebra depois de rodar dev.
+`pnpm knip` imprime `ERROR: Error loading src/payload.config.ts (This module cannot be imported from a Client Component module. It should only be used from a Server Component.)` e **sai com código 0** — acontece no CI (run `35099887216`) e local, sempre que `src/app/(payload)/admin/importMap.js` está no working tree. Esse arquivo está **commitado** (apesar de `knip.json`/OPS99/`.gitignore` o tratarem como artefato de build), importa componentes client e faz o knip carregar `payload.config.ts`; o `server-only` resolve para o entry que lança sob as condições padrão (não `react-server`), e a regra `paths.server-only` do `knip.json` não está sendo aplicada ao specifier de pacote. Sem o arquivo, o knip completa (com 2 unresolved imports esperados de `[[...segments]]`). Por sair 0, o gate fica verde com o knip **degradado em silêncio**: um ERROR que ninguém lê e uma análise possivelmente incompleta (o entry falhou ao carregar).
+
+Correção de rumo (2026-09-16): durante o C178 eu supus que isso quebrava `pnpm push`/`gate:ci` local; na prática o exit é 0 e o push não é afetado. O problema real é o erro silencioso/análise degradada.
 
 ## Fases verificáveis
 
-1. **Diagnóstico + fix** — decidir entre remover `src/app/(payload)/admin/importMap.js` do `entry` do `knip.json` (o arquivo é artefato de build; os unresolved imports de `[[...segments]]` apontam para um artefato inexistente) ou `ignore`-á-lo com justificativa. Provar: `pnpm predev` (gera o artefato) → `pnpm knip` verde; e `pnpm knip` verde sem o artefato.
-2. **Gate** — `pnpm gate:fast` + `pnpm push`; sem tocar no pipeline de CI (o CI já passa).
+1. **Diagnóstico + fix** — confirmar a causa (mapping `paths.server-only` não aplicado vs. `importMap.js` commitado) e corrigir: (a) fazer o mapeamento funcionar (ou trocar por `compilerOptions.paths`/condição `react-server` no knip) e/ou (b) destrackear o `importMap.js` (OPS99 diz que não deveria estar commitado — foi commitado em `fea6c645`/`b03cec7c` para corrigir o admin em branco). Provar: `pnpm knip` sem a linha `Error loading src/payload.config.ts` e com a análise completa (um export morto de teste é detectado).
+2. **Gate** — `pnpm gate:fast` + `pnpm push`; sem tocar no pipeline de CI além do necessário.
 
 ## Já resolvido no simplify/critique (não reabrir)
 
-- Confirmado que o erro é pré-existente e independente do C178 (mesmo erro em `main` com o artefato presente).
+- Confirmado que o comportamento é idêntico no CI e local (o `importMap.js` está commitado, então o CI também vê o arquivo) e que o exit code é 0 — não é um gate quebrado.
 
 ## Explicitamente fora
 
-- Mudar a ordem do CI (knip antes do build já é o caso).
-- Commitar o `importMap.js` (OPS99 proíbe; é artefato de build).
-- Qualquer outra limpeza de `knip.json`.
+- Mudar a ordem do CI (knip antes do build já é o caso; e o build não é o gerador do arquivo no repo).
+- Limpar outros pontos do `knip.json`.
 
 ## Self-score (decisão)
 
-4/5 — fix barato e reversível, causa-raiz nomeada, prova verde clara, sem risco de produção.
+4/5 — fix barato e reversível, causa-raiz nomeada (com correção de rumo registrada), prova objetiva (o ERROR some + export morto detectado), sem risco de produção.
+