Skip to content

Commit 25546e5

Browse files
committed
Remove stale '--force' message from interactive clean preview
The non-force mode now uses an interactive confirmation prompt instead of being a pure dry-run. The old 'Run with --force to delete these items.' message was contradictory when shown right before the confirmation dialog. Remove the stale message and update the test to verify it's no longer present.
1 parent 1fc8030 commit 25546e5

File tree

2 files changed

+3
-4
lines changed

2 files changed

+3
-4
lines changed

cmd/entire/cli/clean.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -246,7 +246,6 @@ func runCleanWithItems(ctx context.Context, w io.Writer, force bool, items []str
246246
fmt.Fprintln(w)
247247
}
248248

249-
fmt.Fprintln(w, "Run with --force to delete these items.")
250249
return nil
251250
}
252251

cmd/entire/cli/clean_test.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -143,9 +143,9 @@ func TestRunClean_PreviewMode(t *testing.T) {
143143
t.Errorf("Should not list '%s', got: %s", paths.MetadataBranchName, output)
144144
}
145145

146-
// Should prompt to use --force
147-
if !strings.Contains(output, "--force") {
148-
t.Errorf("Expected '--force' prompt in output, got: %s", output)
146+
// Should NOT contain stale --force message (interactive prompt handles confirmation)
147+
if strings.Contains(output, "--force") {
148+
t.Errorf("Should not contain '--force' message in interactive mode, got: %s", output)
149149
}
150150

151151
// Branches should still exist (preview mode doesn't delete)

0 commit comments

Comments
 (0)