Skip to content

[Refactor] DTOs to records and drop ingest tone#46

Merged
shinae1023 merged 1 commit into
mainfrom
fix/#38-mock-apply
May 14, 2026
Merged

[Refactor] DTOs to records and drop ingest tone#46
shinae1023 merged 1 commit into
mainfrom
fix/#38-mock-apply

Conversation

@shinae1023
Copy link
Copy Markdown
Member

@shinae1023 shinae1023 commented May 14, 2026

✨ 어떤 이유로 PR를 하셨나요?

  • feature 병합
  • 버그 수정(아래에 issue #를 남겨주세요)
  • 코드 개선
  • 코드 수정
  • 배포
  • 기타(아래에 자세한 내용 기입해주세요)

📋 세부 내용 - 왜 해당 PR이 필요한지 작업 내용을 자세하게 설명해주세요

작업 내용

  • job-posting ingest 요청 DTO를 record로 정리해 mutable setter 의존을 제거했습니다.
  • ingest 경로에서 사용되지 않던 tone 필드를 제거하고, 생성 프롬프트에는 null로 전달되도록 정리했습니다.
  • JobPostingExtractResponse, JobPostingGenerateResponse, JobPostingClassificationResultResponserecord로 전환했습니다.
  • JobPostingAiService의 응답 후처리를 setter 기반 수정 방식에서 새 객체를 반환하는 방식으로 리팩터링했습니다.
  • MockApplyController의 ACTUAL API Swagger 설명을 보완해, 공고 추출/저장/사용자 확인 이후 저장된 공고 ID로 지원이 생성되는 흐름을 명시했습니다.

변경 이유

  • DTO를 immutable하게 정리해 요청/응답 객체의 의도를 더 명확히 하고, 불필요한 setter 사용을 줄이기 위해 리팩터링했습니다.
  • ingest API에서 실제로 받지 않아도 되는 tone 입력을 제거해 요청 스펙을 단순화했습니다.
  • ACTUAL mock apply 플로우가 “저장된 공고 기반 지원 생성”이라는 점이 Swagger에 충분히 드러나지 않아 문서를 보완했습니다.

📸 작업 화면 스크린샷

⚠️ PR하기 전에 확인해주세요

  • 로컬테스트를 진행하셨나요?
  • 머지할 브랜치를 확인하셨나요?
  • 관련 label을 선택하셨나요?

🚨 관련 이슈 번호 [#38]

@shinae1023 shinae1023 self-assigned this May 14, 2026
@shinae1023 shinae1023 added the fix label May 14, 2026
@shinae1023 shinae1023 merged commit a2d1142 into main May 14, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant