feat: Added tutorial and warnings for langchain orchestration template - #572
Open
BrigittaK307 wants to merge 20 commits into
Open
feat: Added tutorial and warnings for langchain orchestration template#572BrigittaK307 wants to merge 20 commits into
BrigittaK307 wants to merge 20 commits into
Conversation
Comment on lines
+2
to
+4
| id: langgraph-orchestration-client | ||
| title: Using Orchestration Client in LangGraph Workflows | ||
| sidebar_label: Orchestration Client in LangGraph |
Member
There was a problem hiding this comment.
Suggested change
| id: langgraph-orchestration-client | |
| title: Using Orchestration Client in LangGraph Workflows | |
| sidebar_label: Orchestration Client in LangGraph | |
| id: langgraph-template-orchestration-client | |
| title: Using Orchestration Templates in LangGraph Agents | |
| sidebar_label: Using Orchestration Templates in LangGraph Agents |
| } | ||
| }); | ||
| ``` | ||
|
|
Member
There was a problem hiding this comment.
[pp] As a tutorial, I would prefer if there was something e.g. at the bottom for both that shows how to combine these into an actual LangGraph (or maybe createAgent()...)
Comment on lines
+235
to
+240
| .addConditionalEdges(START, routeByTurn, [ | ||
| 'firstTurnNode', | ||
| 'conversationNode' | ||
| ]) | ||
| .addEdge('firstTurnNode', END) | ||
| .addEdge('conversationNode', END) |
Member
There was a problem hiding this comment.
[pp] I think there should be edge from START -> firstTurnNode -> conversationNode with a conditional edge from conversationNode to itself or END (in this case perhaps just END).
Contributor
Author
There was a problem hiding this comment.
The graph re-enters from START on every graph.invoke() call. Adapting the firstTurnNode → conversationNode case would cause both nodes to fire on turn 1, making two LLM calls instead of one.
| When initializing the client with an orchestration configuration reference, any messages passed to the `chatCompletion()` or `stream()` methods are automatically appended to the `messages_history` array as they cannot be merged into the stored configuration's prompt template. | ||
| The `messages` property defined in the `template` array are appended after any existing `messagesHistory` entries. | ||
| :::warning | ||
| Messages passed alongside a configuration reference are **automatically routed to `messages_history`** and cannot be merged into the remotely stored configuration's prompt template. See the [LangGraph tutorial](../tutorials/langgraph-template-orchestration-client) for the recommended two-client pattern in multi-turn workflows. |
Member
There was a problem hiding this comment.
Suggested change
| Messages passed alongside a configuration reference are **automatically routed to `messages_history`** and cannot be merged into the remotely stored configuration's prompt template. See the [LangGraph tutorial](../tutorials/langgraph-template-orchestration-client) for the recommended two-client pattern in multi-turn workflows. | |
| Messages passed alongside a configuration reference are **automatically routed to `messages_history`** and cannot be merged into the remotely stored configuration's prompt template. See the [LangGraph tutorial](../tutorials/langgraph-template-orchestration-client) for the recommended two-client pattern in multi-turn workflows with configuration references. |
| const agent = createAgent({ | ||
| model: clientWithTemplate, | ||
| tools: [], | ||
| checkpointSaver: new MemorySaver() |
Member
There was a problem hiding this comment.
Suggested change
| checkpointSaver: new MemorySaver() | |
| checkpointer: new MemorySaver() |
davidkna-sap
reviewed
Sep 1, 2026
Member
There was a problem hiding this comment.
To ease review please enable twoslash on all code blocks via ```ts twoslash
davidkna-sap
reviewed
Sep 2, 2026
davidkna-sap
approved these changes
Sep 3, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Related to https://github.com/orgs/SAP/projects/111/views/1?pane=issue&itemId=141534346&issue=SAP%7Cai-sdk-js-backlog%7C438