plugin-agent-orchestrator 1.0.22 → 1.0.23

package/src/client/skill-hub/tools/InteractionSchemasProvider.tsx

@@ -1,99 +1,132 @@
-import React, { createContext, useContext, useEffect, useState } from 'react';
-import { useAPIClient } from '@nocobase/client';
-import { parseJsonText } from '../utils/jsonFields';
-import { InteractionSchema } from './loopTemplates';
-
-const Ctx = createContext<Map<string, InteractionSchema>>(new Map());
-
-export const useInteractionSchemas = () => useContext(Ctx);
-
-const sanitize = (name: string) =>
-  name
-    .toLowerCase()
-    .replace(/[^a-z0-9_]/g, '_')
-    .replace(/_+/g, '_')
-    .replace(/^_|_$/g, '');
-
-export const InteractionSchemasProvider: React.FC<{ children?: React.ReactNode }> = ({ children }) => {
-  const api = useAPIClient();
-  const [map, setMap] = useState<Map<string, InteractionSchema>>(new Map());
-  const [version, setVersion] = useState(0);
-
-  useEffect(() => {
-    const refresh = () => setVersion((current) => current + 1);
-    window.addEventListener('skill-hub-loop-settings-changed', refresh);
-    return () => window.removeEventListener('skill-hub-loop-settings-changed', refresh);
-  }, []);
-
-  useEffect(() => {
-    let cancelled = false;
-    const extractList = (data: any) => {
-      const value = data?.data?.data ?? data?.data ?? data ?? [];
-      return Array.isArray(value) ? value : [];
-    };
-    const schemaFromLoopConfig = (config: any): InteractionSchema | null => {
-      const schema = parseJsonText<InteractionSchema | null>(config.schema, null);
-      if (schema) {
-        return config.prompt && !schema.prompt ? { ...schema, prompt: config.prompt } : schema;
-      }
-      if (config.prompt) {
-        return {
-          type: 'confirm',
-          prompt: config.prompt,
-        };
-      }
-      return null;
-    };
-
-    Promise.all([
-      api.request({
-        url: 'skillDefinitions:list',
-        params: {
-          filter: { enabled: true },
-          fields: ['id', 'name', 'autoCall', 'interactionSchema'],
-          pageSize: 500,
-        },
-      }),
-      api.request({
-        url: 'skillLoopConfigs:list',
-        params: {
-          filter: { enabled: true },
-          fields: ['skillId', 'enabled', 'schema', 'prompt', 'templateKey'],
-          pageSize: 500,
-        },
-      }).catch(() => ({ data: [] })),
-    ])
-      .then(([skillsResponse, loopConfigsResponse]) => {
-        if (cancelled) return;
-        const next = new Map<string, InteractionSchema>();
-        const skills = extractList(skillsResponse.data);
-        const loopConfigs = extractList(loopConfigsResponse.data);
-        const skillsById = new Map(skills.map((skill: any) => [String(skill.id), skill]));
-
-        for (const s of skills) {
-          if (s.autoCall) continue;
-          const schema = parseJsonText<InteractionSchema | null>(s.interactionSchema, null);
-          if (!schema) continue;
-          next.set(sanitize(s.name), schema);
-        }
-
-        for (const config of loopConfigs) {
-          const skill = skillsById.get(String(config.skillId));
-          if (!skill?.name) continue;
-          const schema = schemaFromLoopConfig(config);
-          if (!schema) continue;
-          next.set(sanitize(skill.name), schema);
-        }
-
-        setMap(next);
-      })
-      .catch(() => {
-        // silently ignore — user may lack permission to list definitions
-      });
-    return () => {
-      cancelled = true;
-    };
-  }, [api, version]);
-
-  return <Ctx.Provider value={map}>{children}</Ctx.Provider>;
-};
+import React, { createContext, useContext, useEffect, useState } from 'react';
+import { useAPIClient } from '@nocobase/client';
+import { parseJsonText } from '../utils/jsonFields';
+import { InteractionSchema } from './loopTemplates';
+
+const Ctx = createContext<Map<string, InteractionSchema>>(new Map());
+
+export const useInteractionSchemas = () => useContext(Ctx);
+
+type ApiClientWithAuth = ReturnType<typeof useAPIClient> & {
+  auth?: {
+    getToken?: () => string | null | undefined;
+    token?: string | null;
+  };
+  app?: {
+    eventBus?: {
+      addEventListener?: (type: string, listener: EventListenerOrEventListenerObject) => void;
+      removeEventListener?: (type: string, listener: EventListenerOrEventListenerObject) => void;
+    };
+  };
+};
+
+const sanitize = (name: string) =>
+  name
+    .toLowerCase()
+    .replace(/[^a-z0-9_]/g, '_')
+    .replace(/_+/g, '_')
+    .replace(/^_|_$/g, '');
+
+export const InteractionSchemasProvider: React.FC<{ children?: React.ReactNode }> = ({ children }) => {
+  const api = useAPIClient() as ApiClientWithAuth;
+  const authToken = api.auth?.getToken?.() || api.auth?.token || '';
+  const [map, setMap] = useState<Map<string, InteractionSchema>>(new Map());
+  const [version, setVersion] = useState(0);
+
+  useEffect(() => {
+    const refresh = () => setVersion((current) => current + 1);
+    const refreshAfterSignIn = (event: Event) => {
+      const token = (event as CustomEvent<{ token?: string | null }>).detail?.token;
+      if (token) {
+        refresh();
+      }
+    };
+    window.addEventListener('skill-hub-loop-settings-changed', refresh);
+    api.app?.eventBus?.addEventListener?.('auth:tokenChanged', refreshAfterSignIn);
+    return () => {
+      window.removeEventListener('skill-hub-loop-settings-changed', refresh);
+      api.app?.eventBus?.removeEventListener?.('auth:tokenChanged', refreshAfterSignIn);
+    };
+  }, [api]);
+
+  useEffect(() => {
+    let cancelled = false;
+    if (!authToken) {
+      return () => {
+        cancelled = true;
+      };
+    }
+    const extractList = (data: any) => {
+      const value = data?.data?.data ?? data?.data ?? data ?? [];
+      return Array.isArray(value) ? value : [];
+    };
+    const schemaFromLoopConfig = (config: any): InteractionSchema | null => {
+      const schema = parseJsonText<InteractionSchema | null>(config.schema, null);
+      if (schema) {
+        return config.prompt && !schema.prompt ? { ...schema, prompt: config.prompt } : schema;
+      }
+      if (config.prompt) {
+        return {
+          type: 'confirm',
+          prompt: config.prompt,
+        };
+      }
+      return null;
+    };
+
+    Promise.all([
+      api.request({
+        url: 'skillDefinitions:list',
+        skipNotify: true,
+        params: {
+          filter: { enabled: true },
+          fields: ['id', 'name', 'autoCall', 'interactionSchema'],
+          pageSize: 500,
+        },
+      }),
+      api
+        .request({
+          url: 'skillLoopConfigs:list',
+          skipNotify: true,
+          params: {
+            filter: { enabled: true },
+            fields: ['skillId', 'enabled', 'schema', 'prompt', 'templateKey'],
+            pageSize: 500,
+          },
+        })
+        .catch(() => ({ data: [] })),
+    ])
+      .then(([skillsResponse, loopConfigsResponse]) => {
+        if (cancelled) return;
+        const next = new Map<string, InteractionSchema>();
+        const skills = extractList(skillsResponse.data);
+        const loopConfigs = extractList(loopConfigsResponse.data);
+        const skillsById = new Map(skills.map((skill: any) => [String(skill.id), skill]));
+
+        for (const s of skills) {
+          if (s.autoCall) continue;
+          const schema = parseJsonText<InteractionSchema | null>(s.interactionSchema, null);
+          if (!schema) continue;
+          next.set(sanitize(s.name), schema);
+        }
+
+        for (const config of loopConfigs) {
+          const skill = skillsById.get(String(config.skillId));
+          if (!skill?.name) continue;
+          const schema = schemaFromLoopConfig(config);
+          if (!schema) continue;
+          next.set(sanitize(skill.name), schema);
+        }
+
+        setMap(next);
+      })
+      .catch(() => {
+        // silently ignore — user may lack permission to list definitions
+      });
+    return () => {
+      cancelled = true;
+    };
+  }, [api, authToken, version]);
+
+  return <Ctx.Provider value={map}>{children}</Ctx.Provider>;
+};

package/src/client/skill-hub/tools/registerSkillLoopCards.ts

@@ -1,58 +1,71 @@
-import { SkillHubCard } from './SkillHubCard';
-import { parseJsonText } from '../utils/jsonFields';
-
-const sanitize = (name: string) =>
-  name
-    .toLowerCase()
-    .replace(/[^a-z0-9_]/g, '_')
-    .replace(/_+/g, '_')
-    .replace(/^_|_$/g, '');
-
-const extractList = (data: any) => {
-  const value = data?.data?.data ?? data?.data ?? data ?? [];
-  return Array.isArray(value) ? value : [];
-};
-
-export async function registerSkillLoopCards(app: any) {
-  const toolsManager = app.aiManager?.toolsManager;
-  if (!toolsManager) return;
-
-  try {
-    const skillsResponse = await app.apiClient.request({
-      url: 'skillDefinitions:list',
-      params: {
-        filter: { enabled: true },
-        fields: ['id', 'name', 'autoCall', 'interactionSchema'],
-        pageSize: 500,
-      },
-    });
-
-    let loopSkillIds = new Set<string>();
-    try {
-      const loopConfigsResponse = await app.apiClient.request({
-        url: 'skillLoopConfigs:list',
-        params: {
-          filter: { enabled: true },
-          fields: ['skillId'],
-          pageSize: 500,
-        },
-      });
-      loopSkillIds = new Set(extractList(loopConfigsResponse.data).map((config: any) => String(config.skillId)));
-      if (loopSkillIds.size > 0) {
-        toolsManager.registerTools('skill_hub_execute', { ui: { card: SkillHubCard } });
-      }
-    } catch {
-      // Older deployments may not have the collection before migration/sync.
-    }
-
-    const skills = extractList(skillsResponse.data);
-    for (const skill of skills) {
-      const hasLoopConfig = loopSkillIds.has(String(skill.id));
-      const hasLegacySchema = !skill.autoCall && !!parseJsonText(skill.interactionSchema, null);
-      if (!hasLoopConfig && !hasLegacySchema) continue;
-      toolsManager.registerTools(`skill_hub_${sanitize(skill.name)}`, { ui: { card: SkillHubCard } });
-    }
-  } catch {
-    // user without ACL or backend unavailable - skip silently
-  }
-}
+import { SkillHubCard } from './SkillHubCard';
+import { parseJsonText } from '../utils/jsonFields';
+
+const sanitize = (name: string) =>
+  name
+    .toLowerCase()
+    .replace(/[^a-z0-9_]/g, '_')
+    .replace(/_+/g, '_')
+    .replace(/^_|_$/g, '');
+
+const extractList = (data: any) => {
+  const value = data?.data?.data ?? data?.data ?? data ?? [];
+  return Array.isArray(value) ? value : [];
+};
+
+const getAuthToken = (app: any) => app?.apiClient?.auth?.getToken?.() || app?.apiClient?.auth?.token || '';
+
+const registerSkillHubCard = (toolsManager: any, name: string) => {
+  try {
+    toolsManager.registerTools(name, { ui: { card: SkillHubCard } });
+  } catch (err: unknown) {
+    if (!(err instanceof Error) || !err.message.includes('override existing keys')) {
+      throw err;
+    }
+  }
+};
+
+export async function registerSkillLoopCards(app: any) {
+  const toolsManager = app.aiManager?.toolsManager;
+  if (!toolsManager) return;
+  registerSkillHubCard(toolsManager, 'skill_hub_execute');
+  if (!getAuthToken(app)) return;
+
+  try {
+    const skillsResponse = await app.apiClient.request({
+      url: 'skillDefinitions:list',
+      skipNotify: true,
+      params: {
+        filter: { enabled: true },
+        fields: ['id', 'name', 'autoCall', 'interactionSchema'],
+        pageSize: 500,
+      },
+    });
+
+    let loopSkillIds = new Set<string>();
+    try {
+      const loopConfigsResponse = await app.apiClient.request({
+        url: 'skillLoopConfigs:list',
+        skipNotify: true,
+        params: {
+          filter: { enabled: true },
+          fields: ['skillId'],
+          pageSize: 500,
+        },
+      });
+      loopSkillIds = new Set(extractList(loopConfigsResponse.data).map((config: any) => String(config.skillId)));
+    } catch {
+      // Older deployments may not have the collection before migration/sync.
+    }
+
+    const skills = extractList(skillsResponse.data);
+    for (const skill of skills) {
+      const hasLoopConfig = loopSkillIds.has(String(skill.id));
+      const hasLegacySchema = !skill.autoCall && !!parseJsonText(skill.interactionSchema, null);
+      if (!hasLoopConfig && !hasLegacySchema) continue;
+      registerSkillHubCard(toolsManager, `skill_hub_${sanitize(skill.name)}`);
+    }
+  } catch {
+    // user without ACL or backend unavailable - skip silently
+  }
+}

package/src/client/tools/registerOrchestratorCards.ts

@@ -1,7 +1,17 @@
-import { PlanApprovalCard } from './PlanApprovalCard';
-
-export async function registerOrchestratorCards(app: any) {
-  const toolsManager = app.aiManager?.toolsManager;
-  if (!toolsManager) return;
-  toolsManager.registerTools('orchestrator_execute_plan', { ui: { card: PlanApprovalCard } });
-}
+import { PlanApprovalCard } from './PlanApprovalCard';
+
+const registerToolCard = (toolsManager: any, name: string) => {
+  try {
+    toolsManager.registerTools(name, { ui: { card: PlanApprovalCard } });
+  } catch (err: unknown) {
+    if (!(err instanceof Error) || !err.message.includes('override existing keys')) {
+      throw err;
+    }
+  }
+};
+
+export async function registerOrchestratorCards(app: any) {
+  const toolsManager = app.aiManager?.toolsManager;
+  if (!toolsManager) return;
+  registerToolCard(toolsManager, 'orchestrator_execute_plan');
+}

package/src/client-v2/components/AIEmployeeSelect.tsx

@@ -0,0 +1,47 @@
+import React from 'react';
+import { Select } from 'antd';
+import { useAIEmployees } from './AIEmployeesContext';
+
+/**
+ * Reusable Select component for AI Employees.
+ * Uses shared AIEmployeesContext instead of making its own API call.
+ */
+export const AIEmployeeSelect: React.FC<{
+  value?: string;
+  onChange?: (value: string) => void;
+  exclude?: string; // username to exclude (prevent self-reference)
+  placeholder?: string;
+}> = ({ value, onChange, exclude, placeholder = 'Select AI Employee...' }) => {
+  const { employees, loading } = useAIEmployees();
+
+  const options = React.useMemo(() => {
+    return employees
+      .filter((emp) => !exclude || emp.username !== exclude)
+      .map((emp) => ({
+        label: emp.nickname,
+        value: emp.username,
+        description: emp.about,
+      }));
+  }, [employees, exclude]);
+
+  return (
+    <Select
+      loading={loading}
+      options={options}
+      value={value}
+      onChange={onChange}
+      placeholder={placeholder}
+      showSearch
+      filterOption={(input, option) =>
+        (option?.label ?? '').toString().toLowerCase().includes(input.toLowerCase()) ||
+        (option?.value ?? '').toString().toLowerCase().includes(input.toLowerCase())
+      }
+      optionRender={(option) => (
+        <div>
+          <div style={{ fontWeight: 500 }}>{option.label}</div>
+          {option.data.description && <div style={{ fontSize: 12, color: '#888' }}>{option.data.description}</div>}
+        </div>
+      )}
+    />
+  );
+};

package/src/client-v2/components/AIEmployeesContext.tsx

@@ -0,0 +1,110 @@
+import React, { createContext, useContext } from 'react';
+import { useRequest } from '../hooks/useApiRequest';
+
+interface AIEmployeeInfo {
+  username: string;
+  nickname: string;
+  about?: string;
+  tools: string[];
+}
+
+interface AIEmployeesContextType {
+  employees: AIEmployeeInfo[];
+  employeeMap: Map<string, string>;
+  toolNamesMap: Map<string, Set<string>>;
+  loading: boolean;
+  refresh: () => void;
+}
+
+const AIEmployeesContext = createContext<AIEmployeesContextType>({
+  employees: [],
+  employeeMap: new Map(),
+  toolNamesMap: new Map(),
+  loading: false,
+  refresh: () => {},
+});
+
+const orchestratorToolNames = new Set([
+  'orchestrator_plan_goal',
+  'orchestrator_execute_plan',
+  'orchestrator_status',
+  'orchestrator_cancel',
+  'external_rag_search',
+  'skill_hub_execute',
+]);
+
+function isToolLikeName(name: string) {
+  return (
+    orchestratorToolNames.has(name) ||
+    name.startsWith('delegate_') ||
+    name.startsWith('dispatch_subagents_') ||
+    name.startsWith('skill_hub_') ||
+    name.startsWith('browser_') ||
+    name.startsWith('drawio-')
+  );
+}
+
+function extractToolNames(skillSettings: any) {
+  const tools = Array.isArray(skillSettings?.tools)
+    ? skillSettings.tools
+        .map((tool: any) => (typeof tool === 'string' ? tool : tool?.name))
+        .filter((name: any): name is string => typeof name === 'string' && name.length > 0)
+    : [];
+
+  const legacyTools = Array.isArray(skillSettings?.skills)
+    ? skillSettings.skills
+        .map((skill: any) => {
+          if (typeof skill === 'string') {
+            return isToolLikeName(skill) ? skill : null;
+          }
+          return skill?.name;
+        })
+        .filter((name: any): name is string => typeof name === 'string' && name.length > 0)
+    : [];
+
+  return Array.from(new Set([...tools, ...legacyTools]));
+}
+
+/**
+ * Shared context provider that fetches aiEmployees once and shares the data
+ * across RulesTab, TracingTab, AgentRunsTab, and AIEmployeeSelect.
+ *
+ * Also exposes each employee's configured tools so RulesTab can warn when a
+ * delegation rule exists but the leader hasn't added the corresponding
+ * delegate_<leader>_to_<sub> tool to its skillSettings.tools.
+ */
+export const AIEmployeesProvider: React.FC<{ children: React.ReactNode }> = ({ children }) => {
+  const { data, loading, refresh } = useRequest({
+    url: 'aiEmployees:list',
+    params: { pageSize: 200 },
+  });
+
+  const value = React.useMemo(() => {
+    const rawEmployees = (data as any)?.data || [];
+    const employees: AIEmployeeInfo[] = rawEmployees.map((emp: any) => {
+      const tools = extractToolNames(emp.skillSettings);
+      return {
+        username: emp.username,
+        nickname: emp.nickname || emp.username,
+        about: emp.about?.substring(0, 80),
+        tools,
+      };
+    });
+
+    const employeeMap = new Map<string, string>();
+    const toolNamesMap = new Map<string, Set<string>>();
+    for (const emp of employees) {
+      employeeMap.set(emp.username, emp.nickname);
+      toolNamesMap.set(emp.username, new Set(emp.tools));
+    }
+
+    return { employees, employeeMap, toolNamesMap, loading, refresh };
+  }, [data, loading, refresh]);
+
+  return <AIEmployeesContext.Provider value={value}>{children}</AIEmployeesContext.Provider>;
+};
+
+/**
+ * Hook to access shared AI employees data.
+ */
+export const useAIEmployees = () => useContext(AIEmployeesContext);