dev to main#169
Merged
Merged
Conversation
- 공고 프로필 색상을 관리하는 JobPostingProfileColor enum 추가 - DEFAULT, RED, ORANGE, GREEN, LIGHTBLUE, BLUE, PINK 색상 지원 - JobPosting 엔티티에 profileColor, postingName, jobTitle 필드 추가 - 공고 생성 및 수정 요청 DTO에 신규 필드와 필수값 검증 추가 - 공고 단건·목록 조회 응답에 프로필 색상, 공고명, 직무명 포함 - AI 공고 추출 및 모의 공고 생성 시 기본 프로필 색상과 직무명 저장 - 모의지원 홈 화면에서 소분류명 대신 실제 저장된 직무명 반환 - 기존 공고 데이터의 프로필 색상, 공고명, 직무명 보정 SQL 추가 - 신규 필드의 생성, 조회, 수정 동작을 검증하는 테스트 추가
…ields [Feat] 공고 프로필 색상, 공고명, 직무명 필드 추가
- 평가 CSV rejectionCodeCounts에서 NONE 제외 - 빈 questionAnalyses에 대한 judge 문장 평가 hallucination 검증 추가 - NLG judge OpenAI usage 기반 input/output token 기록 - EvaluationExitCoordinator 종료 실패 시 System.exit 보장 - 종료 Coordinator 중복 exit 요청 멱등성 테스트 추가 - 관련 평가 회귀 테스트 보강
- Spring exit 반환 코드와 요청 코드를 다르게 설정 - SystemExitOperation이 resolved exit code를 받는지 검증 - 중복 exit 요청 멱등성 검증 유지
…alid-issues [Fix] 평가 Runner 및 NLG judge 검증 보강
[Docs] metric 기반 worker 비동기화 및 처리량 확장 ADR 작성 (#136)
Feat/#163 async worker
- 공고 추출/생성 결과 품질 검증 로직 추가 - string, 미분류 회사 등 placeholder 응답을 오류 처리 - 필수 공고 필드 누락 및 짧은 값 검증 추가 - 동기 ingest와 비동기 worker finalize 저장 전 검증 적용 - 비정상 공고 분석 결과 저장 방지 테스트 추가
- 추출 confidence의 NaN, Infinity, 범위 초과 값을 오류 처리 - worker finalize 검증을 결과 저장보다 먼저 수행하도록 순서 변경 - invalid finalize 입력 시 worker 결과 저장과 성공 처리를 막는 테스트 추가 - 공고 추출 confidence 및 필수 필드 경계값 테스트 보강 - 공고 생성 필수 필드 null 및 최소 길이 검증 테스트 추가
…ing-ingest-quality
- 성공/실패 완료된 worker task는 기존 결과 처리 흐름을 우선 적용 - 신규 finalize 저장 직전에만 공고 품질 검증을 수행하도록 순서 조정 - invalid finalize 입력 테스트를 변경된 검증 순서에 맞게 수정 - CI와 동일한 clean test로 검증
…e-job-posting-ingest-quality
…g-ingest-quality [Fix] 공고 분석 placeholder 결과 검증 추가
- 비교 리포트 output 경로를 절대 경로로 정규화 - 비교 CSV 부모 디렉터리 생성 및 writer flush/close 보장 - 비교 CSV 생성 후 파일 존재, regular file, 0 byte, row 수 검증 - 성공 로그에 working directory, 절대 output 경로, 파일 크기, summary row 수 기록 - 비교 Runner와 BatchService의 실제 파일 생성 테스트 보강
- 비교 리포트 summaryRowCount를 실제 생성 CSV row 수 기준으로 계산 - 비교 CSV를 임시 파일에 작성한 뒤 atomic move로 최종 파일 교체 - atomic move 미지원 시 replace move로 fallback - 실패 시 기존 비교 리포트 보존 및 임시 파일 정리 - 입력 CSV와 생성 CSV row count 검증 테스트 보강
…n-output-validation [Fix] NLG judge 비교 리포트 출력 검증 보강
Feat/#167 get paging
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
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
2 participants
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.
✨ 어떤 이유로 PR를 하셨나요?
📋 세부 내용 - 왜 해당 PR이 필요한지 작업 내용을 자세하게 설명해주세요
📸 작업 화면 스크린샷
🚨 관련 이슈 번호 [ ]