@yancyyu/openhermit 1.6.42 → 1.6.43

package/src/renderer/components/common/TerminalPane.tsx

@@ -1,213 +0,0 @@
-/**
- * TerminalPane - Reusable xterm.js terminal pane.
- *
- * Extracted from SystemManagerView so any team or panel can embed
- * a terminal connected to a PTY process.
- *
- * Lifecycle:
- *  - Mount: creates Terminal + FitAddon, registers SSE listeners
- *  - `spawn()` call or autoSpawn: starts a CLI process via the terminal API
- *  - Unmount: kills PTY, disposes terminal
- */
-import React, { useCallback, useEffect, useImperativeHandle, useRef } from 'react';
-import { FitAddon } from '@xterm/addon-fit';
-import { WebLinksAddon } from '@xterm/addon-web-links';
-import { Terminal } from '@xterm/xterm';
-
-import { api } from '@renderer/api';
-import { cn } from '@renderer/lib/utils';
-
-// =============================================================================
-// Types
-// =============================================================================
-
-export interface TerminalSpawnOptions {
-  command: string;
-  args: string[];
-  cwd: string;
-}
-
-interface TerminalPaneProps {
-  /** Extra class names for the container */
-  className?: string;
-  /** Called after a PTY is spawned successfully */
-  onSpawned?: (ptyId: string) => void;
-  /** Called when the PTY exits */
-  onExit?: (ptyId: string, exitCode: number) => void;
-  /** Auto-spawn config. When provided, auto-spawns on mount. */
-  autoSpawn?: TerminalSpawnOptions;
-}
-
-export interface TerminalPaneRef {
-  spawn: (options: TerminalSpawnOptions) => Promise<void>;
-}
-
-// =============================================================================
-// TerminalPane
-// =============================================================================
-
-export const TerminalPane = React.forwardRef<TerminalPaneRef, TerminalPaneProps>(
-  function TerminalPane({ className, onSpawned, onExit, autoSpawn }, ref) {
-    const hostRef = useRef<HTMLDivElement>(null);
-    const terminalRef = useRef<Terminal | null>(null);
-    const fitAddonRef = useRef<FitAddon | null>(null);
-    const ptyIdRef = useRef<string | null>(null);
-    const spawnedRef = useRef(false);
-
-    const fitTerminal = useCallback(() => {
-      try {
-        fitAddonRef.current?.fit();
-        if (ptyIdRef.current && terminalRef.current) {
-          api.terminal.resize(ptyIdRef.current, terminalRef.current.cols, terminalRef.current.rows);
-        }
-      } catch {
-        // xterm fit can throw when the element is not measurable yet
-      }
-    }, []);
-
-    // Expose spawn method via ref
-    useImperativeHandle(ref, () => ({
-      spawn: async (options: TerminalSpawnOptions) => {
-        // Kill existing PTY if any
-        if (ptyIdRef.current) {
-          try { await api.terminal.kill(ptyIdRef.current); } catch {}
-          ptyIdRef.current = null;
-        }
-        // Clear stale terminal content before spawning new process
-        terminalRef.current?.clear();
-        try {
-          const ptyId = await api.terminal.spawn(options);
-          ptyIdRef.current = ptyId;
-          spawnedRef.current = true;
-          fitTerminal();
-          onSpawned?.(ptyId);
-        } catch (err) {
-          terminalRef.current?.writeln(
-            `\x1b[31m[Failed to spawn: ${err instanceof Error ? err.message : String(err)}]\x1b[0m`
-          );
-        }
-      },
-    }), [fitTerminal, onSpawned]);
-
-    // Create terminal instance
-    useEffect(() => {
-      const host = hostRef.current;
-      if (!host) return;
-
-      const term = new Terminal({
-        cursorBlink: true,
-        convertEol: true,
-        fontFamily: 'JetBrains Mono, SFMono-Regular, Menlo, Monaco, Consolas, monospace',
-        fontSize: 13,
-        lineHeight: 1.28,
-        theme: {
-          background: 'var(--color-surface)',
-          foreground: 'var(--color-text)',
-          cursor: 'var(--color-text)',
-          selectionBackground: 'var(--color-border-emphasis)',
-          black: 'var(--color-surface-sidebar)',
-          red: '#f87171',
-          green: '#86efac',
-          yellow: '#fde68a',
-          blue: 'var(--color-accent)',
-          magenta: '#d8b4fe',
-          cyan: '#67e8f9',
-          white: 'var(--color-text)',
-        },
-      });
-
-      const fitAddon = new FitAddon();
-      term.loadAddon(fitAddon);
-      term.loadAddon(new WebLinksAddon());
-      term.open(host);
-      terminalRef.current = term;
-      fitAddonRef.current = fitAddon;
-      fitTerminal();
-
-      // SSE listeners
-      const dataDispose = api.terminal.onData((_event, ptyId, data) => {
-        if (ptyId === ptyIdRef.current) term.write(data);
-      });
-      const exitDispose = api.terminal.onExit((_event, ptyId, exitCode) => {
-        if (ptyId === ptyIdRef.current) {
-          term.writeln(`\r\n\x1b[90m[process exited with code ${exitCode}]\x1b[0m`);
-          onExit?.(ptyId, exitCode);
-          ptyIdRef.current = null;
-          spawnedRef.current = false;
-        }
-      });
-
-      // Input forwarding
-      const inputDispose = term.onData((data) => {
-        if (ptyIdRef.current) api.terminal.write(ptyIdRef.current, data);
-      });
-
-      // Resize observer
-      const resizeObserver = new ResizeObserver(() => fitTerminal());
-      resizeObserver.observe(host);
-
-      return () => {
-        dataDispose();
-        exitDispose();
-        inputDispose.dispose();
-        resizeObserver.disconnect();
-        if (ptyIdRef.current) {
-          void api.terminal.kill(ptyIdRef.current).catch(() => {});
-        }
-        term.dispose();
-        terminalRef.current = null;
-        fitAddonRef.current = null;
-        ptyIdRef.current = null;
-      };
-    }, [fitTerminal, onExit]);
-
-    // Auto-spawn
-    // Two issues combined to prevent the CLI from ever spawning:
-    //  1. React StrictMode mounts → unmounts → remounts. The first mount set
-    //     spawnedRef = true and started a 200ms timer; cleanup cleared the timer
-    //     but NOT the ref. The second mount saw spawnedRef === true → skipped.
-    //  2. The parent passes an inline object literal as autoSpawn, producing a
-    //     new reference each render. With the raw object in deps, every render
-    //     re-ran the effect, whose cleanup cleared the timeout. spawnedRef was
-    //     already true, so the retry path was blocked.
-    // Fix: reset spawnedRef in cleanup so StrictMode remount can retry, and use
-    // a stable string key derived from autoSpawn content instead of the object.
-    const autoSpawnKey = autoSpawn
-      ? `${autoSpawn.command}\0${autoSpawn.args.join(',')}\0${autoSpawn.cwd}`
-      : undefined;
-
-    useEffect(() => {
-      if (!autoSpawnKey || spawnedRef.current) return;
-      spawnedRef.current = true;
-
-      const spawnOpts = autoSpawn!;
-
-      const doSpawn = async () => {
-        try {
-          const ptyId = await api.terminal.spawn(spawnOpts);
-          ptyIdRef.current = ptyId;
-          fitTerminal();
-          onSpawned?.(ptyId);
-        } catch (err) {
-          terminalRef.current?.writeln(
-            `\x1b[31m[Failed to spawn: ${err instanceof Error ? err.message : String(err)}]\x1b[0m`
-          );
-        }
-      };
-
-      const timer = setTimeout(doSpawn, 200);
-      return () => {
-        clearTimeout(timer);
-        spawnedRef.current = false;
-      };
-      // autoSpawnKey is a stable primitive; autoSpawn is captured via closure.
-      // eslint-disable-next-line react-hooks/exhaustive-deps
-    }, [autoSpawnKey, fitTerminal, onSpawned]);
-
-    return (
-      <div className={cn('size-full overflow-hidden', className)}>
-        <div ref={hostRef} className="size-full" />
-      </div>
-    );
-  }
-);

package/src/renderer/components/team/dialogs/useTeamEditForm.ts

@@ -1,292 +0,0 @@
-import { useEffect, useMemo, useRef, useState } from 'react';
-
-import { api } from '@renderer/api';
-import { useStore } from '@renderer/store';
-import { isTeamProvisioningActive } from '@renderer/store/slices/teamSlice';
-import { useShallow } from 'zustand/react/shallow';
-
-import type { GlobalProvider } from '@shared/types';
-import type { CcAgentType } from '@shared/types/ccConnect';
-
-type SavePhase = 'idle' | 'saving' | 'done';
-
-export interface UseTeamEditFormReturn {
-  loading: boolean;
-  isProvisioning: boolean;
-  name: string;
-  setName: (v: string) => void;
-  description: string;
-  setDescription: (v: string) => void;
-  agentType: string;
-  setAgentType: (v: string) => void;
-  permissionMode: string;
-  setPermissionMode: (v: string) => void;
-  workDir: string;
-  setWorkDir: (v: string) => void;
-  language: string;
-  setLanguage: (v: string) => void;
-  managedSources: string;
-  setManagedSources: (v: string) => void;
-  feishuAllowFrom: string;
-  setFeishuAllowFrom: (v: string) => void;
-  feishuAllowChat: string;
-  setFeishuAllowChat: (v: string) => void;
-  disabledCommandsInput: string;
-  setDisabledCommandsInput: (v: string) => void;
-  providerRef: string;
-  setProviderRef: (v: string) => void;
-  showContextIndicator: boolean;
-  setShowContextIndicator: (v: boolean) => void;
-  replyFooter: boolean;
-  setReplyFooter: (v: boolean) => void;
-  injectSender: boolean;
-  setInjectSender: (v: boolean) => void;
-  color: string;
-  compatibleProviders: GlobalProvider[];
-  canDelete: boolean;
-  /** Current phase of the save+restart lifecycle */
-  savePhase: SavePhase;
-  /** true while saving or restarting (button spinner / disable) */
-  saving: boolean;
-  error: string | null;
-  clearError: () => void;
-  handleSave: () => void;
-}
-
-const PERMISSION_MODE_OPTIONS = [
-  { value: 'default', label: '默认' },
-  { value: 'acceptEdits', label: '自动接受编辑' },
-  { value: 'bypassPermissions', label: '跳过权限确认' },
-  { value: 'plan', label: '计划模式' },
-] as const;
-
-export { PERMISSION_MODE_OPTIONS };
-
-export function useTeamEditForm(teamName: string, open: boolean): UseTeamEditFormReturn {
-  // ── Store reads ──────────────────────────────────────────────
-  const { data, fetchTeams, selectTeam } = useStore(
-    useShallow((s) => ({
-      data: s.selectedTeamName === teamName ? s.selectedTeamData : null,
-      fetchTeams: s.fetchTeams,
-      selectTeam: s.selectTeam,
-    }))
-  );
-  const isProvisioning = useStore((s) => isTeamProvisioningActive(s, teamName));
-
-  // ── Derived defaults ─────────────────────────────────────────
-  const rawSettings = useMemo(
-    () => (data?.settings ?? {}) as Record<string, unknown>,
-    [data?.settings]
-  );
-
-  const defaults = useMemo(() => {
-    const cfg = data?.config;
-    const d = data as Record<string, unknown> | null;
-    return {
-      name: cfg?.name ?? '',
-      description: cfg?.description ?? '',
-      color: cfg?.color ?? '',
-      agentType: cfg?.agentType ?? (d?.harness as string | undefined) ?? 'cursor',
-      workDir: (d?.workDir as string | undefined) ?? cfg?.projectPath ?? '',
-      permissionMode: cfg?.permissionMode ?? (d?.permissionMode as string | undefined) ?? 'default',
-      language:
-        cfg?.language ?? (typeof rawSettings.language === 'string' ? rawSettings.language : 'zh'),
-      managedSources:
-        cfg?.managedSources ??
-        (typeof rawSettings.admin_from === 'string' ? rawSettings.admin_from : '*'),
-      disabledCommands: Array.isArray(cfg?.disabledCommands)
-        ? cfg.disabledCommands
-        : Array.isArray(rawSettings.disabled_commands)
-          ? (rawSettings.disabled_commands as unknown[]).filter(
-              (entry): entry is string => typeof entry === 'string' && entry.trim().length > 0
-            )
-          : [],
-      platformAllowFrom:
-        cfg?.platformAllowFrom ??
-        (typeof rawSettings.platform_allow_from === 'object' &&
-        rawSettings.platform_allow_from !== null &&
-        !Array.isArray(rawSettings.platform_allow_from)
-          ? (rawSettings.platform_allow_from as Record<string, string>)
-          : {}),
-      platformAllowChat:
-        cfg?.platformAllowChat ??
-        (typeof (rawSettings as Record<string, unknown>).platform_allow_chat === 'object' &&
-        (rawSettings as Record<string, unknown>).platform_allow_chat !== null &&
-        !Array.isArray((rawSettings as Record<string, unknown>).platform_allow_chat)
-          ? ((rawSettings as Record<string, unknown>).platform_allow_chat as Record<string, string>)
-          : {}),
-      providerRefs: data?.providerRefs ?? [],
-      globalProviders: data?.globalProviders ?? [],
-      showContextIndicator:
-        cfg?.showContextIndicator ??
-        (typeof rawSettings.show_context_indicator === 'boolean'
-          ? rawSettings.show_context_indicator
-          : true),
-      replyFooter:
-        cfg?.replyFooter ??
-        (typeof rawSettings.reply_footer === 'boolean' ? rawSettings.reply_footer : true),
-      injectSender:
-        cfg?.injectSender ??
-        (typeof rawSettings.inject_sender === 'boolean' ? rawSettings.inject_sender : false),
-    };
-  }, [data, rawSettings]);
-
-  // ── Local form state ─────────────────────────────────────────
-  const [name, setName] = useState(defaults.name);
-  const [description, setDescription] = useState(defaults.description);
-  const [agentType, setAgentType] = useState(defaults.agentType);
-  const [permissionMode, setPermissionMode] = useState(defaults.permissionMode);
-  const [workDir, setWorkDir] = useState(defaults.workDir);
-  const [language, setLanguage] = useState(defaults.language);
-  const [managedSources, setManagedSources] = useState(defaults.managedSources);
-  const [disabledCommandsInput, setDisabledCommandsInput] = useState(
-    defaults.disabledCommands.join(', ')
-  );
-  const [feishuAllowFrom, setFeishuAllowFrom] = useState(defaults.platformAllowFrom.feishu ?? '*');
-  const [feishuAllowChat, setFeishuAllowChat] = useState(defaults.platformAllowChat.feishu ?? '*');
-  const [providerRef, setProviderRef] = useState(defaults.providerRefs[0] ?? '');
-  const [showContextIndicator, setShowContextIndicator] = useState(defaults.showContextIndicator);
-  const [replyFooter, setReplyFooter] = useState(defaults.replyFooter);
-  const [injectSender, setInjectSender] = useState(defaults.injectSender);
-
-  // ── Single async lifecycle state ─────────────────────────────
-  const [savePhase, setSavePhase] = useState<SavePhase>('idle');
-  const [error, setError] = useState<string | null>(null);
-  const saving = savePhase === 'saving';
-
-  // ── Refs ─────────────────────────────────────────────────────
-  const defaultsRef = useRef(defaults);
-  if (defaults.name) {
-    defaultsRef.current = defaults;
-  }
-
-  // ── Reset form when dialog opens ─────────────────────────────
-  const prevOpenRef = useRef(false);
-  useEffect(() => {
-    if (!open || prevOpenRef.current) {
-      prevOpenRef.current = open;
-      return;
-    }
-    prevOpenRef.current = true;
-    const d = defaultsRef.current;
-    setSavePhase('idle');
-    setError(null);
-    setName(d.name);
-    setDescription(d.description);
-    setAgentType(d.agentType);
-    setPermissionMode(d.permissionMode);
-    setWorkDir(d.workDir);
-    setLanguage(d.language);
-    setManagedSources(d.managedSources);
-    setDisabledCommandsInput(d.disabledCommands.join(', '));
-    setFeishuAllowFrom(d.platformAllowFrom.feishu ?? '*');
-    setFeishuAllowChat(d.platformAllowChat.feishu ?? '*');
-    setProviderRef(d.providerRefs[0] ?? '');
-    setShowContextIndicator(d.showContextIndicator);
-    setReplyFooter(d.replyFooter);
-    setInjectSender(d.injectSender);
-  }, [open]);
-
-  // ── Computed values ──────────────────────────────────────────
-  const compatibleProviders = useMemo(
-    () =>
-      defaults.globalProviders.filter(
-        (p) =>
-          !p.agent_types ||
-          p.agent_types.length === 0 ||
-          (p.agent_types as string[]).includes(agentType)
-      ),
-    [defaults.globalProviders, agentType]
-  );
-  const canDelete = teamName !== 'default' && teamName !== 'my-project';
-
-  // ── Actions ──────────────────────────────────────────────────
-  const clearError = (): void => setError(null);
-
-  const handleSave = (): void => {
-    if (!name.trim()) {
-      setError('团队名称不能为空');
-      return;
-    }
-    if (savePhase !== 'idle') return;
-
-    const disabledCommands = disabledCommandsInput
-      .split(',')
-      .map((e) => e.trim())
-      .filter((e) => e.length > 0);
-    const feishu = feishuAllowFrom.trim();
-    const feishuChat = feishuAllowChat.trim();
-
-    setSavePhase('saving');
-    setError(null);
-
-    void (async () => {
-      try {
-        await api.teams.updateConfig(teamName, {
-          name: name.trim(),
-          description: description.trim(),
-          color: defaultsRef.current.color,
-          agentType: agentType.trim() || undefined,
-          workDir: workDir.trim() || undefined,
-          permissionMode: permissionMode.trim() || undefined,
-          showContextIndicator,
-          replyFooter,
-          injectSender,
-          language: language.trim() || undefined,
-          managedSources: managedSources.trim() || undefined,
-          disabledCommands,
-          platformAllowFrom: feishu ? { feishu } : {},
-          platformAllowChat: feishuChat ? { feishu: feishuChat } : {},
-          providerRefs: providerRef ? [providerRef] : [],
-        });
-
-        await Promise.all([fetchTeams(), selectTeam(teamName)]);
-        setSavePhase('done');
-      } catch (err) {
-        setError(err instanceof Error ? err.message : '保存失败');
-        setSavePhase('idle');
-      }
-    })();
-  };
-
-  return {
-    loading: !data,
-    isProvisioning,
-    name,
-    setName,
-    description,
-    setDescription,
-    agentType,
-    setAgentType,
-    permissionMode,
-    setPermissionMode,
-    workDir,
-    setWorkDir,
-    language,
-    setLanguage,
-    managedSources,
-    setManagedSources,
-    feishuAllowFrom,
-    setFeishuAllowFrom,
-    feishuAllowChat,
-    setFeishuAllowChat,
-    disabledCommandsInput,
-    setDisabledCommandsInput,
-    providerRef,
-    setProviderRef,
-    showContextIndicator,
-    setShowContextIndicator,
-    replyFooter,
-    setReplyFooter,
-    injectSender,
-    setInjectSender,
-    color: defaults.color,
-    compatibleProviders,
-    canDelete,
-    savePhase,
-    saving,
-    error,
-    clearError,
-    handleSave,
-  };
-}