fix: handle prerelease bounds in subset (#867)

puneetdixit200 · Puneet Dixit · web-flow · commit 5f3ca133e040 · 2026-05-21T09:09:17.000-07:00
## Summary - Fix `subset()` so prerelease lower and upper bounds are checked against individual comparators without reapplying full range prerelease gating. - Add coverage for `^10.2.0-beta.2` being a subset of `^10.2.0-beta.1`. ## Tests - `npx tap test/ranges/subset.js` - `npm test` - `git diff --check` Fixes #757 Co-authored-by: Puneet Dixit <rvit23bcs086.rvitm@rvei.edu.in>
diff --git a/ranges/subset.js b/ranges/subset.js
@@ -174,7 +174,7 @@ const simpleSubset = (sub, dom, options) => {
         if (higher === c && higher !== gt) {
           return false
         }
-      } else if (gt.operator === '>=' && !satisfies(gt.semver, String(c), options)) {
+      } else if (gt.operator === '>=' && !c.test(gt.semver)) {
         return false
       }
     }
@@ -192,7 +192,7 @@ const simpleSubset = (sub, dom, options) => {
         if (lower === c && lower !== lt) {
           return false
         }
-      } else if (lt.operator === '<=' && !satisfies(lt.semver, String(c), options)) {
+      } else if (lt.operator === '<=' && !c.test(lt.semver)) {
         return false
       }
     }
diff --git a/test/ranges/subset.js b/test/ranges/subset.js
@@ -29,6 +29,7 @@ const cases = [
   ['^1.2.3-pre.0', '>=1.0.0', true, { includePrerelease: true }],
   ['^1.2.3-pre.0', '>=1.2.3-pre.0', true],
   ['^1.2.3-pre.0', '>=1.2.3-pre.0', true, { includePrerelease: true }],
+  ['^10.2.0-beta.2', '^10.2.0-beta.1', true],
   ['>1.2.3-pre.0', '>=1.2.3-pre.0', true],
   ['>1.2.3-pre.0', '>1.2.3-pre.0 || 2', true],
   ['1 >1.2.3-pre.0', '>1.2.3-pre.0', true],

Original file line number	Diff line number	Diff line change
`@@ -174,7 +174,7 @@ const simpleSubset = (sub, dom, options) => {`
`174`	`174`	`if (higher === c && higher !== gt) {`
`175`	`175`	`return false`
`176`	`176`	`}`
`177`		`- } else if (gt.operator === '>=' && !satisfies(gt.semver, String(c), options)) {`
	`177`	`+ } else if (gt.operator === '>=' && !c.test(gt.semver)) {`
`178`	`178`	`return false`
`179`	`179`	`}`
`180`	`180`	`}`
`@@ -192,7 +192,7 @@ const simpleSubset = (sub, dom, options) => {`
`192`	`192`	`if (lower === c && lower !== lt) {`
`193`	`193`	`return false`
`194`	`194`	`}`
`195`		`- } else if (lt.operator === '<=' && !satisfies(lt.semver, String(c), options)) {`
	`195`	`+ } else if (lt.operator === '<=' && !c.test(lt.semver)) {`
`196`	`196`	`return false`
`197`	`197`	`}`
`198`	`198`	`}`