From e1b2f97bfdb0ad97af5399ae46311d2acd1e12de Mon Sep 17 00:00:00 2001
From: YeonGyu-Kim <code.yeon.gyu@gmail.com>
Date: Wed, 15 Apr 2026 11:27:07 +0900
Subject: [PATCH] fix: isPlanFamily regression tests, code-review ->
 review-work in test data (#3312, #3285, #3290)

---
 .../delegate-task/prompt-builder.test.ts      |  4 ++--
 src/tools/delegate-task/tools.test.ts         | 21 +++++++++++++++++++
 2 files changed, 23 insertions(+), 2 deletions(-)

diff --git a/src/tools/delegate-task/prompt-builder.test.ts b/src/tools/delegate-task/prompt-builder.test.ts
index 9c31fdefc..b728e9d70 100644
--- a/src/tools/delegate-task/prompt-builder.test.ts
+++ b/src/tools/delegate-task/prompt-builder.test.ts
@@ -46,7 +46,7 @@ describe("prompt-builder", () => {
       test("#when agent is explore #then system content includes available_skills section", () => {
         // given
         const availableSkills: AvailableSkill[] = [
-          { name: "code-review", description: "Review code quality", location: "project" },
+          { name: "review-work", description: "Review code quality", location: "project" },
         ]
 
         // when
@@ -57,7 +57,7 @@ describe("prompt-builder", () => {
 
         // then
         expect(result).toBeDefined()
-        expect(result).toContain("code-review")
+        expect(result).toContain("review-work")
       })
 
       test("#when availableSkills is empty #then system content does not include available_skills section", () => {
diff --git a/src/tools/delegate-task/tools.test.ts b/src/tools/delegate-task/tools.test.ts
index 650eb7670..5353ed6d5 100644
--- a/src/tools/delegate-task/tools.test.ts
+++ b/src/tools/delegate-task/tools.test.ts
@@ -289,6 +289,27 @@ describe("sisyphus-task", () => {
       expect(result).toBe(false)
     })
 
+    test("returns false for 'Momus (Plan Critic)' - no substring false-positive", () => {
+      //#given / #when
+      const result = isPlanFamily("Momus (Plan Critic)")
+      //#then
+      expect(result).toBe(false)
+    })
+
+    test("returns false for 'Momus - Plan Critic' - no substring false-positive", () => {
+      //#given / #when
+      const result = isPlanFamily("Momus - Plan Critic")
+      //#then
+      expect(result).toBe(false)
+    })
+
+    test("returns false for 'Metis - Plan Consultant' - no substring false-positive", () => {
+      //#given / #when
+      const result = isPlanFamily("Metis - Plan Consultant")
+      //#then
+      expect(result).toBe(false)
+    })
+
     test("returns false for undefined", () => {
       //#given / #when
       const result = isPlanFamily(undefined)