gobby 0.2.5__py3-none-any.whl → 0.2.7__py3-none-any.whl

gobby/tools/summarizer.py

@@ -11,7 +11,8 @@ import logging
 from typing import TYPE_CHECKING, Any
 
 if TYPE_CHECKING:
-    from gobby.config.app import ToolSummarizerConfig
+    from gobby.config.features import ToolSummarizerConfig
+from gobby.prompts import PromptLoader
 
 logger = logging.getLogger(__name__)
 
@@ -20,12 +21,42 @@ MAX_DESCRIPTION_LENGTH = 200
 
 # Module-level config reference (set by init_summarizer_config)
 _config: ToolSummarizerConfig | None = None
+_loader: PromptLoader | None = None
 
+DEFAULT_SUMMARY_PROMPT = """Summarize this MCP tool description in 180 characters or less.
+Keep it to three sentences or less. Be concise and preserve the key functionality.
+Do not add quotes, extra formatting, or code examples.
 
-def init_summarizer_config(config: ToolSummarizerConfig) -> None:
+Description: {description}
+
+Summary:"""
+
+DEFAULT_SUMMARY_SYSTEM_PROMPT = "You are a technical summarizer. Create concise tool descriptions."
+
+DEFAULT_SERVER_DESC_PROMPT = """Write a single concise sentence describing what the '{server_name}' MCP server does based on its tools.
+
+Tools:
+{tools_list}
+
+Description (1 sentence, try to keep under 100 characters):"""
+
+DEFAULT_SERVER_DESC_SYSTEM_PROMPT = "You write concise technical descriptions."
+
+
+def init_summarizer_config(config: ToolSummarizerConfig, project_dir: str | None = None) -> None:
     """Initialize the summarizer with configuration."""
-    global _config
+    from pathlib import Path
+
+    global _config, _loader
     _config = config
+    _loader = PromptLoader(project_dir=Path(project_dir) if project_dir else None)
+    # Register fallbacks
+    _loader.register_fallback("features/tool_summary", lambda: DEFAULT_SUMMARY_PROMPT)
+    _loader.register_fallback("features/tool_summary_system", lambda: DEFAULT_SUMMARY_SYSTEM_PROMPT)
+    _loader.register_fallback("features/server_description", lambda: DEFAULT_SERVER_DESC_PROMPT)
+    _loader.register_fallback(
+        "features/server_description_system", lambda: DEFAULT_SERVER_DESC_SYSTEM_PROMPT
+    )
 
 
 def _get_config() -> ToolSummarizerConfig:
@@ -33,7 +64,12 @@ def _get_config() -> ToolSummarizerConfig:
     if _config is not None:
         return _config
     # Import here to avoid circular imports
-    from gobby.config.app import ToolSummarizerConfig
+    from gobby.config.features import ToolSummarizerConfig
+
+    # Ensure loader defaults exist even if init wasn't called
+    global _loader
+    if _loader is None:
+        _loader = PromptLoader()
 
     return ToolSummarizerConfig()
 
@@ -53,11 +89,30 @@ async def _summarize_description_with_claude(description: str) -> str:
     try:
         from claude_agent_sdk import AssistantMessage, ClaudeAgentOptions, TextBlock, query
 
-        prompt = config.prompt.format(description=description)
+        # Get summary prompt
+        prompt_path = config.prompt_path or "features/tool_summary"
+        try:
+            # We assume _loader is initialized by _get_config() logic or init
+            if _loader is None:
+                raise RuntimeError("Summarizer not initialized")
+            prompt = _loader.render(prompt_path, {"description": description})
+        except (FileNotFoundError, OSError, KeyError, ValueError, RuntimeError) as e:
+            logger.debug(f"Failed to load prompt from {prompt_path}: {e}, using default")
+            prompt = DEFAULT_SUMMARY_PROMPT.format(description=description)
+
+        # Get system prompt
+        sys_prompt_path = config.system_prompt_path or "features/tool_summary_system"
+        try:
+            if _loader is None:
+                raise RuntimeError("Summarizer not initialized")
+            system_prompt = _loader.render(sys_prompt_path, {})
+        except (FileNotFoundError, OSError, KeyError, ValueError, RuntimeError) as e:
+            logger.debug(f"Failed to load system prompt from {sys_prompt_path}: {e}, using default")
+            system_prompt = DEFAULT_SUMMARY_SYSTEM_PROMPT
 
         # Configure for single-turn completion
         options = ClaudeAgentOptions(
-            system_prompt=config.system_prompt,
+            system_prompt=system_prompt,
             max_turns=1,
             model=config.model,
             allowed_tools=[],
@@ -137,14 +192,40 @@ async def generate_server_description(
         # Build tools list for prompt
         tools_list = "\n".join([f"- {t['name']}: {t['description']}" for t in tool_summaries])
 
-        prompt = config.server_description_prompt.format(
-            server_name=server_name,
-            tools_list=tools_list,
+        # Build prompt
+        prompt_path = config.server_description_prompt_path or "features/server_description"
+        context = {
+            "server_name": server_name,
+            "tools_list": tools_list,
+        }
+        try:
+            if _loader is None:
+                _get_config()  # force init
+            if _loader is None:
+                # Still None after _get_config, use default
+                prompt = DEFAULT_SERVER_DESC_PROMPT.format(**context)
+            else:
+                prompt = _loader.render(prompt_path, context)
+        except (FileNotFoundError, OSError, KeyError, ValueError, RuntimeError) as e:
+            logger.debug(f"Failed to load prompt from {prompt_path}: {e}, using default")
+            prompt = DEFAULT_SERVER_DESC_PROMPT.format(**context)
+
+        # Get system prompt
+        sys_prompt_path = (
+            config.server_description_system_prompt_path or "features/server_description_system"
         )
+        try:
+            if _loader is None:
+                system_prompt = DEFAULT_SERVER_DESC_SYSTEM_PROMPT
+            else:
+                system_prompt = _loader.render(sys_prompt_path, {})
+        except (FileNotFoundError, OSError, KeyError, ValueError, RuntimeError) as e:
+            logger.debug(f"Failed to load system prompt from {sys_prompt_path}: {e}, using default")
+            system_prompt = DEFAULT_SERVER_DESC_SYSTEM_PROMPT
 
         # Configure for single-turn completion
         options = ClaudeAgentOptions(
-            system_prompt=config.server_description_system_prompt,
+            system_prompt=system_prompt,
             max_turns=1,
             model=config.model,
             allowed_tools=[],

gobby/tui/api_client.py

@@ -10,7 +10,7 @@ import httpx
 class GobbyAPIClient:
     """HTTP client for communicating with Gobby daemon."""
 
-    def __init__(self, base_url: str = "http://localhost:8765") -> None:
+    def __init__(self, base_url: str = "http://localhost:60887") -> None:
         self.base_url = base_url.rstrip("/")
         self._client: httpx.AsyncClient | None = None
 
@@ -177,17 +177,14 @@ class GobbyAPIClient:
         self,
         task_id: str,
         commit_sha: str | None = None,
-        no_commit_needed: bool = False,
-        override_justification: str | None = None,
+        reason: str | None = None,
     ) -> dict[str, Any]:
         """Close a task."""
         args: dict[str, Any] = {"task_id": task_id}
         if commit_sha:
             args["commit_sha"] = commit_sha
-        if no_commit_needed:
-            args["no_commit_needed"] = True
-        if override_justification:
-            args["override_justification"] = override_justification
+        if reason:
+            args["reason"] = reason
         return await self.call_tool("gobby-tasks", "close_task", args)
 
     async def suggest_next_task(self) -> dict[str, Any]:

gobby/tui/app.py

@@ -158,8 +158,8 @@ class GobbyApp(App[None]):
 
     def __init__(
         self,
-        daemon_url: str = "http://localhost:8765",
-        ws_url: str = "ws://localhost:8766",
+        daemon_url: str = "http://localhost:60887",
+        ws_url: str = "ws://localhost:60888",
     ) -> None:
         super().__init__()
         self.daemon_url = daemon_url
@@ -321,7 +321,9 @@ class GobbyApp(App[None]):
         await self.ws_client.disconnect()
 
 
-def run_tui(daemon_url: str = "http://localhost:8765", ws_url: str = "ws://localhost:8766") -> None:
+def run_tui(
+    daemon_url: str = "http://localhost:60887", ws_url: str = "ws://localhost:60888"
+) -> None:
     """Entry point for the TUI application."""
     app = GobbyApp(daemon_url=daemon_url, ws_url=ws_url)
     app.run()

gobby/tui/screens/orchestrator.py

@@ -826,8 +826,7 @@ class OrchestratorScreen(Widget):
                 return
             await self.api_client.close_task(
                 task_id,
-                no_commit_needed=True,
-                override_justification="Orchestrator approval - manual user review",
+                reason="completed",
             )
             self.notify(f"Approved: {task.get('ref', task_id)}")
 

gobby/tui/screens/tasks.py

@@ -403,16 +403,14 @@ class TasksScreen(Widget):
                     # Note: In real usage, this would need a commit SHA
                     await client.close_task(
                         task_id,
-                        no_commit_needed=True,
-                        override_justification="TUI completion - manual user action",
+                        reason="obsolete",
                     )
                     self.notify(f"Task completed: {task_id}")
 
                 elif button_id == "btn-approve":
                     await client.close_task(
                         task_id,
-                        no_commit_needed=True,
-                        override_justification="TUI approval - manual user review",
+                        reason="obsolete",
                     )
                     self.notify(f"Task approved: {task_id}")
 

gobby/tui/ws_client.py

@@ -19,7 +19,7 @@ class GobbyWebSocketClient:
 
     def __init__(
         self,
-        ws_url: str = "ws://localhost:8766",
+        ws_url: str = "ws://localhost:60888",
         reconnect_interval: float = 5.0,
         max_reconnect_attempts: int = 10,
     ) -> None:

gobby/utils/daemon_client.py

@@ -11,7 +11,7 @@ Example:
     ```python
     from gobby.utils.daemon_client import DaemonClient
 
-    client = DaemonClient(host="localhost", port=8765)
+    client = DaemonClient(host="localhost", port=60887)
 
     # Check daemon health
     is_healthy, error = client.check_health()
@@ -57,7 +57,7 @@ class DaemonClient:
     def __init__(
         self,
         host: str = "localhost",
-        port: int = 8765,
+        port: int = 60887,
         timeout: float = 5.0,
         logger: logging.Logger | None = None,
     ):

gobby/utils/project_context.py

@@ -10,8 +10,7 @@ from pathlib import Path
 from typing import TYPE_CHECKING, Any, cast
 
 if TYPE_CHECKING:
-    from gobby.config.app import ProjectVerificationConfig
-    from gobby.config.features import HooksConfig
+    from gobby.config.features import HooksConfig, ProjectVerificationConfig
 
 logger = logging.getLogger(__name__)
 
@@ -138,7 +137,7 @@ def get_verification_config(cwd: Path | None = None) -> ProjectVerificationConfi
     Returns:
         ProjectVerificationConfig if verification section exists, None otherwise.
     """
-    from gobby.config.app import ProjectVerificationConfig
+    from gobby.config.features import ProjectVerificationConfig
 
     context = get_project_context(cwd)
     if not context:

gobby/utils/status.py

@@ -79,6 +79,11 @@ def fetch_rich_status(http_port: int, timeout: float = 2.0) -> dict[str, Any]:
             status_kwargs["memories_count"] = memory.get("count", 0)
             status_kwargs["memories_avg_importance"] = memory.get("avg_importance", 0.0)
 
+        # Skills
+        skills_data = data.get("skills", {})
+        if skills_data:
+            status_kwargs["skills_total"] = skills_data.get("total", 0)
+
     except (httpx.ConnectError, httpx.TimeoutException):
         # Daemon not responding - return empty
         pass
@@ -117,6 +122,8 @@ def format_status_message(
     # Memory
     memories_count: int | None = None,
     memories_avg_importance: float | None = None,
+    # Skills
+    skills_total: int | None = None,
     **kwargs: Any,
 ) -> str:
     """
@@ -202,6 +209,12 @@ def format_status_message(
             lines.append(f"  Unhealthy: {unhealthy_str}")
         lines.append("")
 
+    # Skills section (only show if we have data)
+    if skills_total is not None:
+        lines.append("Skills:")
+        lines.append(f"  Loaded: {skills_total}")
+        lines.append("")
+
     # Sessions section (only show if we have data)
     if sessions_active is not None or sessions_paused is not None:
         lines.append("Sessions:")