# 작업 보고서: task-104.1

**팀:** dev3-team
**팀장:** 라(Ra)
**작성일:** 2026-03-02
**소요시간:** 5분 50초
**작업 ID:** task-104.1
**근거:** task-100.3 (아누 지시), improvement-plan.md M2+L2

---

## 작업 요약

대시보드 개선 2건:
1. **M2** — 횡단조직(CenterCard) 및 레드팀에 `memberStatus` prop 전달 → 비너스/야누스/마아트/로키 실시간 상태 표시
2. **L2** — `server.py` 기본 패스워드 하드코딩 경고 추가 + `sys` import 정리

---

## 수정 파일 목록

| 파일 | 변경 유형 | 사유 |
|------|----------|------|
| `/home/jay/workspace/dashboard/index.html` | 수정 | M2: CenterCard memberStatus prop 수신, 레드팀 memberStatus 조회 |
| `/home/jay/workspace/dashboard/server.py` | 수정 | L2: 패스워드 경고 + sys import 모듈 레벨 이동 |
| `/home/jay/workspace/teams/dev3/plan-task-104.1.md` | 생성 | 팀장 계획서 의무화 준수 |

---

## GLM 결과 평가

### M2 — index.html

**GLM 구현 내용:**
- `CenterCard` 컴포넌트 시그니처: `({ center })` → `({ center, memberStatus = {} })`
- 내부에서 `memberStatus.members[lead.id]` 조회, `resolvedStatus` 및 `currentTask` 변수 도출
- 레드팀 인라인 JSX를 IIFE `(() => { ... })()` 패턴으로 래핑하여 로컬 변수 사용
- `<CenterCard>` 렌더링 시 `memberStatus={memberStatus}` 전달 추가
- TeamCard의 MemberRow 패턴과 동일한 방식으로 일관성 유지

**검토 결과:** ✅ 1차 검토 통과, 수정 사항 없음
- fallback 체인 `msEntry?.status || lead?.status || 'active'` 적절
- currentTask 표시 추가 (TeamCard 대비 동등 기능) 긍정적

### L2 — server.py

**GLM 구현 내용:**
- line 546: `DASHBOARD_PASSWORD` 우선 조회, fallback `DASHBOARD_PASS`, 최종 기본값 `changeme`
- `__main__` 블록에 경고 3줄 출력 (stderr)
- 단, `import sys`를 `__main__` 블록 내부에 위치시킴

**검토 결과:** ⚠️ 수정 요청 → 팀장 직접 수정
- `import sys`가 `__main__` 블록 내부에 있었음 → 모듈 최상단(`import` 섹션)으로 이동 (파이썬 관례 및 일관성)
- 기능 동작에는 영향 없으나 코드 스타일 불일치

---

## 팀장 검토 수정 사항

**수정 내용:** `server.py` L21 — `import sys` 모듈 최상단 이동
- 수정 전: `__main__` 블록 내부 `import sys`
- 수정 후: `import socketserver` 다음 줄에 `import sys` 추가, `__main__` 블록 내부 `import sys` 제거
- 사유: PEP 8 준수, 모듈 전체에서 sys 사용 가능성 고려

---

## 테스트 결과

### M2 — 수동 테스트 시나리오 (브라우저)
1. 대시보드 접속 후 "횡단조직 & 특수팀" 섹션 확인
2. 비너스(디자인센터), 야누스(DevOps센터), 마아트(QC센터) 각각 StatusDot 표시 확인
3. `/api/status`에서 `member_status.members`에 해당 agent ID 있을 때 동적 상태 반영 확인
4. memberStatus 없을 경우 fallback `lead.status || 'active'` 동작 확인

### L2 — 서버 시작 테스트
- `DASHBOARD_PASSWORD` 미설정 시: stderr에 경고 3줄 출력 확인
- `DASHBOARD_PASSWORD=mypass` 설정 시: 경고 미출력 확인
- 인증 요청 시 `DASHBOARD_PASSWORD` 우선 사용 확인

---

## 셀프 QC (아누 가이드 3.4)

**1. 이 변경이 다른 파일에 영향을 미치는가?**
영향 없음. index.html은 단일 파일 SPA, server.py는 독립 실행 모듈. CenterCard의 prop 추가는 기존 호출부(`<CenterCard center={center} />`)를 신규 호출부(`memberStatus` 추가)로 교체하므로 하위 호환성 문제 없음 (default `= {}`로 처리).

**2. 이 로직의 엣지 케이스는 무엇인가?**
- `lead`가 null인 경우: `CenterCard`에서 `lead && (...)` 조건이 이미 처리. `leadId = lead?.id`의 옵셔널 체이닝으로 추가 방어.
- `memberStatus.members`가 없는 경우: `memberStatus.members || {}` 기본값으로 처리.
- `DASHBOARD_PASSWORD`와 `DASHBOARD_PASS` 모두 없는 경우: `"changeme"` 기본값 유지, 경고 출력.

**3. 이 구현이 작업 지시와 정확히 일치하는가?**
✅ M2: task-103.1 지시사항 4가지 조치 모두 이행 (prop 추가, 내부 조회, 레드팀 적용, TeamCard 패턴 참조).
✅ L2: 환경변수 읽기, 미설정 시 경고, 기본값 유지 — 모두 구현됨.

**4. 에러 처리와 보안은 확인했는가?**
- 보안: 하드코딩 패스워드 노출 경고로 운영자 인식 유도. 기본값 `changeme`는 유지되지만 경고로 치환 의지 전달.
- 에러 처리: `try/except`로 base64 디코딩 실패 처리 (기존 로직, 변경 없음).
- `memberStatus` 조회 시 옵셔널 체이닝 + 기본값으로 NPE 방지.

**5. 테스트가 모든 경로를 커버하는가?**
- M2: memberStatus 있을 때 / 없을 때 두 경로 커버 (manual). 자동화 테스트는 JSX 특성상 브라우저 렌더링 필요.
- L2: 환경변수 설정/미설정 두 경로 커버 (manual). `do_GET` 인증 로직은 기존 테스트 범위 내.

---

## 검토한 대안과 기각 사유

1. **CenterCard를 MemberRow 재사용으로 리팩터링** → 기각: CenterCard는 레이아웃 구조가 달라 인라인 유지가 단순하고 충분
2. **레드팀을 별도 RedTeamCard 컴포넌트로 추출** → 기각: 단일 렌더링 위치이므로 과잉설계
3. **DASHBOARD_PASSWORD 미설정 시 서버 시작 중단** → 기각: task-103.1 지시사항 명시 ("서비스 중단 방지")
4. **DASHBOARD_PASS 환경변수명 유지 (DASHBOARD_PASSWORD 미지원)** → 기각: 명시적이고 혼동 없는 변수명으로 개선 필요

---

## 수정 기록 로그

| 시각 | 담당자 | 파일 | 변경유형 | 사유 |
|------|--------|------|----------|------|
| 2026-03-02 17:27 | 라(Ra) | plan-task-104.1.md | 생성 | 팀장 계획서 의무화 |
| 2026-03-02 17:27 | GLM-5 | index.html | 수정 | M2: memberStatus 전달 |
| 2026-03-02 17:27 | GLM-5 | server.py | 수정 | L2: 패스워드 경고 추가 |
| 2026-03-02 17:33 | 라(Ra) | server.py | 수정 | import sys 모듈 레벨 이동 (코드 스타일) |