Skip to content

chore: add Biome formatter (formatter only) + CI check - #145

Merged
aaronjmars merged 3 commits into
mainfrom
chore/add-biome-formatter
Sep 3, 2026
Merged

chore: add Biome formatter (formatter only) + CI check#145
aaronjmars merged 3 commits into
mainfrom
chore/add-biome-formatter

Conversation

@aaronjmars

Copy link
Copy Markdown
Collaborator

What

Adds the Biome formatter to minitor, formatter only. Linting stays with ESLint (eslint.config.mjs is untouched and still the source of truth for lint rules).

Changes

  • Add @biomejs/biome@^2.5.12 as a dev dependency.
  • Add biome.json: formatter enabled (2-space indent, line width 100), linter and assist disabled. JSON and CSS formatting are disabled to avoid churn on config files, drizzle snapshots, and Tailwind @apply directives.
  • Add format (biome format --write .) and format:check (biome ci .) npm scripts.
  • Add a Format GitHub Actions workflow that runs npm run format:check on push to main and on PRs.
  • Reformat existing JS/TS sources (221 files): 124 .ts, 96 .tsx, 1 .mjs.

Verification

  • npm run format:check exits 0.
  • npm run lint -- --max-warnings=0 still passes (Biome formatting does not conflict with the ESLint config).
  • npm test passes (292/292).

No JSON or CSS files were reformatted.

Adds @biomejs/biome, biome.json (formatter only; linting stays with ESLint),
format and format:check scripts, and a Format CI workflow. Reformats JS/TS
sources. JSON and CSS formatting are disabled to avoid churn.
The committed lockfile listed vitest node_modules/@esbuild/* platform
packages as extraneous (non-optional), so npm ci on the Linux runner hit
EBADPLATFORM on @esbuild/aix-ppc64. Marking them optional lets npm ci
skip non-matching platforms while staying in sync.
@aaronjmars
aaronjmars merged commit 0d3e604 into main Sep 3, 2026
3 checks passed
@aaronjmars
aaronjmars deleted the chore/add-biome-formatter branch September 3, 2026 23:47
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