fix: uvicorn 미설치 시 dashboard 오해 없는 실패 처리 (#178) #184

Merged
jihoson merged 1 commits from feature/issue-178-dashboard-log-order into main 2026-02-20 09:30:48 +09:00
Collaborator

Summary

  • _start_dashboard_server에서 스레드 시작 전 uvicorn import를 동기적으로 검증
  • uvicorn 없을 때 "Dashboard server started" → "failed" 오해 소지 있는 로그 쌍을 제거
  • 대신 즉시 WARNING 출력 후 None 반환

Fixes

Test plan

  • test_start_dashboard_server_returns_none_when_uvicorn_missing 추가 및 통과
  • 기존 dashboard 테스트 2개 유지
  • 전체 테스트 718개 통과

🤖 Generated with Claude Code

## Summary - `_start_dashboard_server`에서 스레드 시작 전 uvicorn import를 동기적으로 검증 - uvicorn 없을 때 "Dashboard server started" → "failed" 오해 소지 있는 로그 쌍을 제거 - 대신 즉시 WARNING 출력 후 None 반환 ## Fixes - closes #178 ## Test plan - [x] `test_start_dashboard_server_returns_none_when_uvicorn_missing` 추가 및 통과 - [x] 기존 dashboard 테스트 2개 유지 - [x] 전체 테스트 718개 통과 🤖 Generated with [Claude Code](https://claude.com/claude-code)
agentson added 1 commit 2026-02-20 09:28:33 +09:00
스레드 시작 전에 uvicorn import를 검증하도록 _start_dashboard_server 수정.
uvicorn 미설치 시 "started" 로그 없이 즉시 WARNING 출력 후 None 반환.

- 사전 import 검증으로 "started" → "failed" 오해 소지 있는 로그 쌍 제거
- uvicorn 미설치 시 명확한 경고 메시지 출력
- test_start_dashboard_server_returns_none_when_uvicorn_missing 테스트 추가

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
jihoson merged commit 17112b864a into main 2026-02-20 09:30:48 +09:00
Sign in to join this conversation.
No Reviewers
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: jihoson/The-Ouroboros#184