Skip to content

Conversation

azu
Copy link
Member

@azu azu commented Jul 3, 2025

fix #1622

azu and others added 2 commits July 3, 2025 23:14
Fixes schema validation errors in MCP server when TextlintMessage
contains additional properties like type, data, index, range, and loc.

- Add explicit properties for all TextlintMessage fields
- Enable additionalProperties via .passthrough() for future extensibility
- Add comprehensive test coverage for schema validation
- Ensure compatibility with complete TextlintMessage interface

Resolves #1622

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
Comment on lines -148 to +149
"additionalProperties": false,
"additionalProperties": true,
Copy link
Member Author

Choose a reason for hiding this comment

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

This is main change.

@azu azu added the Type: Bug Bug or Bug fixes label Jul 3, 2025
@azu azu marked this pull request as ready for review July 3, 2025 14:36
@azu azu merged commit 64c557c into master Jul 3, 2025
17 checks passed
@azu azu deleted the fix-mcp-schema branch July 3, 2025 14:38
@github-actions github-actions bot mentioned this pull request Jul 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Bug Bug or Bug fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

MCP Validation Error occurs in some cases when using MCP server
1 participant