MCP/mcvsphere
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

add idempotentHint to safe-to-retry vCenter tools

Browse Source 
Mark tools that handle already-in-target-state gracefully:
- storage_vmotion: returns "no_migration_needed" if already on target
- convert_to_template: returns "already_template" if already converted
- convert_to_vm: returns "already_vm" if already a VM

This signals to LLM clients that these operations are safe to retry.
This commit is contained in:
Ryan Malloy 2025-12-26 06:23:35 -07:00
parent 359167ec6a
commit b9c411fdf9
1 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
src/esxi_mcp_server/mixins/vcenter_ops.py
View File

@ -23,8 +23,8 @@ class VCenterOpsMixin(MCPMixin):
@mcp_tool(
name="storage_vmotion",
description="Move a VM's disks to a different datastore (Storage vMotion)",
annotations=ToolAnnotations(destructiveHint=True),
description="Move a VM's disks to a different datastore (Storage vMotion). Idempotent if already on target.",
annotations=ToolAnnotations(destructiveHint=True, idempotentHint=True),
)
def storage_vmotion(
self,
@ -161,8 +161,8 @@ class VCenterOpsMixin(MCPMixin):
@mcp_tool(
name="convert_to_template",
description="Convert a VM to a template",
annotations=ToolAnnotations(destructiveHint=True),
description="Convert a VM to a template (idempotent - safe to call on existing template)",
annotations=ToolAnnotations(destructiveHint=True, idempotentHint=True),
)
def convert_to_template(self, vm_name: str) -> dict[str, Any]:
"""Convert a VM to a template.
@ -200,8 +200,8 @@ class VCenterOpsMixin(MCPMixin):
@mcp_tool(
name="convert_to_vm",
description="Convert a template back to a VM",
annotations=ToolAnnotations(destructiveHint=True),
description="Convert a template back to a VM (idempotent - safe to call on existing VM)",
annotations=ToolAnnotations(destructiveHint=True, idempotentHint=True),
)
def convert_to_vm(
self,