Skip to content

build: switch to oxfmt and oxlint - #10

Merged
TheGreatAxios merged 2 commits into
cl-9046-google-tools-test-layoutfrom
cl-9370-google-tools-switch-to-oxfmt-and-oxlint
Sep 27, 2026
Merged

TheGreatAxios merged 2 commits into
cl-9046-google-tools-test-layoutfrom
cl-9370-google-tools-switch-to-oxfmt-and-oxlint

Conversation

@TheGreatAxios

@TheGreatAxios TheGreatAxios commented Sep 26, 2026 •

Copy link
Copy Markdown
Contributor

Closes CL-9370

Summary

  • Add oxfmt and oxlint as exact-pinned devDependencies (the repo had no formatter or linter)
  • Run lint and format:check in CI after install
  • Remove two unused type imports in src/client/client.ts (no-unused-vars)
  • Replace new Array<T>(n) with Array.from<T>({ length: n }) in src/tools/create-tools.ts (unicorn/no-new-array)
  • .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

Stacked on #8.

Commits

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

@TheGreatAxios
TheGreatAxios force-pushed the cl-9370-google-tools-switch-to-oxfmt-and-oxlint branch from fce69fe to 1fabca9 Compare September 26, 2026 02:56
@TheGreatAxios
TheGreatAxios changed the base branch from main to cl-9046-google-tools-test-layout September 26, 2026 02:56
@TheGreatAxios
TheGreatAxios added this pull request to stack #9 September 26, 2026 03:00
@TheGreatAxios
TheGreatAxios merged commit c4e5eda 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