From 07850da79a9abc62323e1440715d738e0db10a8e Mon Sep 17 00:00:00 2001 From: Lex Christopherson Date: Thu, 26 Mar 2026 09:29:47 -0600 Subject: [PATCH] fix: update triage-dispatch static analysis tests for enqueueSidecar helper The static analysis tests check source code for `return "continue"` patterns. After extracting enqueueSidecar(), the return is now via the helper call. Accept both patterns in the assertion. Co-Authored-By: Claude Opus 4.6 (1M context) --- src/resources/extensions/gsd/tests/triage-dispatch.test.ts | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/resources/extensions/gsd/tests/triage-dispatch.test.ts b/src/resources/extensions/gsd/tests/triage-dispatch.test.ts index 34fd149de..025f99677 100644 --- a/src/resources/extensions/gsd/tests/triage-dispatch.test.ts +++ b/src/resources/extensions/gsd/tests/triage-dispatch.test.ts @@ -119,7 +119,7 @@ test("dispatch: triage dispatch keeps the loop in continue mode", () => { postUnitSrc.indexOf("// ── Quick-task dispatch"), ); assert.ok( - triageBlock.includes('return "continue"'), + triageBlock.includes('return "continue"') || triageBlock.includes("return enqueueSidecar("), "triage dispatch should return 'continue' after enqueuing sidecar work", ); }); @@ -320,7 +320,7 @@ test("dispatch: quick-task dispatch keeps the loop in continue mode", () => { postUnitSrc.indexOf("if (s.stepMode)"), ); assert.ok( - quickTaskSection.includes('return "continue"'), + quickTaskSection.includes('return "continue"') || quickTaskSection.includes("return enqueueSidecar("), "quick-task dispatch should return 'continue' after enqueuing sidecar work", ); });