Skip to content

Commit

Permalink
fix(core): Fix coercion of serialized tool and function messages (#7653)
Browse files Browse the repository at this point in the history
  • Loading branch information
jacoblee93 authored Feb 5, 2025
1 parent 1f11644 commit 1efb550
Show file tree
Hide file tree
Showing 2 changed files with 47 additions and 0 deletions.
37 changes: 37 additions & 0 deletions langchain-core/src/messages/tests/base_message.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -404,6 +404,43 @@ describe("Message like coercion", () => {
}),
]);
});
it("should convert serialized messages", async () => {
const originalMessages = [
new SystemMessage({
id: "foobar",
content: "6",
additional_kwargs: {},
}),
new SystemMessage({
content: "6.1",
additional_kwargs: {
__openai_role__: "developer",
},
}),
new HumanMessage({
content: [{ type: "image_url", image_url: { url: "7.1" } }],
}),
new AIMessage({
content: [{ type: "text", text: "8.1" }],
tool_calls: [
{
id: "8.5",
name: "8.4",
args: { "8.2": "8.3" },
type: "tool_call",
},
],
}),
new ToolMessage({
name: undefined,
content: "10.2",
tool_call_id: "10.2",
}),
];
const serialized = JSON.parse(JSON.stringify(originalMessages));
const deserialized = serialized.map(coerceMessageLikeToMessage);
expect(deserialized).toEqual(originalMessages);
});
});

describe("usage_metadata serialized", () => {
Expand Down
10 changes: 10 additions & 0 deletions langchain-core/src/messages/utils.ts
Original file line number Diff line number Diff line change
Expand Up @@ -87,6 +87,16 @@ function _constructMessageFromParams(
className === "SystemMessageChunk"
) {
type = "system";
} else if (
className === "FunctionMessage" ||
className === "FunctionMessageChunk"
) {
type = "function";
} else if (
className === "ToolMessage" ||
className === "ToolMessageChunk"
) {
type = "tool";
} else {
type = "unknown";
}
Expand Down

0 comments on commit 1efb550

Please sign in to comment.