sweatstack-cli 0.8.0__tar.gz → 0.9.0__tar.gz

{sweatstack_cli-0.8.0 → sweatstack_cli-0.9.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: sweatstack-cli
-Version: 0.8.0
+Version: 0.9.0
 Summary: Command-line interface for SweatStack — the sports data platform for developers
 Project-URL: Homepage, https://sweatstack.no
 Project-URL: Documentation, https://docs.sweatstack.no
@@ -88,7 +88,7 @@ sweatstack page deploy
 
 | Command | Description |
 |---------|-------------|
-| `sweatstack app create NAME` | Create a new private OAuth2 application |
+| `sweatstack app create [NAME]` | Create a new private OAuth2 application |
 | `sweatstack app link` | Link current directory to an existing application |
 
 #### `app create` options

{sweatstack_cli-0.8.0 → sweatstack_cli-0.9.0}/README.md

@@ -58,7 +58,7 @@ sweatstack page deploy
 
 | Command | Description |
 |---------|-------------|
-| `sweatstack app create NAME` | Create a new private OAuth2 application |
+| `sweatstack app create [NAME]` | Create a new private OAuth2 application |
 | `sweatstack app link` | Link current directory to an existing application |
 
 #### `app create` options

{sweatstack_cli-0.8.0 → sweatstack_cli-0.9.0}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "sweatstack-cli"
-version = "0.8.0"
+version = "0.9.0"
 description = "Command-line interface for SweatStack — the sports data platform for developers"
 readme = "README.md"
 authors = [{ name = "Aart Goossens", email = "aart@sweatstack.no" }]

{sweatstack_cli-0.8.0 → sweatstack_cli-0.9.0}/src/sweatstack_cli/__init__.py

@@ -1,3 +1,3 @@
 """SweatStack CLI — Command-line interface for the SweatStack platform."""
 
-__version__ = "0.8.0"
+__version__ = "0.9.0"

{sweatstack_cli-0.8.0 → sweatstack_cli-0.9.0}/src/sweatstack_cli/commands/app.py

@@ -17,9 +17,14 @@ from sweatstack_cli.project import CONFIG_FILENAME, write_config
 app = typer.Typer(name="app", help="Manage OAuth2 applications.")
 
 
+def _is_interactive() -> bool:
+    """Check if stdin is a TTY (interactive terminal)."""
+    return sys.stdin.isatty()
+
+
 @app.command()
 def create(
-    name: str = typer.Argument(..., help="Application name."),
+    name: str | None = typer.Argument(None, help="Application name."),
     description: str | None = typer.Option(
         None,
         "--description",
@@ -59,6 +64,32 @@ def create(
         console.print("[red]Error:[/red] --env and --env-file are mutually exclusive.")
         raise typer.Exit(1)
 
+    interactive = _is_interactive()
+
+    # Resolve name
+    if name is None:
+        if not interactive:
+            console.print("[red]Error:[/red] App name is required.")
+            raise typer.Exit(1)
+        name = typer.prompt("App name")
+
+    # Prompt for optional fields in interactive mode
+    if interactive:
+        if description is None:
+            description = typer.prompt(
+                "Description (optional)", default="", show_default=False
+            )
+            if not description:
+                description = None
+        if page is None:
+            page = typer.prompt(
+                "Page slug (optional)", default="", show_default=False
+            )
+            if not page:
+                page = None
+        if not secret:
+            secret = typer.confirm("Generate client secret?", default=False)
+
     body: dict[str, str | bool] = {"name": name}
 
     if description:
@@ -90,7 +121,7 @@ def create(
     # Write sweatstack.toml (protect existing file)
     config_path = Path(CONFIG_FILENAME)
     if config_path.exists():
-        if not sys.stdin.isatty():
+        if not interactive:
             console.print(
                 f"[red]Error:[/red] {CONFIG_FILENAME} already exists. "
                 "Remove it first or run interactively."

{sweatstack_cli-0.8.0 → sweatstack_cli-0.9.0}/tests/test_commands/test_app.py

@@ -3,7 +3,7 @@
 from __future__ import annotations
 
 from pathlib import Path
-from unittest.mock import MagicMock, patch
+from unittest.mock import MagicMock, PropertyMock, patch
 
 import pytest
 from typer.testing import CliRunner
@@ -247,3 +247,106 @@ class TestAppCreate:
             assert result.exit_code == 0
             assert "won't be shown again" not in result.stdout
             assert "written to" in result.stdout
+
+    def test_create_noninteractive_requires_name(self) -> None:
+        """Should error when name is missing in non-interactive mode."""
+        result = runner.invoke(app, ["app", "create"])
+
+        assert result.exit_code == 1
+        assert "App name is required" in result.stdout
+
+    def test_create_interactive_full(self, mock_api: MagicMock, tmp_path: Path) -> None:
+        """Should prompt for all fields when no args given interactively."""
+        mock_api.post.return_value = {"name": "My App", "id": "abc123"}
+
+        with patch("sweatstack_cli.commands.app._is_interactive", return_value=True):
+            with runner.isolated_filesystem(temp_dir=tmp_path):
+                result = runner.invoke(
+                    app, ["app", "create"], input="My App\n\n\nn\n"
+                )
+
+        assert result.exit_code == 0
+        assert "My App" in result.stdout
+        mock_api.post.assert_called_once_with(
+            "/api/v1/applications",
+            json={"name": "My App"},
+        )
+
+    def test_create_interactive_with_values(
+        self, mock_api: MagicMock, tmp_path: Path
+    ) -> None:
+        """Should send all interactively-provided values to the API."""
+        mock_api.post.return_value = {
+            "name": "My App",
+            "id": "abc123",
+            "client_secret": "secret456",
+        }
+
+        with patch("sweatstack_cli.commands.app._is_interactive", return_value=True):
+            with runner.isolated_filesystem(temp_dir=tmp_path):
+                result = runner.invoke(
+                    app,
+                    ["app", "create"],
+                    input="My App\nA cool app\nmy-page\ny\n",
+                )
+
+        assert result.exit_code == 0
+        mock_api.post.assert_called_once_with(
+            "/api/v1/applications",
+            json={
+                "name": "My App",
+                "description": "A cool app",
+                "page": "my-page",
+                "generate_secret": True,
+            },
+        )
+
+    def test_create_interactive_partial(
+        self, mock_api: MagicMock, tmp_path: Path
+    ) -> None:
+        """Should only prompt for fields not provided via args/flags."""
+        mock_api.post.return_value = {"name": "My App", "id": "abc123"}
+
+        with patch("sweatstack_cli.commands.app._is_interactive", return_value=True):
+            with runner.isolated_filesystem(temp_dir=tmp_path):
+                result = runner.invoke(
+                    app, ["app", "create", "My App"], input="\n\nn\n"
+                )
+
+        assert result.exit_code == 0
+        assert "App name" not in result.stdout
+        mock_api.post.assert_called_once_with(
+            "/api/v1/applications",
+            json={"name": "My App"},
+        )
+
+    def test_create_interactive_skips_provided_flags(
+        self, mock_api: MagicMock, tmp_path: Path
+    ) -> None:
+        """Should skip prompts for fields already provided via flags."""
+        mock_api.post.return_value = {
+            "name": "Test App",
+            "id": "abc123",
+            "client_secret": "secret456",
+        }
+
+        with patch("sweatstack_cli.commands.app._is_interactive", return_value=True):
+            with runner.isolated_filesystem(temp_dir=tmp_path):
+                # --secret and --description provided; only page is prompted
+                result = runner.invoke(
+                    app,
+                    ["app", "create", "Test App", "--secret", "-d", "Desc"],
+                    input="\n",
+                )
+
+        assert result.exit_code == 0
+        assert "Generate client secret?" not in result.stdout
+        assert "Description" not in result.stdout
+        mock_api.post.assert_called_once_with(
+            "/api/v1/applications",
+            json={
+                "name": "Test App",
+                "description": "Desc",
+                "generate_secret": True,
+            },
+        )