From 14fda0efee49d0f1adac4e40dcd5352b9a321c4c Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Thu, 3 Sep 2026 13:08:27 +0000
Subject: [PATCH 1/2] =?UTF-8?q?fix:=20aria-label=20no=20l=C3=A1pis=20de=20?=
 =?UTF-8?q?editar=20em=20metas,=20or=C3=A7amento=20e=20investimentos=20(cl?=
 =?UTF-8?q?oses=20#128)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Os três lápis de editar em GoalDialog, BudgetOverrideDialog e
InvestmentEntryDialog envolvem só um ícone Pencil sem aria-label — o
Icon do lucide-react@1.8.0 marca o svg como aria-hidden quando não
recebe filho nem prop de a11y, então o botão chega ao leitor de tela
como role=button name="".

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012WLtBDDRT1KDQqQtNjTgjK
---
 __tests__/unit/a11y-nomes-acessiveis.test.ts  | 76 +++++++++++++++++++
 .../configuracao-mes/BudgetOverrideDialog.tsx |  1 +
 .../investimentos/InvestmentEntryDialog.tsx   |  1 +
 components/metas/GoalDialog.tsx               |  1 +
 4 files changed, 79 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..0567b34
--- /dev/null
+++ b/__tests__/unit/a11y-nomes-acessiveis.test.ts
@@ -0,0 +1,76 @@
+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) ──────────────────────────
+//
+// `lucide-react@1.8.0` (package.json:45) marca o <svg> do ícone como
+// `aria-hidden="true"` quando ele não recebe filho nem prop de a11y — então
+// um <Button size="icon"> cujo único filho é um ícone chega ao leitor de tela
+// como `role=button name=""`. Não há infra de render de componente no
+// projeto (sem jsdom/testing-library); o gate possível é sobre o
+// texto-fonte, mesmo padrão de row-actions.test.ts (#54) e
+// a11y-estado-selecao.test.ts (#107).
+//
+// Cada asserção ancora no bloco JSX específico do botão-alvo, não em
+// `toMatch(/aria-label/)` sobre o arquivo inteiro — os três componentes desta
+// fatia (#128) têm mais de um <Button> no mesmo trecho (ternário de 2 ou 3
+// braços), então uma asserção genérica passaria com qualquer um deles
+// rotulado, deixando o lápis de editar intacto.
+
+describe('InvestmentEntryDialog — lápis de "Editar registro" (#128, site 2)', () => {
+  const source = readFileSync(
+    join(process.cwd(), 'components/investimentos/InvestmentEntryDialog.tsx'),
+    'utf-8'
+  )
+
+  // Ramo `existing` do ternário de três braços (existing / isGlobal / default)
+  const existingBranch = source.match(/existing \? \(([\s\S]*?)\) : isGlobal \? \(/)?.[1]
+
+  it('encontra o ramo `existing` do trigger', () => {
+    expect(existingBranch).toBeDefined()
+  })
+
+  it('rotula o botão do lápis de editar, não o ícone', () => {
+    expect(existingBranch).toMatch(/^\s*aria-label="Editar registro"$/m)
+  })
+
+  it('não rotula os outros dois ramos do ternário (isGlobal / default) com o mesmo texto', () => {
+    const rest = source.slice(source.indexOf(') : isGlobal ? ('))
+    expect(rest).not.toMatch(/aria-label="Editar registro"/)
+  })
+})
+
+describe('BudgetOverrideDialog — lápis de "Editar orçamento" (#128, site 6)', () => {
+  const source = readFileSync(
+    join(process.cwd(), 'components/configuracao-mes/BudgetOverrideDialog.tsx'),
+    'utf-8'
+  )
+
+  it('rotula o botão do lápis de editar, não o ícone', () => {
+    expect(source).toMatch(/^\s*aria-label="Editar orçamento"$/m)
+  })
+})
+
+describe('GoalDialog — lápis de "Editar meta" (#128, site 7)', () => {
+  const source = readFileSync(join(process.cwd(), 'components/metas/GoalDialog.tsx'), 'utf-8')
+
+  // Ramo `else` do ternário `mode === 'create' ? (...) : (...)` — captura até
+  // o primeiro `</Button>` após o separador, não até o primeiro `)}`: o botão
+  // tem `onClick={() => setOpen(true)}`, que termina em `)}` e cortaria a
+  // captura antes do `aria-label` se o limite fosse esse.
+  const editBranch = source.match(/\) : \(([\s\S]*?<\/Button>)/)?.[1]
+
+  it('encontra o ramo de edição do trigger', () => {
+    expect(editBranch).toBeDefined()
+  })
+
+  it('rotula o botão do lápis de editar, não o ícone', () => {
+    expect(editBranch).toMatch(/^\s*aria-label="Editar meta"$/m)
+  })
+
+  it('não rotula o ramo `create` (Nova meta) com o mesmo texto', () => {
+    const createBranch = source.match(/mode === 'create' \? \(([\s\S]*?)\) : \(/)?.[1]
+    expect(createBranch).not.toMatch(/aria-label/)
+  })
+})
diff --git a/components/configuracao-mes/BudgetOverrideDialog.tsx b/components/configuracao-mes/BudgetOverrideDialog.tsx
index 7d71099..e126353 100644
--- a/components/configuracao-mes/BudgetOverrideDialog.tsx
+++ b/components/configuracao-mes/BudgetOverrideDialog.tsx
@@ -127,6 +127,7 @@ export function BudgetOverrideDialog({
         variant="ghost"
         className="h-6 w-6 text-text-tertiary hover:text-text-primary"
         onClick={() => setOpen(true)}
+        aria-label="Editar orçamento"
       >
         <Pencil className="h-3 w-3" />
       </Button>
diff --git a/components/investimentos/InvestmentEntryDialog.tsx b/components/investimentos/InvestmentEntryDialog.tsx
index c96d357..64c4089 100644
--- a/components/investimentos/InvestmentEntryDialog.tsx
+++ b/components/investimentos/InvestmentEntryDialog.tsx
@@ -189,6 +189,7 @@ export function InvestmentEntryDialog({
         variant="ghost"
         className="h-7 w-7 text-text-tertiary hover:text-text-primary"
         onClick={() => setOpen(true)}
+        aria-label="Editar registro"
       >
         <Pencil className="h-3 w-3" />
       </Button>
diff --git a/components/metas/GoalDialog.tsx b/components/metas/GoalDialog.tsx
index 2b13c20..50c39df 100644
--- a/components/metas/GoalDialog.tsx
+++ b/components/metas/GoalDialog.tsx
@@ -158,6 +158,7 @@ export function GoalDialog(props: Props) {
           variant="ghost"
           className="h-7 w-7 text-text-tertiary hover:text-text-primary"
           onClick={() => setOpen(true)}
+          aria-label="Editar meta"
         >
           <Pencil className="h-3.5 w-3.5" />
         </Button>

From 84d8b8e3a5a8cc5c0891ac4f09f5d7279bb148fc Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Thu, 3 Sep 2026 15:22:38 +0000
Subject: [PATCH 2/2] =?UTF-8?q?fix:=20ancora=20asser=C3=A7=C3=A3o=20do=20B?=
 =?UTF-8?q?udgetOverrideDialog=20no=20bot=C3=A3o=20mais=20pr=C3=B3ximo=20d?=
 =?UTF-8?q?o=20=C3=ADcone?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Achado no review de #144: expect(source).toMatch(/aria-label/) sobre o
arquivo inteiro passava mesmo com o rótulo no botão errado ("Usar
padrão"), porque o gatilho não é ramo de ternário e o arquivo tem dois
outros <Button> fora dele. Verificado por mutação: reverter o rótulo
do trigger e colocá-lo no botão de reset deixava o teste anterior
verde; a nova captura (mais próximo do <Pencil>) falha corretamente.

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

diff --git a/__tests__/unit/a11y-nomes-acessiveis.test.ts b/__tests__/unit/a11y-nomes-acessiveis.test.ts
index 0567b34..178282f 100644
--- a/__tests__/unit/a11y-nomes-acessiveis.test.ts
+++ b/__tests__/unit/a11y-nomes-acessiveis.test.ts
@@ -47,8 +47,19 @@ describe('BudgetOverrideDialog — lápis de "Editar orçamento" (#128, site 6)'
     'utf-8'
   )
 
+  // O gatilho aqui não é ramo de ternário — é o único <Button> do bloco de
+  // abertura, mas o arquivo tem outros dois <Button> fora dele (submit e
+  // "Usar padrão"). `toMatch` sobre `source` cru passaria com o rótulo em
+  // qualquer um dos três; a captura precisa mirar o <Button> mais próximo do
+  // ícone Pencil, não a primeira ocorrência de `aria-label` no arquivo.
+  const trigger = source.match(/<Button\b(?:(?!<Button\b)[\s\S])*?<Pencil\b[\s\S]*?<\/Button>/)?.[0]
+
+  it('encontra o bloco do botão de editar (o mais próximo do ícone Pencil)', () => {
+    expect(trigger).toBeDefined()
+  })
+
   it('rotula o botão do lápis de editar, não o ícone', () => {
-    expect(source).toMatch(/^\s*aria-label="Editar orçamento"$/m)
+    expect(trigger).toMatch(/^\s*aria-label="Editar orçamento"$/m)
   })
 })