Fixes pkgxdev/pkgx#650

Author: mxcl

src/utils/hacks.ts

@@ -3,6 +3,7 @@ import { is_what } from "../deps.ts"
 const { isString, isNumber } = is_what
 import * as semver from "./semver.ts"
 import host from "./host.ts"
+import { TeaError } from "../../mod.ts"
 
 export function validatePackageRequirement(project: string, constraint: unknown): PackageRequirement | undefined
 {
@@ -25,8 +26,13 @@ export function validatePackageRequirement(project: string, constraint: unknown)
     throw new Error(`invalid constraint for ${project}: ${constraint}`)
   }
 
+  constraint = semver.Range.parse(constraint as string)
+  if (!constraint) {
+    throw new TeaError("invalid constraint for " + project + ": " + constraint)
+  }
+
   return {
     project,
-    constraint: new semver.Range(constraint as string)
+    constraint: constraint as semver.Range
   }
 }

src/utils/semver.test.ts

@@ -264,9 +264,9 @@ Deno.test("coverage", () => {
 
   assertThrows(() => new semver.Range("1"))
 
-  assertEquals(semver.Range.parse("1"), new semver.Range("^1"))
-  assertEquals(semver.Range.parse("1.1"), new semver.Range("^1.1"))
-  assertEquals(semver.Range.parse("1.1.2"), new semver.Range("~1.1.2"))
+  assertEquals(semver.Range.parse("1")?.toString(), new semver.Range("^1").toString())
+  assertEquals(semver.Range.parse("1.1")?.toString(), new semver.Range("~1.1").toString())
+  assertEquals(semver.Range.parse("1.1.2")?.toString(), new semver.Range(">=1.1.2<1.1.3").toString())
 
   assertEquals(semver.Range.parse("a"), undefined)
 

src/utils/semver.ts

@@ -247,12 +247,7 @@ export class Range {
       // AFAICT this is what people expect
       // verified via https://jubianchi.github.io/semver-check/
 
-      const parts = input.split('.')
-      if (parts.length < 3) {
-        return new Range(`^${input}`)
-      } else {
-        return new Range(`~${input}`)
-      }
+      return new Range(`@${input}`)
     }
   }