From 2f2c8563a5f127aac52d32b5960493ffb44365b8 Mon Sep 17 00:00:00 2001
From: Cursor Agent <cursoragent@cursor.com>
Date: Sun, 2 Aug 2026 23:04:36 +0000
Subject: [PATCH 1/8] =?UTF-8?q?B136=20=E2=80=94=20corrigir=20grade=203?=
 =?UTF-8?q?=C3=972=20no=20In=C3=ADcio=20e=20FAB=20overlay?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- layout=grid no botão com w-full/min-w-0 sem box-content no mobile
- li min-w-0, gap-2, data-layout=grid-3 no strip
- e2e com geometria (boundingBox) em viewport 390
- impl plan docs/plans/inicio-acoes-grade-3cols-2rows-impl.md

Co-authored-by: Francisco Solla <fsolla@users.noreply.github.com>
---
 .../inicio-acoes-grade-3cols-2rows-impl.md    | 62 ++++++++++++++++
 .../dashboard/CampaignHomeActionButton.tsx    | 22 ++++--
 .../dashboard/CampaignHomeActionStrip.tsx     | 16 +++--
 tests/e2e/campaignHomeActions.e2e.spec.ts     | 10 +++
 tests/e2e/campaignMunicipalities.e2e.spec.ts  | 35 ++++++++++
 tests/e2e/helpers/actionGridGeometry.ts       | 70 +++++++++++++++++++
 .../campaignHomeActionButton.unit.spec.tsx    | 12 ++++
 .../campaignQuickActionsDrawer.unit.spec.tsx  |  1 +
 8 files changed, 218 insertions(+), 10 deletions(-)
 create mode 100644 docs/plans/inicio-acoes-grade-3cols-2rows-impl.md
 create mode 100644 tests/e2e/helpers/actionGridGeometry.ts

diff --git a/docs/plans/inicio-acoes-grade-3cols-2rows-impl.md b/docs/plans/inicio-acoes-grade-3cols-2rows-impl.md
new file mode 100644
index 00000000..4d0b6e58
--- /dev/null
+++ b/docs/plans/inicio-acoes-grade-3cols-2rows-impl.md
@@ -0,0 +1,62 @@
+# Impl: Início / FAB — grade 3 colunas × 2 linhas (corrigir orientação)
+
+Status: aprovado
+Atualizado em: 2026-08-02
+Issue: #302
+Intenção: docs/plans/inicio-acoes-grade-3cols-2rows.md
+Appetite restante: ~0,5d (causa real + testes de geometria)
+
+## Leitura da intenção
+
+- **Outcome:** Início mobile e overlay FAB mostram 6 ações staff em **3 colunas × 2 linhas** (verificável em viewport ~390), não 2×3; `md+` Início mantém strip.
+- **O que NÃO negociar:** catálogo/lockdown; sem migration; não reabrir snap/peek; liderança sem layout paralelo.
+- **O que reavaliar:** B132 já trocou `grid-cols-2` → `grid-cols-3`, mas testes só assertam `className` — a causa visual provável é **min-width dos itens da grade** (`min-width: auto` + `box-content` + `w-[5.5rem]` residual) comprimindo para 2 colunas percebidas ou clip pelo `overflow-hidden` do shell de retraction.
+
+## Abordagem recomendada
+
+```mermaid
+flowchart LR
+  Strip["CampaignHomeActionStrip"]
+  Strip -->|grid/responsive lt-md| G["grid-cols-3 + li min-w-0"]
+  Btn["CampaignHomeActionButton"]
+  Btn -->|layout=grid| W["w-full min-w-0 border-box"]
+  E2E["e2e viewport 390"]
+  E2E --> Geo["boundingBox: 3 por linha"]
+```
+
+**Opções consideradas:** A) só reafirmar `grid-cols-3` | B) `min-w-0` + `layout=grid` no botão | C) reduzir círculo/gap  
+**Recomendação:** **B** — corrige sizing real sem redesign; gap-2 só se critique pedir.  
+**Rejeitadas:** A (já feito em B132, insuficiente); C (fora do apetite salvo falha em 390).
+
+### Componentes / mudanças
+
+- **`CampaignHomeActionButton`:** `layout: 'grid'` — `w-full min-w-0 max-w-full`, `box-border` (sem `box-content` na grade).
+- **`CampaignHomeActionStrip`:** `li` com `min-w-0`; `layout='grid'` quando `variant` é `grid` ou `responsive`; `gap-2` no mobile grid (mais folga horizontal em 390px); `data-layout="grid-3"` no `<ul>` para e2e.
+- **`CampaignQuickActionsOverlay`:** sem mudança estrutural (já `variant="grid"`).
+- **Testes:** helper de geometria em e2e (`campaignHomeActions.e2e.spec.ts` + overlay em `campaignMunicipalities.e2e.spec.ts`); unit para `layout=grid` e `min-w-0` no `li`.
+- **Migration:** sem migration.
+
+## Fases verificáveis
+
+1. **Strip + button sizing** — `layout=grid`, `min-w-0`, gap ajustado.
+2. **E2e geometria** — 3 botões na primeira linha, 3 na segunda (staff); overlay FAB idem.
+3. **Gates** — `pnpm gate:fast`; `pnpm push`.
+
+## Rabbit holes / Não escopo (engenharia)
+
+- PWA cache bust (fora do repo); se persistir pós-deploy, runbook separado.
+- Refatorar `HomeChromeRetractionShell` globalmente.
+- Grade em `md+`.
+
+## Riscos e mitigação
+
+- **Leader 2 ações:** grade 3-col deixa 1 célula vazia — aceite da intenção.
+- **Desktop overlay dialog:** `layout=grid` mantém `w-full` (sem `md:w-[5.5rem]`).
+
+## Aceite de engenharia
+
+- [ ] Aceite de produto da intenção ainda coberto
+- [ ] Invariantes AGENTS/engineering-standards
+- [ ] E2e viewport 390 com geometria 3×2 (não só className)
+
+Self-score decision-quality: 5/5
diff --git a/src/components/campaign/dashboard/CampaignHomeActionButton.tsx b/src/components/campaign/dashboard/CampaignHomeActionButton.tsx
index ebc00249..92488302 100644
--- a/src/components/campaign/dashboard/CampaignHomeActionButton.tsx
+++ b/src/components/campaign/dashboard/CampaignHomeActionButton.tsx
@@ -25,17 +25,25 @@ export type CampaignHomeActionButtonProps = {
   href?: string
   onClick?: () => void
   disabled?: boolean
-  layout?: 'strip' | 'responsive'
+  layout?: 'strip' | 'responsive' | 'grid'
 }
 
 const actionControlBaseClassName =
-  'group box-content flex min-h-11 flex-col items-center gap-2 rounded-md text-center outline-none focus-visible:ring-2 focus-visible:ring-ring active:opacity-90 disabled:pointer-events-none disabled:opacity-50'
+  'group flex min-h-11 flex-col items-center gap-2 rounded-md text-center outline-none focus-visible:ring-2 focus-visible:ring-ring active:opacity-90 disabled:pointer-events-none disabled:opacity-50'
 
-const actionStripControlClassName = cn(actionControlBaseClassName, 'w-[5.5rem] shrink-0 snap-start')
+const actionStripControlClassName = cn(
+  actionControlBaseClassName,
+  'box-content w-[5.5rem] shrink-0 snap-start',
+)
 
 const actionResponsiveControlClassName = cn(
   actionControlBaseClassName,
-  'w-full md:w-[5.5rem] md:shrink-0 md:snap-start',
+  'w-full min-w-0 max-w-full md:box-content md:w-[5.5rem] md:max-w-none md:shrink-0 md:snap-start',
+)
+
+const actionGridControlClassName = cn(
+  actionControlBaseClassName,
+  'w-full min-w-0 max-w-full',
 )
 
 const circleClassName =
@@ -84,7 +92,11 @@ export const CampaignHomeActionButton = ({
   })
 
   const className = cn(
-    layout === 'responsive' ? actionResponsiveControlClassName : actionStripControlClassName,
+    layout === 'grid'
+      ? actionGridControlClassName
+      : layout === 'responsive'
+        ? actionResponsiveControlClassName
+        : actionStripControlClassName,
     isInert && 'cursor-default',
   )
   const aria = {
diff --git a/src/components/campaign/dashboard/CampaignHomeActionStrip.tsx b/src/components/campaign/dashboard/CampaignHomeActionStrip.tsx
index 6c4e26eb..c809ff95 100644
--- a/src/components/campaign/dashboard/CampaignHomeActionStrip.tsx
+++ b/src/components/campaign/dashboard/CampaignHomeActionStrip.tsx
@@ -133,13 +133,15 @@ export const CampaignHomeActionStrip = ({
         onClickCapture,
       }
 
+  const isGridLikeLayout = isGridLayout || !isStripLayout
+
   const listClassName = cn(
     'm-0 list-none p-0',
     isStripLayout
       ? 'flex min-w-max snap-x snap-proximity gap-0 px-4 pb-1 md:px-0'
       : isGridLayout
-        ? 'grid grid-cols-3 gap-3'
-        : 'grid grid-cols-3 gap-3 md:flex md:min-w-max md:snap-x md:snap-proximity md:gap-0 md:px-0 md:pb-1',
+        ? 'grid grid-cols-3 gap-2'
+        : 'grid grid-cols-3 gap-2 md:flex md:min-w-max md:snap-x md:snap-proximity md:gap-0 md:px-0 md:pb-1',
   )
 
   const scrollerClassName = cn(
@@ -160,12 +162,16 @@ export const CampaignHomeActionStrip = ({
         className={scrollerClassName}
         {...finePointerPanProps}
       >
-        <ul role="list" className={listClassName}>
+        <ul
+          role="list"
+          className={listClassName}
+          data-layout={isGridLikeLayout ? 'grid-3' : undefined}
+        >
           {actions?.map(({ id, ...button }) => (
-            <li key={id ?? button.href ?? button.label} className="m-0 list-none p-0">
+            <li key={id ?? button.href ?? button.label} className="m-0 min-w-0 list-none p-0">
               <CampaignHomeActionButton
                 {...button}
-                layout={isStripLayout ? 'strip' : 'responsive'}
+                layout={isStripLayout ? 'strip' : isGridLayout ? 'grid' : 'responsive'}
               />
             </li>
           ))}
diff --git a/tests/e2e/campaignHomeActions.e2e.spec.ts b/tests/e2e/campaignHomeActions.e2e.spec.ts
index 8ee00fe5..1b05fecb 100644
--- a/tests/e2e/campaignHomeActions.e2e.spec.ts
+++ b/tests/e2e/campaignHomeActions.e2e.spec.ts
@@ -1,6 +1,11 @@
 import type { Page } from '@playwright/test'
 
 import { WIZARD_MUNICIPALITY_STEP_TITLE } from '../../src/lib/campaignWizardCopy.js'
+import {
+  assertThreeColumnActionGrid,
+  collectActionBoundingBoxes,
+  groupActionsByRow,
+} from './helpers/actionGridGeometry.js'
 import { expect, test } from './fixtures/campaignE2EFixtures.js'
 
 const staffActionLabels = [
@@ -127,6 +132,11 @@ test.describe('Início — catálogo de ações (B45)', () => {
     expect(overflowX).not.toBe('auto')
     expect(overflowX).not.toBe('scroll')
 
+    const actionList = actionsRegion.locator('ul[data-layout="grid-3"]')
+    await expect(actionList).toBeVisible()
+    const boxes = await collectActionBoundingBoxes(actionsRegion, staffActionLabels)
+    assertThreeColumnActionGrid(groupActionsByRow(boxes), 2)
+
     await page.getByRole('link', { name: 'Ajustar votos', exact: true }).click()
     await page.waitForURL(/\/campanha\/acoes\/atualizar-votos/)
     await expect(page.getByLabel('Buscar município')).toBeVisible()
diff --git a/tests/e2e/campaignMunicipalities.e2e.spec.ts b/tests/e2e/campaignMunicipalities.e2e.spec.ts
index 3fcc44fe..eebbf960 100644
--- a/tests/e2e/campaignMunicipalities.e2e.spec.ts
+++ b/tests/e2e/campaignMunicipalities.e2e.spec.ts
@@ -3,6 +3,11 @@ import {
   ensureLeasedConsent,
   SUPPORTER_REGISTRATION_CONSENT_LEASE_KEY,
 } from '../helpers/testDatabaseLease.js'
+import {
+  assertThreeColumnActionGrid,
+  collectActionBoundingBoxes,
+  groupActionsByRow,
+} from './helpers/actionGridGeometry.js'
 import {
   campaignPageChrome,
   checkRadixWhenHydrated,
@@ -483,6 +488,36 @@ test.describe('Municípios — FAB ações rápidas mobile (B126)', () => {
 test.describe('Municípios — FAB overlay polish (B126)', () => {
   test.use({ viewport: { width: 390, height: 844 } })
 
+  const staffActionLabels = [
+    'Ajustar votos',
+    'Registrar sinal',
+    'Mudar tendência',
+    'Atualizar liderança',
+    'Registrar pedido',
+    'Ver esquecidos',
+  ] as const
+
+  test('overlay actions use a 3×2 grid at mobile width (B136)', async ({ campaign, page }) => {
+    const { fixtures } = campaign
+    const coordinator = await fixtures.createCampaignUser('coordinator', {
+      name: fixtures.value('Coordenador B136'),
+    })
+
+    await campaign.login(page, coordinator.email!, coordinator.password)
+    await page.goto(`${campaign.baseURL}/campanha/municipios`)
+    await expect(campaignPageChrome(page, 'Municípios')).toBeVisible()
+
+    await page.getByRole('button', { name: 'Ações rápidas' }).click()
+    const overlay = page.locator('#CampaignQuickActionsOverlay')
+    await expect(overlay).toBeVisible()
+
+    const actionsRegion = overlay.getByLabel('Ações rápidas')
+    await expect(actionsRegion.locator('ul[data-layout="grid-3"]')).toBeVisible()
+
+    const boxes = await collectActionBoundingBoxes(actionsRegion, staffActionLabels)
+    assertThreeColumnActionGrid(groupActionsByRow(boxes), 2)
+  })
+
   test('overlay labels readable and search focus hides action strip', async ({
     campaign,
     page,
diff --git a/tests/e2e/helpers/actionGridGeometry.ts b/tests/e2e/helpers/actionGridGeometry.ts
new file mode 100644
index 00000000..fc0adfc8
--- /dev/null
+++ b/tests/e2e/helpers/actionGridGeometry.ts
@@ -0,0 +1,70 @@
+import type { Locator } from '@playwright/test'
+import { expect } from '@playwright/test'
+
+const ROW_Y_TOLERANCE_PX = 12
+
+export type ActionBoundingBox = {
+  label: string
+  x: number
+  y: number
+  width: number
+  height: number
+}
+
+export async function collectActionBoundingBoxes(
+  container: Locator,
+  labels: readonly string[],
+): Promise<ActionBoundingBox[]> {
+  const boxes: ActionBoundingBox[] = []
+
+  for (const label of labels) {
+    const control = container
+      .getByRole('link', { name: label, exact: true })
+      .or(container.getByRole('button', { name: label, exact: true }))
+    const box = await control.boundingBox()
+    if (!box) {
+      throw new Error(`Missing bounding box for action "${label}"`)
+    }
+    boxes.push({ label, ...box })
+  }
+
+  return boxes
+}
+
+export function groupActionsByRow(
+  boxes: readonly ActionBoundingBox[],
+  tolerancePx = ROW_Y_TOLERANCE_PX,
+): ActionBoundingBox[][] {
+  const sorted = [...boxes].sort((left, right) => left.y - right.y || left.x - right.x)
+  const rows: ActionBoundingBox[][] = []
+
+  for (const box of sorted) {
+    const row = rows.find((entry) => Math.abs(entry[0]!.y - box.y) <= tolerancePx)
+    if (row) {
+      row.push(box)
+    } else {
+      rows.push([box])
+    }
+  }
+
+  for (const row of rows) {
+    row.sort((left, right) => left.x - right.x)
+  }
+
+  return rows
+}
+
+/** Asserts a 3-column grid with `expectedRowCount` rows (staff = 2 rows of 3). */
+export function assertThreeColumnActionGrid(
+  rows: readonly ActionBoundingBox[][],
+  expectedRowCount: number,
+) {
+  expect(rows).toHaveLength(expectedRowCount)
+
+  for (const row of rows) {
+    expect(row).toHaveLength(3)
+    const [first, second, third] = row
+    expect(second.x).toBeGreaterThan(first.x + first.width * 0.4)
+    expect(third.x).toBeGreaterThan(second.x + second.width * 0.4)
+  }
+}
diff --git a/tests/unit/campaignHomeActionButton.unit.spec.tsx b/tests/unit/campaignHomeActionButton.unit.spec.tsx
index 9adec2a1..bb7abd14 100644
--- a/tests/unit/campaignHomeActionButton.unit.spec.tsx
+++ b/tests/unit/campaignHomeActionButton.unit.spec.tsx
@@ -98,6 +98,15 @@ describe('CampaignHomeActionButton', () => {
     expect(control.className).toContain('w-[5.5rem]')
   })
 
+  it('uses full-width grid sizing when layout is grid (B136)', () => {
+    renderActionButton({ label: 'Registrar', icon: BarChart3, layout: 'grid' })
+    const control = screen.getByRole('button', { name: 'Registrar' })
+    expect(control.className).toContain('w-full')
+    expect(control.className).toContain('min-w-0')
+    expect(control.className).not.toContain('box-content')
+    expect(control.className).not.toContain('w-[5.5rem]')
+  })
+
   it('opens the description drawer on long-press when the pointer is coarse', () => {
     resetCampaignCoarsePointerForTests()
     matchMediaMock.mockImplementation((query: string) => ({
@@ -209,7 +218,9 @@ describe('CampaignHomeActionStrip', () => {
     const list = region?.querySelector('ul[role="list"]')
     expect(list).toBeTruthy()
     expect(list?.className).toMatch(/grid-cols-3/)
+    expect(list?.getAttribute('data-layout')).toBe('grid-3')
     expect(list?.className).toMatch(/md:flex/)
+    expect(list?.querySelector('li')?.className).toContain('min-w-0')
     const regionClasses = region?.className.split(/\s+/) ?? []
     expect(regionClasses).not.toContain('overflow-x-auto')
     expect(regionClasses.some((token) => token.includes('md:overflow-x-auto'))).toBe(true)
@@ -229,6 +240,7 @@ describe('CampaignHomeActionStrip', () => {
     const region = container.querySelector('[aria-label="Ações rápidas"]')
     const list = region?.querySelector('ul[role="list"]')
     expect(list?.className).toMatch(/grid-cols-3/)
+    expect(list?.getAttribute('data-layout')).toBe('grid-3')
     expect(list?.className).not.toMatch(/md:flex/)
     expect(region?.className).not.toMatch(/overflow-x-auto/)
   })
diff --git a/tests/unit/campaignQuickActionsDrawer.unit.spec.tsx b/tests/unit/campaignQuickActionsDrawer.unit.spec.tsx
index 160bf305..4e31a38f 100644
--- a/tests/unit/campaignQuickActionsDrawer.unit.spec.tsx
+++ b/tests/unit/campaignQuickActionsDrawer.unit.spec.tsx
@@ -171,6 +171,7 @@ describe('CampaignQuickActionsOverlay (B126)', () => {
     const chrome = document.querySelector('[data-slot="quick-actions-chrome"]')
     const list = chrome?.querySelector('ul[role="list"]')
     expect(list?.className).toContain('grid-cols-3')
+    expect(list?.getAttribute('data-layout')).toBe('grid-3')
     expect(list?.className).not.toMatch(/flex/)
     expect(chrome?.querySelector('[aria-label="Ações rápidas"]')).not.toBeNull()
   })

From 479e08dbd0b79398807fd9f00eb750b1f483421f Mon Sep 17 00:00:00 2001
From: Cursor Agent <cursoragent@cursor.com>
Date: Sun, 2 Aug 2026 23:05:48 +0000
Subject: [PATCH 2/8] refactor(B136): inline grid layout data-attribute guard

Co-authored-by: Francisco Solla <fsolla@users.noreply.github.com>
---
 src/components/campaign/dashboard/CampaignHomeActionStrip.tsx | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/src/components/campaign/dashboard/CampaignHomeActionStrip.tsx b/src/components/campaign/dashboard/CampaignHomeActionStrip.tsx
index c809ff95..71a13f87 100644
--- a/src/components/campaign/dashboard/CampaignHomeActionStrip.tsx
+++ b/src/components/campaign/dashboard/CampaignHomeActionStrip.tsx
@@ -133,8 +133,6 @@ export const CampaignHomeActionStrip = ({
         onClickCapture,
       }
 
-  const isGridLikeLayout = isGridLayout || !isStripLayout
-
   const listClassName = cn(
     'm-0 list-none p-0',
     isStripLayout
@@ -165,7 +163,7 @@ export const CampaignHomeActionStrip = ({
         <ul
           role="list"
           className={listClassName}
-          data-layout={isGridLikeLayout ? 'grid-3' : undefined}
+          data-layout={!isStripLayout ? 'grid-3' : undefined}
         >
           {actions?.map(({ id, ...button }) => (
             <li key={id ?? button.href ?? button.label} className="m-0 min-w-0 list-none p-0">

From 993b9af69b13f5637c5ecd05f4ae2759db09e938 Mon Sep 17 00:00:00 2001
From: Cursor Agent <cursoragent@cursor.com>
Date: Sun, 2 Aug 2026 23:06:28 +0000
Subject: [PATCH 3/8] style: format B136 files with Prettier

---
 .../campaign/dashboard/CampaignHomeActionButton.tsx    |  5 +----
 tests/e2e/campaignHomeActions.e2e.spec.ts              |  2 +-
 tests/e2e/campaignMunicipalities.e2e.spec.ts           | 10 +++++-----
 3 files changed, 7 insertions(+), 10 deletions(-)

diff --git a/src/components/campaign/dashboard/CampaignHomeActionButton.tsx b/src/components/campaign/dashboard/CampaignHomeActionButton.tsx
index 92488302..8e928751 100644
--- a/src/components/campaign/dashboard/CampaignHomeActionButton.tsx
+++ b/src/components/campaign/dashboard/CampaignHomeActionButton.tsx
@@ -41,10 +41,7 @@ const actionResponsiveControlClassName = cn(
   'w-full min-w-0 max-w-full md:box-content md:w-[5.5rem] md:max-w-none md:shrink-0 md:snap-start',
 )
 
-const actionGridControlClassName = cn(
-  actionControlBaseClassName,
-  'w-full min-w-0 max-w-full',
-)
+const actionGridControlClassName = cn(actionControlBaseClassName, 'w-full min-w-0 max-w-full')
 
 const circleClassName =
   'flex size-14 items-center justify-center rounded-full bg-muted text-foreground transition-[transform,colors] duration-150 ease-out motion-reduce:transition-none pointer-fine:group-hover:scale-[1.05] pointer-coarse:group-data-[pressing=true]:scale-[1.05] motion-reduce:scale-100 group-hover:bg-muted/80 group-focus-visible:bg-muted/80'
diff --git a/tests/e2e/campaignHomeActions.e2e.spec.ts b/tests/e2e/campaignHomeActions.e2e.spec.ts
index 1b05fecb..0ff7000c 100644
--- a/tests/e2e/campaignHomeActions.e2e.spec.ts
+++ b/tests/e2e/campaignHomeActions.e2e.spec.ts
@@ -1,12 +1,12 @@
 import type { Page } from '@playwright/test'
 
 import { WIZARD_MUNICIPALITY_STEP_TITLE } from '../../src/lib/campaignWizardCopy.js'
+import { expect, test } from './fixtures/campaignE2EFixtures.js'
 import {
   assertThreeColumnActionGrid,
   collectActionBoundingBoxes,
   groupActionsByRow,
 } from './helpers/actionGridGeometry.js'
-import { expect, test } from './fixtures/campaignE2EFixtures.js'
 
 const staffActionLabels = [
   'Ajustar votos',
diff --git a/tests/e2e/campaignMunicipalities.e2e.spec.ts b/tests/e2e/campaignMunicipalities.e2e.spec.ts
index eebbf960..f7e3d7ad 100644
--- a/tests/e2e/campaignMunicipalities.e2e.spec.ts
+++ b/tests/e2e/campaignMunicipalities.e2e.spec.ts
@@ -3,11 +3,6 @@ import {
   ensureLeasedConsent,
   SUPPORTER_REGISTRATION_CONSENT_LEASE_KEY,
 } from '../helpers/testDatabaseLease.js'
-import {
-  assertThreeColumnActionGrid,
-  collectActionBoundingBoxes,
-  groupActionsByRow,
-} from './helpers/actionGridGeometry.js'
 import {
   campaignPageChrome,
   checkRadixWhenHydrated,
@@ -15,6 +10,11 @@ import {
   expectPostResponse,
   test,
 } from './fixtures/campaignE2EFixtures.js'
+import {
+  assertThreeColumnActionGrid,
+  collectActionBoundingBoxes,
+  groupActionsByRow,
+} from './helpers/actionGridGeometry.js'
 
 /**
  * Core municipality-model journeys per role: coordinator strategy editing, advisor

From 3ac5d303e806efbb041c28bc929f38314ee355dd Mon Sep 17 00:00:00 2001
From: Cursor Agent <cursoragent@cursor.com>
Date: Sun, 2 Aug 2026 23:14:53 +0000
Subject: [PATCH 4/8] =?UTF-8?q?test(B136):=20assert=203=C3=972=20grid=20by?=
 =?UTF-8?q?=20column=20clusters=20(tolerant=20row=20heights)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 tests/e2e/campaignMunicipalities.e2e.spec.ts |  7 ++++
 tests/e2e/helpers/actionGridGeometry.ts      | 39 +++++++++++++++++++-
 2 files changed, 45 insertions(+), 1 deletion(-)

diff --git a/tests/e2e/campaignMunicipalities.e2e.spec.ts b/tests/e2e/campaignMunicipalities.e2e.spec.ts
index f7e3d7ad..f254582f 100644
--- a/tests/e2e/campaignMunicipalities.e2e.spec.ts
+++ b/tests/e2e/campaignMunicipalities.e2e.spec.ts
@@ -513,6 +513,13 @@ test.describe('Municípios — FAB overlay polish (B126)', () => {
 
     const actionsRegion = overlay.getByLabel('Ações rápidas')
     await expect(actionsRegion.locator('ul[data-layout="grid-3"]')).toBeVisible()
+    for (const label of staffActionLabels) {
+      await expect(
+        actionsRegion
+          .getByRole('link', { name: label, exact: true })
+          .or(actionsRegion.getByRole('button', { name: label, exact: true })),
+      ).toBeVisible()
+    }
 
     const boxes = await collectActionBoundingBoxes(actionsRegion, staffActionLabels)
     assertThreeColumnActionGrid(groupActionsByRow(boxes), 2)
diff --git a/tests/e2e/helpers/actionGridGeometry.ts b/tests/e2e/helpers/actionGridGeometry.ts
index fc0adfc8..673c5a02 100644
--- a/tests/e2e/helpers/actionGridGeometry.ts
+++ b/tests/e2e/helpers/actionGridGeometry.ts
@@ -2,6 +2,7 @@ import type { Locator } from '@playwright/test'
 import { expect } from '@playwright/test'
 
 const ROW_Y_TOLERANCE_PX = 12
+const COLUMN_X_TOLERANCE_PX = 24
 
 export type ActionBoundingBox = {
   label: string
@@ -54,11 +55,47 @@ export function groupActionsByRow(
   return rows
 }
 
+function clusterActionsByColumn(
+  boxes: readonly ActionBoundingBox[],
+  tolerancePx = COLUMN_X_TOLERANCE_PX,
+): ActionBoundingBox[][] {
+  const sorted = [...boxes].sort((left, right) => left.x - right.x)
+  const columns: ActionBoundingBox[][] = []
+
+  for (const box of sorted) {
+    const column = columns.find((entry) => Math.abs(entry[0]!.x - box.x) <= tolerancePx)
+    if (column) {
+      column.push(box)
+    } else {
+      columns.push([box])
+    }
+  }
+
+  for (const column of columns) {
+    column.sort((left, right) => left.y - right.y)
+  }
+
+  return columns.sort((left, right) => left[0]!.x - right[0]!.x)
+}
+
 /** Asserts a 3-column grid with `expectedRowCount` rows (staff = 2 rows of 3). */
 export function assertThreeColumnActionGrid(
-  rows: readonly ActionBoundingBox[][],
+  boxes: readonly ActionBoundingBox[],
   expectedRowCount: number,
 ) {
+  expect(boxes).toHaveLength(expectedRowCount * 3)
+
+  const columns = clusterActionsByColumn(boxes)
+  expect(columns).toHaveLength(3)
+  for (const column of columns) {
+    expect(column).toHaveLength(expectedRowCount)
+  }
+
+  const medianHeight = [...boxes].map((box) => box.height).sort((left, right) => left - right)[
+    Math.floor(boxes.length / 2)
+  ]!
+  const rowTolerance = Math.max(ROW_Y_TOLERANCE_PX, medianHeight * 0.35)
+  const rows = groupActionsByRow(boxes, rowTolerance)
   expect(rows).toHaveLength(expectedRowCount)
 
   for (const row of rows) {

From 4cbb65963ba43e132f270e765adad5c0407d486f Mon Sep 17 00:00:00 2001
From: Cursor Agent <cursoragent@cursor.com>
Date: Sun, 2 Aug 2026 23:15:27 +0000
Subject: [PATCH 5/8] fix(B136): pass raw boxes to grid geometry assert

---
 tests/e2e/campaignHomeActions.e2e.spec.ts    | 2 +-
 tests/e2e/campaignMunicipalities.e2e.spec.ts | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/tests/e2e/campaignHomeActions.e2e.spec.ts b/tests/e2e/campaignHomeActions.e2e.spec.ts
index 0ff7000c..c26c9ef1 100644
--- a/tests/e2e/campaignHomeActions.e2e.spec.ts
+++ b/tests/e2e/campaignHomeActions.e2e.spec.ts
@@ -135,7 +135,7 @@ test.describe('Início — catálogo de ações (B45)', () => {
     const actionList = actionsRegion.locator('ul[data-layout="grid-3"]')
     await expect(actionList).toBeVisible()
     const boxes = await collectActionBoundingBoxes(actionsRegion, staffActionLabels)
-    assertThreeColumnActionGrid(groupActionsByRow(boxes), 2)
+    assertThreeColumnActionGrid(boxes, 2)
 
     await page.getByRole('link', { name: 'Ajustar votos', exact: true }).click()
     await page.waitForURL(/\/campanha\/acoes\/atualizar-votos/)
diff --git a/tests/e2e/campaignMunicipalities.e2e.spec.ts b/tests/e2e/campaignMunicipalities.e2e.spec.ts
index f254582f..2d2a43a3 100644
--- a/tests/e2e/campaignMunicipalities.e2e.spec.ts
+++ b/tests/e2e/campaignMunicipalities.e2e.spec.ts
@@ -522,7 +522,7 @@ test.describe('Municípios — FAB overlay polish (B126)', () => {
     }
 
     const boxes = await collectActionBoundingBoxes(actionsRegion, staffActionLabels)
-    assertThreeColumnActionGrid(groupActionsByRow(boxes), 2)
+    assertThreeColumnActionGrid(boxes, 2)
   })
 
   test('overlay labels readable and search focus hides action strip', async ({

From 18c13178846d59b83679b1df1f5ed97b472f0c2c Mon Sep 17 00:00:00 2001
From: Cursor Agent <cursoragent@cursor.com>
Date: Sun, 2 Aug 2026 23:16:11 +0000
Subject: [PATCH 6/8] chore(B136): keep groupActionsByRow internal; drop unused
 import

---
 tests/e2e/campaignHomeActions.e2e.spec.ts    | 1 -
 tests/e2e/campaignMunicipalities.e2e.spec.ts | 1 -
 tests/e2e/helpers/actionGridGeometry.ts      | 2 +-
 3 files changed, 1 insertion(+), 3 deletions(-)

diff --git a/tests/e2e/campaignHomeActions.e2e.spec.ts b/tests/e2e/campaignHomeActions.e2e.spec.ts
index c26c9ef1..c2605f80 100644
--- a/tests/e2e/campaignHomeActions.e2e.spec.ts
+++ b/tests/e2e/campaignHomeActions.e2e.spec.ts
@@ -5,7 +5,6 @@ import { expect, test } from './fixtures/campaignE2EFixtures.js'
 import {
   assertThreeColumnActionGrid,
   collectActionBoundingBoxes,
-  groupActionsByRow,
 } from './helpers/actionGridGeometry.js'
 
 const staffActionLabels = [
diff --git a/tests/e2e/campaignMunicipalities.e2e.spec.ts b/tests/e2e/campaignMunicipalities.e2e.spec.ts
index 2d2a43a3..f4402c0f 100644
--- a/tests/e2e/campaignMunicipalities.e2e.spec.ts
+++ b/tests/e2e/campaignMunicipalities.e2e.spec.ts
@@ -13,7 +13,6 @@ import {
 import {
   assertThreeColumnActionGrid,
   collectActionBoundingBoxes,
-  groupActionsByRow,
 } from './helpers/actionGridGeometry.js'
 
 /**
diff --git a/tests/e2e/helpers/actionGridGeometry.ts b/tests/e2e/helpers/actionGridGeometry.ts
index 673c5a02..859c0b2e 100644
--- a/tests/e2e/helpers/actionGridGeometry.ts
+++ b/tests/e2e/helpers/actionGridGeometry.ts
@@ -32,7 +32,7 @@ export async function collectActionBoundingBoxes(
   return boxes
 }
 
-export function groupActionsByRow(
+function groupActionsByRow(
   boxes: readonly ActionBoundingBox[],
   tolerancePx = ROW_Y_TOLERANCE_PX,
 ): ActionBoundingBox[][] {

From 764343da8ebf291610f2471496f19196c8034690 Mon Sep 17 00:00:00 2001
From: Cursor Agent <cursoragent@cursor.com>
Date: Sun, 2 Aug 2026 23:28:06 +0000
Subject: [PATCH 7/8] test(B128): cover search-only omnibox exports for knip
 graph

---
 tests/unit/listOmniboxB128.unit.spec.ts | 21 +++++++++++++++++++++
 1 file changed, 21 insertions(+)

diff --git a/tests/unit/listOmniboxB128.unit.spec.ts b/tests/unit/listOmniboxB128.unit.spec.ts
index 836311c5..0d49037c 100644
--- a/tests/unit/listOmniboxB128.unit.spec.ts
+++ b/tests/unit/listOmniboxB128.unit.spec.ts
@@ -3,6 +3,9 @@ import { describe, expect, it } from 'vitest'
 import {
   applySearchOnlyOmniboxSuggestion,
   buildSearchOnlyOmniboxChips,
+  buildSearchOnlyOmniboxSuggestions,
+  clearSearchOnlyOmnibox,
+  removeSearchOnlyOmniboxChip,
 } from '@/lib/searchOnlyListOmnibox'
 import {
   applyActivityOmniboxSuggestion,
@@ -266,6 +269,7 @@ describe('list omnibox adapters (B128)', () => {
     expect(applied).toEqual({ kind: 'url', state: { page: 1, q: 'ana' } })
   })
 
+<<<<<<< HEAD
   it('advisor toggles municipality portfolio filter and suggests by name', () => {
     const base = parseAdvisorListParams({})
     const applied = applyAdvisorOmniboxSuggestion({
@@ -288,5 +292,22 @@ describe('list omnibox adapters (B128)', () => {
     })
     const suggestions = filterAdvisorOmniboxSuggestions(seeds, 'feira')
     expect(suggestions.some((entry) => entry.group === 'Município (carteira)')).toBe(true)
+=======
+  it('search-only omnibox clears and removes chips', () => {
+    expect(buildSearchOnlyOmniboxSuggestions('')).toEqual([])
+
+    const removed = removeSearchOnlyOmniboxChip({
+      state: { q: 'x', page: 2 },
+      chipId: 'q',
+      withPageReset: (next) => ({ ...next, page: 1 }),
+    })
+    expect(removed).toEqual({ kind: 'url', state: { page: 1, q: undefined } })
+
+    const cleared = clearSearchOnlyOmnibox({
+      state: { q: 'x' },
+      cleared: {},
+    })
+    expect(cleared).toEqual({ kind: 'clear', state: {} })
+>>>>>>> c2a8e54b (test(B128): cover search-only omnibox exports for knip graph)
   })
 })

From a2e952b3a2f100384e09fc82cc842d5d7fac812f Mon Sep 17 00:00:00 2001
From: Cursor Agent <cursoragent@cursor.com>
Date: Sun, 2 Aug 2026 23:53:33 +0000
Subject: [PATCH 8/8] fix(B136): resolve listOmniboxB128 merge conflict markers
 for CI

---
 tests/unit/listOmniboxB128.unit.spec.ts | 35 ++++++++++++-------------
 1 file changed, 17 insertions(+), 18 deletions(-)

diff --git a/tests/unit/listOmniboxB128.unit.spec.ts b/tests/unit/listOmniboxB128.unit.spec.ts
index 0d49037c..ea40fa00 100644
--- a/tests/unit/listOmniboxB128.unit.spec.ts
+++ b/tests/unit/listOmniboxB128.unit.spec.ts
@@ -269,7 +269,23 @@ describe('list omnibox adapters (B128)', () => {
     expect(applied).toEqual({ kind: 'url', state: { page: 1, q: 'ana' } })
   })
 
-<<<<<<< HEAD
+  it('search-only omnibox clears and removes chips', () => {
+    expect(buildSearchOnlyOmniboxSuggestions('')).toEqual([])
+
+    const removed = removeSearchOnlyOmniboxChip({
+      state: { q: 'x', page: 2 },
+      chipId: 'q',
+      withPageReset: (next) => ({ ...next, page: 1 }),
+    })
+    expect(removed).toEqual({ kind: 'url', state: { page: 1, q: undefined } })
+
+    const cleared = clearSearchOnlyOmnibox({
+      state: { q: 'x' },
+      cleared: {},
+    })
+    expect(cleared).toEqual({ kind: 'clear', state: {} })
+  })
+
   it('advisor toggles municipality portfolio filter and suggests by name', () => {
     const base = parseAdvisorListParams({})
     const applied = applyAdvisorOmniboxSuggestion({
@@ -292,22 +308,5 @@ describe('list omnibox adapters (B128)', () => {
     })
     const suggestions = filterAdvisorOmniboxSuggestions(seeds, 'feira')
     expect(suggestions.some((entry) => entry.group === 'Município (carteira)')).toBe(true)
-=======
-  it('search-only omnibox clears and removes chips', () => {
-    expect(buildSearchOnlyOmniboxSuggestions('')).toEqual([])
-
-    const removed = removeSearchOnlyOmniboxChip({
-      state: { q: 'x', page: 2 },
-      chipId: 'q',
-      withPageReset: (next) => ({ ...next, page: 1 }),
-    })
-    expect(removed).toEqual({ kind: 'url', state: { page: 1, q: undefined } })
-
-    const cleared = clearSearchOnlyOmnibox({
-      state: { q: 'x' },
-      cleared: {},
-    })
-    expect(cleared).toEqual({ kind: 'clear', state: {} })
->>>>>>> c2a8e54b (test(B128): cover search-only omnibox exports for knip graph)
   })
 })