[improve/#69] 적용사항 timeline 근거 의도 정렬 개선

[wantkdd]

✨ 작업 개요

적용사항 timeline 근거가 공통 화면/API 키워드만 보고 잘못 붙지 않도록, 실제 작업 의도 기준으로 근거 발화를 고르는 프롬프트 규칙을 추가했습니다.

📄 작업 내용

• timeline 근거 정렬 규칙 추가
  • 같은 화면/API/도메인명이 겹쳐도 작업 의도가 다르면 같은 application timeline에 포함하지 않도록 명시
  • "어디에 적용하는가"보다 "무엇을 바꾸기로 했는가"를 우선하도록 명시
  • 로딩 스피너 적용과 응답 필드명 통일처럼 공통 범위 키워드가 겹치는 케이스를 예시로 추가
• 전체 회의 분석/적용사항 단독 추출 프롬프트에 동일 규칙 반영
• 프롬프트 계약 테스트 추가

✅ 테스트

• uv run pytest
• uv run ruff check

💬 기타 사항

• 응답 DTO 구조 변경은 없습니다.
• 기존 분석 결과는 재분석 전까지 그대로 유지됩니다.

Closes #69

Summary by CodeRabbit

릴리스 노트

• 새로운 기능

  • 회의 분석의 타임라인 추출 정확도를 향상시키기 위해 근거 정렬 규칙을 강화했습니다. 이제 타임라인에는 실제 적용 의도와 직접 부합하는 발화만 포함되며, 관련성이 낮은 발화는 자동으로 분리됩니다.

• 테스트

  • 타임라인 규칙 검증 테스트를 추가하여 개선 사항이 올바르게 적용되도록 확인했습니다.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 2ea22e44-2dc2-4599-b7a2-0bcbc271f371

📥 Commits

Reviewing files that changed from the base of the PR and between 3930965 and e94ab95.

📒 Files selected for processing (2)

• app/domains/meeting_analysis/services/extraction.py
• tests/test_application_reason_format.py

---

📝 Walkthrough

개요

Gemini 프롬프트에 타임라인 근거 정렬 규칙을 추가하여 적용사항별 근거가 실제 작업 의도와 부합하도록 강화하고, 프롬프트 규칙 검증 테스트를 추가했습니다.

변경 사항

타임라인 근거 의도 정렬

Layer / File(s)	요약
타임라인 근거 정렬 규칙 정의 및 프롬프트 통합 app/domains/meeting_analysis/services/extraction.py	TIMELINE_EVIDENCE_ALIGNMENT_RULE 규칙 리스트를 정의하여 발화 포함/제외 및 적용 의도 기준을 명시하고, APPLICATION_POLICY_PROMPT와 APPLICATIONS_ONLY_PROMPT 조립 시 규칙을 삽입합니다.
프롬프트 규칙 검증 테스트 tests/test_application_reason_format.py	규칙 상수 import를 추가하고, 타임라인 근거 정렬 규칙의 필수 문구 포함 여부와 분석 프롬프트 반영 여부를 검증하는 2개 테스트를 추가합니다.

예상 코드 리뷰 시간

🎯 2 (간단함) | ⏱️ ~10 분

검토자 제안

• Yujin1219

소회

🐰 타임라인 근거 규칙을 정확히 정렬하니,
의도와 발화가 마침내 만나네요.
공통 키워드만으로 헷갈리던 것도,
이제는 작업 의도로 구분하리라. ✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	제목이 PR의 주요 변경사항(적용사항 timeline 근거 의도 정렬 개선)을 명확하고 간결하게 요약하고 있습니다.
Description check	✅ Passed	PR 설명이 템플릿의 모든 필수 섹션(작업 개요, 작업 내용, 관련 이슈, 기타 사항)을 포함하고 있으며 각 섹션이 충분히 작성되었습니다.
Linked Issues check	✅ Passed	PR의 모든 변경사항이 issue #69의 요구사항을 충족합니다: 프롬프트 규칙 추가, 범위 키워드와 작업 의도 분리, 테스트 추가, 응답 구조 유지.
Out of Scope Changes check	✅ Passed	모든 변경사항(프롬프트 규칙 추가 및 테스트 추가)이 issue #69의 범위 내에 있으며, 응답 DTO 구조 변경을 명시적으로 피하고 있습니다.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ 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 improve/wantkdd-timeline-evidence-alignment#69

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[Yujin1219]

굿이요