Skip to content

chore: migrate from biome to oxfmt#1123

Open
B4nan wants to merge 2 commits intochore/migrate-to-oxlintfrom
chore/migrate-to-oxfmt
Open

chore: migrate from biome to oxfmt#1123
B4nan wants to merge 2 commits intochore/migrate-to-oxlintfrom
chore/migrate-to-oxfmt

Conversation

@B4nan
Copy link
Copy Markdown
Member

@B4nan B4nan commented May 5, 2026

Summary

Replaces Biome (formatter) with oxfmt, matching the toolchain in apify-storage-local-js, apify-shared-js, apify-client-js, apify-core, and crawlee. Stacked on #1122 (eslint→oxlint migration); will be retargeted to master once that lands.

Commits

  • chore: add oxfmt tooling — adds oxfmt@0.46.0 devDep, .oxfmtrc.json (printWidth 120, quoteProps preserve, the rest comes from .editorconfig), adds quote_type = single to .editorconfig so oxfmt picks up biome's previous quote style; drops @biomejs/biome and prettier (biome was the only consumer of prettier on this repo besides the markdown/yaml check, which we drop too — matches what other repos did when migrating). Swaps format / format:check and lint-staged to oxfmt.
  • chore: apply oxfmt formatting — pure oxfmt --write reformat across 39 files. Diff is mostly tab/quote/line-wrap normalization that biome and oxfmt disagreed on.

What changed (config side)

  • Drop @biomejs/biome, prettier
  • Add oxfmt@0.46.0
  • Replace biome.json with .oxfmtrc.json
  • Add quote_type = single to .editorconfig (oxfmt reads it; biome had it inline)
  • formatoxfmt --check, format:fixoxfmt
  • lint-staged runs oxfmt + oxlint --fix for code; markdown/yaml no longer auto-formatted (consistent with the other oxfmt'd repos)

Format output

$ pnpm format
oxfmt --check
All matched files use the correct format.
Finished in 18ms on 246 files using 14 threads.

Test plan

  • pnpm lint — clean
  • pnpm format — clean
  • pnpm build — TypeScript + tsdown both succeed

Follow-up

Once this lands, add the chore: apply oxfmt formatting commit SHA to a .git-blame-ignore-revs file so blame skips it.

@B4nan B4nan requested a review from vladfrangu as a code owner May 5, 2026 20:34
@B4nan B4nan added the adhoc Ad-hoc unplanned task added during the sprint. label May 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

adhoc Ad-hoc unplanned task added during the sprint.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants