tuna-agent 0.1.173 → 0.1.175

package/dist/agents/claude-code-adapter.js

@@ -558,14 +558,11 @@ export class ClaudeCodeAdapter {
                 pmSessionId = chatResult.sessionId || pmSessionId;
                 // Update persisted PM session ID
                 savePMState({ taskId: task.id, pmSessionId, repoPath: task.repoPath, savedAt: new Date().toISOString(), agentId: task.agentId });
-                // PM produced a plan with subtasks → break out to execution
+                // PM produced a plan with subtasks → break out to execution.
+                // (No "Got it! <summary>" message — plan_ready already conveys the summary;
+                // sending both duplicated it and added a robotic English prefix.)
                 if (chatResult.plan && chatResult.plan.subtasks.length > 0) {
                     planResult = { plan: chatResult.plan, pmSessionId };
-                    ws.sendPMMessage(task.id, {
-                        sender: 'pm',
-                        content: `Got it! ${chatResult.plan.summary}`,
-                        startedAt: firstChunkIso || undefined,
-                    });
                     break;
                 }
                 // PM responded conversationally — send to app and update latest message
@@ -709,11 +706,8 @@ export class ClaudeCodeAdapter {
                 const { text: answer } = await waitForInput(task.id, pendingInputResolvers);
                 console.log(`[ClaudeCode] User answered: ${answer.substring(0, 80)}`);
                 pmAnswers.push({ question: q.question, answer });
-                ws.sendPMMessage(task.id, {
-                    sender: 'pm',
-                    content: `Got it! Proceeding with: ${answer.substring(0, 100)}${answer.length > 100 ? '...' : ''}`,
-                });
-                // Delay to ensure "Got it!" is saved to DB before next question triggers refetch
+                // (No robotic "Got it! Proceeding with…" ack — the agent just continues.)
+                // Small delay to avoid a refetch race before the next question.
                 await new Promise(resolve => setTimeout(resolve, 200));
             }
         }
@@ -730,22 +724,12 @@ export class ClaudeCodeAdapter {
         // Step 4: Execute subtasks
         console.log(`[ClaudeCode] Starting execution for task ${task.id}`);
         ws.sendProgress(task.id, 'executing', { startedAt: new Date().toISOString() });
-        ws.sendPMMessage(task.id, {
-            sender: 'pm',
-            content: "Starting execution now. I'll keep you updated as each session progresses.",
-        });
         // Step 4-6: Execute plan with follow-up loop
         const allSessions = [];
         let lastStatus = 'done';
         const MAX_FOLLOW_UP_ROUNDS = 10;
         for (let followUpRound = 0; followUpRound <= MAX_FOLLOW_UP_ROUNDS; followUpRound++) {
             ws.sendProgress(task.id, 'executing', { startedAt: new Date().toISOString() });
-            if (followUpRound === 0) {
-                ws.sendPMMessage(task.id, {
-                    sender: 'pm',
-                    content: "Starting execution now. I'll keep you updated as each session progresses.",
-                });
-            }
             const result = await executePlanAndReport(task, plan, ws, pendingInputResolvers, signal, confirmBeforeEdit, onPermissionRequest);
             allSessions.push(...result.sessions);
             lastStatus = result.status;

package/dist/daemon/index.js

@@ -777,7 +777,7 @@ ${skillContent.slice(0, 15000)}`;
                 // PM produced a plan → execute it using shared helper
                 if (chatResult.plan && chatResult.plan.subtasks.length > 0) {
                     const plan = chatResult.plan;
-                    wsClient.sendPMMessage(taskId, { sender: 'pm', content: `Got it! ${plan.summary}`, startedAt: firstChunkIso || undefined });
+                    // (No "Got it! <summary>" — plan_ready already conveys the summary.)
                     wsClient.sendPlanReady(taskId, {
                         summary: plan.summary,
                         subtasks: plan.subtasks.map((s) => ({

package/dist/utils/execution-helpers.js

@@ -65,11 +65,15 @@ export function buildExecutionCallbacks(taskId, plan, ws, resolvers, onPermissio
                     role: subtask.role,
                     description: subtask.description,
                 });
-                await new Promise(resolve => setTimeout(resolve, 100));
-                ws.sendPMMessage(taskId, {
-                    sender: 'system',
-                    content: `${capitalize(subtask.role)} session started`,
-                });
+                // Only announce a "session started" for real multi-role dev teams.
+                // A single 'self' agent doing its own work needs no orchestration noise.
+                if (subtask.role !== 'self') {
+                    await new Promise(resolve => setTimeout(resolve, 100));
+                    ws.sendPMMessage(taskId, {
+                        sender: 'system',
+                        content: `${capitalize(subtask.role)} session started`,
+                    });
+                }
             }
         },
         onSubtaskLog: async (subtaskId, logData) => {
@@ -92,9 +96,13 @@ export function buildExecutionCallbacks(taskId, plan, ws, resolvers, onPermissio
                 const durationStr = durationMs ? `${(durationMs / 1000).toFixed(1)}s` : '';
                 const resultPreview = rawResult ? simplifyMarkdown(rawResult) : '';
                 await new Promise(resolve => setTimeout(resolve, 100));
+                const isSelf = subtask?.role === 'self';
                 ws.sendPMMessage(taskId, {
                     sender: 'pm',
-                    content: `${roleName} completed successfully${durationStr ? ` in ${durationStr}` : ''}${resultPreview ? `\n\n${resultPreview}` : ''}`,
+                    // Single 'self' agent: just deliver the result, no "X completed successfully" scaffolding.
+                    content: isSelf
+                        ? (resultPreview || 'Xong ạ.')
+                        : `${roleName} completed successfully${durationStr ? ` in ${durationStr}` : ''}${resultPreview ? `\n\n${resultPreview}` : ''}`,
                 });
             }
             else if (session.status === 'failed') {
@@ -109,9 +117,10 @@ export function buildExecutionCallbacks(taskId, plan, ws, resolvers, onPermissio
         onSubtaskNeedsInput: async (subtaskId, question) => {
             const subtask = plan.subtasks.find((s) => s.id === subtaskId);
             const roleName = subtask ? capitalize(subtask.role) : 'A session';
+            const askSelf = subtask?.role === 'self';
             ws.sendPMMessage(taskId, {
                 sender: 'pm',
-                content: `${roleName} is asking: ${question.question}`,
+                content: askSelf ? question.question : `${roleName} is asking: ${question.question}`,
                 options: question.options,
                 context: question.context,
             });
@@ -122,10 +131,13 @@ export function buildExecutionCallbacks(taskId, plan, ws, resolvers, onPermissio
                 context: question.context,
             });
             const { text: answer } = await waitForInput(taskId, resolvers);
-            ws.sendPMMessage(taskId, {
-                sender: 'pm',
-                content: `Got it! Passing this to ${roleName}...`,
-            });
+            // No "passing this to X" scaffolding for a single self agent.
+            if (!askSelf) {
+                ws.sendPMMessage(taskId, {
+                    sender: 'pm',
+                    content: `Got it! Passing this to ${roleName}...`,
+                });
+            }
             return answer;
         },
         onLayerStart: async (layerIndex, totalLayers, subtaskIds) => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "tuna-agent",
-  "version": "0.1.173",
+  "version": "0.1.175",
   "description": "Tuna Agent - Run AI coding tasks on your machine",
   "bin": {
     "tuna-agent": "dist/cli/index.js"