Skip to content

chore: migrate linter to @antfu/eslint-config - #6

Merged
dinwwwh merged 1 commit into
mainfrom
claude/migrate-antifu-eslint-f9c9e6
Sep 6, 2026
Merged

chore: migrate linter to @antfu/eslint-config#6
dinwwwh merged 1 commit into
mainfrom
claude/migrate-antifu-eslint-f9c9e6

Conversation

@dinwwwh

@dinwwwh dinwwwh commented Sep 6, 2026

Copy link
Copy Markdown
Member

Replaces oxlint, oxfmt, and ultracite with the same ESLint setup middleapi/orpc uses: @antfu/eslint-config with formatters, eslint-driven lint-staged and CI, and matching Zed and Claude hook settings. The whole codebase is reformatted to that style, and top-level arrow-function consts are now function declarations.

Config

  • lint / lint:fix run eslint --max-warnings=0; CI, lint-staged, and the sponsors sync use them.
  • Dependabot groups eslint, @antfu/eslint-config, and eslint-plugin-* like orpc.
  • ts/method-signature-style stays off so the published types keep their declared shapes; orpc's eslint-plugin-ban rules are left out since they target orpc internals.

Source

  • All oxlint-disable directives are gone: the anti-slop rules have no ESLint counterpart, and hoisted function declarations remove the need for every no-use-before-define suppression but one.
  • Generated fixtures under packages/types/tests are reformatted too; their generator is not in this repo.
  • The import sorter reordered the usage example in the types README, so its comments were reworded to match.

Testing

  • pnpm run lint, pnpm run type:check, and pnpm test (327 tests) pass.
  • No behavior change intended. Sherif's two findings (root package not private, unused apps/* workspace glob) predate this PR.

Replaces oxlint, oxfmt, and ultracite with the ESLint setup used by
middleapi/orpc: @antfu/eslint-config with formatters, eslint-driven
lint-staged and CI, and matching Zed and Claude hook settings.
Top-level arrow-function consts become function declarations, and the
oxlint-only disable directives are dropped.
@dinwwwh
dinwwwh merged commit 8f91134 into main Sep 6, 2026
5 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