- Rename duplicate step 'the registry last persona should be set to' to 'the mock registry last persona should be set to' in tui_persona_state_coverage_steps.py
- Update corresponding feature file to use the new step name
- Fixes AmbiguousStep error that was preventing unit tests from running

@@ -0,0 +27,4 @@
+    When I switch to session "sess-2"
+    Then the active session should have session_id "sess-2"
+
+  Scenario: Close a session

@@ -0,0 +43,4 @@
+    Given the TUI app has 1 session
+    And the active session has name "Default"
+    When I rename the session to "My Session"
+    Then the active session should have name "My Session"

@@ -0,0 +55,4 @@
+    Then the first session should have active persona "analyst"
+    When I switch to the second session
+    Then the second session should have active persona "coder"
+

@@ -0,0 +1 @@
+Subproject commit 435e409df9fb97b9a08bdd8d7ad2374c25331233

@@ -0,0 +13,4 @@
+
+class MockCommandRouter:
+    """Mock command router for testing."""
+

@@ -108,0 +115,4 @@
+                session_id="default",
+                transcript=[],
+                name="Default",
+                created_at=datetime.utcnow().isoformat(),

@@ -19,0 +41,4 @@
+    @property
+    def active_session_id(self) -> str | None:
+        """Return the active session ID."""
+        if self._app is not None and self._app._sessions:  # type: ignore

@@ -109,0 +188,4 @@
+                if session.session_id == close_id:
+                    if len(self.session_ops.sessions) <= 1:
+                        return "Cannot close the last session"
+                    self.session_ops._app._sessions.pop(i)  # type: ignore

@@ -0,0 +13,4 @@
+
+class MockCommandRouter:
+    """Mock command router for testing."""
+

@@ -108,0 +115,4 @@
+                session_id="default",
+                transcript=[],
+                name="Default",
+                created_at=datetime.utcnow().isoformat(),

@@ -19,0 +41,4 @@
+    @property
+    def active_session_id(self) -> str | None:
+        """Return the active session ID."""
+        if self._app is not None and self._app._sessions:  # type: ignore

@@ -109,0 +188,4 @@
+                if session.session_id == close_id:
+                    if len(self.session_ops.sessions) <= 1:
+                        return "Cannot close the last session"
+                    self.session_ops._app._sessions.pop(i)  # type: ignore

@@ -0,0 +134,4 @@
+@then("the BudgetExceededError used should be {expected:f}")
+def step_check_budget_exc_used(context: Context, expected: float) -> None:
+    """Verify BudgetExceededError used."""
+  assert context.budget_exc.used == expected, (

@@ -0,0 +11,4 @@
+from cleveragents.tui.persona.state import PersonaState
+
+
+class MockCommandRouter:

@@ -108,0 +147,4 @@
+                session_id=session_id,
+                transcript=[],
+                name=session_name,
+                created_at=datetime.utcnow().isoformat(),

@@ -19,0 +41,4 @@
+    @property
+    def active_session_id(self) -> str | None:
+        """Return the active session ID."""
+        if self._app is not None and self._app._sessions:  # type: ignore

@@ -109,0 +176,4 @@
+            target_id = tokens[1]
+            for i, session in enumerate(self.session_ops.sessions):
+                if session.session_id == target_id:
+                    self.session_ops._app._active_session_index = i  # type: ignore

@@ -0,0 +133,4 @@
+
+@then("the BudgetExceededError used should be {expected:f}")
+def step_check_budget_exc_used(context: Context, expected: float) -> None:
+    """Verify BudgetExceededError used."""

@@ -0,0 +11,4 @@
+from cleveragents.tui.persona.state import PersonaState
+
+
+class MockCommandRouter:

@@ -108,0 +115,4 @@
+                session_id="default",
+                transcript=[],
+                name="Default",
+                created_at=datetime.utcnow().isoformat(),

@@ -19,0 +41,4 @@
+    @property
+    def active_session_id(self) -> str | None:
+        """Return the active session ID."""
+        if self._app is not None and self._app._sessions:  # type: ignore

@@ -109,0 +176,4 @@
+            target_id = tokens[1]
+            for i, session in enumerate(self.session_ops.sessions):
+                if session.session_id == target_id:
+                    self.session_ops._app._active_session_index = i  # type: ignore

@@ -0,0 +134,4 @@
+@then("the BudgetExceededError used should be {expected:f}")
+def step_check_budget_exc_used(context: Context, expected: float) -> None:
+    """Verify BudgetExceededError used."""
+  assert context.budget_exc.used == expected, (

@@ -0,0 +11,4 @@
+from cleveragents.tui.persona.state import PersonaState
+
+
+class MockCommandRouter:

@@ -228,0 +410,4 @@
+</html>"""
+
+
+def _run_tui_web(app: CleverAgentsTuiApp, *, port: int = 8000) -> int:  # type: ignore[valid-type]