Skip to content

refactor: use START/END from langgraph.graph instead of magic strings #34

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
xbsheng wants to merge 1 commit into
base: main
Choose a base branch
from
+5 −5
Open

refactor: use START/END from langgraph.graph instead of magic strings #34

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 5 additions & 5 deletions src/react_agent/graph.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
from typing import Dict, List, Literal, cast

from langchain_core.messages import AIMessage
from langgraph.graph import StateGraph
from langgraph.graph import END, START, StateGraph
from langgraph.prebuilt import ToolNode
from langgraph.runtime import Runtime

Expand Down Expand Up @@ -42,7 +42,7 @@ async def call_model(
)

# Get the model's response
response = cast( # type: ignore[redundant-cast]
response = cast( # type: ignore[redundant-cast]
AIMessage,
await model.ainvoke(
[{"role": "system", "content": system_message}, *state.messages]
Expand Down Expand Up @@ -74,7 +74,7 @@ async def call_model(

# Set the entrypoint as `call_model`
# This means that this node is the first one called
builder.add_edge("__start__", "call_model")
builder.add_edge(START, "call_model")


def route_model_output(state: State) -> Literal["__end__", "tools"]:
Expand All @@ -86,7 +86,7 @@ def route_model_output(state: State) -> Literal["__end__", "tools"]:
state (State): The current state of the conversation.

Returns:
str: The name of the next node to call ("__end__" or "tools").
str: The name of the next node to call (END or "tools").
"""
last_message = state.messages[-1]
if not isinstance(last_message, AIMessage):
Expand All @@ -95,7 +95,7 @@ def route_model_output(state: State) -> Literal["__end__", "tools"]:
)
# If there is no tool call, then we finish
if not last_message.tool_calls:
return "__end__"
return END
# Otherwise we execute the requested actions
return "tools"

Expand Down