@pennyfarthing/core 11.1.1 → 11.2.0

package/pennyfarthing_scripts/bikerack/tui.py

@@ -5,21 +5,30 @@ Story 103-4: Connection status indicator in TUI header.
 Story 103-6: SprintPanel as default panel on launch.
 Story 103-7: /bc TUI panel focus — subscribe to /ws/focus, switch panels.
 Story 103-9: Panel header chrome — icon + name indicator for active panel.
+Panel navigation: Mount all panels, tab bar, keyboard switching, command palette.
 """
 
 from __future__ import annotations
 
+from functools import partial
 from pathlib import Path
 from typing import Any
 
 from textual.app import App, ComposeResult
 from textual.binding import Binding
+from textual.command import Hit, Hits, Provider
 from textual.containers import VerticalScroll
 from textual.reactive import reactive
 from textual.widgets import Footer, Header, Static
 
 from pennyfarthing_scripts.bc.focus import get_last_panel, save_last_panel
+from pennyfarthing_scripts.bikerack.background_panel import BackgroundPanel
 from pennyfarthing_scripts.bikerack.base_panel import get_panel_icon
+from pennyfarthing_scripts.bikerack.changed_panel import ChangedPanel
+from pennyfarthing_scripts.bikerack.debug_panel import DebugPanel
+from pennyfarthing_scripts.bikerack.diffs_panel import DiffsPanel
+from pennyfarthing_scripts.bikerack.git_panel import GitPanel
+from pennyfarthing_scripts.bikerack.progress_panel import ProgressPanel
 from pennyfarthing_scripts.bikerack.sprint_panel import SprintPanel
 from pennyfarthing_scripts.bikerack.ws_client import ConnectionState, WheelHubClient
 
@@ -30,7 +39,48 @@ STATE_DISPLAY: dict[ConnectionState, str] = {
     ConnectionState.CONNECTING: "[yellow]● Connecting…[/yellow]",
 }
 
-# Human-readable display names for panels
+# Agent role colors for Rich markup (mapped from React AGENT_COLORS)
+AGENT_ROLE_COLORS: dict[str, str] = {
+    "pm": "purple",
+    "sm": "blue",
+    "dev": "green",
+    "tea": "cyan",
+    "reviewer": "red",
+    "architect": "dark_orange",
+    "devops": "bright_cyan",
+    "ux-designer": "magenta",
+    "tech-writer": "white",
+    "orchestrator": "bright_magenta",
+    "ba": "bright_green",
+}
+
+AGENT_ABBREV: dict[str, str] = {
+    "pm": "PM",
+    "sm": "SM",
+    "dev": "DEV",
+    "tea": "TEA",
+    "reviewer": "REV",
+    "architect": "ARC",
+    "devops": "OPS",
+    "ux-designer": "UX",
+    "tech-writer": "TW",
+    "orchestrator": "ORC",
+    "ba": "BA",
+}
+
+# Ordered panel registry: (key, display_name, widget_class)
+# Only panels with implemented widget classes are included.
+PANEL_REGISTRY: list[tuple[str, str]] = [
+    ("sprint", "Sprint"),
+    ("git", "Git"),
+    ("diffs", "Diffs"),
+    ("changed", "Changed"),
+    ("background", "Background"),
+    ("debug", "Debug"),
+    ("progress", "Progress"),
+]
+
+# Human-readable display names for panels (full set for external focus messages)
 PANEL_DISPLAY_NAMES: dict[str, str] = {
     "sprint": "Sprint",
     "git": "Git",
@@ -42,24 +92,106 @@ PANEL_DISPLAY_NAMES: dict[str, str] = {
     "changed": "Changed",
     "ac": "Acceptance Criteria",
     "debug": "Debug",
+    "progress": "Progress",
     "settings": "Settings",
     "tty": "TTY",
 }
 
+# Keys from PANEL_REGISTRY for fast lookup
+_PANEL_KEYS = [key for key, _ in PANEL_REGISTRY]
 
-class PanelIndicator(Static):
-    """Displays the active panel's Nerd Font icon and name."""
 
-    panel_key: reactive[str] = reactive("sprint")
+class PanelTabBar(Static):
+    """Horizontal tab bar showing all available panels with active highlight."""
 
-    def watch_panel_key(self, key: str) -> None:
-        """Update display when the active panel changes."""
-        icon = get_panel_icon(key)
-        name = PANEL_DISPLAY_NAMES.get(key, key.title())
-        if icon:
-            self.update(f"[bold]{icon} {name}[/bold]")
-        else:
-            self.update(f"[bold]{name}[/bold]")
+    active: reactive[str] = reactive("sprint")
+
+    def watch_active(self, key: str) -> None:
+        """Re-render tab bar when active panel changes."""
+        parts: list[str] = []
+        for panel_key, display_name in PANEL_REGISTRY:
+            icon = get_panel_icon(panel_key)
+            idx = _PANEL_KEYS.index(panel_key) + 1
+            prefix = f"{idx}:"
+            if panel_key == key:
+                if icon:
+                    parts.append(f"[bold reverse] {prefix}{icon} {display_name} [/]")
+                else:
+                    parts.append(f"[bold reverse] {prefix}{display_name} [/]")
+            else:
+                if icon:
+                    parts.append(f"[dim]{prefix}{icon} {display_name}[/]")
+                else:
+                    parts.append(f"[dim]{prefix}{display_name}[/]")
+        self.update("  ".join(parts))
+
+
+class AgentHeader(Static):
+    """Displays current agent persona from WheelHub /ws/persona channel."""
+
+    def __init__(self, **kwargs: Any) -> None:
+        super().__init__(**kwargs)
+        self._is_streaming: bool = False
+        self._persona_data: dict[str, Any] = {}
+
+    def _apply_persona(self, data: dict[str, Any]) -> None:
+        """Render persona data into the header."""
+        if data.get("type") == "streaming":
+            self._is_streaming = bool(data.get("isStreaming", False))
+            self._render_header()
+            return
+
+        self._persona_data = data
+        self._is_streaming = bool(data.get("isStreaming", False))
+        self._render_header()
+
+    def _render_header(self) -> None:
+        """Re-render the header from stored state."""
+        data = self._persona_data
+        char = data.get("character", "")
+        role = data.get("role", "")
+        role_desc = data.get("roleDescription", "")
+        quote = data.get("quote", "")
+        style = data.get("style", "")
+        theme = data.get("theme", "")
+
+        if not char:
+            self.update("[dim]Waiting for agent...[/dim]")
+            return
+
+        parts: list[str] = []
+
+        # Role badge
+        if role:
+            abbrev = AGENT_ABBREV.get(role, role.upper()[:3])
+            color = AGENT_ROLE_COLORS.get(role, "bright_magenta")
+            parts.append(f"[bold {color}][{abbrev}][/bold {color}]")
+
+        # Character name
+        parts.append(f"[bold]{char}[/bold]")
+
+        # Theme name
+        if theme:
+            from pennyfarthing_scripts.bikerack.base_panel import humanize_theme
+            parts.append(f"[dim]{humanize_theme(theme)}[/dim]")
+
+        # Streaming indicator
+        if self._is_streaming:
+            parts.append("[bold yellow]⚡[/bold yellow]")
+
+        line = "  ".join(parts)
+
+        # Role description / style subtitle
+        if role_desc:
+            line += f"\n[dim]{role_desc}[/dim]"
+        elif style:
+            line += f"\n[dim]{style}[/dim]"
+
+        # Quote
+        if quote:
+            line += f"\n[italic dim]\"{quote}\"[/italic dim]"
+
+        self.update(line)
 
 
 class ConnectionStatus(Static):
@@ -74,50 +206,212 @@ class ConnectionStatus(Static):
         self.update(STATE_DISPLAY.get(state, "● Unknown"))
 
 
+class PanelCommands(Provider):
+    """Command palette provider for panel switching."""
+
+    async def search(self, query: str) -> Hits:
+        matcher = self.matcher(query)
+        for panel_key, display_name in PANEL_REGISTRY:
+            icon = get_panel_icon(panel_key)
+            label = f"{icon} {display_name}" if icon else display_name
+            score = matcher.match(display_name)
+            if score > 0:
+                yield Hit(
+                    score,
+                    matcher.highlight(label),
+                    partial(self.app.action_switch_panel, panel_key),
+                    help=f"Switch to {display_name} panel",
+                )
+
+
 class BikeRackApp(App):
     """BikeRack TUI application shell."""
 
     TITLE = "BikeRack"
 
+    CSS = """
+    #agent-header {
+        height: auto;
+        max-height: 3;
+        padding: 0 1;
+    }
+    #tab-bar {
+        height: 1;
+    }
+    #connection-status {
+        height: 1;
+    }
+    """
+
+    COMMANDS = App.COMMANDS | {PanelCommands}
+
     BINDINGS = [
         Binding("q", "quit", "Quit"),
+        Binding("1", "switch_panel('sprint')", "Sprint", show=False),
+        Binding("2", "switch_panel('git')", "Git", show=False),
+        Binding("3", "switch_panel('diffs')", "Diffs", show=False),
+        Binding("4", "switch_panel('changed')", "Changed", show=False),
+        Binding("5", "switch_panel('background')", "Background", show=False),
+        Binding("6", "switch_panel('debug')", "Debug", show=False),
+        Binding("7", "switch_panel('progress')", "Progress", show=False),
+        Binding("bracketright", "next_panel", "]Next"),
+        Binding("bracketleft", "prev_panel", "[Prev"),
+        Binding("tab", "next_panel", show=False),
+        Binding("shift+tab", "prev_panel", show=False),
+        Binding("n", "next_diff_file", "Next file", show=False),
+        Binding("p", "prev_diff_file", "Prev file", show=False),
+        Binding("j", "next_epic", show=False),
+        Binding("k", "prev_epic", show=False),
+        Binding("e", "toggle_epic", show=False),
     ]
 
     def __init__(self, client=None, **kwargs):
         super().__init__(**kwargs)
         self._client = client
-        self._focused_panel: str | None = None
+        self._focused_panel: str = "sprint"
         self._previous_panel: str | None = None
 
     def compose(self) -> ComposeResult:
         yield Header()
-        yield PanelIndicator(id="panel-indicator")
+        yield AgentHeader(id="agent-header")
+        yield PanelTabBar(id="tab-bar")
         yield ConnectionStatus(
             STATE_DISPLAY[ConnectionState.DISCONNECTED],
             id="connection-status",
         )
         with VerticalScroll(id="main-content"):
-            yield SprintPanel(client=self._client, id="sprint-panel")
+            yield SprintPanel(client=self._client, id="panel-sprint")
+            yield GitPanel(client=self._client, id="panel-git")
+            yield DiffsPanel(client=self._client, id="panel-diffs")
+            yield ChangedPanel(client=self._client, id="panel-changed")
+            yield BackgroundPanel(client=self._client, id="panel-background")
+            yield DebugPanel(client=self._client, id="panel-debug")
+            yield ProgressPanel(client=self._client, id="panel-progress")
         yield Footer()
 
     async def on_mount(self) -> None:
+        # Restore last panel or default to sprint
         result = get_last_panel()
+        initial = "sprint"
         if result.get("success") and result.get("last_panel"):
-            self._focused_panel = result["last_panel"]
+            last = result["last_panel"]
+            if last in _PANEL_KEYS:
+                initial = last
+
+        self._focused_panel = initial
 
-        # Set initial panel indicator
-        self._update_panel_indicator(self._focused_panel or "sprint")
+        # Hide all panels except the active one
+        for panel_key in _PANEL_KEYS:
+            widget_id = f"panel-{panel_key}"
+            try:
+                widget = self.query_one(f"#{widget_id}")
+                widget.display = (panel_key == initial)
+            except Exception:
+                pass
+
+        # Set tab bar active state
+        self._update_tab_bar(initial)
 
         if self._client is not None:
             self._client.on_state_change(self._on_ws_state_change)
             self._client.subscribe("focus", self._handle_focus_message)
+            self._client.subscribe("persona", self._handle_persona_message)
             self.run_worker(self._client.connect(), exclusive=True, name="ws-client")
 
-    def _update_panel_indicator(self, panel_key: str) -> None:
-        """Update the panel indicator widget with the given panel key."""
+    def action_switch_panel(self, key: str) -> None:
+        """Switch to a panel by key."""
+        if key not in _PANEL_KEYS:
+            return
+        if key == self._focused_panel:
+            return
+
+        # Hide current panel
+        try:
+            current = self.query_one(f"#panel-{self._focused_panel}")
+            current.display = False
+        except Exception:
+            pass
+
+        # Show target panel
+        try:
+            target = self.query_one(f"#panel-{key}")
+            target.display = True
+        except Exception:
+            pass
+
+        self._previous_panel = self._focused_panel
+        self._focused_panel = key
+        save_last_panel(key, project_dir=None)
+        self._update_tab_bar(key)
+
+    def action_next_panel(self) -> None:
+        """Cycle to the next panel."""
         try:
-            indicator = self.query_one("#panel-indicator", PanelIndicator)
-            indicator.panel_key = panel_key
+            idx = _PANEL_KEYS.index(self._focused_panel)
+        except ValueError:
+            idx = 0
+        next_idx = (idx + 1) % len(_PANEL_KEYS)
+        self.action_switch_panel(_PANEL_KEYS[next_idx])
+
+    def action_prev_panel(self) -> None:
+        """Cycle to the previous panel."""
+        try:
+            idx = _PANEL_KEYS.index(self._focused_panel)
+        except ValueError:
+            idx = 0
+        prev_idx = (idx - 1) % len(_PANEL_KEYS)
+        self.action_switch_panel(_PANEL_KEYS[prev_idx])
+
+    def action_next_diff_file(self) -> None:
+        """Advance to next file in diffs panel."""
+        if self._focused_panel == "diffs":
+            try:
+                panel = self.query_one("#panel-diffs", DiffsPanel)
+                panel.next_file()
+            except Exception:
+                pass
+
+    def action_prev_diff_file(self) -> None:
+        """Go to previous file in diffs panel."""
+        if self._focused_panel == "diffs":
+            try:
+                panel = self.query_one("#panel-diffs", DiffsPanel)
+                panel.prev_file()
+            except Exception:
+                pass
+
+    def action_next_epic(self) -> None:
+        """Move to next epic in sprint panel."""
+        if self._focused_panel == "sprint":
+            try:
+                panel = self.query_one("#panel-sprint", SprintPanel)
+                panel.next_epic()
+            except Exception:
+                pass
+
+    def action_prev_epic(self) -> None:
+        """Move to previous epic in sprint panel."""
+        if self._focused_panel == "sprint":
+            try:
+                panel = self.query_one("#panel-sprint", SprintPanel)
+                panel.prev_epic()
+            except Exception:
+                pass
+
+    def action_toggle_epic(self) -> None:
+        """Toggle expand/collapse on selected epic in sprint panel."""
+        if self._focused_panel == "sprint":
+            try:
+                panel = self.query_one("#panel-sprint", SprintPanel)
+                panel.toggle_epic()
+            except Exception:
+                pass
+
+    def _update_tab_bar(self, panel_key: str) -> None:
+        """Update the tab bar widget with the given panel key."""
+        try:
+            tab_bar = self.query_one("#tab-bar", PanelTabBar)
+            tab_bar.active = panel_key
         except Exception:
             pass
 
@@ -135,14 +429,23 @@ class BikeRackApp(App):
             return
 
         focus = message["focus"]
-        if focus is not None:
+        if focus is not None and focus in _PANEL_KEYS:
+            self.action_switch_panel(focus)
+        elif focus is not None:
+            # Panel exists in display names but not implemented — just update state
             self._previous_panel = self._focused_panel
             self._focused_panel = focus
             save_last_panel(focus, project_dir=None)
-            self._update_panel_indicator(focus)
-        else:
-            self._focused_panel = None
-            self._previous_panel = None
+
+    def _handle_persona_message(self, message: dict[str, Any] | None) -> None:
+        """Handle incoming persona channel messages."""
+        if message is None or not isinstance(message, dict):
+            return
+        try:
+            header = self.query_one("#agent-header", AgentHeader)
+            header._apply_persona(message)
+        except Exception:
+            pass
 
     def _on_ws_state_change(self, state: ConnectionState) -> None:
         """Handle WheelHub connection state changes."""
@@ -156,16 +459,19 @@ class BikeRackApp(App):
 DEFAULT_PORT = 2898
 
 
-def main(port: int | None = None, project_dir: Path | None = None) -> None:
+def main(
+    port: int | None = None,
+    project_dir: Path | None = None,
+) -> None:
     """Launch BikeRack TUI as a standalone application.
 
     Args:
-        port: Explicit WheelHub port. If None, reads from .bikerack-port file.
+        port: Explicit WheelHub port. If None, reads from .wheelhub-port file.
         project_dir: Project directory for port file discovery. Defaults to cwd.
     """
     if port is None:
         if project_dir is not None:
-            port_file = project_dir / ".bikerack-port"
+            port_file = project_dir / ".wheelhub-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 .bikerack-port file, fallback to DEFAULT_PORT.
+        """Read port from .wheelhub-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 / ".bikerack-port"
+            port_file = self._project_dir / ".wheelhub-port"
             if port_file.exists():
                 try:
                     return int(port_file.read_text().strip())

package/pennyfarthing_scripts/cli.py

@@ -137,6 +137,11 @@ from pennyfarthing_scripts.epic.cli import epic  # noqa: E402
 
 cli.add_command(epic)
 
+# Import and register consultation group
+from pennyfarthing_scripts.consultation.cli import consultation  # noqa: E402
+
+cli.add_command(consultation)
+
 
 @cli.group()
 def agent():
@@ -144,7 +149,8 @@ def agent():
 
     \b
     Commands:
-      start  - Start an agent session with context
+      start    - Start an agent session with context
+      heatmap  - Visualize context distribution and attention
     """
     pass
 
@@ -193,78 +199,44 @@ def agent_start(
     raise SystemExit(exit_code)
 
 
-@cli.group()
-def workflow():
-    """Workflow state and phase management.
-
-    \b
-    Commands:
-      check       - Check current workflow state
-      phase-check - Verify phase ownership
-      handoff     - Emit handoff marker
-    """
-    pass
-
-
-@workflow.command("check")
-@click.option("--json", "output_json", is_flag=True, help="Output as JSON")
-def workflow_check(output_json: bool):
-    """Check current workflow state.
-
-    Returns the current story ID, phase, and workflow state.
-    """
-    # Lazy import - only load when command is actually invoked
-    from pennyfarthing_scripts.workflow import get_workflow_state
-
-    state = get_workflow_state()
-
-    if output_json:
-        import json
-
-        click.echo(json.dumps(state, indent=2))
-    else:
-        click.echo(f"State: {state.get('state', 'unknown')}")
-        if state.get("story_id"):
-            click.echo(f"Story: {state['story_id']}")
-        if state.get("workflow"):
-            click.echo(f"Workflow: {state['workflow']}")
-        if state.get("phase"):
-            click.echo(f"Phase: {state['phase']}")
-
+@agent.command("heatmap")
+@click.argument("name", required=False)
+@click.option("--all", "show_all", is_flag=True, help="Show summary across all primary agents")
+@click.option("--csv", "csv_output", is_flag=True, help="Output CSV for machine consumption")
+@click.option("--json", "json_output", is_flag=True, help="Output JSON")
+def agent_heatmap(
+    name: str | None,
+    show_all: bool,
+    csv_output: bool,
+    json_output: bool,
+):
+    """Visualize context distribution and attention for agent activation.
 
-@workflow.command("phase-check")
-@click.argument("workflow_name")
-@click.argument("phase")
-def workflow_phase_check(workflow_name: str, phase: str):
-    """Check which agent owns a workflow phase.
+    Shows a heat map of how tokens are distributed across sections of an
+    agent's activation context, with attention scores based on the
+    "Lost in the Middle" U-shaped attention model.
 
     \b
-    Arguments:
-      WORKFLOW_NAME  - The workflow type (tdd, trivial, etc.)
-      PHASE          - The phase to check (red, implement, review, etc.)
+    Examples:
+      pf agent heatmap sm           # Detailed view for SM
+      pf agent heatmap --all        # Summary across all agents
+      pf agent heatmap dev --json   # JSON output for tooling
     """
-    # Lazy import
-    from pennyfarthing_scripts.workflow import get_phase_owner
+    from pennyfarthing_scripts.prime.heatmap import run_heatmap
 
-    owner = get_phase_owner(workflow_name, phase)
-    click.echo(owner)
+    exit_code = run_heatmap(
+        agent_name=name,
+        show_all=show_all,
+        csv_output=csv_output,
+        json_output=json_output,
+    )
+    raise SystemExit(exit_code)
 
 
-@workflow.command("handoff")
-@click.argument("next_agent")
-def workflow_handoff(next_agent: str):
-    """Emit a handoff marker for Cyclist.
+# Import and register workflow group
+from pennyfarthing_scripts.workflow.cli import workflow  # noqa: E402
 
-    \b
-    Arguments:
-      NEXT_AGENT  - The agent to hand off to (tea, dev, reviewer, etc.)
-    """
-    # Output the marker format expected by Cyclist
-    click.echo("---")
-    click.echo("AGENT_COMMAND:")
-    click.echo(f'  marker: "<!-- CYCLIST:HANDOFF:/{next_agent} -->"')
-    click.echo(f'  fallback: "Run `/{next_agent}` to continue"')
-    click.echo("---")
+cli.add_command(workflow)
 
 
 @cli.command("help")

package/pennyfarthing_scripts/consultation/__init__.py

@@ -0,0 +1 @@
+"""Consultation package — Dialogue file management for tandem agent consultation."""