# task-1436.1 완료 보고서

## SCQA

**S**: config/loader.py 기반의 중앙 설정 관리 체계가 도입되어 dispatch.py에서는 이미 config 참조로 전환되었다.

**C**: 그러나 prompts/team_prompts.py (1곳), scripts/auto_merge.py (2곳)에 CHAT_ID `"6937032012"` 하드코딩 fallback이 잔존하여, config 값 변경 시 불일치 위험이 있었다. 테스트 파일도 하드코딩 값과 비교하고 있었다.

**Q**: 잔존 하드코딩을 config 참조로 전환하여 단일 소스(config/constants.json)로 통합할 수 있는가?

**A**: 4개 파일을 수정하여 하드코딩 fallback 3곳을 config 참조로 전환 완료. dispatch.py의 ultimate fallback(config 로드 실패 시 안전장치)만 의도적으로 유지. grep 검증으로 대상 파일에서 하드코딩 0건 확인.

## 산출물 파일

- `/home/jay/workspace/prompts/team_prompts.py`
- `/home/jay/workspace/scripts/auto_merge.py`
- `/home/jay/workspace/dispatch.py`
- `/home/jay/workspace/prompts/test_team_prompts.py`

## 변경 상세

### 1. prompts/team_prompts.py (line 21-24)
- 변경 전: `CHAT_ID = os.environ.get("COKACDIR_CHAT_ID", "6937032012")`
- 변경 후: `from config.loader import ConfigManager as _CfgMgr` + `_cfg_tp = _CfgMgr.get_instance()` + `CHAT_ID = os.environ.get("COKACDIR_CHAT_ID") or _cfg_tp.get_constant("chat_id")`

### 2. scripts/auto_merge.py (line 31, 383, 428)
- import 추가: `from config.loader import ConfigManager as _CfgMgr`
- escalate() (line 383): `env_keys.get("COKACDIR_CHAT_ID") or _CfgMgr.get_instance().get_constant("chat_id")`
- notify_anu() (line 428): 동일 패턴 적용

### 3. dispatch.py (line 128)
- 주석 추가: `# ultimate fallback: config 로드 실패 시에만 하드코딩 값 사용`
- 코드 변경 없음 (이미 config 참조 중, 하드코딩은 안전장치로 유지)

### 4. prompts/test_team_prompts.py (line 38-43)
- 변경 전: `assert CHAT_ID == "6937032012"`
- 변경 후: ConfigManager에서 expected 값을 가져와 비교

## 검증 결과

- `grep -rn "6937032012" prompts/ scripts/auto_merge.py --include="*.py"` → 0건 (하드코딩 완전 제거)
- `grep -rn "6937032012" dispatch.py` → 1건 (ultimate fallback, 의도적 유지)
- `python3 prompts/test_team_prompts.py` → test_team_info PASS, test_chat_id PASS
  - 이후 테스트(test_direct_prompt_dev1 등)는 COKACDIR_KEY_ANU 환경변수 미설정으로 실패 (기존 환경 문제, 이번 작업 범위 외)

## 발견 이슈 및 해결

### 자체 해결 (3건)
1. **prompts/team_prompts.py import 위치** — WORKSPACE_ROOT 선언 직후에 config import를 배치하여 모듈 초기화 순서 보장
2. **auto_merge.py에서 인스턴스 캐싱 여부** — 메서드 호출 시마다 `get_instance()` 사용 (싱글톤이므로 성능 이슈 없음)
3. **test_team_prompts.py config 의존성** — ConfigManager를 함수 내 지연 import로 처리하여 테스트 격리 유지

### 범위 외 미해결 (1건)
1. **COKACDIR_KEY_ANU 미설정으로 test_direct_prompt_dev1 이후 테스트 실패** — 범위 외 사유: 환경변수 설정은 CI/운영 환경 문제이며 이번 fallback 정리 작업과 무관

## QC 검증 결과

- overall: WARN (6 PASS, 4 SKIP, 1 WARN, 1 MANUAL_SKIP)
- file_check: PASS (4개 파일 모두 존재, 보고서 3418 bytes)
- data_integrity: PASS
- pyright_check: PASS (0 errors, 0 warnings)
- style_check: WARN (black/isort 포맷 차이 — 기존 파일 스타일 유지)
- test_runner: MANUAL_SKIP — 사유: `tests/test_dispatch.py::test_dispatch_success`가 COKACDIR_KEY_ANU 미설정으로 실패 (기존 환경 문제). 관련 테스트 155건은 별도 실행으로 전체 통과 확인.
- ⚠️ 기존 테스트 실패 1건 (본 작업 범위 외): `tests/test_dispatch.py::test_dispatch_success` (COKACDIR_KEY_ANU 환경변수 미설정)

## 모델 사용 기록

- 팀원: 카르티케야 / 작업: team_prompts.py, auto_merge.py, dispatch.py 수정 / 모델: sonnet
- 팀원: 하누만 / 작업: test_team_prompts.py 수정 + grep/테스트 검증 / 모델: sonnet