jhony75/eeg-qa
Watch
1
0
Fork
You've already forked eeg-qa
 0
Code Issues Pull requests 1 Projects Releases Packages 3 Wiki Activity Actions

chore: atualizar react-router-dom pra 7.18.3 #25

Closed
jhony75 wants to merge 1 commit from chore/react-router-7-upgrade into develop
pull from: chore/react-router-7-upgrade
merge into: jhony75:develop
Conversation 0 Commits 1 Files changed 3 +48 -27
jhony75 commented 2026-09-05 00:12:07 -03:00
Owner
Copy link

Corrige 2 advisories moderados do npm audit (faixa vulnerável ia até 7.17.0).

Avaliei trocar por TanStack Router e descartei: o valor dele é inferência de tipo de rota ponta-a-ponta, e o frontend é 100% JavaScript (zero .ts/.tsx) — sem TypeScript esse ganho não existe, e reescrever toda a navegação só pra fechar 2 CVEs que um bump de versão já resolve não compensava.

API usada (BrowserRouter/Routes/Route/NavLink/useNavigate/useParams) não mudou entre v6 e v7 — build e testes passam sem nenhuma alteração de código, só o bump de versão.

Corrige 2 advisories moderados do npm audit (faixa vulnerável ia até 7.17.0). Avaliei trocar por TanStack Router e descartei: o valor dele é inferência de tipo de rota ponta-a-ponta, e o frontend é 100% JavaScript (zero `.ts`/`.tsx`) — sem TypeScript esse ganho não existe, e reescrever toda a navegação só pra fechar 2 CVEs que um bump de versão já resolve não compensava. API usada (`BrowserRouter`/`Routes`/`Route`/`NavLink`/`useNavigate`/`useParams`) não mudou entre v6 e v7 — build e testes passam sem nenhuma alteração de código, só o bump de versão.
chore: atualizar react-router-dom pra 7.18.3 (corrige 2 advisories moderados)
All checks were successful
EEG QA CI / validate-and-build (pull_request) Successful in 2m9s
Details
EEG QA CI / Deploy production (pull_request) Has been skipped
Details
EEG QA CI / Deploy development (pull_request) Has been skipped
Details
e5615f06cc 
Faixa vulnerável ia até 7.17.0. Avaliado trocar por TanStack Router e
descartado: o valor dele é inferência de tipo ponta-a-ponta, e o frontend
é 100% JS (zero .ts/.tsx) — sem TypeScript esse ganho não existe, e
reescrever toda a navegação só pra fechar 2 CVEs que um bump resolve não
compensava. API usada (BrowserRouter/Routes/Route/NavLink/useNavigate/
useParams) não mudou entre v6 e v7 — build e testes passam sem alteração
de código.
jhony75 closed this pull request 2026-09-05 02:46:19 -03:00
All checks were successful
EEG QA CI / validate-and-build (pull_request) Successful in 2m9s
Details
EEG QA CI / Deploy production (pull_request) Has been skipped
Details
EEG QA CI / Deploy development (pull_request) Has been skipped
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
jhony75/eeg-qa!25
No description provided.