Switch lint and CI to oxfmt and oxlint

Author: TheGreatAxios

.github/workflows/ci.yml

@@ -11,11 +11,9 @@ concurrency:
   cancel-in-progress: ${{ github.event_name != 'push' }}
 
 jobs:
-  # Prettier, eslint, and typecheck share one runner: one checkout and one
-  # install instead of three of each. Prettier and eslint still run un-cached
-  # in CI: restored result caches can mark files clean against a stale tool
-  # version or config, masking real failures. The --cache flags in the
-  # package.json lint script remain for local speed.
+  # oxfmt, oxlint, and typecheck share one runner: one checkout and one
+  # install instead of three of each. Dummy job names prettier and eslint
+  # stay for protect-main.
   static-analysis:
     runs-on: ubuntu-latest
     steps:
@@ -39,11 +37,11 @@ jobs:
       - name: Install dependencies
         run: bun install --frozen-lockfile
 
-      - name: Prettier
-        run: bunx prettier --check .
+      - name: oxfmt
+        run: bunx oxfmt --check .
 
-      - name: ESLint
-        run: bunx eslint .
+      - name: oxlint
+        run: bunx oxlint .
 
       - name: Typecheck
         run: bun run typecheck

.prettierignore

@@ -34,8 +34,8 @@ When refactoring replaces an old path, delete the old one. No back-compat shims,
 - Bug fixes start with a failing test that reproduces the bug. Do not start by patching.
 - `tests/unit/` shared unit tests and helpers · co-located `src/**/*.test.ts` for module logic · `tests/fixtures/` fixture repos · `tests/integration/` reactor/permission harness. Planned: `tests/e2e/` (fixture-repo runs).
 - A test must not depend on another file having run, or on the default file order. It must pass under `bun test ./src ./tests ./evals ./scripts --randomize`. If a test mutates module-level state or calls `mock.module`, it must restore that state itself (`afterEach`/`afterAll`), not rely on the process happening to reset it. When capturing a module's real exports to restore later, shallow-copy them (`{ ...moduleNamespace }`) at capture time, whether the namespace came from `await import(path)` or a static `import * as ns from "path"` — Bun mutates the live namespace object in place when the module is mocked, so holding a bare reference to it (either form) silently turns into the mocked exports.
-- Never call `mock.module` directly. Bun runs every test file in one process, so a `mock.module` call without its own teardown stays installed for the rest of the run and silently replaces the real module for other files — producing failures in files the change never touched, with no obvious link to the cause and no signal from `tsc` or a per-file run (CL-6967). Use `withMockedModule`/`withMockedModuleDuring` from `tests/helpers/mock-module.ts`, which capture the real module and register their own restore. An eslint rule (`no-restricted-syntax` in `eslint.config.js`) rejects bare `mock.module` calls in `*.test.ts` files.
-- A test earns its place only if a real behavior change can fail it. Document copy, brand colors, marketing assets, and splash text are not behavior: assertions that pin an asset's literal wording, an exact palette hex/ANSI value, or rendered copy fail on copy/design edits and catch no regressions — assert the contract instead (parsing, formatting, ranges, aliases, invariants). Tests are code too: pinning a source file's own text is the same trap. This bar is a review and authorship rule, not an eslint shape match.
+- Never call `mock.module` directly. Bun runs every test file in one process, so a `mock.module` call without its own teardown stays installed for the rest of the run and silently replaces the real module for other files — producing failures in files the change never touched, with no obvious link to the cause and no signal from `tsc` or a per-file run (CL-6967). Use `withMockedModule`/`withMockedModuleDuring` from `tests/helpers/mock-module.ts`, which capture the real module and register their own restore. The oxlint plugin (`corbits/no-bare-mock-module` in `.oxlintrc.json` / `scripts/oxlint-plugin-corbits.js`) rejects bare `mock.module` calls in `*.test.ts` files.
+- A test earns its place only if a real behavior change can fail it. Document copy, brand colors, marketing assets, and splash text are not behavior: assertions that pin an asset's literal wording, an exact palette hex/ANSI value, or rendered copy fail on copy/design edits and catch no regressions — assert the contract instead (parsing, formatting, ranges, aliases, invariants). Tests are code too: pinning a source file's own text is the same trap. This bar is a review and authorship rule, not a linter shape match.
 
 ## Build & Validation
 

.prettierrc.json

@@ -59,6 +59,10 @@ parallel copies under `docs/` or `scripts/notes/`. At cut time: rename
 
 ### Changed
 
+- oxfmt formats the tree and oxlint is the lint gate. Prettier and ESLint are
+  gone. Dummy CI job names `prettier` and `eslint` stay for protect-main.
+  Empty functions and non-null assertions are errors.
+
 - Skywalker may spawn one successor with a changed brief after a failed or
   incomplete-report fleet worker. A parent-initiated interrupt
   (`stop_reason: interrupted`) is a resumable pause — `resume_agent` or