Skip to content

Phase 1: CI debugging refactor — structured output, discover, capabil…#1366

Open
edvandroadolfopereira-bot wants to merge 1 commit into
modelcontextprotocol:mainfrom
edvandroadolfopereira-bot:claude/review-ci-debugging-refactor-4YbAf
Open

Phase 1: CI debugging refactor — structured output, discover, capabil…#1366
edvandroadolfopereira-bot wants to merge 1 commit into
modelcontextprotocol:mainfrom
edvandroadolfopereira-bot:claude/review-ci-debugging-refactor-4YbAf

Conversation

@edvandroadolfopereira-bot
Copy link
Copy Markdown

…ity gating, log capture

  • Add structured output envelope (--structured) with error taxonomy (transport/capability/protocol/application/validation), timing, and log capture for CI-consumable diagnostics
  • Add discover pseudo-method that returns server shape: capabilities, tools, resources, and prompts in a single invocation
  • Add capability gating: methods now fail fast with a clear category if the server lacks the required capability
  • Add --fail-on-error flag to exit 1 on application-level tool errors
  • Add ping method support
  • Capture logging/message notifications from the server and include them in structured output or emit to stderr in raw mode
  • Fix SSE transport in test server: add POST /messages route, remove double-start of SSEServerTransport (connect() already calls start() internally)
  • Add 23 tests covering all new features across stdio, HTTP, and SSE transports
  • Add CLI test step to main.yml workflow
  • Write CI_DEBUGGING_REFACTOR.md design document

Summary

Note: Inspector V2 is under development to address architectural and UX improvements. During this time, V1 contributions should focus on bug fixes and MCP spec compliance. See CONTRIBUTING.md for more details.

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Documentation update
  • Refactoring (no functional changes)
  • Test updates
  • Build/CI improvements

Changes Made

Related Issues

Testing

  • Tested in UI mode
  • Tested in CLI mode
  • Tested with STDIO transport
  • Tested with SSE transport
  • Tested with Streamable HTTP transport
  • Added/updated automated tests
  • Manual testing performed

Test Results and/or Instructions

Screenshots are encouraged to share your testing results for this change.

Checklist

  • Code follows the style guidelines (ran npm run prettier-fix)
  • Self-review completed
  • Code is commented where necessary
  • Documentation updated (README, comments, etc.)

Breaking Changes

Additional Context

…ity gating, log capture

- Add structured output envelope (--structured) with error taxonomy
  (transport/capability/protocol/application/validation), timing, and
  log capture for CI-consumable diagnostics
- Add discover pseudo-method that returns server shape: capabilities,
  tools, resources, and prompts in a single invocation
- Add capability gating: methods now fail fast with a clear category
  if the server lacks the required capability
- Add --fail-on-error flag to exit 1 on application-level tool errors
- Add ping method support
- Capture logging/message notifications from the server and include
  them in structured output or emit to stderr in raw mode
- Fix SSE transport in test server: add POST /messages route,
  remove double-start of SSEServerTransport (connect() already calls
  start() internally)
- Add 23 tests covering all new features across stdio, HTTP, and SSE
  transports
- Add CLI test step to main.yml workflow
- Write CI_DEBUGGING_REFACTOR.md design document
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