Skip to content

[Fix] FABRICATED 충돌 근거 검증 범위 완화 - #255

Merged
whc9999 merged 1 commit into
devfrom
fix/demo-day-analysis-status
Jul 31, 2026
Merged

[Fix] FABRICATED 충돌 근거 검증 범위 완화#255
whc9999 merged 1 commit into
devfrom
fix/demo-day-analysis-status

Conversation

@whc9999

@whc9999 whc9999 commented Jul 31, 2026

Copy link
Copy Markdown
Collaborator

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

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

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

  • 모순, 불일치 등 명시적인 충돌 표현을 유효한 FABRICATED 근거로 인정
  • 팀 프로젝트와 개인 프로젝트 또는 단독 수행 주장의 의미상 대조 검증 추가
  • 단순 프로젝트 언급과 역할 혼재 표현은 충돌 근거에서 계속 제외
  • 운영 환경에서 제거됐던 FABRICATED reason 회귀 테스트 추가

📸 작업 화면 스크린샷

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

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

🚨 관련 이슈 번호 [ ]

Summary by CodeRabbit

  • 개선 사항
    • 분석 결과에서 불일치·모순을 나타내는 직접 충돌 표현을 더욱 정확하게 감지합니다.
    • 팀 프로젝트 진행 사실과 개인 프로젝트 주장 간의 상충을 직접적인 충돌로 판정합니다.
    • 단순한 팀 프로젝트 설명이나 역할 혼재 표현은 충돌로 오인하지 않도록 개선했습니다.

- 모순, 불일치 등 명시적인 충돌 표현을 유효한 FABRICATED 근거로 인정
- 팀 프로젝트와 개인 프로젝트 또는 단독 수행 주장의 의미상 대조 검증 추가
- 단순 프로젝트 언급과 역할 혼재 표현은 충돌 근거에서 계속 제외
- 운영 환경에서 제거됐던 FABRICATED reason 회귀 테스트 추가
@whc9999 whc9999 self-assigned this Jul 31, 2026
@coderabbitai

coderabbitai Bot commented Jul 31, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 2effce3c-045a-425d-86d2-d224fde092f7

📥 Commits

Reviewing files that changed from the base of the PR and between a5969a7 and f0b788c.

📒 Files selected for processing (3)
  • src/main/java/com/jobdri/jobdri_api/domain/analysis/service/sanitization/AnalysisSanitizationRules.java
  • src/test/java/com/jobdri/jobdri_api/domain/analysis/service/core/AnalysisServiceTest.java
  • src/test/java/com/jobdri/jobdri_api/domain/analysis/service/sanitization/AnalysisSanitizationRulesTest.java

📝 Walkthrough

Walkthrough

FABRICATED 직접 충돌 판정에 불일치 표현과 팀·개인 프로젝트 주장 조합 검사가 추가되었습니다. 관련 단위 테스트와 분석 서비스 테스트의 기대값이 변경되었습니다.

Changes

FABRICATED 직접 충돌 판정

Layer / File(s) Summary
직접 충돌 판정 규칙 확장
src/main/java/com/jobdri/jobdri_api/domain/analysis/service/sanitization/AnalysisSanitizationRules.java
FABRICATED_DIRECT_CONFLICT_TERMS에 모순과 서술 불일치 표현을 추가했습니다. 팀 프로젝트 주장과 개인 프로젝트 주장이 함께 포함되면 직접 충돌로 판정합니다.
판정 규칙 검증
src/test/java/com/jobdri/jobdri_api/domain/analysis/service/sanitization/AnalysisSanitizationRulesTest.java, src/test/java/com/jobdri/jobdri_api/domain/analysis/service/core/AnalysisServiceTest.java
명시적 불일치와 팀·개인 프로젝트 대조를 true로 검증합니다. 정상적인 팀 프로젝트 설명과 역할 혼재 표현은 false로 검증합니다. 분석 결과의 reason 기대값을 구체적인 팀·개인 프로젝트 충돌 문구로 변경했습니다.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Possibly related PRs

  • JobDri-Developer/BackEnd#156: AnalysisSanitizationRules의 FABRICATED 직접 충돌 판정을 사용하고 재검증 및 후보 복원 로직을 검증합니다.
  • JobDri-Developer/BackEnd#253: 동일한 규칙 클래스에서 FABRICATED 직접 충돌 판정 기준을 변경합니다.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/demo-day-analysis-status

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@whc9999
whc9999 merged commit 7e7e104 into dev Jul 31, 2026
1 of 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