Merge pull request #49 from UiPath/fix/consistent-CLI-design

fix: use ConsoleLogger

Author: radu-mocanu

pyproject.toml

@@ -1,11 +1,11 @@
 [project]
 name = "uipath-langchain"
-version = "0.0.95"
+version = "0.0.96"
 description = "UiPath Langchain"
 readme = { file = "README.md", content-type = "text/markdown" }
 requires-python = ">=3.10"
 dependencies = [
-    "uipath>=2.0.23, <2.1.0",
+    "uipath>=2.0.28, <2.1.0",
     "langgraph>=0.2.70",
     "langchain-core>=0.3.34",
     "langgraph-checkpoint-sqlite>=2.0.3",

src/uipath_langchain/_cli/cli_init.py

@@ -4,14 +4,15 @@
 import uuid
 from typing import Any, Dict
 
-import click
 from langgraph.graph.state import CompiledStateGraph
+from uipath._cli._utils._console import ConsoleLogger
 from uipath._cli._utils._parse_ast import generate_bindings_json  # type: ignore
 from uipath._cli.middlewares import MiddlewareResult
-from uipath._cli.spinner import Spinner
 
 from ._utils._graph import LangGraphConfig
 
+console = ConsoleLogger()
+
 
 def resolve_refs(schema, root=None):
     """Recursively resolves $ref references in a JSON schema."""
@@ -95,15 +96,13 @@ def generate_schema_from_graph(graph: CompiledStateGraph) -> Dict[str, Any]:
 
 async def langgraph_init_middleware_async(entrypoint: str) -> MiddlewareResult:
     """Middleware to check for langgraph.json and create uipath.json with schemas"""
-    spinner = Spinner("Initializing UiPath project...")
     config = LangGraphConfig()
     if not config.exists:
         return MiddlewareResult(
             should_continue=True
         )  # Continue with normal flow if no langgraph.json
 
     try:
-        spinner.start()
         config.load_config()
         entrypoints = []
         all_bindings = {"version": "2.0", "resources": []}
@@ -134,8 +133,8 @@ async def langgraph_init_middleware_async(entrypoint: str) -> MiddlewareResult:
                         if "resources" in file_bindings:
                             all_bindings["resources"] = file_bindings["resources"]
                 except Exception as e:
-                    print(
-                        f"⚠️ Warning: Could not generate bindings for {graph.file_path}: {str(e)}"
+                    console.warning(
+                        f"Warning: Could not generate bindings for {graph.file_path}: {str(e)}"
                     )
 
                 new_entrypoint: dict[str, Any] = {
@@ -148,21 +147,18 @@ async def langgraph_init_middleware_async(entrypoint: str) -> MiddlewareResult:
                 entrypoints.append(new_entrypoint)
 
             except Exception as e:
-                spinner.stop()
-                print(f"Error during graph load: {e}")
+                console.error(f"Error during graph load: {e}")
                 return MiddlewareResult(
                     should_continue=False,
-                    error_message=f"❌ Failed to load graph '{graph.name}': {str(e)}",
                     should_include_stacktrace=True,
                 )
             finally:
                 await graph.cleanup()
 
         if entrypoint and not entrypoints:
-            spinner.stop()
+            console.error(f"Error: No graph found with name '{entrypoint}'")
             return MiddlewareResult(
                 should_continue=False,
-                error_message=f"❌ Error: No graph found with name '{entrypoint}'",
             )
 
         uipath_config = {"entryPoints": entrypoints, "bindings": all_bindings}
@@ -177,25 +173,22 @@ async def langgraph_init_middleware_async(entrypoint: str) -> MiddlewareResult:
             try:
                 with open(mermaid_file_path, "w") as f:
                     f.write(mermaid_content)
+                console.success(f" Created '{mermaid_file_path}' file.")
             except Exception as write_error:
-                spinner.stop()
+                console.error(
+                    f"Error writing mermaid file for '{graph_name}': {str(write_error)}"
+                )
                 return MiddlewareResult(
                     should_continue=False,
-                    error_message=f"❌ Error writing mermaid file for '{graph_name}': {str(write_error)}",
                     should_include_stacktrace=True,
                 )
-        spinner.stop()
-        return MiddlewareResult(
-            should_continue=False,
-            info_message=click.style("✓ ", fg="green", bold=True)
-            + f" Configuration file {config_path} created successfully.",
-        )
+        console.success(f" Created '{config_path}' file.")
+        return MiddlewareResult(should_continue=False)
 
     except Exception as e:
-        spinner.stop()
+        console.error(f"Error processing langgraph configuration: {str(e)}")
         return MiddlewareResult(
             should_continue=False,
-            error_message=f"❌ Error processing langgraph configuration: {str(e)}",
             should_include_stacktrace=True,
         )
 

src/uipath_langchain/_cli/cli_new.py

@@ -2,8 +2,10 @@
 import shutil
 
 import click
+from uipath._cli._utils._console import ConsoleLogger
 from uipath._cli.middlewares import MiddlewareResult
-from uipath._cli.spinner import Spinner
+
+console = ConsoleLogger()
 
 
 def generate_script(target_directory):
@@ -41,36 +43,29 @@ def generate_pyproject(target_directory, project_name):
 
 def langgraph_new_middleware(name: str) -> MiddlewareResult:
     """Middleware to create demo langchain agent"""
-    spinner = Spinner("Creating demo agent...")
+
     directory = os.getcwd()
 
     try:
-        generate_script(directory)
-        click.echo(click.style("✓ ", fg="green", bold=True) + "Created main.py file")
-        click.echo(
-            click.style("✓ ", fg="green", bold=True) + "Created langgraph.json file"
-        )
-        generate_pyproject(directory, name)
-        click.echo(
-            click.style("✓ ", fg="green", bold=True) + "Created pyproject.toml file"
-        )
-        os.system("uv sync")
-        spinner.start()
-        ctx = click.get_current_context()
-        init_cmd = ctx.parent.command.get_command(ctx, "init")  # type: ignore
-        ctx.invoke(init_cmd)
-        spinner.stop()
-        click.echo(
-            click.style("✓ ", fg="green", bold=True) + " Agent created successfully."
-        )
-        return MiddlewareResult(
-            should_continue=False,
-            info_message="""Usage example: ` uipath run agent '{"topic": "UiPath"}' `""",
-        )
+        with console.spinner(f"Creating new agent {name} in current directory ..."):
+            generate_pyproject(directory, name)
+            generate_script(directory)
+            console.success("Created 'main.py' file.")
+            console.success("Created 'langgraph.json' file.")
+            generate_pyproject(directory, name)
+            console.success("Created 'pyproject.toml' file.")
+            os.system("uv sync")
+            ctx = click.get_current_context()
+            init_cmd = ctx.parent.command.get_command(ctx, "init")  # type: ignore
+            ctx.invoke(init_cmd)
+            console.config(
+                " Please ensure to define either ANTHROPIC_API_KEY or OPENAI_API_KEY in your .env file."
+            )
+            console.hint(""" Run agent: uipath run agent '{"topic": "UiPath"}'""")
+        return MiddlewareResult(should_continue=False)
     except Exception as e:
-        spinner.stop()
+        console.error(f"Error creating demo agent {str(e)}")
         return MiddlewareResult(
             should_continue=False,
-            error_message=f"❌ Error creating demo agent {str(e)}",
             should_include_stacktrace=True,
         )