Adjust snapshot name when destination not provided: append 3 char hex

Author: anisaoshafi

cmd/snapshot.go

@@ -33,7 +33,7 @@ func newSnapshotSaveCmd(cfg *env.Env) *cobra.Command {
 
 Pass [destination] as an absolute or relative path for the exported file:
 
-  lstk snapshot save                    # saves to ./snapshot-<YYYY-MM-DDTHH-mm-ss>.zip
+  lstk snapshot save                    # saves to ./snapshot-<YYYY-MM-DDTHH-mm-ss>-<hex>.zip
   lstk snapshot save ./my-snapshot.zip  # saves to ./my-snapshot.zip
   lstk snapshot save /tmp/my-state      # saves to /tmp/my-state.zip
 

internal/snapshot/destination.go

@@ -1,6 +1,7 @@
 package snapshot
 
 import (
+	"crypto/rand"
 	"errors"
 	"fmt"
 	"os"
@@ -18,11 +19,13 @@ var (
 
 // ParseDestination resolves the user-supplied path to an absolute local path.
 // When dest is empty, a default name based on now (UTC) is used, e.g.
-// "snapshot-2026-05-11T21-04-32.zip", saved in the current working directory.
+// "snapshot-2026-05-11T21-04-32-a3f.zip", saved in the current working directory.
 // The returned path always has a .zip extension.
 func ParseDestination(dest string, now time.Time) (string, error) {
 	if dest == "" {
-		dest = "./" + now.UTC().Format("snapshot-2006-01-02T15-04-05")
+		b := make([]byte, 2)
+		_, _ = rand.Read(b)
+		dest = "./" + now.UTC().Format("snapshot-2006-01-02T15-04-05") + "-" + fmt.Sprintf("%x", b)[:3]
 	} else {
 		lower := strings.ToLower(dest)
 		switch {

internal/snapshot/destination_test.go

@@ -3,6 +3,7 @@ package snapshot_test
 import (
 	"os"
 	"path/filepath"
+	"regexp"
 	"runtime"
 	"strings"
 	"testing"
@@ -30,20 +31,21 @@ func TestParseDestination(t *testing.T) {
 	require.NoError(t, os.Mkdir(subDir, 0o755))
 
 	type testCase struct {
-		name          string // optional; uses input when empty
-		input         string
-		wantPath      string
-		wantErr       string
-		wantRemoteErr bool
-		wantSchemeErr bool
+		name           string // optional; uses input when empty
+		input          string
+		wantPath       string
+		wantPathRegexp string // used instead of wantPath when the result contains a random component
+		wantErr        string
+		wantRemoteErr  bool
+		wantSchemeErr  bool
 	}
 
 	tests := []testCase{
 		// --- default (empty input) ---
 		{
-			name:     "default",
-			input:    "",
-			wantPath: filepath.Join(wd, "snapshot-2026-05-11T21-04-32.zip"),
+			name:           "default",
+			input:          "",
+			wantPathRegexp: regexp.QuoteMeta(filepath.Join(wd, "snapshot-2026-05-11T21-04-32-")) + `[0-9a-f]{3}\.zip`,
 		},
 
 		// --- local paths ---
@@ -178,7 +180,11 @@ func TestParseDestination(t *testing.T) {
 				return
 			}
 			require.NoError(t, err)
-			assert.Equal(t, tc.wantPath, got)
+			if tc.wantPathRegexp != "" {
+				assert.Regexp(t, tc.wantPathRegexp, got)
+			} else {
+				assert.Equal(t, tc.wantPath, got)
+			}
 		})
 	}
 }