Skip to content

feat: convert package-evaluator from skill to agent#19

Merged
yujeong-jeon merged 1 commit into
mainfrom
feature/18
May 13, 2026
Merged

feat: convert package-evaluator from skill to agent#19
yujeong-jeon merged 1 commit into
mainfrom
feature/18

Conversation

@yujeong-jeon

Copy link
Copy Markdown
Contributor

Related Issue

Describe your changes

package-evaluator 플러그인을 skill (command) 기반에서 agent 기반으로 변환했습니다.

주요 변경사항

  1. 아키텍처 변경

    • commands/evaluate-package.md (skill) 제거
    • agents/evaluate-package.md (agent) 생성
    • plugin.jsonagents 필드 추가
  2. Agent 기능 강화

    • Trigger 설정: 패키지 설치 논의 시 자동 호출
    • LLM 통합: 설치 전 평가 결과 확인 필수
    • 자동 경로 탐지: 스크립트 경로 자동 감지
    • 명확한 출력: 메트릭 + 판정 + 다음 액션
  3. 사용성 개선

    • README 업데이트: 자연어 사용법으로 변경
    • 에러 처리: Chrome CDP 미실행 시 명확한 안내
    • 규칙 강화: "평가 없이 설치하지 말 것" 명시

Request

  • Agent 기반 구조와 trigger 설정이 적절한지 검토 부탁드립니다
  • LLM이 자동으로 호출될 수 있는지 확인해주세요

@yujeong-jeon yujeong-jeon requested a review from a team as a code owner May 11, 2026 07:29

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ Code review skipped — your organization's overage spend limit has been reached.

Code review is billed via overage credits. To resume reviews, an organization admin can raise the monthly limit at claude.ai/admin-settings/claude-code.

Once credits are available, push a new commit or reopen this pull request to trigger a review.

@Ohjaeyeop Ohjaeyeop left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

확인했습니다!

@yujeong-jeon yujeong-jeon merged commit ee6f928 into main May 13, 2026
3 checks passed
@yujeong-jeon yujeong-jeon deleted the feature/18 branch May 13, 2026 05:53
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.

2 participants