fix(client): circular function calls leads to memory leak

[dzhulk]

Summary

Fixed circular calls of json validation in ToolsTab. checkValidationErrors() calls validateJson() which calls onChange() which calls checkValidationErrors() again.

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

[dzhulk]

proof:

[olaservo]

Thanks, LGTM - tested with a few tools with JSON form fields as well as simpler inputs.