Improve rendering of the path in the success message

Author: anisaoshafi

internal/snapshot/destination.go

@@ -17,6 +17,22 @@ var (
 	ErrUnknownScheme = errors.New("unrecognized destination scheme")
 )
 
+// displayPath shortens abs for human-readable output:
+// under cwd → ./rel, under home → ~/rel, otherwise unchanged.
+func displayPath(abs, cwd, home string) string {
+	if cwd != "" {
+		if rel, err := filepath.Rel(cwd, abs); err == nil && !strings.HasPrefix(rel, "..") {
+			return "./" + filepath.ToSlash(rel)
+		}
+	}
+	if home != "" {
+		if rel, err := filepath.Rel(home, abs); err == nil && !strings.HasPrefix(rel, "..") {
+			return "~/" + filepath.ToSlash(rel)
+		}
+	}
+	return abs
+}
+
 // 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-a3f.zip", saved in the current working directory.

internal/snapshot/destination_test.go

@@ -16,6 +16,72 @@ import (
 
 
 
+func TestDisplayPath(t *testing.T) {
+	t.Parallel()
+
+	base := t.TempDir()
+	cwd := filepath.Join(base, "projects", "lstk")
+	home := filepath.Join(base, "home")
+
+	tests := []struct {
+		name string
+		abs  string
+		cwd  string
+		home string
+		want string
+	}{
+		{
+			name: "under cwd",
+			abs:  filepath.Join(cwd, "snap.zip"),
+			cwd:  cwd, home: home,
+			want: "./snap.zip",
+		},
+		{
+			name: "under cwd subdir",
+			abs:  filepath.Join(cwd, "exports", "snap.zip"),
+			cwd:  cwd, home: home,
+			want: "./exports/snap.zip",
+		},
+		{
+			name: "under home but not cwd",
+			abs:  filepath.Join(home, "snap.zip"),
+			cwd:  cwd, home: home,
+			want: "~/snap.zip",
+		},
+		{
+			name: "under home subdir",
+			abs:  filepath.Join(home, "downloads", "snap.zip"),
+			cwd:  cwd, home: home,
+			want: "~/downloads/snap.zip",
+		},
+		{
+			name: "unrelated to both",
+			abs:  filepath.Join(base, "other", "snap.zip"),
+			cwd:  cwd, home: home,
+			want: filepath.Join(base, "other", "snap.zip"),
+		},
+		{
+			name: "empty cwd falls back to home",
+			abs:  filepath.Join(home, "snap.zip"),
+			cwd:  "", home: home,
+			want: "~/snap.zip",
+		},
+		{
+			name: "empty cwd and home returns absolute",
+			abs:  filepath.Join(base, "snap.zip"),
+			cwd:  "", home: "",
+			want: filepath.Join(base, "snap.zip"),
+		},
+	}
+
+	for _, tc := range tests {
+		t.Run(tc.name, func(t *testing.T) {
+			t.Parallel()
+			assert.Equal(t, tc.want, snapshot.DisplayPath(tc.abs, tc.cwd, tc.home))
+		})
+	}
+}
+
 func TestParseDestination(t *testing.T) {
 	t.Parallel()
 	wd, err := os.Getwd()

internal/snapshot/export_test.go

@@ -0,0 +1,3 @@
+package snapshot
+
+var DisplayPath = displayPath

internal/snapshot/save.go

@@ -44,7 +44,9 @@ func Save(ctx context.Context, rt runtime.Runtime, containers []config.Container
 	defer func() {
 		sink.Emit(output.SpinnerStop())
 		if retErr == nil {
-			sink.Emit(output.MessageEvent{Severity: output.SeveritySuccess, Text: fmt.Sprintf("Snapshot saved to %s", dest)})
+			cwd, _ := os.Getwd()
+			home, _ := os.UserHomeDir()
+			sink.Emit(output.MessageEvent{Severity: output.SeveritySuccess, Text: fmt.Sprintf("Snapshot saved to %s", displayPath(dest, cwd, home))})
 		}
 	}()
 

test/integration/snapshot_save_test.go

@@ -90,7 +90,7 @@ func TestSnapshotSaveCustomPath(t *testing.T) {
 	)
 	require.NoError(t, err, "lstk snapshot save failed: %s", stderr)
 	assert.Contains(t, stdout, "Snapshot saved")
-	assert.Contains(t, stdout, outPath)
+	assert.Contains(t, stdout, "./my-snap.zip")
 
 	data, err := os.ReadFile(outPath)
 	require.NoError(t, err, "output file should exist")