Skip to content

Add React Doctor and clear GUI/src diagnostics#140

Open
Wibias wants to merge 4 commits into
lidge-jun:mainfrom
Wibias:chore/add-react-doctor
Open

Add React Doctor and clear GUI/src diagnostics#140
Wibias wants to merge 4 commits into
lidge-jun:mainfrom
Wibias:chore/add-react-doctor

Conversation

@Wibias

@Wibias Wibias commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add React Doctor for the GUI: local doctor / doctor:full scripts, changed-scope pre-push when gui/ changes, and advisory GitHub Action CI.
  • Clear React Doctor findings across the GUI (a11y, native dialogs, TanStack Query data fetching, component splits) and across src/ (perf loops/clones, Zod 4 APIs, update integrity check, related cleanups).
  • Verified clean with full-repo and GUI react-doctor scans (No issues found).

Test plan

  • bun run typecheck
  • bun test --isolate ./tests/
  • cd gui && bun run doctor:full → no issues
  • From repo root: bun x react-doctor@latest --verbose --scope full → no issues
  • Smoke GUI: open dashboard, providers, models, auth modals after Query migration
  • Confirm advisory React Doctor workflow runs on this PR without blocking merge

Wibias added 4 commits July 17, 2026 05:50
Wire doctor scripts and changed-scope pre-push when gui/ changes; keep CI advisory via react-doctor.yml scanning gui/.
Resolve GUI diagnostics (a11y, dialogs, TanStack Query, etc.) and src performance/security findings so full-repo doctor scans report clean.
Derive poll data from React Query instead of mirroring into useState via effects, and sync editable slices with render-time state adjusts.
Use a controlled clock and a wider stale window so the replacement lock is not immediately re-staled and unlinked when same-bytes rewrites leave mtime unchanged.
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