Update npm dependencies and pin eslint-plugin-react-hooks via pnpm over… - #106
buildcanada-fabro[bot] wants to merge 11 commits into
Conversation
Fabro-Run: 01M3AM0MMWXN5W6Y5BJ82P3FJY Fabro-Completed: 2 ⚒️ Generated with [Fabro](https://fabro.sh) Co-Authored-By: Fabro <noreply@fabro.sh>
Fabro-Run: 01M3AM0MMWXN5W6Y5BJ82P3FJY Fabro-Completed: 3 ⚒️ Generated with [Fabro](https://fabro.sh) Co-Authored-By: Fabro <noreply@fabro.sh>
Fabro-Run: 01M3AM0MMWXN5W6Y5BJ82P3FJY Fabro-Completed: 4 ⚒️ Generated with [Fabro](https://fabro.sh) Co-Authored-By: Fabro <noreply@fabro.sh>
Fabro-Run: 01M3AM0MMWXN5W6Y5BJ82P3FJY Fabro-Completed: 5 ⚒️ Generated with [Fabro](https://fabro.sh) Co-Authored-By: Fabro <noreply@fabro.sh>
Fabro-Run: 01M3AM0MMWXN5W6Y5BJ82P3FJY Fabro-Completed: 6 ⚒️ Generated with [Fabro](https://fabro.sh) Co-Authored-By: Fabro <noreply@fabro.sh>
Fabro-Run: 01M3AM0MMWXN5W6Y5BJ82P3FJY Fabro-Completed: 7 ⚒️ Generated with [Fabro](https://fabro.sh) Co-Authored-By: Fabro <noreply@fabro.sh>
Fabro-Run: 01M3AM0MMWXN5W6Y5BJ82P3FJY Fabro-Completed: 8 ⚒️ Generated with [Fabro](https://fabro.sh) Co-Authored-By: Fabro <noreply@fabro.sh>
Fabro-Run: 01M3AM0MMWXN5W6Y5BJ82P3FJY Fabro-Completed: 9 ⚒️ Generated with [Fabro](https://fabro.sh) Co-Authored-By: Fabro <noreply@fabro.sh>
Fabro-Run: 01M3AM0MMWXN5W6Y5BJ82P3FJY Fabro-Completed: 10 ⚒️ Generated with [Fabro](https://fabro.sh) Co-Authored-By: Fabro <noreply@fabro.sh>
|
| importers: | ||
|
|
||
| .: | ||
| dependencies: | ||
| '@base-ui/react': | ||
| specifier: ^1.3.0 | ||
| version: 1.3.0(@types/react@19.2.14)(react-dom@19.2.3(react@19.2.3))(react@19.2.3) | ||
| version: 1.8.0(@types/react@19.2.14)(date-fns@4.4.0)(react-dom@19.2.3(react@19.2.3))(react@19.2.3) | ||
| '@buildcanada/charts': |
There was a problem hiding this comment.
Survey select lacks interaction coverage
This updates the Base UI select used by the voter survey from 1.3.0 to 1.8.0, but there is no component or browser test for choosing an option and submitting its value. The existing build and library tests cannot catch a regression in that survey interaction. Add a focused test for it.
Prompt To Fix With AI
This is a comment left during a code review.
Path: pnpm-lock.yaml
Line: 17
Comment:
**Survey select lacks interaction coverage**
This updates the Base UI select used by the voter survey from 1.3.0 to 1.8.0, but there is no component or browser test for choosing an option and submitting its value. The existing build and library tests cannot catch a regression in that survey interaction. Add a focused test for it.
---
For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
Refreshes the dependency tree (package.json + pnpm-lock.yaml) with all updates that were verified to build and pass lint/tests. Also adds a pnpm
overridesentry pinningeslint-config-next > eslint-plugin-react-hooksto7.0.1to resolve a version mismatch pulled in transitively byeslint-config-next@16.1.7.Notable version bumps
next16.1.6 → 16.1.7,eslint-config-next16.1.6 → 16.1.7,next-auth4.24.13 → 4.24.15jose^5.9.6 → ^6.2.12openai^5.15.0 → ^7.20.0three/@types/three^0.185.1 → ^0.186.0@types/node^20 → ^26.6.2tsxunpinned range → pinned4.21.0@react-three/*, etc.)These were bundled together because they built cleanly and had no breaking API usage detected in this codebase. Any major upgrade found to actually change API surface used here is tracked separately via its own issue rather than included in this batch.
Fabro Details
Ran 9 stages in 58m 14s for $0.61
Ran
DependencyUpdate.fabro(10 nodes and 11 edges)⚒️ Generated with Fabro