Merge pull request 'development' (#133) from development into main

Reviewed-on: #133

.claude-plugin/marketplace.json

@@ -6,7 +6,7 @@
   },
   "metadata": {
     "description": "Project management plugins with Gitea and NetBox integrations",
-    "version": "3.2.0"
+    "version": "3.1.0"
   },
   "plugins": [
     {

CHANGELOG.md

@@ -4,23 +4,14 @@ All notable changes to the Leo Claude Marketplace will be documented in this fil
 
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
 
-## [Unreleased]
+## [3.1.2] - 2026-01-23
-
-*Changes staged for the next release*
-
----
-
-## [3.2.0] - 2026-01-24
 
 ### Added
 - **git-flow:** `/commit` now detects protected branches before committing
   - Warns when on protected branch (main, master, development, staging, production)
   - Offers to create feature branch automatically instead of committing directly
   - Configurable via `GIT_PROTECTED_BRANCHES` environment variable
-- **netbox:** Platform and primary_ip parameters added to device update tools
+  - Resolves issue where commits to protected branches would fail on push
-- **claude-config-maintainer:** Auto-enforce mandatory behavior rules via SessionStart hook
-- **scripts:** `release.sh` - Versioning workflow script for consistent releases
-- **scripts:** `verify-hooks.sh` - Verify all hooks are command type
 
 ### Changed
 - **doc-guardian:** Hook switched from `prompt` type to `command` type
@@ -28,24 +19,14 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
   - New `notify.sh` bash script guarantees exact output behavior
   - Only notifies for config file changes (commands/, agents/, skills/, hooks/)
   - Silent exit for all other files - no blocking possible
-- **All hooks:** Converted to command type with stricter plugin prefix enforcement
+- **All hooks:** Stricter plugin prefix enforcement
-  - All hooks now mandate `[plugin-name]` prefix with "NO EXCEPTIONS" rule
+  - All prompts now mandate `[plugin-name]` prefix with "NO EXCEPTIONS" rule
   - Simplified output formats with word limits
   - Consistent structure across projman, pr-review, code-sentinel, doc-guardian
-- **CLAUDE.md:** Replaced destructive "ALWAYS CLEAR CACHE" rule with "VERIFY AND RESTART"
-  - Cache clearing mid-session breaks MCP tools
-  - Added guidance for proper plugin development workflow
 
 ### Fixed
-- **cmdb-assistant:** Complete MCP tool schemas for update operations (#138)
+- Protected branch workflow: Claude no longer commits directly to protected branches and then fails on push (fixes #109)
-- **netbox:** Shorten tool names to meet 64-char API limit (#134)
+- doc-guardian hook no longer blocks workflow - switched to command hook that can't be overridden by model (fixes #110)
-- **cmdb-assistant:** Correct NetBox API URL format in setup wizard (#132)
-- **gitea/projman:** Type safety for `create_label_smart`, curl-based debug-report (#124)
-- **netbox:** Add diagnostic logging for JSON parse errors (#121)
-- **labels:** Add duplicate check before creating labels (#116)
-- **hooks:** Convert ALL hooks to command type with proper prefixes (#114)
-- Protected branch workflow: Claude no longer commits directly to protected branches (fixes #109)
-- doc-guardian hook no longer blocks workflow (fixes #110)
 
 ---
 

CLAUDE.md

@@ -286,56 +286,13 @@ See `docs/DEBUGGING-CHECKLIST.md` for systematic troubleshooting.
 - `/debug-report` - Run full diagnostics, create issue if needed
 - `/debug-review` - Investigate and propose fixes
 
-## Versioning Workflow
+## Versioning Rules
 
-This project follows [SemVer](https://semver.org/) and [Keep a Changelog](https://keepachangelog.com).
+- Version displayed ONLY in main `README.md` title: `# Leo Claude Marketplace - vX.Y.Z`
-
+- `CHANGELOG.md` is authoritative for version history
-### Version Locations (must stay in sync)
+- Follow [SemVer](https://semver.org/): MAJOR.MINOR.PATCH
-
+- On release: Update README title → CHANGELOG → marketplace.json → plugin.json files
-| Location | Format | Example |
-|----------|--------|---------|
-| Git tags | `vX.Y.Z` | `v3.2.0` |
-| README.md title | `# Leo Claude Marketplace - vX.Y.Z` | `v3.2.0` |
-| marketplace.json | `"version": "X.Y.Z"` | `3.2.0` |
-| CHANGELOG.md | `## [X.Y.Z] - YYYY-MM-DD` | `[3.2.0] - 2026-01-24` |
-
-### During Development
-
-**All changes go under `[Unreleased]` in CHANGELOG.md.** Never create a versioned section until release time.
-
-```markdown
-## [Unreleased]
-
-### Added
-- New feature description
-
-### Fixed
-- Bug fix description
-```
-
-### Creating a Release
-
-Use the release script to ensure consistency:
-
-```bash
-./scripts/release.sh 3.2.0
-```
-
-The script will:
-1. Validate `[Unreleased]` section has content
-2. Replace `[Unreleased]` with `[3.2.0] - YYYY-MM-DD`
-3. Update README.md title
-4. Update marketplace.json version
-5. Commit and create git tag
-
-### SemVer Guidelines
-
-| Change Type | Version Bump | Example |
-|-------------|--------------|---------|
-| Bug fixes only | PATCH (x.y.**Z**) | 3.1.1 → 3.1.2 |
-| New features (backwards compatible) | MINOR (x.**Y**.0) | 3.1.2 → 3.2.0 |
-| Breaking changes | MAJOR (**X**.0.0) | 3.2.0 → 4.0.0 |
 
 ---
 
-**Last Updated:** 2026-01-24
+**Last Updated:** 2026-01-23

README.md

@@ -1,4 +1,4 @@
-# Leo Claude Marketplace - v3.2.0
+# Leo Claude Marketplace - v3.1.1
 
 A collection of Claude Code plugins for project management, infrastructure automation, and development workflows.
 
@@ -106,11 +106,11 @@ Full Gitea API integration for project management.
 
 | Category | Tools |
 |----------|-------|
-| Issues | `list_issues`, `get_issue`, `create_issue`, `update_issue`, `add_comment`, `aggregate_issues` |
+| Issues | `list_issues`, `get_issue`, `create_issue`, `update_issue`, `add_comment` |
-| Labels | `get_labels`, `suggest_labels`, `create_label`, `create_label_smart` |
+| Labels | `get_labels`, `suggest_labels`, `create_label` |
-| Wiki | `list_wiki_pages`, `get_wiki_page`, `create_wiki_page`, `update_wiki_page`, `create_lesson`, `search_lessons` |
+| Wiki | `list_wiki_pages`, `get_wiki_page`, `create_wiki_page`, `create_lesson`, `search_lessons` |
-| Milestones | `list_milestones`, `get_milestone`, `create_milestone`, `update_milestone`, `delete_milestone` |
+| Milestones | `list_milestones`, `get_milestone`, `create_milestone`, `update_milestone` |
-| Dependencies | `list_issue_dependencies`, `create_issue_dependency`, `remove_issue_dependency`, `get_execution_order` |
+| Dependencies | `list_issue_dependencies`, `create_issue_dependency`, `get_execution_order` |
 | **Pull Requests** | `list_pull_requests`, `get_pull_request`, `get_pr_diff`, `get_pr_comments`, `create_pr_review`, `add_pr_comment` *(NEW in v3.0.0)* |
 | Validation | `validate_repo_org`, `get_branch_protection` |
 
@@ -245,8 +245,7 @@ leo-claude-mktplace/
 ├── docs/                          # Documentation
 │   ├── CANONICAL-PATHS.md         # Path reference
 │   └── CONFIGURATION.md           # Setup guide
-├── scripts/                       # Setup scripts
+└── scripts/                       # Setup scripts
-└── CHANGELOG.md                   # Version history
 ```
 
 ## Documentation

mcp-servers/netbox/mcp_server/server.py

@@ -103,19 +103,7 @@ TOOL_DEFINITIONS = {
         'properties': {
             'id': {'type': 'integer', 'description': 'Site ID'},
             'name': {'type': 'string', 'description': 'New name'},
-            'slug': {'type': 'string', 'description': 'New slug'},
+            'status': {'type': 'string', 'description': 'New status'}
-            'status': {'type': 'string', 'description': 'Status'},
-            'region': {'type': 'integer', 'description': 'Region ID'},
-            'group': {'type': 'integer', 'description': 'Site group ID'},
-            'tenant': {'type': 'integer', 'description': 'Tenant ID'},
-            'facility': {'type': 'string', 'description': 'Facility name'},
-            'time_zone': {'type': 'string', 'description': 'Time zone'},
-            'description': {'type': 'string', 'description': 'Description'},
-            'physical_address': {'type': 'string', 'description': 'Physical address'},
-            'shipping_address': {'type': 'string', 'description': 'Shipping address'},
-            'latitude': {'type': 'number', 'description': 'Latitude'},
-            'longitude': {'type': 'number', 'description': 'Longitude'},
-            'comments': {'type': 'string', 'description': 'Comments'}
         },
         'required': ['id']
     },
@@ -148,14 +136,7 @@ TOOL_DEFINITIONS = {
     },
     'dcim_update_location': {
         'description': 'Update an existing location',
-        'properties': {
+        'properties': {'id': {'type': 'integer', 'description': 'Location ID'}},
-            'id': {'type': 'integer', 'description': 'Location ID'},
-            'name': {'type': 'string', 'description': 'New name'},
-            'slug': {'type': 'string', 'description': 'New slug'},
-            'site': {'type': 'integer', 'description': 'Site ID'},
-            'parent': {'type': 'integer', 'description': 'Parent location ID'},
-            'description': {'type': 'string', 'description': 'Description'}
-        },
         'required': ['id']
     },
     'dcim_delete_location': {
@@ -190,18 +171,7 @@ TOOL_DEFINITIONS = {
     },
     'dcim_update_rack': {
         'description': 'Update an existing rack',
-        'properties': {
+        'properties': {'id': {'type': 'integer', 'description': 'Rack ID'}},
-            'id': {'type': 'integer', 'description': 'Rack ID'},
-            'name': {'type': 'string', 'description': 'New name'},
-            'site': {'type': 'integer', 'description': 'Site ID'},
-            'location': {'type': 'integer', 'description': 'Location ID'},
-            'status': {'type': 'string', 'description': 'Status'},
-            'role': {'type': 'integer', 'description': 'Role ID'},
-            'tenant': {'type': 'integer', 'description': 'Tenant ID'},
-            'u_height': {'type': 'integer', 'description': 'Rack height in U'},
-            'description': {'type': 'string', 'description': 'Description'},
-            'comments': {'type': 'string', 'description': 'Comments'}
-        },
         'required': ['id']
     },
     'dcim_delete_rack': {
@@ -228,12 +198,7 @@ TOOL_DEFINITIONS = {
     },
     'dcim_update_manufacturer': {
         'description': 'Update an existing manufacturer',
-        'properties': {
+        'properties': {'id': {'type': 'integer', 'description': 'Manufacturer ID'}},
-            'id': {'type': 'integer', 'description': 'Manufacturer ID'},
-            'name': {'type': 'string', 'description': 'New name'},
-            'slug': {'type': 'string', 'description': 'New slug'},
-            'description': {'type': 'string', 'description': 'Description'}
-        },
         'required': ['id']
     },
     'dcim_delete_manufacturer': {
@@ -265,16 +230,7 @@ TOOL_DEFINITIONS = {
     },
     'dcim_update_device_type': {
         'description': 'Update an existing device type',
-        'properties': {
+        'properties': {'id': {'type': 'integer', 'description': 'Device type ID'}},
-            'id': {'type': 'integer', 'description': 'Device type ID'},
-            'manufacturer': {'type': 'integer', 'description': 'Manufacturer ID'},
-            'model': {'type': 'string', 'description': 'Model name'},
-            'slug': {'type': 'string', 'description': 'New slug'},
-            'u_height': {'type': 'number', 'description': 'Height in rack units'},
-            'is_full_depth': {'type': 'boolean', 'description': 'Is full depth'},
-            'description': {'type': 'string', 'description': 'Description'},
-            'comments': {'type': 'string', 'description': 'Comments'}
-        },
         'required': ['id']
     },
     'dcim_delete_device_type': {
@@ -303,14 +259,7 @@ TOOL_DEFINITIONS = {
     },
     'dcim_update_device_role': {
         'description': 'Update an existing device role',
-        'properties': {
+        'properties': {'id': {'type': 'integer', 'description': 'Device role ID'}},
-            'id': {'type': 'integer', 'description': 'Device role ID'},
-            'name': {'type': 'string', 'description': 'New name'},
-            'slug': {'type': 'string', 'description': 'New slug'},
-            'color': {'type': 'string', 'description': 'Hex color code'},
-            'vm_role': {'type': 'boolean', 'description': 'Can be assigned to VMs'},
-            'description': {'type': 'string', 'description': 'Description'}
-        },
         'required': ['id']
     },
     'dcim_delete_device_role': {
@@ -341,13 +290,7 @@ TOOL_DEFINITIONS = {
     },
     'dcim_update_platform': {
         'description': 'Update an existing platform',
-        'properties': {
+        'properties': {'id': {'type': 'integer', 'description': 'Platform ID'}},
-            'id': {'type': 'integer', 'description': 'Platform ID'},
-            'name': {'type': 'string', 'description': 'New name'},
-            'slug': {'type': 'string', 'description': 'New slug'},
-            'manufacturer': {'type': 'integer', 'description': 'Manufacturer ID'},
-            'description': {'type': 'string', 'description': 'Description'}
-        },
         'required': ['id']
     },
     'dcim_delete_platform': {
@@ -443,18 +386,7 @@ TOOL_DEFINITIONS = {
     },
     'dcim_update_interface': {
         'description': 'Update an existing interface',
-        'properties': {
+        'properties': {'id': {'type': 'integer', 'description': 'Interface ID'}},
-            'id': {'type': 'integer', 'description': 'Interface ID'},
-            'name': {'type': 'string', 'description': 'New name'},
-            'type': {'type': 'string', 'description': 'Interface type'},
-            'enabled': {'type': 'boolean', 'description': 'Interface enabled'},
-            'mtu': {'type': 'integer', 'description': 'MTU'},
-            'mac_address': {'type': 'string', 'description': 'MAC address'},
-            'description': {'type': 'string', 'description': 'Description'},
-            'mode': {'type': 'string', 'description': 'VLAN mode'},
-            'untagged_vlan': {'type': 'integer', 'description': 'Untagged VLAN ID'},
-            'tagged_vlans': {'type': 'array', 'description': 'Tagged VLAN IDs'}
-        },
         'required': ['id']
     },
     'dcim_delete_interface': {
@@ -488,15 +420,7 @@ TOOL_DEFINITIONS = {
     },
     'dcim_update_cable': {
         'description': 'Update an existing cable',
-        'properties': {
+        'properties': {'id': {'type': 'integer', 'description': 'Cable ID'}},
-            'id': {'type': 'integer', 'description': 'Cable ID'},
-            'type': {'type': 'string', 'description': 'Cable type'},
-            'status': {'type': 'string', 'description': 'Cable status'},
-            'label': {'type': 'string', 'description': 'Cable label'},
-            'color': {'type': 'string', 'description': 'Cable color'},
-            'length': {'type': 'number', 'description': 'Cable length'},
-            'length_unit': {'type': 'string', 'description': 'Length unit'}
-        },
         'required': ['id']
     },
     'dcim_delete_cable': {
@@ -584,15 +508,7 @@ TOOL_DEFINITIONS = {
     },
     'ipam_update_vrf': {
         'description': 'Update an existing VRF',
-        'properties': {
+        'properties': {'id': {'type': 'integer', 'description': 'VRF ID'}},
-            'id': {'type': 'integer', 'description': 'VRF ID'},
-            'name': {'type': 'string', 'description': 'New name'},
-            'rd': {'type': 'string', 'description': 'Route distinguisher'},
-            'tenant': {'type': 'integer', 'description': 'Tenant ID'},
-            'enforce_unique': {'type': 'boolean', 'description': 'Enforce unique IPs'},
-            'description': {'type': 'string', 'description': 'Description'},
-            'comments': {'type': 'string', 'description': 'Comments'}
-        },
         'required': ['id']
     },
     'ipam_delete_vrf': {
@@ -631,19 +547,7 @@ TOOL_DEFINITIONS = {
     },
     'ipam_update_prefix': {
         'description': 'Update an existing prefix',
-        'properties': {
+        'properties': {'id': {'type': 'integer', 'description': 'Prefix ID'}},
-            'id': {'type': 'integer', 'description': 'Prefix ID'},
-            'prefix': {'type': 'string', 'description': 'Prefix in CIDR notation'},
-            'status': {'type': 'string', 'description': 'Status'},
-            'site': {'type': 'integer', 'description': 'Site ID'},
-            'vrf': {'type': 'integer', 'description': 'VRF ID'},
-            'vlan': {'type': 'integer', 'description': 'VLAN ID'},
-            'role': {'type': 'integer', 'description': 'Role ID'},
-            'tenant': {'type': 'integer', 'description': 'Tenant ID'},
-            'is_pool': {'type': 'boolean', 'description': 'Is a pool'},
-            'description': {'type': 'string', 'description': 'Description'},
-            'comments': {'type': 'string', 'description': 'Comments'}
-        },
         'required': ['id']
     },
     'ipam_delete_prefix': {
@@ -694,18 +598,7 @@ TOOL_DEFINITIONS = {
     },
     'ipam_update_ip_address': {
         'description': 'Update an existing IP address',
-        'properties': {
+        'properties': {'id': {'type': 'integer', 'description': 'IP address ID'}},
-            'id': {'type': 'integer', 'description': 'IP address ID'},
-            'address': {'type': 'string', 'description': 'IP address with prefix length'},
-            'status': {'type': 'string', 'description': 'Status'},
-            'vrf': {'type': 'integer', 'description': 'VRF ID'},
-            'tenant': {'type': 'integer', 'description': 'Tenant ID'},
-            'dns_name': {'type': 'string', 'description': 'DNS name'},
-            'description': {'type': 'string', 'description': 'Description'},
-            'comments': {'type': 'string', 'description': 'Comments'},
-            'assigned_object_type': {'type': 'string', 'description': 'Object type to assign to'},
-            'assigned_object_id': {'type': 'integer', 'description': 'Object ID to assign to'}
-        },
         'required': ['id']
     },
     'ipam_delete_ip_address': {
@@ -770,18 +663,7 @@ TOOL_DEFINITIONS = {
     },
     'ipam_update_vlan': {
         'description': 'Update an existing VLAN',
-        'properties': {
+        'properties': {'id': {'type': 'integer', 'description': 'VLAN ID'}},
-            'id': {'type': 'integer', 'description': 'VLAN ID'},
-            'vid': {'type': 'integer', 'description': 'VLAN ID number'},
-            'name': {'type': 'string', 'description': 'VLAN name'},
-            'status': {'type': 'string', 'description': 'Status'},
-            'site': {'type': 'integer', 'description': 'Site ID'},
-            'group': {'type': 'integer', 'description': 'VLAN group ID'},
-            'tenant': {'type': 'integer', 'description': 'Tenant ID'},
-            'role': {'type': 'integer', 'description': 'Role ID'},
-            'description': {'type': 'string', 'description': 'Description'},
-            'comments': {'type': 'string', 'description': 'Comments'}
-        },
         'required': ['id']
     },
     'ipam_delete_vlan': {
@@ -891,17 +773,16 @@ TOOL_DEFINITIONS = {
         'properties': {'id': {'type': 'integer', 'description': 'Provider ID'}},
         'required': ['id']
     },
-    # NOTE: circuit_types tools shortened to meet 28-char limit
+    'circuits_list_circuit_types': {
-    'circ_list_types': {
         'description': 'List all circuit types in NetBox',
         'properties': {'name': {'type': 'string', 'description': 'Filter by name'}}
     },
-    'circ_get_type': {
+    'circuits_get_circuit_type': {
         'description': 'Get a specific circuit type by ID',
         'properties': {'id': {'type': 'integer', 'description': 'Circuit type ID'}},
         'required': ['id']
     },
-    'circ_create_type': {
+    'circuits_create_circuit_type': {
         'description': 'Create a new circuit type',
         'properties': {
             'name': {'type': 'string', 'description': 'Type name'},
@@ -944,20 +825,19 @@ TOOL_DEFINITIONS = {
         'properties': {'id': {'type': 'integer', 'description': 'Circuit ID'}},
         'required': ['id']
     },
-    # NOTE: circuit_terminations tools shortened to meet 28-char limit
+    'circuits_list_circuit_terminations': {
-    'circ_list_terminations': {
         'description': 'List all circuit terminations in NetBox',
         'properties': {
             'circuit_id': {'type': 'integer', 'description': 'Filter by circuit ID'},
             'site_id': {'type': 'integer', 'description': 'Filter by site ID'}
         }
     },
-    'circ_get_termination': {
+    'circuits_get_circuit_termination': {
         'description': 'Get a specific circuit termination by ID',
         'properties': {'id': {'type': 'integer', 'description': 'Termination ID'}},
         'required': ['id']
     },
-    'circ_create_termination': {
+    'circuits_create_circuit_termination': {
         'description': 'Create a new circuit termination',
         'properties': {
             'circuit': {'type': 'integer', 'description': 'Circuit ID'},
@@ -968,18 +848,16 @@ TOOL_DEFINITIONS = {
     },
 
     # ==================== Virtualization Tools ====================
-    # NOTE: Tool names shortened from 'virtualization_' to 'virt_' to meet
+    'virtualization_list_cluster_types': {
-    # 28-char limit (Claude API 64-char limit minus 36-char prefix)
-    'virt_list_cluster_types': {
         'description': 'List all cluster types in NetBox',
         'properties': {'name': {'type': 'string', 'description': 'Filter by name'}}
     },
-    'virt_get_cluster_type': {
+    'virtualization_get_cluster_type': {
         'description': 'Get a specific cluster type by ID',
         'properties': {'id': {'type': 'integer', 'description': 'Cluster type ID'}},
         'required': ['id']
     },
-    'virt_create_cluster_type': {
+    'virtualization_create_cluster_type': {
         'description': 'Create a new cluster type',
         'properties': {
             'name': {'type': 'string', 'description': 'Type name'},
@@ -987,16 +865,16 @@ TOOL_DEFINITIONS = {
         },
         'required': ['name', 'slug']
     },
-    'virt_list_cluster_groups': {
+    'virtualization_list_cluster_groups': {
         'description': 'List all cluster groups in NetBox',
         'properties': {'name': {'type': 'string', 'description': 'Filter by name'}}
     },
-    'virt_get_cluster_group': {
+    'virtualization_get_cluster_group': {
         'description': 'Get a specific cluster group by ID',
         'properties': {'id': {'type': 'integer', 'description': 'Cluster group ID'}},
         'required': ['id']
     },
-    'virt_create_cluster_group': {
+    'virtualization_create_cluster_group': {
         'description': 'Create a new cluster group',
         'properties': {
             'name': {'type': 'string', 'description': 'Group name'},
@@ -1004,7 +882,7 @@ TOOL_DEFINITIONS = {
         },
         'required': ['name', 'slug']
     },
-    'virt_list_clusters': {
+    'virtualization_list_clusters': {
         'description': 'List all clusters in NetBox',
         'properties': {
             'name': {'type': 'string', 'description': 'Filter by name'},
@@ -1013,12 +891,12 @@ TOOL_DEFINITIONS = {
             'site_id': {'type': 'integer', 'description': 'Filter by site ID'}
         }
     },
-    'virt_get_cluster': {
+    'virtualization_get_cluster': {
         'description': 'Get a specific cluster by ID',
         'properties': {'id': {'type': 'integer', 'description': 'Cluster ID'}},
         'required': ['id']
     },
-    'virt_create_cluster': {
+    'virtualization_create_cluster': {
         'description': 'Create a new cluster',
         'properties': {
             'name': {'type': 'string', 'description': 'Cluster name'},
@@ -1029,27 +907,17 @@ TOOL_DEFINITIONS = {
         },
         'required': ['name', 'type']
     },
-    'virt_update_cluster': {
+    'virtualization_update_cluster': {
         'description': 'Update an existing cluster',
-        'properties': {
+        'properties': {'id': {'type': 'integer', 'description': 'Cluster ID'}},
-            'id': {'type': 'integer', 'description': 'Cluster ID'},
-            'name': {'type': 'string', 'description': 'New name'},
-            'type': {'type': 'integer', 'description': 'Cluster type ID'},
-            'group': {'type': 'integer', 'description': 'Cluster group ID'},
-            'site': {'type': 'integer', 'description': 'Site ID'},
-            'status': {'type': 'string', 'description': 'Status'},
-            'tenant': {'type': 'integer', 'description': 'Tenant ID'},
-            'description': {'type': 'string', 'description': 'Description'},
-            'comments': {'type': 'string', 'description': 'Comments'}
-        },
         'required': ['id']
     },
-    'virt_delete_cluster': {
+    'virtualization_delete_cluster': {
         'description': 'Delete a cluster',
         'properties': {'id': {'type': 'integer', 'description': 'Cluster ID'}},
         'required': ['id']
     },
-    'virt_list_vms': {
+    'virtualization_list_virtual_machines': {
         'description': 'List all virtual machines in NetBox',
         'properties': {
             'name': {'type': 'string', 'description': 'Filter by name'},
@@ -1058,12 +926,12 @@ TOOL_DEFINITIONS = {
             'status': {'type': 'string', 'description': 'Filter by status'}
         }
     },
-    'virt_get_vm': {
+    'virtualization_get_virtual_machine': {
         'description': 'Get a specific virtual machine by ID',
         'properties': {'id': {'type': 'integer', 'description': 'VM ID'}},
         'required': ['id']
     },
-    'virt_create_vm': {
+    'virtualization_create_virtual_machine': {
         'description': 'Create a new virtual machine',
         'properties': {
             'name': {'type': 'string', 'description': 'VM name'},
@@ -1076,45 +944,29 @@ TOOL_DEFINITIONS = {
         },
         'required': ['name']
     },
-    'virt_update_vm': {
+    'virtualization_update_virtual_machine': {
         'description': 'Update an existing virtual machine',
-        'properties': {
+        'properties': {'id': {'type': 'integer', 'description': 'VM ID'}},
-            'id': {'type': 'integer', 'description': 'VM ID'},
-            'name': {'type': 'string', 'description': 'New name'},
-            'status': {'type': 'string', 'description': 'Status'},
-            'cluster': {'type': 'integer', 'description': 'Cluster ID'},
-            'site': {'type': 'integer', 'description': 'Site ID'},
-            'role': {'type': 'integer', 'description': 'Role ID'},
-            'tenant': {'type': 'integer', 'description': 'Tenant ID'},
-            'platform': {'type': 'integer', 'description': 'Platform ID'},
-            'vcpus': {'type': 'number', 'description': 'Number of vCPUs'},
-            'memory': {'type': 'integer', 'description': 'Memory in MB'},
-            'disk': {'type': 'integer', 'description': 'Disk in GB'},
-            'primary_ip4': {'type': 'integer', 'description': 'Primary IPv4 address ID'},
-            'primary_ip6': {'type': 'integer', 'description': 'Primary IPv6 address ID'},
-            'description': {'type': 'string', 'description': 'Description'},
-            'comments': {'type': 'string', 'description': 'Comments'}
-        },
         'required': ['id']
     },
-    'virt_delete_vm': {
+    'virtualization_delete_virtual_machine': {
         'description': 'Delete a virtual machine',
         'properties': {'id': {'type': 'integer', 'description': 'VM ID'}},
         'required': ['id']
     },
-    'virt_list_vm_ifaces': {
+    'virtualization_list_vm_interfaces': {
         'description': 'List all VM interfaces in NetBox',
         'properties': {
             'virtual_machine_id': {'type': 'integer', 'description': 'Filter by VM ID'},
             'name': {'type': 'string', 'description': 'Filter by name'}
         }
     },
-    'virt_get_vm_iface': {
+    'virtualization_get_vm_interface': {
         'description': 'Get a specific VM interface by ID',
         'properties': {'id': {'type': 'integer', 'description': 'Interface ID'}},
         'required': ['id']
     },
-    'virt_create_vm_iface': {
+    'virtualization_create_vm_interface': {
         'description': 'Create a new VM interface',
         'properties': {
             'virtual_machine': {'type': 'integer', 'description': 'VM ID'},
@@ -1252,18 +1104,16 @@ TOOL_DEFINITIONS = {
     },
 
     # ==================== Wireless Tools ====================
-    # NOTE: Tool names shortened from 'wireless_' to 'wlan_' to meet
+    'wireless_list_wireless_lan_groups': {
-    # 28-char limit (Claude API 64-char limit minus 36-char prefix)
-    'wlan_list_groups': {
         'description': 'List all wireless LAN groups in NetBox',
         'properties': {'name': {'type': 'string', 'description': 'Filter by name'}}
     },
-    'wlan_get_group': {
+    'wireless_get_wireless_lan_group': {
         'description': 'Get a specific wireless LAN group by ID',
         'properties': {'id': {'type': 'integer', 'description': 'WLAN group ID'}},
         'required': ['id']
     },
-    'wlan_create_group': {
+    'wireless_create_wireless_lan_group': {
         'description': 'Create a new wireless LAN group',
         'properties': {
             'name': {'type': 'string', 'description': 'Group name'},
@@ -1271,7 +1121,7 @@ TOOL_DEFINITIONS = {
         },
         'required': ['name', 'slug']
     },
-    'wlan_list_lans': {
+    'wireless_list_wireless_lans': {
         'description': 'List all wireless LANs in NetBox',
         'properties': {
             'ssid': {'type': 'string', 'description': 'Filter by SSID'},
@@ -1279,12 +1129,12 @@ TOOL_DEFINITIONS = {
             'status': {'type': 'string', 'description': 'Filter by status'}
         }
     },
-    'wlan_get_lan': {
+    'wireless_get_wireless_lan': {
         'description': 'Get a specific wireless LAN by ID',
         'properties': {'id': {'type': 'integer', 'description': 'WLAN ID'}},
         'required': ['id']
     },
-    'wlan_create_lan': {
+    'wireless_create_wireless_lan': {
         'description': 'Create a new wireless LAN',
         'properties': {
             'ssid': {'type': 'string', 'description': 'SSID'},
@@ -1294,14 +1144,14 @@ TOOL_DEFINITIONS = {
         },
         'required': ['ssid']
     },
-    'wlan_list_links': {
+    'wireless_list_wireless_links': {
         'description': 'List all wireless links in NetBox',
         'properties': {
             'ssid': {'type': 'string', 'description': 'Filter by SSID'},
             'status': {'type': 'string', 'description': 'Filter by status'}
         }
     },
-    'wlan_get_link': {
+    'wireless_get_wireless_link': {
         'description': 'Get a specific wireless link by ID',
         'properties': {'id': {'type': 'integer', 'description': 'Link ID'}},
         'required': ['id']
@@ -1407,52 +1257,6 @@ TOOL_DEFINITIONS = {
 }
 
 
-# Map shortened tool names to (category, method_name) for routing.
-# This is necessary because tool names were shortened to meet the 28-character
-# limit imposed by Claude API's 64-character tool name limit minus the
-# 36-character prefix used by Claude Code for MCP tools.
-TOOL_NAME_MAP = {
-    # Virtualization tools (virt_ -> virtualization category)
-    'virt_list_cluster_types': ('virtualization', 'list_cluster_types'),
-    'virt_get_cluster_type': ('virtualization', 'get_cluster_type'),
-    'virt_create_cluster_type': ('virtualization', 'create_cluster_type'),
-    'virt_list_cluster_groups': ('virtualization', 'list_cluster_groups'),
-    'virt_get_cluster_group': ('virtualization', 'get_cluster_group'),
-    'virt_create_cluster_group': ('virtualization', 'create_cluster_group'),
-    'virt_list_clusters': ('virtualization', 'list_clusters'),
-    'virt_get_cluster': ('virtualization', 'get_cluster'),
-    'virt_create_cluster': ('virtualization', 'create_cluster'),
-    'virt_update_cluster': ('virtualization', 'update_cluster'),
-    'virt_delete_cluster': ('virtualization', 'delete_cluster'),
-    'virt_list_vms': ('virtualization', 'list_virtual_machines'),
-    'virt_get_vm': ('virtualization', 'get_virtual_machine'),
-    'virt_create_vm': ('virtualization', 'create_virtual_machine'),
-    'virt_update_vm': ('virtualization', 'update_virtual_machine'),
-    'virt_delete_vm': ('virtualization', 'delete_virtual_machine'),
-    'virt_list_vm_ifaces': ('virtualization', 'list_vm_interfaces'),
-    'virt_get_vm_iface': ('virtualization', 'get_vm_interface'),
-    'virt_create_vm_iface': ('virtualization', 'create_vm_interface'),
-
-    # Circuits tools (circ_ -> circuits category, for shortened names only)
-    'circ_list_types': ('circuits', 'list_circuit_types'),
-    'circ_get_type': ('circuits', 'get_circuit_type'),
-    'circ_create_type': ('circuits', 'create_circuit_type'),
-    'circ_list_terminations': ('circuits', 'list_circuit_terminations'),
-    'circ_get_termination': ('circuits', 'get_circuit_termination'),
-    'circ_create_termination': ('circuits', 'create_circuit_termination'),
-
-    # Wireless tools (wlan_ -> wireless category)
-    'wlan_list_groups': ('wireless', 'list_wireless_lan_groups'),
-    'wlan_get_group': ('wireless', 'get_wireless_lan_group'),
-    'wlan_create_group': ('wireless', 'create_wireless_lan_group'),
-    'wlan_list_lans': ('wireless', 'list_wireless_lans'),
-    'wlan_get_lan': ('wireless', 'get_wireless_lan'),
-    'wlan_create_lan': ('wireless', 'create_wireless_lan'),
-    'wlan_list_links': ('wireless', 'list_wireless_links'),
-    'wlan_get_link': ('wireless', 'get_wireless_link'),
-}
-
-
 class NetBoxMCPServer:
     """MCP Server for NetBox integration"""
 
@@ -1526,20 +1330,11 @@ class NetBoxMCPServer:
                 )]
 
     async def _route_tool(self, name: str, arguments: dict):
-        """Route tool call to appropriate handler.
+        """Route tool call to appropriate handler."""
-
-        Tool names may be shortened (e.g., 'virt_list_vms' instead of
-        'virtualization_list_virtual_machines') to meet the 28-character
-        limit. TOOL_NAME_MAP handles the translation to actual method names.
-        """
-        # Check if this is a mapped short name
-        if name in TOOL_NAME_MAP:
-            category, method_name = TOOL_NAME_MAP[name]
-        else:
-            # Fall back to original logic for unchanged tools
         parts = name.split('_', 1)
         if len(parts) != 2:
             raise ValueError(f"Invalid tool name format: {name}")
+
         category, method_name = parts[0], parts[1]
 
         # Map category to tool class

scripts/release.sh

@@ -1,172 +0,0 @@
-#!/bin/bash
-# release.sh - Create a new release with version consistency
-#
-# Usage: ./scripts/release.sh X.Y.Z
-#
-# This script ensures all version references are updated consistently:
-# 1. CHANGELOG.md - [Unreleased] becomes [X.Y.Z] - YYYY-MM-DD
-# 2. README.md - Title updated to vX.Y.Z
-# 3. marketplace.json - version field updated
-# 4. Git commit and tag created
-#
-# Prerequisites:
-# - Clean working directory (no uncommitted changes)
-# - [Unreleased] section in CHANGELOG.md with content
-# - On development branch
-
-set -e
-
-VERSION=$1
-DATE=$(date +%Y-%m-%d)
-
-# Colors for output
-RED='\033[0;31m'
-GREEN='\033[0;32m'
-YELLOW='\033[1;33m'
-NC='\033[0m' # No Color
-
-error() { echo -e "${RED}ERROR: $1${NC}" >&2; exit 1; }
-warn() { echo -e "${YELLOW}WARNING: $1${NC}"; }
-success() { echo -e "${GREEN}$1${NC}"; }
-info() { echo -e "$1"; }
-
-# Validate arguments
-if [ -z "$VERSION" ]; then
-    echo "Usage: ./scripts/release.sh X.Y.Z"
-    echo ""
-    echo "Example: ./scripts/release.sh 3.2.0"
-    echo ""
-    echo "This will:"
-    echo "  1. Update CHANGELOG.md [Unreleased] -> [X.Y.Z] - $(date +%Y-%m-%d)"
-    echo "  2. Update README.md title to vX.Y.Z"
-    echo "  3. Update marketplace.json version to X.Y.Z"
-    echo "  4. Commit with message 'chore: release vX.Y.Z'"
-    echo "  5. Create git tag vX.Y.Z"
-    exit 1
-fi
-
-# Validate version format
-if ! [[ "$VERSION" =~ ^[0-9]+\.[0-9]+\.[0-9]+$ ]]; then
-    error "Invalid version format. Use X.Y.Z (e.g., 3.2.0)"
-fi
-
-# Check we're in the right directory
-if [ ! -f "CHANGELOG.md" ] || [ ! -f "README.md" ] || [ ! -f ".claude-plugin/marketplace.json" ]; then
-    error "Must run from repository root (CHANGELOG.md, README.md, .claude-plugin/marketplace.json must exist)"
-fi
-
-# Check for clean working directory
-if [ -n "$(git status --porcelain)" ]; then
-    warn "Working directory has uncommitted changes"
-    echo ""
-    git status --short
-    echo ""
-    read -p "Continue anyway? [y/N] " -n 1 -r
-    echo
-    if [[ ! $REPLY =~ ^[Yy]$ ]]; then
-        exit 1
-    fi
-fi
-
-# Check current branch
-BRANCH=$(git branch --show-current)
-if [ "$BRANCH" != "development" ] && [ "$BRANCH" != "main" ]; then
-    warn "Not on development or main branch (current: $BRANCH)"
-    read -p "Continue anyway? [y/N] " -n 1 -r
-    echo
-    if [[ ! $REPLY =~ ^[Yy]$ ]]; then
-        exit 1
-    fi
-fi
-
-# Check [Unreleased] section has content
-if ! grep -q "## \[Unreleased\]" CHANGELOG.md; then
-    error "CHANGELOG.md missing [Unreleased] section"
-fi
-
-# Check if tag already exists
-if git tag -l | grep -q "^v$VERSION$"; then
-    error "Tag v$VERSION already exists"
-fi
-
-info ""
-info "=== Release v$VERSION ==="
-info ""
-
-# Show what will change
-info "Changes to be made:"
-info "  CHANGELOG.md: [Unreleased] -> [$VERSION] - $DATE"
-info "  README.md: title -> v$VERSION"
-info "  marketplace.json: version -> $VERSION"
-info "  Git: commit + tag v$VERSION"
-info ""
-
-# Preview CHANGELOG [Unreleased] content
-info "Current [Unreleased] content:"
-info "---"
-sed -n '/^## \[Unreleased\]/,/^## \[/p' CHANGELOG.md | head -30
-info "---"
-info ""
-
-read -p "Proceed with release? [y/N] " -n 1 -r
-echo
-if [[ ! $REPLY =~ ^[Yy]$ ]]; then
-    info "Aborted"
-    exit 0
-fi
-
-info ""
-info "Updating files..."
-
-# 1. Update CHANGELOG.md
-# Replace [Unreleased] with [X.Y.Z] - DATE and add new [Unreleased] section
-sed -i "s/^## \[Unreleased\]$/## [Unreleased]\n\n*Changes staged for the next release*\n\n---\n\n## [$VERSION] - $DATE/" CHANGELOG.md
-
-# Remove the placeholder text if it exists after the new [Unreleased]
-sed -i '/^\*Changes staged for the next release\*$/d' CHANGELOG.md
-
-# Clean up any double blank lines
-sed -i '/^$/N;/^\n$/d' CHANGELOG.md
-
-success "  CHANGELOG.md updated"
-
-# 2. Update README.md title
-sed -i "s/^# Leo Claude Marketplace - v[0-9]\+\.[0-9]\+\.[0-9]\+$/# Leo Claude Marketplace - v$VERSION/" README.md
-success "  README.md updated"
-
-# 3. Update marketplace.json version
-sed -i "s/\"version\": \"[0-9]\+\.[0-9]\+\.[0-9]\+\"/\"version\": \"$VERSION\"/" .claude-plugin/marketplace.json
-success "  marketplace.json updated"
-
-info ""
-info "Files updated. Review changes:"
-info ""
-git diff --stat
-info ""
-git diff CHANGELOG.md | head -40
-info ""
-
-read -p "Commit and tag? [y/N] " -n 1 -r
-echo
-if [[ ! $REPLY =~ ^[Yy]$ ]]; then
-    warn "Changes made but not committed. Run 'git checkout -- .' to revert."
-    exit 0
-fi
-
-# Commit
-git add CHANGELOG.md README.md .claude-plugin/marketplace.json
-git commit -m "chore: release v$VERSION"
-success "  Committed"
-
-# Tag
-git tag "v$VERSION"
-success "  Tagged v$VERSION"
-
-info ""
-success "=== Release v$VERSION created ==="
-info ""
-info "Next steps:"
-info "  1. Review the commit: git show HEAD"
-info "  2. Push to remote: git push && git push --tags"
-info "  3. Merge to main if on development branch"
-info ""