Skip to content

Conversation

@madhav165
Copy link
Collaborator

@madhav165 madhav165 commented Sep 8, 2025

🐛 Bug-fix PR


📌 Summary

Fixes tool input from checkboxes in test tool UI.

  • Passes True when checked
  • Passes False when unchecked

💡 Fix Description

Add hidden checkboxes which are False by default for boolean fields in tool input schema

🧪 Verification

Check Command Status
Lint suite make lint pass
Unit tests make test pass
Manual regression no longer fails steps / screenshots Tested with wiki-get-article-content

📐 MCP Compliance (if relevant)

  • No breaking change to MCP clients

✅ Checklist

  • Code formatted (make black isort pre-commit)
  • No secrets/credentials committed

@madhav165 madhav165 self-assigned this Sep 8, 2025
Signed-off-by: Madhav Kandukuri <madhav165@gmail.com>
@crivetimihai crivetimihai merged commit 3a01346 into main Sep 9, 2025
32 of 34 checks passed
@crivetimihai crivetimihai deleted the fix-checkbox-2 branch September 9, 2025 08:40
madhav165 added a commit that referenced this pull request Sep 9, 2025
Signed-off-by: Madhav Kandukuri <madhav165@gmail.com>
rakdutta pushed a commit to rakdutta/mcp-context-forge that referenced this pull request Sep 10, 2025
Signed-off-by: Madhav Kandukuri <madhav165@gmail.com>
vk-playground pushed a commit to vk-playground/mcp-context-forge that referenced this pull request Sep 14, 2025
Signed-off-by: Madhav Kandukuri <madhav165@gmail.com>
vk-playground pushed a commit to vk-playground/mcp-context-forge that referenced this pull request Sep 14, 2025
Signed-off-by: Madhav Kandukuri <madhav165@gmail.com>
vk-playground pushed a commit to vk-playground/mcp-context-forge that referenced this pull request Sep 16, 2025
Signed-off-by: Madhav Kandukuri <madhav165@gmail.com>
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