From 5c70405a8552c5cb966e3a5a22d32d91376f4633 Mon Sep 17 00:00:00 2001 From: sneak Date: Sat, 26 Jul 2025 03:31:09 +0200 Subject: [PATCH] Fix snapshot list to fail on manifest errors - Remove error suppression for manifest decoding errors - Manifest read/deserialize errors now fail immediately with clear error messages - This ensures we catch format mismatches and other issues early --- internal/cli/snapshot.go | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/internal/cli/snapshot.go b/internal/cli/snapshot.go index 78f0a00..26ea8b0 100644 --- a/internal/cli/snapshot.go +++ b/internal/cli/snapshot.go @@ -548,8 +548,7 @@ func (app *SnapshotApp) List(ctx context.Context, jsonOutput bool) error { // Try to download manifest to get size totalSize, err := app.getManifestSize(ctx, snapshotID) if err != nil { - log.Warn("Failed to get manifest size", "id", snapshotID, "error", err) - totalSize = 0 + return fmt.Errorf("failed to get manifest size for %s: %w", snapshotID, err) } snapshots = append(snapshots, SnapshotInfo{