Skip to content

fix: 에이전트 목록 조회 범위 안내와 계약 검증 보완 - #21

Merged
jeonsworld merged 1 commit into
mainfrom
fix/mcp-scope-recovery-guidance
Oct 26, 2026
Merged

jeonsworld merged 1 commit into
mainfrom
fix/mcp-scope-recovery-guidance

Conversation

@jeonsworld

@jeonsworld jeonsworld commented Oct 26, 2026 •

Copy link
Copy Markdown
Contributor

구 MCP schema를 따라 list_multi_agents(scope=all)을 호출하면 운영 API에서 거절되는 문제를 줄이도록 조회 안내와 계약 검증을 보완합니다.

  • 목록 조회는 current|specified를 사용합니다. 전체 프로젝트 요청은 프로젝트별로 조회하며 저장된 현재 선택을 유지하고 소속·부분 실패·페이지를 구분합니다.
  • MCP 서버 c4301d0에서 추출한 목록 도구 schema와 정상·거절 예제를 추가합니다. all이 다시 허용되면 실패하는 회귀 검증을 포함합니다.
  • Plugin·생성 manifest·uploader 버전을 1.5.3으로 맞춥니다.

검증: Python 58개, Node uploader 36개 테스트, manifest 동기화·패키지·입출력 계약 검사 통과. 독립 합성 비교에서 기존 지침의 all 선택이 후보에서 프로젝트별 specified 조회로 바뀌었습니다. GET 405·검증 미통과 처리는 기존 지침도 올바르게 수행하여 개선 실적으로 계산하지 않았습니다. 실제 host 설치와 운영 API 실행은 검증하지 않았습니다.

서버 변경: https://github.com/wanteddev/ennoia-mcp-server/pull/241

Summary by CodeRabbit

  • 개선 사항
    • 에이전트 목록은 현재 프로젝트 또는 지정한 프로젝트 범위에서 조회하도록 안내가 업데이트되었습니다. 전체 프로젝트의 에이전트를 확인할 때는 프로젝트별 조회 결과를 모으며, 현재 선택은 유지합니다.
    • 연결 상태 확인 중 GET 요청이 405를 반환하더라도 초기화와 도구 목록 조회가 성공했다면 재로그인이나 계정 전환을 시작하지 않습니다. 검증이 유효하지 않은 결과는 저장하지 않습니다.
  • 버전 업데이트
    • Ennoia 플러그인 버전이 1.5.3으로 업데이트되었습니다.

@coderabbitai

coderabbitai Bot commented Oct 26, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Caution

Review failed

The pull request is closed.

Walkthrough

목록 조회 도구의 스키마와 입력 계약 사례를 추가했습니다. 응답 안내와 합성 평가 자료에는 조회 범위 및 연결 상태 기준을 기록했습니다. 플러그인 버전과 관련 픽스처 revision도 갱신했습니다.

Changes

목록 조회 범위와 계약

Layer / File(s) Summary
도구 계약과 검증
tests/fixtures/ennoia-tool-contracts.json, tests/fixtures/ennoia-contract-cases.json, tests/test_contracts.py, tests/fixtures/ennoia-tools.json, plugins/ennoia/..., .claude-plugin/marketplace.json
list_multi_agents와 list_ennoia_projects의 스키마 및 입력 사례를 추가했습니다. list_multi_agents의 all 범위와 지원되지 않는 추가 필드는 무효 사례로 기록하고, 계약 검증 테스트를 추가했습니다. 관련 revision과 플러그인 버전은 1.5.3으로 갱신했습니다.
범위 지침과 합성 평가
plugins/ennoia/references/response-guide.md, evals/mcp-scope-scenarios.json, evals/mcp-scope-rubric.md, evals/README.md, evals/2026-10-26-mcp-scope-summary.md
목록 API의 전체 프로젝트 조회 절차와 범위 기준을 문서화했습니다. 인증된 GET의 405 응답과 valid=false 사례를 포함한 합성 시나리오 및 평가 기준을 추가하고, 평가와 실제 API 실행을 구분했습니다.

Priority: Not assessed

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

Change: Bug fix

Possibly related PRs

Merge Risk: Minimal · up to c4db1

No actionable issue remains from this review. The change is mergeable after normal checks; the fixture’s match to the live server has not been independently verified.

Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 26, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-26T13:24:26.840019Z c4db144 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@jeonsworld jeonsworld self-assigned this Oct 26, 2026
@jeonsworld
jeonsworld merged commit b38cbba into main Oct 26, 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