Skip to content

Feat: Add JD matching logic for required skills - #397

Merged
Muskankr merged 5 commits into
Muskankr:mainfrom
Atharv094:feature/jd-matching
Jul 28, 2026
Merged

Feat: Add JD matching logic for required skills#397
Muskankr merged 5 commits into
Muskankr:mainfrom
Atharv094:feature/jd-matching

Conversation

@Atharv094

@Atharv094 Atharv094 commented Jul 26, 2026

Copy link
Copy Markdown
Contributor

Closes: #18
## 📝 Description

This PR implements Job Description (JD) matching for the AI Resume Analyzer.

Previously, the backend ignored the submitted job_description field and
matched the resume against generic hardcoded roles. Now, if a user pastes a job
description on the frontend, the backend dynamically extracts required skills
from that JD text and compares the resume strictly against those skills to
generate an accurate match percentage and missing keywords list.

## 🔗 Related Issue

Closes #18

## ✅ Type of Change

- [ ] 🐞 Bug fix (non-breaking change that fixes an issue)
- [x] ✨ New feature (non-breaking change that adds functionality)
- [ ] 📚 Documentation update
- [ ] 💅 UI/UX improvement
- [ ] ♻️ Refactor (no functional change)
- [ ] ⚡ Performance improvement

## 🧪 How Has This Been Tested?

- [x] Frontend builds (`npm run build`) and lints (`npm run lint`) clean
- [ ] Backend tests pass (`python manage.py test analyzer`)
- [x] Manually tested in the browser / API client

## 📋 Checklist

- [x] My code follows the project's style and conventions
- [x] I have linked the related issue above
- [x] I have read the [Code of Conduct](CODE_OF_CONDUCT.md)

@Atharv094
Atharv094 requested a review from Muskankr as a code owner July 26, 2026 10:30
@vercel

vercel Bot commented Jul 26, 2026

Copy link
Copy Markdown

@Atharv094 is attempting to deploy a commit to the a2521729-9889's projects Team on Vercel.

A member of the Team first needs to authorize it.

@Muskankr

Copy link
Copy Markdown
Owner

@Atharv094 , resolve conflicts

@Atharv094

Copy link
Copy Markdown
Contributor Author

@Muskankr I will solve the conflicts asap please don't unassign

@Muskankr
Muskankr merged commit 8e9f302 into Muskankr:main Jul 28, 2026
1 of 2 checks passed
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.

Job description matching: compare resume against a pasted JD

2 participants