fix: isPlanFamily regression tests, code-review -> review-work in test data (#3312, #3285, #3290)

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", () => {

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)