Skip to content

chore(apps): add Biome formatter and format check for all apps - #1193

Merged
aaronjmars merged 4 commits into
mainfrom
chore/apps-formatter
Oct 7, 2026
Merged

aaronjmars merged 4 commits into
mainfrom
chore/apps-formatter

Conversation

@aaronjmars

Copy link
Copy Markdown
Collaborator

Adds a code formatter for the four apps (cli, dashboard, mcp-server, webhook). None of them had one.

What changes:

  • Root biome.json, formatter only. Linting stays with each app's eslint. Biome is pinned to 2.5.15 and run via npx, so the root stays dependency-free.
  • Per-app overrides keep the existing style: single quotes and no semicolons in cli and dashboard; double quotes and semicolons in mcp-server, webhook and the dashboard *.test.ts files. Width 120, 2-space indent.
  • npm run format / npm run format:check at the repo root.
  • New format - biome job in ci-apps (CI Gate picks it up), and the command added to the operator skill's ci.md (both copies).
  • The second commit is a one-time mechanical reformat (141 files, +7546/-3109). It is whitespace and punctuation only, and the existing typecheck, lint, test and build jobs re-verify it. No open PR touches apps/, so there are no conflicts.

Review tip: read the first commit; the second is just the output of npm run format.

One root biome.json covers apps/cli, dashboard, mcp-server and webhook.
Formatter only; linting stays with each app eslint. Per-app overrides keep
the existing quote and semicolon style. Biome pinned to 2.5.15.
Mechanical reformat only, no logic changes: npm run format.
bin/generate-skill-icons owns that file and ci-skills-json diffs it byte for byte.
@aaronjmars
aaronjmars merged commit fe7df9a into main Oct 7, 2026
8 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