titan-cli 0.1.4__py3-none-any.whl → 0.1.6__py3-none-any.whl

titan_cli/ui/tui/screens/workflow_execution.py

@@ -146,16 +146,16 @@ class WorkflowExecutionScreen(BaseScreen):
                 thread=True
             )
 
-        except (WorkflowNotFoundError, WorkflowExecutionError):
-            pass
-            # TODO Create empty error screen
-            # self._update_workflow_info(f"[red]Error: {e}[/red]")
-        except Exception:
-            pass
-            # TODO Create empty error screen
-            # self._update_workflow_info(
-            #     f"[red]Unexpected error: {type(e).__name__} - {e}[/red]"
-            # )
+        except (WorkflowNotFoundError, WorkflowExecutionError) as e:
+            self._output(f"[red]{Icons.ERROR} Error loading workflow: {e}[/red]")
+            self._output("[dim]Press ESC or Q to return[/dim]")
+        except Exception as e:
+            import traceback
+            error_details = traceback.format_exc()
+            self._output(f"[red]{Icons.ERROR} Unexpected error loading workflow:[/red]")
+            self._output(f"[red]{type(e).__name__}: {e}[/red]")
+            self._output(f"[dim]{error_details}[/dim]")
+            self._output("[dim]Press ESC or Q to return[/dim]")
 
     def _execute_workflow(self) -> None:
         """Execute the workflow in a background thread."""
@@ -493,18 +493,19 @@ class WorkflowExecutionContent(Widget):
 
     def on_descendant_mount(self, event) -> None:
         """Auto-scroll when any widget is mounted as a descendant."""
-        # Don't auto-scroll if we're mounting a PromptInput (it will handle its own scroll)
-        from titan_cli.ui.tui.textual_components import PromptInput
+        from titan_cli.ui.tui.widgets import PromptInput, StepContainer
 
-        # Skip scroll only if the widget itself is a PromptInput
-        # (not if it's a child of PromptInput, to avoid blocking scroll after PromptInput is removed)
-        if not isinstance(event.widget, PromptInput):
+        # Auto-scroll for StepContainers (new steps being added)
+        # StepContainer itself triggers scroll via begin_step(), but this is a safety net
+        if isinstance(event.widget, StepContainer):
+            self._scroll_to_end()
+        # Also scroll for other widgets, but skip PromptInput (it handles its own scroll)
+        elif not isinstance(event.widget, PromptInput):
             self._scroll_to_end()
 
     def handle_event(self, message) -> None:
         """Handle workflow events generically."""
         from titan_cli.ui.tui.textual_workflow_executor import TextualWorkflowExecutor
-        from titan_cli.ui.tui.widgets import Panel
 
         if isinstance(message, TextualWorkflowExecutor.WorkflowStarted):
             # Track nested workflow depth
@@ -513,44 +514,21 @@ class WorkflowExecutionContent(Widget):
             self.append_output(f"\n[bold cyan]🚀 Starting workflow: {message.workflow_name}[/bold cyan]")
 
         elif isinstance(message, TextualWorkflowExecutor.StepStarted):
-            # Format differently for nested workflows
-            if self._workflow_depth > 0:
-                # Nested workflow: show with indentation, no step number
-                indent = "  " * self._workflow_depth
-                self.append_output(f"[cyan]{indent}→ Step {message.step_index}: {message.step_name}[/cyan]")
-            else:
-                # Top-level workflow: show with step number
-                self.append_output(f"[cyan]→ Step {message.step_index}: {message.step_name}[/cyan]")
+            # StepContainer now handles step titles, so we don't display anything here
+            pass
 
         elif isinstance(message, TextualWorkflowExecutor.StepCompleted):
-            # Apply indentation for nested workflows
-            if self._workflow_depth > 0:
-                indent = "  " * self._workflow_depth
-                self.append_output(f"[green]{indent}{Icons.SUCCESS} Completed: {message.step_name}[/green]\n")
-            else:
-                self.append_output(f"[green]{Icons.SUCCESS} Completed: {message.step_name}[/green]\n")
+            # StepContainer now handles step completion (green border), so we don't display anything here
+            pass
 
         elif isinstance(message, TextualWorkflowExecutor.StepFailed):
-            # Mount error panel
-            try:
-                self.mount(Panel(f"Failed: {message.step_name} - {message.error_message}", panel_type="error"))
-                self._scroll_to_end()
-            except Exception:
-                pass
-
-            if message.on_error == "continue":
-                indent = "  " * self._workflow_depth if self._workflow_depth > 0 else ""
-                self.append_output(f"[yellow]{indent}   {Icons.WARNING}  Continuing despite error[/yellow]\n")
-            else:
-                self.append_output("")
+            # Steps handle their own error display via ctx.textual
+            # Executor only coordinates execution, no visual output needed
+            pass
 
         elif isinstance(message, TextualWorkflowExecutor.StepSkipped):
-            # Mount warning panel for skipped steps
-            try:
-                self.mount(Panel(f"Skipped: {message.step_name}", panel_type="warning"))
-                self._scroll_to_end()
-            except Exception:
-                pass
+            # StepContainer now handles step skips (yellow border), so we don't display the panel
+            pass
 
         elif isinstance(message, TextualWorkflowExecutor.WorkflowCompleted):
             # Track nested workflow depth
@@ -586,7 +564,7 @@ class WorkflowExecutionContent(Widget):
                 pass
 
         elif isinstance(message, TextualWorkflowExecutor.WorkflowFailed):
-            # Show error toast for workflow failure
+            # Steps handle their own error display via ctx.textual
+            # Just show a simple notification without duplicating the error message
             self.app.notify(f"❌ Workflow failed at step: {message.step_name}", severity="error", timeout=10)
-            self.append_output(f"[red]{message.error_message}[/red]")
 

titan_cli/ui/tui/screens/workflows.py

@@ -16,6 +16,7 @@ from titan_cli.core.workflows.workflow_filter_service import WorkflowFilterServi
 from titan_cli.core.workflows.workflow_sources import WorkflowInfo
 from titan_cli.ui.tui.screens.workflow_execution import WorkflowExecutionScreen
 from titan_cli.ui.tui.icons import Icons
+from titan_cli.ui.tui.widgets import StyledOption
 from .base import BaseScreen
 
 class WorkflowsScreen(BaseScreen):
@@ -170,11 +171,14 @@ class WorkflowsScreen(BaseScreen):
             workflows_to_show = workflows
 
         for wf_info in workflows_to_show:
-            label = f"{wf_info.name.capitalize()}"
-            description = f"{wf_info.description}"
-            options.append(
-                Option(f"{label}\n[dim]{description}[/dim]", id=wf_info.name)
+            styled_opt = StyledOption(
+                id=wf_info.name,
+                title=wf_info.name.capitalize(),
+                description=wf_info.description
             )
+            # Convert StyledOption to Option with markup
+            prompt = f"[bold]{styled_opt.title}[/bold]\n[dim]{styled_opt.description}[/dim]"
+            options.append(Option(prompt, id=styled_opt.id))
 
         return options if options else [Option("No workflows found", id="none", disabled=True)]