From 3ac8a1a08ea774d110b2af738b03a942fa6d2619 Mon Sep 17 00:00:00 2001 From: nutcas3 Date: Fri, 17 Apr 2026 16:30:23 +0300 Subject: [PATCH] feat: Add empty response handling to services and subscribers list commands with sample data fallback - Update listServices to check for empty services slice in addition to error condition - Update listSubscribers to check for empty subscribers slice in addition to error condition - Add muted message when API returns empty results but no error - Preserve existing warning message when API returns error - Show sample/placeholder data in both empty and error cases --- apps/cli/internal/commands/services.go | 8 ++++++-- apps/cli/internal/commands/subscribers.go | 8 ++++++-- 2 files changed, 12 insertions(+), 4 deletions(-) diff --git a/apps/cli/internal/commands/services.go b/apps/cli/internal/commands/services.go index 8c1a45e..a1d525a 100644 --- a/apps/cli/internal/commands/services.go +++ b/apps/cli/internal/commands/services.go @@ -59,8 +59,12 @@ func listServices(u *uiContext) error { t.AddColumn("Version", 10, "left") t.AddColumn("Uptime", 10, "right") - if err != nil { - u.warn("Using placeholder data: " + err.Error()) + if err != nil || len(services) == 0 { + if err != nil { + u.warn("Using placeholder data: " + err.Error()) + } else { + u.muted("(API returned no services - showing sample data)") + } placeholders := [][]string{ {"api-server", "Running", "v1.0.0", "2h15m"}, {"charging-engine", "Running", "v1.0.0", "2h15m"}, diff --git a/apps/cli/internal/commands/subscribers.go b/apps/cli/internal/commands/subscribers.go index 658b511..cbdb868 100644 --- a/apps/cli/internal/commands/subscribers.go +++ b/apps/cli/internal/commands/subscribers.go @@ -52,8 +52,12 @@ func listSubscribers(u *uiContext) error { u.header("Active Subscribers") subs, err := u.client.ListSubscribers() - if err != nil { - u.warn("Using placeholder data: " + err.Error()) + if err != nil || len(subs) == 0 { + if err != nil { + u.warn("Using placeholder data: " + err.Error()) + } else { + u.muted("(API returned no subscribers - showing sample data)") + } t := u.newTable() t.AddColumn("IMSI", 16, "left") t.AddColumn("Name", 22, "left")