Skip to content

build: switch to oxfmt and oxlint - #49

Merged
TheGreatAxios merged 2 commits into
cl-mailbox-e2e-test-layoutfrom
cl-9382-mailbox-switch-to-oxfmt-and-oxlint
Sep 27, 2026
Merged

TheGreatAxios merged 2 commits into
cl-mailbox-e2e-test-layoutfrom
cl-9382-mailbox-switch-to-oxfmt-and-oxlint

Conversation

@TheGreatAxios

@TheGreatAxios TheGreatAxios commented Sep 26, 2026 •

Copy link
Copy Markdown
Contributor

Closes CL-9382

Summary

  • Stacked on test: e2e suites in e2e/, units only for load-bearing logic #46, the tip of the open stack, so the reformat covers every file on it
  • Add oxfmt and oxlint as exact-pinned devDependencies (the repo had no formatter or linter)
  • Run lint and format:check in the test workflow after install
  • Remove an unused import in e2e/native-store.test.ts (no-unused-vars)
  • e2e/migrations.test.ts: / DESC$/.test(column) becomes column.endsWith(" DESC") (unicorn/prefer-string-starts-ends-with)
  • .oxfmtrc.json sets printWidth: 80 (Prettier's width); everything else is oxfmt's defaults, including sorting package.json keys
  • .oxlintrc.json turns the correctness category from warn to error; no rules are disabled
  • lint runs oxlint, format runs oxfmt, format:check runs oxfmt --check

Commits

  1. build: switch to oxfmt and oxlint: tooling, configs and lint fixes
  2. style: format with oxfmt: mechanical reformat only (26 files changed, 314 insertions(+), 205 deletions(-)), safe to skip in review

@TheGreatAxios
TheGreatAxios force-pushed the cl-9382-mailbox-switch-to-oxfmt-and-oxlint branch from 0c848d2 to b728d5e Compare September 26, 2026 03:10
@TheGreatAxios
TheGreatAxios changed the base branch from main to cl-mailbox-e2e-test-layout September 26, 2026 03:10
@TheGreatAxios
TheGreatAxios added this pull request to stack #50 September 26, 2026 03:10
@TheGreatAxios
TheGreatAxios force-pushed the cl-9382-mailbox-switch-to-oxfmt-and-oxlint branch from b728d5e to 5ff3afa Compare September 27, 2026 00:45
@TheGreatAxios
TheGreatAxios force-pushed the cl-9382-mailbox-switch-to-oxfmt-and-oxlint branch from 5ff3afa to 50f21c9 Compare September 27, 2026 01:23
@TheGreatAxios
TheGreatAxios merged commit 57c79f1 into main Sep 27, 2026
2 checks passed
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.

1 participant