Skip to content

chore(deps): update dependency eslint-plugin-react-hooks to v7 - #27

Open
renovate-appwrite[bot] wants to merge 1 commit into
mainfrom
renovate/major-react-monorepo
Open

renovate-appwrite[bot] wants to merge 1 commit into
mainfrom
renovate/major-react-monorepo

Conversation

@renovate-appwrite

Copy link
Copy Markdown

This PR contains the following updates:

Package Change Age Confidence
eslint-plugin-react-hooks (source) ^5.0.0 → ^7.0.0 age confidence

Release Notes

facebook/react (eslint-plugin-react-hooks)

v7.1.1

Compare Source

Note: 7.1.0 accidentally removed the component-hook-factories rule, causing errors for users who referenced it in their ESLint config. This is now fixed.

  • Add deprecated no-op component-hook-factories rule for backwards compatibility. (@​mofeiZ in #​36307)

v7.1.0

Compare Source

This release adds ESLint v10 support, improves performance by skipping compilation for non-React files, and includes compiler lint improvements including better set-state-in-effect detection, improved ref validation, and more helpful error reporting.

v7.0.1

Compare Source

v7.0.0

Compare Source

This release slims down presets to just 2 configurations (recommended and recommended-latest), and all compiler rules are enabled by default.

  • Breaking: Removed recommended-latest-legacy and flat/recommended configs. The plugin now provides recommended (legacy and flat configs with all recommended rules), and recommended-latest (legacy and flat configs with all recommended rules plus new bleeding edge experimental compiler rules). (@​poteto in #​34757)

v6.1.1

Compare Source

Note: 6.1.0 accidentally allowed use of recommended without flat config, causing errors when used with ESLint v9's defineConfig() helper. This has been fixed in 6.1.1.

v6.1.0

Compare Source

Note: Version 6.0.0 was mistakenly released and immediately deprecated and untagged on npm. This is the first official 6.x major release and includes breaking changes.

  • Breaking: Require Node.js 18 or newer. (@​michaelfaith in #​32458)
  • Breaking: Flat config is now the default recommended preset. Legacy config moved to recommended-legacy. (@​michaelfaith in #​32457)
  • New Violations: Disallow calling use within try/catch blocks. (@​poteto in #​34040)
  • New Violations: Disallow calling useEffectEvent functions in arbitrary closures. (@​jbrown215 in #​33544)
  • Handle React.useEffect in addition to useEffect in rules-of-hooks. (@​Ayc0 in #​34076)
  • Added react-hooks settings config option that to accept additionalEffectHooks that are used across exhaustive-deps and rules-of-hooks rules. (@​jbrown215) in #​34497

v6.0.0

Compare Source

Accidentally released. See 6.1.0 for the actual changes.


Configuration

📅 Schedule: (UTC)

  • Branch creation
    • Only on Monday (* * * * 1)
  • Automerge
    • At any time (no schedule defined)

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

This PR has been generated by Mend Renovate CLI.

@renovate-appwrite renovate-appwrite Bot added the dependencies Pull requests that update a dependency file label Nov 9, 2026
@appwrite

appwrite Bot commented Nov 9, 2026 •

Copy link
Copy Markdown

🔵 Tier A · Mergeable after minor fixes

The dependency installs, but the new recommended rules introduce a minor lint regression that should be addressed.

Updates eslint-plugin-react-hooks from the 5.x range to 7.x and refreshes the npm lockfile, including the plugin's newly required parser and validation dependencies. The 7.x release also changes the rules exposed by the recommended preset consumed by the repository's ESLint configuration.

Verdict New comments Fixed Still open
✅ Approved 1 0 0
Finding Where
🟡 The upgrade adds a lint error under the configured recommended preset package-lock.json:2750
Fix with agent prompt
### Issue 1
package-lock.json:2750
**The upgrade adds a lint error under the configured recommended preset**

With `eslint.config.js` spreading `reactHooks.configs.recommended.rules`, this version enables compiler rules that the old preset did not include. Running `npm run lint` now reports `react-hooks/immutability` at `src/App.jsx:30` because `getItems` is called before its declaration, adding a new lint failure to this upgrade.

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.
📂 Walkthrough · 2
File Change
package.json Raises the declared eslint-plugin-react-hooks dependency to ^7.0.0.
package-lock.json Locks eslint-plugin-react-hooks to 7.1.1 and adds its transitive dependencies.

Reviewed bf238f9 · Details · Comment @appwrite review to re-run, or mention @appwrite with a question.

@appwrite appwrite Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔵 Tier A · Looks good to merge. Summary

Comment thread package-lock.json
"version": "5.2.0",
"resolved": "https://registry.npmjs.org/eslint-plugin-react-hooks/-/eslint-plugin-react-hooks-5.2.0.tgz",
"integrity": "sha512-+f15FfK64YQwZdJNELETdn5ibXEUQmW1DZL6KXhNnc2heoy/sg9VJJeT7n8TlMWouzWqSWavFkIhHyIbIAEapg==",
"version": "7.1.1",

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The upgrade adds a lint error under the configured recommended preset

With eslint.config.js spreading reactHooks.configs.recommended.rules, this version enables compiler rules that the old preset did not include. Running npm run lint now reports react-hooks/immutability at src/App.jsx:30 because getItems is called before its declaration, adding a new lint failure to this upgrade.

Prompt To Fix With AI
This is a comment left during a code review.
Path: package-lock.json
Line: 2750

Comment:
**The upgrade adds a lint error under the configured recommended preset**

With `eslint.config.js` spreading `reactHooks.configs.recommended.rules`, this version enables compiler rules that the old preset did not include. Running `npm run lint` now reports `react-hooks/immutability` at `src/App.jsx:30` because `getItems` is called before its declaration, adding a new lint failure to this upgrade.

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.

🟡 Minor · testing · Reply if this doesn't apply.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants