Skip to content

build: switch to oxfmt and oxlint - #63

Merged
TheGreatAxios merged 2 commits into
cl-9364-memory-build-with-bun-in-prepackfrom
cl-9377-memory-switch-to-oxfmt-and-oxlint
Sep 27, 2026
Merged

TheGreatAxios merged 2 commits into
cl-9364-memory-build-with-bun-in-prepackfrom
cl-9377-memory-switch-to-oxfmt-and-oxlint

Conversation

@TheGreatAxios

@TheGreatAxios TheGreatAxios commented Sep 26, 2026 •

Copy link
Copy Markdown
Contributor

Closes CL-9377

Summary

  • Stacked on build: run prepack with bun #62, 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 CI after install
  • Remove unused imports and a dead grant() test helper (no-unused-vars): sql in src/services/timeline.ts, authorize in src/memory.ts, RerankClientConfig in src/services/search.ts, mock in src/core/degrade-metrics.test.ts, createInMemoryGrantStore/GrantRule/grant() in src/routes/deps.test.ts
  • Replace new Array(n).fill(x) with Array.from({ length: n }, () => x) in src/core/degrade-metrics.ts and three test files (unicorn/no-new-array)
  • Two inline oxlint-disable-next-line unicorn/no-thenable comments in src/services/transform.test.ts and src/services/search.test.ts: those fakes stand in for drizzle's query builder, which is thenable on purpose
  • .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 (75 files changed, 1935 insertions(+), 1283 deletions(-)), safe to skip in review

@TheGreatAxios
TheGreatAxios force-pushed the cl-9377-memory-switch-to-oxfmt-and-oxlint branch from 14298e8 to 8a4ee65 Compare September 26, 2026 03:19
@TheGreatAxios
TheGreatAxios changed the base branch from release-0.2.0 to cl-9364-memory-build-with-bun-in-prepack September 26, 2026 03:19
@TheGreatAxios
TheGreatAxios added this pull request to stack #61 September 26, 2026 03:19
@TheGreatAxios
TheGreatAxios force-pushed the cl-9377-memory-switch-to-oxfmt-and-oxlint branch from 8a4ee65 to 9df3fc7 Compare September 27, 2026 00:32
@TheGreatAxios
TheGreatAxios merged commit 49be195 into main Sep 27, 2026
2 of 4 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