-
Notifications
You must be signed in to change notification settings - Fork 0
Open
Labels
area/pipelineReview pipeline (L1/L2/L3)Review pipeline (L1/L2/L3)priority/mediumNice to haveNice to havesize/M<200 lines<200 linestech-debtTechnical debt cleanupTechnical debt cleanup
Description
Summary
설정 스키마에 정의되어 있지만 런타임에서 무시되는 필드 9개 + MCP 1개. 사용자가 설정해도 아무 효과 없어 혼동 유발.
목록
| # | 필드 | 위치 | 상태 | 권장 |
|---|---|---|---|---|
| 1 | config.plugins |
config.ts:234 |
완전 미구현 (읽는 코드 없음) | #185 결정 후 처리 |
| 2 | supporters.pickStrategy |
config.ts:110 |
round-robin 미구현, 항상 randomPick() |
구현 또는 enum에서 제거 |
| 3 | supporters.personaAssignment |
config.ts:113 |
fixed 미구현, 항상 randomElement() |
구현 또는 enum에서 제거 |
| 4 | declarativeReviewers.constraints.tierMin |
config.ts:159 |
L0 모델 선택에서 미사용 | 구현 또는 제거 |
| 5 | declarativeReviewers.constraints.preferProviders |
config.ts:160 |
L0 모델 선택에서 미사용 | 구현 또는 제거 |
| 6 | config.github.postSuggestions |
config.ts:232 |
mapper에 전달 안 됨 | 연결 또는 제거 |
| 7 | config.github.collapseDiscussions |
config.ts:232 |
mapper에 전달 안 됨 | 연결 또는 제거 |
| 8 | agent.label |
config.ts:38 |
런타임에서 안 읽음 | 문서화 또는 제거 |
| 9 | modelRouter.strategy |
l0.ts:87 |
단일값 enum (thompson-sampling), 값 안 읽음 |
제거 (확장 시 재추가) |
| 10 | MCP reviewer_count |
mcp/helpers.ts:16 |
받기만 하고 runPipeline()에 미전달 |
전달 또는 제거 |
수정 방향
각 필드별로:
- 간단히 구현 가능 (2, 3, 6, 7, 10) → 구현
- L0 로직 변경 필요 (4, 5) → 제거하고 L0 고도화 시 재추가
- 의미 없음 (8, 9) → 제거
- 별도 이슈 (1) → feat: plugins/ 시스템 + rules/ 엔진 — 연결 vs 삭제 결정 #185 참조
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
area/pipelineReview pipeline (L1/L2/L3)Review pipeline (L1/L2/L3)priority/mediumNice to haveNice to havesize/M<200 lines<200 linestech-debtTechnical debt cleanupTechnical debt cleanup