^0 != @0.0.0

Author: mxcl

.github/deno-to-node.ts

@@ -5,6 +5,8 @@ import SemVer from "../src/utils/semver.ts";
 
 await emptyDir("./dist");
 
+const test = !Deno.args.includes("--no-test");
+
 const version = (() => {
   try {
     return new SemVer(Deno.args[0]).toString();
@@ -17,6 +19,7 @@ const version = (() => {
 await build({
   entryPoints: ["./mod.ts"],
   outDir: "./dist",
+  test,
   shims: {
     deno: true,
     crypto: true,

.github/workflows/cd.npm.yml

@@ -6,7 +6,7 @@ on:
       - published
 
 concurrency:
-  group: cd/vx/${{ github.event.release.tag_name }}
+  group: cd/npm/${{ github.event.release.tag_name }}
   cancel-in-progress: true
 
 permissions:

.github/workflows/cd.yml

@@ -35,7 +35,7 @@ jobs:
       - uses: actions/checkout@v4
       - uses: pkgxdev/dev@v0
 
-      - run: deno task dnt ${{ inputs.version }}
+      - run: deno task dnt ${{ inputs.version }} --no-test
 
       - run: |
           mv dist libpkgx-${{inputs.version}}
@@ -45,9 +45,8 @@ jobs:
         run:
           gh release upload
             --clobber
+            v${{ inputs.version }}
             libpkgx-${{ inputs.version }}.tar.xz
         env:
-          # using this token rather than github.token due to `release not found` bug
-          # https://github.com/cli/cli/issues/5252
-          GH_TOKEN: ${{ secrets.TEMP_JACOBS_GITHUB_PAT }}
+          GH_TOKEN: ${{ github.token }}
           GH_REPO: pkgxdev/libpkgx

src/utils/semver.test.ts

@@ -237,6 +237,20 @@ Deno.test("semver", async test => {
       assertFalse(a.satisfies(new SemVer("1.0.0")))
     })
 
+    await test.step("^0 string is not @0.0.0", () => {
+      const a = new semver.Range("^0")
+      assertEquals(a.toString(), "^0")
+
+      const b = new semver.Range("^0.0")
+      assertEquals(b.toString(), "~0") //NOTE strictly should be ~0.0 but this is fine
+
+      const c = new semver.Range("^1")
+      assertEquals(c.toString(), "^1")
+
+      const d = new semver.Range("^1.0")
+      assertEquals(d.toString(), "^1")
+    })
+
     //FIXME this *should* work
     // await test.step("^11,^12…^11.3,^12.2", () => {
     //   const a = new semver.Range("^11,^12")

src/utils/semver.ts

@@ -205,9 +205,17 @@ export class Range {
       return this.set.map(v => {
         if (!isArray(v)) return `=${v.toString()}`
         const [v1, v2] = v
-        if (v1.major > 0 && v2.major == v1.major + 1 && v2.minor == 0 && v2.patch == 0) {
+        if (v2.major == v1.major + 1 && v2.minor == 0 && v2.patch == 0) {
           const v = chomp(v1)
-          return `^${v}`
+          if (v1.major == 0) {
+            if (v1.components.length == 1) {
+              return `^0`
+            } else {
+              return `>=${v}<1`
+            }
+          } else {
+            return `^${v}`
+          }
         } else if (v2.major == v1.major && v2.minor == v1.minor + 1 && v2.patch == 0) {
           const v = chomp(v1)
           return `~${v}`