Skip to content

Display provider icons in status bar text#87

Merged
kargnas merged 3 commits intomainfrom
codex-replace-provider-names-with-icons
Feb 19, 2026
Merged

Display provider icons in status bar text#87
kargnas merged 3 commits intomainfrom
codex-replace-provider-names-with-icons

Conversation

@kargnas
Copy link
Member

@kargnas kargnas commented Feb 19, 2026

Summary

  • update the toggle label to refer to provider icons and keep existing behavior for provider names
  • strip provider names from status text formatting and render icons alongside usage/cost values via the icon view
  • add logging around status updates and keep icon-only handling synchronized with icon state

Testing

  • Not run (not requested)

@op-gg-ai-devops
Copy link
Contributor

상태바에 텍스트로만 나오던 공급자 이름을 아이콘으로도 표시할 수 있게 StatusBarIconView 드로잉 로직이랑 컨트롤러를 싹 고쳤네.


텍스트 대신 아이콘 박아넣는 거 아주 깔끔하고 좋아, 공간 절약도 되고 시인성도 좋아지겠네. drawTintedIcon에서 매번 이미지 새로 그리고 lockFocus 하는 건 상태바 업데이트 빈도 생각하면 당장은 괜찮겠지만, 나중에 호출 빈도 늘어나면 캐싱 고려해봐. 그리고 providerIconSize 12, yOffset 같은 수치들 하드코딩 된 거 나중에 MenuDesignToken 같은 걸로 싹 정리하면 유지보수하기 더 편할 것 같아. formatAlertText에서 안 쓰는 파라미터(_) 깔끔하게 처리한 건 굿잡.


이건 그냥 짧은 프리뷰일 뿐이야. 버그 체크나 보안 점검 같은 디테일한 라인별 리뷰가 필요하면 @opgginc/op-gg-ai-devops 리뷰해줘라고 댓글 달거나, @opgginc/op-gg-ai-devops 팀을 리뷰어로 추가해줘(커밋 푸시할 때마다 자동으로 리뷰해줌).

@kargnas
Copy link
Member Author

kargnas commented Feb 19, 2026

@opgginc/op-gg-ai-devops

@op-gg-ai-devops
Copy link
Contributor

op-gg-ai-devops bot commented Feb 19, 2026

✅ AI Code Review Completed

리뷰 완료했고 중대한 버그/보안/성능 이슈는 발견되지 않음. README.md 문구 수정 커밋은 로컬에만 있고 maintainerCanModify=false라 푸시 못했음. CI는 Build and Release가 진행 중임.

Reviewed with openai/gpt-5.2-codex


📋 View Full Logs

Copy link
Contributor

@op-gg-ai-devops op-gg-ai-devops bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 OP.GG DevOps AI Code Review

변경 요약

  • 상태바 텍스트에서 공급자 이름을 제거하고 아이콘 기반 표시로 전환됨.
  • Gemini 아이콘 크기 토큰 추가 및 상태바 렌더링에 공급자 아이콘 레이어 추가됨.

리스크/이슈

  • 중대한 버그, 보안, 성능 문제는 발견되지 않음.

문서 업데이트

  • README.md의 "Show Provider Name" 항목을 "Show Provider Icon"으로 수정하는 커밋을 로컬에서 생성했지만, maintainerCanModify=false라 PR 브랜치에 푸시할 수 없음. 작성자가 동일 변경을 반영하거나 수정 권한을 활성화해야 함.

테스트/검증

  • 패키지 매니저 락파일이 없어 의존성 설치를 수행하지 못함.
  • Linux 환경에서 Xcode 빌드/테스트를 실행할 수 없어 테스트 미실행.

CI/CD 상태

  • Build and Release: 진행 중 (최종 상태 확인 필요).
  • 나머지 워크플로는 성공.

리뷰 언어 설정 안내 (AGENTS.md)

  • AGENTS.md에 기본 리뷰 언어를 명시해줘. 예: Default review language: English

이 PR에 새 커밋이 푸시되면 전체 커밋 기준으로 자동 재리뷰됨. PR 댓글에 @opgginc/op-gg-ai-devops 멘션하면 다시 리뷰함. 예: @opgginc/op-gg-ai-devops README 문구 확인, @opgginc/op-gg-ai-devops 상태바 아이콘 크기 검토, @opgginc/op-gg-ai-devops StatusBarIconView 렌더링 비용 점검

@kargnas kargnas assigned kargnas and unassigned kargnas Feb 19, 2026
@kargnas kargnas merged commit 745650b into main Feb 19, 2026
11 checks passed
@kargnas kargnas deleted the codex-replace-provider-names-with-icons branch February 19, 2026 10:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant