Skip to content

test: cover tool metadata diffs#60

Open
koriyoshi2041 wants to merge 1 commit into
SamMorrowDrums:mainfrom
koriyoshi2041:rios/tool-description-diff-test
Open

test: cover tool metadata diffs#60
koriyoshi2041 wants to merge 1 commit into
SamMorrowDrums:mainfrom
koriyoshi2041:rios/tool-description-diff-test

Conversation

@koriyoshi2041

Copy link
Copy Markdown

Summary

  • Add a focused regression test for compareProbeResults()
  • Verify tool description changes are reported under the tools endpoint diff
  • Verify nested input schema description changes are also reported under the tools endpoint diff

Why

Tool descriptions and schema descriptions are part of an MCP server's public interface. This test makes that behavior explicit so future diff changes do not accidentally stop reporting metadata drift.

Validation

  • npm test -- --runInBand src/__tests__/diff.test.ts
  • npm run typecheck
  • npm run lint
  • npm run format:check
  • npm run check

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