Skip to content

Migrate to Oxlint and Oxfmt.#888

Closed
cpojer wants to merge 2 commits intoopenclaw:mainfrom
cpojer:oxc-migrate
Closed

Migrate to Oxlint and Oxfmt.#888
cpojer wants to merge 2 commits intoopenclaw:mainfrom
cpojer:oxc-migrate

Conversation

@cpojer
Copy link
Copy Markdown
Member

@cpojer cpojer commented Jan 14, 2026

This PR migrates to Oxfmt and Oxlint.

Prompt: Migrate from Biome to Oxlint and Oxfmt. At the end, all references to Biome should be removed and replaced with the corresponding config and mentions of Oxlint and Oxfmt. Make sure to use the latest version of Oxlint and Oxfmt.

I did a few manual changes like locking the printWidth to 80 like it was with Biome. The first commit is the tooling change, the second commit is a run of pnpm format.

@steipete
Copy link
Copy Markdown
Contributor

is oxfmt >> biome now?

  • Medium: pnpm lint:fix no longer formats. Previously ran Biome format;
    now only oxlint --fix, so lint:fix leaves oxfmt --check failures.
    Suggest chain pnpm format:fix or rename. package.json:93
  • Low: Biome ignore directives remain; oxlint won’t honor them, so
    suppressions may be lost/misleading. Convert to oxlint disables or
    remove if rules gone. src/agents/pi-tools.types.ts:3, src/channels/
    plugins/types.plugin.ts:30
  • Low: CI matrix now runs pnpm lint twice (plain + after oxfmt
    --check); duplicate signal/job names. Consider rename second task to
    format or drop plain lint. .github/workflows/ci.yml:65, .github/
    workflows/ci.yml:78

(no worries, taking over)

@steipete steipete self-assigned this Jan 14, 2026
@steipete
Copy link
Copy Markdown
Contributor

Migrated on main instead. Oxfmt/Oxlint config + scripts/CI updated, Biome removed, repo reformatted.

Gate: pnpm lint/build/test/e2e OK. test:live hit Anthropic 429 rate limit; test:docker:all not run.

@steipete steipete closed this Jan 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants