@lobehub/lobehub 2.0.0-next.306 → 2.0.0-next.308

package/packages/builtin-tool-group-management/src/client/Inspector/ExecuteAgentTask/index.tsx

@@ -32,6 +32,7 @@ export const ExecuteAgentTaskInspector = memo<BuiltinInspectorProps<ExecuteTaskP
     const { t } = useTranslation('plugin');
 
     const agentId = args?.agentId || partialArgs?.agentId;
+    const taskTitle = args?.title || partialArgs?.title;
 
     // Get active group ID and agent from store
     const activeGroupId = useAgentGroupStore(agentGroupSelectors.activeGroupId);
@@ -42,12 +43,47 @@ export const ExecuteAgentTaskInspector = memo<BuiltinInspectorProps<ExecuteTaskP
     );
     const theme = useTheme();
 
-    if (isArgumentsStreaming && !agent) {
-      return (
-        <div className={cx(styles.root, shinyTextStyles.shinyText)}>
-          <span>{t('builtins.lobe-group-management.apiName.executeAgentTask')}</span>
-        </div>
-      );
+    if (isArgumentsStreaming) {
+      if (!agent && !taskTitle)
+        return (
+          <div className={cx(styles.root, shinyTextStyles.shinyText)}>
+            <span>{t('builtins.lobe-group-management.apiName.executeAgentTask')}</span>
+          </div>
+        );
+      if (agent) {
+        return (
+          <Flexbox
+            align={'center'}
+            className={cx(styles.root, isArgumentsStreaming && shinyTextStyles.shinyText)}
+            gap={8}
+            horizontal
+          >
+            <span className={styles.title}>
+              {t('builtins.lobe-group-management.inspector.executeAgentTask.assignTo')}
+            </span>
+            {agent && (
+              <>
+                <Avatar
+                  avatar={agent.avatar || DEFAULT_AVATAR}
+                  background={agent.backgroundColor || theme.colorBgContainer}
+                  shape={'square'}
+                  size={24}
+                  title={agent.title || undefined}
+                />
+                <span>{agent?.title}</span>
+              </>
+            )}
+            {taskTitle && (
+              <>
+                <span className={styles.title}>
+                  {t('builtins.lobe-group-management.inspector.executeAgentTask.task')}
+                </span>
+                <span className={highlightTextStyles.primary}>{taskTitle}</span>
+              </>
+            )}
+          </Flexbox>
+        );
+      }
     }
 
     const agentName = agent?.title || agentId;
@@ -60,7 +96,7 @@ export const ExecuteAgentTaskInspector = memo<BuiltinInspectorProps<ExecuteTaskP
         horizontal
       >
         <span className={styles.title}>
-          {t('builtins.lobe-group-management.inspector.executeAgentTask.title')}
+          {t('builtins.lobe-group-management.inspector.executeAgentTask.assignTo')}
         </span>
         {agent && (
           <Avatar
@@ -71,7 +107,15 @@ export const ExecuteAgentTaskInspector = memo<BuiltinInspectorProps<ExecuteTaskP
             title={agent.title || undefined}
           />
         )}
-        {agentName && <span className={highlightTextStyles.primary}>{agentName}</span>}
+        {agentName && <span>{agentName}</span>}
+        {taskTitle && (
+          <>
+            <span className={styles.title}>
+              {t('builtins.lobe-group-management.inspector.executeAgentTask.task')}
+            </span>
+            <span className={highlightTextStyles.primary}>{taskTitle}</span>
+          </>
+        )}
       </Flexbox>
     );
   },

package/packages/builtin-tool-group-management/src/client/Render/ExecuteTask/index.tsx

@@ -1,15 +1,12 @@
 'use client';
 
 import { BuiltinRenderProps } from '@lobechat/types';
-import { Avatar, Flexbox, Text } from '@lobehub/ui';
+import { Flexbox, Text } from '@lobehub/ui';
 import { createStaticStyles } from 'antd-style';
 import { Clock } from 'lucide-react';
 import { memo } from 'react';
 import { useTranslation } from 'react-i18next';
 
-import { useAgentGroupStore } from '@/store/agentGroup';
-import { agentGroupSelectors } from '@/store/agentGroup/selectors';
-
 import type { ExecuteTaskParams, ExecuteTaskState } from '../../../types';
 
 const styles = createStaticStyles(({ css, cssVar }) => ({
@@ -40,14 +37,6 @@ const ExecuteTaskRender = memo<BuiltinRenderProps<ExecuteTaskParams, ExecuteTask
   ({ args }) => {
     const { t } = useTranslation('tool');
 
-    // Get agent info from store
-    const activeGroupId = useAgentGroupStore(agentGroupSelectors.activeGroupId);
-    const agent = useAgentGroupStore((s) =>
-      args?.agentId && activeGroupId
-        ? agentGroupSelectors.getAgentByIdFromGroup(activeGroupId, args.agentId)(s)
-        : undefined,
-    );
-
     const timeoutMinutes = args?.timeout ? Math.round(args.timeout / 60_000) : 30;
 
     return (
@@ -55,15 +44,7 @@ const ExecuteTaskRender = memo<BuiltinRenderProps<ExecuteTaskParams, ExecuteTask
         {/* Header: Agent info + Timeout */}
         <Flexbox align={'center'} gap={12} horizontal justify={'space-between'}>
           <Flexbox align={'center'} flex={1} gap={12} horizontal style={{ minWidth: 0 }}>
-            <Avatar
-              avatar={agent?.avatar || '🤖'}
-              background={agent?.backgroundColor || undefined}
-              size={24}
-              style={{ borderRadius: 8, flexShrink: 0 }}
-            />
-            <span className={styles.agentTitle}>
-              {agent?.title || t('agentGroupManagement.executeTask.intervention.unknownAgent')}
-            </span>
+            <span className={styles.agentTitle}>{args?.title}</span>
           </Flexbox>
           <Flexbox align="center" className={styles.timeout} gap={4} horizontal>
             <Clock size={14} />

package/packages/builtin-tool-group-management/src/executor.test.ts

@@ -271,18 +271,6 @@ describe('GroupManagementExecutor', () => {
     });
   });
 
-  describe('getAgentInfo', () => {
-    it('should return error when no groupId in context', async () => {
-      const ctx = createMockContext();
-
-      const result = await groupManagementExecutor.getAgentInfo({ agentId: 'agent-1' }, ctx);
-
-      // No groupId means we can't get agent info
-      expect(result.success).toBe(false);
-      expect(result.stop).toBeUndefined();
-    });
-  });
-
   describe('executeAgentTask', () => {
     beforeEach(() => {
       vi.clearAllMocks();
@@ -292,7 +280,7 @@ describe('GroupManagementExecutor', () => {
       const ctx = createMockContext();
 
       const result = await groupManagementExecutor.executeAgentTask(
-        { agentId: 'agent-1', task: 'Do something' },
+        { agentId: 'agent-1', task: 'Do something', title: 'Test Task' },
         ctx,
       );
 
@@ -303,6 +291,7 @@ describe('GroupManagementExecutor', () => {
         agentId: 'agent-1',
         task: 'Do something',
         timeout: undefined,
+        title: 'Test Task',
         type: 'executeAgentTask',
       });
     });
@@ -326,7 +315,7 @@ describe('GroupManagementExecutor', () => {
       );
 
       await groupManagementExecutor.executeAgentTask(
-        { agentId: 'agent-1', task: 'Do something', timeout: 30000 },
+        { agentId: 'agent-1', task: 'Do something', timeout: 30000, title: 'Test Task' },
         ctx,
       );
 
@@ -351,7 +340,7 @@ describe('GroupManagementExecutor', () => {
       const ctx = createMockContext();
 
       const result = await groupManagementExecutor.executeAgentTask(
-        { agentId: 'agent-1', task: 'Do something' },
+        { agentId: 'agent-1', task: 'Do something', title: 'Test Task' },
         ctx,
       );
 
@@ -363,7 +352,7 @@ describe('GroupManagementExecutor', () => {
       const ctx = createMockContext();
 
       const result = await groupManagementExecutor.executeAgentTask(
-        { agentId: 'agent-1', task: 'Do something', timeout: 60000 },
+        { agentId: 'agent-1', task: 'Do something', timeout: 60000, title: 'Test Task' },
         ctx,
       );
 
@@ -372,6 +361,7 @@ describe('GroupManagementExecutor', () => {
         agentId: 'agent-1',
         task: 'Do something',
         timeout: 60000,
+        title: 'Test Task',
         type: 'executeAgentTask',
       });
     });

package/packages/builtin-tool-group-management/src/executor.ts

@@ -11,7 +11,6 @@ import {
   CreateWorkflowParams,
   DelegateParams,
   ExecuteTaskParams,
-  GetAgentInfoParams,
   GroupManagementApiName,
   GroupManagementIdentifier,
   InterruptParams,
@@ -19,43 +18,12 @@ import {
   SummarizeParams,
   VoteParams,
 } from '@lobechat/builtin-tool-group-management';
-import { formatAgentProfile } from '@lobechat/prompts';
 import { BaseExecutor, type BuiltinToolContext, type BuiltinToolResult } from '@lobechat/types';
 
-import { agentGroupSelectors, useAgentGroupStore } from '@/store/agentGroup';
-
 class GroupManagementExecutor extends BaseExecutor<typeof GroupManagementApiName> {
   readonly identifier = GroupManagementIdentifier;
   protected readonly apiEnum = GroupManagementApiName;
 
-  // ==================== Agent Info ====================
-
-  getAgentInfo = async (
-    params: GetAgentInfoParams,
-    ctx: BuiltinToolContext,
-  ): Promise<BuiltinToolResult> => {
-    const { groupId } = ctx;
-
-    if (!groupId) {
-      return {
-        content: JSON.stringify({ error: 'No group context available', success: false }),
-        success: false,
-      };
-    }
-
-    const agent = agentGroupSelectors.getAgentByIdFromGroup(
-      groupId,
-      params.agentId,
-    )(useAgentGroupStore.getState());
-
-    if (!agent) {
-      return { content: `Agent "${params.agentId}" not found in this group`, success: false };
-    }
-
-    // Return formatted agent profile for the supervisor
-    return { content: formatAgentProfile(agent), state: agent, success: true };
-  };
-
   // ==================== Communication Coordination ====================
 
   speak = async (params: SpeakParams, ctx: BuiltinToolContext): Promise<BuiltinToolResult> => {
@@ -225,12 +193,12 @@ class GroupManagementExecutor extends BaseExecutor<typeof GroupManagementApiName
     _ctx: BuiltinToolContext,
   ): Promise<BuiltinToolResult> => {
     // TODO: Implement conversation summarization
+    const focusInfo = params.focus ? ` with focus on "${params.focus}"` : '';
+    const preserveInfo = params.preserveRecent
+      ? ` (preserving ${params.preserveRecent} recent messages)`
+      : '';
     return {
-      content: JSON.stringify({
-        focus: params.focus,
-        message: 'Summarization not yet implemented',
-        preserveRecent: params.preserveRecent,
-      }),
+      content: `Summarization not yet implemented${focusInfo}${preserveInfo}`,
       success: true,
     };
   };
@@ -243,23 +211,16 @@ class GroupManagementExecutor extends BaseExecutor<typeof GroupManagementApiName
   ): Promise<BuiltinToolResult> => {
     // TODO: Implement workflow creation
     return {
-      content: JSON.stringify({
-        message: 'Workflow creation not yet implemented',
-        name: params.name,
-        steps: params.steps,
-      }),
+      content: `Workflow creation not yet implemented for "${params.name}" with ${params.steps.length} steps`,
       success: true,
     };
   };
 
   vote = async (params: VoteParams, _ctx: BuiltinToolContext): Promise<BuiltinToolResult> => {
     // TODO: Implement voting mechanism
+    const optionLabels = params.options.map((o) => o.label).join(', ');
     return {
-      content: JSON.stringify({
-        message: 'Voting not yet implemented',
-        options: params.options,
-        question: params.question,
-      }),
+      content: `Voting not yet implemented for question: "${params.question}" with options: ${optionLabels}`,
       success: true,
     };
   };

package/packages/builtin-tool-group-management/src/manifest.ts

@@ -8,23 +8,6 @@ export const GroupManagementIdentifier = 'lobe-group-management';
 export const GroupManagementManifest: BuiltinToolManifest = {
   /* eslint-disable sort-keys-fix/sort-keys-fix */
   api: [
-    // ==================== Agent Info ====================
-    {
-      description:
-        'Get detailed information about a specific agent, including their capabilities, available tools, and configuration. Use this to check if an agent has tools before deciding between speak vs executeAgentTask.',
-      name: GroupManagementApiName.getAgentInfo,
-      parameters: {
-        properties: {
-          agentId: {
-            description: 'The ID of the agent to get information about.',
-            type: 'string',
-          },
-        },
-        required: ['agentId'],
-        type: 'object',
-      },
-    },
-
     // ==================== Communication Coordination ====================
     {
       description:
@@ -112,6 +95,10 @@ export const GroupManagementManifest: BuiltinToolManifest = {
             description: 'The ID of the agent to execute the task.',
             type: 'string',
           },
+          title: {
+            description: 'Brief title describing what this task does (shown in UI).',
+            type: 'string',
+          },
           task: {
             description:
               'Clear description of the task to perform. Be specific about expected deliverables.',
@@ -130,7 +117,7 @@ export const GroupManagementManifest: BuiltinToolManifest = {
             type: 'boolean',
           },
         },
-        required: ['agentId', 'task'],
+        required: ['agentId', 'title', 'task'],
         type: 'object',
       },
     },

package/packages/builtin-tool-group-management/src/systemRole.ts

@@ -150,9 +150,6 @@ When a user's request is broad or unclear, ask 1-2 focused questions to understa
 <core_capabilities>
 ## Tool Categories
 
-**Agent Info:**
-- **getAgentInfo**: Get agent details including their tools and capabilities - **Use this to check if an agent has tools before deciding speak vs executeAgentTask**
-
 **Speaking (Shared Context, No Tools):**
 - **speak**: Single agent responds synchronously in group context
 - **broadcast**: Multiple agents respond in parallel in group context
@@ -241,9 +238,6 @@ Action: executeAgentTasks({
 </workflow_patterns>
 
 <tool_usage_guidelines>
-**Agent Info:**
-- getAgentInfo: \`agentId\` - **Use this to check if an agent has tools before deciding speak vs executeAgentTask**
-
 **Speaking:**
 - speak: \`agentId\`, \`instruction\` (optional guidance)
 - broadcast: \`agentIds\` (array), \`instruction\` (optional shared guidance)
@@ -259,8 +253,7 @@ Action: executeAgentTasks({
 </tool_usage_guidelines>
 
 <best_practices>
-1. **Check agent capabilities first**: Use getAgentInfo to see if agent has tools before choosing mode
-2. **Don't over-engineer**: Simple questions → speak; Complex tasks requiring tools → executeAgentTask
+1. **Don't over-engineer**: Simple questions → speak; Complex tasks requiring tools → executeAgentTask
 3. **Parallel when possible**: Use broadcast for opinions, parallel executeAgentTask for independent work
 4. **Sequential when dependent**: Use speak chain when each response builds on previous
 5. **Be explicit with task instructions**: For executeAgentTask, clearly describe expected deliverables

package/packages/builtin-tool-group-management/src/types.ts

@@ -7,10 +7,6 @@
  * are handled by group-agent-builder tool. This tool focuses on orchestration.
  */
 export const GroupManagementApiName = {
-  // ==================== Agent Info ====================
-  /** Get detailed information about an agent */
-  getAgentInfo: 'getAgentInfo',
-
   // ==================== Communication Coordination ====================
   /** Let a specific agent speak (synchronous, immediate response) */
   speak: 'speak',
@@ -41,12 +37,6 @@ export const GroupManagementApiName = {
 export type GroupManagementApiNameType =
   (typeof GroupManagementApiName)[keyof typeof GroupManagementApiName];
 
-// ==================== Agent Info Params ====================
-
-export interface GetAgentInfoParams {
-  agentId: string;
-}
-
 // ==================== Communication Params ====================
 
 export interface SpeakParams {
@@ -90,6 +80,8 @@ export interface ExecuteTaskParams {
   skipCallSupervisor?: boolean;
   task: string;
   timeout?: number;
+  /** Brief title describing what this task does (shown in UI) */
+  title: string;
 }
 
 export interface TaskItem {

package/packages/builtin-tool-local-system/src/ExecutionRuntime/index.ts

@@ -71,8 +71,14 @@ export class LocalSystemExecutionRuntime {
 
       const state: LocalFileListState = { listResults: result };
 
+      const fileList = result.map((f) => `  ${f.isDirectory ? '[D]' : '[F]'} ${f.name}`).join('\n');
+      const content =
+        result.length > 0
+          ? `Found ${result.length} item(s) in ${args.path}:\n${fileList}`
+          : `Directory ${args.path} is empty`;
+
       return {
-        content: JSON.stringify(result),
+        content,
         state,
         success: true,
       };
@@ -91,8 +97,11 @@ export class LocalSystemExecutionRuntime {
 
       const state: LocalReadFileState = { fileContent: result };
 
+      const lineInfo = args.loc ? ` (lines ${args.loc[0]}-${args.loc[1]})` : '';
+      const content = `File: ${args.path}${lineInfo}\n\n${result.content}`;
+
       return {
-        content: JSON.stringify(result),
+        content,
         state,
         success: true,
       };
@@ -111,8 +120,11 @@ export class LocalSystemExecutionRuntime {
 
       const state: LocalReadFilesState = { filesContent: results };
 
+      const fileContents = results.map((r) => `=== ${r.filename} ===\n${r.content}`).join('\n\n');
+      const content = `Read ${results.length} file(s):\n\n${fileContents}`;
+
       return {
-        content: JSON.stringify(results),
+        content,
         state,
         success: true,
       };
@@ -131,8 +143,12 @@ export class LocalSystemExecutionRuntime {
 
       const state: LocalFileSearchState = { searchResults: result };
 
+      const fileList = result.map((f) => `  ${f.path}`).join('\n');
+      const content =
+        result.length > 0 ? `Found ${result.length} file(s):\n${fileList}` : 'No files found';
+
       return {
-        content: JSON.stringify(result),
+        content,
         state,
         success: true,
       };
@@ -154,14 +170,14 @@ export class LocalSystemExecutionRuntime {
       const successCount = results.filter((r) => r.success).length;
       const failedCount = results.length - successCount;
 
-      let message = '';
+      let content = '';
 
       if (allSucceeded) {
-        message = `Successfully moved ${results.length} item(s).`;
+        content = `Successfully moved ${results.length} item(s).`;
       } else if (someFailed) {
-        message = `Moved ${successCount} item(s) successfully. Failed to move ${failedCount} item(s).`;
+        content = `Moved ${successCount} item(s) successfully. Failed to move ${failedCount} item(s).`;
       } else {
-        message = `Failed to move all ${results.length} item(s).`;
+        content = `Failed to move all ${results.length} item(s).`;
       }
 
       const state: LocalMoveFilesState = {
@@ -171,7 +187,7 @@ export class LocalSystemExecutionRuntime {
       };
 
       return {
-        content: JSON.stringify({ message, results }),
+        content,
         state,
         success: true,
       };
@@ -197,7 +213,7 @@ export class LocalSystemExecutionRuntime {
         };
 
         return {
-          content: JSON.stringify({ message: result.error, success: false }),
+          content: `Failed to rename file: ${result.error}`,
           state,
           success: false,
         };
@@ -210,10 +226,7 @@ export class LocalSystemExecutionRuntime {
       };
 
       return {
-        content: JSON.stringify({
-          message: `Successfully renamed file ${args.path} to ${args.newName}.`,
-          success: true,
-        }),
+        content: `Successfully renamed file ${args.path} to ${args.newName}`,
         state,
         success: true,
       };
@@ -232,20 +245,14 @@ export class LocalSystemExecutionRuntime {
 
       if (!result.success) {
         return {
-          content: JSON.stringify({
-            message: result.error || '写入文件失败',
-            success: false,
-          }),
+          content: `Failed to write file: ${result.error || 'Unknown error'}`,
           error: result.error,
           success: false,
         };
       }
 
       return {
-        content: JSON.stringify({
-          message: `成功写入文件 ${args.path}`,
-          success: true,
-        }),
+        content: `Successfully wrote to ${args.path}`,
         success: true,
       };
     } catch (error) {
@@ -301,22 +308,28 @@ export class LocalSystemExecutionRuntime {
     try {
       const result: RunCommandResult = await this.localFileService.runCommand(args);
 
-      let message: string;
+      const parts: string[] = [];
 
       if (result.success) {
         if (result.shell_id) {
-          message = `Command started in background with shell_id: ${result.shell_id}`;
+          parts.push(`Command started in background with shell_id: ${result.shell_id}`);
         } else {
-          message = `Command completed successfully.`;
+          parts.push('Command completed successfully.');
         }
       } else {
-        message = `Command failed: ${result.error}`;
+        parts.push(`Command failed: ${result.error}`);
       }
 
+      if (result.stdout) parts.push(`Output:\n${result.stdout}`);
+      if (result.stderr) parts.push(`Stderr:\n${result.stderr}`);
+      if (result.exit_code !== undefined) parts.push(`Exit code: ${result.exit_code}`);
+
+      const message = parts[0];
+      const content = parts.join('\n\n');
       const state: RunCommandState = { message, result };
 
       return {
-        content: JSON.stringify(result),
+        content,
         state,
         success: result.success,
       };
@@ -337,10 +350,14 @@ export class LocalSystemExecutionRuntime {
         ? `Output retrieved. Running: ${result.running}`
         : `Failed: ${result.error}`;
 
+      const parts: string[] = [message];
+      if (result.output) parts.push(`Output:\n${result.output}`);
+      if (result.error) parts.push(`Error: ${result.error}`);
+
       const state: GetCommandOutputState = { message, result };
 
       return {
-        content: JSON.stringify(result),
+        content: parts.join('\n\n'),
         state,
         success: result.success,
       };
@@ -364,7 +381,7 @@ export class LocalSystemExecutionRuntime {
       const state: KillCommandState = { message, result };
 
       return {
-        content: JSON.stringify(result),
+        content: message,
         state,
         success: result.success,
       };
@@ -389,8 +406,19 @@ export class LocalSystemExecutionRuntime {
 
       const state: GrepContentState = { message, result };
 
+      let content = message;
+      if (result.success && result.matches.length > 0) {
+        const matchList = result.matches
+          .slice(0, 20)
+          .map((m) => `  ${m}`)
+          .join('\n');
+        const moreInfo =
+          result.matches.length > 20 ? `\n  ... and ${result.matches.length - 20} more` : '';
+        content = `${message}:\n${matchList}${moreInfo}`;
+      }
+
       return {
-        content: JSON.stringify(result),
+        content,
         state,
         success: result.success,
       };
@@ -411,8 +439,19 @@ export class LocalSystemExecutionRuntime {
 
       const state: GlobFilesState = { message, result };
 
+      let content = message;
+      if (result.success && result.files.length > 0) {
+        const fileList = result.files
+          .slice(0, 50)
+          .map((f) => `  ${f}`)
+          .join('\n');
+        const moreInfo =
+          result.files.length > 50 ? `\n  ... and ${result.files.length - 50} more` : '';
+        content = `${message}:\n${fileList}${moreInfo}`;
+      }
+
       return {
-        content: JSON.stringify(result),
+        content,
         state,
         success: result.success,
       };