shotgun-sh 0.2.8.dev2__py3-none-any.whl → 0.3.3.dev1__py3-none-any.whl

shotgun/tui/screens/pipx_migration.py

@@ -0,0 +1,205 @@
+"""Migration notice screen for pipx users."""
+
+from __future__ import annotations
+
+from typing import TYPE_CHECKING
+
+from textual import on
+from textual.app import ComposeResult
+from textual.containers import Container, Horizontal, VerticalScroll
+from textual.events import Resize
+from textual.screen import ModalScreen
+from textual.widgets import Button, Label, Markdown
+
+from shotgun.tui.layout import COMPACT_HEIGHT_THRESHOLD
+
+if TYPE_CHECKING:
+    pass
+
+
+class PipxMigrationScreen(ModalScreen[None]):
+    """Modal screen warning pipx users about migration to uvx."""
+
+    CSS = """
+        PipxMigrationScreen {
+            align: center middle;
+        }
+
+        #migration-container {
+            width: 90;
+            height: auto;
+            max-height: 90%;
+            border: thick $error;
+            background: $surface;
+            padding: 2;
+        }
+
+        #migration-content {
+            height: 1fr;
+            padding: 1 0;
+        }
+
+        #buttons-container {
+            height: auto;
+            padding: 2 0 1 0;
+        }
+
+        #action-buttons {
+            width: 100%;
+            height: auto;
+            align: center middle;
+        }
+
+        .action-button {
+            margin: 0 1;
+            min-width: 20;
+        }
+
+        #migration-status {
+            height: auto;
+            padding: 1;
+            min-height: 1;
+            text-align: center;
+        }
+
+        /* Compact styles for short terminals */
+        #migration-container.compact {
+            padding: 1;
+            max-height: 98%;
+        }
+
+        #migration-content.compact {
+            padding: 0;
+        }
+
+        #buttons-container.compact {
+            padding: 1 0 0 0;
+        }
+
+        #migration-status.compact {
+            padding: 0;
+        }
+    """
+
+    BINDINGS = [
+        ("escape", "dismiss", "Continue Anyway"),
+        ("ctrl+c", "app.quit", "Quit"),
+    ]
+
+    def compose(self) -> ComposeResult:
+        """Compose the migration notice modal."""
+        with Container(id="migration-container"):
+            with VerticalScroll(id="migration-content"):
+                yield Markdown(
+                    """
+## We've Switched to uvx
+
+We've switched from `pipx` to `uvx` as the primary installation method due to critical build issues with our `kuzu` dependency.
+
+### The Problem
+Users with pipx encounter cmake build errors during installation because pip falls back to building from source instead of using pre-built binary wheels.
+
+### The Solution: uvx
+- ✅ **No build tools required** - Binary wheels enforced
+- ✅ **10-100x faster** - Much faster than pipx
+- ✅ **Better reliability** - No cmake/build errors
+
+### How to Migrate
+
+**1. Uninstall shotgun-sh from pipx:**
+```bash
+pipx uninstall shotgun-sh
+```
+
+**2. Install uv:**
+```bash
+curl -LsSf https://astral.sh/uv/install.sh | sh
+```
+Or with Homebrew: `brew install uv`
+
+**3. Run shotgun-sh with uvx:**
+```bash
+uvx shotgun-sh
+```
+Or install permanently: `uv tool install shotgun-sh`
+
+---
+
+### Need Help?
+
+**Discord:** https://discord.gg/5RmY6J2N7s
+
+**Full Migration Guide:** https://github.com/shotgun-sh/shotgun/blob/main/docs/PIPX_MIGRATION.md
+"""
+                )
+
+                with Container(id="buttons-container"):
+                    yield Label("", id="migration-status")
+                    with Horizontal(id="action-buttons"):
+                        yield Button(
+                            "Copy Instructions to Clipboard",
+                            variant="default",
+                            id="copy-instructions",
+                            classes="action-button",
+                        )
+                        yield Button(
+                            "Continue Anyway",
+                            variant="primary",
+                            id="continue",
+                            classes="action-button",
+                        )
+
+    def on_mount(self) -> None:
+        """Focus the continue button and ensure scroll starts at top."""
+        self.query_one("#continue", Button).focus()
+        self.query_one("#migration-content", VerticalScroll).scroll_home(animate=False)
+        # Apply compact layout if starting in a short terminal
+        self._apply_compact_layout(self.app.size.height < COMPACT_HEIGHT_THRESHOLD)
+
+    @on(Resize)
+    def handle_resize(self, event: Resize) -> None:
+        """Adjust layout based on terminal height."""
+        self._apply_compact_layout(event.size.height < COMPACT_HEIGHT_THRESHOLD)
+
+    def _apply_compact_layout(self, compact: bool) -> None:
+        """Apply or remove compact layout classes for short terminals."""
+        container = self.query_one("#migration-container")
+        content = self.query_one("#migration-content")
+        buttons_container = self.query_one("#buttons-container")
+        status = self.query_one("#migration-status")
+
+        if compact:
+            container.add_class("compact")
+            content.add_class("compact")
+            buttons_container.add_class("compact")
+            status.add_class("compact")
+        else:
+            container.remove_class("compact")
+            content.remove_class("compact")
+            buttons_container.remove_class("compact")
+            status.remove_class("compact")
+
+    @on(Button.Pressed, "#copy-instructions")
+    def _copy_instructions(self) -> None:
+        """Copy all migration instructions to clipboard."""
+        instructions = """# Step 1: Uninstall from pipx
+pipx uninstall shotgun-sh
+
+# Step 2: Install uv
+curl -LsSf https://astral.sh/uv/install.sh | sh
+
+# Step 3: Run shotgun with uvx
+uvx shotgun-sh"""
+        status_label = self.query_one("#migration-status", Label)
+        try:
+            import pyperclip  # type: ignore[import-untyped]  # noqa: PGH003
+
+            pyperclip.copy(instructions)
+            status_label.update("✓ Copied migration instructions to clipboard!")
+        except ImportError:
+            status_label.update("⚠️ Clipboard not available. See instructions above.")
+
+    @on(Button.Pressed, "#continue")
+    def _continue(self) -> None:
+        """Dismiss the modal and continue."""
+        self.dismiss()

shotgun/tui/screens/provider_config.py

@@ -7,11 +7,13 @@ from typing import TYPE_CHECKING, cast
 from textual import on
 from textual.app import ComposeResult
 from textual.containers import Horizontal, Vertical
+from textual.events import Resize
 from textual.reactive import reactive
 from textual.screen import Screen
 from textual.widgets import Button, Input, Label, ListItem, ListView, Markdown, Static
 
 from shotgun.agents.config import ConfigManager, ProviderType
+from shotgun.tui.layout import COMPACT_HEIGHT_THRESHOLD
 
 if TYPE_CHECKING:
     from ..app import ShotgunApp
@@ -77,6 +79,38 @@ class ProviderConfigScreen(Screen[None]):
         #provider-list {
             padding: 1;
         }
+        #provider-status {
+            height: auto;
+            padding: 0 1;
+            min-height: 1;
+        }
+        #provider-status.error {
+            color: $error;
+        }
+
+        /* Compact styles for short terminals */
+        ProviderConfigScreen.compact #titlebox {
+            margin: 0;
+            padding: 0;
+            border: none;
+        }
+
+        ProviderConfigScreen.compact #provider-config-summary {
+            display: none;
+        }
+
+        ProviderConfigScreen.compact #provider-links {
+            display: none;
+        }
+
+        ProviderConfigScreen.compact #provider-list {
+            margin: 0;
+            padding: 0;
+        }
+
+        ProviderConfigScreen.compact #provider-actions {
+            padding: 0;
+        }
     """
 
     BINDINGS = [
@@ -97,12 +131,13 @@ class ProviderConfigScreen(Screen[None]):
                 "Don't have an API Key? Use these links to get one: [OpenAI](https://platform.openai.com/api-keys) | [Anthropic](https://console.anthropic.com) | [Google Gemini](https://aistudio.google.com)",
                 id="provider-links",
             )
-        yield ListView(*self._build_provider_items(), id="provider-list")
+        yield ListView(*self._build_provider_items_sync(), id="provider-list")
         yield Input(
             placeholder=self._input_placeholder(self.selected_provider),
             password=True,
             id="api-key",
         )
+        yield Label("", id="provider-status")
         with Horizontal(id="provider-actions"):
             yield Button("Save key \\[ENTER]", variant="primary", id="save")
             yield Button("Authenticate", variant="success", id="authenticate")
@@ -110,8 +145,6 @@ class ProviderConfigScreen(Screen[None]):
             yield Button("Done \\[ESC]", id="done")
 
     def on_mount(self) -> None:
-        self.refresh_provider_status()
-        self._update_done_button_visibility()
         list_view = self.query_one(ListView)
         if list_view.children:
             list_view.index = 0
@@ -121,13 +154,35 @@ class ProviderConfigScreen(Screen[None]):
         self.query_one("#authenticate", Button).display = False
         self.set_focus(self.query_one("#api-key", Input))
 
+        # Refresh UI asynchronously
+        self.run_worker(self._refresh_ui(), exclusive=False)
+
+        # Apply layout based on terminal height
+        self._apply_layout_for_height(self.app.size.height)
+
+    @on(Resize)
+    def handle_resize(self, event: Resize) -> None:
+        """Adjust layout based on terminal height."""
+        self._apply_layout_for_height(event.size.height)
+
+    def _apply_layout_for_height(self, height: int) -> None:
+        """Apply appropriate layout based on terminal height."""
+        if height < COMPACT_HEIGHT_THRESHOLD:
+            self.add_class("compact")
+        else:
+            self.remove_class("compact")
+
     def on_screenresume(self) -> None:
         """Refresh provider status when screen is resumed.
 
         This ensures the UI reflects any provider changes made elsewhere.
         """
-        self.refresh_provider_status()
-        self._update_done_button_visibility()
+        self.run_worker(self._refresh_ui(), exclusive=False)
+
+    async def _refresh_ui(self) -> None:
+        """Refresh provider status and button visibility."""
+        await self.refresh_provider_status()
+        await self._update_done_button_visibility()
 
     def action_done(self) -> None:
         self.dismiss()
@@ -170,7 +225,11 @@ class ProviderConfigScreen(Screen[None]):
         if not self.is_mounted:
             return
 
-        # Show/hide UI elements based on provider type
+        # Show/hide UI elements based on provider type asynchronously
+        self.run_worker(self._update_provider_ui(provider), exclusive=False)
+
+    async def _update_provider_ui(self, provider: ProviderType) -> None:
+        """Update UI elements based on selected provider."""
         is_shotgun = provider == "shotgun"
 
         input_widget = self.query_one("#api-key", Input)
@@ -183,7 +242,7 @@ class ProviderConfigScreen(Screen[None]):
             save_button.display = False
 
             # Only show Authenticate button if shotgun is NOT already configured
-            if self._has_provider_key("shotgun"):
+            if await self._has_provider_key("shotgun"):
                 auth_button.display = False
             else:
                 auth_button.display = True
@@ -200,22 +259,29 @@ class ProviderConfigScreen(Screen[None]):
         app = cast("ShotgunApp", self.app)
         return app.config_manager
 
-    def refresh_provider_status(self) -> None:
+    async def refresh_provider_status(self) -> None:
         """Update the list view entries to reflect configured providers."""
         for provider_id in get_configurable_providers():
             label = self.query_one(f"#label-{provider_id}", Label)
-            label.update(self._provider_label(provider_id))
+            label.update(await self._provider_label(provider_id))
 
-    def _update_done_button_visibility(self) -> None:
+    async def _update_done_button_visibility(self) -> None:
         """Show/hide Done button based on whether any provider keys are configured."""
         done_button = self.query_one("#done", Button)
-        has_keys = self.config_manager.has_any_provider_key()
+        has_keys = await self.config_manager.has_any_provider_key()
         done_button.display = has_keys
 
-    def _build_provider_items(self) -> list[ListItem]:
+    def _build_provider_items_sync(self) -> list[ListItem]:
+        """Build provider items synchronously for compose().
+
+        Labels will be populated with status asynchronously in on_mount().
+        """
         items: list[ListItem] = []
         for provider_id in get_configurable_providers():
-            label = Label(self._provider_label(provider_id), id=f"label-{provider_id}")
+            # Create labels with placeholder text - will be updated in on_mount()
+            label = Label(
+                self._provider_display_name(provider_id), id=f"label-{provider_id}"
+            )
             items.append(ListItem(label, id=f"provider-{provider_id}"))
         return items
 
@@ -225,11 +291,10 @@ class ProviderConfigScreen(Screen[None]):
         provider_id = item.id.removeprefix("provider-")
         return provider_id if provider_id in get_configurable_providers() else None
 
-    def _provider_label(self, provider_id: str) -> str:
+    async def _provider_label(self, provider_id: str) -> str:
         display = self._provider_display_name(provider_id)
-        status = (
-            "Configured" if self._has_provider_key(provider_id) else "Not configured"
-        )
+        has_key = await self._has_provider_key(provider_id)
+        status = "Configured" if has_key else "Not configured"
         return f"{display} · {status}"
 
     def _provider_display_name(self, provider_id: str) -> str:
@@ -244,53 +309,67 @@ class ProviderConfigScreen(Screen[None]):
     def _input_placeholder(self, provider_id: str) -> str:
         return f"{self._provider_display_name(provider_id)} API key"
 
-    def _has_provider_key(self, provider_id: str) -> bool:
+    async def _has_provider_key(self, provider_id: str) -> bool:
         """Check if provider has a configured API key."""
         if provider_id == "shotgun":
             # Check shotgun key directly
-            config = self.config_manager.load()
+            config = await self.config_manager.load()
             return self.config_manager._provider_has_api_key(config.shotgun)
         else:
             # Check LLM provider key
             try:
                 provider = ProviderType(provider_id)
-                return self.config_manager.has_provider_key(provider)
+                return await self.config_manager.has_provider_key(provider)
             except ValueError:
                 return False
 
     def _save_api_key(self) -> None:
+        self.run_worker(self._do_save_api_key(), exclusive=True)
+
+    async def _do_save_api_key(self) -> None:
+        """Async implementation of API key saving."""
         input_widget = self.query_one("#api-key", Input)
         api_key = input_widget.value.strip()
+        status_label = self.query_one("#provider-status", Label)
 
         if not api_key:
-            self.notify("Enter an API key before saving.", severity="error")
+            status_label.update("❌ Enter an API key before saving.")
+            status_label.add_class("error")
             return
 
         try:
-            self.config_manager.update_provider(
+            await self.config_manager.update_provider(
                 self.selected_provider,
                 api_key=api_key,
             )
         except Exception as exc:  # pragma: no cover - defensive; textual path
-            self.notify(f"Failed to save key: {exc}", severity="error")
+            status_label.update(f"❌ Failed to save key: {exc}")
+            status_label.add_class("error")
             return
 
         input_widget.value = ""
-        self.refresh_provider_status()
-        self._update_done_button_visibility()
-        self.notify(
-            f"Saved API key for {self._provider_display_name(self.selected_provider)}."
+        await self.refresh_provider_status()
+        await self._update_done_button_visibility()
+        status_label.update(
+            f"✓ Saved API key for {self._provider_display_name(self.selected_provider)}."
         )
+        status_label.remove_class("error")
 
     def _clear_api_key(self) -> None:
+        self.run_worker(self._do_clear_api_key(), exclusive=True)
+
+    async def _do_clear_api_key(self) -> None:
+        """Async implementation of API key clearing."""
+        status_label = self.query_one("#provider-status", Label)
         try:
-            self.config_manager.clear_provider_key(self.selected_provider)
+            await self.config_manager.clear_provider_key(self.selected_provider)
         except Exception as exc:  # pragma: no cover - defensive; textual path
-            self.notify(f"Failed to clear key: {exc}", severity="error")
+            status_label.update(f"❌ Failed to clear key: {exc}")
+            status_label.add_class("error")
             return
 
-        self.refresh_provider_status()
-        self._update_done_button_visibility()
+        await self.refresh_provider_status()
+        await self._update_done_button_visibility()
         self.query_one("#api-key", Input).value = ""
 
         # If we just cleared shotgun, show the Authenticate button
@@ -298,9 +377,10 @@ class ProviderConfigScreen(Screen[None]):
             auth_button = self.query_one("#authenticate", Button)
             auth_button.display = True
 
-        self.notify(
-            f"Cleared API key for {self._provider_display_name(self.selected_provider)}."
+        status_label.update(
+            f"✓ Cleared API key for {self._provider_display_name(self.selected_provider)}."
         )
+        status_label.remove_class("error")
 
     async def _start_shotgun_auth(self) -> None:
         """Launch Shotgun Account authentication flow."""
@@ -311,5 +391,6 @@ class ProviderConfigScreen(Screen[None]):
 
         # Refresh provider status after auth completes
         if result:
-            self.refresh_provider_status()
-            # Notify handled by auth screen
+            await self.refresh_provider_status()
+            # Auto-dismiss provider config screen after successful auth
+            self.dismiss()

shotgun/tui/screens/shared_specs/__init__.py

@@ -0,0 +1,21 @@
+"""Shared specs TUI screens and dialogs."""
+
+from shotgun.tui.screens.shared_specs.create_spec_dialog import CreateSpecDialog
+from shotgun.tui.screens.shared_specs.models import (
+    CreateSpecResult,
+    ShareSpecsAction,
+    ShareSpecsResult,
+    UploadScreenResult,
+)
+from shotgun.tui.screens.shared_specs.share_specs_dialog import ShareSpecsDialog
+from shotgun.tui.screens.shared_specs.upload_progress_screen import UploadProgressScreen
+
+__all__ = [
+    "CreateSpecDialog",
+    "CreateSpecResult",
+    "ShareSpecsAction",
+    "ShareSpecsDialog",
+    "ShareSpecsResult",
+    "UploadProgressScreen",
+    "UploadScreenResult",
+]