Limit session reset to assignment wakes only

This commit is contained in:
Dotta
2026-03-05 07:39:30 -06:00
parent 758a5538c5
commit 67eb5e5734
2 changed files with 5 additions and 9 deletions

View File

@@ -93,22 +93,22 @@ describe("shouldResetTaskSessionForWake", () => {
expect(shouldResetTaskSessionForWake({ wakeReason: "issue_assigned" })).toBe(true); expect(shouldResetTaskSessionForWake({ wakeReason: "issue_assigned" })).toBe(true);
}); });
it("resets session context on mention wake comment", () => { it("does not reset session context on mention wake comment", () => {
expect( expect(
shouldResetTaskSessionForWake({ shouldResetTaskSessionForWake({
wakeReason: "issue_comment_mentioned", wakeReason: "issue_comment_mentioned",
wakeCommentId: "comment-1", wakeCommentId: "comment-1",
}), }),
).toBe(true); ).toBe(false);
}); });
it("resets session context when commentId is present", () => { it("does not reset session context when commentId is present", () => {
expect( expect(
shouldResetTaskSessionForWake({ shouldResetTaskSessionForWake({
wakeReason: "issue_commented", wakeReason: "issue_commented",
commentId: "comment-2", commentId: "comment-2",
}), }),
).toBe(true); ).toBe(false);
}); });
it("does not reset for comment wakes", () => { it("does not reset for comment wakes", () => {

View File

@@ -199,11 +199,7 @@ export function shouldResetTaskSessionForWake(
contextSnapshot: Record<string, unknown> | null | undefined, contextSnapshot: Record<string, unknown> | null | undefined,
) { ) {
const wakeReason = readNonEmptyString(contextSnapshot?.wakeReason); const wakeReason = readNonEmptyString(contextSnapshot?.wakeReason);
if (wakeReason === "issue_assigned") return true; return wakeReason === "issue_assigned";
const wakeCommentId =
readNonEmptyString(contextSnapshot?.wakeCommentId) ??
readNonEmptyString(contextSnapshot?.commentId);
return wakeCommentId != null;
} }
function deriveCommentId( function deriveCommentId(