Fix TUI Ralph using cached API client for completion verification

The TUI passed its cached APIClient to the Ralph orchestrator, causing
verify_feature_completed to return stale responses. Feature status
mutated externally by the coding agent via MCP was invisible to the
cached client, leading to false retry loops. Create a separate uncached
client for the orchestrator.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: shuveb

pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "mfbt-cli"
-version = "0.1.7"
+version = "0.1.8"
 description = "CLI tool for the mfbt platform — interactive TUI and subcommands for managing mfbt projects"
 readme = "README.md"
 requires-python = ">=3.10"

src/mfbt/tui/screens/ralph_panel.py

@@ -56,6 +56,7 @@ def __init__(
         self._config = config
         self._phase_id = phase_id
         self._orchestrator: Any = None
+        self._orchestrator_client: APIClient | None = None
         self._ralph_config: Any = None
         self._phases_progress: dict[str, dict] = {}
         self._orphan_modules: list[dict[str, Any]] = []
@@ -373,6 +374,12 @@ def stop(self) -> None:
                 self._orchestrator.stop()
             except Exception:
                 pass
+        if self._orchestrator_client is not None:
+            try:
+                self._orchestrator_client.close()
+            except Exception:
+                pass
+            self._orchestrator_client = None
 
     def pause_orchestrator(self) -> None:
         """Show pause modal — lets user continue or stop the orchestration loop."""
@@ -509,7 +516,7 @@ def _run_single_retry(
         from mfbt.commands.ralph.progress import resolve_feature_task
         from mfbt.commands.ralph.types import FeatureOutcome
 
-        feature = resolve_feature_task(self._api_client, feature_info)
+        feature = resolve_feature_task(self._orchestrator_client, feature_info)
         if feature is None:
             def _notify_error() -> None:
                 self.app.notify(
@@ -655,6 +662,29 @@ def _glisten_tick(self) -> None:
         except Exception:
             pass
 
+    def _build_uncached_client(self) -> APIClient:
+        """Create an APIClient without response caching for the orchestrator.
+
+        The orchestrator needs fresh API responses (especially for
+        verify_feature_completed) because feature status is mutated
+        externally by the coding agent via MCP.
+        """
+        from mfbt.api_client import APIClient
+        from mfbt.auth_flow import run_interactive_auth
+        from mfbt.token_manager import TokenManager
+
+        base_url = self._config.get("base_url", "https://app.mfbt.ai")
+        tm = TokenManager(base_url)
+
+        def _on_auth_required() -> bool:
+            from rich.console import Console
+
+            c = Console()
+            c.print("\n[yellow]Session expired. Re-authenticating...[/yellow]")
+            return run_interactive_auth(base_url, tm, c)
+
+        return APIClient(base_url, tm, cache=None, on_auth_required=_on_auth_required)
+
     def _run_orchestrator(self) -> None:
         """Run the orchestrator in a worker thread."""
         from mfbt.commands.ralph.log_capture import RalphLogger
@@ -668,7 +698,8 @@ def _run_orchestrator(self) -> None:
             log_base_dir, self._ralph_config.coding_agent
         )
 
+        self._orchestrator_client = self._build_uncached_client()
         self._orchestrator = RalphOrchestrator(
-            self._ralph_config, self._api_client, display, logger=ralph_logger
+            self._ralph_config, self._orchestrator_client, display, logger=ralph_logger
         )
         self._orchestrator.run()