From 162cbc3bd25ab5af9b8169baf0f070385aec339f Mon Sep 17 00:00:00 2001 From: chasqui Date: Wed, 11 Nov 2026 09:23:37 -0600 Subject: [PATCH] fix(app): prevent localized values from breaking app list --- cli/app/list.go | 37 +++++++++++++++++++++++++------------ 1 file changed, 25 insertions(+), 12 deletions(-) diff --git a/cli/app/list.go b/cli/app/list.go index d5076e73..222e3f27 100644 --- a/cli/app/list.go +++ b/cli/app/list.go @@ -117,10 +117,10 @@ Use "--status/-S" flag to query all servers for the live deployment status.`), log.Fatal(err) } - status := i18n.G("unknown") - version := i18n.G("unknown") - chaos := i18n.G("unknown") - chaosVersion := i18n.G("unknown") + status := "unknown" + version := "unknown" + chaos := "unknown" + chaosVersion := "unknown" if statusMeta, ok := statuses[app.StackName()]; ok { if currentVersion, exists := statusMeta["version"]; exists { if currentVersion != "" { @@ -181,9 +181,9 @@ Use "--status/-S" flag to query all servers for the live deployment status.`), if len(newUpdates) == 0 { if version == "unknown" { - appStats.Upgrade = i18n.G("unknown") + appStats.Upgrade = "unknown" } else { - appStats.Upgrade = i18n.G("latest") + appStats.Upgrade = "latest" stats.LatestCount++ } } else { @@ -256,13 +256,12 @@ Use "--status/-S" flag to query all servers for the live deployment status.`), } row = append(row, []string{ - appStat.Status, - chaosStatus, - appStat.Version, - appStat.Upgrade}..., - ) + displayUnknown(appStat.Status), + displayUnknown(chaosStatus), + displayUnknown(appStat.Version), + displayUpgrade(appStat.Upgrade), + }...) } - rows = append(rows, row) } @@ -283,6 +282,20 @@ Use "--status/-S" flag to query all servers for the live deployment status.`), }, } +func displayUnknown(value string) string { + if value == "unknown" { + return i18n.G("unknown") + } + return value +} + +func displayUpgrade(value string) string { + if value == "latest" { + return i18n.G("latest") + } + return displayUnknown(value) +} + var ( status bool recipeFilter string -- 2.54.0