Skip to content

Commit bbc6549

Browse files
committed
fix: address reviewer import and streaming feedback
1 parent cf347f9 commit bbc6549

3 files changed

Lines changed: 25 additions & 16 deletions

File tree

src/agent/profiles/react_to_me.py

Lines changed: 18 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -25,9 +25,7 @@ def __init__(
2525
super().__init__(llm, embedding)
2626

2727
# Create runnables (tasks & tools)
28-
streaming_llm = llm.model_copy(update={"streaming": True})
29-
30-
self.unsafe_answer_generator = create_unsafe_answer_generator(streaming_llm)
28+
self.unsafe_answer_generator = create_unsafe_answer_generator(llm)
3129
self.reactome_rag: Runnable = create_reactome_rag(
3230
llm, embedding, streaming=True
3331
)
@@ -75,21 +73,25 @@ async def generate_unsafe_response(
7573
else str(final_answer_message)
7674
)
7775

78-
updated_state = dict(state)
79-
updated_state.update(
80-
chat_history=[
76+
history = list(state.get("chat_history", []))
77+
history.extend(
78+
[
8179
HumanMessage(state["user_input"]),
8280
(
8381
final_answer_message
8482
if hasattr(final_answer_message, "content")
8583
else AIMessage(final_answer)
8684
),
87-
],
85+
]
86+
)
87+
88+
return ReactToMeState(
89+
**state,
90+
chat_history=history,
8891
answer=final_answer,
8992
safety=SAFETY_UNSAFE,
9093
additional_content={"search_results": []},
9194
)
92-
return ReactToMeState(**updated_state)
9395

9496
async def call_model(
9597
self, state: ReactToMeState, config: RunnableConfig
@@ -106,15 +108,18 @@ async def call_model(
106108
},
107109
config,
108110
)
109-
updated_state = dict(state)
110-
updated_state.update(
111-
chat_history=[
111+
history = list(state.get("chat_history", []))
112+
history.extend(
113+
[
112114
HumanMessage(state["user_input"]),
113115
AIMessage(result["answer"]),
114-
],
116+
]
117+
)
118+
return ReactToMeState(
119+
**state,
120+
chat_history=history,
115121
answer=result["answer"],
116122
)
117-
return ReactToMeState(**updated_state)
118123

119124

120125
def create_reactome_graph(

src/agent/tasks/unsafe_answer.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,8 @@ def create_unsafe_answer_generator(llm: BaseChatModel) -> Runnable:
3333
3434
You must not provide any workaround, implicit answer, or redirection toward unsafe content.
3535
"""
36+
streaming_llm = llm.model_copy(update={"streaming": True})
37+
3638
prompt = ChatPromptTemplate.from_messages(
3739
[
3840
("system", system_prompt),
@@ -43,4 +45,4 @@ def create_unsafe_answer_generator(llm: BaseChatModel) -> Runnable:
4345
]
4446
)
4547

46-
return prompt | llm
48+
return prompt | streaming_llm

src/tools/preprocessing/__init__.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,5 +2,7 @@
22
Preprocessing utilities with reusable workflows and state definitions.
33
"""
44

5-
from .state import PreprocessingState # noqa: F401
6-
from .workflow import create_preprocessing_workflow # noqa: F401
5+
from tools.preprocessing.state import PreprocessingState
6+
from tools.preprocessing.workflow import create_preprocessing_workflow
7+
8+
__all__ = ["PreprocessingState", "create_preprocessing_workflow"]

0 commit comments

Comments
 (0)