Nits

Author: anisaoshafi

cmd/snapshot.go

@@ -30,13 +30,13 @@ func newSnapshotSaveCmd(cfg *env.Env) *cobra.Command {
 	return &cobra.Command{
 		Use:   "save [destination]",
 		Short: "Save a snapshot of the emulator state",
-		Long: `Save a snapshot of the running emulator's state to a local file.
+		Long: `Save a snapshot of the running emulator's state.
 
-The destination must be a file path. Use a path prefix to save locally:
+Pass [destination] as an absolute or relative path for the exported file:
 
-  lstk snapshot save                  # saves to ./snapshot-2026-05-11T21-04-32
-  lstk snapshot save ./my-snapshot    # saves to ./my-snapshot
-  lstk snapshot save /tmp/my-state    # saves to /tmp/my-state
+  lstk snapshot save                    # saves to ./snapshot-<YYYY-MM-DDTHH-mm-ss>.zip
+  lstk snapshot save ./my-snapshot.zip  # saves to ./my-snapshot.zip
+  lstk snapshot save /tmp/my-state      # saves to /tmp/my-state.zip
 
 Cloud destinations are not yet supported.`,
 		Args:    cobra.MaximumNArgs(1),

internal/snapshot/destination.go

@@ -1,13 +1,16 @@
 package snapshot
 
 import (
+	"errors"
 	"fmt"
 	"os"
 	"path/filepath"
 	"strings"
 	"time"
 )
 
+var ErrCloudNotSupported = errors.New("cloud destinations are not yet supported — use a file path like ./my-snapshot.zip")
+
 // ParseDestination resolves the user-supplied path to an absolute local path,
 // or returns an error for cloud/bare names. When dest is empty, a default name
 // based on now (UTC) is used, e.g. "snapshot-2026-05-11T21-04-32.zip".
@@ -16,10 +19,10 @@ func ParseDestination(dest string, now time.Time) (string, error) {
 	if dest == "" {
 		dest = "./" + now.UTC().Format("snapshot-2006-01-02T15-04-05")
 	} else if strings.Contains(dest, "://") {
-		return "", fmt.Errorf("cloud destinations are not yet supported — use a file path like ./my-snapshot.zip")
+		return "", ErrCloudNotSupported
 	} else if !strings.HasPrefix(dest, ".") && !strings.HasPrefix(dest, "~") && !filepath.IsAbs(dest) && filepath.Base(dest) == dest {
 		// bare name with no path separators: reserved for future cloud pod names
-		return "", fmt.Errorf("cloud destinations are not yet supported — use a file path like ./my-snapshot.zip")
+		return "", ErrCloudNotSupported
 	}
 	if dest == "~" || strings.HasPrefix(dest, "~/") || strings.HasPrefix(dest, `~\`) {
 		home, err := os.UserHomeDir()

internal/snapshot/destination_test.go

@@ -79,17 +79,14 @@ func TestParseDestination(t *testing.T) {
 		},
 		{
 			input:        "my-pod",
-			wantErr:      "cloud destinations are not yet supported",
 			wantCloudErr: true,
 		},
 		{
 			input:        "cloud://my-pod",
-			wantErr:      "cloud destinations are not yet supported",
 			wantCloudErr: true,
 		},
 		{
 			input:        "s3://bucket/key",
-			wantErr:      "cloud destinations are not yet supported",
 			wantCloudErr: true,
 		},
 	}
@@ -109,9 +106,10 @@ func TestParseDestination(t *testing.T) {
 			if tc.wantErr != "" {
 				require.Error(t, err)
 				assert.Contains(t, err.Error(), tc.wantErr)
-				if tc.wantCloudErr {
-					assert.Contains(t, err.Error(), "./my-snapshot.zip")
-				}
+				return
+			}
+			if tc.wantCloudErr {
+				require.ErrorIs(t, err, snapshot.ErrCloudNotSupported)
 				return
 			}
 			require.NoError(t, err)