Standardize CLI confirmation skip flag to --yes/-y across all packages

Author: davegaeddert

plain-cache/plain/cache/cli.py

@@ -20,10 +20,10 @@ def clear_expired() -> None:
 
 
 @cli.command()
-@click.option("--force", is_flag=True)
-def clear_all(force: bool) -> None:
+@click.option("--yes", "-y", is_flag=True, help="Skip confirmation prompt.")
+def clear_all(yes: bool) -> None:
     """Clear all cache entries"""
-    if not force and not click.confirm(
+    if not yes and not click.confirm(
         "Are you sure you want to delete all cache items?"
     ):
         return

plain-jobs/plain/jobs/cli.py

@@ -165,9 +165,10 @@ def stats() -> None:
 
 
 @cli.command()
-def purge() -> None:
+@click.option("--yes", "-y", is_flag=True, help="Skip confirmation prompt.")
+def purge(yes: bool) -> None:
     """Delete all pending and running jobs"""
-    if not click.confirm(
+    if not yes and not click.confirm(
         "Are you sure you want to clear all running and pending jobs? This will delete all current Jobs and JobRequests"
     ):
         return

plain-observer/plain/observer/README.md

@@ -118,7 +118,7 @@ plain observer span <span_id>
 
 # Clear all trace data
 plain observer clear
-plain observer clear --force
+plain observer clear --yes
 ```
 
 The `observer request` command makes a request with tracing automatically enabled and returns structured JSON output including query counts, duplicate detection, issue analysis, and a span tree. The `--user` flag accepts a user ID or email.

plain-observer/plain/observer/cli.py

@@ -261,8 +261,8 @@ def span_to_dict(span: Span) -> dict[str, Any]:
 
 
 @observer_cli.command()
-@click.option("--force", is_flag=True, help="Skip confirmation prompt.")
-def clear(force: bool) -> None:
+@click.option("--yes", "-y", is_flag=True, help="Skip confirmation prompt.")
+def clear(yes: bool) -> None:
     """Clear all observer data"""
     query = Trace.query.all()
     trace_count = query.count()
@@ -271,7 +271,7 @@ def clear(force: bool) -> None:
         click.echo("No traces to clear.")
         return
 
-    if not force:
+    if not yes:
         confirm_msg = f"Are you sure you want to clear {trace_count} trace(s)? This will delete all observer data."
         click.confirm(confirm_msg, abort=True)
 

plain-portal/plain/portal/cli.py

@@ -52,9 +52,10 @@ def cli() -> None:
     default=DEFAULT_RELAY_HOST,
     hidden=True,
 )
-def start(writable: bool, timeout: int, relay_host: str) -> None:
+@click.option("--yes", "-y", is_flag=True, help="Skip confirmation prompt.")
+def start(writable: bool, timeout: int, relay_host: str, yes: bool) -> None:
     """Start a portal session on the remote machine."""
-    if writable:
+    if writable and not yes:
         if not click.confirm(
             "This session allows writes to the production database. Continue?"
         ):

plain-postgres/plain/postgres/backups/cli.py

@@ -138,11 +138,13 @@ def delete_backup(backup_name: str) -> None:
 
 
 @cli.command("clear")
-@click.confirmation_option(prompt="Are you sure you want to delete all backups?")
-def clear_backups() -> None:
+@click.option("--yes", "-y", is_flag=True, help="Skip confirmation prompt.")
+def clear_backups(yes: bool) -> None:
     """Clear all database backups"""
     backups_handler = DatabaseBackups()
     backups = backups_handler.find_backups()
+    if not yes:
+        click.confirm("Are you sure you want to delete all backups?", abort=True)
     for backup in backups:
         backup.delete()
     click.secho("All backups deleted", fg="green")

plain-postgres/plain/postgres/cli/core.py

@@ -61,8 +61,9 @@ def shell(parameters: tuple[str, ...]) -> None:
 @cli.command("drop-unknown-tables")
 @click.option(
     "--yes",
+    "-y",
     is_flag=True,
-    help="Skip confirmation prompt (for non-interactive use).",
+    help="Skip confirmation prompt.",
 )
 def drop_unknown_tables(yes: bool) -> None:
     """Drop all tables not associated with a Plain model"""

plain-postgres/plain/postgres/cli/migrations.py

@@ -777,8 +777,9 @@ def print_deps(node: Any) -> str:
 @cli.command("prune")
 @click.option(
     "--yes",
+    "-y",
     is_flag=True,
-    help="Skip confirmation prompt (for non-interactive use).",
+    help="Skip confirmation prompt.",
 )
 def prune(yes: bool) -> None:
     """Remove stale migration records from the database"""