Repository navigation
fix: 개발 직무 분석의 주 문서를 첫 문서가 아니라 포트폴리오로 고른다 - #56
Merged
kimwuju merged 1 commit intoOct 20, 2026
Merged
Conversation
API는 사용자가 정한 순서 그대로 documents를 보내므로 [RESUME, PORTFOLIO]처럼 이력서가 앞에 올 수 있다. 지금은 documents[0]을 주 문서로 써서 Stage1 검증·텍스트 품질 판정·페이지 복구(#42)·이미지 예산 우선권이 이력서에 쓰이고, 실제 포트폴리오는 복구 없는 추가 문서 경로로 갔다. type이 PORTFOLIO인 첫 문서를 주 문서로 고르고, 분석 입력의 문서 번호(doc-N)는 보낸 순서 그대로 둔다. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
danjee2009
reviewed
Oct 20, 2026
danjee2009
left a comment
Member
There was a problem hiding this comment.
검토 결과: 확인된 수정 필요 결함은 없습니다. 개발 직군 문서 요청에서 PORTFOLIO 문서를 주 문서로 선택하고 원래 문서 순서를 유지하는 처리와 회귀 테스트를 검토했습니다. 테스트는 직접 재실행하지 않았습니다.
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
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.
무엇을
#52 리뷰 코멘트([P2] 이력서를 먼저 두면 이력서가 포트폴리오로 처리됨)를 고칩니다.
원인: 개발 직무 경로가 문서 종류와 상관없이
documents[0]을 주 문서로 썼습니다. API는 사용자가 정한 순서 그대로documents를 보내고, Web도 첫 줄에 이력서를 고를 수 있어서[RESUME, PORTFOLIO]요청이 실제로 옵니다. 이때 다음과 같이 됐습니다.documentExtraction보고도 이력서 기준이었습니다변경
dispatch_applicant_profile.py):type이PORTFOLIO인 첫 문서를 주 문서로 고릅니다. 개발 직무는 포트폴리오가 필수라 접수 검증을 통과한 요청에는 항상 있습니다.doc-N)와 순서는 보낸 그대로입니다portfolioUrl)과 문서가 하나인 요청은 동작이 같습니다docs/api.md): "documents[0]에만 적용한다"던 복구·보고서 설명을 주 문서 기준으로 고쳤습니다test_dev_documents.py):[RESUME, PORTFOLIO]요청에서 Stage1과 텍스트 복구에 포트폴리오가 쓰이는지, 문서 번호가 순서대로인지 확인합니다. 수정 전 코드에서는 Stage1에resume.pdf가 전달되어 실패합니다검증
ruff check,ruff format --check,mypy src tests,lint-imports통과pytest: 1306 passed, 20 skipped병합 순서
#52(
codex/general-job-analysis) 위에 쌓았습니다. 이 PR을 먼저 #52 브랜치에 병합하거나, 커밋을 #52 브랜치로 옮겨도 됩니다.🤖 Generated with Claude Code