chore: pin peer dependency version ranges#566
Conversation
Replace `*` wildcards with bounded ranges in published packages and pin the root tsconfig devDependency: - packages/ethers, packages/wagmi: peerDependencies `@react-native-community/netinfo` and `react-native-get-random-values` changed from `*` to `>=11.0.0` / `>=1.11.0` - root tsconfig devDependency `*` -> `7.0.0` Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
🦋 Changeset detectedLatest commit: b3c8126 The changes in this PR will be included in the next version bump. This PR includes changesets to release 9 packages
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 |
|
Claude finished @ignaciosantise's task in 28s —— View job PR Review
License compliance: ✅ No new dependencies introduced — only version range adjustments on pre-existing deps. Found 2 issue(s)Issue 1: peerDependency floor tightening may cause install-time failures for consumers on old netinfo versionsID: brk-ethers-wagmi-netinfo-peerdep-floor-a3c2 Context:
Recommendation: The tightening is semantically correct. Consider whether Issue 2: peerDependency floor tightening may cause install-time failures for consumers on old react-native-get-random-values versionsID: brk-ethers-wagmi-getrandomvalues-peerdep-floor-b7d4 Context:
Recommendation: Same as above — the floor is accurate and should stay. Consider bumping the changeset from Additional observation: The changeset bumps all 9 packages for version alignment, but only |
There was a problem hiding this comment.
Pull request overview
This PR updates the monorepo’s published package metadata to avoid * wildcards by pinning peer dependency ranges (and one root devDependency) to explicit versions/ranges, aiming to make consumer installs more predictable.
Changes:
- Replace
*peerDependency wildcards with explicit lower-bounded ranges for@react-native-community/netinfoandreact-native-get-random-valuesin theethersandwagmipackages. - Pin the root
tsconfigdevDependency to7.0.0. - Sync
yarn.lockand add a Changeset to bump all packages (patch).
Reviewed changes
Copilot reviewed 4 out of 5 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| yarn.lock | Updated lockfile to reflect the new peer/dev dependency constraints. |
| packages/wagmi/package.json | Replaced wildcard peer deps with explicit version floors for netinfo + random-values. |
| packages/ethers/package.json | Replaced wildcard peer deps with explicit version floors for netinfo + random-values. |
| package.json | Pinned tsconfig devDependency from * to 7.0.0. |
| .changeset/cuddly-heads-type.md | Adds a patch Changeset to keep package versions aligned. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| "@react-native-community/netinfo": ">=11.0.0", | ||
| "@walletconnect/react-native-compat": ">=2.16.1", | ||
| "react": ">=18", | ||
| "react-native": ">=0.72", | ||
| "react-native-get-random-values": "*", | ||
| "react-native-get-random-values": ">=1.11.0", |
| "@react-native-community/netinfo": ">=11.0.0", | ||
| "@walletconnect/react-native-compat": ">=2.16.1", | ||
| "react": ">=18", | ||
| "react-native": ">=0.72", | ||
| "react-native-get-random-values": "*" | ||
| "react-native-get-random-values": ">=1.11.0" |
Summary
Removes
*wildcards from published package metadata so consumers installing the SDK via npm get bounded, predictable version ranges instead of "any version ever published."Changes
packages/ethers&packages/wagmi— peerDependencies@react-native-community/netinfoandreact-native-get-random-valueschanged from*to>=11.0.0and>=1.11.0respectively (floors match the versions this repo builds/tests against; consistent with the existing>=peer-dep convention).package.json—tsconfigdevDependency pinned from*to7.0.0(latest; same version that was already resolving). Not published, but removes the only remaining wildcard in the repo's own install.yarn.locksynced.All regular
dependenciesin the published packages were already exact-pinned — no change needed there.Notes
🤖 Generated with Claude Code