Merge pull request #68 from Garsson-io/worktree-260317-1224-7026

fix: route dev case notifications to main group

Author: aviadr1

src/ipc-github-issues.test.ts

@@ -356,6 +356,57 @@ describe('case_create auto-creates GitHub issue for dev cases', () => {
     );
   });
 
+  // INVARIANT: Dev case notifications go to main group, not source group
+  // SUT: processTaskIpc case_create notification routing
+  test('dev case created from non-main group notifies main group', async () => {
+    mockedCreateGitHubIssue.mockResolvedValue({
+      success: true,
+      issueUrl: 'https://github.com/Garsson-io/kaizen/issues/60',
+      issueNumber: 60,
+    });
+
+    await processTaskIpc(
+      {
+        type: 'case_create',
+        description: 'Fix widget rendering',
+        caseType: 'dev',
+        chatJid: 'tg:222',
+        requestId: 'req-dev-routing',
+      } as any,
+      'telegram_work',
+      false,
+      deps,
+    );
+
+    // Notification should go to main group (tg:111), not source group (tg:222)
+    expect(sendMessage).toHaveBeenCalledWith(
+      'tg:111',
+      expect.stringContaining('dev case created'),
+    );
+  });
+
+  // INVARIANT: Work case notifications go to source group, not main group
+  test('work case created from non-main group notifies source group', async () => {
+    await processTaskIpc(
+      {
+        type: 'case_create',
+        description: 'Process client order',
+        caseType: 'work',
+        chatJid: 'tg:222',
+        requestId: 'req-work-routing',
+      } as any,
+      'telegram_work',
+      false,
+      deps,
+    );
+
+    // Notification should go to source group (tg:222), not main
+    expect(sendMessage).toHaveBeenCalledWith(
+      'tg:222',
+      expect.stringContaining('work case created'),
+    );
+  });
+
   test('work case does NOT create GitHub issue', async () => {
     await processTaskIpc(
       {

src/ipc.ts

@@ -1080,12 +1080,17 @@ export async function processTaskIpc(
         }),
       );
 
-      // Notify user
-      if (resolvedChatJid) {
+      // Notify user — dev cases notify main group, work cases notify source group
+      const notifyJid =
+        caseType === 'dev'
+          ? Object.entries(registeredGroups).find(([, g]) => g.isMain)?.[0] ||
+            resolvedChatJid
+          : resolvedChatJid;
+      if (notifyJid) {
         const issueInfo = issueUrl ? `\nGitHub: ${issueUrl}` : '';
         deps
           .sendMessage(
-            resolvedChatJid,
+            notifyJid,
             `📋 New ${caseType} case created: ${name}\n${d.description.slice(0, 200)}${issueInfo}`,
           )
           .catch(() => {
@@ -1115,10 +1120,12 @@ export async function processTaskIpc(
           initiatorChannel: undefined,
           githubIssue: d.githubIssue,
         });
-        // Notify user about the suggestion
-        const targetJid = Object.entries(registeredGroups).find(
-          ([, g]) => g.folder === sourceGroup,
-        )?.[0];
+        // Notify main group about the dev suggestion
+        const targetJid =
+          Object.entries(registeredGroups).find(([, g]) => g.isMain)?.[0] ||
+          Object.entries(registeredGroups).find(
+            ([, g]) => g.folder === sourceGroup,
+          )?.[0];
         if (targetJid) {
           deps
             .sendMessage(