@yancyyu/openhermit 1.6.28 → 1.6.30

package/src/renderer/store/slices/extensionsSlice.ts

@@ -18,9 +18,6 @@ import { findPaneByTabId, updatePane } from '../utils/paneHelpers';
 
 import type { AppState } from '../types';
 import type {
-  ApiKeyEntry,
-  ApiKeySaveRequest,
-  ApiKeyStorageStatus,
   EnrichedPlugin,
   ExtensionOperationState,
   InstalledMcpEntry,
@@ -74,12 +71,13 @@ export interface ExtensionsSlice {
   mcpInstallProgress: Record<string, ExtensionOperationState>;
   installErrors: Record<string, string>; // keyed by scoped operation key
 
-  // ── API Keys ──
-  apiKeys: ApiKeyEntry[];
-  apiKeysLoading: boolean;
-  apiKeysError: string | null;
-  apiKeySaving: boolean;
-  apiKeyStorageStatus: ApiKeyStorageStatus | null;
+  // ── Toast notifications ──
+  extensionToasts: Array<{
+    id: string;
+    type: 'success' | 'error' | 'warning' | 'info';
+    title: string;
+    message?: string;
+  }>;
 
   // ── Skills catalog cache ──
   skillsUserCatalog: SkillCatalogItem[];
@@ -111,6 +109,14 @@ export interface ExtensionsSlice {
   applySkillImport: (request: SkillImportRequest) => Promise<SkillDetail | null>;
   deleteSkill: (request: SkillDeleteRequest) => Promise<void>;
 
+  // ── Toast actions ──
+  addExtensionToast: (
+    type: 'success' | 'error' | 'warning' | 'info',
+    title: string,
+    message?: string
+  ) => void;
+  dismissExtensionToast: (id: string) => void;
+
   // ── Mutation actions ──
   installPlugin: (request: PluginInstallRequest) => Promise<void>;
   uninstallPlugin: (pluginId: string, scope?: InstallScope, projectPath?: string) => Promise<void>;
@@ -123,12 +129,6 @@ export interface ExtensionsSlice {
     projectPath?: string
   ) => Promise<void>;
 
-  // ── API Keys actions ──
-  fetchApiKeys: () => Promise<void>;
-  fetchApiKeyStorageStatus: () => Promise<void>;
-  saveApiKey: (request: ApiKeySaveRequest) => Promise<void>;
-  deleteApiKey: (id: string) => Promise<void>;
-
   // ── Tab opener ──
   openExtensionsTab: () => void;
 
@@ -337,11 +337,6 @@ function getSkillsCatalogKey(projectPath?: string): string {
   return projectPath ?? USER_SKILLS_CATALOG_KEY;
 }
 
-function upsertApiKeyEntry(entries: ApiKeyEntry[], entry: ApiKeyEntry): ApiKeyEntry[] {
-  const nextEntries = entries.filter((candidate) => candidate.id !== entry.id);
-  return [entry, ...nextEntries];
-}
-
 /** Duration to show "success" state before returning to idle */
 const SUCCESS_DISPLAY_MS = 2_000;
 const PROJECT_SCOPE_REQUIRED_MESSAGE =
@@ -397,12 +392,7 @@ export const createExtensionsSlice: StateCreator<AppState, [], [], ExtensionsSli
   pluginInstallProgress: {},
   mcpInstallProgress: {},
   installErrors: {},
-
-  apiKeys: [],
-  apiKeysLoading: false,
-  apiKeysError: null,
-  apiKeySaving: false,
-  apiKeyStorageStatus: null,
+  extensionToasts: [],
 
   skillsUserCatalog: [],
   skillsProjectCatalogByProjectPath: {},
@@ -631,6 +621,11 @@ export const createExtensionsSlice: StateCreator<AppState, [], [], ExtensionsSli
         const diagnosticsRecord = Object.fromEntries(
           diagnostics.map((entry) => [getMcpDiagnosticKey(entry.name, entry.scope), entry] as const)
         );
+        const failedServers = diagnostics.filter((d) => d.status === 'failed');
+        if (failedServers.length > 0) {
+          const names = failedServers.map((s) => s.name).join(', ');
+          get().addExtensionToast('warning', 'MCP 连接异常', `${names} 连接失败`);
+        }
         const checkedAt = Date.now();
         set({
           mcpDiagnostics: diagnosticsRecord,
@@ -956,6 +951,11 @@ export const createExtensionsSlice: StateCreator<AppState, [], [], ExtensionsSli
       set((prev) => ({
         pluginInstallProgress: { ...prev.pluginInstallProgress, [operationKey]: 'success' },
       }));
+      get().addExtensionToast(
+        'success',
+        '插件已安装',
+        `已安装到 claudecode (${effectiveRequest.scope ?? 'user'})`
+      );
 
       // Refresh catalog to pick up new installed state
       void get().fetchPluginCatalog(
@@ -1183,6 +1183,7 @@ export const createExtensionsSlice: StateCreator<AppState, [], [], ExtensionsSli
       set((prev) => ({
         mcpInstallProgress: { ...prev.mcpInstallProgress, [progressKey]: 'success' },
       }));
+      get().addExtensionToast('success', 'MCP 服务器已安装', `已安装 ${request.serverName}`);
 
       scheduleMcpSuccessReset(progressKey, set);
     } catch (err) {
@@ -1264,6 +1265,7 @@ export const createExtensionsSlice: StateCreator<AppState, [], [], ExtensionsSli
       set((prev) => ({
         mcpInstallProgress: { ...prev.mcpInstallProgress, [operationKey]: 'success' },
       }));
+      get().addExtensionToast('success', 'MCP 服务器已卸载');
 
       scheduleMcpSuccessReset(operationKey, set);
     } catch (err) {
@@ -1276,92 +1278,25 @@ export const createExtensionsSlice: StateCreator<AppState, [], [], ExtensionsSli
     }
   },
 
-  // ── API Keys fetch ──
-  fetchApiKeys: async () => {
-    if (!api.apiKeys) return;
-
-    set({ apiKeysLoading: true, apiKeysError: null });
-    try {
-      const keys = await api.apiKeys.list();
-      set({ apiKeys: keys, apiKeysLoading: false });
-    } catch (err) {
-      set({
-        apiKeysLoading: false,
-        apiKeysError: err instanceof Error ? err.message : 'Failed to load API keys',
-      });
-    }
-  },
-
-  fetchApiKeyStorageStatus: async () => {
-    if (!api.apiKeys) return;
-    try {
-      const status = await api.apiKeys.getStorageStatus();
-      set({ apiKeyStorageStatus: status });
-    } catch {
-      // Non-critical — UI will just not show the info icon
-    }
-  },
-
-  // ── API Key save ──
-  saveApiKey: async (request: ApiKeySaveRequest) => {
-    if (!api.apiKeys) return;
-
-    set({ apiKeySaving: true, apiKeysError: null });
-    try {
-      const savedKey = await api.apiKeys.save(request);
-      const warnings: string[] = [];
-
-      try {
-        const keys = await api.apiKeys.list();
-        set({ apiKeys: keys });
-      } catch (listError) {
-        warnings.push(
-          listError instanceof Error
-            ? `API key saved, but failed to refresh key list. ${listError.message}`
-            : 'API key saved, but failed to refresh key list.'
-        );
+  // ── Toast notifications ──
+  addExtensionToast: (type, title, message) => {
+    const id = `toast-${Date.now()}-${Math.random().toString(36).slice(2, 8)}`;
+    set((prev) => ({
+      extensionToasts: [...prev.extensionToasts, { id, type, title, message }],
+    }));
+    if (type === 'success') {
+      setTimeout(() => {
         set((prev) => ({
-          apiKeys: upsertApiKeyEntry(prev.apiKeys, savedKey),
+          extensionToasts: prev.extensionToasts.filter((t) => t.id !== id),
         }));
-      }
-
-      await refreshConfiguredCliStatus(get());
-      const refreshError = get().cliStatusError;
-      if (refreshError) {
-        warnings.push(`API key saved, but failed to refresh provider status. ${refreshError}`);
-      }
-      set({ apiKeySaving: false, apiKeysError: warnings.length > 0 ? warnings.join(' ') : null });
-    } catch (err) {
-      set({
-        apiKeySaving: false,
-        apiKeysError: err instanceof Error ? err.message : 'Failed to save API key',
-      });
-      throw err; // Re-throw so the dialog can show the error
+      }, 3000);
     }
   },
 
-  // ── API Key delete ──
-  deleteApiKey: async (id: string) => {
-    if (!api.apiKeys) return;
-
-    try {
-      await api.apiKeys.delete(id);
-      set((prev) => ({
-        apiKeys: prev.apiKeys.filter((k) => k.id !== id),
-      }));
-      await refreshConfiguredCliStatus(get());
-      const refreshError = get().cliStatusError;
-      set({
-        apiKeysError: refreshError
-          ? `API key deleted, but failed to refresh provider status. ${refreshError}`
-          : null,
-      });
-    } catch (err) {
-      set({
-        apiKeysError: err instanceof Error ? err.message : 'Failed to delete API key',
-      });
-      throw err;
-    }
+  dismissExtensionToast: (id) => {
+    set((prev) => ({
+      extensionToasts: prev.extensionToasts.filter((t) => t.id !== id),
+    }));
   },
 
   // ── Tab opener ──

package/src/renderer/store/slices/scheduleSlice.ts

@@ -39,6 +39,9 @@ export interface ScheduleSlice {
 
   /** Open a standalone Schedules tab (or focus existing) */
   openSchedulesTab(): void;
+
+  /** Open the standalone task overview tab. */
+  openTasksTab(): void;
 }
 
 // =============================================================================
@@ -246,4 +249,22 @@ export const createScheduleSlice: StateCreator<AppState, [], [], ScheduleSlice>
     // Ensure schedules are fresh when opening
     void get().fetchSchedules();
   },
+
+  openTasksTab: () => {
+    const state = get();
+    const focusedPane = state.paneLayout.panes.find((p) => p.id === state.paneLayout.focusedPaneId);
+    const existingTab = focusedPane?.tabs.find((tab) => tab.type === 'tasks');
+    if (existingTab) {
+      state.setActiveTab(existingTab.id);
+      return;
+    }
+
+    state.openTab({
+      type: 'tasks',
+      label: '任务',
+    });
+
+    void get().fetchAllTasks();
+    void get().fetchSchedules();
+  },
 });

package/src/renderer/store/slices/teamSlice.ts

@@ -904,7 +904,9 @@ function pruneOptimisticMessages(
   canonical: readonly InboxMessage[]
 ): InboxMessage[] {
   if (optimistic.length === 0) {
-    return [];
+    // Preserve the input reference so selectTeamMessages' identity cache stays
+    // warm across no-op head refreshes (otherwise every poll churns `messages`).
+    return optimistic as InboxMessage[];
   }
 
   const canonicalIds = new Set(
@@ -913,10 +915,14 @@ function pruneOptimisticMessages(
       .filter((messageId) => messageId.length > 0)
   );
 
-  return optimistic.filter((message) => {
+  const pruned = optimistic.filter((message) => {
     const messageId = typeof message.messageId === 'string' ? message.messageId.trim() : '';
     return !messageId || !canonicalIds.has(messageId);
   });
+
+  // Nothing was actually pruned — return the original reference so downstream
+  // identity checks (merged-messages selector) can short-circuit re-renders.
+  return pruned.length === optimistic.length ? (optimistic as InboxMessage[]) : pruned;
 }
 
 function clearPendingReplyRefreshTimer(teamName: string): void {
@@ -2067,6 +2073,7 @@ export interface TeamSlice {
     delayMs?: number
   ) => void;
   sendTeamMessage: (teamName: string, request: SendMessageRequest) => Promise<SendMessageResult>;
+  addOptimisticTeamMessage: (teamName: string, message: InboxMessage) => void;
   crossTeamTargets: {
     teamName: string;
     displayName: string;
@@ -3988,6 +3995,39 @@ export const createTeamSlice: StateCreator<AppState, [], [], TeamSlice> = (set,
   },
 
   sendTeamMessage: async (teamName: string, request: SendMessageRequest) => {
+    const optimisticMessageId =
+      request.messageId ?? `optimistic-${Date.now()}-${Math.random().toString(36).slice(2, 8)}`;
+    const optimisticTimestamp = request.timestamp ?? nowIso();
+    const requestWithMessageId: SendMessageRequest = {
+      ...request,
+      messageId: optimisticMessageId,
+      timestamp: optimisticTimestamp,
+    };
+    const optimisticMessage: InboxMessage = {
+      from: requestWithMessageId.from ?? 'user',
+      to: requestWithMessageId.to ?? requestWithMessageId.member,
+      text: requestWithMessageId.text,
+      timestamp: optimisticTimestamp,
+      read: true,
+      taskRefs: requestWithMessageId.taskRefs?.length ? requestWithMessageId.taskRefs : undefined,
+      actionMode: requestWithMessageId.actionMode,
+      summary: requestWithMessageId.summary,
+      color: requestWithMessageId.color,
+      messageId: optimisticMessageId,
+      relayOfMessageId: requestWithMessageId.relayOfMessageId,
+      source: requestWithMessageId.source ?? 'user_sent',
+      attachments: requestWithMessageId.attachments?.length
+        ? requestWithMessageId.attachments
+        : undefined,
+      leadSessionId: requestWithMessageId.leadSessionId,
+      conversationId: requestWithMessageId.conversationId,
+      replyToConversationId: requestWithMessageId.replyToConversationId,
+      toolSummary: requestWithMessageId.toolSummary,
+      toolCalls: requestWithMessageId.toolCalls,
+      messageKind: requestWithMessageId.messageKind,
+      slashCommand: requestWithMessageId.slashCommand,
+      commandOutput: requestWithMessageId.commandOutput,
+    };
     set({
       sendingMessage: true,
       sendMessageError: null,
@@ -3995,35 +4035,25 @@ export const createTeamSlice: StateCreator<AppState, [], [], TeamSlice> = (set,
       sendMessageDebugDetails: null,
       lastSendMessageResult: null,
     });
+    set((state) => ({
+      teamMessagesByName: {
+        ...state.teamMessagesByName,
+        [teamName]: upsertOptimisticTeamMessage(
+          getTeamMessagesCacheEntry(state, teamName),
+          optimisticMessage
+        ),
+      },
+    }));
     try {
       const result = await unwrapIpc('team:sendMessage', () =>
-        api.teams.sendMessage(teamName, request)
+        api.teams.sendMessage(teamName, requestWithMessageId)
       );
       const runtimeDeliveryFailed =
         result.runtimeDelivery?.attempted === true && result.runtimeDelivery.delivered === false;
       const runtimeDeliveryDiagnostics = buildOpenCodeRuntimeDeliveryDiagnostics(result);
-      const optimisticMessage: InboxMessage = {
-        from: request.from ?? 'user',
-        to: request.to ?? request.member,
-        text: request.text,
-        timestamp: request.timestamp ?? nowIso(),
-        read: true,
-        taskRefs: request.taskRefs?.length ? request.taskRefs : undefined,
-        actionMode: request.actionMode,
-        summary: request.summary,
-        color: request.color,
+      const confirmedOptimisticMessage: InboxMessage = {
+        ...optimisticMessage,
         messageId: result.messageId,
-        relayOfMessageId: request.relayOfMessageId,
-        source: request.source ?? 'user_sent',
-        attachments: request.attachments?.length ? request.attachments : undefined,
-        leadSessionId: request.leadSessionId,
-        conversationId: request.conversationId,
-        replyToConversationId: request.replyToConversationId,
-        toolSummary: request.toolSummary,
-        toolCalls: request.toolCalls,
-        messageKind: request.messageKind,
-        slashCommand: request.slashCommand,
-        commandOutput: request.commandOutput,
       };
       set((state) => ({
         sendingMessage: false,
@@ -4035,7 +4065,7 @@ export const createTeamSlice: StateCreator<AppState, [], [], TeamSlice> = (set,
           ...state.teamMessagesByName,
           [teamName]: upsertOptimisticTeamMessage(
             getTeamMessagesCacheEntry(state, teamName),
-            optimisticMessage
+            confirmedOptimisticMessage
           ),
         },
       }));
@@ -4053,6 +4083,18 @@ export const createTeamSlice: StateCreator<AppState, [], [], TeamSlice> = (set,
     }
   },
 
+  addOptimisticTeamMessage: (teamName: string, message: InboxMessage) => {
+    set((state) => ({
+      teamMessagesByName: {
+        ...state.teamMessagesByName,
+        [teamName]: upsertOptimisticTeamMessage(
+          getTeamMessagesCacheEntry(state, teamName),
+          message
+        ),
+      },
+    }));
+  },
+
   fetchCrossTeamTargets: async () => {
     set({ crossTeamTargetsLoading: true });
     try {

package/src/renderer/types/tabs.ts

@@ -86,6 +86,7 @@ export interface Tab {
     | 'report'
     | 'extensions'
     | 'schedules'
+    | 'tasks'
     | 'graph';
 
   /** Session ID (required when type === 'session') */

package/src/shared/types/api.ts

@@ -46,6 +46,7 @@ import type {
   BoardTaskExactLogSummariesResponse,
   BoardTaskLogStreamResponse,
   BoardTaskLogStreamSummary,
+  CollabTask,
   CreateTaskRequest,
   CrossTeamMessage,
   CrossTeamSendRequest,
@@ -723,6 +724,33 @@ export interface CrossTeamAPI {
   getOutbox: (teamName: string) => Promise<CrossTeamMessage[]>;
 }
 
+// =============================================================================
+// Collaboration Board API
+// =============================================================================
+
+export interface CollabBoardAPI {
+  getBoard: () => Promise<{ tasks: CollabTask[] }>;
+  getTask: (dispatchId: string) => Promise<{ task: CollabTask }>;
+  getEvents: (dispatchId: string) => Promise<{ events: import('./team').CollabTaskEvent[] }>;
+  accept: (teamSlug: string, dispatchId: string) => Promise<{ ok: boolean; taskId: string }>;
+  reject: (teamSlug: string, dispatchId: string, reason?: string) => Promise<{ ok: boolean }>;
+  deliver: (teamSlug: string, dispatchId: string, result: string) => Promise<{ ok: boolean }>;
+  approve: (teamSlug: string, dispatchId: string) => Promise<{ ok: boolean }>;
+  revision: (teamSlug: string, dispatchId: string, feedback: string) => Promise<{ ok: boolean }>;
+  dispatch: (
+    fromTeam: string,
+    toTeam: string,
+    subject: string,
+    opts?: {
+      description?: string;
+      deadlineMinutes?: number;
+      needsHumanReview?: boolean;
+      messageId?: string;
+      sessionKey?: string;
+    }
+  ) => Promise<{ ok: boolean; dispatchId: string; status: string; message: string }>;
+}
+
 // =============================================================================
 // Schedule API
 // =============================================================================
@@ -1060,6 +1088,7 @@ export interface ElectronAPI extends RecentProjectsElectronApi {
 
   // Cross-Team Communication API
   crossTeam: CrossTeamAPI;
+  collab: CollabBoardAPI;
 
   // Review API
   review: ReviewAPI;
@@ -1093,6 +1122,35 @@ export interface ElectronAPI extends RecentProjectsElectronApi {
   // Extension Store — Skills Catalog API (Electron-only, optional)
   skills?: SkillsCatalogAPI;
 
+  // Extension Store — Credentials (project env, MCP credentials)
+  credentials?: {
+    getStatus: () => Promise<{ encryption: string; storagePath: string } | null>;
+    getProjectEnv: (projectPath: string) => Promise<Record<string, string>>;
+    saveProjectEnv: (projectPath: string, vars: Record<string, string>) => Promise<void>;
+    scanRequired: (
+      projectPath: string,
+      mcpServers: {
+        name: string;
+        envVars?: { name: string; isRequired: boolean; description?: string };
+      }[],
+      skillReqs: {
+        name: string;
+        envVars: { name: string; isRequired?: boolean; description?: string }[];
+      }[]
+    ) => Promise<{
+      required: {
+        name: string;
+        isRequired: boolean;
+        description?: string;
+        source: string;
+        value?: string;
+      }[];
+    }>;
+    resolveAgentEnv: (projectPath: string) => Promise<Record<string, string>>;
+    getSkillGlobalEnv: (skillFolderName: string) => Promise<Record<string, string>>;
+    saveSkillGlobalEnv: (skillFolderName: string, vars: Record<string, string>) => Promise<void>;
+  };
+
   // Extension Store — API Keys Management (Electron-only, optional)
   apiKeys?: ApiKeysAPI;
 

package/src/shared/types/extensions/index.ts

@@ -41,6 +41,7 @@ export type {
   CreateSkillRequest,
   DeleteSkillRequest,
   SkillCatalogItem,
+  SkillEnvVarDef,
   SkillDeleteRequest,
   SkillDetail,
   SkillDirectoryFlags,

package/src/shared/types/extensions/mcp.ts

@@ -111,6 +111,7 @@ export interface McpInstallRequest {
   projectPath?: string; // required for 'project' scope
   envValues: Record<string, string>;
   headers: McpHeaderDef[]; // for HTTP/SSE servers (CLI --header flag)
+  harnessType?: string; // which harness to install to (defaults to claudecode)
 }
 
 // ── Custom install request (bypasses registry, user provides spec) ──────────
@@ -122,6 +123,7 @@ export interface McpCustomInstallRequest {
   installSpec: McpInstallSpec; // user provides directly
   envValues: Record<string, string>;
   headers: McpHeaderDef[];
+  harnessType?: string; // which harness to install to
 }
 
 // ── Search result wrapper ──────────────────────────────────────────────────

package/src/shared/types/extensions/plugin.ts

@@ -14,7 +14,7 @@ export interface PluginCatalogItem {
   name: string; // display name only
 
   // Metadata
-  source: 'official';
+  source: 'official' | 'local'; // 'local' = discovered from a user-registered custom marketplace
   description: string;
   category: string; // open-ended string, derived from marketplace.json
   author?: { name: string; email?: string };
@@ -71,6 +71,7 @@ export interface PluginInstallRequest {
   pluginId: string; // canonical key — main resolves qualifiedName from catalog
   scope: InstallScope;
   projectPath?: string; // required for repo-scoped installs ('project' or 'local')
+  harnessType?: string; // which harness to install to (defaults to claudecode)
 }
 
 // ── Filters (renderer-only concern) ────────────────────────────────────────

package/src/shared/types/extensions/skill.ts

@@ -12,6 +12,12 @@ export type SkillInvocationMode = 'auto' | 'manual-only';
 
 export type SkillIssueSeverity = 'info' | 'warning' | 'error';
 
+export interface SkillEnvVarDef {
+  name: string;
+  description?: string;
+  isRequired?: boolean;
+}
+
 export interface SkillDirectoryFlags {
   hasScripts: boolean;
   hasReferences: boolean;
@@ -57,6 +63,7 @@ export interface SkillCatalogItem {
   isValid: boolean;
   issues: SkillValidationIssue[];
   modifiedAt: number;
+  requiredEnv?: SkillEnvVarDef[];
 }
 
 export interface SkillDetail {