chore(deps): upgrade all dependencies (2026-04-16)#7
Open
devin-ai-integration[bot] wants to merge 12 commits into
Open
chore(deps): upgrade all dependencies (2026-04-16)#7devin-ai-integration[bot] wants to merge 12 commits into
devin-ai-integration[bot] wants to merge 12 commits into
Conversation
- @testing-library/jest-dom: 5.16.3 → 5.17.0 - @testing-library/react: 12.1.4 → 12.1.5 - react-transition-group: 4.4.2 → 4.4.5 - react-scripts: 5.0.0 → 5.0.1 Co-Authored-By: Phil Bedford <phil.bedford@cognition.ai>
- classnames: 2.3.1 → 2.5.1 - react-countdown: 2.3.2 → 2.3.6 - react-icons: 4.3.1 → 4.12.0 - sass: 1.49.9 → 1.99.0 - lint-staged: 12.3.7 → 12.5.0 Co-Authored-By: Phil Bedford <phil.bedford@cognition.ai>
Co-Authored-By: Phil Bedford <phil.bedford@cognition.ai>
Co-Authored-By: Phil Bedford <phil.bedford@cognition.ai>
Reformatted source files to match prettier 3 style. Co-Authored-By: Phil Bedford <phil.bedford@cognition.ai>
Co-Authored-By: Phil Bedford <phil.bedford@cognition.ai>
Co-Authored-By: Phil Bedford <phil.bedford@cognition.ai>
Co-Authored-By: Phil Bedford <phil.bedford@cognition.ai>
Migrated from ReactDOM.render to createRoot API. Co-Authored-By: Phil Bedford <phil.bedford@cognition.ai>
…nd @testing-library/react from 12.1.5 to 16.3.2 Co-Authored-By: Phil Bedford <phil.bedford@cognition.ai>
…ttier Co-Authored-By: Phil Bedford <phil.bedford@cognition.ai>
Author
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
React 19 removed findDOMNode. react-transition-group v4 requires nodeRef prop to avoid calling findDOMNode internally. Co-Authored-By: Phil Bedford <phil.bedford@cognition.ai>
Author
Frontend Verification Results — All 5 Tests PASSED
Key Validations
Full video recording of the walkthrough is attached to the PR session. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Upgrades all 15 outdated npm dependencies across patch, minor, and major versions. The most significant change is the React 17 → 19 migration, which required updating
src/index.jsfrom the legacyReactDOM.renderAPI to thecreateRootAPI and addingnodeRefprops toCSSTransitioncomponents to avoidfindDOMNodecrashes.Patch upgrades
Minor upgrades
Major upgrades
Source code changes
src/index.js: Migrated fromReactDOM.render()tocreateRoot()(required for React 18+)src/components/Alert/Alert.jsx: AddeduseRef+nodeRefprop toCSSTransitionto avoidfindDOMNodecrash in React 19src/components/Modal/Modal.jsx: AddednodeRefprop toCSSTransitionto avoidfindDOMNodecrash in React 19src/App.jsx,src/hooks/useOnClickOutside.js,src/lib/words.js: Trailing comma formatting from Prettier 3src/styles/_transitionStyles.scss: Multi-linetransitionproperty formatting from Prettier 3Skipped packages
None — all outdated dependencies were successfully upgraded.
Validation
npm run check)react-scripts build)react-scripts test)main—App.test.jsxmissingAlertProviderwrapper)Review & Testing Checklist for Human
react-transition-groupv4 now usesnodeRefinstead offindDOMNode; test the info modal, settings modal, stats modal, and virtual keyboard to confirm transitions animate correctlyAlertcomponent'sCSSTransitionwas updated withnodeRef; trigger an alert (e.g., submit an invalid word) and confirm the fade-in/fade-out animation workssass@1.99may emit deprecation warnings for legacy SCSS features; React 19 may warn about legacy patternsreact-scripts@5.0.1+ React 19 is acceptable — CRA is unmaintained and react-scripts 5 was built for React 17/18; consider whether migrating off CRA is neededRecommended test plan: Start the dev server (
npm start), open http://localhost:3000, play through a full game (type guesses, see color feedback, win/lose), open each modal (info, settings with dark mode toggle, stats), and check the browser console for errors throughout.Notes
maindue toApp.test.jsxrendering<App>without the required<AlertProvider>wrapper. This is a pre-existing issue, not introduced by this PR.^) for most deps, exact versions forreact-scriptsandprettier(matching the originalpackage.json).react-scripts).ReactDOM.findDOMNode(), whichreact-transition-groupv4 calls internally whennodeRefis not provided. AddednodeRefto bothCSSTransitionusages (Alert.jsx,Modal.jsx) to prevent runtimeTypeError.Link to Devin session: https://app.devin.ai/sessions/d9c58d9621134031a7edca274004275a
Requested by: @dr-phil