fix: address PR review findings

- Keep decodeErr as a hard failure in ensureDaemon even when
  ROBOREV_SKIP_VERSION_CHECK=1 is set, so non-daemon processes
  returning HTTP 200 are not falsely accepted.
- Trim trailing newline in compact mode to prevent TUI layout
  overflow by one line on every render frame.

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

Author: mariusvniekerk

cmd/roborev/daemon_lifecycle.go

@@ -143,10 +143,17 @@ func ensureDaemon() error {
 			}
 			decodeErr := json.NewDecoder(resp.Body).Decode(&status)
 
-			// Fail closed: restart if decode fails, version empty, or mismatch
-			if !skipVersionCheck && (decodeErr != nil || status.Version == "" || status.Version != version.Version) {
+			// Always fail on decode errors (response is not a valid daemon)
+			if decodeErr != nil {
 				if verbose {
-					fmt.Printf("Daemon version mismatch or unreadable (daemon: %s, cli: %s), restarting...\n", status.Version, version.Version)
+					fmt.Printf("Daemon response unreadable, restarting...\n")
+				}
+				return restartDaemon()
+			}
+			// Skip version mismatch check when env var is set
+			if !skipVersionCheck && (status.Version == "" || status.Version != version.Version) {
+				if verbose {
+					fmt.Printf("Daemon version mismatch (daemon: %s, cli: %s), restarting...\n", status.Version, version.Version)
 				}
 				return restartDaemon()
 			}
@@ -165,10 +172,17 @@ func ensureDaemon() error {
 		}
 		decodeErr := json.NewDecoder(resp.Body).Decode(&status)
 
-		// Fail closed: restart if decode fails, version empty, or mismatch
-		if !skipVersionCheck && (decodeErr != nil || status.Version == "" || status.Version != version.Version) {
+		// Always fail on decode errors (response is not a valid daemon)
+		if decodeErr != nil {
+			if verbose {
+				fmt.Printf("Daemon response unreadable, restarting...\n")
+			}
+			return restartDaemon()
+		}
+		// Skip version mismatch check when env var is set
+		if !skipVersionCheck && (status.Version == "" || status.Version != version.Version) {
 			if verbose {
-				fmt.Printf("Daemon version mismatch or unreadable (daemon: %s, cli: %s), restarting...\n", status.Version, version.Version)
+				fmt.Printf("Daemon version mismatch (daemon: %s, cli: %s), restarting...\n", status.Version, version.Version)
 			}
 			return restartDaemon()
 		}

cmd/roborev/tui/render_queue.go

@@ -306,10 +306,16 @@ func (m model) renderQueueView() string {
 		b.WriteString(renderHelpTable(m.queueHelpRows(), m.width))
 	}
 
-	b.WriteString("\x1b[K") // Clear to end of line (no newline at end)
-	b.WriteString("\x1b[J") // Clear to end of screen to prevent artifacts
+	output := b.String()
+	if compact {
+		// Trim trailing newline to avoid layout overflow (compact has no
+		// help footer to consume the final line).
+		output = strings.TrimSuffix(output, "\n")
+	}
+	output += "\x1b[K" // Clear to end of line (no newline at end)
+	output += "\x1b[J" // Clear to end of screen to prevent artifacts
 
-	return b.String()
+	return output
 }
 func (m model) calculateColumnWidths(idWidth int) columnWidths {
 	// Fixed widths: ID (idWidth), Status (8), P/F (3), Queued (12), Elapsed (8), Addressed (9)