fix: correctly handle version check success with no updates

- Return latest version even when no update is needed
- Show "You're running the latest version" with version number
- Fix logic to properly distinguish between check failure and up-to-date

The issue was that when running the latest version, we returned an empty
string which made it look like the check failed, when it actually succeeded.

🤖 Generated with [Claude Code](https://claude.ai/code)

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

Author: kevinelliott

cmd/version.go

@@ -45,11 +45,11 @@ func runVersion(cmd *cobra.Command, args []string) {
 			fmt.Printf("   Latest version:  %s\n", latestVersion)
 			fmt.Printf("\n   Update with: brew upgrade agentpipe\n")
 			fmt.Printf("   Or download from: https://github.com/kevinelliott/agentpipe/releases/latest\n")
-		} else if latestVersion == "" {
-			// Silently skip if we couldn't determine the latest version (likely rate limited)
-			fmt.Printf("   ℹ️  Update check unavailable at this time\n")
+		} else if latestVersion != "" {
+			fmt.Printf("   ✅ You're running the latest version! (%s)\n", latestVersion)
 		} else {
-			fmt.Printf("   ✅ You're running the latest version!\n")
+			// Couldn't determine the latest version
+			fmt.Printf("   ℹ️  Update check unavailable at this time\n")
 		}
 	}
 }

internal/version/version.go

@@ -50,7 +50,8 @@ func CheckForUpdate() (bool, string, error) {
 	}
 	defer resp.Body.Close()
 
-	if resp.StatusCode != http.StatusFound {
+	// Check if we got a redirect (302 or 301)
+	if resp.StatusCode != http.StatusFound && resp.StatusCode != http.StatusMovedPermanently {
 		// Try the API as a fallback (will hit rate limits but worth a try)
 		return checkViaAPI()
 	}
@@ -78,7 +79,8 @@ func CheckForUpdate() (bool, string, error) {
 		return true, latestTag, nil
 	}
 
-	return false, "", nil
+	// Return false with the latest version so we know the check succeeded
+	return false, latestTag, nil
 }
 
 // checkViaAPI is a fallback that uses the GitHub API (subject to rate limits)