mcp-windows-automation

Tool annotations

No tools have read-only/destructive annotations

Add readOnlyHint or destructiveHint annotations to every tool so hosts can warn users before invoking.

All four hints declared on every tool

442/442 tools missing one or more hints — mysql_fragmentation_extensive_analysis (missing: readOnlyHint, destructiveHint, idempotentHint, openWorldHint); mysql_index_optimization_suggestions (missing: readOnlyHint, destructiveHint, idempotentHint, openWorldHint); mysql_historical_slow_query_analysis (missing: readOnlyHint, destructiveHint, idempotentHint, openWorldHint), +439 more. OpenAI's directory rejects tools where any of the four hints are missing or non-boolean.

For every tool, set all four hints (readOnlyHint, destructiveHint, idempotentHint, openWorldHint) to explicit true/false values that match the handler’s actual behaviour.

Destructive tools are labelled

2 tools perform destructive updates without destructiveHint — clean_temp_files deletes at line 13576 (item.unlink()); sync_directories deletes at line 14259 ((target / rel_path).unlink())

Add destructiveHint:true to any tool whose handler calls .delete(), .upsert(), .update(), unlink, rm, DELETE, DROP, REPLACE INTO, or any operation that overwrites existing data.

Descriptions match behaviour

35 tools describe read intent but their handlers mutate — check_ml_system_processes (line 9089: result = subprocess.run(); wifi_profiles_list (line 11092: result = subprocess.run(); wifi_show_profile_details (line 11183: result = subprocess.run()

Rename the tool, rewrite the description, or move the side-effect into a separate clearly-named tool.

Tool inputs are validated

Only 212/442 tool handlers declare input schemas (48%)

Declare an inputSchema with zod/joi/yup on every tool definition.

Tool handlers catch errors

435/442 tool handlers wrap calls in try/catch (98%)

Wrap each tool handler body in try/catch and return a structured error response.

Tests exist

No test files found

Add tests that exercise each declared tool.

Tool description accuracy

39 tools have description/behavior mismatches: check_ml_system_processes: description implies read-only but handler writes/deletes/executes; wifi_profiles_list: description implies read-only but handler writes/deletes/executes; wifi_show_profile_details: description implies read-only but handler writes/deletes/executes

Update tool descriptions to accurately reflect all capabilities — especially write, delete, or execute operations.