fix(cmdb-assistant): complete MCP tool schemas for update operations #138

lmiranda · 2026-01-24T17:19:29Z

lmiranda commented

2026-01-24 17:19:29 +00:00

Summary

Expand parameter schemas for 15 update tools that previously only exposed the id field
The underlying Python implementation already supported all fields via **kwargs, but Claude could not discover available parameters
Each tool now exposes all commonly-used optional fields matching the NetBox API

Updated Tools

Virtualization:

update_virtual_machine - Added: name, status, cluster, site, role, tenant, platform, vcpus, memory, disk, primary_ip4, primary_ip6, description, comments
update_cluster - Added: name, type, group, site, status, tenant, description, comments

DCIM:

update_site, update_location, update_rack, update_manufacturer, update_device_type, update_device_role, update_platform, update_interface, update_cable

IPAM:

update_vrf, update_prefix, update_ip_address, update_vlan

Test plan

Reinstall plugin in test project
Run ToolSearch for virt_update_vm and verify schema shows all new fields
Test updating a VM with description/comments fields
Verify no regressions in existing functionality

Fixes #137

🤖 Generated with Claude Code

## Summary - Expand parameter schemas for 15 update tools that previously only exposed the `id` field - The underlying Python implementation already supported all fields via `**kwargs`, but Claude could not discover available parameters - Each tool now exposes all commonly-used optional fields matching the NetBox API ## Updated Tools **Virtualization:** - `update_virtual_machine` - Added: name, status, cluster, site, role, tenant, platform, vcpus, memory, disk, primary_ip4, primary_ip6, description, comments - `update_cluster` - Added: name, type, group, site, status, tenant, description, comments **DCIM:** - `update_site`, `update_location`, `update_rack`, `update_manufacturer`, `update_device_type`, `update_device_role`, `update_platform`, `update_interface`, `update_cable` **IPAM:** - `update_vrf`, `update_prefix`, `update_ip_address`, `update_vlan` ## Test plan - [ ] Reinstall plugin in test project - [ ] Run `ToolSearch` for `virt_update_vm` and verify schema shows all new fields - [ ] Test updating a VM with description/comments fields - [ ] Verify no regressions in existing functionality Fixes #137 🤖 Generated with [Claude Code](https://claude.com/claude-code)

lmiranda added 1 commit 2026-01-24 17:19:30 +00:00

fix(cmdb-assistant): complete MCP tool schemas for update operations 70d6963d0d

Expand parameter schemas for 15 update tools that previously only exposed
the 'id' field. The underlying Python implementation already supported
all fields via **kwargs, but Claude couldn't discover available parameters.

Updated tools:
- virtualization: update_virtual_machine, update_cluster
- dcim: update_site, update_location, update_rack, update_manufacturer,
        update_device_type, update_device_role, update_platform,
        update_interface, update_cable
- ipam: update_vrf, update_prefix, update_ip_address, update_vlan

Each tool now exposes all commonly-used optional fields matching the
NetBox API, following the pattern established by dcim_update_device.

Fixes #137

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

lmiranda referenced this pull request

2026-01-24 17:19:46 +00:00

[Diagnostic] cmdb-assistant: Incomplete parameter schemas for update operations #137

lmiranda merged commit 98df35a33e into development

2026-01-24 17:22:01 +00:00

lmiranda deleted branch fix/issue-137-update-schema-completeness

2026-01-24 17:22:01 +00:00

lmiranda referenced this issue from a commit

2026-01-24 17:22:02 +00:00

Merge pull request 'fix(cmdb-assistant): complete MCP tool schemas for update operations' (#138) from fix/issue-137-update-schema-completeness into development

Sign in to join this conversation.