fix: add proper semver comparison for updates

Author: FelixIsaac

claude-code-sync

@@ -606,6 +606,23 @@ get_remote_version() {
     echo "$content" | grep -oP 'VERSION="\K[^"]+' | head -1
 }
 
+compare_semver() {
+    # Returns: 1 if v1 > v2, 2 if v1 < v2, 0 if equal
+    local v1="$1" v2="$2"
+    local IFS='.'
+    read -ra parts1 <<< "$v1"
+    read -ra parts2 <<< "$v2"
+
+    for i in 0 1 2; do
+        local p1="${parts1[$i]:-0}"
+        local p2="${parts2[$i]:-0}"
+
+        if (( p1 > p2 )); then echo 1; return; fi
+        if (( p1 < p2 )); then echo 2; return; fi
+    done
+    echo 0
+}
+
 cmd_check_update() {
     log_info "Checking for updates..."
 
@@ -620,11 +637,17 @@ cmd_check_update() {
     echo "Local version:  v$VERSION"
     echo "Remote version: v$remote_version"
 
-    if [[ "$VERSION" == "$remote_version" ]]; then
+    local cmp
+    cmp=$(compare_semver "$remote_version" "$VERSION")
+
+    if [[ "$cmp" == "0" ]]; then
         log_success "You're up to date!"
-    else
+    elif [[ "$cmp" == "1" ]]; then
         echo ""
         echo -e "Update available! Run: ${BLUE}claude-code-sync update${NC}"
+    else
+        echo ""
+        echo -e "${YELLOW}You're ahead of remote (dev version?)${NC}"
     fi
 }
 
@@ -638,9 +661,15 @@ cmd_update() {
         die "Could not fetch remote version. Check your internet connection."
     fi
 
-    if [[ "$VERSION" == "$remote_version" ]]; then
+    local cmp
+    cmp=$(compare_semver "$remote_version" "$VERSION")
+
+    if [[ "$cmp" == "0" ]]; then
         log_success "Already up to date (v$VERSION)"
         return
+    elif [[ "$cmp" == "2" ]]; then
+        log_warn "Local version (v$VERSION) is newer than remote (v$remote_version). Skipping."
+        return
     fi
 
     log_info "Updating v$VERSION -> v$remote_version..."

claude-code-sync.ps1

@@ -628,6 +628,23 @@ function Get-RemoteVersion {
     return $null
 }
 
+function Compare-SemVer {
+    # Returns: 1 if v1 > v2, -1 if v1 < v2, 0 if equal
+    param([string]$v1, [string]$v2)
+
+    $parts1 = $v1 -split '\.' | ForEach-Object { [int]$_ }
+    $parts2 = $v2 -split '\.' | ForEach-Object { [int]$_ }
+
+    for ($i = 0; $i -lt 3; $i++) {
+        $p1 = if ($i -lt $parts1.Count) { $parts1[$i] } else { 0 }
+        $p2 = if ($i -lt $parts2.Count) { $parts2[$i] } else { 0 }
+
+        if ($p1 -gt $p2) { return 1 }
+        if ($p1 -lt $p2) { return -1 }
+    }
+    return 0
+}
+
 function Invoke-CheckUpdate {
     Write-Info "Checking for updates..."
     $remoteVersion = Get-RemoteVersion
@@ -640,12 +657,16 @@ function Invoke-CheckUpdate {
     Write-Host "Local version:  v$VERSION"
     Write-Host "Remote version: v$remoteVersion"
 
-    if ($VERSION -eq $remoteVersion) {
+    $cmp = Compare-SemVer $remoteVersion $VERSION
+    if ($cmp -eq 0) {
         Write-Success "You're up to date!"
-    } else {
+    } elseif ($cmp -gt 0) {
         Write-Host ""
         Write-Host "Update available! Run: " -NoNewline
         Write-Host "claude-code-sync update" -ForegroundColor Cyan
+    } else {
+        Write-Host ""
+        Write-Host "You're ahead of remote (dev version?)" -ForegroundColor Yellow
     }
 }
 
@@ -657,9 +678,13 @@ function Invoke-Update {
         throw "Could not fetch remote version. Check your internet connection."
     }
 
-    if ($VERSION -eq $remoteVersion) {
+    $cmp = Compare-SemVer $remoteVersion $VERSION
+    if ($cmp -eq 0) {
         Write-Success "Already up to date (v$VERSION)"
         return
+    } elseif ($cmp -lt 0) {
+        Write-Warn "Local version (v$VERSION) is newer than remote (v$remoteVersion). Skipping."
+        return
     }
 
     Write-Info "Updating v$VERSION -> v$remoteVersion..."