tmux-team 4.0.0 → 4.2.0

package/src/commands/talk.test.ts

@@ -43,6 +43,25 @@ function createMockTmux(): Tmux & {
     getCurrentPaneId() {
       return null;
     },
+    resolvePaneTarget(target: string) {
+      return target;
+    },
+    getAgentRegistry() {
+      return { paneRegistry: {}, agents: {} };
+    },
+    setAgentRegistration() {},
+    clearAgentRegistration() {
+      return false;
+    },
+    listTeams() {
+      return {};
+    },
+    listTeamPanes() {
+      return [];
+    },
+    removeTeam() {
+      return { removed: 0, agents: [] };
+    },
   };
   return mock;
 }
@@ -239,7 +258,8 @@ describe('buildMessage (via cmdTalk)', () => {
         pollInterval: 0.1,
         captureLines: 100,
         maxCaptureLines: 2000,
-        preambleEvery: 3, pasteEnterDelayMs: 500,
+        preambleEvery: 3,
+        pasteEnterDelayMs: 500,
       },
     };
 
@@ -448,6 +468,27 @@ describe('cmdTalk - basic send', () => {
     expect(ui.errors[0]).toContain("Agent 'unknown' not found");
   });
 
+  it('requires explicit team when a missing agent appears in shared teams', async () => {
+    const tmux = createMockTmux();
+    vi.spyOn(tmux, 'listTeams').mockReturnValue({
+      frontend: ['codex'],
+      release: ['codex'],
+    });
+    const ui = createMockUI();
+    const ctx = createContext({
+      tmux,
+      ui,
+      paths: createTestPaths(testDir),
+      config: { paneRegistry: {} },
+    });
+
+    await expect(cmdTalk(ctx, 'codex', 'Hello')).rejects.toThrow('exit(3)');
+
+    expect(ui.errors).toEqual([
+      "Agent 'codex' is in multiple shared teams: frontend, release. Specify one with --team <team>.",
+    ]);
+  });
+
   it('outputs JSON when --json flag is set', async () => {
     const tmux = createMockTmux();
     const ui = createMockUI();
@@ -551,7 +592,8 @@ describe('cmdTalk - --wait mode', () => {
           pollInterval: 0.01,
           captureLines: 100,
           maxCaptureLines: 2000,
-          preambleEvery: 3, pasteEnterDelayMs: 500,
+          preambleEvery: 3,
+          pasteEnterDelayMs: 500,
         },
       },
     });
@@ -595,7 +637,8 @@ describe('cmdTalk - --wait mode', () => {
           pollInterval: 0.01,
           captureLines: 100,
           maxCaptureLines: 2000,
-          preambleEvery: 3, pasteEnterDelayMs: 500,
+          preambleEvery: 3,
+          pasteEnterDelayMs: 500,
         },
       },
     });
@@ -626,7 +669,8 @@ describe('cmdTalk - --wait mode', () => {
           pollInterval: 0.02,
           captureLines: 100,
           maxCaptureLines: 2000,
-          preambleEvery: 3, pasteEnterDelayMs: 500,
+          preambleEvery: 3,
+          pasteEnterDelayMs: 500,
         },
       },
     });
@@ -674,7 +718,8 @@ describe('cmdTalk - --wait mode', () => {
           pollInterval: 0.01,
           captureLines: 100,
           maxCaptureLines: 2000,
-          preambleEvery: 3, pasteEnterDelayMs: 500,
+          preambleEvery: 3,
+          pasteEnterDelayMs: 500,
         },
       },
     });
@@ -710,7 +755,8 @@ describe('cmdTalk - --wait mode', () => {
           pollInterval: 0.01,
           captureLines: 100,
           maxCaptureLines: 2000,
-          preambleEvery: 3, pasteEnterDelayMs: 500,
+          preambleEvery: 3,
+          pasteEnterDelayMs: 500,
         },
       },
     });
@@ -739,7 +785,8 @@ describe('cmdTalk - --wait mode', () => {
           pollInterval: 0.01,
           captureLines: 100,
           maxCaptureLines: 2000,
-          preambleEvery: 3, pasteEnterDelayMs: 500,
+          preambleEvery: 3,
+          pasteEnterDelayMs: 500,
         },
       },
     });
@@ -791,7 +838,8 @@ describe('cmdTalk - --wait mode', () => {
           pollInterval: 0.05,
           captureLines: 100,
           maxCaptureLines: 2000,
-          preambleEvery: 3, pasteEnterDelayMs: 500,
+          preambleEvery: 3,
+          pasteEnterDelayMs: 500,
         },
         paneRegistry: {
           codex: { pane: '10.1' },
@@ -837,7 +885,8 @@ describe('cmdTalk - --wait mode', () => {
           pollInterval: 0.02,
           captureLines: 100,
           maxCaptureLines: 2000,
-          preambleEvery: 3, pasteEnterDelayMs: 500,
+          preambleEvery: 3,
+          pasteEnterDelayMs: 500,
         },
         paneRegistry: {
           codex: { pane: '10.1' },
@@ -892,7 +941,8 @@ describe('cmdTalk - --wait mode', () => {
           pollInterval: 0.02,
           captureLines: 100,
           maxCaptureLines: 2000,
-          preambleEvery: 3, pasteEnterDelayMs: 500,
+          preambleEvery: 3,
+          pasteEnterDelayMs: 500,
         },
         paneRegistry: {
           codex: { pane: '10.1' },
@@ -1003,7 +1053,8 @@ describe('cmdTalk - nonce collision handling', () => {
           pollInterval: 0.01,
           captureLines: 100,
           maxCaptureLines: 2000,
-          preambleEvery: 3, pasteEnterDelayMs: 500,
+          preambleEvery: 3,
+          pasteEnterDelayMs: 500,
         },
       },
     });
@@ -1058,7 +1109,8 @@ describe('cmdTalk - JSON output contract', () => {
           pollInterval: 0.01,
           captureLines: 100,
           maxCaptureLines: 2000,
-          preambleEvery: 3, pasteEnterDelayMs: 500,
+          preambleEvery: 3,
+          pasteEnterDelayMs: 500,
         },
       },
     });
@@ -1099,7 +1151,8 @@ describe('cmdTalk - JSON output contract', () => {
           pollInterval: 0.01,
           captureLines: 100,
           maxCaptureLines: 2000,
-          preambleEvery: 3, pasteEnterDelayMs: 500,
+          preambleEvery: 3,
+          pasteEnterDelayMs: 500,
         },
       },
     });
@@ -1141,7 +1194,8 @@ describe('cmdTalk - JSON output contract', () => {
           pollInterval: 0.01,
           captureLines: 100,
           maxCaptureLines: 2000,
-          preambleEvery: 3, pasteEnterDelayMs: 500,
+          preambleEvery: 3,
+          pasteEnterDelayMs: 500,
         },
       },
     });
@@ -1178,7 +1232,8 @@ describe('cmdTalk - JSON output contract', () => {
           pollInterval: 0.01,
           captureLines: 100,
           maxCaptureLines: 2000,
-          preambleEvery: 3, pasteEnterDelayMs: 500,
+          preambleEvery: 3,
+          pasteEnterDelayMs: 500,
         },
       },
     });
@@ -1228,7 +1283,8 @@ describe('cmdTalk - JSON output contract', () => {
           pollInterval: 0.02,
           captureLines: 100,
           maxCaptureLines: 2000,
-          preambleEvery: 3, pasteEnterDelayMs: 500,
+          preambleEvery: 3,
+          pasteEnterDelayMs: 500,
         },
         paneRegistry: {
           codex: { pane: '10.1' },
@@ -1310,7 +1366,16 @@ describe('cmdTalk - end marker detection', () => {
       ui,
       paths: createTestPaths(testDir),
       flags: { wait: true, json: true, timeout: 0.5 },
-      config: { defaults: { timeout: 0.5, pollInterval: 0.01, captureLines: 100, maxCaptureLines: 2000, preambleEvery: 3, pasteEnterDelayMs: 500 } },
+      config: {
+        defaults: {
+          timeout: 0.5,
+          pollInterval: 0.01,
+          captureLines: 100,
+          maxCaptureLines: 2000,
+          preambleEvery: 3,
+          pasteEnterDelayMs: 500,
+        },
+      },
     });
 
     await cmdTalk(ctx, 'claude', 'Test message');
@@ -1344,7 +1409,16 @@ describe('cmdTalk - end marker detection', () => {
       ui,
       paths: createTestPaths(testDir),
       flags: { wait: true, json: true, timeout: 0.5 },
-      config: { defaults: { timeout: 0.5, pollInterval: 0.01, captureLines: 100, maxCaptureLines: 2000, preambleEvery: 3, pasteEnterDelayMs: 500 } },
+      config: {
+        defaults: {
+          timeout: 0.5,
+          pollInterval: 0.01,
+          captureLines: 100,
+          maxCaptureLines: 2000,
+          preambleEvery: 3,
+          pasteEnterDelayMs: 500,
+        },
+      },
     });
 
     await cmdTalk(ctx, 'claude', 'Test');
@@ -1378,7 +1452,16 @@ Line 4 final`;
       ui,
       paths: createTestPaths(testDir),
       flags: { wait: true, json: true, timeout: 0.5 },
-      config: { defaults: { timeout: 0.5, pollInterval: 0.01, captureLines: 100, maxCaptureLines: 2000, preambleEvery: 3, pasteEnterDelayMs: 500 } },
+      config: {
+        defaults: {
+          timeout: 0.5,
+          pollInterval: 0.01,
+          captureLines: 100,
+          maxCaptureLines: 2000,
+          preambleEvery: 3,
+          pasteEnterDelayMs: 500,
+        },
+      },
     });
 
     await cmdTalk(ctx, 'claude', 'Test');
@@ -1409,7 +1492,16 @@ Line 4 final`;
       ui,
       paths: createTestPaths(testDir),
       flags: { wait: true, json: true, timeout: 0.5 },
-      config: { defaults: { timeout: 0.5, pollInterval: 0.01, captureLines: 100, maxCaptureLines: 2000, preambleEvery: 3, pasteEnterDelayMs: 500 } },
+      config: {
+        defaults: {
+          timeout: 0.5,
+          pollInterval: 0.01,
+          captureLines: 100,
+          maxCaptureLines: 2000,
+          preambleEvery: 3,
+          pasteEnterDelayMs: 500,
+        },
+      },
     });
 
     await cmdTalk(ctx, 'claude', 'Test');
@@ -1446,7 +1538,16 @@ Line 4 final`;
       ui,
       paths: createTestPaths(testDir),
       flags: { wait: true, json: true, timeout: 0.5 },
-      config: { defaults: { timeout: 0.5, pollInterval: 0.01, captureLines: 100, maxCaptureLines: 2000, preambleEvery: 3, pasteEnterDelayMs: 500 } },
+      config: {
+        defaults: {
+          timeout: 0.5,
+          pollInterval: 0.01,
+          captureLines: 100,
+          maxCaptureLines: 2000,
+          preambleEvery: 3,
+          pasteEnterDelayMs: 500,
+        },
+      },
     });
 
     await cmdTalk(ctx, 'claude', 'Test');
@@ -1482,7 +1583,16 @@ Line 4 final`;
       ui,
       paths: createTestPaths(testDir),
       flags: { wait: true, json: true, timeout: 0.5 },
-      config: { defaults: { timeout: 0.5, pollInterval: 0.01, captureLines: 200, maxCaptureLines: 2000, preambleEvery: 3, pasteEnterDelayMs: 500 } },
+      config: {
+        defaults: {
+          timeout: 0.5,
+          pollInterval: 0.01,
+          captureLines: 200,
+          maxCaptureLines: 2000,
+          preambleEvery: 3,
+          pasteEnterDelayMs: 500,
+        },
+      },
     });
 
     await cmdTalk(ctx, 'claude', 'Test');

package/src/commands/talk.ts

@@ -186,7 +186,10 @@ function extractWithExpandableCapture(
 
     if (instructionLineIndex !== -1 && instructionLineIndex < endMarkerLineIndex) {
       // Instruction visible: extract from after instruction to marker
-      const response = lines.slice(instructionLineIndex + 1, endMarkerLineIndex).join('\n').trim();
+      const response = lines
+        .slice(instructionLineIndex + 1, endMarkerLineIndex)
+        .join('\n')
+        .trim();
       return { response, truncated: false };
     }
 
@@ -316,6 +319,21 @@ function buildMessage(message: string, agentName: string, ctx: Context): string
   return `[SYSTEM: ${preamble}]\n\n${message}`;
 }
 
+function teamHintForMissingAgent(ctx: Context, target: string): string | null {
+  if (ctx.flags.team) return null;
+
+  const matches = Object.entries(ctx.tmux.listTeams())
+    .filter(([, agents]) => agents.includes(target))
+    .map(([teamName]) => teamName)
+    .sort();
+
+  if (matches.length === 0) return null;
+  if (matches.length === 1) {
+    return `Agent '${target}' is in shared team '${matches[0]}'. Specify it: tmt talk ${target} "..." --team ${matches[0]}`;
+  }
+  return `Agent '${target}' is in multiple shared teams: ${matches.join(', ')}. Specify one with --team <team>.`;
+}
+
 export async function cmdTalk(ctx: Context, target: string, message: string): Promise<void> {
   const { ui, config, tmux, flags, exit } = ctx;
   const waitEnabled = Boolean(flags.wait) || config.mode === 'wait';
@@ -386,6 +404,11 @@ export async function cmdTalk(ctx: Context, target: string, message: string): Pr
   // Single agent
   if (!config.paneRegistry[target]) {
     const available = Object.keys(config.paneRegistry).join(', ');
+    const teamHint = teamHintForMissingAgent(ctx, target);
+    if (teamHint) {
+      ui.error(teamHint);
+      exit(ExitCodes.PANE_NOT_FOUND);
+    }
     ui.error(`Agent '${target}' not found. Available: ${available || 'none'}`);
     exit(ExitCodes.PANE_NOT_FOUND);
   }
@@ -619,7 +642,8 @@ export async function cmdTalk(ctx: Context, target: string, message: string): Pr
       );
 
       // Clean Gemini CLI UI artifacts
-      const response = target === 'gemini' ? cleanGeminiResponse(extractedResponse) : extractedResponse;
+      const response =
+        target === 'gemini' ? cleanGeminiResponse(extractedResponse) : extractedResponse;
 
       if (!flags.json && isTTY) {
         process.stdout.write('\r' + ' '.repeat(80) + '\r');
@@ -883,7 +907,8 @@ async function cmdTalkAllWait(
         );
 
         // Clean Gemini CLI UI artifacts
-        const response = state.agent === 'gemini' ? cleanGeminiResponse(extractedResponse) : extractedResponse;
+        const response =
+          state.agent === 'gemini' ? cleanGeminiResponse(extractedResponse) : extractedResponse;
         state.response = response;
         state.truncated = truncated;
         state.status = 'completed';