fix: improve error handling and JSON safety in ResumeEvaluator #168

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Open

Dubey123f wants to merge 1 commit into interviewstreet:main from Dubey123f:fix/resume-evaluator-json-error

Dubey123f commented Oct 25, 2025 •

edited

Loading

🐞 Problem

The existing evaluator.py passed an unsupported format parameter to provider.chat(), causing runtime errors.

✅ Fix

Removed the invalid format kwarg from LLM API calls.
Added robust JSON parsing with graceful JSONDecodeError handling.
Added provider validation and improved logging.

🧪 Testing

Tested locally using mock LLM responses.
JSON parsing verified for malformed and valid responses.

🔍 Impact

This ensures stable resume evaluation across all model providers and prevents hard crashes during LLM calls.


          fix: improve error handling and JSON safety in ResumeEvaluator

7e592b9

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

Labels

None yet