diff options
Diffstat (limited to 'docs')
| -rw-r--r-- | docs/TODO.md | 180 |
1 files changed, 103 insertions, 77 deletions
diff --git a/docs/TODO.md b/docs/TODO.md index 57c57c8..d8e5e6a 100644 --- a/docs/TODO.md +++ b/docs/TODO.md @@ -5,102 +5,128 @@ ## Current State - **268 tests passing**, lint clean -- 6 microservices (data-collector, strategy-engine, order-executor, portfolio-manager, backtester, api) + CLI + shared library -- 8 trading strategies (RSI, Grid, MACD, Bollinger, EMA Crossover, VWAP, Volume Profile, Combined) -- Operations: SQLAlchemy ORM, Alembic migrations, structlog, Telegram alerts, resilience, Prometheus + Grafana -- Monitoring: Prometheus, Grafana (provisioned dashboard), Loki + Promtail log aggregation -- Security: Bearer token auth on /health and /metrics endpoints -- CI/CD: Gitea Actions workflow, standalone CI script -- REST API: FastAPI with portfolio, orders, strategies endpoints -- Advanced risk: trailing stop-loss, volatility-based position sizing, max open positions -- Multi-exchange: ccxt factory supporting any exchange (Binance, Bybit, OKX, Kraken, etc.) +- 6 microservices + CLI + shared library +- 8 strategies, REST API, Prometheus/Grafana/Loki, CI/CD, multi-exchange, advanced risk --- -## All Completed - -### Critical (Deployment Blockers) -- [x] Alembic 초기 마이그레이션 (6 테이블) -- [x] Docker Compose 헬스체크 포트 노출 (8080-8083) -- [x] numpy 의존성 추가 - -### High Priority -- [x] CLI 명령어 구현 (backtest, strategy, portfolio, data) -- [x] Backtester sys.path 해킹 제거 (STRATEGIES_DIR 환경변수) -- [x] Integration tests (7개) -- [x] Dockerfile 의존성 정리 + PYTHONPATH 설정 - -### Medium Priority -- [x] Edge case tests (30개: zero volume, empty data, extreme values, reset, notifier) -- [x] DB transaction rollback + transaction() context manager -- [x] Portfolio snapshots (주기적 저장 + Telegram 일일 요약) -- [x] Strategy parameter validation (41 tests, 7 strategies) -- [x] VWAP division by zero 버그 수정 - -### Low Priority -- [x] Grafana dashboard provisioning (7-panel trading overview) -- [x] Multi-exchange support (ccxt factory) -- [x] REST API (FastAPI: portfolio, orders, strategies) -- [x] Strategy combination framework (weighted voting) -- [x] Advanced risk management (trailing stop, volatility sizing, position limits) -- [x] Loki + Promtail log aggregation -- [x] CI/CD pipeline (Gitea Actions + scripts/ci.sh) -- [x] Docker E2E test script (scripts/e2e-test.sh) -- [x] Security hardening (bearer token auth on endpoints) +## Critical (기능 오류 — 즉시 수정 필요) + +### 1. Strategy Engine 멀티 심볼 버그 +- **파일:** `services/strategy-engine/src/strategy_engine/main.py:54-63` +- **문제:** `for symbol in config.symbols:` 안에 `while True` 루프가 있어서 첫 번째 심볼만 처리 +- **영향:** BTC/USDT, ETH/USDT, SOL/USDT 설정 시 BTC/USDT만 영원히 처리 +- **수정:** `asyncio.create_task()`로 각 심볼 병렬 처리 + +### 2. Portfolio Snapshot 첫 실행 지연 +- **파일:** `services/portfolio-manager/src/portfolio_manager/main.py:48` +- **문제:** `snapshot_loop()`이 `asyncio.sleep()` 먼저 실행 → 첫 스냅샷이 24시간 후 +- **수정:** 스냅샷을 먼저 찍고 sleep, 또는 시작 시 즉시 한 번 실행 + +### 3. .env.example 누락 필드 +- **파일:** `.env.example` +- **누락:** `RISK_TRAILING_STOP_PCT`, `RISK_MAX_OPEN_POSITIONS`, `RISK_VOLATILITY_LOOKBACK`, `RISK_VOLATILITY_SCALE` +- **수정:** 4개 필드 추가 + +### 4. API 서비스 헬스체크 없음 +- **파일:** `docker-compose.yml` — api 서비스에 `healthcheck` 블록 없음 +- **수정:** curl 기반 healthcheck 추가 + +### 5. API 엔드포인트 에러 처리 없음 +- **파일:** `services/api/src/trading_api/routers/*.py` +- **문제:** DB 연결 실패 시 500 에러 + 스택 트레이스 노출 +- **수정:** try/except + HTTPException으로 적절한 에러 응답 + +### 6. Alembic 하드코딩된 DB 인증정보 +- **파일:** `shared/alembic.ini:3` +- **문제:** `postgresql+asyncpg://postgres:postgres@localhost:5432/trading` — docker-compose는 `trading:trading` 사용 +- **수정:** `DATABASE_URL` 환경변수 우선 참조 (env.py에서 이미 처리하지만 alembic.ini 기본값 불일치) --- -## Future Ideas (Not Planned) +## High Priority (Production 전) -- WebSocket 기반 실시간 대시보드 -- 백테스트 결과 시각화 (matplotlib/plotly 차트) -- 분산 추적 (OpenTelemetry) -- 멀티 타임프레임 전략 -- 머신러닝 기반 시그널 필터 -- 주문 유형 확장 (LIMIT, STOP_LIMIT, OCO) -- Kubernetes 배포 설정 +### 7. Binance WebSocket 전용 문제 +- **파일:** `services/data-collector/src/data_collector/binance_ws.py` +- **문제:** `exchange_id` 설정은 있지만 WebSocket은 Binance 하드코딩 +- **수정:** 거래소별 WebSocket 추상화 또는 ccxt pro의 watchOHLCV 사용 + +### 8. Backtester Config 상속 오류 +- **파일:** `services/backtester/src/backtester/config.py` +- **문제:** `BacktestConfig(BaseSettings)` — 다른 서비스는 `Settings` 상속 +- **수정:** `class BacktestConfig(Settings):`로 변경, 중복 `database_url` 제거 + +### 9. CI에 Docker 빌드 테스트 없음 +- **파일:** `scripts/ci.sh` +- **문제:** pytest + ruff만 실행, Dockerfile 빌드 검증 없음 +- **수정:** `docker compose build` 단계 추가 + +### 10. Health Port 오프셋 문서화 +- **파일:** 각 서비스 main.py +- **문제:** `health_port + 0/1/2/3` 하드코딩, 변경 시 혼란 +- **수정:** 각 서비스별 `HEALTH_PORT` 환경변수 또는 문서화 --- -## Quick Start (개발 환경) +## Medium Priority (안정화) -```bash -# 1. 환경 설정 -cp .env.example .env -# .env에 BINANCE_API_KEY, BINANCE_API_SECRET 입력 +### 11. 테스트 미커버 모듈 +- CLI 명령어 6개 (backtest, data, portfolio, strategy, trade, service) +- API 라우터 3개 (portfolio, orders, strategies) +- BinanceWebSocket 클래스 +- 각 서비스 main.py 실행 루프 -# 2. 의존성 설치 -pip install -e shared/ +### 12. Realized PnL 미추적 +- **파일:** `services/portfolio-manager/src/portfolio_manager/main.py:32` +- **문제:** `realized_pnl=Decimal("0")` 하드코딩 — 실현 손익 미계산 +- **수정:** 매도 시 실현 PnL 누적 → 스냅샷에 반영 -# 3. 인프라 실행 -make infra +### 13. Test Warning 수정 +- **파일:** `shared/tests/test_db.py:315` +- **문제:** `RuntimeWarning: coroutine was never awaited` +- **수정:** `MagicMock()` → `AsyncMock()` -# 4. DB 마이그레이션 -make migrate +### 14. Order Executor last_id 추적 +- **파일:** `services/order-executor/src/order_executor/main.py:92-96` +- **문제:** `last_id`가 실제로 업데이트되지 않음 (항상 "$") +- **영향:** 재시작 시 메시지 유실 가능 +- **수정:** Redis consumer group 사용 또는 last_id 추적 -# 5. 테스트 -make test +--- -# 6. 서비스 실행 -make up +## Future Ideas (미계획) -# 7. API 접근 -curl http://localhost:8000/api/v1/strategies -curl http://localhost:8000/api/v1/portfolio/positions +- WebSocket 기반 실시간 대시보드 +- 백테스트 결과 시각화 (matplotlib/plotly) +- OpenTelemetry 분산 추적 +- 멀티 타임프레임 전략 +- 머신러닝 시그널 필터 +- 주문 유형 확장 (LIMIT, STOP_LIMIT, OCO) +- Kubernetes 배포 +- 알림 채널 확장 (Discord, Slack) +- 백테스트 결과 DB 저장 및 비교 -# 8. 모니터링 -docker compose --profile monitoring up -d -# Grafana: http://localhost:3000 (admin/admin) -# Prometheus: http://localhost:9090 +--- + +## Estimated Effort -# 9. CLI 사용 -trading strategy list -trading backtest run --strategy rsi --symbol BTCUSDT --from 2025-01-01 --to 2025-12-31 -trading portfolio show +| 카테고리 | 예상 작업량 | +|----------|------------| +| Critical (1-6) | 2-3시간 | +| High (7-10) | 4-6시간 | +| Medium (11-14) | 1-2주 | -# 10. E2E 테스트 -make e2e +--- -# 11. CI -make ci +## Quick Start + +```bash +cp .env.example .env # API 키 입력 +pip install -e shared/ # 의존성 설치 +make infra # Redis + PostgreSQL 시작 +make migrate # DB 마이그레이션 +make test # 268 tests 실행 +make up # 서비스 시작 +make e2e # E2E 테스트 +curl localhost:8000/health # API 헬스체크 ``` |