fix: preserve spinner state and use carriage return for animation

- Reassign model after Update() to preserve spinner state between iterations
- Add \r carriage return so frames overwrite instead of printing new lines
- Add \n after final success message for clean line ending

commit: bd06446e4255d656da8fe1d2d4ce56aadaa44ae9
author: brice · 8mo ago · 2026-02-13 18:35 UTC
parent: 0ad73b1
signature: unsigned

1 file · +3 -2

M internal/adapters/in/cli/images.go +3 -2


@@ -178,13 +178,14 @@ func pruneWithSpinner(ctx context.Context, client imagesClient, req dto.ImagePru
178	178		model.SetMessage(cliRenderSuccess("Pruning complete"))
179	179		updatedModel, _ := model.Update(spinner.TickMsg{})
180	180		if m, ok := updatedModel.(components.SpinnerModel); ok {
181		-	fmt.Print(m.View())
	181	+	fmt.Print("\r" + m.View() + "\n")
182	182		}
183	183		return result.resp, nil
184	184		case <-time.After(100 * time.Millisecond):
185	185		updatedModel, _ := model.Update(spinner.TickMsg{})
186	186		if m, ok := updatedModel.(components.SpinnerModel); ok {
187		-	fmt.Print(m.View())
	187	+	model = m
	188	+	fmt.Print("\r" + m.View())
188	189		}
189	190		}
190	191		}

 			model.SetMessage(cliRenderSuccess("Pruning complete"))
 			updatedModel, _ := model.Update(spinner.TickMsg{})
 			if m, ok := updatedModel.(components.SpinnerModel); ok {
-				fmt.Print(m.View())
+				fmt.Print("\r" + m.View() + "\n")
 			}
 			return result.resp, nil
 		case <-time.After(100 * time.Millisecond):
 			updatedModel, _ := model.Update(spinner.TickMsg{})
 			if m, ok := updatedModel.(components.SpinnerModel); ok {
-				fmt.Print(m.View())
+				model = m
+				fmt.Print("\r" + m.View())
 			}
 		}
 	}