fix(ai): use valid synthetic Responses message ids closes #5148
This commit is contained in:
@@ -178,7 +178,7 @@ export function convertResponsesMessages<TApi extends Api>(
|
|||||||
const textBlock = block as TextContent;
|
const textBlock = block as TextContent;
|
||||||
const parsedSignature = parseTextSignature(textBlock.textSignature);
|
const parsedSignature = parseTextSignature(textBlock.textSignature);
|
||||||
const fallbackMessageId =
|
const fallbackMessageId =
|
||||||
textBlockIndex === 0 ? `pi_msg_${msgIndex}` : `pi_msg_${msgIndex}_${textBlockIndex}`;
|
textBlockIndex === 0 ? `msg_pi_${msgIndex}` : `msg_pi_${msgIndex}_${textBlockIndex}`;
|
||||||
textBlockIndex++;
|
textBlockIndex++;
|
||||||
// OpenAI requires id to be max 64 characters
|
// OpenAI requires id to be max 64 characters
|
||||||
let msgId = parsedSignature?.id;
|
let msgId = parsedSignature?.id;
|
||||||
|
|||||||
@@ -42,7 +42,7 @@ describe("OpenAI Responses message ID conversion", () => {
|
|||||||
)
|
)
|
||||||
.map((item) => item.id);
|
.map((item) => item.id);
|
||||||
|
|
||||||
expect(messageIds).toEqual(["pi_msg_1", "pi_msg_1_1"]);
|
expect(messageIds).toEqual(["msg_pi_1", "msg_pi_1_1"]);
|
||||||
expect(new Set(messageIds).size).toBe(messageIds.length);
|
expect(new Set(messageIds).size).toBe(messageIds.length);
|
||||||
});
|
});
|
||||||
});
|
});
|
||||||
|
|||||||
Reference in New Issue
Block a user