Merge pull request #15268 from nikmilson/fix-semver-ranges-parsing

sokra · web-flow · commit 4bc61504b2f2 · 2022-02-01T13:18:39.000+01:00
accept whitespaces after semver range operators
diff --git a/lib/util/semver.js b/lib/util/semver.js
@@ -89,7 +89,9 @@ exports.versionLt = versionLt;
  */
 exports.parseRange = str => {
 	const splitAndConvert = str => {
-		return str.split(".").map(item => (`${+item}` === item ? +item : item));
+		return str
+			.split(".")
+			.map(item => (item !== "NaN" && `${+item}` === item ? +item : item));
 	};
 	// see https://docs.npmjs.com/misc/semver#range-grammar for grammar
 	const parsePartial = str => {
@@ -131,13 +133,15 @@ exports.parseRange = str => {
 		return [-range[0] - 1, ...range.slice(1)];
 	};
 	const parseSimple = str => {
-		// simple     ::= primitive | partial | tilde | caret
-		// primitive  ::= ( '<' | '>' | '>=' | '<=' | '=' ) partial
-		// tilde      ::= '~' partial
-		// caret      ::= '^' partial
+		// simple       ::= primitive | partial | tilde | caret
+		// primitive    ::= ( '<' | '>' | '>=' | '<=' | '=' | '!' ) ( ' ' ) * partial
+		// tilde        ::= '~' ( ' ' ) * partial
+		// caret        ::= '^' ( ' ' ) * partial
 		const match = /^(\^|~|<=|<|>=|>|=|v|!)/.exec(str);
 		const start = match ? match[0] : "";
-		const remainder = parsePartial(str.slice(start.length));
+		const remainder = parsePartial(
+			start.length ? str.slice(start.length).trim() : str.trim()
+		);
 		switch (start) {
 			case "^":
 				if (remainder.length > 1 && remainder[1] === 0) {
@@ -191,11 +195,14 @@ exports.parseRange = str => {
 		return [, ...arr, ...items.slice(1).map(() => fn)];
 	};
 	const parseRange = str => {
-		// range      ::= hyphen | simple ( ' ' simple ) * | ''
-		// hyphen     ::= partial ' - ' partial
-		const items = str.split(" - ");
+		// range      ::= hyphen | simple ( ' ' ( ' ' ) * simple ) * | ''
+		// hyphen     ::= partial ( ' ' ) * ' - ' ( ' ' ) * partial
+		const items = str.split(/\s+-\s+/);
 		if (items.length === 1) {
-			const items = str.trim().split(/\s+/g).map(parseSimple);
+			const items = str
+				.trim()
+				.split(/(?<=[-0-9A-Za-z])\s+/g)
+				.map(parseSimple);
 			return combine(items, 2);
 		}
 		const a = parsePartial(items[0]);
diff --git a/test/SemVer.unittest.js b/test/SemVer.unittest.js
@@ -129,12 +129,27 @@ describe("SemVer", () => {
 
 	describe("parseRange", () => {
 		const cases = {
-			"=3": ["3", "v3", "3.x", "3.X", "3.x.x", "3.*", "3.*.*", "^3", "^3.x"],
-			"=3.0": ["3.0", "v3.0", "3.0.x", "3.0.X", "3.0.*", "~3.0"],
-			"^3.4": ["^3.4.*"],
-			"3.4 - 6.5": [">=3.4 <=6.5"],
-			"<=3.4": ["<3.4 || =3.4"],
-			">3.4": [">=3.4 !3.4"]
+			"5 || 6 || 7.x.x": ["5.x.x || 6.x || 7"],
+			"1 - 2": ["1   -   2"],
+			"=3": [
+				"3",
+				"v3",
+				"3.x",
+				"3.X",
+				"3.x.x",
+				"3.*",
+				"3.*.*",
+				"^3",
+				"^3.x",
+				"= 3"
+			],
+			"=3.0": ["3.0", "v3.0", "3.0.x", "3.0.X", "3.0.*", "~3.0", "= 3.0"],
+			"^3.4": ["^3.4.*", "^ 3.4"],
+			"3.4 - 6.5": [">=3.4 <=6.5", ">= 3.4 <= 6.5"],
+			"<=3.4": ["<3.4 || =3.4", "<= 3.4"],
+			">3.4": [">=3.4 !3.4", "> 3.4"],
+			"1.2.3-alpha.x.x": ["1.2.3-alpha", "1.2.3-alpha+build.25"],
+			"1.2.3-NaN": ["1.2.3-NaN"]
 		};
 		for (const key of Object.keys(cases)) {
 			describe(key, () => {
