Reduce config polling and align tag separators (#76)

Author: devnullvoid

CHANGELOG.md

@@ -32,6 +32,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - **Group Mode Source Profile Preservation**: Fixed VMs losing their cluster association after operations in group mode, which caused "source profile not set" errors. The SourceProfile field is now preserved when refreshing individual VM data.
 - **Group Mode Initial Loading Feedback**: Added header loading message "Loading guest agent data" during initial startup in group mode. Previously, profile names would appear in the UI without warning after a silent enrichment process.
 - **CI Cache Noise**: Removed redundant Go module cache restore in CI to prevent tar "File exists" errors during lint/test/build jobs.
+- **Group Mode Config Polling**: Avoided cross-profile cluster resource polling when only tags change, reducing delays after saving guest configuration.
 
 ## [1.0.16] - 2026-01-01
 

internal/ui/components/vm_config.go

@@ -195,8 +195,11 @@ func NewVMConfigPage(app *App, vm *api.VM, config *api.VMConfig, saveFn func(*ap
 							expectedName = page.config.Hostname
 						}
 
+						nameChanged := expectedName != "" && expectedName != vm.Name
+						tagsChanged := page.config.TagsExplicit && page.config.Tags != vm.Tags
+
 						// Use the dedicated polling function
-						app.pollForConfigChange(vm, expectedName)
+						app.pollForConfigChange(vm, expectedName, nameChanged, page.config.Tags, tagsChanged)
 					}()
 				}
 			})
@@ -428,33 +431,44 @@ func showResizeStorageModal(app *App, vm *api.VM) {
 // pollForConfigChange polls the Proxmox API to verify that a configuration change has propagated
 // 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) {
+func (app *App) pollForConfigChange(vm *api.VM, expectedName string, nameChanged bool, expectedTags string, tagsChanged bool) {
 	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
+	if !nameChanged {
+		maxAttempts = 10
+	}
 	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 := client.GetVMConfig(vm)
-		configUpdated := false
+		configUpdated := true
 
 		if err == nil && config != nil {
-			if vm.Type == api.VMTypeQemu && config.Name == expectedName {
-				configUpdated = true
-			} else if vm.Type == api.VMTypeLXC && config.Hostname == expectedName {
-				configUpdated = true
+			if nameChanged {
+				if vm.Type == api.VMTypeQemu && config.Name != expectedName {
+					configUpdated = false
+				} else if vm.Type == api.VMTypeLXC && config.Hostname != expectedName {
+					configUpdated = false
+				}
+			}
+
+			if tagsChanged && config.Tags != expectedTags {
+				configUpdated = false
 			}
+		} else {
+			configUpdated = false
 		}
 
-		// If config is updated, also check if cluster resources reflect the change
-		if configUpdated {
+		// If config is updated, also check if cluster resources reflect the change when needed
+		if configUpdated && nameChanged {
 			// Use the existing GetVmList function to check cluster resources
-			vmList, err := app.client.GetVmList(context.Background())
+			vmList, err := client.GetVmList(context.Background())
 			if err == nil {
 				for _, vmData := range vmList {
 					if resType, exists := vmData["type"].(string); exists && resType == vm.Type {
@@ -473,6 +487,11 @@ func (app *App) pollForConfigChange(vm *api.VM, expectedName string) {
 					}
 				}
 			}
+		} else if configUpdated {
+			app.QueueUpdateDraw(func() {
+				app.manualRefresh()
+			})
+			return
 		}
 	}
 

pkg/api/vm_config.go

@@ -17,7 +17,7 @@ type VMConfig struct {
 	Memory      int64  `json:"memory,omitempty"` // in bytes
 	Description string `json:"description,omitempty"`
 	OnBoot      *bool  `json:"onboot,omitempty"`
-	// Tags is a comma-separated list of guest tags.
+	// Tags is a semicolon-separated list of guest tags.
 	Tags string `json:"tags,omitempty"`
 	// TagsExplicit controls whether tags should be included in update payloads.
 	TagsExplicit bool `json:"-"`

pkg/api/vm_config_test.go

@@ -26,7 +26,7 @@ func TestVMConfig_ParseAndBuild(t *testing.T) {
 				"cpu":         "host",
 				"maxmem":      16384.0,
 				"boot":        "order=scsi0;net0",
-				"tags":        "prod,db",
+				"tags":        "prod;db",
 			},
 			expected: &VMConfig{
 				Name:         "test-vm",
@@ -38,7 +38,7 @@ func TestVMConfig_ParseAndBuild(t *testing.T) {
 				CPUType:      "host",
 				MaxMem:       16384,
 				BootOrder:    "order=scsi0;net0",
-				Tags:         "prod,db",
+				Tags:         "prod;db",
 				TagsExplicit: true,
 			},
 		},

pkg/api/vm_types.go

@@ -56,7 +56,7 @@ type VM struct {
 	// Administrative and cluster information
 	HAState  string `json:"hastate,omitempty"`  // High availability state
 	Lock     string `json:"lock,omitempty"`     // Lock status if VM is locked
-	Tags     string `json:"tags,omitempty"`     // Comma-separated tags
+	Tags     string `json:"tags,omitempty"`     // Semicolon-separated tags
 	Template bool   `json:"template,omitempty"` // Whether this is a template
 	Pool     string `json:"pool,omitempty"`     // Resource pool assignment