Skip to content

fix: 개발 직무 분석의 주 문서를 첫 문서가 아니라 포트폴리오로 고른다 - #56

Merged
kimwuju merged 1 commit into
codex/general-job-analysisfrom
fix/dev-profile-portfolio-document
Oct 20, 2026
Merged

kimwuju merged 1 commit into
codex/general-job-analysisfrom
fix/dev-profile-portfolio-document

Conversation

@kimwuju

@kimwuju kimwuju commented Oct 20, 2026

Copy link
Copy Markdown
Contributor

무엇을

#52 리뷰 코멘트([P2] 이력서를 먼저 두면 이력서가 포트폴리오로 처리됨)를 고칩니다.

원인: 개발 직무 경로가 문서 종류와 상관없이 documents[0]을 주 문서로 썼습니다. API는 사용자가 정한 순서 그대로 documents를 보내고, Web도 첫 줄에 이력서를 고를 수 있어서 [RESUME, PORTFOLIO] 요청이 실제로 옵니다. 이때 다음과 같이 됐습니다.

  • Stage1 검증, 텍스트 품질 판정, 페이지 복구(Feature/#41 pdf text recovery #42), 이미지 예산 우선권이 이력서에 쓰였습니다
  • 실제 포트폴리오는 복구 없는 추가 문서 경로로 가고, 남은 이미지 예산만 받았습니다
  • 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

API는 사용자가 정한 순서 그대로 documents를 보내므로 [RESUME, PORTFOLIO]처럼
이력서가 앞에 올 수 있다. 지금은 documents[0]을 주 문서로 써서 Stage1 검증·텍스트
품질 판정·페이지 복구(#42)·이미지 예산 우선권이 이력서에 쓰이고, 실제 포트폴리오는
복구 없는 추가 문서 경로로 갔다. type이 PORTFOLIO인 첫 문서를 주 문서로 고르고,
분석 입력의 문서 번호(doc-N)는 보낸 순서 그대로 둔다.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

@danjee2009 danjee2009 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

검토 결과: 확인된 수정 필요 결함은 없습니다. 개발 직군 문서 요청에서 PORTFOLIO 문서를 주 문서로 선택하고 원래 문서 순서를 유지하는 처리와 회귀 테스트를 검토했습니다. 테스트는 직접 재실행하지 않았습니다.

@kimwuju
kimwuju merged commit 4d5e3fc into codex/general-job-analysis Oct 20, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants