From 89a4d22354a703b321e02dcef17cc4a0dafa3f42 Mon Sep 17 00:00:00 2001 From: acamq <179265037+acamq@users.noreply.github.com> Date: Sun, 8 Mar 2026 09:12:13 -0600 Subject: [PATCH] test(think-mode): update tests for variant-only behavior Update test assertions to verify hook only sets output.message.variant and no longer modifies output.message.model. Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-opencode) Co-authored-by: Sisyphus --- src/hooks/think-mode/index.test.ts | 14 ++++---------- 1 file changed, 4 insertions(+), 10 deletions(-) diff --git a/src/hooks/think-mode/index.test.ts b/src/hooks/think-mode/index.test.ts index e135412db..34c1ba55c 100644 --- a/src/hooks/think-mode/index.test.ts +++ b/src/hooks/think-mode/index.test.ts @@ -43,7 +43,7 @@ describe("createThinkModeHook", () => { clearThinkModeState(sessionID) }) - it("sets high variant and switches model when think keyword is present", async () => { + it("sets high variant when think keyword is present", async () => { // given const hook = createThinkModeHook() const input = createHookInput({ @@ -58,13 +58,10 @@ describe("createThinkModeHook", () => { // then expect(output.message.variant).toBe("high") - expect(output.message.model).toEqual({ - providerID: "github-copilot", - modelID: "claude-opus-4-6-high", - }) + expect(output.message.model).toBeUndefined() }) - it("supports dotted model IDs by switching to normalized high variant", async () => { + it("sets high variant for dotted model IDs", async () => { // given const hook = createThinkModeHook() const input = createHookInput({ @@ -79,10 +76,7 @@ describe("createThinkModeHook", () => { // then expect(output.message.variant).toBe("high") - expect(output.message.model).toEqual({ - providerID: "github-copilot", - modelID: "gpt-5-4-high", - }) + expect(output.message.model).toBeUndefined() }) it("skips when message variant is already set", async () => {