fix: useBlocker useEffect infinite loop

[fernandojbf]

• Fix the infinite loop present in useBlocker.
• Introduces some unit tests to find if the hook breaks the basic functionalities

I believe that will fix remix issue introduced by react-router -> remix-run/remix#6704

Feedback always welcome
(hope the .changeset is correct 😕 😃)

Cheers

[changeset-bot]

🦋 Changeset detected

Latest commit: 152fa35

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 4 packages

Name	Type
react-router	Patch
react-router-dom	Patch
react-router-dom-v5-compat	Patch
react-router-native	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[remix-cla-bot]

Thank you for signing the Contributor License Agreement. Let's get this merged! 🥳

[brophdawg11]

Thank you for the PR - but I'm afraid we can't use this approach as the solution because useId is only available in React 18, and we support down to React 16.8 at the moment. We can't use 18-only features until we drop support for earlier versions in an eventual major semver release, which is not planned anytime soon.

I think I have an idea of how we can solve this though so I'll try to get a separate PR up once I can validate the approach.

[fernandojbf]

oh forgot about that. 🤦

The unique incompatible part would be the generation of the id.