Skip to content

Commit 6fc57c0

Browse files
author
Valentina Bojan
committed
feat: address feedback [AL-228, AL-241]
1 parent 714bf0e commit 6fc57c0

4 files changed

Lines changed: 12 additions & 11 deletions

File tree

src/uipath_langchain/agent/guardrails/actions/escalate_action.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@
1616
from ...exceptions import AgentTerminationException
1717
from ...react.types import AgentGuardrailsGraphState
1818
from ..types import ExecutionStage
19-
from ..utils import get_message_text
19+
from ..utils import get_message_content
2020
from .base_action import GuardrailAction, GuardrailActionNode
2121

2222

@@ -378,7 +378,7 @@ def _extract_llm_escalation_content(
378378
if isinstance(last_message, ToolMessage):
379379
return last_message.content
380380

381-
content = get_message_text(last_message)
381+
content = get_message_content(last_message)
382382
return json.dumps(content) if content else ""
383383

384384
# For AI messages, process tool calls if present
@@ -396,14 +396,14 @@ def _extract_llm_escalation_content(
396396
):
397397
content_list.append(json.dumps(args["content"]))
398398

399-
message_content = get_message_text(last_message)
399+
message_content = get_message_content(last_message)
400400
if message_content:
401401
content_list.append(message_content)
402402

403403
return json.dumps(content_list)
404404

405405
# Fallback for other message types
406-
return get_message_text(last_message)
406+
return get_message_content(last_message)
407407

408408

409409
def _extract_agent_escalation_content(

src/uipath_langchain/agent/guardrails/guardrail_nodes.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212
)
1313

1414
from uipath_langchain.agent.guardrails.types import ExecutionStage
15-
from uipath_langchain.agent.guardrails.utils import get_message_text
15+
from uipath_langchain.agent.guardrails.utils import get_message_content
1616
from uipath_langchain.agent.react.types import AgentGuardrailsGraphState
1717

1818
logger = logging.getLogger(__name__)
@@ -64,7 +64,7 @@ def create_llm_guardrail_node(
6464
def _payload_generator(state: AgentGuardrailsGraphState) -> str:
6565
if not state.messages:
6666
return ""
67-
return get_message_text(state.messages[-1])
67+
return get_message_content(state.messages[-1])
6868

6969
return _create_guardrail_node(
7070
guardrail,
@@ -85,7 +85,7 @@ def create_agent_init_guardrail_node(
8585
def _payload_generator(state: AgentGuardrailsGraphState) -> str:
8686
if not state.messages:
8787
return ""
88-
return get_message_text(state.messages[-1])
88+
return get_message_content(state.messages[-1])
8989

9090
return _create_guardrail_node(
9191
guardrail,
@@ -173,7 +173,7 @@ def _payload_generator(state: AgentGuardrailsGraphState) -> str:
173173
if args is not None:
174174
return json.dumps(args)
175175

176-
return get_message_text(state.messages[-1])
176+
return get_message_content(state.messages[-1])
177177

178178
return _create_guardrail_node(
179179
guardrail,
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
from langchain_core.messages import AnyMessage, HumanMessage, SystemMessage
22

33

4-
def get_message_text(msg: AnyMessage) -> str:
4+
def get_message_content(msg: AnyMessage) -> str:
55
if isinstance(msg, (HumanMessage, SystemMessage)):
66
return msg.content if isinstance(msg.content, str) else str(msg.content)
77
return str(getattr(msg, "content", "")) if hasattr(msg, "content") else ""

src/uipath_langchain/agent/react/agent.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,8 @@
1414
from .guardrails.guardrails_subgraph import (
1515
create_agent_init_guardrails_subgraph,
1616
create_agent_terminate_guardrails_subgraph,
17-
create_tools_guardrails_subgraph, create_llm_guardrails_subgraph,
17+
create_llm_guardrails_subgraph,
18+
create_tools_guardrails_subgraph,
1819
)
1920
from .init_node import (
2021
create_init_node,
@@ -99,7 +100,7 @@ def create_agent(
99100
builder.add_node(tool_name, tool_node)
100101

101102
terminate_with_guardrails_subgraph = create_agent_terminate_guardrails_subgraph(
102-
(AgentGraphNode.TERMINATE, terminate_node),
103+
(AgentGraphNode.GUARDED_TERMINATE, terminate_node),
103104
guardrails,
104105
)
105106
builder.add_node(AgentGraphNode.TERMINATE, terminate_with_guardrails_subgraph)

0 commit comments

Comments
 (0)