prune: make dry-run the default, require --force to delete

Change the prune command to be safe by default. Running 'tk prune'
now previews what would be deleted without actually deleting anything.
Users must pass -f/--force to confirm the destructive operation.

Author: cpjolicoeur

CHANGELOG.md

@@ -7,7 +7,7 @@
 - `prune` command to delete old closed tickets
   - `--days=N` to set age threshold (default: 7 days)
   - `--all` to prune regardless of age
-  - `--dry-run` to preview without deleting
+  - `-f/--force` to actually delete (dry-run preview by default)
 - `closed_at` timestamp field set when tickets are closed
 - Dependency protection: closed tickets referenced by open/in-progress tickets are never pruned
 

README.md

@@ -101,10 +101,10 @@ Commands:
   ready                    List open/in-progress tickets with deps resolved
   blocked                  List open/in-progress tickets with unresolved deps
   closed [--limit=N]       List recently closed tickets (default 20, by mtime)
-  prune [options]          Delete old closed tickets
+  prune [options]          Delete old closed tickets (dry-run by default)
     --days=N               Only prune tickets closed > N days ago [default: 7]
     --all                  Prune all eligible closed tickets
-    --dry-run              Preview what would be deleted
+    -f, --force            Actually delete (default is dry-run preview)
   show <id>                Display ticket
   edit <id>                Open ticket in $EDITOR
   add-note <id> [text]     Append timestamped note (or pipe via stdin)

ticket

@@ -1209,7 +1209,7 @@ cmd_prune() {
 
     local days=7
     local prune_all=0
-    local dry_run=0
+    local force=0
 
     # Parse arguments
     while [[ $# -gt 0 ]]; do
@@ -1227,13 +1227,13 @@ cmd_prune() {
                 prune_all=1
                 shift
                 ;;
-            --dry-run)
-                dry_run=1
+            -f|--force)
+                force=1
                 shift
                 ;;
             *)
                 echo "Error: unknown option '$1'" >&2
-                echo "Usage: $(basename "$0") prune [--days=N] [--all] [--dry-run]" >&2
+                echo "Usage: $(basename "$0") prune [--days=N] [--all] [--force]" >&2
                 return 1
                 ;;
         esac
@@ -1325,11 +1325,11 @@ cmd_prune() {
 
         # Output the ticket info
         local prefix=""
-        [[ "$dry_run" -eq 1 ]] && prefix="[DRY-RUN] "
+        [[ "$force" -eq 0 ]] && prefix="[DRY-RUN] "
         echo "${prefix}${id}: $title (closed $closed_display)"
 
-        # Delete unless dry-run
-        if [[ "$dry_run" -eq 0 ]]; then
+        # Delete only if --force is set
+        if [[ "$force" -eq 1 ]]; then
             rm "$filepath"
         fi
 
@@ -1338,8 +1338,8 @@ cmd_prune() {
 
     # Summary line
     if [[ "$count" -gt 0 ]]; then
-        if [[ "$dry_run" -eq 1 ]]; then
-            echo "Would prune $count tickets"
+        if [[ "$force" -eq 0 ]]; then
+            echo "Would prune $count tickets (use --force to delete)"
         else
             echo "Pruned $count tickets"
         fi
@@ -1379,10 +1379,10 @@ Commands:
   ready                    List open/in-progress tickets with deps resolved
   blocked                  List open/in-progress tickets with unresolved deps
   closed [--limit=N]       List recently closed tickets (default 20, by mtime)
-  prune [options]          Delete old closed tickets
+  prune [options]          Delete old closed tickets (dry-run by default)
     --days=N               Only prune tickets closed > N days ago [default: 7]
     --all                  Prune all eligible closed tickets
-    --dry-run              Preview what would be deleted
+    -f, --force            Actually delete (default is dry-run preview)
   show <id>                Display ticket
   edit <id>                Open ticket in \$EDITOR
   add-note <id> [text]     Append timestamped note (or pipe via stdin)