fix: accept loose peer ranges in peers check (#12150)

minbang930 · codex · web-flow · commit 3188ae7026af · 2026-06-16T20:20:53.000+02:00
Run peer dependency range checks with semver loose parsing while preserving prerelease inclusion. This prevents pnpm peers check from reporting react-native-reanimated@4.4.0 as unmet for ranges like >=3.16.0 || >=4.0.0-. Add a lockfile regression fixture for #12149 and a patch changeset for the peer checker and pnpm CLI. Closes #12149. Co-authored-by: OpenAI Codex <codex@openai.com>
diff --git a/.changeset/loose-peers-check.md b/.changeset/loose-peers-check.md
@@ -0,0 +1,6 @@
+---
+"@pnpm/deps.inspection.peers-checker": patch
+"pnpm": patch
+---
+
+Fixed `pnpm peers check` to accept loose peer dependency ranges such as `>=3.16.0 || >=4.0.0-` when the installed peer version satisfies the range [#12149](https://github.com/pnpm/pnpm/issues/12149).
diff --git a/deps/inspection/peers-checker/src/checkPeerDependencies.ts b/deps/inspection/peers-checker/src/checkPeerDependencies.ts
@@ -135,7 +135,7 @@ function extractVersion (ref: string, alias: string, packages: PackageSnapshots)
 
 function satisfies (version: string, range: string): boolean {
   if (range === '*') return true
-  return semver.satisfies(version, range, { includePrerelease: true })
+  return semver.satisfies(version, range, { includePrerelease: true, loose: true })
 }
 
 function filterPeerDependencyIssues (
diff --git a/deps/inspection/peers-checker/test/__fixtures__/with-loose-peer-range/package.json b/deps/inspection/peers-checker/test/__fixtures__/with-loose-peer-range/package.json
@@ -0,0 +1,8 @@
+{
+  "name": "with-loose-peer-range",
+  "version": "1.0.0",
+  "dependencies": {
+    "@gorhom/bottom-sheet": "5.2.14",
+    "react-native-reanimated": "4.4.0"
+  }
+}
diff --git a/deps/inspection/peers-checker/test/__fixtures__/with-loose-peer-range/pnpm-lock.yaml b/deps/inspection/peers-checker/test/__fixtures__/with-loose-peer-range/pnpm-lock.yaml
diff --git a/deps/inspection/peers-checker/test/checkPeerDependencies.test.ts b/deps/inspection/peers-checker/test/checkPeerDependencies.test.ts
@@ -58,6 +58,19 @@ test('reports no issues for satisfied peer dependencies', async () => {
   expect(Object.keys(projectIssues.missing)).toHaveLength(0)
 })
 
+test('reports no issues for satisfied loose peer dependency ranges', async () => {
+  const fixture = f.find('with-loose-peer-range')
+  const issues = await checkPeerDependencies([fixture], {
+    lockfileDir: fixture,
+    checkWantedLockfileOnly: true,
+  })
+
+  const projectIssues = issues['.']
+  expect(projectIssues).toBeDefined()
+  expect(Object.keys(projectIssues.bad)).toHaveLength(0)
+  expect(Object.keys(projectIssues.missing)).toHaveLength(0)
+})
+
 test('respects peerDependencyRules.allowAny', async () => {
   const fixture = f.find('with-unmet-peers')
   const issues = await checkPeerDependencies([fixture], {

Original file line number	Diff line number	Diff line change
`@@ -135,7 +135,7 @@ function extractVersion (ref: string, alias: string, packages: PackageSnapshots)`
`135`	`135`
`136`	`136`	`function satisfies (version: string, range: string): boolean {`
`137`	`137`	`if (range === '*') return true`
`138`		`- return semver.satisfies(version, range, { includePrerelease: true })`
	`138`	`+ return semver.satisfies(version, range, { includePrerelease: true, loose: true })`
`139`	`139`	`}`
`140`	`140`
`141`	`141`	`function filterPeerDependencyIssues (`