[wook10]

배경

노션 DB 감사 중 발견한 Execution 명세 불일치 2건.

1. 실행 취소·중지

"체험자는 QUEUED·ASSIGNED 에서만 취소할 수 있다. RUNNING 이후 중지는 관리자 기능으로 처리한다." — 인데 지금까지 구현은 RUNNING 중에도 체험자가 '실행 중지' 버튼으로 멈출 수 있었다.

2. 실행 상태 전달

"Server → Web 실시간 갱신은 SSE를 사용한다. 연결이 끊기면 재연결 후 현재 상태를 다시 조회해 화면을 복구한다." — 인데 지금까지는 600ms 간격 폴링이었다(코드에 "SSE 우선 검토, 지금은 폴링"이라는 의도적 보류 코멘트가 남아있었음).

변경

실행 취소·중지

• features/execution/model/types.ts: CANCELLABLE_STATUSES(queued·assigned)·isCancellableStatus 추가 (기존 TERMINAL_STATUSES 패턴과 동일)
• mocks.ts: 취소 요청이 QUEUED·ASSIGNED 일 때만 실제로 취소 처리 — RUNNING 이면 상태를 그대로 반환
• RunReadyWorkspace.tsx: RUNNING 중엔 '실행 중지' 버튼 대신 비활성 '실행 중…' 표시

실행 상태 전달

• features/execution/lib/executionStream.ts (신규): SSE 구독 함수. 네이티브 EventSource 대신 fetch + ReadableStream 을 직접 읽는다 — 테스트 환경(jsdom)엔 EventSource 가 없어 폴리필 없이는 검증할 수 없고, fetch 기반이라야 MSW로 그대로 테스트할 수 있다. 연결이 끊기면 지수 백오프(500ms~5s)로 재연결한다.
• mocks.ts: GET /api/executions/:id/stream 추가 — 연결 즉시 현재 상태부터 보내고(재연결 시 "현재 상태를 다시 조회"를 만족), 상태가 실제로 바뀔 때만 push. 종료 상태에서 스트림을 닫아 재연결하지 않는다.
• hooks.ts: useExecutionState 가 폴링 대신 SSE 구독으로 쿼리 캐시를 갱신. 초기 1회 REST 조회는 유지(마운트 직후·refetch() 즉시 호출용).

"실행 상태 전달"은 실제 백엔드가 없어 SSE 계약(이벤트 포맷 등)을 알 수 없는 상태에서 진행한 것이라, 나중에 실제 Poppy-Server가 정해지면 프레임 형식을 다시 맞춰야 할 수 있다.

테스트

• executionApi.test.ts: RUNNING 에서 취소해도 상태가 안 바뀌는 것 확인
• executionStream.test.ts (신규): 초기 상태 즉시 수신, 상태 전환 push 확인(대기→배정→진행→완료), 완료 시 스트림 종료, 취소 시 프레임 수신
• ExperienceView.test.tsx: 배정 대기 중 취소 동작 유지, RUNNING 중엔 중지 버튼이 없고 '처음으로'로 로컬 초기화만 되는 것 확인
• 전체 93/93 통과, typecheck/lint/build/format:check 클린
• CDP로 실제 브라우저에서 폴링 없이 대기→배정→진행(취소 불가)→완료 전환이 SSE push 로만 이루어짐을 확인

🤖 Generated with Claude Code

[inryeok-bot]

리뷰 결과

변경된 6개 파일을 검토했으며, 수정이 필요한 문제를 찾지 못했습니다.

심각도	개수
Critical	0
High	0
Medium	0
Low	0

완료

변경 사항을 검토했으며 게시할 인라인 코멘트가 없습니다.

검토한 head: 6c23050faa46