feat(updater): report pack version in summaries (#69)

Author: Caedis

README.md

@@ -242,6 +242,7 @@ gtnh-daily-updater update-all main-client alt-server
 ## State, Paths, and Merge Behavior
 
 - Local state is stored at `<instance-dir>/.gtnh-daily-updater.json`
+- That state records `display_version`, the pack version as shown in game, so the next run can report what you upgraded from; instances updated before it existed fall back to the config version once
 - On Prism/MultiMC layouts, game files are resolved under `<instance-dir>/.minecraft/`
 - On server/other layouts, game files are resolved directly under `<instance-dir>/`
 - Config files are tracked in a git repo at `<game-dir>/.gtnh-configs/` on a `local` branch; pack updates are applied via `git merge -X theirs` (pack wins on conflicts)

cmd/status.go

@@ -22,7 +22,7 @@ var statusCmd = &cobra.Command{
 				instanceDir = *p.InstanceDir
 			}
 		}
-		return updater.Status(context.Background(), instanceDir, getGithubToken())
+		return updater.Status(context.Background(), instanceDir, getGithubToken(), getCurseForgeKey())
 	},
 }
 

cmd/summary_test.go

@@ -0,0 +1,72 @@
+package cmd
+
+import "testing"
+
+func TestVersionTransition(t *testing.T) {
+	tests := []struct {
+		name string
+		old  string
+		new  string
+		want string
+	}{
+		{
+			name: "version moved",
+			old:  "2.9.x (Daily 647) - 2026-07-27",
+			new:  "2.9.x (Daily 648) - 2026-07-28",
+			want: "2.9.x (Daily 647) - 2026-07-27 → 2.9.x (Daily 648) - 2026-07-28",
+		},
+		{
+			name: "version held still",
+			old:  "2.9.x (Daily 653+) - 2026-07-30",
+			new:  "2.9.x (Daily 653+) - 2026-07-30",
+			want: "2.9.x (Daily 653+) - 2026-07-30 (pack version unchanged)",
+		},
+		{
+			// Instances updated before display versions were tracked report the
+			// config tag on the left; the two sides differ, so it still reads
+			// as a transition.
+			name: "migration from a config tag",
+			old:  "2.9.0-nightly-2026-07-29-02",
+			new:  "2.9.x (Daily 653) - 2026-07-30",
+			want: "2.9.0-nightly-2026-07-29-02 → 2.9.x (Daily 653) - 2026-07-30",
+		},
+	}
+
+	for _, tc := range tests {
+		t.Run(tc.name, func(t *testing.T) {
+			if got := versionTransition(tc.old, tc.new); got != tc.want {
+				t.Errorf("versionTransition(%q, %q) = %q, want %q", tc.old, tc.new, got, tc.want)
+			}
+		})
+	}
+}
+
+func TestVersionCell(t *testing.T) {
+	tests := []struct {
+		name string
+		old  string
+		new  string
+		want string
+	}{
+		{
+			name: "version moved",
+			old:  "2.9.x (Daily 647) - 2026-07-27",
+			new:  "2.9.x (Daily 648) - 2026-07-28",
+			want: "2.9.x (Daily 647) - 2026-07-27 → 2.9.x (Daily 648) - 2026-07-28",
+		},
+		{
+			name: "version held still carries no note",
+			old:  "2.9.x (Daily 653+) - 2026-07-30",
+			new:  "2.9.x (Daily 653+) - 2026-07-30",
+			want: "2.9.x (Daily 653+) - 2026-07-30",
+		},
+	}
+
+	for _, tc := range tests {
+		t.Run(tc.name, func(t *testing.T) {
+			if got := versionCell(tc.old, tc.new); got != tc.want {
+				t.Errorf("versionCell(%q, %q) = %q, want %q", tc.old, tc.new, got, tc.want)
+			}
+		})
+	}
+}

cmd/update.go

@@ -50,12 +50,12 @@ var updateCmd = &cobra.Command{
 			return nil
 		}
 
-		logging.Infof("\nUpdate complete: %s → %s\n", result.OldVersion, result.NewVersion)
+		logging.Infof("\nUpdate complete: %s\n", versionTransition(result.OldVersion, result.NewVersion))
 		logging.Infof("  Mods: %d added, %d removed, %d updated, %d unchanged\n",
 			result.Added, result.Removed, result.Updated, result.Unchanged)
 
 		if result.ConfigUpdated {
-			logging.Infoln("  Pack configs: updated to new version")
+			logging.Infof("  Pack configs: %s → %s\n", result.OldConfigVersion, result.NewConfigVersion)
 		}
 
 		if len(result.StampedFiles) > 0 {
@@ -81,6 +81,24 @@ func init() {
 	rootCmd.AddCommand(updateCmd)
 }
 
+// versionTransition renders the headline form: an arrow when the pack version
+// moved, otherwise the current version with a note.
+func versionTransition(old, new string) string {
+	if old == new {
+		return new + " (pack version unchanged)"
+	}
+	return old + " → " + new
+}
+
+// versionCell renders the table form: an arrow when the pack version moved,
+// otherwise just the version. No prose — it sits in a column.
+func versionCell(old, new string) string {
+	if old == new {
+		return new
+	}
+	return old + " → " + new
+}
+
 func joinSkipped(s []string) string {
 	if len(s) == 0 {
 		return ""

cmd/update_all.go

@@ -136,11 +136,11 @@ var updateAllCmd = &cobra.Command{
 				continue
 			}
 
-			logging.Infof("\nUpdate complete: %s → %s\n", res.OldVersion, res.NewVersion)
+			logging.Infof("\nUpdate complete: %s\n", versionTransition(res.OldVersion, res.NewVersion))
 			logging.Infof("  Mods: %d added, %d removed, %d updated, %d unchanged\n",
 				res.Added, res.Removed, res.Updated, res.Unchanged)
 			if res.ConfigUpdated {
-				logging.Infoln("  Pack configs: updated to new version")
+				logging.Infof("  Pack configs: %s → %s\n", res.OldConfigVersion, res.NewConfigVersion)
 			}
 			if len(res.StampedFiles) > 0 {
 				logging.Infof("  Version stamped into %d file(s)\n", len(res.StampedFiles))
@@ -158,8 +158,8 @@ var updateAllCmd = &cobra.Command{
 				continue
 			}
 			modsChanged := r.result.Added + r.result.Removed + r.result.Updated
-			logging.Infof("  %-20s  OK     %s → %s   %d updated\n",
-				r.name, r.result.OldVersion, r.result.NewVersion, modsChanged)
+			logging.Infof("  %-20s  OK     %s   %d updated\n",
+				r.name, versionCell(r.result.OldVersion, r.result.NewVersion), modsChanged)
 		}
 
 		return firstErr

internal/config/state.go

@@ -11,13 +11,17 @@ import (
 const StateFile = ".gtnh-daily-updater.json"
 
 type LocalState struct {
-	Side          string                  `json:"side"`
-	Mode          string                  `json:"mode,omitempty"`
-	ManifestDate  string                  `json:"manifest_date"`
-	ConfigVersion string                  `json:"config_version"`
-	Mods          map[string]InstalledMod `json:"mods"`
-	ExcludeMods   []string                `json:"exclude_mods,omitempty"`
-	ExtraMods     map[string]ExtraModSpec `json:"extra_mods,omitempty"`
+	Side          string `json:"side"`
+	Mode          string `json:"mode,omitempty"`
+	ManifestDate  string `json:"manifest_date"`
+	ConfigVersion string `json:"config_version"`
+	// DisplayVersion is the pack version as shown in game, e.g.
+	// "2.9.x (Daily 648) - 2026-07-28". Empty on state files written before
+	// this was tracked.
+	DisplayVersion string                  `json:"display_version,omitempty"`
+	Mods           map[string]InstalledMod `json:"mods"`
+	ExcludeMods    []string                `json:"exclude_mods,omitempty"`
+	ExtraMods      map[string]ExtraModSpec `json:"extra_mods,omitempty"`
 }
 
 type ExtraModSpec struct {

internal/config/state_test.go

@@ -98,3 +98,49 @@ func TestGameDir(t *testing.T) {
 		}
 	})
 }
+
+func TestSaveAndLoadDisplayVersion(t *testing.T) {
+	t.Parallel()
+
+	tmp := t.TempDir()
+	s := &LocalState{
+		Side:           "client",
+		ManifestDate:   "2026-07-28",
+		ConfigVersion:  "2.9.0-nightly-2026-07-28",
+		DisplayVersion: "2.9.x (Daily 648) - 2026-07-28",
+	}
+	if err := s.Save(tmp); err != nil {
+		t.Fatalf("Save failed: %v", err)
+	}
+
+	loaded, err := Load(tmp)
+	if err != nil {
+		t.Fatalf("Load failed: %v", err)
+	}
+	if loaded.DisplayVersion != "2.9.x (Daily 648) - 2026-07-28" {
+		t.Errorf("DisplayVersion = %q, want %q", loaded.DisplayVersion, "2.9.x (Daily 648) - 2026-07-28")
+	}
+}
+
+func TestLoadStateWithoutDisplayVersion(t *testing.T) {
+	t.Parallel()
+
+	tmp := t.TempDir()
+	older := `{
+  "side": "client",
+  "manifest_date": "2026-07-27",
+  "config_version": "2.9.0-nightly-2026-07-27",
+  "mods": {}
+}`
+	if err := os.WriteFile(filepath.Join(tmp, StateFile), []byte(older), 0o644); err != nil {
+		t.Fatal(err)
+	}
+
+	loaded, err := Load(tmp)
+	if err != nil {
+		t.Fatalf("Load failed: %v", err)
+	}
+	if loaded.DisplayVersion != "" {
+		t.Errorf("DisplayVersion = %q, want empty for a pre-feature state file", loaded.DisplayVersion)
+	}
+}