Skip to content

Commit 92718d5

Browse files
committed
Comment why summary_fn is popped from node opts
1 parent 5d4fb4c commit 92718d5

1 file changed

Lines changed: 2 additions & 0 deletions

File tree

temporalio/contrib/langgraph/_plugin.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -276,6 +276,8 @@ def execute(
276276
"""Prepare a node or task to execute as an activity or inline in the workflow."""
277277
opts = kwargs or {}
278278
execute_in = opts.pop("execute_in")
279+
# Remove control keys before opts is splatted into execute_activity
280+
# below; summary_fn is consumed here, not a Temporal activity option.
279281
node_summary_fn = opts.pop("summary_fn", None)
280282
if node_summary_fn is not None and opts.get("summary") is not None:
281283
raise ValueError(

0 commit comments

Comments
 (0)