fix: reject 'name' parameter for find action in all cases
- Simplify validation to reject 'name' parameter whenever present for find action - Remove ambiguity when both 'name' and 'search_term' are provided - Update error message to clarify that 'name' should be removedmain
parent
d5292567a3
commit
c38fc330d2
|
|
@ -71,10 +71,10 @@ def register_manage_gameobject_tools(mcp: FastMCP):
|
||||||
try:
|
try:
|
||||||
# Validate parameter usage to prevent silent failures
|
# Validate parameter usage to prevent silent failures
|
||||||
if action == "find":
|
if action == "find":
|
||||||
if name is not None and search_term is None:
|
if name is not None:
|
||||||
return {
|
return {
|
||||||
"success": False,
|
"success": False,
|
||||||
"message": "For 'find' action, use 'search_term' parameter, not 'name'. Example: search_term='Player', search_method='by_name'"
|
"message": "For 'find' action, use 'search_term' parameter, not 'name'. Remove 'name' parameter. Example: search_term='Player', search_method='by_name'"
|
||||||
}
|
}
|
||||||
if search_term is None:
|
if search_term is None:
|
||||||
return {
|
return {
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue