Group model quotas by reset window and add dividers#67
Conversation
|
기존 Gemini 로직을 일단 이건 맛보기 리뷰고, 버그나 보안 문제까지 탈탈 털어주는 진짜 리뷰가 필요하면 댓글에 |
|
@opgginc/op-gg-ai-devops |
✅ AI Code Review Completed
모델 그룹화 로직과 Antigravity 파싱 개선이 잘 적용되었습니다. 구분선 표시 조건에 대한 작은 수정 제안을 남겼으니 확인 부탁드립니다. 수고하셨습니다! Reviewed with |
There was a problem hiding this comment.
🤖 OP.GG DevOps AI Code Review
요약
이번 PR은 모델별 사용량 표시 로직을 개선하여, 동일한 초기화(Reset) 시간과 사용량을 가진 모델들을 그룹화해서 보여주는 기능을 추가했군요. 또한 Antigravity 프로바이더에 대해서도 Reset 시간을 파싱하도록 업데이트했습니다. 전반적으로 코드 구조가 깔끔해졌고 UI 가독성이 좋아질 것 같습니다.
주요 변경 사항
- 그룹화 로직 추상화:
GeminiModelUsageGrouper를ModelUsageGrouper로 일반화하여 다른 프로바이더에서도 재사용 가능하게 변경한 점이 좋습니다. - Antigravity 개선: Antigravity 모델들의 Reset 시간을 파싱하여 상세 정보에 표시하도록 했습니다.
- UI 통일:
addGroupedModelUsageSection헬퍼 함수를 통해 여러 프로바이더의 상세 메뉴 UI를 일관성 있게 유지하도록 했습니다.
제안 사항
ProviderMenuBuilder에서 구분선(Divider)을 추가하는 조건(didGroup)에 대해 작은 개선 제안을 드립니다. 그룹핑이 전혀 일어나지 않은 경우(모든 모델의 윈도우가 다른 경우)에도 구분선이 필요할 수 있는데, 현재 로직으로는 생략될 수 있습니다.
CI/CD 파이프라인은 모두 통과(✅)했습니다. 문서화 상태도 양호합니다.
CopilotMonitor/CopilotMonitor/Helpers/ProviderMenuBuilder.swift
Outdated
Show resolved
Hide resolved
Co-authored-by: op-gg-ai-devops[bot] <255644809+op-gg-ai-devops[bot]@users.noreply.github.com>
What
Add shared model usage grouping (same reset window + same remaining%) with horizontal dividers between groups.
Why
Gemini CLI already groups identical reset windows in the UI; extend the same behavior to other providers and keep the divider logic centralized.
Changes
ModelUsageGrouper(stricter key; no grouping when reset is missing)DetailedUsage.modelResetTimesVerification
make lint-swiftxcodebuild -scheme CopilotMonitor -configuration Debug clean build/tmp/provider_debug.log(look foradding N divider(s) between model groups)