Merge pull request #537 from UiPath/fix/terminal_style_improve

fix: add input validations in terminal

Author: cristipufu

pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "uipath"
-version = "2.1.32"
+version = "2.1.33"
 description = "Python SDK and CLI for UiPath Platform, enabling programmatic interaction with automation services, process management, and deployment tools."
 readme = { file = "README.md", content-type = "text/markdown" }
 requires-python = ">=3.10"

src/uipath/_cli/_dev/_terminal/__init__.py

@@ -123,8 +123,9 @@ async def action_resume(self) -> None:
                 input = json.loads(input_data)
             except json.JSONDecodeError:
                 return
-            details_panel.current_run.input_data = input
+            details_panel.current_run.resume_data = input
             asyncio.create_task(self._execute_runtime(details_panel.current_run))
+            details_panel.switch_tab("run-tab")
 
     async def action_execute_run(self) -> None:
         """Execute a new run with UiPath runtime."""
@@ -161,7 +162,6 @@ async def _execute_runtime(self, run: ExecutionRun):
         try:
             context: UiPathRuntimeContext = self.runtime_factory.new_context(
                 entrypoint=run.entrypoint,
-                input_json=run.input_data,
                 trace_id=str(uuid4()),
                 execution_id=run.id,
                 logs_min_level=env.get("LOG_LEVEL", "INFO"),
@@ -172,8 +172,10 @@ async def _execute_runtime(self, run: ExecutionRun):
 
             if run.status == "suspended":
                 context.resume = True
+                context.input_json = run.resume_data
                 self._add_info_log(run, f"Resuming execution: {run.entrypoint}")
             else:
+                context.input_json = run.input_data
                 self._add_info_log(run, f"Starting execution: {run.entrypoint}")
 
             run.status = "running"

src/uipath/_cli/_dev/_terminal/_components/_details.py

@@ -1,4 +1,3 @@
-import json
 from typing import Dict, List, Optional
 
 from textual.app import ComposeResult
@@ -29,14 +28,7 @@ def show_span_details(self, trace_msg: TraceMessage):
         details_log = self.query_one("#span-details", RichLog)
         details_log.clear()
 
-        # Format span details
         details_log.write(f"[bold cyan]Span: {trace_msg.span_name}[/bold cyan]")
-        details_log.write(f"[dim]Trace ID: {trace_msg.trace_id}[/dim]")
-        details_log.write(f"[dim]Span ID: {trace_msg.span_id}[/dim]")
-        details_log.write(f"[dim]Run ID: {trace_msg.run_id}[/dim]")
-
-        if trace_msg.parent_span_id:
-            details_log.write(f"[dim]Parent Span: {trace_msg.parent_span_id}[/dim]")
 
         details_log.write("")  # Empty line
 
@@ -68,6 +60,16 @@ def show_span_details(self, trace_msg: TraceMessage):
             for key, value in trace_msg.attributes.items():
                 details_log.write(f"  {key}: {value}")
 
+        details_log.write("")  # Empty line
+
+        # Format span details
+        details_log.write(f"[dim]Trace ID: {trace_msg.trace_id}[/dim]")
+        details_log.write(f"[dim]Span ID: {trace_msg.span_id}[/dim]")
+        details_log.write(f"[dim]Run ID: {trace_msg.run_id}[/dim]")
+
+        if trace_msg.parent_span_id:
+            details_log.write(f"[dim]Parent Span: {trace_msg.parent_span_id}[/dim]")
+
 
 class RunDetailsPanel(Container):
     """Panel showing traces and logs for selected run with tabbed interface."""
@@ -144,10 +146,65 @@ def show_run(self, run: ExecutionRun):
         # Clear and rebuild traces tree using TraceMessage objects
         self._rebuild_spans_tree()
 
+    def switch_tab(self, tab_id: str) -> None:
+        """Switch to a specific tab by id (e.g. 'run-tab', 'traces-tab')."""
+        tabbed = self.query_one(TabbedContent)
+        tabbed.active = tab_id
+
     def _update_resume_tab(self, run: ExecutionRun) -> None:
         resume_panel = self.query_one("#resume-panel", ResumePanel)
         resume_panel.display = run.status == "suspended"
 
+    def _flatten_values(self, value: object, prefix: str = "") -> list[str]:
+        """Flatten nested dict/list structures into dot-notation paths."""
+        lines: list[str] = []
+
+        if value is None:
+            lines.append(f"{prefix}: [dim]—[/dim]" if prefix else "[dim]—[/dim]")
+
+        elif isinstance(value, dict):
+            if not value:
+                lines.append(f"{prefix}: {{}}" if prefix else "{}")
+            else:
+                for k, v in value.items():
+                    new_prefix = f"{prefix}.{k}" if prefix else k
+                    lines.extend(self._flatten_values(v, new_prefix))
+
+        elif isinstance(value, list):
+            if not value:
+                lines.append(f"{prefix}: []" if prefix else "[]")
+            else:
+                for i, item in enumerate(value):
+                    new_prefix = f"{prefix}[{i}]"
+                    lines.extend(self._flatten_values(item, new_prefix))
+
+        elif isinstance(value, str):
+            if prefix:
+                for line in value.splitlines():
+                    lines.append(f"{prefix}: {line}")
+            else:
+                lines.extend(value.splitlines())
+
+        else:
+            if prefix:
+                lines.append(f"{prefix}: {value}")
+            else:
+                lines.append(str(value))
+
+        return lines
+
+    def _write_block(
+        self, log: RichLog, title: str, data: object, style: str = "white"
+    ) -> None:
+        """Pretty-print a block with flattened dot-notation paths."""
+        log.write(f"[bold {style}]{title.upper()}:[/bold {style}]")
+        log.write("[dim]" + "=" * 50 + "[/dim]")
+
+        for line in self._flatten_values(data):
+            log.write(line)
+
+        log.write("")
+
     def _show_run_details(self, run: ExecutionRun):
         """Display detailed information about the run in the Details tab."""
         self._update_resume_tab(run)
@@ -202,35 +259,16 @@ def _show_run_details(self, run: ExecutionRun):
 
         run_details_log.write("")
 
-        # Input section
-        if hasattr(run, "input_data") and run.input_data is not None:
-            run_details_log.write("[bold green]INPUT:[/bold green]")
-            run_details_log.write("[dim]" + "=" * 50 + "[/dim]")
-
-            # Handle different input types
-            if isinstance(run.input_data, str):
-                run_details_log.write(run.input_data)
-            elif isinstance(run.input_data, dict):
-                run_details_log.write(json.dumps(run.input_data, indent=2))
-            else:
-                run_details_log.write(str(run.input_data))
-
-            run_details_log.write("")
+        if hasattr(run, "input_data"):
+            self._write_block(run_details_log, "Input", run.input_data, style="green")
 
-        # Output section
-        if hasattr(run, "output_data") and run.output_data is not None:
-            run_details_log.write("[bold magenta]OUTPUT:[/bold magenta]")
-            run_details_log.write("[dim]" + "=" * 50 + "[/dim]")
-
-            # Handle different output types
-            if isinstance(run.output_data, str):
-                run_details_log.write(run.output_data)
-            elif isinstance(run.output_data, dict):
-                run_details_log.write(json.dumps(run.output_data, indent=2))
-            else:
-                run_details_log.write(str(run.output_data))
+        if hasattr(run, "resume_data") and run.resume_data:
+            self._write_block(run_details_log, "Resume", run.resume_data, style="green")
 
-            run_details_log.write("")
+        if hasattr(run, "output_data"):
+            self._write_block(
+                run_details_log, "Output", run.output_data, style="magenta"
+            )
 
         # Error section (if applicable)
         if hasattr(run, "error") and run.error:

src/uipath/_cli/_dev/_terminal/_components/_json_input.py

@@ -0,0 +1,23 @@
+import json
+
+from textual.widgets import TextArea
+
+
+class JsonInput(TextArea):
+    """TextArea that validates JSON on change."""
+
+    def validate_json(self) -> bool:
+        text = self.text.strip()
+        if not text:
+            self.remove_class("invalid")
+            return True
+        try:
+            json.loads(text)
+            self.remove_class("invalid")
+            return True
+        except json.JSONDecodeError:
+            self.add_class("invalid")
+            return False
+
+    def on_text_area_changed(self, event: TextArea.Changed) -> None:
+        self.validate_json()

src/uipath/_cli/_dev/_terminal/_components/_new.py

@@ -7,6 +7,8 @@
 from textual.reactive import reactive
 from textual.widgets import Button, Select, TabbedContent, TabPane, TextArea
 
+from ._json_input import JsonInput
+
 
 def mock_json_from_schema(schema: Dict[str, Any]) -> Dict[str, Any]:
     props = schema.get("properties", {})
@@ -74,7 +76,7 @@ def compose(self) -> ComposeResult:
                         allow_blank=False,
                     )
 
-                    yield TextArea(
+                    yield JsonInput(
                         text=self.initial_input,
                         language="json",
                         id="json-input",

src/uipath/_cli/_dev/_terminal/_components/_resume.py

@@ -4,6 +4,8 @@
 from textual.containers import Container, Horizontal, Vertical
 from textual.widgets import Button, TextArea
 
+from ._json_input import JsonInput
+
 
 class ResumePanel(Container):
     """Panel for resuming a suspended run."""
@@ -13,7 +15,7 @@ def __init__(self, **kwargs):
 
     def compose(self) -> ComposeResult:
         with Vertical():
-            yield TextArea(
+            yield JsonInput(
                 text=json.dumps({"value": ""}, indent=2),
                 language="json",
                 id="resume-json-input",

src/uipath/_cli/_dev/_terminal/_models/_execution.py

@@ -16,6 +16,7 @@ def __init__(self, entrypoint: str, input_data: Dict[str, Any]):
         self.id = str(uuid4())[:8]
         self.entrypoint = entrypoint
         self.input_data = input_data
+        self.resume_data: Optional[Dict[str, Any]] = None
         self.output_data: Optional[Dict[str, Any]] = None
         self.start_time = datetime.now()
         self.end_time: Optional[datetime] = None

src/uipath/_cli/_dev/_terminal/_styles/terminal.tcss

@@ -218,3 +218,7 @@ SpanDetailsDisplay {
     text-style: bold;
     border: none;
 }
+
+TextArea.invalid {
+    border: tall red;
+}