Skip to content

fix: SuperAgent 검증·테스트·저장 안내 보강 - #15

Merged
jeonsworld merged 2 commits into
mainfrom
fix/superagent-authoring-guidance
Oct 6, 2026
Merged

jeonsworld merged 2 commits into
mainfrom
fix/superagent-authoring-guidance

Conversation

@jeonsworld

@jeonsworld jeonsworld commented Oct 2, 2026 •

Copy link
Copy Markdown
Contributor

SuperAgent가 schema에 노출돼도 안전 테스트를 지원하지 않는 경우에 검증·저장 가능성과 실제 실행 성공을 구분하도록 ennoia-build-agent 안내를 보강합니다.

  • 명시적인 테스트 미지원이면 sync/async 테스트를 시작하지 않고, 검증과 저장 입력이 유효하면 요청된 저장을 진행합니다. 사용자가 테스트 성공을 저장 조건으로 정했다면 저장을 보류합니다.
  • SuperAgent 구조·이름 중복·위임 edge 규칙과 discovery/compile 지원 불일치의 처리 기준을 추가합니다. 검증 실패 우회와 사용자 모델·구성의 임의 변경을 막고 일반 graph 및 이전 schema의 테스트 경로를 유지합니다.
  • 합성 평가 입력 7개, 별도 판정 기준과 비교 결과를 추가하고 패키지를 1.4.4로 올립니다.

검증: Python 45개·Node 36개 테스트, manifest sync·패키지·평가 결과 검증, Claude strict validator 2개, git diff --check 통과. 독립 리뷰에서 중요한 이슈가 없었습니다. 합성 비교에서 기존 Skill도 핵심 행동을 충족했으므로 성능·성공률 개선은 주장하지 않습니다. 실제 host 설치·MCP 저장·runtime 실행은 검증하지 않았습니다.

PR #14(1.4.3)가 main에 병합되어 최신 main을 반영하고 충돌을 해소했습니다. 양쪽 안내와 main의 동적 버전 검사를 보존했으며, 이 PR은 1.4.4를 유지합니다. 충돌 해소 후 위 로컬 검증을 다시 통과했습니다.

관련: wanteddev/ennoia-mcp-server#184, backend 수정 wanteddev/llm-orchestrator#935.

Summary by CodeRabbit

  • 개선 사항
    • 테스트 지원 여부를 확인한 뒤 테스트를 진행하며, 지원되지 않는 경우에는 해당 그래프의 테스트를 시작하지 않습니다.
    • 안전 테스트를 지원하지 않더라도 저장 조건이 충족되면 기존 저장 절차를 진행합니다. 테스트 성공이 필수 조건이면 저장을 보류하고 조건 변경 여부를 확인합니다.
    • 구조 검증 경고와 테스트 미지원 상태를 구분해 안내하며, 검증 실패나 저장 완료 여부를 실제 응답에 근거해 판단하도록 절차를 명확히 했습니다.
  • 문서
    • SuperAgent의 그래프 구성과 검증·테스트·저장 기준을 보완했습니다.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

@coderabbitai

coderabbitai Bot commented Oct 2, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Essentials

Run ID: cbb439a9-5da0-4b11-a93f-875dafa47f69

📥 Commits

Reviewing files that changed from the base of the PR and between cee6cdd and 34c1479.

📒 Files selected for processing (14)
  • .claude-plugin/marketplace.json
  • evals/2026-10-03-superagent-summary.md
  • evals/README.md
  • evals/superagent-rubric.md
  • evals/superagent-scenarios.json
  • plugins/ennoia/.claude-plugin/plugin.json
  • plugins/ennoia/.codex-plugin/plugin.json
  • plugins/ennoia/mcp/file-uploader.mjs
  • plugins/ennoia/plugin.json
  • plugins/ennoia/skills/ennoia-build-agent/SKILL.md
  • plugins/ennoia/skills/ennoia-build-agent/references/graph-authoring.md
  • plugins/ennoia/skills/ennoia-build-agent/references/save-and-retry.md
  • tests/test_file_uploader.mjs
  • tests/test_validation.py

Included review availability: This review used your included allowance. 0 included reviews remain after this review. Your included PR review attempts over the past 7 days set your current allowance at 4 reviews per hour. Your free on-demand review promotion remains active until October 9, 2026 at 6:00 PM UTC.


Walkthrough

플러그인 버전을 1.4.4로 통일했습니다. SuperAgent의 테스트 지원, 구조 검증, 저장 조건을 설명하는 지침과 7개 합성 평가 시나리오 및 결과 문서를 추가했습니다.

Changes

플러그인 버전 동기화

Layer / File(s) Summary
버전 기준 및 검증 갱신
.claude-plugin/marketplace.json, plugins/ennoia/..., tests/test_file_uploader.mjs, tests/test_validation.py
플러그인 선언과 업로더 서버 버전을 1.4.4로 변경했습니다. 초기화 응답 테스트와 버전 드리프트 검사의 기대값도 갱신했습니다.

SuperAgent 지원 지침 및 평가

Layer / File(s) Summary
SuperAgent 검증 및 저장 지침
plugins/ennoia/skills/ennoia-build-agent/SKILL.md, plugins/ennoia/skills/ennoia-build-agent/references/*
테스트 지원 여부, 구조 전용 검증, 저장 전 유효성, 컴파일 오류와 graph 구조 규칙을 처리하는 기준을 추가했습니다.
SuperAgent 평가 시나리오와 판정 기준
evals/superagent-scenarios.json, evals/superagent-rubric.md
테스트 미지원, 구조 검증 실패, 중복 하위 agent 이름, 일반 agent의 지원 여부를 다루는 7개 합성 시나리오와 합격 기준을 추가했습니다.
SuperAgent 평가 범위 및 결과
evals/README.md, evals/2026-10-03-superagent-summary.md
평가 범위, 결과 해석의 제한, 기존 arm 처리 사례의 집계 규칙, 대상별 SHA-256 해시를 문서화했습니다.

Priority: ➖ Normal

Estimated code review effort: 3 (Moderate) | ~20 minutes

Change: Bug fix

Merge Risk: ⚪ Minimal · up to 34c14

No actionable issue is established in the supplied changes. The PR is mergeable after normal checks, subject to the stated release ordering with PR #14.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 2 functions across 3 files. (11 skipped: 1… Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed SuperAgent의 검증·테스트·저장 안내를 보강하는 주요 변경을 구체적으로 요약합니다.
Full details: Docstring Coverage

Explanation

Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 2 functions across 3 files. (11 skipped: 11 unsupported.)

  • Fix all pre-merge checks with AI
✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.


Comment @coderabbitai help to get the list of available commands.

@jeonsworld jeonsworld self-assigned this Oct 6, 2026
@jeonsworld
jeonsworld merged commit 5d82a6a into main Oct 6, 2026
6 of 7 checks passed
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