Skip to content

fix: 리뷰 반영(공백 정규화, 재채점 콜백 대상, SAMPLE 판정, allow-sample 기본값) - #110

Merged
kimwuju merged 3 commits into
mainfrom
feat/company-dataset-api
Oct 19, 2026
Merged

kimwuju merged 3 commits into
mainfrom
feat/company-dataset-api

Conversation

@kimwuju

@kimwuju kimwuju commented Oct 19, 2026 •

Copy link
Copy Markdown
Contributor
  • 패키지 글 필드의 앞뒤 공백을 검증 전에 빼고 그 값을 저장한다. 공백을 뺀 길이로 검증을 통과한 파일이 VARCHAR 길이에 걸려 저장 단계에서 실패하던 것을 막는다.
  • 재채점 결과가 접수 응답보다 먼저 오면 세션의 지난 작업 행(FAILED)에 쓰던 것을, 이 작업의 행을 새로 만들도록 바꿨다. 이어서 만들어지는 PENDING 행이 결과를 가리지 않는다.
  • V38(main의 V37__add_interview_answer_mode와 겹치지 않게 번호를 옮김): 출처가 없는 기업까지 SAMPLE로 표시한 V35 계산을 가져오기 판정(출처가 있고 모두 SAMPLE)과 맞춰 다시 계산한다.
  • app.company.allow-sample을 비우면 prod 프로필이 아닐 때만 허용한다. 개발·스테이징은 설정을 더하지 않아도 샘플이 보이고, true/false 외 값은 기동을 막는다.

{연관된 이슈 번호}

Task

  • 작업 내용 1
  • 작업 내용 2

작업 내용에 관한 이미지 첨부가 가능합니다.

Screenshot (optional)

ETC

기타 덧붙일 내용을 작성해 주세요.
없다면 삭제해 주세요.

아래 내용은 삭제해 주세요

  1. 작업 내용은 간결히 작성해 주세요. 가능한 경우 이미지를 첨부해 주세요.
  2. 기타 덧붙임 내용에는 리뷰 요구사항 등을 작성 할 수 있습니다.
  3. Pull-Request 제목은 branch 이름과 동일하게 작성해 주세요.

kimwuju and others added 2 commits October 19, 2026 13:41
- 패키지 글 필드의 앞뒤 공백을 검증 전에 빼고 그 값을 저장한다. 공백을 뺀 길이로 검증을
  통과한 파일이 VARCHAR 길이에 걸려 저장 단계에서 실패하던 것을 막는다.
- 재채점 결과가 접수 응답보다 먼저 오면 세션의 지난 작업 행(FAILED)에 쓰던 것을, 이 작업의
  행을 새로 만들도록 바꿨다. 이어서 만들어지는 PENDING 행이 결과를 가리지 않는다.
- V37: 출처가 없는 기업까지 SAMPLE로 표시한 V35 계산을 가져오기 판정(출처가 있고 모두 SAMPLE)과
  맞춰 다시 계산한다.
- app.company.allow-sample을 비우면 prod 프로필이 아닐 때만 허용한다. 개발·스테이징은 설정을
  더하지 않아도 샘플이 보이고, true/false 외 값은 기동을 막는다.

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

Copy link
Copy Markdown
Contributor

[P1] Flyway 마이그레이션 버전이 중복되어 서버가 기동하지 않습니다.
V37__company_sample_requires_source.sql:4
현재 main에 이미 V37__add_interview_answer_mode.sql이 있습니다. 이 PR을 병합하면 V37이 두 개가 되어 Flyway 초기화가 실패합니다

main에 V37__add_interview_answer_mode.sql이 들어와 같은 번호의
V37__company_sample_requires_source.sql 때문에 Flyway가 기동하지 못했다.
보정 마이그레이션을 V38로 바꾸고 테스트가 읽는 경로를 맞춘다.

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.

기업 패키지 정규화와 저장 검증 일치, 재채점 콜백의 작업 행 선택, SAMPLE 판정 마이그레이션 및 프로필 기본값 변경을 관련 테스트와 함께 검토했습니다. 확인된 실질적 문제는 없습니다. PR의 CI 테스트 체크가 통과한 것을 확인했으며, 별도로 로컬 전체 테스트는 실행하지 않았습니다.

@kimwuju
kimwuju merged commit 6acc8e4 into main Oct 19, 2026
1 check 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.

3 participants