feat: adaptive exam input, graph canvas, voice, confetti#161
Conversation
…tion UX - Expand AdaptiveInput (MCQ, graph drawing types) and refactor GraphCanvas - Inline exam logic into exam page; remove useExamLogic hook - Add weakness-paper API, VoiceDictationButton, and confetti on milestones - Coach shell and sidebar a11y; coach-voice styling for feedback blocks - PDF viewer sticky controls; study technique modal and UI polish Made-with: Cursor
Made-with: Cursor
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 13 minutes and 22 seconds. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (21)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request introduces a "weakness paper" generation feature using an AI service, adds a voice dictation component for accessibility, and implements performance optimizations through memoization and a layered rendering system for the graph canvas. It also enhances the user experience with celebration effects and improved UI transitions. Feedback suggests refining the API timeout logic to prevent resource exhaustion and handling uninitialized canvas states more explicitly in the graph component.
| Generate the full practice paper Markdown now.`; | ||
|
|
||
| const controller = new AbortController(); | ||
| const timeoutId = setTimeout(() => controller.abort(), 15_000); |
There was a problem hiding this comment.
The timeout is set to 15 seconds, but the rate limit is 4 requests per minute. If multiple requests are pending, this could lead to resource exhaustion. Consider reducing the timeout or implementing a more robust queueing mechanism.
|
|
||
| const getGraphCoordinates = (e: React.MouseEvent<HTMLCanvasElement>) => { | ||
| const canvas = canvasRef.current; | ||
| if (!canvas) return { x: 0, y: 0, cx: 0, cy: 0 }; |
There was a problem hiding this comment.
The function returns a default object { x: 0, y: 0, cx: 0, cy: 0 } when the canvas is null. This might mask issues where the canvas is not properly initialized or mounted. Consider throwing an error or handling the null case more explicitly if this is an unexpected state.
1 participant
Rebases and lands the work from closed #160 onto
mainafter #159 merged (stacked base no longer needed).Includes reviewer follow-ups from the original PR.
Made with Cursor