From 0f6f3269202ce89660c304b0c6df8f4b08b5d663 Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Wed, 2 Sep 2026 13:09:22 +0000
Subject: [PATCH 1/2] =?UTF-8?q?fix:=20DeleteButton=20do=20DS=20ganha=20ari?=
 =?UTF-8?q?a-label=20no=20gatilho=20de=20exclus=C3=A3o=20(closes=20#126)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

O trigger `<Button>` do DeleteButton (components/ui/delete-button.tsx) é
icon-only e não tinha aria-label, aria-labelledby, title ou sr-only. O
Trash2 do lucide-react@1.8.0 marca o svg como aria-hidden quando não recebe
filho nem prop de a11y, então o role=button ficava sem nenhuma fonte de
nome (WCAG 4.1.2). Afeta 6 pontos de render em /categorias, /contas,
/metas e /investimentos — o botão destrutivo do DS.

Adiciona __tests__/unit/a11y-nomes-acessiveis.test.ts, seguindo o padrão
de gate por texto-fonte de row-actions.test.ts (#54) e
a11y-estado-selecao.test.ts (#107): ancora a asserção no bloco do <Button>
que é o gatilho (contém o onClick de abrir o dialog e o ícone Trash2), não
no arquivo inteiro.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_014vttNSZKJfEaaPukZWKhvB
---
 __tests__/unit/a11y-nomes-acessiveis.test.ts | 31 ++++++++++++++++++++
 components/ui/delete-button.tsx              |  1 +
 2 files changed, 32 insertions(+)
 create mode 100644 __tests__/unit/a11y-nomes-acessiveis.test.ts

diff --git a/__tests__/unit/a11y-nomes-acessiveis.test.ts b/__tests__/unit/a11y-nomes-acessiveis.test.ts
new file mode 100644
index 0000000..e65e6fa
--- /dev/null
+++ b/__tests__/unit/a11y-nomes-acessiveis.test.ts
@@ -0,0 +1,31 @@
+import { describe, it, expect } from 'vitest'
+import { readFileSync } from 'node:fs'
+import { join } from 'node:path'
+
+// ─── Botões só de ícone sem nome acessível (#106) ──────────────────────────
+//
+// Não há infra de render de componente no projeto (sem jsdom/testing-library).
+// O gate possível é sobre o texto-fonte, padrão já usado em row-actions.test.ts
+// (#54) e a11y-estado-selecao.test.ts (#107).
+//
+// A asserção precisa ancorar no <Button> que envolve o ícone, não no arquivo
+// inteiro — senão um aria-label em qualquer outro botão do mesmo arquivo
+// deixaria o teste verde com o gatilho de exclusão ainda sem nome.
+
+describe('DeleteButton — gatilho de exclusão com nome acessível (#126)', () => {
+  const source = readFileSync(join(process.cwd(), 'components/ui/delete-button.tsx'), 'utf-8')
+
+  // Bloco do <Button> que é o gatilho (contém o onClick={() => setOpen(true)}
+  // e o ícone Trash2) — não o botão "Excluir" do Dialog/Drawer, nem "Cancelar".
+  const trigger = source.match(
+    /<Button\b[^]*?onClick=\{\(\) => setOpen\(true\)\}[^]*?<Trash2[^]*?<\/Button>/
+  )?.[0]
+
+  it('encontra o bloco do gatilho', () => {
+    expect(trigger).toBeDefined()
+  })
+
+  it('expõe aria-label no <Button> do gatilho, não no ícone', () => {
+    expect(trigger).toMatch(/^\s*aria-label="[^"]+"\s*$/m)
+  })
+})
diff --git a/components/ui/delete-button.tsx b/components/ui/delete-button.tsx
index 6296fcf..b8c3b47 100644
--- a/components/ui/delete-button.tsx
+++ b/components/ui/delete-button.tsx
@@ -51,6 +51,7 @@ export function DeleteButton({
       variant="ghost"
       className="h-7 w-7 text-text-secondary hover:text-negative"
       onClick={() => setOpen(true)}
+      aria-label="Excluir"
     >
       <Trash2 className="h-3.5 w-3.5" />
     </Button>

From 9f9f16d185d379b93b89837a2ec7003f257f19f1 Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Wed, 2 Sep 2026 19:21:37 +0000
Subject: [PATCH 2/2] =?UTF-8?q?fix:=20gate=20ancora=20no=20Button=20mais?=
 =?UTF-8?q?=20pr=C3=B3ximo=20do=20=C3=ADcone,=20n=C3=A3o=20no=20primeiro?=
 =?UTF-8?q?=20do=20arquivo?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Nit de revisão em #139: o regex anterior (<Button\b[^]*?onClick=...[^]*?<Trash2...)
podia atravessar um <Button> anterior no mesmo arquivo, caso ele existisse antes
do gatilho — a proteção dependia da ordem do fonte (o gatilho ser o primeiro
<Button>), não do regex em si. Sem falso-positivo hoje, mas frágil.

Troca por findClosestButtonWithIcon(source, iconTag), com lookahead negativa
que impede o match de atravessar outro "<Button" antes de alcançar o ícone-alvo
— mesmo helper que o PR #140 (issue #127) introduz ao criar este mesmo arquivo.
Validado que a lookahead rejeita o cenário do reviewer (Button rotulado antes
do gatilho, gatilho ainda sem nome) e que o comportamento correto (rejeita
pré-fix, aceita pós-fix) se mantém.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_014vttNSZKJfEaaPukZWKhvB
---
 __tests__/unit/a11y-nomes-acessiveis.test.ts | 23 +++++++++++++-------
 1 file changed, 15 insertions(+), 8 deletions(-)

diff --git a/__tests__/unit/a11y-nomes-acessiveis.test.ts b/__tests__/unit/a11y-nomes-acessiveis.test.ts
index e65e6fa..1f43808 100644
--- a/__tests__/unit/a11y-nomes-acessiveis.test.ts
+++ b/__tests__/unit/a11y-nomes-acessiveis.test.ts
@@ -10,18 +10,25 @@ import { join } from 'node:path'
 //
 // A asserção precisa ancorar no <Button> que envolve o ícone, não no arquivo
 // inteiro — senão um aria-label em qualquer outro botão do mesmo arquivo
-// deixaria o teste verde com o gatilho de exclusão ainda sem nome.
+// deixaria o teste verde com o gatilho ainda sem nome.
+
+function findClosestButtonWithIcon(source: string, iconTag: string): string | undefined {
+  // Ancora no <Button> MAIS PRÓXIMO do ícone: a lookahead negativa impede o
+  // match de atravessar outro "<Button" antes de alcançar o ícone-alvo — sem
+  // isso um regex guloso/lazy simples poderia casar a partir de um <Button>
+  // anterior no mesmo arquivo, desde que os marcadores (onClick, ícone,
+  // </Button>) só existam depois dele.
+  const pattern = new RegExp(
+    `<Button\\b(?:(?!<Button\\b)[\\s\\S])*?<${iconTag}\\b[\\s\\S]*?<\\/Button>`
+  )
+  return source.match(pattern)?.[0]
+}
 
 describe('DeleteButton — gatilho de exclusão com nome acessível (#126)', () => {
   const source = readFileSync(join(process.cwd(), 'components/ui/delete-button.tsx'), 'utf-8')
+  const trigger = findClosestButtonWithIcon(source, 'Trash2')
 
-  // Bloco do <Button> que é o gatilho (contém o onClick={() => setOpen(true)}
-  // e o ícone Trash2) — não o botão "Excluir" do Dialog/Drawer, nem "Cancelar".
-  const trigger = source.match(
-    /<Button\b[^]*?onClick=\{\(\) => setOpen\(true\)\}[^]*?<Trash2[^]*?<\/Button>/
-  )?.[0]
-
-  it('encontra o bloco do gatilho', () => {
+  it('encontra o bloco do botão gatilho (o mais próximo do ícone Trash2)', () => {
     expect(trigger).toBeDefined()
   })