Fix tag separator and group config polling

Author: devnullvoid

CHANGELOG.md

@@ -12,7 +12,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - **Node Disk SMART Information**: Node details now display disk health status and SMART data for all attached disks, including disk type (SSD/HDD), size, model, and health status (PASSED/FAILED).
 - **System Update Notifications**: Node details now show available system package updates with version information, displaying up to 5 pending updates with a count of any additional updates.
 - **Icon Toggle**: Added multiple ways to control icons/emojis throughout the UI. Icons are enabled by default. To disable: use `--show-icons=false` CLI flag, `PVETUI_SHOW_ICONS=false` environment variable, or `show_icons: false` in YAML config. (#75)
-- **Guest Tags**: Added editable tag list for VM/LXC configuration, using a comma-separated tag field in the Edit Configuration form. (#76)
+- **Guest Tags**: Added editable tag list for VM/LXC configuration, using a semicolon-separated tag field in the Edit Configuration form. (#76)
 - **SSH Jump Host Support**: Added SSH jump host (bastion host) configuration for accessing Proxmox environments through an intermediate SSH server. Configure via config wizard, CLI flags (`--ssh-jumphost-addr`, `--ssh-jumphost-user`, `--ssh-jumphost-keyfile`, `--ssh-jumphost-port`), environment variables (`PVETUI_SSH_JUMPHOST_ADDR`, `PVETUI_SSH_JUMPHOST_USER`, `PVETUI_SSH_JUMPHOST_KEYFILE`, `PVETUI_SSH_JUMPHOST_PORT`), or YAML config (`ssh_jump_host` section). Per-profile configuration supported.
 - **Release Announcements**: GoReleaser now posts release announcements to Mastodon and Bluesky when the required secrets are configured.
 

internal/ui/components/vm_config.go

@@ -118,8 +118,8 @@ func NewVMConfigPage(app *App, vm *api.VM, config *api.VMConfig, saveFn func(*ap
 	})
 
 	// Tags
-	initialTags := strings.TrimSpace(config.Tags)
-	form.AddInputField("Tags (comma-separated)", initialTags, 40, nil, func(text string) {
+	initialTags := normalizeTags(config.Tags)
+	form.AddInputField("Tags (semicolon-separated)", initialTags, 40, nil, func(text string) {
 		page.config.Tags = normalizeTags(text)
 		page.config.TagsExplicit = true
 	})
@@ -302,7 +302,7 @@ func normalizeTags(raw string) string {
 		}
 		cleaned = append(cleaned, trimmed)
 	}
-	return strings.Join(cleaned, ",")
+	return strings.Join(cleaned, ";")
 }
 
 // showResizeStorageModal displays a modal for resizing a storage volume.
@@ -429,13 +429,18 @@ func showResizeStorageModal(app *App, vm *api.VM) {
 // to both the config endpoint and the cluster resources endpoint before refreshing the UI.
 // This prevents race conditions where config is updated but cluster resources still show old names.
 func (app *App) pollForConfigChange(vm *api.VM, expectedName string) {
+	client, err := app.getClientForVM(vm)
+	if err != nil {
+		client = app.client
+	}
+
 	// Poll every 500ms for up to 15 seconds (increased timeout for cluster resources propagation)
 	maxAttempts := 30
 	for attempt := 0; attempt < maxAttempts; attempt++ {
 		time.Sleep(500 * time.Millisecond)
 
 		// First check if the config endpoint has the new name using the existing API function
-		config, err := app.client.GetVMConfig(vm)
+		config, err := client.GetVMConfig(vm)
 		configUpdated := false
 
 		if err == nil && config != nil {