fix: stable custom metadata field ordering and clean display

- Change registerMetadata to accept goja.Value directly to preserve JavaScript object key order
- Disable SetAutoWrapText to prevent line combining and spacing issues
- Custom fields now display in definition order without extra spaces

Author: speier

CHANGELOG.md

@@ -7,7 +7,13 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
-## [0.6.3] - 2025-10-22
+## [0.6.4] - 2025-01-07
+
+### Fixed
+- Custom metadata field ordering now stable and consistent across runs
+- Removed extra leading spaces from custom field values in table display
+
+## [0.6.3] - 2025-01-07
 
 ### Changed
 - **Enhanced metadata display in `comet list --details`**

internal/cli/print.go

@@ -54,7 +54,7 @@ func PrintStacksList(stacks *schema.Stacks, details bool) {
 	headers = append(headers, "path")
 
 	table.SetHeader(headers)
-	table.SetAutoWrapText(true)
+	table.SetAutoWrapText(false)
 	table.SetRowLine(true)
 	table.SetColWidth(20)
 	table.SetAutoMergeCells(false)
@@ -96,19 +96,18 @@ func PrintStacksList(stacks *schema.Stacks, details bool) {
 						if i+1 < len(orderedSlice) {
 							k := fmt.Sprintf("%v", orderedSlice[i])
 							v := orderedSlice[i+1]
-							line := fmt.Sprintf("%-20s", fmt.Sprintf("%s=%v", k, v))
-							customPairs = append(customPairs, line)
+							customPairs = append(customPairs, fmt.Sprintf("%s=%v", k, v))
 						}
 					}
 				} else if customMap, ok := s.Metadata.Custom.(map[string]any); ok {
 					// Fallback to map (unordered)
 					for k, v := range customMap {
-						line := fmt.Sprintf("%-20s", fmt.Sprintf("%s=%v", k, v))
-						customPairs = append(customPairs, line)
+						customPairs = append(customPairs, fmt.Sprintf("%s=%v", k, v))
 					}
 				}
 
 				if len(customPairs) > 0 {
+					// Join without manual padding - let tablewriter handle it
 					custom = strings.Join(customPairs, "\n")
 				}
 			}

internal/parser/js/js.go

@@ -194,46 +194,56 @@ func (vm *jsinterpreter) registerStack(stack *schema.Stack) func(string, map[str
 	}
 }
 
-func (vm *jsinterpreter) registerMetadata(stack *schema.Stack) func(map[string]interface{}) {
-	return func(meta map[string]interface{}) {
+func (vm *jsinterpreter) registerMetadata(stack *schema.Stack) func(goja.Value) {
+	return func(metaValue goja.Value) {
 		log.Debug("register metadata", "stack", stack.Name)
 
 		metadata := &schema.Metadata{}
 
-		if desc, ok := meta["description"].(string); ok {
-			metadata.Description = desc
+		// Get the metadata object
+		metaObj := metaValue.ToObject(vm.rt)
+		if metaObj == nil {
+			return
 		}
 
-		if owner, ok := meta["owner"].(string); ok {
-			metadata.Owner = owner
+		// Extract description
+		if descVal := metaObj.Get("description"); descVal != nil && !goja.IsUndefined(descVal) {
+			if desc, ok := descVal.Export().(string); ok {
+				metadata.Description = desc
+			}
 		}
 
-		if tags, ok := meta["tags"].([]interface{}); ok {
-			metadata.Tags = make([]string, 0, len(tags))
-			for _, tag := range tags {
-				if tagStr, ok := tag.(string); ok {
-					metadata.Tags = append(metadata.Tags, tagStr)
-				}
+		// Extract owner
+		if ownerVal := metaObj.Get("owner"); ownerVal != nil && !goja.IsUndefined(ownerVal) {
+			if owner, ok := ownerVal.Export().(string); ok {
+				metadata.Owner = owner
 			}
 		}
 
-		// Store custom fields preserving order
-		if customObj, ok := meta["custom"].(map[string]interface{}); ok {
-			// Use goja to get object keys in order
-			if vm.rt != nil {
-				customValue := vm.rt.ToValue(meta["custom"])
-				if obj := customValue.ToObject(vm.rt); obj != nil {
-					keys := obj.Keys()
-					orderedCustom := make([]interface{}, 0, len(keys)*2)
-					for _, key := range keys {
-						val := obj.Get(key)
-						orderedCustom = append(orderedCustom, key, val.Export())
+		// Extract tags
+		if tagsVal := metaObj.Get("tags"); tagsVal != nil && !goja.IsUndefined(tagsVal) {
+			if tagsExport := tagsVal.Export(); tagsExport != nil {
+				if tags, ok := tagsExport.([]interface{}); ok {
+					metadata.Tags = make([]string, 0, len(tags))
+					for _, tag := range tags {
+						if tagStr, ok := tag.(string); ok {
+							metadata.Tags = append(metadata.Tags, tagStr)
+						}
 					}
-					metadata.Custom = orderedCustom
 				}
-			} else {
-				// Fallback to map
-				metadata.Custom = customObj
+			}
+		}
+
+		// Extract custom fields preserving order
+		if customVal := metaObj.Get("custom"); customVal != nil && !goja.IsUndefined(customVal) {
+			if customObj := customVal.ToObject(vm.rt); customObj != nil {
+				keys := customObj.Keys()
+				orderedCustom := make([]interface{}, 0, len(keys)*2)
+				for _, key := range keys {
+					val := customObj.Get(key)
+					orderedCustom = append(orderedCustom, key, val.Export())
+				}
+				metadata.Custom = orderedCustom
 			}
 		}