Skip to content

chore(lint): adopt oxc-config-mantine for the oxlint ruleset - #43

Merged
gfazioli merged 1 commit into
masterfrom
chore/oxc-config-mantine
Jul 27, 2026
Merged

chore(lint): adopt oxc-config-mantine for the oxlint ruleset#43
gfazioli merged 1 commit into
masterfrom
chore/oxc-config-mantine

Conversation

@gfazioli

@gfazioli gfazioli commented Jul 27, 2026

Copy link
Copy Markdown
Owner

Summary

Mantine 9.5 migrated its toolchain to oxc and published the shared config as oxc-config-mantine. This adopts the linter half of it: the hand-maintained oxlint.config.ts is replaced by a short oxlint.config.mjs that spreads the shared ruleset.

The formatter half is deliberately not adopted: that config orders imports via an importOrder key which oxfmt does not support (the key appears nowhere in oxfmt's dist; neither 0.55 nor 0.60 reacts to it, and upstream's own files show the declared order is not applied). Adopting it would silently drop the import sorting we get from sortImports.groups. .oxfmtrc.json is unchanged.

Two overrides on top of the shared ruleset:

  • curly: ['error', 'multi-line'] — upstream's curly: 'error' requires braces everywhere and conflicts with oxfmt collapsing single-statement bodies onto the condition line.
  • ignorePatterns adds docs/.next, which our docs sites have and upstream's list lacks.

Config-only change: no code files touched, no release.

Test plan

  • yarn test (syncpack + format + typecheck + lint + jest)
  • yarn build

Summary by CodeRabbit

  • Chores
    • Updated linting to use the project’s explicit configuration.
    • Improved formatting consistency by requiring braces for multi-line statements.
    • Excluded generated documentation output from lint checks.
    • Added configuration support for the project’s linting setup.

@gfazioli
gfazioli merged commit c8f0981 into master Jul 27, 2026
1 of 2 checks passed
@gfazioli
gfazioli deleted the chore/oxc-config-mantine branch July 27, 2026 17:07
@coderabbitai

coderabbitai Bot commented Jul 27, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: c58c7d81-8df4-4d43-9abf-575eae0f33db

📥 Commits

Reviewing files that changed from the base of the PR and between 1c1e2a4 and 8d2db18.

⛔ Files ignored due to path filters (1)
  • yarn.lock is excluded by !**/yarn.lock, !**/*.lock
📒 Files selected for processing (3)
  • oxlint.config.mjs
  • oxlint.config.ts
  • package.json

📝 Walkthrough

Walkthrough

Oxlint now uses oxlint.config.mjs explicitly, with formatter-aligned curly enforcement and an additional docs/.next ignore pattern. The previous TypeScript configuration was removed, and oxc-config-mantine was added as a development dependency.

Changes

Oxlint configuration migration

Layer / File(s) Summary
Explicit Oxlint configuration
oxlint.config.mjs, package.json, oxlint.config.ts
The npm script selects oxlint.config.mjs; the configuration updates curly and ignored paths, while the previous TypeScript configuration is removed and oxc-config-mantine is added.

Estimated code review effort: 2 (Simple) | ~10 minutes

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/oxc-config-mantine

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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