Skip to content

fix: validate recommendation interests#849

Open
gTanusri wants to merge 1 commit into
komalharshita:mainfrom
gTanusri:fix/recommend-valid-interests
Open

fix: validate recommendation interests#849
gTanusri wants to merge 1 commit into
komalharshita:mainfrom
gTanusri:fix/recommend-valid-interests

Conversation

@gTanusri

Copy link
Copy Markdown

Summary

  • Added the missing VALID_INTERESTS list used by recommendation input validation.
  • Fixed duplicate project IDs in data/projects.json so dataset validation no longer crashes recommendation requests.
  • Restored recommender compatibility for existing tests and JSON skill-array inputs.
  • Returned recommendation results as a project array to match the existing API/frontend contract.
  • Added focused regression tests for interest validation.

Related Issue

Closes #822

Verification

  • python -m pytest tests/test_recommender_validation.py -q
  • python -m pytest tests -q

@vercel

vercel Bot commented Jun 11, 2026

Copy link
Copy Markdown

@servis is attempting to deploy a commit to the komalsony234-1530's projects Team on Vercel.

A member of the Team first needs to authorize it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: NameError crash on /api/recommend — VALID_INTERESTS is undefined

1 participant