Remove deprecated channel-based spinner API

cmd/workspace/repos/overrides.go

@@ -141,10 +141,10 @@ func updateOverride(updateCmd *cobra.Command, updateReq *workspace.UpdateRepoReq
 func repoArgumentToRepoID(ctx context.Context, w *databricks.WorkspaceClient, args []string) (int64, error) {
 	// ---- Begin copy from cmd/workspace/repos/repos.go ----
 	if len(args) == 0 {
-		promptSpinner := cmdio.Spinner(ctx)
-		promptSpinner <- "No REPO_ID argument specified. Loading names for Repos drop-down."
+		sp := cmdio.NewSpinner(ctx)
+		sp.Update("No REPO_ID argument specified. Loading names for Repos drop-down.")
 		names, err := w.Repos.RepoInfoPathToIdMap(ctx, workspace.ListReposRequest{})
-		close(promptSpinner)
+		sp.Close()
 		if err != nil {
 			return 0, fmt.Errorf("failed to load names for Repos drop-down. Please manually specify required arguments. Original error: %w", err)
 		}

libs/cmdio/io.go

@@ -181,11 +181,6 @@ func RunSelect(ctx context.Context, prompt *promptui.Select) (int, string, error
 	return prompt.Run()
 }
 
-func Spinner(ctx context.Context) chan string {
-	c := fromContext(ctx)
-	return c.Spinner(ctx)
-}
-
 // NewSpinner creates a new spinner for displaying progress indicators.
 // The returned spinner should be closed when done to release resources.
 //

libs/cmdio/spinner.go

@@ -165,31 +165,3 @@ func (c *cmdIO) NewSpinner(ctx context.Context) *spinner {
 
 	return sp
 }
-
-// Spinner returns a channel for updating spinner status messages.
-// Send messages to update the suffix, close the channel to stop.
-// The spinner runs until the channel is closed or context is cancelled.
-func (c *cmdIO) Spinner(ctx context.Context) chan string {
-	updates := make(chan string)
-	sp := c.NewSpinner(ctx)
-
-	// Bridge goroutine: channel -> spinner.Update()
-	go func() {
-		defer sp.Close()
-
-		for {
-			select {
-			case <-ctx.Done():
-				return
-			case msg, ok := <-updates:
-				if !ok {
-					// Channel closed
-					return
-				}
-				sp.Update(msg)
-			}
-		}
-	}()
-
-	return updates
-}