@pennyfarthing/core 11.2.1 → 11.2.2

package/pennyfarthing_scripts/bikerack/audit_log_panel.py

@@ -8,7 +8,7 @@ Uses native Textual widgets (DataTable) instead of Rich renderables.
 
 from __future__ import annotations
 
-from datetime import datetime, timezone
+from datetime import UTC, datetime
 from typing import Any
 
 from rich.console import Console
@@ -19,6 +19,18 @@ from pennyfarthing_scripts.bikerack.base_panel import PANEL_ICONS, BasePanel
 
 MAX_INPUT_LENGTH = 80
 
+# Tool name → Rich style (matches GUI CSS custom properties)
+_TOOL_COLORS: dict[str, str] = {
+    "read": "blue",
+    "write": "dark_orange",
+    "bash": "green",
+    "glob": "medium_purple1",
+    "grep": "dark_cyan",
+    "edit": "yellow",
+    "task": "hot_pink",
+    "skill": "dim",
+}
+
 # Shared fallback console for offline column/cell measurement
 _FALLBACK_CONSOLE = Console()
 
@@ -58,8 +70,9 @@ class AuditLogPanel(BasePanel):
     def handle_message(self, message: dict[str, Any] | None) -> None:
         """Handle incoming WebSocket messages for tool events.
 
-        Overrides BasePanel.handle_message to manipulate DataTable directly
-        instead of going through render_panel/post_message.
+        Mutates the internal DataTable then delegates to
+        super().handle_message() which calls render_panel() →
+        post_message(DataReceived) for thread-safe Textual repaint.
         """
         if not self._mounted or message is None:
             return
@@ -81,6 +94,11 @@ class AuditLogPanel(BasePanel):
             span = message.get("span")
             if isinstance(span, dict):
                 self._add_row(span)
+        else:
+            return
+
+        # Trigger repaint through BasePanel's message system
+        super().handle_message(message)
 
     def _add_row(self, span: dict[str, Any]) -> None:
         """Add a single tool event row to the DataTable."""
@@ -104,8 +122,32 @@ class AuditLogPanel(BasePanel):
         self._table.add_row(time_str, tool_name, input_text, result)
 
     def render_panel(self, payload: dict[str, Any]) -> Any:
-        """Not used — handle_message manages DataTable directly."""
-        return self._table
+        """Render current DataTable rows as a Rich Table renderable."""
+        from rich.table import Table as RichTable
+        from rich.text import Text
+
+        if self._table.row_count == 0:
+            return Text("No audit events yet", style="dim italic")
+
+        rich_table = RichTable(show_header=True, expand=True, box=None)
+        rich_table.add_column("Time", style="dim", no_wrap=True)
+        rich_table.add_column("Tool", no_wrap=True)
+        rich_table.add_column("Input")
+        rich_table.add_column("Result", justify="center", no_wrap=True)
+
+        for row_key in reversed(list(self._table.rows)):
+            row_data = self._table.get_row(row_key)
+            time_str, tool_name, input_text, result = (str(c) for c in row_data)
+            tool_style = _TOOL_COLORS.get(tool_name.lower(), "bold cyan")
+            result_style = "green" if result == "\u2713" else "red" if result == "\u2717" else "dim"
+            rich_table.add_row(
+                time_str,
+                Text(tool_name, style=f"bold {tool_style}"),
+                Text(input_text, style="dim") if input_text else Text(""),
+                Text(result, style=result_style),
+            )
+
+        return rich_table
 
 
 def _format_timestamp(ts: Any) -> str:
@@ -113,7 +155,7 @@ def _format_timestamp(ts: Any) -> str:
     if ts is None:
         return "\u2014"
     try:
-        dt = datetime.fromtimestamp(float(ts) / 1000, tz=timezone.utc)
+        dt = datetime.fromtimestamp(float(ts) / 1000, tz=UTC)
         return dt.strftime("%H:%M:%S")
     except (ValueError, TypeError, OSError):
         return "\u2014"

package/pennyfarthing_scripts/bikerack/launcher.py

@@ -24,8 +24,8 @@ def is_process_alive(pid: int) -> bool:
 
 
 def cleanup_files(project_dir: Path) -> None:
-    """Clean up .wheelhub-port, .wheelhub-pid, and .wheelhub-gui-pid files."""
-    for name in (".wheelhub-port", ".wheelhub-pid", ".wheelhub-gui-pid"):
+    """Clean up .bikerack-port, .wheelhub-pid, and .wheelhub-gui-pid files."""
+    for name in (".bikerack-port", ".wheelhub-pid", ".wheelhub-gui-pid"):
         try:
             (project_dir / name).unlink()
         except FileNotFoundError:
@@ -33,9 +33,9 @@ def cleanup_files(project_dir: Path) -> None:
 
 
 def read_port_file(project_dir: Path) -> int | None:
-    """Read port from .wheelhub-port file. Returns None if not found."""
+    """Read port from .bikerack-port file. Returns None if not found."""
     try:
-        return int((project_dir / ".wheelhub-port").read_text().strip())
+        return int((project_dir / ".bikerack-port").read_text().strip())
     except (FileNotFoundError, ValueError):
         return None
 
@@ -104,8 +104,8 @@ def start_wheelhub(project_dir: Path) -> subprocess.Popen:
 def poll_for_port_file(
     project_dir: Path, timeout: float = 5.0, interval: float = 0.1
 ) -> int:
-    """Poll for .wheelhub-port file, return port number."""
-    port_file = project_dir / ".wheelhub-port"
+    """Poll for .bikerack-port file, return port number."""
+    port_file = project_dir / ".bikerack-port"
     deadline = time.monotonic() + timeout
 
     while True:

package/pennyfarthing_scripts/bikerack/progress_panel.py

@@ -265,7 +265,6 @@ class ProgressPanel(BasePanel):
         line = Text()
         line.append("Git  ", style="bold")
 
-        repo_parts: list[str] = []
         for repo in repos:
             if not isinstance(repo, dict):
                 continue

package/pennyfarthing_scripts/bikerack/sprint_panel.py

@@ -228,7 +228,7 @@ class SprintPanel(Widget):
         """Mark unmounted so WS callbacks are ignored."""
         self._mounted = False
 
-    def focus(self, scroll_visible: bool = True) -> "SprintPanel":
+    def focus(self, scroll_visible: bool = True) -> SprintPanel:
         """Delegate focus to the tree widget."""
         try:
             tree = self.query_one("#sprint-tree", Tree)

package/pennyfarthing_scripts/bikerack/story_detail_data.py

@@ -107,9 +107,12 @@ def _parse_session_file(session_path: str) -> dict[str, Any]:
 
 def _get_sprint_story_status(story_id: str) -> str | None:
     """Try to get story status from pf sprint CLI."""
+    import sys
+
     try:
         out = subprocess.run(
-            ["pf", "sprint", "story", "field", story_id, "status"],
+            [sys.executable, "-m", "pennyfarthing_scripts.cli",
+             "sprint", "story", "field", story_id, "status"],
             capture_output=True,
             text=True,
             timeout=10,

package/pennyfarthing_scripts/bikerack/story_detail_screen.py

@@ -88,7 +88,8 @@ class StoryDetailScreen(Screen):
             name = _format_assignee(assignee)
             if name:
                 meta_parts.append(name)
-        header_text.append(f"\n{' \u00b7 '.join(meta_parts)}", style="dim")
+        separator = " \u00b7 "
+        header_text.append(f"\n{separator.join(meta_parts)}", style="dim")
         yield Static(header_text, id="dossier-header")
 
         # Context section (only if context files exist)

package/pennyfarthing_scripts/bikerack/tui.py

@@ -10,6 +10,7 @@ Panel navigation: Mount all panels, tab bar, keyboard switching, command palette
 
 from __future__ import annotations
 
+import os
 from functools import partial
 from pathlib import Path
 from typing import Any
@@ -392,6 +393,11 @@ class BikeRackApp(App):
         height: auto;
         width: 1fr;
     }
+    #project-dir {
+        height: 1;
+        padding: 0 1;
+        color: $text-muted;
+    }
     Tabs {
         dock: top;
     }
@@ -444,8 +450,12 @@ class BikeRackApp(App):
         self._programmatic_tab_count: int = 0
 
     def compose(self) -> ComposeResult:
+        project_dir_name = Path(
+            os.environ.get("CYCLIST_PROJECT_DIR", os.getcwd())
+        ).name
         yield Header()
         yield AgentHeader(id="agent-header")
+        yield Static(f"[dim]{project_dir_name}[/dim]", id="project-dir")
         yield Tabs(*_build_panel_tabs(), id="tab-bar")
         yield ConnectionStatus(
             STATE_DISPLAY[ConnectionState.DISCONNECTED],
@@ -693,7 +703,7 @@ def main(
     """Launch BikeRack TUI as a standalone application.
 
     Args:
-        port: Explicit WheelHub port. If None, reads from .wheelhub-port file.
+        port: Explicit WheelHub port. If None, reads from .bikerack-port file.
         project_dir: Project directory for port file discovery. Defaults to cwd.
     """
     # Detect terminal image protocol BEFORE App.run() claims the terminal
@@ -703,7 +713,7 @@ def main(
 
     if port is None:
         if project_dir is not None:
-            port_file = project_dir / ".wheelhub-port"
+            port_file = project_dir / ".bikerack-port"
             if port_file.exists():
                 try:
                     port = int(port_file.read_text().strip())

package/pennyfarthing_scripts/bikerack/ws_client.py

@@ -75,14 +75,14 @@ class WheelHubClient:
             cb(new_state)
 
     def discover_port(self) -> int:
-        """Read port from .wheelhub-port file, fallback to DEFAULT_PORT.
+        """Read port from .bikerack-port file, fallback to DEFAULT_PORT.
 
         Priority: explicit port > port file > DEFAULT_PORT.
         """
         if self._port is not None:
             return self._port
         if self._project_dir is not None:
-            port_file = self._project_dir / ".wheelhub-port"
+            port_file = self._project_dir / ".bikerack-port"
             if port_file.exists():
                 try:
                     return int(port_file.read_text().strip())

package/pennyfarthing_scripts/common/config.py

@@ -82,11 +82,38 @@ def load_yaml_config(path: Path) -> dict[str, Any] | None:
         return yaml.safe_load(f)
 
 
-def load_pennyfarthing_config() -> dict[str, Any]:
+def load_pennyfarthing_config(project_root: Path | None = None) -> dict[str, Any]:
     """Load .pennyfarthing/config.local.yaml.
 
+    Args:
+        project_root: Project root path (defaults to auto-detect)
+
     Returns:
         Config dict, or empty dict if not found
     """
-    config_path = get_project_root() / ".pennyfarthing" / "config.local.yaml"
+    root = project_root or get_project_root()
+    config_path = root / ".pennyfarthing" / "config.local.yaml"
     return load_yaml_config(config_path) or {}
+
+
+def save_pennyfarthing_config_key(
+    key: str, value: Any, project_root: Path | None = None
+) -> None:
+    """Set a top-level key in .pennyfarthing/config.local.yaml.
+
+    Creates the file if it doesn't exist. Preserves existing keys.
+
+    Args:
+        key: Top-level key (e.g., "sprint")
+        value: Value to set (dict, str, etc.)
+        project_root: Project root path (defaults to auto-detect)
+    """
+    root = project_root or get_project_root()
+    config_path = root / ".pennyfarthing" / "config.local.yaml"
+
+    config = load_yaml_config(config_path) or {}
+    config[key] = value
+
+    config_path.parent.mkdir(parents=True, exist_ok=True)
+    with open(config_path, "w") as f:
+        yaml.dump(config, f, default_flow_style=False, sort_keys=False)

package/pennyfarthing_scripts/consultation/cli.py

@@ -6,7 +6,7 @@ the bash wrapper dialogue-manager.sh interface.
 
 from __future__ import annotations
 
-from datetime import datetime, timezone
+from datetime import UTC, datetime
 from pathlib import Path
 
 import click
@@ -44,7 +44,7 @@ def init(story_id: str, workflow: str, leader: str, partner: str) -> None:
         workflow=workflow,
         leader=leader,
         partner=partner,
-        started_at=datetime.now(timezone.utc).strftime("%Y-%m-%dT%H:%M:%SZ"),
+        started_at=datetime.now(UTC).strftime("%Y-%m-%dT%H:%M:%SZ"),
     )
 
     content = create_dialogue_content(header)
@@ -80,7 +80,7 @@ def append(story_id: str, question: str, recommendation: str, confidence: str) -
 
     exchange = DialogueExchange(
         number=next_num,
-        timestamp=datetime.now(timezone.utc).strftime("%H:%M"),
+        timestamp=datetime.now(UTC).strftime("%H:%M"),
         leader="",
         partner="",
         question=question,

package/pennyfarthing_scripts/context.py

@@ -269,7 +269,7 @@ def detect_cyclist(project_dir: str | None = None) -> bool:
 
     Checks:
     1. CYCLIST env var set to '1' (Electron mode - definitive)
-    2. .wheelhub-port file exists AND port is responding (Web mode)
+    2. .bikerack-port file exists AND port is responding (Web mode)
     """
     # Env var is definitive - set by Cyclist when it spawns Claude
     if os.environ.get("CYCLIST") == "1":
@@ -284,8 +284,8 @@ def detect_cyclist(project_dir: str | None = None) -> bool:
     )
 
     port_files = [
-        Path(project_dir) / "packages" / "cyclist" / ".wheelhub-port",
-        Path(os.getcwd()) / ".wheelhub-port",
+        Path(project_dir) / "packages" / "cyclist" / ".bikerack-port",
+        Path(os.getcwd()) / ".bikerack-port",
     ]
 
     for port_file in port_files:

package/pennyfarthing_scripts/git/hooks_installer.py

@@ -10,7 +10,6 @@ Usage via CLI:
 
 from __future__ import annotations
 
-import re
 from pathlib import Path
 
 from pennyfarthing_scripts.common.config import get_project_root
@@ -70,8 +69,8 @@ def install_git_hooks(project_root: Path | None = None) -> int:
         return 1
 
     print("Installing git hooks with .d/ dispatcher pattern...")
-    print(f"  Source: pennyfarthing-dist/scripts/hooks/")
-    print(f"  Dest:   .git/hooks/")
+    print("  Source: pennyfarthing-dist/scripts/hooks/")
+    print("  Dest:   .git/hooks/")
     print()
 
     for source_file, dest_name in HOOKS:

package/pennyfarthing_scripts/git/status_all.py

@@ -291,7 +291,7 @@ async def main(brief: bool = False) -> int:
     Returns:
         0 if all repos clean, 1 if any have changes/unpushed
     """
-    from pennyfarthing_scripts.git.repos import load_repos_config, get_repo_paths
+    from pennyfarthing_scripts.git.repos import get_repo_paths, load_repos_config
 
     repos_with_upstream: list[tuple[str, Path, str]] = []
     repo_paths = get_repo_paths()

package/pennyfarthing_scripts/git/worktree.py

@@ -199,7 +199,7 @@ def remove_worktree(name: str) -> int:
         shutil.rmtree(wt_path)
 
     # Prune worktree references
-    for repo_name, cfg in repos.items():
+    for _repo_name, cfg in repos.items():
         full_path = (project_root / cfg.path).resolve()
         if full_path.exists():
             _git(["worktree", "prune"], full_path)
@@ -277,7 +277,7 @@ def show_worktree_status() -> int:
             if repo_wt.exists():
                 branch, _ = _git(["branch", "--show-current"], repo_wt)
                 status_out, _ = _git(["status", "--short"], repo_wt)
-                count = len([l for l in status_out.split("\n") if l.strip()]) if status_out else 0
+                count = len([line for line in status_out.split("\n") if line.strip()]) if status_out else 0
                 print(f"  {repo_name} ({cfg.repo_type}): {branch} ({count} uncommitted)")
 
         # Check for session files referencing this worktree

package/pennyfarthing_scripts/hooks/__init__.py

@@ -30,7 +30,7 @@ import yaml
 
 # WheelHub port file - central coordination server for all communication
 # Per ADR-0004: "the hub where all communication converges"
-CYCLIST_PORT_FILE = ".wheelhub-port"
+CYCLIST_PORT_FILE = ".bikerack-port"
 
 # Default port if file not found
 DEFAULT_CYCLIST_PORT = 7431
@@ -48,7 +48,7 @@ def find_project_root(start_dir: Path | None = None) -> Path | None:
     """Find the project root by looking for marker files.
 
     Searches for (in order):
-    1. .wheelhub-port (WheelHub is running)
+    1. .bikerack-port (WheelHub is running)
     2. .pennyfarthing directory
     3. .claude directory
 
@@ -84,7 +84,7 @@ def read_port_file(file_name: str, project_root: Path | None = None) -> int | No
     """Read a port number from a Cyclist port file.
 
     Args:
-        file_name: Name of the port file (e.g. .wheelhub-port)
+        file_name: Name of the port file (e.g. .bikerack-port)
         project_root: Project root directory (auto-detected if not provided)
 
     Returns:
@@ -140,6 +140,7 @@ class CyclistSettings:
     permission_mode: str = "manual"  # plan, manual, accept
     relay_mode: bool = False
     bell_mode: bool = False
+    git_monitor: bool = False
     theme: str | None = None
 
 
@@ -206,6 +207,10 @@ def load_settings(project_root: Path | None = None) -> CyclistSettings:
     if "bell_mode" in workflow and isinstance(workflow["bell_mode"], bool):
         settings.bell_mode = workflow["bell_mode"]
 
+    # Handle git_monitor
+    if "git_monitor" in workflow and isinstance(workflow["git_monitor"], bool):
+        settings.git_monitor = workflow["git_monitor"]
+
     return settings