Skip to content

frontend: React Compiler compatibility layer#2289

Merged
malinskibeniamin merged 1 commit intoreact-doctor/1-infrastructurefrom
react-doctor/2-compiler-compat
Mar 11, 2026
Merged

frontend: React Compiler compatibility layer#2289
malinskibeniamin merged 1 commit intoreact-doctor/1-infrastructurefrom
react-doctor/2-compiler-compat

Conversation

@malinskibeniamin
Copy link
Contributor

Summary

Add React Compiler compatibility across 60 components to prepare for incremental react-doctor rule enablement.

  • Add 'use no memo' directives to components using incompatible libraries (TanStack Table, react-hook-form, Zustand)
  • Add React Compiler source filter in rsbuild.config.ts to skip compilation for problematic modules
  • Add useApiStoreHook typed wrapper for Zustand store selectors
  • Fix related type errors in layout and auth components

Changes

  • frontend/rsbuild.config.ts — React Compiler source filter
  • frontend/src/state/backend-api.tsuseApiStoreHook wrapper
  • frontend/src/components/layout/header.tsx — type fix for useApiStoreHook
  • frontend/src/components/layout/sidebar.tsx — type fix for useApiStoreHook
  • frontend/src/components/require-auth.tsx — type fix for useApiStoreHook
  • 57 component files — 'use no memo' directive additions

React Doctor Score: 100/100

PR 2 of 6 in the react-doctor incremental rollout. Depends on #2287.

Test Plan

  • bun run type:check passes
  • bun run doctor passes with 100/100
  • No runtime regressions (directives are compile-time only)

🤖 Generated with Claude Code

- Add 'use no memo' directives to 60 components using incompatible
  libraries (TanStack Table, react-hook-form, Zustand stores)
- Add React Compiler source filter in rsbuild.config.ts
- Add useApiStoreHook typed wrapper for Zustand store selectors
- Fix type errors in header, sidebar, and require-auth components

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@secpanda
Copy link

secpanda commented Mar 11, 2026

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@malinskibeniamin malinskibeniamin merged commit 123ba17 into react-doctor/1-infrastructure Mar 11, 2026
15 checks passed
@malinskibeniamin malinskibeniamin deleted the react-doctor/2-compiler-compat branch March 11, 2026 14:34
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.

2 participants