UNIFI-7 - trckr

← UNIFI-7

feature

Created: 2026-02-02 • Updated: 2026-02-02

Relationships Loading...

## Description
Implement network/VLAN management commands.

## Tasks
- network list - list all networks
- network get <id> - get network details
- network create --name --vlan-id --purpose --subnet - create network
- network update <id> --name --vlan-id - update network
- network delete <id> - delete network
- network refs <id> - get network references

## Docs Reference
- ~/github/llm-code-docs/docs/web-scraped/unifi-api/network/getnetworksoverviewpage.md
- ~/github/llm-code-docs/docs/web-scraped/unifi-api/network/getnetworkdetails.md
- ~/github/llm-code-docs/docs/web-scraped/unifi-api/network/createnetwork.md
- ~/github/llm-code-docs/docs/web-scraped/unifi-api/network/updatenetwork.md
- ~/github/llm-code-docs/docs/web-scraped/unifi-api/network/deletenetwork.md
- ~/github/llm-code-docs/docs/web-scraped/unifi-api/network/getnetworkreferences.md

## QA Instructions
1. Full CRUD cycle works: create -> read -> update -> delete
2. Create with required fields only succeeds
3. Create with invalid VLAN ID fails with clear error
4. --format json outputs match OpenAPI schema

Pull Requests (comma-separated URLs)

Attachments

Comments (3)

code-verification-qa · 2026-02-02

## QA Verification Complete ✅ **All acceptance criteria met:** 1. ✅ Full CRUD cycle works (create → read → update → delete) 2. ✅ Create with required fields only succeeds 3. ✅ Create with invalid VLAN ID fails with clear error message 4. ✅ JSON output format matches OpenAPI schema **Test Results:** - All 300 tests pass (26 new network tests added) - Test execution time: 0.62s - No failures or regressions **Implementation verified:** - 6 client methods (get_network_list, get_network_detail, create_network, update_network, delete_network, get_network_references) - 5 CLI commands (network get, create, update, delete, refs) - Safety features: VLAN validation, confirmation prompts, reference checking **Detailed QA report:** /tmp/qa-UNIFI-7.md **Status:** VERIFIED - Ready for production

agent · 2026-02-02

## Implementation Complete Implemented network CRUD commands via the Network API (connector proxy): ### New Client Methods (unifi_cli.py:786-900) - `get_network_list()` - list networks with pagination/filtering - `get_network_detail()` - get single network details - `create_network()` - create new network - `update_network()` - update existing network - `delete_network()` - delete network with cascade/force options - `get_network_references()` - get references to a network ### New CLI Commands (unifi_cli.py:2494-2676) - `network get <id>` - get network by ID - `network create --name --vlan-id [--management] [--enabled/--disabled]` - `network update <id> --name --vlan-id --management --enabled/--disabled` - `network delete <id> [-y] [--cascade] [--force] [--check-refs]` - `network refs <id>` - show network references ### Safety Features - VLAN ID validation (0-4094) - `--check-refs` flag on delete to warn about referenced resources - Confirmation prompt before delete (skip with -y) - `--cascade` and `--force` options for delete ### Testing - 26 new tests added (tests/test_cli.py:2762-3059) - All 300 tests pass

triage-agent · 2026-02-02

## Feature Triage Complete **Status:** PARTIALLY IMPLEMENTED **Summary:** The `network list` command already exists with full text/json/yaml output support. Five additional commands need to be implemented: `network get`, `network create`, `network update`, `network delete`, and `network refs`. **Implementation Location:** - Client methods: After `get_networks()` method (line 588 in `unifi_cli.py`) - CLI commands: After `network_list` command (line 2365 in `unifi_cli.py`) **Complexity:** Low-Medium (~430 lines of code + tests) **Key Notes:** 1. Existing patterns from firewall zone CRUD can be followed 2. All network operations require `--site-id` 3. Issue mentions `--subnet` and `--purpose` params not found in API docs - may need clarification 4. Recommend adding `--check-refs` flag to delete for safety See attached triage report for full details.