Fix status verbose checksums, list file count, and restore --force

- status --verbose: keep new checksums after compare (not empty map lookups)
- list: use len(manifest.Files) instead of snapshot filename length
- restore --force: remove existing path before creating symlinks
- Add regression tests; move design doc to docs/PROJECT_DESIGN.md; drop SYSTEMPROMPT.md

Author: Cod-e-Codes

README.md

@@ -24,12 +24,12 @@
 
 Download the latest release from [GitHub Releases](https://github.com/Cod-e-Codes/ignoregrets/releases/latest):
 
-- Linux (amd64): `ignoregrets_0.1.3_linux_amd64.tar.gz`
-- Linux (arm64): `ignoregrets_0.1.3_linux_arm64.tar.gz`
-- macOS (amd64): `ignoregrets_0.1.3_darwin_amd64.tar.gz`
-- macOS (arm64): `ignoregrets_0.1.3_darwin_arm64.tar.gz`
-- Windows (amd64): `ignoregrets_0.1.3_windows_amd64.zip`
-- Windows (arm64): `ignoregrets_0.1.3_windows_arm64.zip`
+- Linux (amd64): `ignoregrets_0.1.4_linux_amd64.tar.gz`
+- Linux (arm64): `ignoregrets_0.1.4_linux_arm64.tar.gz`
+- macOS (amd64): `ignoregrets_0.1.4_darwin_amd64.tar.gz`
+- macOS (arm64): `ignoregrets_0.1.4_darwin_arm64.tar.gz`
+- Windows (amd64): `ignoregrets_0.1.4_windows_amd64.zip`
+- Windows (arm64): `ignoregrets_0.1.4_windows_arm64.zip`
 
 SHA256 checksums are provided in `checksums.txt` on the release page.
 

RELEASE_NOTES.md

@@ -1,3 +1,36 @@
+# ignoregrets v0.1.4
+
+Bug fix release for status, list, and restore behavior.
+
+## Changes
+
+- Fix `status --verbose` printing empty new checksums after comparison removed entries from the working map
+- Fix `list` showing snapshot filename length instead of manifest file count
+- Fix `restore --force` failing when replacing an existing symlink at the target path
+- Add regression tests for all three fixes
+- Move project design doc to `docs/PROJECT_DESIGN.md` and remove obsolete `SYSTEMPROMPT.md`
+
+## Installation
+
+### Windows
+1. Download `ignoregrets_v0.1.4_windows_amd64.exe`
+2. Rename to `ignoregrets.exe`
+3. Move to a directory in your PATH
+
+### Linux
+1. Download `ignoregrets_v0.1.4_linux_amd64`
+2. Make executable: `chmod +x ignoregrets_v0.1.4_linux_amd64`
+3. Move to `/usr/local/bin/ignoregrets`
+
+### macOS
+1. Download `ignoregrets_v0.1.4_darwin_amd64`
+2. Make executable: `chmod +x ignoregrets_v0.1.4_darwin_amd64`
+3. Move to `/usr/local/bin/ignoregrets`
+
+For more information, see the [README](README.md).
+
+---
+
 # ignoregrets v0.1.2
 
 Patch release with code quality improvements.

SYSTEMPROMPT.md

@@ -28,10 +28,10 @@ Snapshots are sorted by commit hash and timestamp.`,
 		}
 
 		type snapshotInfo struct {
-			path      string
 			commit    string
 			timestamp time.Time
 			index     int
+			fileCount int
 		}
 
 		var snapshots []snapshotInfo
@@ -53,10 +53,10 @@ Snapshots are sorted by commit hash and timestamp.`,
 				}
 
 				snapshots = append(snapshots, snapshotInfo{
-					path:      file.Name(),
 					commit:    manifest.CommitHash,
 					timestamp: manifest.Timestamp,
 					index:     manifest.Index,
+					fileCount: len(manifest.Files),
 				})
 			}
 		}
@@ -89,7 +89,7 @@ Snapshots are sorted by commit hash and timestamp.`,
 			fmt.Printf("  [%d] %s (%d files)\n",
 				s.index,
 				s.timestamp.Format("2006-01-02 15:04:05"),
-				len(s.path))
+				s.fileCount)
 		}
 
 		return nil

cmd/list.go

@@ -0,0 +1,90 @@
+package cmd
+
+import (
+	"archive/tar"
+	"compress/gzip"
+	"encoding/json"
+	"os"
+	"path/filepath"
+	"testing"
+	"time"
+
+	"github.com/Cod-e-Codes/ignoregrets/internal/snapshot"
+)
+
+func TestListUsesManifestFileCount(t *testing.T) {
+	dir := t.TempDir()
+	snapshotsDir := filepath.Join(dir, ".ignoregrets", "snapshots")
+	if err := os.MkdirAll(snapshotsDir, 0755); err != nil {
+		t.Fatal(err)
+	}
+
+	manifest := &snapshot.Manifest{
+		CommitHash: "abc123",
+		Timestamp:  time.Date(2024, 1, 2, 15, 4, 0, 0, time.UTC),
+		Index:      0,
+		Files: map[string]string{
+			"a.txt": "111",
+			"b.txt": "222",
+			"c.txt": "333",
+		},
+	}
+
+	// Filename length is unrelated to archived file count.
+	longName := "abc123_20240102T1504_0.tar.gz"
+	snapshotPath := filepath.Join(snapshotsDir, longName)
+	if err := writeTestSnapshot(snapshotPath, manifest); err != nil {
+		t.Fatal(err)
+	}
+
+	f, err := os.Open(snapshotPath)
+	if err != nil {
+		t.Fatal(err)
+	}
+	readManifest, err := snapshot.ReadManifest(f)
+	f.Close()
+	if err != nil {
+		t.Fatal(err)
+	}
+
+	fileCount := len(readManifest.Files)
+	if fileCount != 3 {
+		t.Fatalf("fileCount = %d, want 3 from manifest", fileCount)
+	}
+	if fileCount == len(longName) {
+		t.Fatal("file count must not equal snapshot filename length")
+	}
+}
+
+func writeTestSnapshot(path string, manifest *snapshot.Manifest) error {
+	file, err := os.Create(path)
+	if err != nil {
+		return err
+	}
+	defer file.Close()
+
+	gw := gzip.NewWriter(file)
+	defer gw.Close()
+
+	tw := tar.NewWriter(gw)
+	defer tw.Close()
+
+	manifestData, err := json.Marshal(manifest)
+	if err != nil {
+		return err
+	}
+
+	hdr := &tar.Header{
+		Name: "manifest.json",
+		Mode: 0644,
+		Size: int64(len(manifestData)),
+	}
+	if err := tw.WriteHeader(hdr); err != nil {
+		return err
+	}
+	if _, err := tw.Write(manifestData); err != nil {
+		return err
+	}
+
+	return nil
+}