improve: domain-aware skill suggestions and expanded JD parser coverage#2
Merged
Merged
Conversation
Owner
mibali
commented
May 4, 2026
mibali
commented
- JD parser now extracts MLOps and domain-specific tools (KFP, Ray, SkyPilot, DCGM, MLflow, DVC, Kubeflow, Seldon, BentoML, Triton, Argo, Vertex AI, SageMaker, Prefect, Dagster, vLLM, and ~30 more)
- _extractBulletsAndSentences now captures plain-text requirement lines and splits on '/' separators, fixing missed sub-requirements like 'deploying ML orchestration tools (e.g. NV Ray, KFP, SkyPilot)'
- Added suggestDomainSkills() to CVTailor with static coverage for 7 domains (MLOps, data engineering, DevOps, ML scientist, frontend, backend, cloud) used as a fallback
- /api/cv/analyze fires a parallel LLM call to dynamically suggest domain-common tools for any job title not covered by the static map; falls back to static suggestions on timeout or failure
- Popup shows a new 'Also common in this domain' section with checkboxes; confirmed domain skills are included in the tailor request
- JD parser now extracts MLOps and domain-specific tools (KFP, Ray, SkyPilot, DCGM, MLflow, DVC, Kubeflow, Seldon, BentoML, Triton, Argo, Vertex AI, SageMaker, Prefect, Dagster, vLLM, and ~30 more) - _extractBulletsAndSentences now captures plain-text requirement lines and splits on '/' separators, fixing missed sub-requirements like 'deploying ML orchestration tools (e.g. NV Ray, KFP, SkyPilot)' - Added suggestDomainSkills() to CVTailor with static coverage for 7 domains (MLOps, data engineering, DevOps, ML scientist, frontend, backend, cloud) used as a fallback - /api/cv/analyze fires a parallel LLM call to dynamically suggest domain-common tools for any job title not covered by the static map; falls back to static suggestions on timeout or failure - Popup shows a new 'Also common in this domain' section with checkboxes; confirmed domain skills are included in the tailor request Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.