fix: trim repeated slack thread context payloads (#32133) (thanks @sourman)

This commit is contained in:
Peter Steinberger
2026-03-02 21:29:21 +00:00
parent 7a99027ef6
commit 2438fde6d9
2 changed files with 3 additions and 0 deletions

View File

@@ -510,6 +510,7 @@ describe("slack prepareSlackMessage inbound contract", () => {
expect(prepared).toBeTruthy();
expect(prepared!.ctxPayload.IsFirstThreadTurn).toBe(true);
expect(prepared!.ctxPayload.ThreadStarterBody).toBe("starter");
expect(prepared!.ctxPayload.ThreadHistoryBody).toContain("assistant reply");
expect(prepared!.ctxPayload.ThreadHistoryBody).toContain("follow-up question");
expect(prepared!.ctxPayload.ThreadHistoryBody).not.toContain("current message");
@@ -558,6 +559,7 @@ describe("slack prepareSlackMessage inbound contract", () => {
expect(prepared!.ctxPayload.ThreadHistoryBody).toBeUndefined();
// Thread starter should also be skipped for existing sessions
expect(prepared!.ctxPayload.ThreadStarterBody).toBeUndefined();
expect(prepared!.ctxPayload.ThreadLabel).toContain("Slack thread");
// Replies API should only be called once (for thread starter lookup, not history)
expect(replies).toHaveBeenCalledTimes(1);
});