fix(status): mirror checked_at/is_prerelease in status update object; mark FR-002 delta follow-up

Review follow-ups on the US1 status slice (specs/079-upgrade-nudge):

- StatusUpdateInfo now carries checked_at (RFC 3339 string, staleness
  signal for consumers of `status -o json`) and is_prerelease, so it is
  an accurate mirror of the /api/v1/info update object as the code
  comment and docs claim (FR-019/FR-021). Extraction + JSON wire-format
  tests extended; docs example updated.
- Added explicit TODO(spec-079/FR-002) markers at the two surfaces that
  will grow the "N releases / M weeks behind" delta (statusVersionSuffix
  and the checker's one-shot Info log). The delta needs the release list
  + publish dates, which the checker does not fetch yet — owned by a
  later 079 slice, intentionally not in this one.
- FR-004 once-per-version (vs once-per-process) Info announce is kept
  as-is: the operative clause is "MUST NOT repeatedly log the same
  availability on a timer", and per-version re-announce matches the
  Web UI banner's per-version dismissal semantics (FR-005); a genuinely
  newer release is new information. Locked in by checker_test.go.

Testing: go test -race ./internal/updatecheck/... ./cmd/mcpproxy/...;
golangci-lint v2 (.github/.golangci.yml) on touched packages: clean.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

Author: Dumbris

cmd/mcpproxy/status_cmd.go

@@ -45,6 +45,8 @@ type StatusUpdateInfo struct {
 	Available     bool   `json:"available"`
 	LatestVersion string `json:"latest_version,omitempty"`
 	ReleaseURL    string `json:"release_url,omitempty"`
+	CheckedAt     string `json:"checked_at,omitempty"` // RFC 3339, as serialized by the daemon
+	IsPrerelease  bool   `json:"is_prerelease,omitempty"`
 	CheckError    string `json:"check_error,omitempty"`
 }
 
@@ -300,6 +302,12 @@ func extractStatusUpdate(infoData map[string]interface{}) *StatusUpdateInfo {
 	if v, ok := updateData["release_url"].(string); ok {
 		u.ReleaseURL = v
 	}
+	if v, ok := updateData["checked_at"].(string); ok {
+		u.CheckedAt = v
+	}
+	if v, ok := updateData["is_prerelease"].(bool); ok {
+		u.IsPrerelease = v
+	}
 	if v, ok := updateData["check_error"].(string); ok {
 		u.CheckError = v
 	}
@@ -309,6 +317,11 @@ func extractStatusUpdate(infoData map[string]interface{}) *StatusUpdateInfo {
 // statusVersionSuffix renders the update annotation appended to the Version
 // line, mirroring doctor's presentation. A failed or not-yet-completed check
 // renders nothing (quiet on failure; the error stays in JSON for diagnostics).
+//
+// TODO(spec-079/FR-002): extend the annotation with the human-readable
+// "N releases / M weeks behind" delta once internal/updatecheck computes it
+// (requires the release list + publish dates, not just the latest release;
+// additive per FR-021). This function is the single rendering point.
 func statusVersionSuffix(u *StatusUpdateInfo) string {
 	if u == nil || u.CheckError != "" {
 		return ""

cmd/mcpproxy/status_update_test.go

@@ -153,9 +153,10 @@ func TestExtractStatusUpdate(t *testing.T) {
 			"version": "v0.45.0",
 			"update": map[string]interface{}{
 				"available":      true,
-				"latest_version": "v0.46.0",
-				"release_url":    "https://github.com/smart-mcp-proxy/mcpproxy-go/releases/tag/v0.46.0",
-				"is_prerelease":  false,
+				"latest_version": "v0.46.0-rc.1",
+				"release_url":    "https://github.com/smart-mcp-proxy/mcpproxy-go/releases/tag/v0.46.0-rc.1",
+				"checked_at":     "2026-07-02T10:00:00Z",
+				"is_prerelease":  true,
 			},
 		}
 
@@ -166,12 +167,18 @@ func TestExtractStatusUpdate(t *testing.T) {
 		if !u.Available {
 			t.Error("expected Available=true")
 		}
-		if u.LatestVersion != "v0.46.0" {
-			t.Errorf("expected LatestVersion v0.46.0, got %q", u.LatestVersion)
+		if u.LatestVersion != "v0.46.0-rc.1" {
+			t.Errorf("expected LatestVersion v0.46.0-rc.1, got %q", u.LatestVersion)
 		}
-		if u.ReleaseURL != "https://github.com/smart-mcp-proxy/mcpproxy-go/releases/tag/v0.46.0" {
+		if u.ReleaseURL != "https://github.com/smart-mcp-proxy/mcpproxy-go/releases/tag/v0.46.0-rc.1" {
 			t.Errorf("unexpected ReleaseURL %q", u.ReleaseURL)
 		}
+		if u.CheckedAt != "2026-07-02T10:00:00Z" {
+			t.Errorf("expected CheckedAt to be preserved, got %q", u.CheckedAt)
+		}
+		if !u.IsPrerelease {
+			t.Error("expected IsPrerelease=true")
+		}
 	})
 
 	t.Run("check error preserved for machine output", func(t *testing.T) {
@@ -210,6 +217,8 @@ func TestStatusJSONIncludesUpdate(t *testing.T) {
 			Available:     true,
 			LatestVersion: "v0.46.0",
 			ReleaseURL:    "https://github.com/smart-mcp-proxy/mcpproxy-go/releases/tag/v0.46.0",
+			CheckedAt:     "2026-07-02T10:00:00Z",
+			IsPrerelease:  true,
 		},
 	}
 
@@ -234,13 +243,21 @@ func TestStatusJSONIncludesUpdate(t *testing.T) {
 		t.Errorf("expected update.latest_version v0.46.0, got %q", result.Update.LatestVersion)
 	}
 
-	// Field name in the wire format must be "update" (snake_case contract).
+	// Field names in the wire format must match the /api/v1/info update
+	// object (snake_case contract, FR-021).
 	var raw map[string]interface{}
 	if err := json.Unmarshal([]byte(output), &raw); err != nil {
 		t.Fatalf("invalid JSON: %v", err)
 	}
-	if _, ok := raw["update"]; !ok {
-		t.Error("expected raw JSON key 'update'")
+	rawUpdate, ok := raw["update"].(map[string]interface{})
+	if !ok {
+		t.Fatal("expected raw JSON key 'update'")
+	}
+	if v, ok := rawUpdate["checked_at"].(string); !ok || v != "2026-07-02T10:00:00Z" {
+		t.Errorf("expected raw JSON key 'update.checked_at', got %v", rawUpdate["checked_at"])
+	}
+	if v, ok := rawUpdate["is_prerelease"].(bool); !ok || !v {
+		t.Errorf("expected raw JSON key 'update.is_prerelease'=true, got %v", rawUpdate["is_prerelease"])
 	}
 }
 

docs/cli/status-command.md

@@ -138,7 +138,8 @@ mcpproxy status -o json
   "update": {
     "available": true,
     "latest_version": "v1.3.0",
-    "release_url": "https://github.com/smart-mcp-proxy/mcpproxy-go/releases/tag/v1.3.0"
+    "release_url": "https://github.com/smart-mcp-proxy/mcpproxy-go/releases/tag/v1.3.0",
+    "checked_at": "2026-07-02T10:00:00Z"
   }
 }
 ```
@@ -151,6 +152,8 @@ When the daemon is running, `status` surfaces the result of the background updat
 - **Up to date**: `Version: v1.3.0 (latest)`
 - **Check failed or not yet completed** (offline, rate-limited): the version is shown without any annotation. In JSON output the `update.check_error` field retains the failure reason for diagnostics.
 
+In machine-readable output (`-o json`/`-o yaml`) the `update` object also carries `checked_at` (when the last successful check ran, so consumers can judge staleness) and `is_prerelease` (whether the offered version is a prerelease), matching the `/api/v1/info` contract.
+
 ## API Key Masking
 
 By default, the API key is masked showing only the first 4 and last 4 characters:

internal/updatecheck/checker.go

@@ -167,6 +167,10 @@ func (c *Checker) updateVersionInfo(release *GitHubRelease, checkError string) {
 	case updateAvailable && latestVersion != c.announcedVersion:
 		// Announce each newly detected version exactly once per process;
 		// subsequent ticks for the same version log at Debug only.
+		// TODO(spec-079/FR-002): include the "N releases / M weeks behind"
+		// delta here once the checker fetches the release list + publish
+		// dates (a later 079 slice extending VersionInfo, additive per
+		// FR-021).
 		c.announcedVersion = latestVersion
 		c.logger.Info("Update available",
 			zap.String("current", c.version),