Skip to content

refactor: add reusable validation utilities for MCP tools - #315

Open
KAishwarya2429 wants to merge 2 commits into
openMF:mainfrom
KAishwarya2429:add-validation-utils
Open

refactor: add reusable validation utilities for MCP tools#315
KAishwarya2429 wants to merge 2 commits into
openMF:mainfrom
KAishwarya2429:add-validation-utils

Conversation

@KAishwarya2429

Copy link
Copy Markdown

This PR introduces reusable validation utilities for MCP tools.

Changes

  • Added validation_utils.py for common validation logic
  • Implemented validate_id and validate_required_fields
  • Added unit tests for validation utilities

Purpose

  • Reduce duplication across MCP tools
  • Improve consistency in validation and error handling
  • Enhance maintainability of the codebase

Looking forward to feedback!

@gyanLM10

gyanLM10 commented Apr 7, 2026

Copy link
Copy Markdown
Contributor

I also saw in the commit history something like “Add Smart Suggestion Engine for MCP tools” — this was already discussed in a previous PR, and the suggestion engine logic doesn’t belong in the MCP server layer.

@KAishwarya2429

Copy link
Copy Markdown
Author

Hi @gyanLM10, thanks for reviewing!
I’ll check my commit history and ensure only validation utility changes are included in this PR. If any unrelated changes are present, I will remove them and update the PR accordingly.

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.

3 participants