ivada-infra/DOCS
3
0
Fork 0
Code Issues 1 Pull Requests 5 Actions Packages Projects Releases Wiki Activity

docs: 콜드메일 IR 분석 레거시 함수 제거 및 단일 진입점 통일 추가

Browse Source 
- slack_handler.py 레거시 함수 제거 (119줄)
- 단일 진입점 통일 (interactive.py → coldmail_service.py)
- 코드 중복 제거 및 유지보수성 향상
This commit is contained in:
Claude-51124 2025-12-23 15:04:35 +09:00
parent e2b1a013f2
commit 4da0326034
1 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
journey/troubleshooting/251216_coldmail_ir_analysis_improvements.md
View File

@ -24,9 +24,11 @@
- **성능 향상**: 약 4.9배 속도 개선 - **성능 향상**: 약 4.9배 속도 개선
- 수정: `app/services/ir_deck_analyzer.py` - 수정: `app/services/ir_deck_analyzer.py`
### 2. 콜드메일 IR 분석 아키텍처 개선 (a9af970) ### 2. 콜드메일 IR 분석 아키텍처 개선 (a9af970, bba1f5d)
- **skill-slack API 전환**: `WebClient` 직접 호출 대신 skill-slack API 사용 - **skill-slack API 전환**: `WebClient` 직접 호출 대신 skill-slack API 사용
- **원칙 준수**: rb8001(brain) → skill-slack(hand) 위임 아키텍처 준수 - **원칙 준수**: rb8001(brain) → skill-slack(hand) 위임 아키텍처 준수
- **레거시 함수 제거**: `slack_handler.py`의 `_handle_coldmail_analyze_ir`, `_handle_coldmail_analyze_ir_no` 제거 (119줄 삭제)
- **단일 진입점 통일**: 모든 IR 분석 요청이 `interactive.py` → `coldmail_service.py` 경로로 통일
- 수정: `app/router/slack_handler.py`, `app/services/slack/coldmail_service.py` - 수정: `app/router/slack_handler.py`, `app/services/slack/coldmail_service.py`
### 3. IR 분석 요청 방식 변경 (3a6e8d5) ### 3. IR 분석 요청 방식 변경 (3a6e8d5)
@ -73,6 +75,7 @@
### 아키텍처 원칙 준수 ### 아키텍처 원칙 준수
- skill-slack API 사용으로 서비스 간 책임 분리 명확화 - skill-slack API 사용으로 서비스 간 책임 분리 명확화
- rb8001은 비즈니스 로직, skill-slack은 Slack API 호출 담당 - rb8001은 비즈니스 로직, skill-slack은 Slack API 호출 담당
- 레거시 코드 제거로 단일 진입점 보장, 코드 중복 제거 및 유지보수성 향상
### 사용자 경험 개선 ### 사용자 경험 개선
- 자동 분석보다 사용자 확인 후 분석이 더 나은 UX 제공 - 자동 분석보다 사용자 확인 후 분석이 더 나은 UX 제공