Skip to content

fix(license): use useApiStoreHook to fix react-doctor errors#2301

Merged
jvorcak merged 1 commit intomasterfrom
fix/react-doctor-hooks
Mar 12, 2026
Merged

fix(license): use useApiStoreHook to fix react-doctor errors#2301
jvorcak merged 1 commit intomasterfrom
fix/react-doctor-hooks

Conversation

@malinskibeniamin
Copy link
Contributor

@malinskibeniamin malinskibeniamin commented Mar 12, 2026

Summary

  • Replace useStore(useApiStore, ...) with useApiStoreHook(...) in license notification components
  • Passing a hook (useApiStore) as a value argument to useStore violates React Compiler rules
  • useApiStoreHook is an existing wrapper that encapsulates the same pattern
  • Fixes react-doctor CI check (6 errors → 0, score 99 → 100)

Test plan

  • Verify react-doctor CI passes (100/100)
  • Verify license notification banners still render correctly
  • Verify overview license notification still works

🤖 Generated with Claude Code

Replace useStore(useApiStore, ...) with useApiStoreHook(...) in license
notification components. Passing a hook as a value argument to another
hook violates React Compiler rules. useApiStoreHook is an existing
wrapper that encapsulates the same call pattern.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@malinskibeniamin malinskibeniamin added bug Something isn't working frontend ui/ux labels Mar 12, 2026
@malinskibeniamin malinskibeniamin self-assigned this Mar 12, 2026
@jvorcak jvorcak merged commit c6f8444 into master Mar 12, 2026
16 checks passed
@jvorcak jvorcak deleted the fix/react-doctor-hooks branch March 12, 2026 18:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working frontend ui/ux

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants