Skip to content

chore(format): migrate from biome to oxfmt#243

Merged
chhoumann merged 2 commits into
masterfrom
chore/oxfmt-migration
Jul 6, 2026
Merged

chore(format): migrate from biome to oxfmt#243
chhoumann merged 2 commits into
masterfrom
chore/oxfmt-migration

Conversation

@chhoumann

@chhoumann chhoumann commented Jul 6, 2026

Copy link
Copy Markdown
Owner

Completes the vite-plus toolchain move started in #242: oxfmt replaces biome for the format gate (biome's lint half was already redundant next to oxlint). oxfmt was already vendored via obsidian-e2e's vite-plus pack; it's now a direct devDep.

  • Same file scope as before (configs, tests/e2e, scripts - src was never format-gated).
  • .oxfmtrc.json sets useTabs to match the repo; everything else is oxfmt defaults, which means a one-time reformat of the covered files: sortPackageJson puts package.json keys in conventional order, and wrapping gets denser prettier-style (net -443 lines).
  • New format script for write-mode alongside format:check.
  • format:check now runs in ~110ms.

Verified: format:check, oxlint, typecheck, build, 876/876 tests all green; package.json/manifest.json remain valid JSON for npm, semantic-release, and version-bump.mjs.


Open in Devin Review

oxfmt (already vendored via the vite-plus pack) replaces biome for the
format gate over the same file scope. Config: useTabs to match the
repo; otherwise oxfmt defaults, so this includes a one-time reformat
of the covered non-src files (sorted package.json keys, denser
prettier-style wrapping). Adds a 'format' write script alongside
format:check.
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jul 6, 2026

Copy link
Copy Markdown

Deploying podnotes with  Cloudflare Pages  Cloudflare Pages

Latest commit: 2866950
Status: ✅  Deploy successful!
Preview URL: https://d5b5b76c.podnotes.pages.dev
Branch Preview URL: https://chore-oxfmt-migration.podnotes.pages.dev

View logs

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Devin Review found 1 potential issue.

Open in Devin Review

Comment thread package.json
@chhoumann chhoumann merged commit 821480d into master Jul 6, 2026
3 checks passed
@chhoumann chhoumann deleted the chore/oxfmt-migration branch July 6, 2026 16:12
@github-actions

github-actions Bot commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 2.17.3 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant