chore(deps): add wanted dependencies to remove yarn warnings

[kelset]

Summary

I wanted to start working on a thing but this barrage of warnings was very annoying so ended up doing this instead: a very small PR to take care of some warnings during yarn install.

It doesn't change anything (the versions are the ones already used all around the repo), just makes yarn happier.

Also, while doing that I found a dependency that was lying there unused for 2 years so took care of it.

Changelog

[General] [Fixed] - add wanted dependencies to remove yarn warnings

Test Plan

Before

After

[analysis-bot]

Platform	Engine	Arch	Size (bytes)	Diff
ios	-	universal	n/a	--

Base commit: fd91748
Branch: main

[facebook-github-bot]

@robhogan has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

[robhogan]

We have a convention (internally enforced, possibly?) of alphabetically ordering dependencies but otherwise LGTM - thanks @kelset

[kelset]

@robhogan no problem, thanks for the suggestions, I've applied them! Is there some tooling that we could use to automate this? Kind of like prettier but for package.json? 😅

[facebook-github-bot]

@robhogan has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

[robhogan]

@robhogan no problem, thanks for the suggestions, I've applied them! Is there some tooling that we could use to automate this? Kind of like prettier but for package.json? 😅

Yeah we should, tbh I thought we had an autofix lint rule for it already. Yarn and NPM auto-sort whenever they modify the file iirc, but it looks like we don't have any check - package-json/sort-collections would work. CC @cipolleschi

[react-native-bot]

This pull request was successfully merged by @kelset in 8422c53.

^{When will my fix make it into a release? | Upcoming Releases}

[facebook-github-bot]

This pull request has been reverted by cf5addf.

[robhogan]

Reopening as this was reverted due to an (apparently) missing internal dependency that previously worked transitively.

Meta folks: See D40716713 before re-landing.

[kelset]

@robhogan quick check, anything I can do to help get this in? do you want me to rebase it on top of main?

[robhogan]

@robhogan quick check, anything I can do to help get this in? do you want me to rebase it on top of main?

Sorry about this, nothing you can do from the OSS end. Removing the @reactions/component dependency is tripping up Flow internally for reasons that are unclear at the moment, some config/resolution bug - we're working on it.

If it'd help you to get this in, you could submit a new PR (has to be new - this one can't be re-imported now it's been committed) without the @reactions/component removal, that should be a straightforward merge.

Otherwise, I'm hoping we can unblock this within a few days.

[kelset]

@robhogan here you go: #35122

[kelset]

the other one got merged, so I'll close this one off