---
task_id: task-1870
team: dev2-team
status: done
created_at: 2026-04-16
---

# task-1870: 와치독 .done 경합(race condition) 버그 수정

## SCQA

**S**: session-watchdog.sh가 systemd timer에 의해 2분마다 실행되며, running 상태 태스크의 PID/heartbeat를 검사하여 stalled 봇을 자동 재위임한다.

**C**: task-1861_2.1+1 사례에서 봇이 timer end(11:21:12) → 보고서 작성 → .done 생성(11:22:54) 순서로 정상 완료 중이었으나, timer end 직후 PID가 사라지자 와치독이 11:21:12에 STALLED로 오판하여 불필요한 재위임(+2)을 실행했다. timer end → .done 생성 사이 약 1분 42초의 race window가 존재.

**Q**: timer end 후 .done 생성까지의 race window에서 와치독의 오판 재위임을 방지할 수 있는가?

**A**: 3중 방어 로직을 추가하여 race condition을 완전 차단. (1) .done/.acked/.clear 파일 확인 확장, (2) STALLED 판정 직전 status 재확인, (3) end_time 존재 시 스킵. 테스트 55/55 전체 통과, 문법 검증 통과. 기존 재위임 기능에 대한 회귀 없음.

## 수정 내용

### 수정 1: .done 파일 확인 확장 (session-watchdog.sh:79)
- 기존: `.done` 파일만 확인
- 변경: `.done`, `.done.acked`, `.done.clear` 3종 모두 확인
- 효과: 완료 처리 과정의 모든 상태 파일을 인식

### 수정 2: STALLED 판정 전 status 재확인 (session-watchdog.sh:151-156)
- 추가: task-timers.json에서 현재 status를 재조회
- status가 `running`이 아니면 (completed/done 등) 스킵
- 효과: timer end가 status를 변경한 직후의 race window 차단

### 수정 3: end_time 존재 시 스킵 (session-watchdog.sh:158-163)
- 추가: end_time 필드가 존재하면 이미 timer end가 완료된 task
- .done 파일이 아직 생성되지 않았어도 스킵
- 효과: timer end → .done 생성 사이 가장 중요한 방어선

### 테스트 업데이트 (test_session_watchdog.sh)
- `judge_task` 함수에 동일한 3중 방어 로직 반영
- TC3a: `.done.acked` 파일 존재 시 pass 판정 검증
- TC3b: `status=completed` 시 pass 판정 검증 (race condition 방어)
- TC3c: `end_time` 존재 시 pass 판정 검증 (timer end 완료)

## 산출물

- `/home/jay/workspace/scripts/session-watchdog.sh`
- `/home/jay/workspace/scripts/tests/test_session_watchdog.sh`

## 발견 이슈 및 해결

### 자체 해결 (3건)
1. **기존 .done 확인이 불완전** — `.acked`와 `.clear` 파일을 OR 조건에 추가하여 완료 과정 전체를 커버
2. **STALLED 판정에 status 재확인 없음** — jq로 status를 재조회하는 guard 추가
3. **end_time 존재 시에도 STALLED 판정** — end_time 확인 guard 추가

## 검증 시나리오 결과

1. 봇이 정상 완료 중(timer end → .done 생성 전)일 때 → end_time guard에 의해 재위임 스킵 (TC3c 통과)
2. 봇이 실제로 죽었을 때(PID 없음 + heartbeat 10분+ + .done 없음 + end_time 없음) → 정상 STALLED 판정 (TC2 통과)
3. 기존 재위임 기능 회귀 없음 → TC1~TC14 + BONUS 전체 55건 통과

## 테스트 결과

```
테스트 실행: bash scripts/tests/test_session_watchdog.sh
결과: 55/55 (0건 실패)
문법 검사: bash -n session-watchdog.sh → OK
```

## QC 체크리스트
- [x] 1. 영향 파일: session-watchdog.sh, test_session_watchdog.sh (2파일만, 다른 파일 영향 없음)
- [x] 2. 엣지 케이스: .done 없음+.acked 있음, status=completed, end_time 있음+status=running → 모두 테스트됨
- [x] 3. 작업 지시와 정확히 일치: 3가지 수정사항 모두 구현
- [x] 4. 에러 처리: jq 에러 시 2>/dev/null 방어 적용
- [x] 5. 테스트 커버리지: 기존 52건 + 신규 3건 = 55건 전체 통과
- [x] 6. 미해결 이슈 없음
- [x] 7. 아키텍처 원칙 위반 없음 (bash script, SOLID 해당 없음)
- [x] 8. 인터페이스 변경 없음 (내부 로직 추가만)

## 모델 사용 기록
- 팀원: 토르 / 작업: session-watchdog.sh 수정 (3중 방어 로직) / 모델: sonnet
- 팀원: 헤임달 / 작업: test_session_watchdog.sh 테스트 업데이트 + 신규 TC 3건 / 모델: sonnet

## 세션 통계
- 총 도구 호출: 9회

### 수정 파일 목록
- /home/jay/workspace/scripts/tests/test_session_watchdog.sh: 3회 (Edit)
- /home/jay/workspace/scripts/session-watchdog.sh: 2회 (Edit)
- bash_cmd: 2회 (Bash)
- /home/jay/workspace/memory/reports/task-1870.md: 1회 (Write)
- /home/jay/workspace/memory/tasks/task-1870.md: 1회 (dispatch)

### 도구 사용 현황
- Edit: 5회
- Bash: 2회
- Write: 1회
- dispatch: 1회