Commit 8fcbb99
Fix boolean parameter handling in ListRoutes MCP tool (#279)
* Fix boolean parameter handling in ListRoutes MCP tool
The ListRoutes tool was throwing a type error when boolean parameters
(except_vendor, only_vendor) were passed because str_replace() was
being called on boolean values. This fix adds type checking to handle
boolean parameters separately from string parameters.
The error was: "str_replace(): Argument #3 ($subject) must be of type
array|string, true given"
Boolean parameters are now properly passed to the artisan command
without string manipulation, while string parameters continue to have
wildcard characters sanitized.
* Fix boolean parameter handling in ListRoutes MCP tool
The ListRoutes tool was throwing a type error when boolean parameters
(except_vendor, only_vendor) were passed because str_replace() was
being called on boolean values. This fix adds type checking to handle
boolean parameters separately from string parameters.
The error was: "str_replace(): Argument #3 ($subject) must be of type
array|string, true given"
Boolean parameters are now properly passed to the artisan command
without string manipulation, while string parameters continue to have
wildcard characters sanitized.
---------
Co-authored-by: systempath <[email protected]>
Co-authored-by: Pushpak Chhajed <[email protected]>1 parent 3eb3dcc commit 8fcbb99
1 file changed
+7
-3
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
64 | 64 | | |
65 | 65 | | |
66 | 66 | | |
67 | | - | |
68 | | - | |
69 | | - | |
| 67 | + | |
| 68 | + | |
| 69 | + | |
| 70 | + | |
| 71 | + | |
| 72 | + | |
| 73 | + | |
70 | 74 | | |
71 | 75 | | |
72 | 76 | | |
| |||
0 commit comments