ci(lint): add oxlint and fail CI on any warning (#828)

.coding-ci.yaml

@@ -19,7 +19,9 @@ stages:
         cmds:
         - node --version && npm --version
         - npm ci --ignore-scripts
-        - echo "=== Lint (typecheck) ==="
+        - echo "=== Lint ==="
+        - npm run lint
+        - echo "=== Typecheck ==="
         - npx tsc --noEmit
         - echo "=== Unit Tests + Coverage ==="
         - npx vitest run --coverage

.github/CONTRIBUTING.md

@@ -15,11 +15,14 @@ npm install
 ```bash
 npm run build          # Build with tsup → dist/
 npx tsc --noEmit       # Type check
+npm run lint           # oxlint; CI fails on any warning
 npx vitest run         # Run unit tests
 npx vitest run --coverage
 npm run test:e2e       # E2E tests (optional, requires a live test repo)
 ```
 
+`npm run lint` needs Node ^20.19 or >=22.12 (oxlint's requirement); the CLI itself still supports Node 20.
+
 ### Running your local build
 
 ```bash
@@ -75,7 +78,7 @@ See [docs/providers.md](../docs/providers.md) for how to add a new git provider.
 
 1. Fork the repo and create a feature branch from the latest `origin/main`. Prefer a git worktree for code changes when practical.
 2. Write tests for your change (we target 80%+ coverage).
-3. Run `npx vitest run` and `npx tsc --noEmit` — both must pass.
+3. Run `npx vitest run`, `npx tsc --noEmit` and `npm run lint` — all must pass.
 4. Use conventional commits where possible: `feat:`, `fix:`, `chore:`, `docs:`, `refactor:`, `test:`.
 5. Open a PR with a clear description: what's the problem, what's the fix, anything reviewers should pay attention to.
 

.github/PULL_REQUEST_TEMPLATE.md

@@ -19,6 +19,7 @@ Thanks for opening a PR! Please fill out this template to help reviewers underst
 <!-- How did you verify this works? Include commands you ran, new tests added, etc. -->
 
 - [ ] `npx tsc --noEmit` passes
+- [ ] `npm run lint` passes
 - [ ] `npx vitest run` passes
 - [ ] Added/updated tests for the change
 

.github/workflows/ci.yml

@@ -36,6 +36,9 @@ jobs:
       - name: Install dependencies
         run: npm ci --ignore-scripts
 
+      - name: Lint
+        run: npm run lint
+
       - name: Type check
         run: npx tsc --noEmit
 

.oxlintrc.json

@@ -0,0 +1,13 @@
+{
+  "$schema": "./node_modules/oxlint/configuration_schema.json",
+  "rules": {
+    "no-unused-vars": [
+      "warn",
+      {
+        "argsIgnorePattern": "^_",
+        "varsIgnorePattern": "^_",
+        "ignoreRestSiblings": true
+      }
+    ]
+  }
+}

AGENTS.md

@@ -2,7 +2,7 @@
 
 CLI for syncing team skills, rules, docs, and env across AI coding tools. Package: [`teamai-cli`](https://www.npmjs.com/package/teamai-cli).
 
-TypeScript, Node 20+, tsup (ESM), Vitest. Commands: `npm run build`, `npx tsc --noEmit`, `npx vitest run`, `npm run test:e2e`.
+TypeScript, Node 20+ (`npm run lint` needs ^20.19 or >=22.12), tsup (ESM), Vitest. Commands: `npm run build`, `npx tsc --noEmit`, `npm run lint`, `npx vitest run`, `npm run test:e2e`.
 
 ## Git
 

CLAUDE.md

@@ -2,7 +2,7 @@
 
 CLI for syncing team skills, rules, docs, and env across AI coding tools. Package: [`teamai-cli`](https://www.npmjs.com/package/teamai-cli).
 
-TypeScript, Node 20+, tsup (ESM), Vitest. Commands: `npm run build`, `npx tsc --noEmit`, `npx vitest run`, `npm run test:e2e`.
+TypeScript, Node 20+ (`npm run lint` needs ^20.19 or >=22.12), tsup (ESM), Vitest. Commands: `npm run build`, `npx tsc --noEmit`, `npm run lint`, `npx vitest run`, `npm run test:e2e`.
 
 ## Git