fix: add curly braces for oxlint, update moduleResolution to bundler

- Add braces to all single-line if/for-of statements (eslint/curly: all)
- Change moduleResolution from node to bundler in tsconfig.json and
  tsconfig.dts.json (node10 removed in tsgo)

Author: jdalton

.config/esbuild.config.mjs

@@ -51,7 +51,9 @@ function buildPathMap(modulePaths) {
 
   for (const longPath of modulePaths) {
     const info = extractPackageInfo(longPath)
-    if (!info) continue
+    if (!info) {
+      continue
+    }
 
     const shortPath = `${info.packageName}/${info.subpath}`
     if (!shortPathGroups.has(shortPath)) {
@@ -123,7 +125,9 @@ function createPathShorteningPlugin() {
               }
 
               function collectPaths(node) {
-                if (!node || typeof node !== 'object') return
+                if (!node || typeof node !== 'object') {
+                  return
+                }
 
                 if (
                   node.type === 'StringLiteral' &&
@@ -133,11 +137,14 @@ function createPathShorteningPlugin() {
                 }
 
                 for (const key of Object.keys(node)) {
-                  if (key === 'start' || key === 'end' || key === 'loc')
+                  if (key === 'start' || key === 'end' || key === 'loc') {
                     continue
+                  }
                   const value = node[key]
                   if (Array.isArray(value)) {
-                    for (const item of value) collectPaths(item)
+                    for (const item of value) {
+                      collectPaths(item)
+                    }
                   } else {
                     collectPaths(value)
                   }
@@ -169,7 +176,9 @@ function createPathShorteningPlugin() {
               }
 
               function applyReplacements(node) {
-                if (!node || typeof node !== 'object') return
+                if (!node || typeof node !== 'object') {
+                  return
+                }
 
                 if (
                   node.type === 'StringLiteral' &&
@@ -187,11 +196,14 @@ function createPathShorteningPlugin() {
                 }
 
                 for (const key of Object.keys(node)) {
-                  if (key === 'start' || key === 'end' || key === 'loc')
+                  if (key === 'start' || key === 'end' || key === 'loc') {
                     continue
+                  }
                   const value = node[key]
                   if (Array.isArray(value)) {
-                    for (const item of value) applyReplacements(item)
+                    for (const item of value) {
+                      applyReplacements(item)
+                    }
                   } else {
                     applyReplacements(value)
                   }

.config/tsconfig.dts.json

@@ -5,9 +5,10 @@
     "declarationMap": false,
     "emitDeclarationOnly": true,
     "module": "commonjs",
-    "moduleResolution": "node",
+    "moduleResolution": "bundler",
     "outDir": "../dist",
     "rootDir": "../src",
+    "types": ["node"],
     "verbatimModuleSyntax": false
   },
   "include": ["../src/**/*.ts"]

src/vers.ts

@@ -123,29 +123,43 @@ function parseSemver(version: string): SemverParts {
  */
 function comparePrereleases(a: string[], b: string[]): number {
   // No prerelease has higher precedence than any prerelease
-  if (a.length === 0 && b.length === 0) return 0
-  if (a.length === 0) return 1
-  if (b.length === 0) return -1
+  if (a.length === 0 && b.length === 0) {
+    return 0
+  }
+  if (a.length === 0) {
+    return 1
+  }
+  if (b.length === 0) {
+    return -1
+  }
 
   const len = Math.min(a.length, b.length)
   for (let i = 0; i < len; i += 1) {
     const ai = a[i]!
     const bi = b[i]!
-    if (ai === bi) continue
+    if (ai === bi) {
+      continue
+    }
     const aNum = RegExpPrototypeTest(DIGITS_ONLY, ai)
     const bNum = RegExpPrototypeTest(DIGITS_ONLY, bi)
     // Numeric identifiers always have lower precedence than alphanumeric
     if (aNum && bNum) {
       const diff = Number(ai) - Number(bi)
-      if (diff !== 0) return diff < 0 ? -1 : 1
+      if (diff !== 0) {
+        return diff < 0 ? -1 : 1
+      }
     } else if (aNum) {
       return -1
     } else if (bNum) {
       return 1
     } else {
       // Alphanumeric: lexicographic comparison
-      if (ai < bi) return -1
-      if (ai > bi) return 1
+      if (ai < bi) {
+        return -1
+      }
+      if (ai > bi) {
+        return 1
+      }
     }
   }
   // Larger set of pre-release fields has higher precedence
@@ -164,12 +178,20 @@ function compareSemver(a: string, b: string): -1 | 0 | 1 {
   const pa = parseSemver(a)
   const pb = parseSemver(b)
   // Compare major.minor.patch
-  if (pa.major !== pb.major) return pa.major < pb.major ? -1 : 1
-  if (pa.minor !== pb.minor) return pa.minor < pb.minor ? -1 : 1
-  if (pa.patch !== pb.patch) return pa.patch < pb.patch ? -1 : 1
+  if (pa.major !== pb.major) {
+    return pa.major < pb.major ? -1 : 1
+  }
+  if (pa.minor !== pb.minor) {
+    return pa.minor < pb.minor ? -1 : 1
+  }
+  if (pa.patch !== pb.patch) {
+    return pa.patch < pb.patch ? -1 : 1
+  }
   // Compare prerelease
   const pre = comparePrereleases(pa.prerelease, pb.prerelease)
-  if (pre !== 0) return pre < 0 ? -1 : 1
+  if (pre !== 0) {
+    return pre < 0 ? -1 : 1
+  }
   return 0
 }
 
@@ -396,10 +418,16 @@ class Vers {
         }
         // Version >= lower bound — check upper bound
         const next = ranges[i + 1]
-        if (!next) return true
+        if (!next) {
+          return true
+        }
         const cmpNext = compareSemver(version, next.version)
-        if (next.comparator === '<' && cmpNext < 0) return true
-        if (next.comparator === '<=' && cmpNext <= 0) return true
+        if (next.comparator === '<' && cmpNext < 0) {
+          return true
+        }
+        if (next.comparator === '<=' && cmpNext <= 0) {
+          return true
+        }
         // Outside this range's upper bound — advance past it and try next range
         i += 1
       } else if (c.comparator === '>') {
@@ -413,17 +441,27 @@ class Vers {
         }
         // Version > lower bound — check upper bound
         const next = ranges[i + 1]
-        if (!next) return true
+        if (!next) {
+          return true
+        }
         const cmpNext = compareSemver(version, next.version)
-        if (next.comparator === '<' && cmpNext < 0) return true
-        if (next.comparator === '<=' && cmpNext <= 0) return true
+        if (next.comparator === '<' && cmpNext < 0) {
+          return true
+        }
+        if (next.comparator === '<=' && cmpNext <= 0) {
+          return true
+        }
         // Outside this range's upper bound — advance past it and try next range
         i += 1
       } else if (c.comparator === '<' || c.comparator === '<=') {
         // Leading less-than without a preceding lower bound
         const cmpVal = compareSemver(version, c.version)
-        if (c.comparator === '<' && cmpVal < 0) return true
-        if (c.comparator === '<=' && cmpVal <= 0) return true
+        if (c.comparator === '<' && cmpVal < 0) {
+          return true
+        }
+        if (c.comparator === '<=' && cmpVal <= 0) {
+          return true
+        }
         // Not in this range — continue to next
       }
     }

tsconfig.json

@@ -2,9 +2,10 @@
   "extends": "./.config/tsconfig.base.json",
   "compilerOptions": {
     "module": "commonjs",
-    "moduleResolution": "node",
+    "moduleResolution": "bundler",
     "outDir": "dist",
     "rootDir": "src",
+    "types": ["node"],
     "verbatimModuleSyntax": false,
     "noEmit": false,
     "noEmitOnError": false,