Skip to content

chore: triagem dos parâmetros não usados (lint 210 → 150) - #175

Merged
iacapuca merged 2 commits into
mainfrom
chore/type-lint-cleanup
Aug 5, 2026
Merged

chore: triagem dos parâmetros não usados (lint 210 → 150)#175
iacapuca merged 2 commits into
mainfrom
chore/type-lint-cleanup

Conversation

@iacapuca

@iacapuca iacapuca commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator

Triagem dos 65 avisos no-unused-vars do tipo Parameter, um a um, checando call sites. Nenhuma mudança de comportamento.

O que foi feito

  • parâmetro posicional → prefixo _
  • prop desestruturada não usada → removida do padrão de desestruturação. Interfaces intactas, então nenhum call site quebra
  • os cinco key do Table.tsx({ key: _key, ...props }). Não são render props: são IIFEs que existem para tirar o key do objeto espalhado (React 19 avisa ao spreadar props com key). Remover quebraria; todo elemento já define key explicitamente

O que ficou warnando de propósito

Sete avisos foram preservados sem prefixo, porque apontam para bugs reais. Silenciar seria cimentar o defeito:

parâmetro issue
streetData, lat, lon #172 — requisição feita e resultado descartado
selectedTypes, bounds #172 — filtro e viewport ignorados
selectedInfra, execucaoCicloviaria #171 — geometria mock renderizada

Números

antes depois
lint 210 150
typecheck 281 281 (inalterado)
build

Contexto

A triagem revelou que os avisos não eram ruído uniforme — apontavam para uma feature substancialmente não funcional. Cinco issues abertas com o caminho do código e a cadeia até a tela:

As categorias Variable (49) e Identifier (18) não foram tocadas — ficam para um próximo passo.

🤖 Generated with Claude Code

iacapuca and others added 2 commits August 5, 2026 11:18
Aplicado por `oxlint --fix`. Dentro de character class, `\/` e `\#`
sao equivalentes a `/` e `#` — comportamento identico.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01NzvzBUT8kw8SbR4bsGab97
Triagem dos 65 parametros nao usados, um a um, checando call sites.
60 eram genuinamente mortos; 5 apontavam para bugs reais e ficaram
warnando de proposito.

- parametro posicional: prefixo `_`
- prop desestruturada nao usada: removida do padrao (interface intacta,
  nenhum call site quebra)
- os cinco `key` do Table.tsx viraram `key: _key` — sao um workaround
  correto para o React 19 nao receber `key` dentro de spread do
  react-table v7, remover quebraria

Preservados sem prefixo, para o lint continuar cobrando:
  streetData, lat, lon        -> #172
  selectedTypes, bounds       -> #172
  selectedInfra, execucaoCicloviaria -> #171

Lint 210 -> 150. Typecheck inalterado em 281, build passa.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01NzvzBUT8kw8SbR4bsGab97
@github-actions

github-actions Bot commented Aug 5, 2026

Copy link
Copy Markdown

🚀 Preview deploy

URL https://6ad5cc59-ameciclo.ti-8b9.workers.dev
Version 6ad5cc59-a793-4f4e-a895-c0df277cb041
Commit 77fee14e1562b756479738d40f6ca82a364994e6

Updates on every push. Production is unchanged.

@iacapuca
iacapuca merged commit aee880a into main Aug 5, 2026
3 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant