-
Notifications
You must be signed in to change notification settings - Fork 0
Open
Labels
priority/lowBacklogBacklogsize/S<50 lines<50 linestech-debtTechnical debt cleanupTechnical debt cleanup
Description
Summary
코드 중복 정리 + 빌드/린트 에러 수정.
중복 상수/유틸
| 항목 | 중복 위치 | 수정 |
|---|---|---|
SEVERITY_ORDER |
core/types, notifications/webhook.ts, discord-live.ts, l2/moderator.ts (4곳) | shared로 통합 |
DECISION_COLORS |
webhook.ts, discord-live.ts (2곳) | shared로 통합 |
truncate() |
webhook.ts, discord-live.ts (2곳) | shared/utils로 이동 |
severityColor |
DebatePanel.tsx, DiffViewer.tsx, theme.ts (3곳) | theme.ts에서 export |
dirExists/fileExists/readJsonFile |
CLI 5개 파일 복사붙여넣기 | shared/utils/fs.ts에서 export |
TUI 버전 v1.1.0 |
2곳 하드코딩 | env var 또는 package.json 읽기 |
빌드/린트
테스트 실패 (1건)
src/tests/init-multiselect.test.ts:462
"buildMultiProviderConfig() > supports cli backend"
expected: 'cli' received: 'claude'
CLI backend 매핑 로직과 테스트 기대값 불일치.
ESLint errors (7건) — 전부 unused vars
| 파일 | 미사용 심볼 |
|---|---|
cli/commands/costs.ts:23 |
CostSummary |
cli/commands/init.ts:621 |
detectApiKeys |
cli/commands/models.ts:7 |
BanditArm |
core/pipeline/orchestrator.ts:112 |
pipelineStartMs |
mcp/index.ts:9 |
z |
web/pages/ReviewDetail.tsx:16 |
Severity |
web/pages/ReviewDetail.tsx:18 |
DiffIssueMarker |
ESLint warnings (11건)
react-hooks/exhaustive-deps6건 (Costs.tsx, Sessions.tsx)no-explicit-any4건 (examples, init.ts)- unused eslint-disable 1건
lint 스크립트 누락
package.json에 "lint" 스크립트 없음 → pnpm lint가 Android SDK lint 실행. "lint": "eslint packages/*/src/" 추가 필요.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
priority/lowBacklogBacklogsize/S<50 lines<50 linestech-debtTechnical debt cleanupTechnical debt cleanup