Move lint to a custom script to avoid hardcoding dependency versions

Author: wojtekmaj

.github/workflows/ci.yml

@@ -30,8 +30,11 @@ jobs:
         with:
           node-version: lts/*
 
+      - uses: pnpm/action-setup@1e1c8eafbd745f64b1ef30a7d7ed7965034c486c
+        name: Install pnpm
+
       - name: 🔠 Lint project
-        run: node --run lint:ci
+        run: node scripts/lint.ts
 
   test:
     runs-on: ubuntu-latest

package.json

@@ -19,7 +19,7 @@
     "knip": "knip",
     "knip:fix": "knip --fix",
     "lint": "vite lint && vite fmt --check",
-    "lint:ci": "npx oxlint@1.42.0 & npx oxfmt@0.27.0 --check",
+    "lint:ci": "node scripts/lint.ts",
     "lint:fix": "vite lint --fix && vite fmt",
     "generate": "nuxt generate",
     "npmx-connector": "pnpm --filter npmx-connector dev",

scripts/lint.ts

@@ -0,0 +1,45 @@
+/**
+ * This script runs oxlint and oxfmt in a CI environment, without the need to install the entire
+ * project. It reads the required versions from vite-plus entry in pnpm-lock.yaml and executes
+ * the linters accordingly. It's "stupid by design" so it could work in minimal Node.js environments.
+ */
+
+import { spawnSync } from 'node:child_process'
+import { readFileSync } from 'node:fs'
+
+const lockfileContent = readFileSync('pnpm-lock.yaml', 'utf8')
+const lines = lockfileContent.split(/\n/)
+
+const startIndex = lines.findIndex(line => line.includes('vite-plus@'))
+
+if (startIndex === -1) {
+  throw new Error('Could not find "vite-plus@" in pnpm-lock.yaml.')
+}
+
+function getDependencyVersion(dependencyName: string): string {
+  const versionRegex = new RegExp(`^\\s+${dependencyName}: ([^(\\s]+)`)
+
+  for (let index = startIndex + 1; index < lines.length; index += 1) {
+    const match = lines[index].match(versionRegex)
+
+    if (match) {
+      return match[1]
+    }
+  }
+
+  throw new Error(`Could not resolve ${dependencyName} version from pnpm-lock.yaml.`)
+}
+
+const oxlintVersion = getDependencyVersion('oxlint')
+const oxfmtVersion = getDependencyVersion('oxfmt')
+
+const runCommand = (command: string, args: string[]) => {
+  const result = spawnSync(command, args, { stdio: 'inherit' })
+
+  if (result.status) {
+    process.exit(result.status)
+  }
+}
+
+runCommand('pnpx', [`oxlint@${oxlintVersion}`])
+runCommand('pnpx', [`oxfmt@${oxfmtVersion}`, '--check'])