`) — nenhum teste vivo produz os 3 nomes do purge; comentários "Onda0 shared" corrigidos; `withExclusiveTestDatabaseLease` colapsado (era cópia do privado); rollback no down-test quando a sessão falta; cleanup no finally nos testes F1 (auto-curáveis — um run falho do próprio teste não polui). Testes novos: contrato da janela exclusiva (waiter SHARED bloqueado), purge em ensure/setup, self-heal do canônico (repro do run abortado, 2 fixtures), F1b. **Contraprova:** 7 arquivos da key × 6 workers → **12/12 verdes** (baseline 1/4; designs intermediários: cascata 15s / FK); suíte int completa 83 arquivos/701 testes; unit 1938; e2e paridade com baseline (falhas pré-existentes de chat IA/webauthn/mic reproduzem no código original). Sem migration/access/Consent/`src/`. Intenção [`escala-dry-pos-d9.md`](plans/escala-dry-pos-d9.md); impl [`escala-dry-pos-d9-impl.md`](plans/escala-dry-pos-d9-impl.md).
diff --git a/docs/plans/escala-dry-pos-b197-impl.md b/docs/plans/escala-dry-pos-b197-impl.md
new file mode 100644
index 00000000..cbce4485
--- /dev/null
+++ b/docs/plans/escala-dry-pos-b197-impl.md
@@ -0,0 +1,133 @@
+# Impl: Assessores — menu do picker de colunas derivado da mesma fonte do AdvisorsTable
+
+Status: concluído (executado 2026-08-11)
+Atualizado em: 2026-08-11
+Issue: #642
+Intenção: docs/plans/escala-dry-pos-b197.md
+Appetite restante: ~0,5 dia (herdado)
+
+## Leitura da intenção
+
+- **Outcome:** o menu do picker de colunas de assessores nasce da mesma fonte
+ que os `| ` da tabela — adicionar coluna num lugar atualiza o outro de
+ graça. Zero mudança de comportamento.
+- **O que NÃO negociar:** mesmos ids (`name` mandatory, `email`, `phone`,
+ `municipalities`, `actions` — contrato do cookie `campaign_columns`), mesmos
+ labels pt-BR, mesma ordem, exceção do draft intacta (input de e-mail na
+ linha-draft mesmo com a coluna oculta), e o não-escopo travado de não portar
+ o `AdvisorsTable` para `CampaignTable`.
+- **O que reavaliar:** o plano de intenção já recomenda a Opção A (fonte única
+ no módulo da tabela); a reavaliação aqui é de engenharia fina — onde a
+ constante vive, como o ` | ` consome, como o `emailHidden` deriva.
+
+## Abordagem recomendada
+
+```mermaid
+flowchart LR
+ S[advisorTableColumns
AdvisorsTable.tsx] --> H[ | via resolveVisibleColumns]
+ S --> P[advisorPickerColumns =
toCampaignColumnPickerColumns]
+ P --> T[CampaignColumnPickerTrailing
assessores/page.tsx]
+```
+
+**Opções consideradas:** A | B | C | D
+**Recomendação:** A — porque segue o precedente em produção
+(`supporterColumns`/`supporterPickerColumns` em `SupporterList.tsx` — o RSC de
+`apoiadores` importa a constante serializável do módulo do componente),
+reusa os helpers B17 (`toCampaignColumnPickerColumns`, `resolveVisibleColumns`)
+e mantém o diff em 2 arquivos.
+**Rejeitadas:**
+
+- **B — lib nova (`src/lib/advisorTableColumns.ts`):** só há dois consumidores
+ e um é o mesmo módulo que renderiza; módulo lib com 1 call site real é
+ camada sem volatilidade (depth check — pass-through).
+- **C — status quo (comentário):** o achado que originou a Issue é exatamente
+ que o comentário é insuficiente; rejeitada pela intenção.
+- **D — extrair só id/label/mandatory e manter widths/sr-only no JSX:** deixa
+ dois lugares para tocar ao adicionar coluna (layout do header) e obrigaria
+ exceção manual no map para o caso `Ações`; a classe de header é propriedade
+ da coluna, declarada junto.
+
+### Decisões de engenharia
+
+1. **Fonte única em `AdvisorsTable.tsx` (client).** Novo tipo local
+ `AdvisorTableColumn { id, label, mandatory?, headerClassName?,
+visuallyHiddenLabel? }` e constante `advisorTableColumns` com as 5 colunas
+ atuais (incluindo `w-[20%]`/`w-[24%]`/`w-[14%]`/`w-28 text-right` e
+ `visuallyHiddenLabel: true` em `Ações`, que preserva o ``).
+2. **` | ` consomem `resolveVisibleColumns(advisorTableColumns,
+columnVisibility.hiddenColumnIds)`** — o mesmo helper B17 das outras listas:
+ filtra por cookie respeitando `mandatory`, e `emailHidden` passa a derivar
+ de `visibleColumns.some(id === 'email')`. **Descoberta do /simplify (fix
+ incorporado):** o corpo renderiza as células incondicionalmente (só `email`
+ tem ` | ` condicional + exceção do draft), então `phone`/`municipalities`/
+ `actions` são `mandatory: true` — um id escondido por cookie velho derruba
+ o ` | ` sobre um corpo de 5 células (table-fixed) se não for; o checkbox
+ do picker vira "sempre visível" (honesto em vez do toggle inerte do B197).
+ O corpo (células + exceção do draft) fica intocado.
+3. \*\*`export const advisorPickerColumns = toCampaignColumnPickerColumns(advisorTableColumns)`
+ no módulo da tabela — mesma forma de `supporterPickerColumns`.
+4. **`assessores/page.tsx` importa `advisorPickerColumns`** e passa direto ao
+ `CampaignColumnPickerTrailing`; removidos o array local, o comentário do
+ espelho e os imports órfãos (`toCampaignColumnPickerColumns`,
+ `CampaignColumnPickerColumn`).
+5. **e2e `campaignColumnPicker` (assessores):** asserts novos pinando o
+ contrato mandatory — `Nome`/`Celular` "sempre visível" (checked + disabled).
+
+### Componentes / mudanças
+
+- **`AdvisorsTable.tsx`** (`src/components/campaign/advisor/`): declara a
+ fonte única, exporta `advisorPickerColumns`, header por map; corpo intacto.
+- **`assessores/page.tsx`** (`src/app/(campaign)/campanha/(app)/assessores/`):
+ consome a fonte exportada.
+- **Migration:** sem migration.
+- **Access / Consent:** sem mudança (acesso já gateado no page).
+- **UI:** Impeccable A — sem superfície nova; refactor estrutura que preserva
+ DOM/aria (sr-only de `Ações` mantido).
+
+### Dados → forma
+
+- Forma escolhida: colunas como dados serializáveis no módulo do componente,
+ convertidas na fonte via helper B17. Alternativas rejeitadas: lib separada
+ (sem call site extra) e manutenção do espelho (o próprio defeito).
+
+## Fases verificáveis
+
+1. **Fonte única + header** — `AdvisorsTable.tsx`: tipo + constante + export +
+ map do header com `resolveVisibleColumns`; `emailHidden` derivado.
+2. **Consumo no page** — `assessores/page.tsx`: import + remoção do array
+ local e imports órfãos.
+3. **Gates** — `pnpm exec tsc --noEmit`, `pnpm lint`, `pnpm format:check`,
+ `pnpm test` (unit: `campaignColumnVisibility` — lib intacta), e2e
+ `campaignColumnPicker` (spec de assessores cobre exatamente os comportamentos
+ preservados: default oculto, draft com input, religar, persistência);
+ `pnpm exec knip` (removidos imports órfãos), `pnpm check:cycles`.
+4. **Fechamento** — status do plano de intenção → concluído; entrada curta em
+ `docs/CHANGELOG-AGENTS.md`.
+
+## Rabbit holes / Não escopo (engenharia)
+
+- Não portar `AdvisorsTable` para `CampaignTable` (decisão travada do B197).
+- Não mexer em labels/ordem/ids — contrato do cookie `campaign_columns`.
+- Não criar coluna nova, não tocar células do corpo além de `emailHidden`
+ derivado.
+- Sem novo unit test: o comportamento é idêntico e coberto pelo e2e existente;
+ testar a constante exigiria importar o módulo client inteiro (lucide/next/
+ sonner) — custo sem retorno para um espelho que deixou de existir.
+
+## Riscos e mitigação
+
+- **Import de client module em RSC:** precedente em produção
+ (`supporterPickerColumns` em `apoiadores/page.tsx`); valor serializável
+ (array de `{id,label,mandatory}`).
+- **Layout `table-fixed`:** classes de largura movem para o mesmo ` | `, na
+ mesma ordem — DOM e classes idênticos, só que gerados por map com `key`.
+- **`emailHidden` divergente do header:** eliminado por construção — deriva do
+ mesmo `visibleColumns` que renderiza o header.
+
+## Aceite de engenharia
+
+- [x] Aceite de produto da intenção ainda coberto (mesma fonte, zero mudança)
+- [x] Invariantes AGENTS/engineering-standards (edit the owner: a tabela já
+ possui o concern do menu; reusa lib B17)
+- [x] Testes de domínio: sem mudança de access/write path; e2e existente é o
+ guard do comportamento
diff --git a/docs/plans/escala-dry-pos-b197.md b/docs/plans/escala-dry-pos-b197.md
index cf6270f1..804b706c 100644
--- a/docs/plans/escala-dry-pos-b197.md
+++ b/docs/plans/escala-dry-pos-b197.md
@@ -1,6 +1,6 @@
# Escala e DRY pós-B197 (picker de colunas de assessores)
-Status: pendente (capture-review-debts pós-B197, gate humano confirmado)
+Status: concluído (Issue #642, impl aprovado e executado 2026-08-11)
Atualizado em: 2026-08-11
Item do roadmap: fill-in **B197+** pós-[#627](https://github.com/fsolla/teqo/issues/627)
Impeccable: A — sem superfície UI nova (DRY/estrutura)
diff --git a/src/app/(campaign)/campanha/(app)/assessores/page.tsx b/src/app/(campaign)/campanha/(app)/assessores/page.tsx
index 6cfca92c..4583372a 100644
--- a/src/app/(campaign)/campanha/(app)/assessores/page.tsx
+++ b/src/app/(campaign)/campanha/(app)/assessores/page.tsx
@@ -3,7 +3,7 @@ import config from '@payload-config'
import { getPayload } from 'payload'
import { AdvisorFilters } from '@/components/campaign/advisor/AdvisorFilters'
-import { AdvisorsTable } from '@/components/campaign/advisor/AdvisorsTable'
+import { AdvisorsTable, advisorPickerColumns } from '@/components/campaign/advisor/AdvisorsTable'
import { CampaignColumnPickerTrailing } from '@/components/campaign/shared/CampaignColumnPickerTrailing'
import { CampaignListFooter } from '@/components/campaign/shared/CampaignListFooter'
import {
@@ -11,10 +11,6 @@ import {
CampaignListResults,
} from '@/components/campaign/shared/CampaignListPending'
import { CampaignPageShell } from '@/components/campaign/shell/CampaignPageShell'
-import {
- toCampaignColumnPickerColumns,
- type CampaignColumnPickerColumn,
-} from '@/lib/campaignColumnVisibility'
import { campaignPageMetadataFromCatalog } from '@/lib/campaignPageChrome'
import { getMunicipalityCatalogEntry } from '@/lib/municipalityCatalog'
import { hasAdvisorListActiveFilters } from '@/utilities/advisor/advisorListFilters'
@@ -40,17 +36,6 @@ type AdvisorsPageProps = {
searchParams: Promise>
}
-// B197 — labels mirror the `| ` copy of `AdvisorsTable`; the table's own
-// columns are hand-rolled (not `CampaignTableColumn`), so the picker menu is
-// spelled out here instead of derived.
-const advisorPickerColumns: readonly CampaignColumnPickerColumn[] = [
- { id: 'name', label: 'Nome', mandatory: true },
- { id: 'email', label: 'E-mail' },
- { id: 'phone', label: 'Celular' },
- { id: 'municipalities', label: 'Municípios' },
- { id: 'actions', label: 'Ações' },
-]
-
export default async function AdvisorsPage({ searchParams }: AdvisorsPageProps) {
const rawSearchParams = await searchParams
const [, payload] = await Promise.all([
@@ -82,7 +67,7 @@ export default async function AdvisorsPage({ searchParams }: AdvisorsPageProps)
trailing={
}
/>
diff --git a/src/components/campaign/advisor/AdvisorsTable.tsx b/src/components/campaign/advisor/AdvisorsTable.tsx
index 32032355..d35dfd19 100644
--- a/src/components/campaign/advisor/AdvisorsTable.tsx
+++ b/src/components/campaign/advisor/AdvisorsTable.tsx
@@ -34,7 +34,11 @@ import {
TableHeader,
TableRow,
} from '@/components/ui/Table'
-import type { CampaignColumnVisibility } from '@/lib/campaignColumnVisibility'
+import {
+ resolveVisibleColumns,
+ toCampaignColumnPickerColumns,
+ type CampaignColumnVisibility,
+} from '@/lib/campaignColumnVisibility'
import { campaignInlineInputClassName } from '@/lib/campaignInlineInput'
import type { MunicipalityPortfolioIndexEntry } from '@/lib/municipalityPortfolio'
import {
@@ -54,6 +58,37 @@ type DraftAdvisor = {
municipalityIds: number[]
}
+// B197+ — single source for the ` | `s and the picker menu: a column added
+// here appears in both, and `resolveVisibleColumns` keeps the cookie filtering
+// honest (a `mandatory` column is never dropped). Only what the body renders
+// conditionally is hideable: the cells are hand-wired per column id (draft row
+// AND data row), so `email` is the sole non-mandatory column — the others must
+// stay mandatory or a hidden id would drop the header under a 5-cell body.
+type AdvisorTableColumn = {
+ id: string
+ label: string
+ mandatory?: boolean
+ headerClassName?: string
+ /** Renders the label visually-hidden — the icon-only actions column. */
+ visuallyHiddenLabel?: boolean
+}
+
+const advisorTableColumns: readonly AdvisorTableColumn[] = [
+ { id: 'name', label: 'Nome', mandatory: true, headerClassName: 'w-[20%]' },
+ { id: 'email', label: 'E-mail', headerClassName: 'w-[24%]' },
+ { id: 'phone', label: 'Celular', mandatory: true, headerClassName: 'w-[14%]' },
+ { id: 'municipalities', label: 'Municípios', mandatory: true },
+ {
+ id: 'actions',
+ label: 'Ações',
+ mandatory: true,
+ headerClassName: 'w-28 text-right',
+ visuallyHiddenLabel: true,
+ },
+]
+
+export const advisorPickerColumns = toCampaignColumnPickerColumns(advisorTableColumns)
+
type AdvisorsTableProps = {
rows: AdvisorRowViewModel[]
municipalityIndex: MunicipalityPortfolioIndexEntry[]
@@ -108,10 +143,15 @@ export const AdvisorsTable = ({
const [isCreating, startCreate] = useTransition()
// B197 — email starts hidden on this list (cookie default); the picker
- // writes the cookie and the server re-renders with a fresh value. The
- // create-draft row below is the ONE exception: email is required to create
- // an advisor, so its input renders even while the column is hidden.
- const emailHidden = columnVisibility.hiddenColumnIds.includes('email')
+ // writes the cookie and the server re-renders with a fresh value. B197+:
+ // the visible set is what the ` | `s render, so this exception below can
+ // never disagree with the header. The create-draft row is the ONE exception:
+ // email is required to create an advisor, so its input renders always.
+ const visibleColumns = resolveVisibleColumns(
+ advisorTableColumns,
+ columnVisibility.hiddenColumnIds,
+ )
+ const emailHidden = !visibleColumns.some((column) => column.id === 'email')
const refresh = () => router.refresh()
@@ -220,13 +260,15 @@ export const AdvisorsTable = ({
- Nome
- {!emailHidden ? E-mail : null}
- Celular
- Municípios
-
- Ações
-
+ {visibleColumns.map((column) => (
+
+ {column.visuallyHiddenLabel ? (
+ {column.label}
+ ) : (
+ column.label
+ )}
+
+ ))}
diff --git a/tests/e2e/campaignColumnPicker.e2e.spec.ts b/tests/e2e/campaignColumnPicker.e2e.spec.ts
index 16d1fcb8..c01808eb 100644
--- a/tests/e2e/campaignColumnPicker.e2e.spec.ts
+++ b/tests/e2e/campaignColumnPicker.e2e.spec.ts
@@ -172,6 +172,17 @@ test.describe('E-mail oculto por padrão nas listas de pessoas (B197)', () => {
await page.getByRole('button', { name: 'Novo assessor', exact: true }).click()
await expect(page.getByLabel('E-mail do novo assessor')).toBeVisible()
+ // B197+ — the table body only conditions the email column, so every other
+ // column is mandatory in the single source: their checkboxes are locked.
+ await picker.click()
+ const nameCheckbox = page.getByRole('checkbox', { name: 'Nome sempre visível' })
+ await expect(nameCheckbox).toBeChecked()
+ await expect(nameCheckbox).toBeDisabled()
+ const phoneCheckbox = page.getByRole('checkbox', { name: 'Celular sempre visível' })
+ await expect(phoneCheckbox).toBeChecked()
+ await expect(phoneCheckbox).toBeDisabled()
+ await page.keyboard.press('Escape')
+
// Re-enabling paints the column and the choice survives a reload.
await picker.click()
await page.getByRole('checkbox', { name: 'E-mail', exact: true }).click()
| |