Adding confirmation to override a file during export and convert oper…

…ation in cli

marimo/_cli/convert/commands.py

@@ -8,6 +8,7 @@
 from marimo._cli.convert.markdown import convert_from_md
 from marimo._cli.convert.utils import load_external_file
 from marimo._cli.print import echo
+from marimo._cli.utils import prompt_to_overwrite
 from marimo._convert.ipynb import convert_from_ipynb
 from marimo._utils.paths import maybe_make_dirs
 
@@ -63,13 +64,12 @@ def convert(
 
     if output:
         output_path = Path(output)
-        if output_path.exists():
-            overwrite = click.confirm(
+        if prompt_to_overwrite(output_path):
+            confirmed = click.confirm(
                 f"Warning: The file '{output}' already exists. Overwrite?",
                 default=False,
             )
-            if not overwrite:
-                echo("No changes made. File was not overwritten.")
+            if not confirmed:
                 return
         # Make dirs if needed
         maybe_make_dirs(output)

marimo/_cli/export/commands.py

@@ -10,6 +10,7 @@
 
 from marimo._cli.parse_args import parse_args
 from marimo._cli.print import echo, green
+from marimo._cli.utils import prompt_to_overwrite
 from marimo._dependencies.dependencies import DependencyManager
 from marimo._server.export import (
     ExportResult,
@@ -57,16 +58,6 @@ def watch_and_export(
 
     def write_data(data: str) -> None:
         if output:
-            output_path = Path(output)
-            # Check if the file exists
-            if output_path.exists():
-                overwrite = click.confirm(
-                    f"Warning: The file '{output}' already exists. Overwrite?",
-                    default=False,
-                )
-                if not overwrite:
-                    echo("No changes made. File was not overwritten.")
-                    return
             # Make dirs if needed
             maybe_make_dirs(output)
             with open(output, "w", encoding="utf-8") as f:
@@ -76,6 +67,16 @@ def write_data(data: str) -> None:
             echo(data)
         return
 
+    if output:
+        output_path = Path(output)
+        if prompt_to_overwrite(output_path):
+            confirmed = click.confirm(
+                f"Warning: The file '{output_path}' already exists. Overwrite?",
+                default=False,
+            )
+            if not confirmed:
+                return
+
     # No watch, just run once
     if not watch:
         result = export_callback(marimo_path)

marimo/_cli/utils.py

@@ -0,0 +1,21 @@
+# Copyright 2024 Marimo. All rights reserved.
+
+from pathlib import Path
+from sys import stdin
+
+from marimo._config.settings import GLOBAL_SETTINGS
+
+
+def prompt_to_overwrite(path: Path) -> bool:
+    if GLOBAL_SETTINGS.YES:
+        return False
+
+    # Check if not in an interactive terminal
+    # default to False
+    if not stdin.isatty():
+        return False
+
+    if path.exists():
+        return True
+
+    return False