Conversation
Owner
gfazioli
commented
Apr 3, 2026
gfazioli
commented
- Upgrade @mantine/core and @mantine/hooks to 9.0.0 across all workspaces
- Upgrade TypeScript to 6.0.2 and React to 19.2.4
- Replace ESLint with oxlint for faster linting (add oxlint.config.ts, remove eslint.config.mjs)
- Replace Prettier with oxfmt for formatting (add .oxfmtrc.json, remove .prettierrc.mjs)
- Upgrade Storybook to v10.3.4 with @storybook/react-vite
- Remove tsconfig.eslint.json as it is no longer needed with oxlint
- Update package keywords to mantine-v9 and README to reference Mantine 9.x
- Update CLAUDE.md to reflect new tooling and Mantine 9 requirement
- Update CI workflow and build configuration for new toolchain
- Refresh yarn.lock with all updated dependencies
* Upgrade @mantine/core and @mantine/hooks to 9.0.0 across all workspaces * Upgrade TypeScript to 6.0.2 and React to 19.2.4 * Replace ESLint with oxlint for faster linting (add oxlint.config.ts, remove eslint.config.mjs) * Replace Prettier with oxfmt for formatting (add .oxfmtrc.json, remove .prettierrc.mjs) * Upgrade Storybook to v10.3.4 with @storybook/react-vite * Remove tsconfig.eslint.json as it is no longer needed with oxlint * Update package keywords to mantine-v9 and README to reference Mantine 9.x * Update CLAUDE.md to reflect new tooling and Mantine 9 requirement * Update CI workflow and build configuration for new toolchain * Refresh yarn.lock with all updated dependencies
There was a problem hiding this comment.
Pull request overview
Upgrades the library and tooling stack to Mantine v9, TypeScript v6, React 19, and switches lint/format tooling to the OxC ecosystem while updating Storybook and CI configuration.
Changes:
- Upgrade core dependencies (Mantine 9, TypeScript 6, React 19) and refresh lockfile/toolchain configs.
- Replace ESLint/Prettier with oxlint/oxfmt and update related configs/scripts.
- Update Storybook setup and CI workflow to match the new stack.
Reviewed changes
Copilot reviewed 18 out of 20 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| tsconfig.json | Switch TS module resolution to bundler for modern toolchains. |
| tsconfig.eslint.json | Remove ESLint-specific tsconfig (no longer needed). |
| tsconfig.build.json | Narrow build include to package/src and set rootDir for declarations output. |
| README.md | Document Mantine 9.x / React requirement. |
| package/src/use-qr-code-download.ts | Adjust ref typings used by the download hook. |
| package/src/QRCode.tsx | Update component implementation for Mantine 9 migration (notably ref handling). |
| package/package.json | Update package description/keywords and reorder metadata. |
| package.json | Swap to oxlint/oxfmt, update dependency versions (Mantine/TS/React/Storybook, etc.). |
| oxlint.config.ts | Add oxlint configuration and rule set. |
| jsdom.mocks.cjs | Simplify Jest DOM mocks setup. |
| eslint.config.mjs | Remove old ESLint configuration. |
| docs/tsconfig.json | Modernize docs TS config (target/lib formatting and bundler resolution). |
| docs/package.json | Upgrade docs dependencies to Mantine 9 / TS 6 / Next patch updates. |
| CLAUDE.md | Update contributor/dev notes for the new tooling and requirements. |
| .storybook/preview.tsx | Update preview globals/decorators to use MantineProvider color scheme controls. |
| .storybook/main.ts | Update Storybook config for v10 + react-vite and adjust addons/core settings. |
| .prettierrc.mjs | Remove Prettier configuration. |
| .oxfmtrc.json | Add oxfmt configuration (import order, ignores, formatting). |
| .github/workflows/pull_request.yml | Update CI to use .nvmrc via node-version-file and rename workflow. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Provides a minimal 21×21 QR matrix for tests - Corrects esbuild-jest default export resolution - Enables component to render full SVG content in unit tests
- Forward ref to the root Box element - Enables parent components to access the QRCode DOM node - Updated props handling to preserve ref while passing other props to useProps
- Simplify props handling by passing _props directly to useProps - Eliminate ref extraction and forwarding to Box - Avoid unnecessary ref prop in QRCode component
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.