Skip to content

Conversation

@DayuanJiang
Copy link
Owner

Summary

  • Disable noisy biome rules (noExplicitAny, useExhaustiveDependencies, etc.)
  • Fix memory leak in file-preview-list.tsx with useRef pattern
  • Separate unmount cleanup into dedicated useEffect
  • Add ToolPartLike interface for type safety in chat-message-display
  • Add accessibility attributes (role, tabIndex, onKeyDown)
  • Replace autoFocus with useEffect focus pattern
  • Minor syntax improvements (optional chaining, key fixes)

Changes

File Change
biome.json Disable noisy linting rules
file-preview-list.tsx Fix memory leak with useRef, separate unmount cleanup
chat-message-display.tsx Type safety, accessibility improvements
chat-input.tsx Textarea auto-resize fix
Others Minor syntax cleanup

Test plan

  • Biome check passes
  • Code reviewed by 3 agents + plan agent

- Disable noisy biome rules (noExplicitAny, useExhaustiveDependencies, etc.)
- Fix memory leak in file-preview-list.tsx with useRef pattern
- Separate unmount cleanup into dedicated useEffect
- Add ToolPartLike interface for type safety in chat-message-display
- Add accessibility attributes (role, tabIndex, onKeyDown)
- Replace autoFocus with useEffect focus pattern
- Minor syntax improvements (optional chaining, key fixes)
@vercel
Copy link

vercel bot commented Dec 6, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
next-ai-draw-io Ready Ready Preview Comment Dec 6, 2025 7:18am

@DayuanJiang DayuanJiang merged commit e893bd6 into main Dec 6, 2025
4 checks passed
@DayuanJiang DayuanJiang deleted the fix/biome-lint-and-memory-leak branch December 6, 2025 07:18
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