diff --git a/docs/CHANGELOG-AGENTS.md b/docs/CHANGELOG-AGENTS.md index 36e695af..fe05b565 100644 --- a/docs/CHANGELOG-AGENTS.md +++ b/docs/CHANGELOG-AGENTS.md @@ -4,6 +4,8 @@ Conteúdo movido do `AGENTS.md` em 2026-07-30 (fatiamento do paradigma de agente --- +**Recently resolved (2026-08-10):** **B193** (Issue #542) — card de município mobile denso + edit-where-you-see + última atualização expandível. O card de `/campanha/municipios` no mobile deixou o grid de labels e virou composição do wireframe: cabeçalho nome+território com **posição de voto 2022 à direita** (percentual em cima; colocação · votos embaixo), **barra de cenários** (pessimista·central·otimista + marcador do ativo — `VoteEstimateScenarioStrip` como trigger da sheet de votos existente), **chips** classe/tendência/nível (tendência e nível com label à esquerda, dentro do trigger — tocar no label abre a sheet; classe com razão via tooltip que abre no toque), **3 grupos de avatares com label** (Assessores/Lideranças/Dobradinhas — um avatar por entidade, wrap, sem cap; leitura de assessores para não-coordenador), rodapé **"Última atualização há X dias" + chevron que expande o limite inferior** revelando o último card de atualização (badges + corpo + autor · data, sem moldura) + CTA "Registrar atualização" (sheet existente); **sem atualização → o rodapé vira CTA direto** (sem chevron/expansão); prioritário = **borda lateral direita 6px** `border-primary` no lugar do flag; Salvador read-only; "Cobertura" (conta da cadeira) sai do card (fica na tabela/detalhe — gate humano); rodapé mede a idade da **atualização** (não o frescor update∨pledge — coluna desktop intacta). **C109 corrigida no caminho (a "sheet quebra ao abrir" era real):** os callbacks dos controles (autosave `onOpenChange`, `handleOpenChange` do nível) são recriados por render — qualquer re-render com a sheet aberta (digitar no sheet!) fazia o host descartar a sessão e o corpo sumia; sessões do host agora são **keyed por id de sessão** (imune a identidade) e o **footer customizado** (submit + `DialogClose`) passou a ser **registrado e renderizado pelo host dentro da árvore do Drawer** (portal perdia o Dialog context — o e2e antigo só abria a sheet de tendência, sem footer custom, por isso a C109 não reproduzia). Controles ganharam prop `trigger`/`triggerClassName` (precedente `RelationChipCell.trigger`); `MunicipalityRelationAvatarStack` ganhou modo wrap. Dado: `MunicipalityListViewModel.lastUpdate` (batch por página via `loadMunicipalityLastUpdates`, scoped por portfólio). E2E novos (390×844): barra abre sheet e reflete valores salvos, chip Nível abre a sheet, borda de prioridade (6px + cor), expansão/CTA vazio. Sem migration/access/Consent. Intenção [`municipios-card-mobile-edit-where-you-see.md`](plans/municipios-card-mobile-edit-where-you-see.md); impl [`municipios-card-mobile-edit-where-you-see-impl.md`](plans/municipios-card-mobile-edit-where-you-see-impl.md). + **Recently resolved (2026-08-10):** **OPS26** (Issue #571) — `worktree next`/`plan` **no terminal** terminam com o TUI do opencode aberto no worktree: a função `worktree()` de `.agents/shell/worktree.sh` marca `TEQO_WORKTREE_TERMINAL=1` e o script imprime a diretiva `launch opencode --model deepseek/deepseek-v4-flash --auto [--prompt /work-issue]` **antes** do `cd` (que segue sendo a última linha — contrato OPS24 intacto); a função aplica o cd e executa a linha (sem `exec`: sair do TUI volta ao shell no worktree; sem auto-`kill`). `next` envia `/work-issue` (o comando OPS25 roda o ciclo completo); `plan` abre **sem prompt** — a CLI do opencode não tem pre-fill sem submit (`--prompt` sempre auto-envia; fallback acordado: `/plan-` + autocomplete; gap registrado para o repo do opencode). Presets são constantes em `scripts/lib/worktree.mjs` (`OPENCODE_PRESET_MODEL`, `OPENCODE_SKILL_COMMAND_BY_PURPOSE`, `opencodeLaunchDirective` — pura, unit-testada; trocar de modelo é editar a constante). **Sem o marcador (comando `/worktree` do opencode, automação) a diretiva não é impressa** — nunca abre TUI aninhado; `--stay` suprime cd e launch; `kill` intocado. Docs: AGENTS.md ("Per-worktree environments"), header/uso do script, `.opencode/commands/worktree.md`, skill `worktree-next-issue`. Sem migration/dados/access/UI. Intenção [`worktree-go-abre-opencode-com-presets.md`](plans/worktree-go-abre-opencode-com-presets.md); impl [`worktree-go-abre-opencode-com-presets-impl.md`](plans/worktree-go-abre-opencode-com-presets-impl.md). **Recently resolved (2026-08-10):** **OPS24** (Issue #569) — `--go` virou o **padrão** do worktree CLI e `kill` volta ao main: `worktree next`/`plan` imprimem `cd ` na última linha **por padrão** (antes exigiam `--go`); `--stay` suprime; `--go` explícito continua aceito como no-op (não quebra hábito/automação). `kill` troca a mensagem "Volte ao main: cd …" (que nunca casava com o sed `s/^cd //p` da função shell) pela **linha máquina `cd
` como última linha** — depois do drop dos DBs, que rodava antes — e **não aceita `--stay`** (ficar em cwd destruído é exatamente o bug corrigido; `--go` no kill é no-op). Vale nas duas superfícies que aplicam o cd: função `worktree()` de `.agents/shell/worktree.sh` (agora sempre captura/applica, `--stay` retorna cedo) e comando `/worktree` do opencode (agente aplica o `cd` da última linha sempre, exceto `--stay`). Docs: header/uso do script, AGENTS.md ("Per-worktree environments"), skills `worktree-next-issue`/`local-database`. Sem migration/dados/access; derivação pura (`worktree-env.mjs`/`worktree.mjs` libs) intocada. Intenção [`worktree-go-default-e-kill-volta-ao-main.md`](plans/worktree-go-default-e-kill-volta-ao-main.md); impl [`worktree-go-default-e-kill-volta-ao-main-impl.md`](plans/worktree-go-default-e-kill-volta-ao-main-impl.md). diff --git a/docs/plans/municipios-card-mobile-edit-where-you-see-impl.md b/docs/plans/municipios-card-mobile-edit-where-you-see-impl.md new file mode 100644 index 00000000..493dd652 --- /dev/null +++ b/docs/plans/municipios-card-mobile-edit-where-you-see-impl.md @@ -0,0 +1,99 @@ +# Impl: B193 — Card de município mobile: composição densa + edit-where-you-see + última atualização expandível + +Status: aprovado +Atualizado em: 2026-08-10 +Issue: #542 +Intenção: docs/plans/municipios-card-mobile-edit-where-you-see.md +Appetite restante: herdado (~1–1,5 dia eng; recomposição de card + fiação de affordances sobre sheets existentes) + +## Leitura da intenção + +- **Outcome:** no mobile (`md:hidden`), o card de `/campanha/municipios` vira composição densa na ordem do wireframe (cabeçalho nome+território com posição de voto 2022 à direita; barra de cenários com marcador do ativo; chips classe/tendência/nível; pilhas de avatares sem cap; rodapé última atualização com chevron) com edit-where-you-see em bottom sheets já existentes; toque atravessado abre o detalhe; prioritário ganha borda lateral direita ~6px; rodapé expande revelando o último card de atualização + CTA de registrar; sem atualização → CTA direto. +- **O que NÃO negociar:** assimetria staff/leader (estimativas nunca para liderança — página já é `gate: noLeader`); B13: classe nunca sem razão acessível; Salvador (linha agregada) read-only; desktop da lista e demais listas inalterados; não adicionar dado novo ao card; expansão ≠ feed completo; não recriar editors (reusar variantes `sheet`). +- **O que reavaliar:** (1) a intenção assume que os controles `variant="sheet"` servem o card novo como estão — os **triggers são internos** a cada controle, e a composição densa exige trigger customizado por alvo (barra de cenários, chip com label à esquerda, pilha de avatares com wrap); (2) a intenção não diz de onde vem o "último card de atualização" — o view model da lista **não carrega a última atualização** (só `lastUpdateAt`/`lastSignalAt`), então é dado novo no _load_ da página (não no card); (3) o rodapé hoje usa frescor (max atualização/pledge) — a expansão exige idade da **atualização**, não do sinal. + +## Abordagem recomendada + +```mermaid +flowchart LR + page["municipios/page.tsx
loadMunicipalityListPageBundle"] -->|"+ lastUpdate por página"| vm["MunicipalityListViewModel
+ lastUpdate: MunicipalityUpdateViewModel | null"] + vm --> cards["MunicipalityListMobileCards (novo)"] + cards --> card["MunicipalityMobileCard (client, estado expandido)"] + card --> bar["ExpectedVotesControl + trigger barra (VoteEstimateScenarioStrip)"] + card --> chips["Trend/Level controls + trigger chip label-esquerda"] + card --> avatars["RelationEditor + trigger pilha wrap (3 grupos)"] + card --> footer["Rodapé: expande | CTA direto (UpdateControl)"] +``` + +**Opções consideradas:** + +- **A — Controles ganham prop `trigger` opcional (recomendada).** Cada controle de edição (`MunicipalityListExpectedVotesControl`, `MunicipalityListLevelControl`, `MunicipalityListTrendControl`, `MunicipalityRelationEditor` + os 3 wrappers de relação) passa a aceitar um `trigger` customizado que substitui o interno; o `CampaignCellEditOverlay` já tem o `aria-label` que cobre a11y de qualquer conteúdo. O card compõe label+valor/barra/pilha dentro do botão, então o toque no **label** abre a sheet (aceite), e o trigger continua `relative` acima do overlay `after:inset-0` (invariante B42 documentado no próprio `CampaignCellEditOverlay`). +- **B — Card posiciona label e trigger como irmãos, e o label abre a sheet via handler próprio.** Duplica o estado de abertura do sheet (dois triggers para um overlay) e fura a maquinaria `openSheet/dismissSheet` do host compartilhado — o host guarda `activeOnOpenChangeRef` e o segundo trigger entraria em conflito. Rejeitada: é a C109 que se descartou; quebraria a invariante "um control = um onOpenChange". +- **C — Recriar o editor no card (componentes novos de sheet).** Rejeitada: rabbit hole "recriar editors" nomeado na intenção; anti-goal explícito. +- **D — Barra/pilhas como affordances estáticas + navegação para a página de edição.** Rejeitada: viola edit-where-you-see (editar "sem sair do card", aceite). + +**Decisões de engenharia:** + +1. **Dado da expansão — última atualização no view model** (custo de reverter: médio — toca loader e tipo). + - Opções: (a) `lastUpdate: MunicipalityUpdateViewModel | null` no `MunicipalityListViewModel`, carregado em batch por página; (b) fetch per-card no client (N+1, e o read precisa de access server-side); (c) `lastUpdateAt` sozinho (não dá o corpo/autoria do card). + - Recomendação: (a) — um `payload.find` em `municipalityUpdate` com `where: { municipality: { in: idsDaPágina } }`, `sort: '-createdAt'`, select enxuto, `overrideAccess: false` (o `canReadMunicipalityUpdate` escopa por portfólio), primeiro doc por município; nomes de autor via `loadCampaignUserNamesByIds` (mesmo padrão de `loadMunicipalityUpdatesFeed`). Alternativas rejeitadas: (b) porque explode o número de reads por página e expõe o access no client; (c) porque o card expandido precisa de corpo/polaridade/autor (aceite). + - Gatilho de revisitação (barato, não agora): se um município tiver centenas de updates, o batch puxa tudo — `DISTINCT ON` via drizzle quando houver medição (mesma política de `supporterListOverviewAggregate`). +2. **Cobertura (conta da cadeira) sai do card mobile.** O wireframe não a inclui; o card atual mostra "Cobertura" (goal coverage). Fica na tabela desktop e no detalhe. Sinalizo no gate como remoção de dado da superfície (não é "dado novo", é composição nova do wireframe). +3. **Rodapé passa a medir a idade da ATUALIZAÇÃO, não o frescor (update∨pledge).** + - O wireframe é explícito ("Última atualização há X dias", "sem nenhuma atualização registrada → CTA direto") e o estado vazio é chaveado na existência de update, não de pledge. O `SignalAgeReadout`/frescor continua na coluna desktop (`lastSignal`), intacto. A cor de "sinal frio" (`estimate-pending`) é mantida no rodapé quando a **atualização** está fria (≥21 dias), preservando o alerta da fila. + - Sinalizo como divergência pequena de produto no gate (a intenção herda o vocabulário frescor; troco para update-only no card). +4. **Borda de prioridade** — `border-r-[6px] border-primary` no `
` quando `isPriority && isStaffView && !isCity`, mantendo o `border-b` separador do B184. O `MunicipalityPriorityIndicator` (flag) sai do card mobile; a coluna desktop e o detalhe não mudam. +5. **Pilha de avatares com wrap** — estender `MunicipalityRelationAvatarStack` (owner, não twin) com modo wrap: `flex-wrap`, sem cap (`maxVisible` indefinido = todos), avatares densos (`size-7`); os três grupos (Assessores/Lideranças/Dobradinhas) são label + pilha dentro do trigger de cada relation editor via prop `trigger` (decisão A). +6. **Chip de classe** — mantém `TerritorialClassCardReadout` (só o rótulo + tooltip com a razão; `CampaignHoverTooltip` já abre no toque — `openOnTouch` default true, satisfaz "razão via tooltip/sheet" sem linha extra). Sem sheet nova. +7. **Barra de cenários** — trigger = `VoteEstimateScenarioStrip` com `markerMode="active-only"` + `labelMode="all"` (os três valores; ativo em destaque — igual ao canvas). Cenário ativo vem do `MunicipalityEstimateScenarioProvider` (já envolve a lista staff). Cidade: sem barra. +8. **Expansão** — estado `expanded` por card (`MunicipalityMobileCard` client); rodapé é ` + {expanded ? ( +
+
+
+ + {municipalityUpdatePolarityLabels[update.polarity]} + + {update.urgent ? Urgente : null} + {update.adversarySignal ? Adversário : null} +
+

+ {update.body ?? 'Sem texto.'} +

+

+ {update.authorName} + + {dateTimeFormatter.format(new Date(update.createdAt))} +

+
+ {updateControl(registerCta)} +
+ ) : null} + + ) : ( + updateControl(registerCta) + )} + + ) : null} +
+ ) +} diff --git a/src/components/campaign/municipality/MunicipalityVotePositionReadout.tsx b/src/components/campaign/municipality/MunicipalityVotePositionReadout.tsx index 0a7a9526..a9d96e58 100644 --- a/src/components/campaign/municipality/MunicipalityVotePositionReadout.tsx +++ b/src/components/campaign/municipality/MunicipalityVotePositionReadout.tsx @@ -11,16 +11,23 @@ export type MunicipalityVotePositionReadoutLayout = 'table' | 'card' | 'search' export const MunicipalityVotePositionReadout = ({ position, layout, + className, }: { position: MunicipalityVoteRankEntry layout: MunicipalityVotePositionReadoutLayout + /** B193 — the dense mobile card header right-aligns the card layout. */ + className?: string }) => { const share = formatVoteSharePercent(position.share) const rank = formatPlacementOrdinal(position.rank) + // B193 — the dense card reads "colocação · votos" per the wireframe; the + // table keeps "votos · colocação" (desktop untouched by B193). const metaLine = layout === 'search' ? formatElectionNumber(position.votes) - : `${formatElectionNumber(position.votes)} · ${rank}` + : layout === 'card' + ? `${rank} · ${formatElectionNumber(position.votes)}` + : `${formatElectionNumber(position.votes)} · ${rank}` const ariaLabel = `${share} da votação estadual, ${formatElectionNumber(position.votes)} votos, ${rank} de ${formatElectionNumber(position.totalUnits)}` return ( @@ -28,6 +35,7 @@ export const MunicipalityVotePositionReadout = ({ className={cn( 'flex flex-col gap-0.5 tabular-nums', layout === 'table' || layout === 'search' ? 'items-end text-right' : 'text-sm', + className, )} aria-label={ariaLabel} > diff --git a/src/components/campaign/shared/CampaignCellEditOverlay.tsx b/src/components/campaign/shared/CampaignCellEditOverlay.tsx index a0f6cf68..8a0e51c3 100644 --- a/src/components/campaign/shared/CampaignCellEditOverlay.tsx +++ b/src/components/campaign/shared/CampaignCellEditOverlay.tsx @@ -109,6 +109,28 @@ export const CampaignCellEditOverlay = ({ }: CampaignCellEditOverlayProps) => { const sharedSheet = useCampaignListSheet() const wasOpenRef = useRef(open) + /** C109 — the shared drawer session this control opened (see the host). */ + const sessionIdRef = useRef(null) + // Latest values for the mount-only cleanup below (never stale, no deps). + const sharedSheetRef = useRef(sharedSheet) + sharedSheetRef.current = sharedSheet + const onOpenChangeRef = useRef(onOpenChange) + onOpenChangeRef.current = onOpenChange + + // The control may unmount with its session open (a filter collapse behind + // the modal): dismiss the host so no orphan drawer keeps a frozen chrome + // and a dead body. The host's session guard makes the call a no-op when + // another control already took over. + useEffect( + () => () => { + const sheet = sharedSheetRef.current + const sessionId = sessionIdRef.current + if (sessionId !== null && sheet) { + sheet.dismissSheet(sessionId, onOpenChangeRef.current) + } + }, + [], + ) // Miss #52: without the provider every opened cell keeps its own Drawer // root. The fallback below stays for prod resilience, but in dev the @@ -179,23 +201,48 @@ export const CampaignCellEditOverlay = ({ wasOpenRef.current = open if (!open) { - if (wasOpen) sharedSheet.dismissSheet(onOpenChange) + if (wasOpen && sessionIdRef.current !== null) { + sharedSheet.dismissSheet(sessionIdRef.current, onOpenChange) + sessionIdRef.current = null + } return } - sharedSheet.openSheet({ - title, - description, - footer, - sheetBodyClassName, - onOpenChange, - }) + // Only the false→true transition opens a session: the controls' callbacks + // are recreated per render (autosave `onOpenChange`, level `handleOpenChange`), + // and calling `openSheet` on every re-render made the host dismiss the + // session it just started (C109 — "sheet quebra ao abrir"). Consequence: + // `title`/`description`/`hasCustomFooter` are frozen while the drawer is + // open — every caller passes static chrome, and the live parts (footer + // buttons/spinner) flow through `setFooterContent` below. + if (!wasOpen) { + sessionIdRef.current = sharedSheet.openSheet({ + title, + description, + hasCustomFooter: Boolean(footer), + sheetBodyClassName, + onOpenChange, + }) + } }, [variant, sharedSheet, open, title, description, footer, sheetBodyClassName, onOpenChange]) + // C109 — the custom footer is REGISTERED here and rendered by the host + // INSIDE the Drawer's React tree; a portal would keep the Dialog context + // away from its `DialogClose`. Re-registers on every change (spinner, + // label) while the session stays active; a stale session's call is a no-op + // in the host. + useLayoutEffect(() => { + if (variant !== 'sheet' || !sharedSheet) return + const sessionId = sessionIdRef.current + if (sessionId === null) return + sharedSheet.setFooterContent(sessionId, open ? (footer ?? null) : null) + }, [variant, sharedSheet, open, footer]) + if (variant === 'sheet' && sharedSheet) { - const showPortals = open && sharedSheet.isActiveSheet(onOpenChange) + const showPortals = + open && sessionIdRef.current !== null && sharedSheet.isActiveSheet(sessionIdRef.current) const bodyTarget = showPortals ? sharedSheet.bodyPortalRef.current : null - // portalRevision keeps this render in sync after the drawer mounts its target. + // portalRevision keeps this render in sync after the drawer mounts its targets. void sharedSheet.portalRevision return ( diff --git a/src/components/campaign/shared/CampaignListSheetHost.tsx b/src/components/campaign/shared/CampaignListSheetHost.tsx index 92c2b0ae..21c971f4 100644 --- a/src/components/campaign/shared/CampaignListSheetHost.tsx +++ b/src/components/campaign/shared/CampaignListSheetHost.tsx @@ -25,18 +25,7 @@ import { cn } from '@/lib/utils' export type CampaignListSheetChrome = { title: string description?: string - /** - * Custom footer for a sheet whose commit is an explicit submit (E14's - * "Registrar movimento", B42's "Registrar atualização"). Rendered INLINE in - * the host's DrawerFooter so its controls resolve the Drawer's context — - * the cell's own tree has no `Drawer.Root`, and a portaled footer keeps the - * source context and crashes base-ui's `DialogClose` (C109). The element is - * still CREATED by the cell (form state, `form` attribute association live - * there); only its render location moves — so it can rely on props/state - * captured by the cell and on providers ABOVE the sheet provider, never on - * providers local to the cell. - */ - footer?: ReactNode + hasCustomFooter: boolean sheetBodyClassName?: string onOpenChange: (open: boolean) => void } @@ -45,9 +34,19 @@ type CampaignListSheetContextValue = { bodyPortalRef: RefObject /** Bumps when the drawer body mount target attaches — portals read this. */ portalRevision: number - openSheet: (chrome: CampaignListSheetChrome) => void - dismissSheet: (onOpenChange: (open: boolean) => void) => void - isActiveSheet: (onOpenChange: (open: boolean) => void) => boolean + /** Opens the shared drawer; returns the session id the caller must keep. */ + openSheet: (chrome: CampaignListSheetChrome) => number + /** Closes the drawer only when `sessionId` is the active session. */ + dismissSheet: (sessionId: number, onOpenChange: (open: boolean) => void) => void + isActiveSheet: (sessionId: number) => boolean + /** + * C109 — the caller's custom footer (submit + `DrawerCloseButton`) must + * render INSIDE the drawer's React tree: the overlay lives outside the + * `Drawer`, so a portal keeps the Dialog context away from `DialogClose`. + * The host renders the registered node for the ACTIVE session only — a + * stale session's close render can never wipe the open drawer's footer. + */ + setFooterContent: (sessionId: number, footer: ReactNode | null) => void } const CampaignListSheetContext = createContext(null) @@ -57,9 +56,7 @@ export const useCampaignListSheet = () => useContext(CampaignListSheetContext) /** * One bottom Drawer for an entire list surface (B42 mobile cards). Cell bodies * portal into it so we never mount hundreds of Drawer roots and never sync form - * trees through provider state (which tripped React #185 in prod). Custom - * footers travel in the chrome instead of portaling: a portaled footer keeps - * the cell's context, which has no `Drawer.Root`, and crashes (C109). + * trees through provider state (which tripped React #185 in prod). */ export const CampaignListSheetProvider = ({ children }: { children: ReactNode }) => { const titleRef = useRef(null) @@ -67,27 +64,45 @@ export const CampaignListSheetProvider = ({ children }: { children: ReactNode }) const [open, setOpen] = useState(false) const [chrome, setChrome] = useState(null) const [portalRevision, setPortalRevision] = useState(0) + const [footerContent, setFooterContent] = useState(null) const activeOnOpenChangeRef = useRef<((open: boolean) => void) | null>(null) + /** + * C109 (2026-08-10, B193) — sheet sessions are keyed by a monotonically + * increasing id instead of the control's `onOpenChange` identity: the + * autosave/level callbacks are recreated per render, so ANY re-render of an + * open control (a keystroke, `isPending`) used to look like "a different + * control opened" — the host dismissed the old session, the drawer stayed + * up with its body portal torn down, and the next `DialogClose` crashed + * outside its root. A session id survives re-renders; only an explicit + * open/close moves it. + */ + const sessionCounterRef = useRef(0) + const activeSessionRef = useRef(null) const attachBodyPortal = useCallback((node: HTMLDivElement | null) => { bodyPortalRef.current = node if (node) setPortalRevision((value) => value + 1) }, []) - const dismissSheet = useCallback((onOpenChange: (open: boolean) => void) => { - if (activeOnOpenChangeRef.current !== onOpenChange) return + const dismissSheet = useCallback((sessionId: number, onOpenChange: (open: boolean) => void) => { + if (activeSessionRef.current !== sessionId) return setOpen(false) + activeSessionRef.current = null activeOnOpenChangeRef.current = null setChrome(null) + setFooterContent(null) onOpenChange(false) }, []) - const openSheet = useCallback((next: CampaignListSheetChrome) => { + const openSheet = useCallback((next: CampaignListSheetChrome): number => { + const session = ++sessionCounterRef.current const previousOnOpenChange = activeOnOpenChangeRef.current if (previousOnOpenChange && previousOnOpenChange !== next.onOpenChange) { previousOnOpenChange(false) } activeOnOpenChangeRef.current = next.onOpenChange + activeSessionRef.current = session + setFooterContent(null) setChrome((current) => { if ( current && @@ -95,35 +110,43 @@ export const CampaignListSheetProvider = ({ children }: { children: ReactNode }) current.title === next.title && current.description === next.description && current.sheetBodyClassName === next.sheetBodyClassName && - // Deliberately NOT `hasCustomFooter`: a custom footer is a fresh JSX - // element on every cell render, and this comparison is what forces the - // host to adopt it — `isPending`/spinner freshness depends on the - // footer being re-rendered here (the cell no longer renders it itself). - current.footer === next.footer + current.hasCustomFooter === next.hasCustomFooter ) { return current } return next }) setOpen(true) + return session }, []) const isActiveSheet = useCallback( - (onOpenChange: (open: boolean) => void) => - open && activeOnOpenChangeRef.current === onOpenChange, + (sessionId: number) => open && activeSessionRef.current === sessionId, [open], ) + const registerFooter = useCallback((sessionId: number, footer: ReactNode | null) => { + if (activeSessionRef.current !== sessionId || !openRef.current) return + setFooterContent(footer) + }, []) + + const openRef = useRef(open) + openRef.current = open + const handleOpenChange = useCallback( (nextOpen: boolean) => { if (nextOpen) return + const sessionId = activeSessionRef.current const onOpenChange = activeOnOpenChangeRef.current - if (!onOpenChange) { + if (sessionId === null || !onOpenChange) { setOpen(false) + activeSessionRef.current = null + activeOnOpenChangeRef.current = null setChrome(null) + setFooterContent(null) return } - dismissSheet(onOpenChange) + dismissSheet(sessionId, onOpenChange) }, [dismissSheet], ) @@ -135,8 +158,9 @@ export const CampaignListSheetProvider = ({ children }: { children: ReactNode }) openSheet, dismissSheet, isActiveSheet, + setFooterContent: registerFooter, }), - [portalRevision, openSheet, dismissSheet, isActiveSheet], + [portalRevision, openSheet, dismissSheet, isActiveSheet, registerFooter], ) return ( @@ -161,7 +185,13 @@ export const CampaignListSheetProvider = ({ children }: { children: ReactNode }) )} /> - {chrome.footer ?? Fechar} + {/* C109 — the custom footer renders HERE, inside the Drawer's + React tree, so its `DialogClose` has the root context; the + body keeps the portal (its content never needs Dialog). */} + {footerContent ? ( +
{footerContent}
+ ) : null} + {!chrome.hasCustomFooter ? Fechar : null} ) : null} diff --git a/src/components/campaign/shared/MunicipalityRelationAvatarStack.tsx b/src/components/campaign/shared/MunicipalityRelationAvatarStack.tsx index d2f8c833..3a46f928 100644 --- a/src/components/campaign/shared/MunicipalityRelationAvatarStack.tsx +++ b/src/components/campaign/shared/MunicipalityRelationAvatarStack.tsx @@ -1,6 +1,7 @@ import type { ReactNode } from 'react' import { Avatar, AvatarFallback } from '@/components/ui/Avatar' +import { cn } from '@/lib/utils' import { campaignUserInitials } from '@/utilities/campaignUserProfile' export type MunicipalityRelationEntry = { @@ -16,19 +17,32 @@ export const MunicipalityRelationAvatarStack = ({ entries, emptyState, maxVisible = 3, + wrap = false, }: { entries: MunicipalityRelationEntry[] emptyState: ReactNode maxVisible?: number + /** + * B193 — dense card mode: every entry gets its own avatar (no cap) and the + * row wraps to the available width instead of overlapping in a fixed stack. + */ + wrap?: boolean }) => { if (entries.length === 0) return emptyState + const visible = wrap ? entries : entries.slice(0, maxVisible) + return ( -
-
- {entries.slice(0, maxVisible).map((entry) => ( - - +
+
+ {visible.map((entry) => ( + + {/* The full names ride in the sr-only span below (or the trigger's + aria-label); initials alone would be announced twice. */} + diff --git a/src/components/campaign/shared/MunicipalityRelationEditor.tsx b/src/components/campaign/shared/MunicipalityRelationEditor.tsx index 17f5f874..36b9e783 100644 --- a/src/components/campaign/shared/MunicipalityRelationEditor.tsx +++ b/src/components/campaign/shared/MunicipalityRelationEditor.tsx @@ -29,7 +29,17 @@ export type MunicipalityRelationMutationResult = } | { status: 'error'; message: string } -type MunicipalityRelationEditorProps = { +/** + * B193 — the closed-display override the dense mobile card passes through the + * relation wrappers (Advisors/Lideranças/Dobradinhas) to this editor. + */ +export type MunicipalityRelationTriggerProps = { + trigger?: (entries: MunicipalityRelationEntry[], emptyState: ReactNode) => ReactNode + /** B193 — dense card trigger styling override (no min-height/hover pill). */ + triggerClassName?: string +} + +type MunicipalityRelationEditorProps = MunicipalityRelationTriggerProps & { municipalityName: string currentIDs: number[] knownEntries?: MunicipalityRelationEntry[] @@ -76,6 +86,8 @@ export const MunicipalityRelationEditor = ({ onToggle, onCreate, onCreated, + trigger, + triggerClassName, }: MunicipalityRelationEditorProps) => { const [open, setOpen] = useState(false) const [selectedIDs, setSelectedIDs] = useState(currentIDs) @@ -270,10 +282,15 @@ export const MunicipalityRelationEditor = ({ triggerBusy={isPending} statusMessage={statusMessage} tooltipContent={tooltipContent} + triggerClassName={triggerClassName} contentClassName="w-80 p-0" sheetBodyClassName="px-0 pt-2" trigger={ - + trigger ? ( + trigger(selectedEntries, emptyState) + ) : ( + + ) } >
diff --git a/src/components/campaign/shared/MunicipalityStateDeputyRelationCell.tsx b/src/components/campaign/shared/MunicipalityStateDeputyRelationCell.tsx index cabd2658..a5509603 100644 --- a/src/components/campaign/shared/MunicipalityStateDeputyRelationCell.tsx +++ b/src/components/campaign/shared/MunicipalityStateDeputyRelationCell.tsx @@ -6,13 +6,14 @@ import type { MunicipalityRelationEntry } from '@/components/campaign/shared/Mun import { MunicipalityRelationEditor, type MunicipalityRelationMutationResult, + type MunicipalityRelationTriggerProps, } from '@/components/campaign/shared/MunicipalityRelationEditor' import { stateDeputyDisplayName } from '@/lib/stateDeputyNameParty' import type { CampaignFormActionState } from '@/utilities/campaignFormActionError' import { firstFormActionMessage } from '@/utilities/campaignFormFields' import type { StateDeputyRelationOption } from '@/utilities/campaignRelationOptions' -type MunicipalityStateDeputyRelationCellProps = { +type MunicipalityStateDeputyRelationCellProps = MunicipalityRelationTriggerProps & { municipalityId: number municipalityName: string stateDeputyIDs: number[] @@ -64,6 +65,8 @@ export const MunicipalityStateDeputyRelationCell = ({ commitAction, createAction, editorVariant, + trigger, + triggerClassName, }: MunicipalityStateDeputyRelationCellProps) => { const toggle = async ( stateDeputyID: number, @@ -122,6 +125,8 @@ export const MunicipalityStateDeputyRelationCell = ({ }` } emptyState={—} + trigger={trigger} + triggerClassName={triggerClassName} createLabel={(name) => `Criar dobradinha “${name}”`} createMaxLength={120} sortSelected={false} diff --git a/src/utilities/municipality/municipalityPageData.ts b/src/utilities/municipality/municipalityPageData.ts index ac57d42f..f95e551e 100644 --- a/src/utilities/municipality/municipalityPageData.ts +++ b/src/utilities/municipality/municipalityPageData.ts @@ -58,6 +58,7 @@ import { territorialClassSortWeight, type MunicipalityTerritorialClassification, } from '@/utilities/municipality/municipalityTerritorialClass' +import { loadMunicipalityLastUpdates } from '@/utilities/municipality/municipalityUpdatePageData' import { loadMunicipalityLeadershipSummaries, municipalityListSelect, @@ -603,13 +604,22 @@ export const loadMunicipalityListPageBundle = async ( // B155 — the leaderships of the visible page only (staff surfaces; the // leader view already returned above). One reverse batch: ids per município // for the rows, contact names for the chips. - const leadershipBundle = isStaff - ? await loadMunicipalityLeadershipSummaries( - payload, - user, - pageDocs.map((municipality) => municipality.id), - ) - : null + const [leadershipBundle, lastUpdatesByMunicipalityID] = await Promise.all([ + isStaff + ? loadMunicipalityLeadershipSummaries( + payload, + user, + pageDocs.map((municipality) => municipality.id), + ) + : null, + isStaff + ? loadMunicipalityLastUpdates( + payload, + user, + pageDocs.map((d) => d.id), + ) + : new Map(), + ]) return { municipalities: pageDocs.map((municipality) => { @@ -622,6 +632,7 @@ export const loadMunicipalityListPageBundle = async ( leadershipBundle?.leadershipIDsByMunicipality.get(municipality.id) ?? [], isCity, isCity ? classBySlug?.get(SALVADOR_CITY_SLUG) : undefined, + lastUpdatesByMunicipalityID.get(municipality.id) ?? null, ) }), totalDocs, diff --git a/src/utilities/municipality/municipalityUpdatePageData.ts b/src/utilities/municipality/municipalityUpdatePageData.ts index 82be2b91..252f57e5 100644 --- a/src/utilities/municipality/municipalityUpdatePageData.ts +++ b/src/utilities/municipality/municipalityUpdatePageData.ts @@ -38,6 +38,39 @@ export const parseMunicipalityUpdateFeedParams = ( return { page } } +/** One doc → view-model mapping, shared by the feed and the list batch (B193). */ +const toMunicipalityUpdateViewModel = ( + update: MunicipalityUpdate, + authorNameById: ReadonlyMap, +): MunicipalityUpdateViewModel => { + const polarity = update.polarity as MunicipalityUpdatePolarity | undefined + return { + id: update.id, + authorName: authorNameById.get(relationshipId(update.author) ?? -1) ?? 'Usuário', + createdAt: update.createdAt, + body: update.body ?? null, + polarity: polarity && municipalityUpdatePolarities.includes(polarity) ? polarity : 'neutra', + urgent: Boolean(update.urgent), + activeVolunteers: update.activeVolunteers ?? null, + newSupports: update.newSupports ?? null, + adversarySignal: Boolean(update.adversarySignal), + } +} + +const loadUpdateAuthorNames = async ( + payload: Payload, + updates: MunicipalityUpdate[], +): Promise> => { + const authorIDs = [ + ...new Set( + updates + .map((update) => relationshipId(update.author)) + .filter((id): id is number => id !== null), + ), + ] + return loadCampaignUserNamesByIds(payload, authorIDs) +} + export const loadMunicipalityUpdatesFeed = async ( payload: Payload, user: CampaignUser, @@ -62,31 +95,11 @@ export const loadMunicipalityUpdatesFeed = async ( overrideAccess: false, }) - const authorIDs = [ - ...new Set( - result.docs - .map((update) => relationshipId((update as MunicipalityUpdate).author)) - .filter((id): id is number => id !== null), - ), - ] - const authorNameById = await loadCampaignUserNamesByIds(payload, authorIDs) + const docs = result.docs as MunicipalityUpdate[] + const authorNameById = await loadUpdateAuthorNames(payload, docs) return { - updates: result.docs.map((update) => { - const doc = update as MunicipalityUpdate - const polarity = doc.polarity as MunicipalityUpdatePolarity | undefined - return { - id: doc.id, - authorName: authorNameById.get(relationshipId(doc.author) ?? -1) ?? 'Usuário', - createdAt: doc.createdAt, - body: doc.body ?? null, - polarity: polarity && municipalityUpdatePolarities.includes(polarity) ? polarity : 'neutra', - urgent: Boolean(doc.urgent), - activeVolunteers: doc.activeVolunteers ?? null, - newSupports: doc.newSupports ?? null, - adversarySignal: Boolean(doc.adversarySignal), - } - }), + updates: docs.map((update) => toMunicipalityUpdateViewModel(update, authorNameById)), totalDocs: result.totalDocs, totalPages: result.totalPages, page: result.page ?? state.page, @@ -102,3 +115,52 @@ export const loadMunicipalityUpdatesPreview = async ( 0, 3, ) + +/** + * B193 — the LAST update per listed municipality, for the mobile card's + * expandable footer. One batch over the visible page ids: docs arrive sorted + * by `-createdAt`, so the first row per municipality is the latest. The read + * runs with `overrideAccess: false`, so `canReadMunicipalityUpdate` scopes it + * to the actor's portfolio exactly like the C89 feed loader. + */ +export const loadMunicipalityLastUpdates = async ( + payload: Payload, + user: CampaignUser, + municipalityIDs: readonly number[], +): Promise> => { + const ids = [...new Set(municipalityIDs)] + if (ids.length === 0) return new Map() + + const result = await payload.find({ + collection: 'municipalityUpdate', + where: { municipality: { in: ids } }, + depth: 0, + limit: 0, + pagination: false, + sort: '-createdAt', + select: { + municipality: true, + author: true, + polarity: true, + urgent: true, + activeVolunteers: true, + newSupports: true, + adversarySignal: true, + body: true, + createdAt: true, + }, + user, + overrideAccess: false, + }) + + const authorNameById = await loadUpdateAuthorNames(payload, result.docs as MunicipalityUpdate[]) + + const lastByMunicipality = new Map() + for (const update of result.docs as MunicipalityUpdate[]) { + const municipalityID = relationshipId(update.municipality) + if (municipalityID === null || lastByMunicipality.has(municipalityID)) continue + lastByMunicipality.set(municipalityID, toMunicipalityUpdateViewModel(update, authorNameById)) + } + + return lastByMunicipality +} diff --git a/src/utilities/municipality/municipalityViewModels.ts b/src/utilities/municipality/municipalityViewModels.ts index 7a448259..39a8fd85 100644 --- a/src/utilities/municipality/municipalityViewModels.ts +++ b/src/utilities/municipality/municipalityViewModels.ts @@ -24,6 +24,7 @@ import { type MunicipalityTerritorialClassification, type TerritorialFactor, } from '@/utilities/municipality/municipalityTerritorialClass' +import type { MunicipalityUpdateViewModel } from '@/utilities/municipality/municipalityUpdatePageData' import type { StateDeputySummary } from '@/utilities/stateDeputyData' import type { MunicipalityPledgeAggregate } from '@/utilities/votePledgeViews' import { createEmptyMunicipalityPledgeAggregate } from '@/utilities/votePledgeViews' @@ -81,6 +82,13 @@ export type MunicipalityListViewModel = { * ordering. */ lastSignalAt: string | null + /** + * B193 — the LAST municipality update (body, polarity, author), for the + * mobile card's expandable footer; `null` when nothing was ever recorded + * (the footer becomes a direct register CTA). Staff-only — the leader never + * reaches the list page. + */ + lastUpdate: MunicipalityUpdateViewModel | null expectedVotes: VoteEstimateScenarioViewModel politicalTrendStatus: PoliticalTrendStatus | null politicalTrendNote: string | null @@ -109,6 +117,8 @@ export const toMunicipalityListViewModel = ( isCity = false, /** B178 — override for the city's AGGREGATE class (the artifact lookup by slug would miss). */ territorialClassOverride?: MunicipalityTerritorialClassification, + /** B193 — the last municipality update for the mobile card footer (staff-only). */ + lastUpdate: MunicipalityUpdateViewModel | null = null, ): MunicipalityListViewModel => { const territorialClass = territorialClassOverride ?? computeMunicipalityTerritorialClass(municipality.slug) @@ -136,6 +146,7 @@ export const toMunicipalityListViewModel = ( municipality.lastUpdateAt ?? null, pledges?.lastPledgeAt ?? null, ), + lastUpdate, expectedVotes: toVoteEstimateScenarioViewModel(municipality.expectedVotes), politicalTrendStatus: municipality.politicalTrend?.status ?? null, politicalTrendNote: municipality.politicalTrend?.note ?? null, diff --git a/tests/e2e/campaignMunicipalities.e2e.spec.ts b/tests/e2e/campaignMunicipalities.e2e.spec.ts index c543d99b..da75d312 100644 --- a/tests/e2e/campaignMunicipalities.e2e.spec.ts +++ b/tests/e2e/campaignMunicipalities.e2e.spec.ts @@ -786,15 +786,13 @@ test.describe('Municípios — cards no celular (B42)', () => { // Anywhere outside a control is the município's own link: the click lands on // the stretched `after:inset-0` overlay, which belongs to the card heading. - // The tap goes on a field LABEL inside the metrics grid on purpose — the - // first version of this card positioned each grid cell instead of each - // control, which lifted the labels and their padding above the overlay and - // made half the card a tap that neither edited nor navigated. + // B193 recomposed the card and every label became a trigger, so the tap goes + // on the card's top-left padding corner — the one area no control covers. // `force`: Playwright's hit-target check retries forever here precisely // BECAUSE the stretched overlay intercepts the pointer — which is the - // behavior under test. Force dispatches the tap at the label's point, the - // overlay receives it (as a user's finger would), and the card navigates. - await card.getByText('Tendência', { exact: true }).click({ force: true }) + // behavior under test. Force dispatches the tap at that point, the overlay + // receives it (as a user's finger would), and the card navigates. + await card.click({ position: { x: 6, y: 6 }, force: true }) await expect(page).toHaveURL(`${campaign.baseURL}/campanha/municipios/${municipality.slug}`) await expect(campaignPageChrome(page, municipality.name)).toBeVisible() }) @@ -847,9 +845,12 @@ test.describe('Municípios — cards no celular (B42)', () => { // The toast is intentionally transient; the persisted freshness label is // the stable success contract and must arrive after the server response. - await expect( - card.getByRole('button', { name: `Registrar atualização em ${municipality.name} — hoje` }), - ).toBeVisible() + // B193 — once the município HAS an update, the card footer becomes the + // expandable "Última atualização" row (the register CTA lives inside the + // expansion), so the freshness contract is asserted on the footer itself. + const updateFooter = card.getByRole('button', { name: /^Última atualização/ }) + await expect(updateFooter).toContainText('hoje') + await updateFooter.click() // Reopen: the chrome is rebuilt fresh, and the custom footer comes back. await card.getByRole('button', { name: /^Registrar atualização em/ }).click() @@ -890,6 +891,9 @@ test.describe('Municípios — filtro e cards sem moldura no celular (B184)', () await campaign.login(page, coordinator.email!, password) await page.goto(`${campaign.baseURL}/campanha/municipios`) + // C106 — dynamic pages stream a transient hidden `#S:*` copy of the shell; + // strict-mode locators below would match BOTH copies while it settles. + await page.waitForFunction(() => document.querySelectorAll('div[id^="S:"]').length === 0) // The mobile standard hides the field label; the input keeps its name via // aria-label, and the filter form is a borderless sticky bar. @@ -955,6 +959,184 @@ test.describe('Municípios — filtro e cards sem moldura no celular (B184)', () }) }) +/** + * B193 (2026-08-10): the mobile card is a dense composition — scenario bar + * with the active marker, chips (classe/tendência/nível), three labelled + * avatar groups — where every data point is its own bottom-sheet trigger + * (edit-where-you-see), the priority município gets a right accent border, + * and the "Última atualização" footer expands to reveal the last update card + * (or becomes a direct register CTA when nothing was ever recorded). + */ +test.describe('Municípios — card denso mobile (B193)', () => { + test.use({ viewport: { width: 390, height: 844 } }) + + test('scenario bar opens the estimate sheet and reflects saved values', async ({ + campaign, + page, + }) => { + const { fixtures } = campaign + const coordinator = await fixtures.createCampaignUser('coordinator', { + name: fixtures.value('Coordenadora B193'), + }) + const municipality = await fixtures.claimMunicipality() + + await campaign.login(page, coordinator.email!, coordinator.password) + await page.goto( + `${campaign.baseURL}/campanha/municipios?q=${encodeURIComponent(municipality.name)}`, + ) + // `?q=` matches name PREFIXES ("Coribe" also returns "São Félix do + // Coribe"), so the card is anchored on the municipality's own name link. + const card = page + .locator('[data-view="mobile-cards"] article') + .filter({ has: page.getByRole('link', { name: municipality.name, exact: true }) }) + .first() + await expect(card).toBeVisible() + + await card.getByRole('button', { name: /Editar votos estimados/ }).click() + const drawer = page.getByRole('dialog', { name: 'Editar votos estimados' }) + await expect(drawer).toBeVisible() + + await drawer.getByLabel('Pessimista', { exact: true }).fill('7500') + await drawer.getByLabel('Média', { exact: true }).fill('8000') + await drawer.getByLabel('Otimista', { exact: true }).fill('10000') + // The autosave announces "Salvando…" through the trigger's own live region + // (next to the button, outside the drawer); closing flushes the save. + await drawer.getByRole('button', { name: 'Fechar' }).click() + await expect(drawer).toBeHidden() + + // The strip is the live autosave value: the card shows all three + // scenarios after the save echoed back. + await expect(card.getByText('7.500')).toBeVisible() + await expect(card.getByText('8.000')).toBeVisible() + await expect(card.getByText('10.000')).toBeVisible() + await expect(page).toHaveURL( + `${campaign.baseURL}/campanha/municipios?q=${encodeURIComponent(municipality.name)}`, + ) + }) + + test('the Nível chip (label included) opens the level sheet', async ({ campaign, page }) => { + const { fixtures } = campaign + const coordinator = await fixtures.createCampaignUser('coordinator', { + name: fixtures.value('Coordenadora B193 Nível'), + }) + const municipality = await fixtures.claimMunicipality() + + await campaign.login(page, coordinator.email!, coordinator.password) + await page.goto( + `${campaign.baseURL}/campanha/municipios?q=${encodeURIComponent(municipality.name)}`, + ) + // `?q=` matches name PREFIXES ("Coribe" also returns "São Félix do + // Coribe"), so the card is anchored on the municipality's own name link. + const card = page + .locator('[data-view="mobile-cards"] article') + .filter({ has: page.getByRole('link', { name: municipality.name, exact: true }) }) + .first() + await expect(card).toBeVisible() + + await card.getByText('Nível', { exact: true }).click() + const drawer = page.getByRole('dialog', { name: 'Registrar nível de envolvimento' }) + await expect(drawer).toBeVisible() + await expect(drawer.getByText(municipality.name, { exact: true })).toBeVisible() + await expect(page).toHaveURL( + `${campaign.baseURL}/campanha/municipios?q=${encodeURIComponent(municipality.name)}`, + ) + }) + + test('priority município gets the right accent border (B193)', async ({ campaign, page }) => { + const { fixtures } = campaign + const coordinator = await fixtures.createCampaignUser('coordinator', { + name: fixtures.value('Coordenadora B193 Prio'), + }) + const municipality = await fixtures.claimMunicipality() + await fixtures.payload.update({ + collection: 'municipality', + id: municipality.id, + data: { priority: 'alta' }, + depth: 0, + }) + + await campaign.login(page, coordinator.email!, coordinator.password) + await page.goto( + `${campaign.baseURL}/campanha/municipios?q=${encodeURIComponent(municipality.name)}`, + ) + // `?q=` matches name PREFIXES ("Coribe" also returns "São Félix do + // Coribe"), so the card is anchored on the municipality's own name link. + const card = page + .locator('[data-view="mobile-cards"] article') + .filter({ has: page.getByRole('link', { name: municipality.name, exact: true }) }) + .first() + await expect(card).toBeVisible() + + await expect(card).toHaveCSS('border-right-width', '6px') + // The accent color is a token contract, not a literal: assert the utility + // class that paints it (`border-r-primary`) instead of the resolved color. + await expect(card).toHaveClass(/border-r-primary/) + }) + + test('footer expands to the last update card and offers register; empty footer is a direct CTA', async ({ + campaign, + page, + }) => { + const { fixtures } = campaign + const coordinator = await fixtures.createCampaignUser('coordinator', { + name: fixtures.value('Coordenadora B193 Upd'), + }) + const withUpdate = await fixtures.claimMunicipality() + await fixtures.payload.create({ + collection: 'municipalityUpdate', + data: { + municipality: withUpdate.id, + author: coordinator.id, + polarity: 'boa', + body: `Fato de campo B193 em ${withUpdate.name}`, + }, + depth: 0, + }) + + await campaign.login(page, coordinator.email!, coordinator.password) + await page.goto( + `${campaign.baseURL}/campanha/municipios?q=${encodeURIComponent(withUpdate.name)}`, + ) + // `?q=` matches name prefixes — anchor on the municipality's own name link. + const card = page + .locator('[data-view="mobile-cards"] article') + .filter({ has: page.getByRole('link', { name: withUpdate.name, exact: true }) }) + .first() + await expect(card).toBeVisible() + + // The footer button expands the card bottom and reveals the last update + // card (no surface change) plus the register CTA. + const footer = card.getByRole('button', { name: /Última atualização/ }) + await expect(footer).toHaveAttribute('aria-expanded', 'false') + await footer.click() + await expect(footer).toHaveAttribute('aria-expanded', 'true') + await expect(card.getByText(`Fato de campo B193 em ${withUpdate.name}`)).toBeVisible() + await expect(card.getByText('Boa', { exact: true })).toBeVisible() + + const registerCta = card.getByRole('button', { name: /Registrar atualização em/ }) + await registerCta.click() + const drawer = page.getByRole('dialog', { name: 'Registrar atualização' }) + await expect(drawer).toBeVisible() + await expect(drawer.getByText(withUpdate.name, { exact: true })).toBeVisible() + await drawer.getByRole('button', { name: 'Cancelar' }).click() + await expect(drawer).toBeHidden() + + // A município with no update at all has no expandable footer: the footer + // IS the register CTA, without chevron or expansion. + const empty = await fixtures.claimMunicipality() + await page.goto(`${campaign.baseURL}/campanha/municipios?q=${encodeURIComponent(empty.name)}`) + const emptyCard = page + .locator('[data-view="mobile-cards"] article') + .filter({ has: page.getByRole('link', { name: empty.name, exact: true }) }) + .first() + await expect(emptyCard).toBeVisible() + await expect(emptyCard.getByRole('button', { name: /Última atualização/ })).toHaveCount(0) + const emptyCta = emptyCard.getByRole('button', { name: /Registrar atualização em/ }) + await emptyCta.click() + await expect(page.getByRole('dialog', { name: 'Registrar atualização' })).toBeVisible() + }) +}) + /** * B126: FAB quick-actions overlay — no persistent drawer; opens on demand. */ diff --git a/tests/e2e/campaignUpdatesMobile.e2e.spec.ts b/tests/e2e/campaignUpdatesMobile.e2e.spec.ts index 7de7e788..d0995d68 100644 --- a/tests/e2e/campaignUpdatesMobile.e2e.spec.ts +++ b/tests/e2e/campaignUpdatesMobile.e2e.spec.ts @@ -24,6 +24,10 @@ const seedUpdates = async ( const { fixtures } = campaign const coordinator = await fixtures.createCampaignUser('coordinator') const municipality = await fixtures.claimMunicipality() + // Explicitly increasing timestamps: rows created back-to-back can share the + // same Postgres `now()` (transaction-scoped), which makes the feed's + // `sort: -createdAt` unstable and the "#8 on top" assertion flaky. + const base = Date.now() for (let i = 0; i < 8; i += 1) { await fixtures.payload.create({ collection: 'municipalityUpdate', @@ -32,6 +36,7 @@ const seedUpdates = async ( author: coordinator.id, polarity: i % 2 === 0 ? 'boa' : 'ruim', body: `Fato de campo C106 #${i + 1} em ${municipality.name}`, + createdAt: new Date(base + (i + 1) * 1000).toISOString(), }, depth: 0, }) diff --git a/tests/unit/campaignCellEditOverlay.unit.spec.ts b/tests/unit/campaignCellEditOverlay.unit.spec.ts index 243a881d..183dc676 100644 --- a/tests/unit/campaignCellEditOverlay.unit.spec.ts +++ b/tests/unit/campaignCellEditOverlay.unit.spec.ts @@ -89,7 +89,9 @@ const overlayCases: OverlayCase[] = [ }, { name: 'votos estimados', - triggerLabel: `Editar votos estimados em ${MUNICIPALITY_NAME} — Média: 1.200`, + // B193 — the votes trigger announces the whole interval (the strip shows + // all three scenarios), not just the active one. + triggerLabel: `Editar votos estimados em ${MUNICIPALITY_NAME} — Pessimista: sem estimativa; Média: 1.200; Otimista: sem estimativa`, drawerTitle: 'Editar votos estimados', element: (variant) => createElement(MunicipalityListExpectedVotesControl, { diff --git a/tests/unit/campaignComponents.unit.spec.ts b/tests/unit/campaignComponents.unit.spec.ts index 9294f659..800fed71 100644 --- a/tests/unit/campaignComponents.unit.spec.ts +++ b/tests/unit/campaignComponents.unit.spec.ts @@ -80,6 +80,7 @@ const responsiveMunicipality = stub({ priority: 'normal', lastUpdateAt: null, lastSignalAt: null, + lastUpdate: null, expectedVotes: toVoteEstimateScenarioViewModel(null), politicalTrendStatus: null, politicalTrendNote: null, @@ -221,6 +222,7 @@ describe('campaign visual foundation', () => { priority: 'alta', lastUpdateAt: null, lastSignalAt: null, + lastUpdate: null, expectedVotes: { pessimistic: null, central: 1500, optimistic: null }, politicalTrendStatus: 'favoravel', politicalTrendNote: null, @@ -267,6 +269,7 @@ describe('campaign visual foundation', () => { priority: 'normal', lastUpdateAt: null, lastSignalAt: null, + lastUpdate: null, expectedVotes: toVoteEstimateScenarioViewModel(null), politicalTrendStatus: null, politicalTrendNote: null, @@ -361,6 +364,7 @@ describe('campaign visual foundation', () => { leadershipIDs: [], lastUpdateAt: null, lastSignalAt: null, + lastUpdate: null, expectedVotes: toVoteEstimateScenarioViewModel(null), politicalTrendStatus: null, politicalTrendNote: null, @@ -421,6 +425,7 @@ describe('campaign visual foundation', () => { stateDeputyIDs: [], lastUpdateAt: null, lastSignalAt: null, + lastUpdate: null, expectedVotes: toVoteEstimateScenarioViewModel(null), politicalTrendStatus: null, politicalTrendNote: null, @@ -553,6 +558,21 @@ describe('campaign visual foundation', () => { priority: 'alta', lastUpdateAt: staleSignal, lastSignalAt: staleSignal, + // B193 — the mobile card footer's "Última atualização" age comes + // from the LAST UPDATE (not the pledge-merged frescor): a stale + // update paints the footer cold, and the register CTA lives inside + // the collapsed expansion. + lastUpdate: { + id: 99, + authorName: 'Maria', + createdAt: staleSignal, + body: 'Visita ao comitê da região.', + polarity: 'boa', + urgent: false, + activeVolunteers: null, + newSupports: null, + adversarySignal: false, + }, expectedVotes: toVoteEstimateScenarioViewModel(null), politicalTrendStatus: null, politicalTrendNote: null, @@ -575,9 +595,16 @@ describe('campaign visual foundation', () => { // "Sem assessor" still appears in the Assessores column (the avatar stack's // empty state); the status badge is what escalates to "Sem responsável". expect(html).toContain('Sem responsável') - expect(html).toContain(`há ${staleDays} dias`) + // The mobile footer's age is the last UPDATE's age (B193): the update is + // 40 days old, so the footer reads "há 40 dias" and paints cold — and the + // expansion stays collapsed (the update body is not in the DOM). + expect(html).toContain('Última atualização há 40 dias') + // The cold class is applied ONLY on the mobile footer (the desktop column + // uses `data-signal="cold"` on the readout's own span), so this pins the + // footer's cold paint without relying on the desktop tree. + expect(html).toContain('text-estimate-pending-foreground') + expect(html).not.toContain('Visita ao comitê da região.') expect(html).toContain('data-signal="cold"') - expect(html).toContain('Última atualização') }) it('reads "Sem sinal" when nothing was ever recorded for the município', () => { @@ -600,6 +627,7 @@ describe('campaign visual foundation', () => { priority: 'normal', lastUpdateAt: null, lastSignalAt: null, + lastUpdate: null, expectedVotes: toVoteEstimateScenarioViewModel(null), politicalTrendStatus: null, politicalTrendNote: null, @@ -644,6 +672,7 @@ describe('campaign visual foundation', () => { priority: 'normal', lastUpdateAt: null, lastSignalAt: null, + lastUpdate: null, expectedVotes: toVoteEstimateScenarioViewModel(null), politicalTrendStatus: null, politicalTrendNote: null, @@ -694,6 +723,7 @@ describe('campaign visual foundation', () => { priority: 'alta', lastUpdateAt: null, lastSignalAt: null, + lastUpdate: null, expectedVotes: toVoteEstimateScenarioViewModel(null), politicalTrendStatus: null, politicalTrendNote: null, @@ -892,6 +922,7 @@ describe('campaign visual foundation', () => { priority: 'normal' as const, lastUpdateAt: null, lastSignalAt: null, + lastUpdate: null, expectedVotes: toVoteEstimateScenarioViewModel(null), politicalTrendStatus: null, politicalTrendNote: null, diff --git a/tests/unit/codebaseConventions.unit.spec.ts b/tests/unit/codebaseConventions.unit.spec.ts index 6584da75..ae0ecbe5 100644 --- a/tests/unit/codebaseConventions.unit.spec.ts +++ b/tests/unit/codebaseConventions.unit.spec.ts @@ -552,6 +552,10 @@ describe('advisor scope fragment comes from access/shared.ts', () => { // B155 — the municipality list surface reads `leadership.municipalities` // by reverse batch; same distinct form as `leadershipData.ts` above. 'src/utilities/municipality/municipalityViewModels.ts', + // B193 — the mobile card footer batches the LAST update per listed + // municipality (a domain filter over the visible page ids); the actor + // scope still comes from `overrideAccess: false` + `user`. + 'src/utilities/municipality/municipalityUpdatePageData.ts', 'src/utilities/visit/visitPlannerData.ts', 'src/utilities/votePledgeData.ts', // B178 — the city overview batches activities across the 19 Salvador zone