fix(init): honor piped confirmation for 'init --here'; only fail-fast on empty stdin

The first version of this fix short-circuited on '_stdin_is_interactive()' (isatty) before typer.confirm, which broke 'init --here' when confirmation is piped (e.g. 'echo y | specify init --here' / CliRunner input='y\n') -- a non-TTY pipe with valid input was wrongly rejected, regressing test_init_here_without_force_preserves_shared_infra. Instead, call typer.confirm normally (piped 'y'/'n' is honored) and catch the Abort/EOFError it raises only when stdin is empty, converting that to the actionable '--force' guidance. This keeps the UX win for the no-input case without rejecting piped input.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Author: jawwad-ali

src/specify_cli/commands/init.py

@@ -229,19 +229,21 @@ def init(
                     console.print(
                         "[cyan]--force supplied: skipping confirmation and proceeding with merge[/cyan]"
                     )
-                elif not _stdin_is_interactive():
-                    # No TTY to confirm on: fail fast with actionable guidance
-                    # instead of blocking on typer.confirm (which would read EOF
-                    # and abort unhelpfully). Mirrors the named-project path,
-                    # which already errors and points to --force.
-                    console.print(
-                        "[red]Error:[/red] Current directory is not empty and no "
-                        "interactive terminal is available to confirm. Re-run with "
-                        "[bold]--force[/bold] to merge into it."
-                    )
-                    raise typer.Exit(1)
                 else:
-                    response = typer.confirm("Do you want to continue?")
+                    try:
+                        response = typer.confirm("Do you want to continue?")
+                    except (typer.Abort, EOFError):
+                        # No confirmation input available (non-interactive session
+                        # with empty stdin): fail fast with actionable guidance
+                        # instead of the bare "Aborted." Piped input (e.g. "y") is
+                        # still honored above. Mirrors the named-project path,
+                        # which already points to --force.
+                        console.print(
+                            "[red]Error:[/red] Current directory is not empty and no "
+                            "confirmation input is available. Re-run with "
+                            "[bold]--force[/bold] to merge into it."
+                        )
+                        raise typer.Exit(1) from None
                     if not response:
                         console.print("[yellow]Operation cancelled[/yellow]")
                         raise typer.Exit(0)

tests/integrations/test_cli.py

@@ -121,16 +121,14 @@ def fail_select(*_args, **_kwargs):
         data = json.loads((project / ".specify" / "integration.json").read_text(encoding="utf-8"))
         assert data["integration"] == specify_cli.DEFAULT_INIT_INTEGRATION
 
-    def test_init_here_nonempty_noninteractive_errors_with_force_guidance(self, tmp_path, monkeypatch):
-        """`init --here` on a non-empty directory must not block on a confirmation
-        prompt when there is no interactive terminal: it should fail fast with
-        guidance to use --force, instead of reading EOF and aborting unhelpfully."""
+    def test_init_here_nonempty_noninteractive_errors_with_force_guidance(self, tmp_path):
+        """`init --here` on a non-empty directory with no confirmation input (empty
+        stdin) must fail fast with guidance to use --force, instead of the bare
+        'Aborted.' from an EOF on typer.confirm. CliRunner with no `input=` provides
+        empty stdin, so typer.confirm raises Abort, which the command converts to the
+        actionable error."""
         from typer.testing import CliRunner
         from specify_cli import app
-        from specify_cli.commands import init as init_mod
-
-        # Deterministically exercise the non-interactive branch.
-        monkeypatch.setattr(init_mod, "_stdin_is_interactive", lambda: False)
 
         project = tmp_path / "nonempty-here"
         project.mkdir()