Merge pull request #45 from blockopsnetwork/chore/update-tui-upgrade-display

Update tui upgrade display

Author: haroldsphinx

cmd/tui.go

@@ -100,6 +100,7 @@ type tuiModel struct {
 	cancelThinking      context.CancelFunc
 	showHelp            bool
 	animationFrame      int
+	updateNotice        string
 
 	isStreaming        bool
 	streamingMessageID string
@@ -615,11 +616,7 @@ func (m *tuiModel) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
 		}
 
 	case updateNotice:
-		m.messages = append(m.messages, ChatMessage{
-			Role:      "system",
-			Content:   fmt.Sprintf("Update available: %s. Run `ponos upgrade` to update.", msg.latest),
-			Timestamp: time.Now(),
-		})
+		m.updateNotice = msg.latest
 		m.updateViewportContent()
 
 	case tea.Msg:
@@ -692,6 +689,9 @@ func (m *tuiModel) View() string {
 	sections = append(sections, m.textarea.View())
 
 	leftHelp := "/help for help and / to see available commands"
+	if m.updateNotice != "" {
+		leftHelp = fmt.Sprintf("Update available: %s (run `ponos upgrade`) • %s", m.updateNotice, leftHelp)
+	}
 	rightHelp := "⌘P to generate a command to know what ponos can do for you"
 	if m.loading {
 		indicator := m.getAnimatedIndicator()

cmd/update_check.go

@@ -4,6 +4,7 @@ import (
 	"encoding/json"
 	"net/http"
 	"os"
+	"strconv"
 	"strings"
 	"time"
 
@@ -25,10 +26,50 @@ func (tui *PonosAgentTUI) checkForUpdates(program *tea.Program) {
 	if err != nil || latest == "" {
 		return
 	}
-	if latest == version {
-		return
+	if isNewerVersion(version, latest) {
+		program.Send(updateNotice{latest: latest})
+	}
+}
+
+func isNewerVersion(current, latest string) bool {
+	currentParts, ok := parseSemver(current)
+	if !ok {
+		return false
+	}
+	latestParts, ok := parseSemver(latest)
+	if !ok {
+		return false
+	}
+	for i := 0; i < 3; i++ {
+		if latestParts[i] > currentParts[i] {
+			return true
+		}
+		if latestParts[i] < currentParts[i] {
+			return false
+		}
+	}
+	return false
+}
+
+func parseSemver(value string) ([3]int, bool) {
+	var out [3]int
+	clean := strings.TrimPrefix(strings.TrimSpace(value), "v")
+	parts := strings.Split(clean, ".")
+	if len(parts) < 3 {
+		return out, false
+	}
+	for i := 0; i < 3; i++ {
+		part := parts[i]
+		if dash := strings.Index(part, "-"); dash >= 0 {
+			part = part[:dash]
+		}
+		n, err := strconv.Atoi(part)
+		if err != nil {
+			return out, false
+		}
+		out[i] = n
 	}
-	program.Send(updateNotice{latest: latest})
+	return out, true
 }
 
 func fetchLatestTag() (string, error) {

cmd/upgrade.go

@@ -300,5 +300,8 @@ func replaceBinary(targetPath, newBinaryPath string) error {
 		return fmt.Errorf("failed to write temp binary: %w", err)
 	}
 
-	return os.Rename(tmpName, targetPath)
+	if err := os.Rename(tmpName, targetPath); err != nil {
+		return err
+	}
+	return os.Chmod(targetPath, 0o755)
 }