chore(tooling): adopt Oxlint and Oxfmt

.pre-commit-config.yaml

@@ -20,8 +20,8 @@
 # Priority groups (prek runs same-priority hooks in parallel):
 #   0  — General file fixers (whitespace, EOF, line endings)
 #   4  — SPDX header insertion (--fix)
-#   5  — Shell / JS / TS formatters (shfmt, Biome)
-#   6  — Fixes that should follow formatters (ruff check --fix, Biome lint --write)
+#   5  — Shell / JS / TS formatters (shfmt, Oxfmt)
+#   6  — Oxlint fixes after formatting
 #  10  — Linters and read-only checks
 #  20  — Project-level checks (vitest, coverage, ratchet)
 
@@ -111,22 +111,22 @@ repos:
 
   - repo: local
     hooks:
-      - id: biome-format
-        name: Biome format
-        entry: npx biome format --write --no-errors-on-unmatched
+      - id: oxfmt
+        name: Oxfmt
+        entry: bash tools/lint/format-added-files.sh --write
         language: system
-        files: ^(biome\.json|package(-lock)?\.json|nemoclaw/package(-lock)?\.json|commitlint\.config\.js|bin/.*\.js|scripts/.*\.js|test/.*\.js|.*\.ts)$
+        files: \.(cjs|cts|js|jsx|mjs|mts|ts|tsx)$
         pass_filenames: true
         priority: 5
 
   # ── Priority 6: auto-fix after formatting ─────────────────────────────────
   - repo: local
     hooks:
-      - id: biome-lint-fix
-        name: Biome lint fixes
-        entry: npx biome lint --write
+      - id: oxlint-fix
+        name: Oxlint fixes
+        entry: npx oxlint --fix --no-error-on-unmatched-pattern
         language: system
-        files: ^(commitlint\.config\.js|bin/.*\.js|scripts/.*\.js|test/.*\.js|.*\.ts)$
+        files: \.(cjs|cts|js|jsx|mjs|mts|ts|tsx)$
         pass_filenames: true
         priority: 6
 
@@ -159,6 +159,14 @@ repos:
 
   - repo: local
     hooks:
+      - id: oxlint-type-aware
+        name: Oxlint type-aware rules
+        entry: npx oxlint --config oxlint.type-aware.config.ts --no-error-on-unmatched-pattern
+        language: system
+        files: ^nemoclaw/src/.*\.ts$
+        pass_filenames: true
+        priority: 10
+
       - id: validate-config-schemas
         name: Validate config files against JSON schemas
         entry: npx tsx scripts/validate-configs.mts

AGENTS.md

@@ -74,7 +74,7 @@ Package-specific guides:
 | Run the broad repo-wide pre-commit and coverage baseline | `npm run check` |
 | Type-check CLI | `npm run typecheck:cli` |
 | Type-check plugin and plugin tests | `npm --prefix nemoclaw run typecheck` |
-| Auto-format | `npm run format` |
+| Auto-format added JavaScript and TypeScript files that Oxfmt does not exclude | `npm run format` |
 | Build docs | `npm run docs` |
 | Serve docs locally | `npm run docs:live` |
 
@@ -153,13 +153,15 @@ For shell scripts use `#` comments. For Markdown use HTML comments.
 
 - `bin/` launcher and remaining `scripts/*.js`: **CommonJS** (`require`/`module.exports`), Node.js 22.19+
 - `test/`: **ESM** (`import`/`export`)
-- Biome config in `biome.json`
+- Oxlint uses `oxlint.config.ts`. The isolated `oxlint.type-aware.config.ts` configuration enforces `typescript/no-floating-promises` for plugin sources.
+
+- Use `eslint-plugin-sonarjs` only for the `oxlint.config.ts` cognitive-complexity rules documented in [`tools/lint/DEPENDENCY-REVIEW.md`](tools/lint/DEPENDENCY-REVIEW.md).
 - Keep function complexity low; existing complexity hotspots are tracked separately
 - Unused vars pattern: prefix with `_`
 
 ### TypeScript
 
-- Plugin code in `nemoclaw/src/` is linted and formatted by the root Biome config
+- Oxlint lints plugin code in `nemoclaw/src/`. Oxfmt formats added plugin files that it does not exclude.
 - CLI type-checking via `tsconfig.cli.json`
 - Plugin production and test type-checking via `npm --prefix nemoclaw run typecheck`, using
   `nemoclaw/tsconfig.json` and `nemoclaw/tsconfig.test.json`
@@ -271,7 +273,8 @@ If the command trace contains no reviewer-request write, report the event as an
 ### Gotchas
 
 - `npm install` at root triggers `prek install` which sets up git hooks. If hooks fail, check that `core.hooksPath` is unset: `git config --unset core.hooksPath`
-- The `nemoclaw/` subdirectory has its own `package.json` and `node_modules`, while sharing the root Biome config — it's a separate npm project
+- The `nemoclaw/` subdirectory has its own `package.json` and `node_modules`.
+  It is a separate npm project that shares the root Oxlint and Oxfmt configuration files.
 - SPDX headers are auto-inserted by pre-commit hooks; don't worry about adding them manually
 - Coverage thresholds are ratcheted in `ci/coverage-threshold-*.json` — new code should not decrease CLI or plugin coverage
 - The `.claude/skills` symlink points to `.agents/skills` — both paths resolve to the same content

CONTRIBUTING.md

@@ -206,7 +206,7 @@ These are the primary npm scripts for day-to-day development:
 | `npm run check` | Run the broad repo-wide pre-commit and full CLI/plugin coverage baseline |
 | `npm run check:diff` | Compatibility alias for `npm run validate:pr` |
 | `npm run checks` | Compatibility alias for `npm run checks:repository`; prints scope guidance before delegating |
-| `npm run format` | Auto-format Biome-supported source files |
+| `npm run format` | Auto-format added JavaScript and TypeScript files that Oxfmt does not exclude |
 | `npm run typecheck:cli` | Type-check the root TypeScript project using `tsconfig.cli.json` |
 | `npm --prefix nemoclaw run typecheck` | Type-check plugin production and test sources without emitting files |
 | `npm test` | Build package artifacts and run every non-live Vitest project for broad changes |
@@ -383,6 +383,11 @@ All git hooks are managed by [prek](https://prek.j178.dev/), a fast, single-bina
 | **commit-msg** | commitlint (Conventional Commits) |
 | **pre-push** | Path-scoped incremental CLI/plugin TypeScript checks and checked-JavaScript checks |
 
+The Oxfmt hook formats added JavaScript and TypeScript files that its configuration does not exclude.
+The hook reads its base ref from `NEMOCLAW_FORMAT_BASE_REF`, which defaults to `origin/main`.
+A file is added when the selected base commit has no file at that path.
+If Git cannot resolve the ref named by `NEMOCLAW_FORMAT_BASE_REF`, the Oxfmt hook exits with status 2 before invoking Oxfmt.
+
 For PR preparation, normal `pre-commit`, `commit-msg`, and `pre-push` hooks are valid verification when they pass and were not bypassed with `--no-verify`.
 If hooks were skipped, missing, failed, or uncertain, refresh the remote-tracking base with `git fetch origin main`, then run `npm run validate:pr` once to reproduce those checks for the current diff.
 

ci/test-file-size-budget.json

@@ -2,13 +2,13 @@
   "$comment": "SPDX-FileCopyrightText: Copyright (c) 2026 NVIDIA CORPORATION & AFFILIATES. All rights reserved.\nSPDX-License-Identifier: Apache-2.0",
   "defaultMaxLines": 1500,
   "legacyMaxLines": {
-    "nemoclaw/src/commands/migration-state.test.ts": 1301,
+    "nemoclaw/src/commands/migration-state.test.ts": 1300,
     "src/lib/inference/nim.test.ts": 2067,
     "src/lib/onboard/preflight.test.ts": 1875,
     "test/generate-openclaw-config.test.ts": 1915,
     "test/install-preflight.test.ts": 3310,
     "test/nemoclaw-start.test.ts": 4790,
-    "test/onboard-messaging.test.ts": 2036,
+    "test/onboard-messaging.test.ts": 2035,
     "test/onboard-selection.test.ts": 4178
   }
 }