Skip to content

Fix flashcards API endpoints - #490

Open
atul-upadhyay-7 wants to merge 2 commits into
FireFistisDead:masterfrom
atul-upadhyay-7:fix-flashcards-449
Open

Fix flashcards API endpoints#490
atul-upadhyay-7 wants to merge 2 commits into
FireFistisDead:masterfrom
atul-upadhyay-7:fix-flashcards-449

Conversation

@atul-upadhyay-7

Copy link
Copy Markdown
Contributor

Fixes #449. Renamed RAG endpoints to match frontend and added Express proxy routes.

Copilot AI review requested due to automatic review settings June 7, 2026 12:39
@vercel

vercel Bot commented Jun 7, 2026

Copy link
Copy Markdown

@atul-upadhyay-7 is attempting to deploy a commit to the firefistisdead's projects Team on Vercel.

A member of the Team first needs to authorize it.

@coderabbitai

coderabbitai Bot commented Jun 7, 2026

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

@atul-upadhyay-7, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 59 minutes

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: eaf4cee5-4e8b-45a3-ad0b-67c480461713

📥 Commits

Reviewing files that changed from the base of the PR and between e976c8b and b1576a7.

📒 Files selected for processing (3)
  • rag-service/main.py
  • server.js
  • server.test.js
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot added backend Express or API gateway work feature A new feature or improvement fix A targeted fix or cleanup frontend Frontend-related work rag-service FastAPI / model service work labels Jun 7, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Adds “flashcards” functionality by proxying new session flashcard endpoints through the Node server to the RAG service, updates the RAG service route paths to match, and adjusts tests to use a signed Supabase JWT.

Changes:

  • Add /sessions/flashcards and /sessions/flashcards/progress endpoints to server.js with request validation and RAG error propagation.
  • Rename/align RAG service endpoints to /sessions/flashcards and /sessions/flashcards/progress.
  • Extend server tests to cover the new endpoints and update Authorization headers to use a signed JWT.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 7 comments.

File Description
server.test.js Updates auth header handling and adds tests for new flashcards endpoints + internal token forwarding.
server.js Adds validated proxy routes for flashcard generation and progress updates to the RAG service.
rag-service/main.py Aligns FastAPI route paths with the new server proxy endpoints and marks them as protected routes.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread server.test.js
Comment thread server.test.js
Comment thread server.test.js
Comment thread server.test.js
Comment thread server.js
Comment thread rag-service/main.py
Comment thread rag-service/main.py
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend Express or API gateway work feature A new feature or improvement fix A targeted fix or cleanup frontend Frontend-related work rag-service FastAPI / model service work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Flashcard generation and progress tracking are completely non-functional — frontend API routes don't match RAG service endpoints

2 participants