Skip to content

tooling: oxfmt + stricter oxlint - #54

Merged
Octember merged 2 commits into
mainfrom
chore/no-mutable-vars
Aug 24, 2026
Merged

tooling: oxfmt + stricter oxlint#54
Octember merged 2 commits into
mainfrom
chore/no-mutable-vars

Conversation

@Octember

@Octember Octember commented Aug 24, 2026

Copy link
Copy Markdown
Owner

Summary

  • Enable oxfmt (bun run fmt / fmt:check) and CI Format
  • Stricter oxlint: style + restriction on, noisy rules off
  • JSON.parse only in parseJson(); callers pass a zod schema

Test plan

  • bun run typecheck && bun run lint && bun run fmt:check && bun test

Add oxfmt (single quotes, no semis, 120) with bun run fmt / fmt:check
and a CI Format step. Ignore packs and .review/CORPUS.md so taste text
is not rewritten. JSON.parse now lives only in parseJson(); callers
pass a schema. Sweep status and GraphQL reads use the same path.

Co-authored-by: Cursor <cursoragent@cursor.com>

@exe-dev-github-integration exe-dev-github-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM ✅

Enable the next oxlint categories with the noisy rules off (magic numbers,
no-ternary, one-var, no-sync, …). Auto-fixable findings (curly, replaceAll,
length checks) go through oxlint --fix; oxfmt sorts imports. Semantic leftovers
that neither tool can rewrite stay off.

Co-authored-by: Cursor <cursoragent@cursor.com>
@Octember Octember changed the title tooling: enable oxfmt tooling: oxfmt + stricter oxlint Aug 24, 2026
@Octember
Octember merged commit 7035d0b into main Aug 24, 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