fix(tui): report a confirmed download cancellation

A cancel the engine-manager acknowledged retired the entry but left the
status line on "canceling ...", since only a failure updated it. Report
the cancellation, and keep it when the pull's own error-free reply
settles afterwards.

Signed-off-by: Chris Kelsey <ckelsey@nvidia.com>

Author: ckelseynv

services/nvpair-tui/ui/engines.go

@@ -76,6 +76,9 @@ type engineOpMsg struct {
 type enginePullDoneMsg struct {
 	pull enginePull
 	err  error
+	// canceled marks a cancel the engine-manager acknowledged, which only
+	// happens once the transfer has stopped.
+	canceled bool
 }
 
 var (
@@ -218,8 +221,11 @@ func (v *enginesView) Update(msg tea.Msg) tea.Cmd {
 
 	case enginePullDoneMsg:
 		v.retire(msg.pull)
-		if msg.err != nil {
+		switch {
+		case msg.err != nil:
 			v.status = fmt.Sprintf("pull %s %s failed: %s", msg.pull.engine, msg.pull.model, msg.err.Error())
+		case msg.canceled:
+			v.status = fmt.Sprintf("pull %s %s: canceled", msg.pull.engine, msg.pull.model)
 		}
 		return nil
 
@@ -340,7 +346,7 @@ func (v *enginesView) decodeCancel(pull enginePull) func(*rpc.Message, error) te
 		if err != nil {
 			return engineOpMsg{what: "cancel " + pull.model, engine: pull.engine, err: err}
 		}
-		return enginePullDoneMsg{pull: pull}
+		return enginePullDoneMsg{pull: pull, canceled: true}
 	}
 }
 

services/nvpair-tui/ui/engines_test.go

@@ -148,6 +148,15 @@ func TestConfirmedCancelRetiresTheDownload(t *testing.T) {
 	if _, ok := v.newestPull("ollama"); ok {
 		t.Error("a confirmed cancel left the download offered as a cancel target")
 	}
+	if want := "pull ollama llama3.2: canceled"; v.status != want {
+		t.Errorf("status = %q, want %q", v.status, want)
+	}
+	// The pull's own reply settles after the cancel and carries no error; it
+	// must not replace the outcome the cancel reported.
+	v.Update(enginePullDoneMsg{pull: pull})
+	if want := "pull ollama llama3.2: canceled"; v.status != want {
+		t.Errorf("status after the pull settled = %q, want %q", v.status, want)
+	}
 }
 
 // A non-terminal frame says the download is still going, so it must leave the