shotgun-sh 0.1.11.dev1__py3-none-any.whl → 0.1.12.dev1__py3-none-any.whl

shotgun/tui/filtered_codebase_service.py

@@ -0,0 +1,46 @@
+"""Filtered codebase service that restricts access to current directory's codebase only."""
+
+from pathlib import Path
+
+from shotgun.codebase.models import CodebaseGraph
+from shotgun.codebase.service import CodebaseService
+
+
+class FilteredCodebaseService(CodebaseService):
+    """CodebaseService subclass that filters graphs to only those accessible from CWD.
+
+    This ensures TUI agents can only see and access the codebase indexed from the
+    current working directory, providing isolation between different project directories.
+    """
+
+    def __init__(self, storage_dir: Path | str):
+        """Initialize the filtered service.
+
+        Args:
+            storage_dir: Directory to store graph databases
+        """
+        super().__init__(storage_dir)
+        self._cwd = str(Path.cwd().resolve())
+
+    async def list_graphs(self) -> list[CodebaseGraph]:
+        """List only graphs accessible from the current working directory.
+
+        Returns:
+            Filtered list of CodebaseGraph objects accessible from CWD
+        """
+        # Use the existing filtering logic from list_graphs_for_directory
+        return await super().list_graphs_for_directory(self._cwd)
+
+    async def list_graphs_for_directory(
+        self, directory: Path | str | None = None
+    ) -> list[CodebaseGraph]:
+        """List graphs for directory - always filters to CWD for TUI context.
+
+        Args:
+            directory: Ignored in TUI context, always uses CWD
+
+        Returns:
+            Filtered list of CodebaseGraph objects accessible from CWD
+        """
+        # Always use CWD regardless of what directory is passed
+        return await super().list_graphs_for_directory(self._cwd)

shotgun/tui/screens/chat.py

@@ -1,6 +1,5 @@
 import asyncio
 import logging
-from collections.abc import Iterable
 from dataclasses import dataclass
 from pathlib import Path
 from typing import cast
@@ -20,7 +19,7 @@ from textual.containers import Container, Grid
 from textual.reactive import reactive
 from textual.screen import ModalScreen, Screen
 from textual.widget import Widget
-from textual.widgets import Button, DirectoryTree, Input, Label, Markdown, Static
+from textual.widgets import Button, Label, Markdown, Static
 
 from shotgun.agents.agent_manager import (
     AgentManager,
@@ -44,10 +43,11 @@ from shotgun.codebase.core.manager import CodebaseAlreadyIndexedError
 from shotgun.posthog_telemetry import track_event
 from shotgun.sdk.codebase import CodebaseSDK
 from shotgun.sdk.exceptions import CodebaseNotFoundError, InvalidPathError
-from shotgun.sdk.services import get_codebase_service
 from shotgun.tui.commands import CommandHandler
+from shotgun.tui.filtered_codebase_service import FilteredCodebaseService
 from shotgun.tui.screens.chat_screen.hint_message import HintMessage
 from shotgun.tui.screens.chat_screen.history import ChatHistory
+from shotgun.utils import get_shotgun_home
 
 from ..components.prompt_input import PromptInput
 from ..components.spinner import Spinner
@@ -167,11 +167,6 @@ class ModeIndicator(Widget):
         return f"[bold $text-accent]{mode_title}{status_icon} mode[/][$foreground-muted] ({description})[/]"
 
 
-class FilteredDirectoryTree(DirectoryTree):
-    def filter_paths(self, paths: Iterable[Path]) -> Iterable[Path]:
-        return [path for path in paths if path.is_dir()]
-
-
 class CodebaseIndexPromptScreen(ModalScreen[bool]):
     """Modal dialog asking whether to index the detected codebase."""
 
@@ -226,105 +221,6 @@ class CodebaseIndexPromptScreen(ModalScreen[bool]):
         self.dismiss(True)
 
 
-class CodebaseIndexScreen(ModalScreen[CodebaseIndexSelection | None]):
-    """Modal dialog for choosing a repository and name to index."""
-
-    DEFAULT_CSS = """
-        CodebaseIndexScreen {
-            align: center middle;
-            background: rgba(0, 0, 0, 0.0);
-        }
-        CodebaseIndexScreen > #index-dialog {
-            width: 80%;
-            max-width: 80;
-            height: 80%;
-            max-height: 40;
-            border: wide $primary;
-            padding: 1;
-            layout: vertical;
-            background: $surface;
-        }
-
-        #index-dialog DirectoryTree {
-            height: 1fr;
-            border: solid $accent;
-            overflow: auto;
-        }
-
-        #index-dialog-controls {
-            layout: horizontal;
-            align-horizontal: right;
-            padding-top: 1;
-        }
-    """
-
-    def __init__(self, start_path: Path | None = None) -> None:
-        super().__init__()
-        self.start_path = Path(start_path or Path.cwd())
-        self.selected_path: Path | None = self.start_path
-
-    def compose(self) -> ComposeResult:
-        with Container(id="index-dialog"):
-            yield Label("Index a codebase", id="index-dialog-title")
-            yield FilteredDirectoryTree(self.start_path, id="index-directory-tree")
-            yield Input(
-                placeholder="Enter a name for the codebase",
-                id="index-codebase-name",
-            )
-            with Container(id="index-dialog-controls"):
-                yield Button("Cancel", id="index-cancel")
-                yield Button(
-                    "Index",
-                    id="index-confirm",
-                    variant="primary",
-                    disabled=True,
-                )
-
-    def on_mount(self) -> None:
-        name_input = self.query_one("#index-codebase-name", Input)
-        if not name_input.value and self.selected_path:
-            name_input.value = self.selected_path.name
-        self._update_confirm()
-
-    def _update_confirm(self) -> None:
-        confirm = self.query_one("#index-confirm", Button)
-        name_input = self.query_one("#index-codebase-name", Input)
-        confirm.disabled = not (self.selected_path and name_input.value.strip())
-
-    @on(DirectoryTree.DirectorySelected, "#index-directory-tree")
-    def handle_directory_selected(self, event: DirectoryTree.DirectorySelected) -> None:
-        event.stop()
-        selected = event.path if event.path.is_dir() else event.path.parent
-        self.selected_path = selected
-        name_input = self.query_one("#index-codebase-name", Input)
-        if not name_input.value:
-            name_input.value = selected.name
-        self._update_confirm()
-
-    @on(Input.Changed, "#index-codebase-name")
-    def handle_name_changed(self, event: Input.Changed) -> None:
-        event.stop()
-        self._update_confirm()
-
-    @on(Button.Pressed, "#index-cancel")
-    def handle_cancel(self, event: Button.Pressed) -> None:
-        event.stop()
-        self.dismiss(None)
-
-    @on(Button.Pressed, "#index-confirm")
-    def handle_confirm(self, event: Button.Pressed) -> None:
-        event.stop()
-        name_input = self.query_one("#index-codebase-name", Input)
-        if not self.selected_path:
-            self.dismiss(None)
-            return
-        selection = CodebaseIndexSelection(
-            repo_path=self.selected_path,
-            name=name_input.value.strip(),
-        )
-        self.dismiss(selection)
-
-
 class ChatScreen(Screen[None]):
     CSS_PATH = "chat.tcss"
 
@@ -349,7 +245,9 @@ class ChatScreen(Screen[None]):
         super().__init__()
         # Get the model configuration and services
         model_config = get_provider_model()
-        codebase_service = get_codebase_service()
+        # Use filtered service in TUI to restrict access to CWD codebase only
+        storage_dir = get_shotgun_home() / "codebases"
+        codebase_service = FilteredCodebaseService(storage_dir)
         self.codebase_sdk = CodebaseSDK()
 
         # Create shared deps without system_prompt_fn (agents provide their own)
@@ -423,6 +321,7 @@ class ChatScreen(Screen[None]):
             self.mount_hint(help_text_with_codebase(already_indexed=True))
             return
 
+        # Ask user if they want to index the current directory
         should_index = await self.app.push_screen_wait(CodebaseIndexPromptScreen())
         if not should_index:
             self.mount_hint(help_text_empty_dir())
@@ -430,7 +329,10 @@ class ChatScreen(Screen[None]):
 
         self.mount_hint(help_text_with_codebase(already_indexed=False))
 
-        self.index_codebase_command()
+        # Auto-index the current directory with its name
+        cwd_name = cur_dir.name
+        selection = CodebaseIndexSelection(repo_path=cur_dir, name=cwd_name)
+        self.call_later(lambda: self.index_codebase(selection))
 
     def watch_mode(self, new_mode: AgentType) -> None:
         """React to mode changes by updating the agent manager."""
@@ -529,12 +431,16 @@ class ChatScreen(Screen[None]):
     @on(PartialResponseMessage)
     def handle_partial_response(self, event: PartialResponseMessage) -> None:
         self.partial_message = event.message
-
         history = self.query_one(ChatHistory)
-        history.update_messages(
-            self.messages + cast(list[ModelMessage | HintMessage], event.messages)
+
+        # Only update messages if the message list changed
+        new_message_list = self.messages + cast(
+            list[ModelMessage | HintMessage], event.messages
         )
+        if len(new_message_list) != len(history.items):
+            history.update_messages(new_message_list)
 
+        # Always update the partial response (reactive property handles the update)
         history.partial_response = self.partial_message
 
     def _clear_partial_response(self) -> None:
@@ -640,16 +546,11 @@ class ChatScreen(Screen[None]):
         return self.placeholder_hints.get_placeholder_for_mode(mode)
 
     def index_codebase_command(self) -> None:
-        start_path = Path.cwd()
-
-        def handle_result(result: CodebaseIndexSelection | None) -> None:
-            if result:
-                self.call_later(lambda: self.index_codebase(result))
-
-        self.app.push_screen(
-            CodebaseIndexScreen(start_path=start_path),
-            handle_result,
-        )
+        # Simplified: always index current working directory with its name
+        cur_dir = Path.cwd().resolve()
+        cwd_name = cur_dir.name
+        selection = CodebaseIndexSelection(repo_path=cur_dir, name=cwd_name)
+        self.call_later(lambda: self.index_codebase(selection))
 
     def delete_codebase_command(self) -> None:
         self.app.push_screen(

shotgun/tui/screens/chat_screen/history.py

@@ -41,7 +41,6 @@ class PartialResponseWidget(Widget):  # TODO: doesn't work lol
         self.item = item
 
     def compose(self) -> ComposeResult:
-        yield Markdown(markdown="**partial response**")
         if self.item is None:
             pass
         elif self.item.kind == "response":
@@ -82,12 +81,14 @@ class ChatHistory(Widget):
         self.items: Sequence[ModelMessage | HintMessage] = []
         self.vertical_tail: VerticalTail | None = None
         self.partial_response = None
+        self._rendered_count = 0  # Track how many messages have been mounted
 
     def compose(self) -> ComposeResult:
         self.vertical_tail = VerticalTail()
 
+        filtered = list(self.filtered_items())
         with self.vertical_tail:
-            for item in self.filtered_items():
+            for item in filtered:
                 if isinstance(item, ModelRequest):
                     yield UserQuestionWidget(item)
                 elif isinstance(item, HintMessage):
@@ -97,6 +98,9 @@ class ChatHistory(Widget):
             yield PartialResponseWidget(self.partial_response).data_bind(
                 item=ChatHistory.partial_response
             )
+
+        # Track how many messages were rendered during initial compose
+        self._rendered_count = len(filtered)
         self.call_later(self.autoscroll)
 
     def filtered_items(self) -> Generator[ModelMessage | HintMessage, None, None]:
@@ -138,17 +142,43 @@ class ChatHistory(Widget):
             yield next_item
 
     def update_messages(self, messages: list[ModelMessage | HintMessage]) -> None:
-        """Update the displayed messages without recomposing."""
+        """Update the displayed messages using incremental mounting."""
         if not self.vertical_tail:
             return
 
         self.items = messages
-        self.refresh(recompose=True)
-        self.call_later(self.autoscroll)
+        filtered = list(self.filtered_items())
+
+        # Only mount new messages that haven't been rendered yet
+        if len(filtered) > self._rendered_count:
+            new_messages = filtered[self._rendered_count :]
+            for item in new_messages:
+                widget: Widget
+                if isinstance(item, ModelRequest):
+                    widget = UserQuestionWidget(item)
+                elif isinstance(item, HintMessage):
+                    widget = HintMessageWidget(item)
+                elif isinstance(item, ModelResponse):
+                    widget = AgentResponseWidget(item)
+                else:
+                    continue
+
+                # Mount before the PartialResponseWidget
+                self.vertical_tail.mount(widget, before=self.vertical_tail.children[-1])
+
+            self._rendered_count = len(filtered)
+            self.call_after_refresh(self.autoscroll)
 
     def autoscroll(self) -> None:
+        """Smoothly scroll to the bottom after new content is added."""
         if self.vertical_tail:
-            self.vertical_tail.scroll_end(animate=False, immediate=False, force=True)
+            # Scroll to absolute bottom - more reliable during recomposition
+            self.vertical_tail.scroll_end(animate=False, force=True)
+
+    def watch_partial_response(self, partial_response: ModelMessage | None) -> None:
+        """Trigger autoscroll when partial response updates during streaming."""
+        if partial_response is not None:
+            self.call_after_refresh(self.autoscroll)
 
 
 class UserQuestionWidget(Widget):

{shotgun_sh-0.1.11.dev1.dist-info → shotgun_sh-0.1.12.dev1.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: shotgun-sh
-Version: 0.1.11.dev1
+Version: 0.1.12.dev1
 Summary: AI-powered research, planning, and task management CLI tool
 Project-URL: Homepage, https://shotgun.sh/
 Project-URL: Repository, https://github.com/shotgun-sh/shotgun

{shotgun_sh-0.1.11.dev1.dist-info → shotgun_sh-0.1.12.dev1.dist-info}/RECORD

@@ -103,13 +103,14 @@ shotgun/sdk/models.py,sha256=X9nOTUHH0cdkQW1NfnMEDu-QgK9oUsEISh1Jtwr5Am4,5496
 shotgun/sdk/services.py,sha256=J4PJFSxCQ6--u7rb3Ta-9eYtlYcxcbnzrMP6ThyCnw4,705
 shotgun/tui/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 shotgun/tui/app.py,sha256=rNi1a2vIhu385-DylE1rYgVj3_cok65X_z1lEamrFqo,3445
+shotgun/tui/filtered_codebase_service.py,sha256=lJ8gTMhIveTatmvmGLP299msWWTkVYKwvY_2FhuL2s4,1687
 shotgun/tui/styles.tcss,sha256=ETyyw1bpMBOqTi5RLcAJUScdPWTvAWEqE9YcT0kVs_E,121
 shotgun/tui/commands/__init__.py,sha256=8D5lvtpqMW5-fF7Bg3oJtUzU75cKOv6aUaHYYszydU8,2518
 shotgun/tui/components/prompt_input.py,sha256=Ss-htqraHZAPaehGE4x86ij0veMjc4UgadMXpbdXr40,2229
 shotgun/tui/components/spinner.py,sha256=ovTDeaJ6FD6chZx_Aepia6R3UkPOVJ77EKHfRmn39MY,2427
 shotgun/tui/components/splash.py,sha256=vppy9vEIEvywuUKRXn2y11HwXSRkQZHLYoVjhDVdJeU,1267
 shotgun/tui/components/vertical_tail.py,sha256=GavHXNMq1X8hc0juDLKDWTW9seRLk3VlhBBMl60uPG0,439
-shotgun/tui/screens/chat.py,sha256=nlu3yHS6HAnloqA_v4xnFeM9BleUyIWY2fHV1P_7yBc,30083
+shotgun/tui/screens/chat.py,sha256=0fIERVokK06L-kIko_q57gx85j9M95hNpx7OPVZUGjc,27067
 shotgun/tui/screens/chat.tcss,sha256=2Yq3E23jxsySYsgZf4G1AYrYVcpX0UDW6kNNI0tDmtM,437
 shotgun/tui/screens/directory_setup.py,sha256=lIZ1J4A6g5Q2ZBX8epW7BhR96Dmdcg22CyiM5S-I5WU,3237
 shotgun/tui/screens/provider_config.py,sha256=A_tvDHF5KLP5PV60LjMJ_aoOdT3TjI6_g04UIUqGPqM,7126
@@ -117,7 +118,7 @@ shotgun/tui/screens/splash.py,sha256=E2MsJihi3c9NY1L28o_MstDxGwrCnnV7zdq00MrGAsw
 shotgun/tui/screens/chat_screen/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 shotgun/tui/screens/chat_screen/command_providers.py,sha256=55JIH9T8QnyHRsMoXhOi87FiVM-d6o7OKpCe82uDP9I,7840
 shotgun/tui/screens/chat_screen/hint_message.py,sha256=WOpbk8q7qt7eOHTyyHvh_IQIaublVDeJGaLpsxEk9FA,933
-shotgun/tui/screens/chat_screen/history.py,sha256=pmgtvPDcY4dyxLyMH0RrVmf8qBouW1hM99VSUn7XGPQ,8950
+shotgun/tui/screens/chat_screen/history.py,sha256=qITuZH1dqeVlbtExh6EkLqcwQfjFt6Kn4VYqRHwS3iM,10356
 shotgun/tui/utils/__init__.py,sha256=cFjDfoXTRBq29wgP7TGRWUu1eFfiIG-LLOzjIGfadgI,150
 shotgun/tui/utils/mode_progress.py,sha256=lseRRo7kMWLkBzI3cU5vqJmS2ZcCjyRYf9Zwtvc-v58,10931
 shotgun/utils/__init__.py,sha256=WinIEp9oL2iMrWaDkXz2QX4nYVPAm8C9aBSKTeEwLtE,198
@@ -125,8 +126,8 @@ shotgun/utils/env_utils.py,sha256=8QK5aw_f_V2AVTleQQlcL0RnD4sPJWXlDG46fsHu0d8,10
 shotgun/utils/file_system_utils.py,sha256=l-0p1bEHF34OU19MahnRFdClHufThfGAjQ431teAIp0,1004
 shotgun/utils/source_detection.py,sha256=Co6Q03R3fT771TF3RzB-70stfjNP2S4F_ArZKibwzm8,454
 shotgun/utils/update_checker.py,sha256=IgzPHRhS1ETH7PnJR_dIx6lxgr1qHpCkMTgzUxvGjhI,7586
-shotgun_sh-0.1.11.dev1.dist-info/METADATA,sha256=sxMCruZxOqBiw8AFfFi4VkhrPJvFYKi2Y0mamnMLFpk,11197
-shotgun_sh-0.1.11.dev1.dist-info/WHEEL,sha256=qtCwoSJWgHk21S1Kb4ihdzI2rlJ1ZKaIurTj_ngOhyQ,87
-shotgun_sh-0.1.11.dev1.dist-info/entry_points.txt,sha256=asZxLU4QILneq0MWW10saVCZc4VWhZfb0wFZvERnzfA,45
-shotgun_sh-0.1.11.dev1.dist-info/licenses/LICENSE,sha256=YebsZl590zCHrF_acCU5pmNt0pnAfD2DmAnevJPB1tY,1065
-shotgun_sh-0.1.11.dev1.dist-info/RECORD,,
+shotgun_sh-0.1.12.dev1.dist-info/METADATA,sha256=HArvIlxxxiQ9eSFZIhSyW_87DTijn5s4qn1RFMuONUc,11197
+shotgun_sh-0.1.12.dev1.dist-info/WHEEL,sha256=qtCwoSJWgHk21S1Kb4ihdzI2rlJ1ZKaIurTj_ngOhyQ,87
+shotgun_sh-0.1.12.dev1.dist-info/entry_points.txt,sha256=asZxLU4QILneq0MWW10saVCZc4VWhZfb0wFZvERnzfA,45
+shotgun_sh-0.1.12.dev1.dist-info/licenses/LICENSE,sha256=YebsZl590zCHrF_acCU5pmNt0pnAfD2DmAnevJPB1tY,1065
+shotgun_sh-0.1.12.dev1.dist-info/RECORD,,