@yancyyu/openhermit 1.5.8

package/src/renderer/components/settings/NotificationTriggerSettings/hooks/useAddTriggerFormHandlers.ts

@@ -0,0 +1,218 @@
+/**
+ * Hook for AddTriggerForm event handlers.
+ * Extracts handler logic from AddTriggerForm for mode changes, content type changes, etc.
+ */
+
+import { useCallback } from 'react';
+
+import { getAvailableMatchFields } from '../utils/trigger';
+
+import type { AddTriggerFormStateReturn } from './useAddTriggerFormState';
+import type { RepositoryDropdownItem } from '@renderer/components/settings/hooks/useSettingsConfig';
+import type {
+  NotificationTrigger,
+  TriggerContentType,
+  TriggerMatchField,
+  TriggerMode,
+} from '@renderer/types/data';
+
+interface UseAddTriggerFormHandlersOptions {
+  formState: AddTriggerFormStateReturn;
+  validatePattern: (pattern: string) => boolean;
+  clearPreview: () => void;
+}
+
+export interface AddTriggerFormHandlersReturn {
+  handleModeChange: (newMode: TriggerMode) => void;
+  handleContentTypeChange: (newContentType: TriggerContentType) => void;
+  handleToolNameChange: (newToolName: string) => void;
+  handleAddRepository: (item: RepositoryDropdownItem) => void;
+  handleRemoveIgnorePattern: (idx: number) => void;
+  handleAddIgnorePattern: (pattern: string) => void;
+  handleRemoveRepository: (idx: number) => void;
+  handleMatchPatternChange: (value: string) => void;
+  handleTokenThresholdChange: (value: string) => void;
+  handleCancel: () => void;
+  buildNewTrigger: (generateId: () => string) => Omit<NotificationTrigger, 'isBuiltin'>;
+}
+
+/**
+ * Hook for managing AddTriggerForm event handlers.
+ */
+export function useAddTriggerFormHandlers({
+  formState,
+  validatePattern,
+  clearPreview,
+}: UseAddTriggerFormHandlersOptions): AddTriggerFormHandlersReturn {
+  const {
+    name,
+    toolName,
+    mode,
+    contentType,
+    matchField,
+    matchPattern,
+    tokenThreshold,
+    tokenType,
+    ignorePatterns,
+    repositoryIds,
+    color,
+    setMode,
+    setContentType,
+    setToolName,
+    setMatchField,
+    setMatchPattern,
+    setTokenThreshold,
+    setIgnorePatterns,
+    setRepositoryIds,
+    setIsExpanded,
+    resetForm,
+  } = formState;
+
+  // When mode changes, adjust content type defaults
+  const handleModeChange = useCallback(
+    (newMode: TriggerMode) => {
+      setMode(newMode);
+      if (newMode === 'error_status') {
+        setContentType('tool_result');
+      }
+    },
+    [setMode, setContentType]
+  );
+
+  // When content type changes, reset matchField to first available option
+  const handleContentTypeChange = useCallback(
+    (newContentType: TriggerContentType) => {
+      setContentType(newContentType);
+      const newMatchFields = getAvailableMatchFields(newContentType, toolName || undefined);
+      setMatchField(newMatchFields[0]?.value || '');
+      // Reset tool name if not applicable
+      if (newContentType !== 'tool_use' && newContentType !== 'tool_result') {
+        setToolName('');
+      }
+    },
+    [toolName, setContentType, setMatchField, setToolName]
+  );
+
+  // When tool name changes, reset matchField to first available option
+  const handleToolNameChange = useCallback(
+    (newToolName: string) => {
+      setToolName(newToolName);
+      const newMatchFields = getAvailableMatchFields(contentType, newToolName || undefined);
+      setMatchField(newMatchFields[0]?.value || '');
+    },
+    [contentType, setToolName, setMatchField]
+  );
+
+  // Handler for adding repository
+  const handleAddRepository = useCallback(
+    (item: RepositoryDropdownItem) => {
+      if (!repositoryIds.includes(item.id)) {
+        setRepositoryIds([...repositoryIds, item.id]);
+      }
+    },
+    [repositoryIds, setRepositoryIds]
+  );
+
+  // Handler for removing ignore pattern
+  const handleRemoveIgnorePattern = useCallback(
+    (idx: number) => {
+      const newPatterns = [...ignorePatterns];
+      newPatterns.splice(idx, 1);
+      setIgnorePatterns(newPatterns);
+    },
+    [ignorePatterns, setIgnorePatterns]
+  );
+
+  // Handler for adding ignore pattern
+  const handleAddIgnorePattern = useCallback(
+    (pattern: string) => {
+      setIgnorePatterns([...ignorePatterns, pattern]);
+    },
+    [ignorePatterns, setIgnorePatterns]
+  );
+
+  // Handler for removing repository
+  const handleRemoveRepository = useCallback(
+    (idx: number) => {
+      const newIds = [...repositoryIds];
+      newIds.splice(idx, 1);
+      setRepositoryIds(newIds);
+    },
+    [repositoryIds, setRepositoryIds]
+  );
+
+  // Handler for match pattern change with validation
+  const handleMatchPatternChange = useCallback(
+    (value: string) => {
+      setMatchPattern(value);
+      validatePattern(value);
+    },
+    [setMatchPattern, validatePattern]
+  );
+
+  // Handler for token threshold change
+  const handleTokenThresholdChange = useCallback(
+    (value: string) => {
+      const val = value.replace(/\D/g, '');
+      setTokenThreshold(parseInt(val) || 0);
+    },
+    [setTokenThreshold]
+  );
+
+  // Handler for cancel button
+  const handleCancel = useCallback(() => {
+    resetForm();
+    clearPreview();
+    setIsExpanded(false);
+  }, [resetForm, clearPreview, setIsExpanded]);
+
+  // Build new trigger object from form state
+  const buildNewTrigger = useCallback(
+    (generateId: () => string): Omit<NotificationTrigger, 'isBuiltin'> => {
+      return {
+        id: `custom-${generateId()}`,
+        name: name.trim(),
+        enabled: true,
+        contentType,
+        mode,
+        ...(mode === 'error_status' && { requireError: true }),
+        ...(mode === 'content_match' &&
+          matchField && { matchField: matchField as TriggerMatchField }),
+        ...(mode === 'content_match' && matchPattern && { matchPattern }),
+        ...(mode === 'token_threshold' && { tokenThreshold, tokenType }),
+        ...((contentType === 'tool_use' || contentType === 'tool_result') &&
+          toolName && { toolName }),
+        ...(ignorePatterns.length > 0 && { ignorePatterns }),
+        ...(repositoryIds.length > 0 && { repositoryIds }),
+        color,
+      };
+    },
+    [
+      name,
+      contentType,
+      mode,
+      matchField,
+      matchPattern,
+      tokenThreshold,
+      tokenType,
+      toolName,
+      ignorePatterns,
+      repositoryIds,
+      color,
+    ]
+  );
+
+  return {
+    handleModeChange,
+    handleContentTypeChange,
+    handleToolNameChange,
+    handleAddRepository,
+    handleRemoveIgnorePattern,
+    handleAddIgnorePattern,
+    handleRemoveRepository,
+    handleMatchPatternChange,
+    handleTokenThresholdChange,
+    handleCancel,
+    buildNewTrigger,
+  };
+}

package/src/renderer/components/settings/NotificationTriggerSettings/hooks/useAddTriggerFormState.ts

@@ -0,0 +1,135 @@
+/**
+ * Hook for AddTriggerForm state management.
+ * Extracts all useState calls and resetForm logic from AddTriggerForm.
+ */
+
+import { useCallback, useState } from 'react';
+
+import type { TriggerContentType, TriggerMode, TriggerTokenType } from '@renderer/types/data';
+import type { TriggerColor } from '@shared/constants/triggerColors';
+
+interface AddTriggerFormState {
+  // Section 1: General Info
+  name: string;
+  toolName: string;
+
+  // Section 2: Trigger Condition
+  mode: TriggerMode;
+
+  // Section 3: Dynamic Configuration
+  // Content match settings
+  contentType: TriggerContentType;
+  matchField: string;
+  matchPattern: string;
+
+  // Token threshold settings
+  tokenThreshold: number;
+  tokenType: TriggerTokenType;
+
+  // Section 4: Advanced
+  ignorePatterns: string[];
+
+  // Section 5: Repository Scope
+  repositoryIds: string[];
+
+  // Display
+  color: TriggerColor;
+
+  // UI state
+  isExpanded: boolean;
+}
+
+export interface AddTriggerFormStateReturn extends AddTriggerFormState {
+  setName: (name: string) => void;
+  setToolName: (toolName: string) => void;
+  setMode: (mode: TriggerMode) => void;
+  setContentType: (contentType: TriggerContentType) => void;
+  setMatchField: (matchField: string) => void;
+  setMatchPattern: (matchPattern: string) => void;
+  setTokenThreshold: (threshold: number) => void;
+  setTokenType: (tokenType: TriggerTokenType) => void;
+  setIgnorePatterns: (patterns: string[]) => void;
+  setRepositoryIds: (ids: string[]) => void;
+  setColor: (color: TriggerColor) => void;
+  setIsExpanded: (expanded: boolean) => void;
+  resetForm: () => void;
+}
+
+/**
+ * Hook for managing AddTriggerForm state.
+ */
+export function useAddTriggerFormState(): AddTriggerFormStateReturn {
+  // Section 1: General Info
+  const [name, setName] = useState('');
+  const [toolName, setToolName] = useState<string>('');
+
+  // Section 2: Trigger Condition
+  const [mode, setMode] = useState<TriggerMode>('error_status');
+
+  // Section 3: Dynamic Configuration
+  // Content match settings
+  const [contentType, setContentType] = useState<TriggerContentType>('tool_result');
+  const [matchField, setMatchField] = useState<string>('content');
+  const [matchPattern, setMatchPattern] = useState('');
+
+  // Token threshold settings
+  const [tokenThreshold, setTokenThreshold] = useState<number>(1000);
+  const [tokenType, setTokenType] = useState<TriggerTokenType>('total');
+
+  // Section 4: Advanced
+  const [ignorePatterns, setIgnorePatterns] = useState<string[]>([]);
+
+  // Section 5: Repository Scope
+  const [repositoryIds, setRepositoryIds] = useState<string[]>([]);
+
+  // Display
+  const [color, setColor] = useState<TriggerColor>('red');
+
+  // UI state
+  const [isExpanded, setIsExpanded] = useState(false);
+
+  const resetForm = useCallback(() => {
+    setName('');
+    setToolName('');
+    setMode('error_status');
+    setContentType('tool_result');
+    setMatchField('content');
+    setMatchPattern('');
+    setTokenThreshold(1000);
+    setTokenType('total');
+    setIgnorePatterns([]);
+    setRepositoryIds([]);
+    // Intentionally do NOT reset color — preserve last-used color across triggers
+  }, []);
+
+  return {
+    // State values
+    name,
+    toolName,
+    mode,
+    contentType,
+    matchField,
+    matchPattern,
+    tokenThreshold,
+    tokenType,
+    ignorePatterns,
+    repositoryIds,
+    color,
+    isExpanded,
+
+    // Setters
+    setName,
+    setToolName,
+    setMode,
+    setContentType,
+    setMatchField,
+    setMatchPattern,
+    setTokenThreshold,
+    setTokenType,
+    setIgnorePatterns,
+    setRepositoryIds,
+    setColor,
+    setIsExpanded,
+    resetForm,
+  };
+}

package/src/renderer/components/settings/NotificationTriggerSettings/hooks/useRepositoryLookup.ts

@@ -0,0 +1,47 @@
+/**
+ * Hook to convert repository IDs to RepositoryDropdownItem[] for display.
+ * Used by TriggerCard and AddTriggerForm to show selected repositories.
+ */
+
+import { useMemo } from 'react';
+
+import { useStore } from '@renderer/store';
+
+import type { RepositoryDropdownItem } from '@renderer/components/settings/hooks/useSettingsConfig';
+
+/**
+ * Converts an array of repository IDs to RepositoryDropdownItem[] for display.
+ * Searches repository groups to find matching repositories.
+ */
+export function useRepositoryLookup(repositoryIds: string[]): RepositoryDropdownItem[] {
+  const repositoryGroups = useStore((state) => state.repositoryGroups);
+
+  return useMemo((): RepositoryDropdownItem[] => {
+    const items: RepositoryDropdownItem[] = [];
+
+    for (const repositoryId of repositoryIds) {
+      // Find repository group by ID
+      const group = repositoryGroups.find((g) => g.id === repositoryId);
+      if (group) {
+        items.push({
+          id: group.id,
+          name: group.name,
+          path: group.worktrees[0]?.path ?? '',
+          worktreeCount: group.worktrees.length,
+          totalSessions: group.totalSessions,
+        });
+      } else {
+        // If not found, create a placeholder item
+        items.push({
+          id: repositoryId,
+          name: repositoryId,
+          path: '',
+          worktreeCount: 0,
+          totalSessions: 0,
+        });
+      }
+    }
+
+    return items;
+  }, [repositoryIds, repositoryGroups]);
+}

package/src/renderer/components/settings/NotificationTriggerSettings/hooks/useTriggerCardState.ts

@@ -0,0 +1,281 @@
+/**
+ * Hook for TriggerCard local state and callback handlers.
+ * Extracts state management logic from TriggerCard component.
+ */
+
+import { useCallback, useState } from 'react';
+
+import { deriveMode, getAvailableMatchFields } from '../utils/trigger';
+
+import type { RepositoryDropdownItem } from '@renderer/components/settings/hooks/useSettingsConfig';
+import type {
+  NotificationTrigger,
+  TriggerContentType,
+  TriggerMatchField,
+  TriggerMode,
+  TriggerTokenType,
+} from '@renderer/types/data';
+import type { TriggerColor } from '@shared/constants/triggerColors';
+
+interface UseTriggerCardStateOptions {
+  trigger: NotificationTrigger;
+  onUpdate: (updates: Partial<NotificationTrigger>) => Promise<void>;
+  validatePattern: (pattern: string) => boolean;
+}
+
+interface UseTriggerCardStateReturn {
+  // UI state
+  isExpanded: boolean;
+  setIsExpanded: (value: boolean) => void;
+  editingName: boolean;
+  setEditingName: (value: boolean) => void;
+
+  // Local form values
+  localName: string;
+  setLocalName: (value: string) => void;
+  localPattern: string;
+  localMode: TriggerMode;
+  localTokenThreshold: number;
+  localTokenType: TriggerTokenType;
+
+  // Handlers
+  handleToggleEnabled: () => void;
+  handleNameSave: () => void;
+  handlePatternBlur: () => void;
+  handlePatternChange: (value: string) => void;
+  handleContentTypeChange: (value: TriggerContentType) => void;
+  handleToolNameChange: (value: string) => void;
+  handleMatchFieldChange: (value: string) => void;
+  handleModeChange: (newMode: TriggerMode) => void;
+  handleTokenThresholdChange: (value: number) => void;
+  handleTokenThresholdBlur: () => void;
+  handleTokenTypeChange: (value: TriggerTokenType) => void;
+  handleAddIgnorePattern: (pattern: string) => void;
+  handleRemoveIgnorePattern: (index: number) => void;
+  handleAddRepository: (item: RepositoryDropdownItem) => void;
+  handleRemoveRepository: (index: number) => void;
+  handleColorChange: (color: TriggerColor) => void;
+}
+
+/**
+ * Manages TriggerCard local state and provides memoized callback handlers.
+ */
+export function useTriggerCardState({
+  trigger,
+  onUpdate,
+  validatePattern,
+}: UseTriggerCardStateOptions): UseTriggerCardStateReturn {
+  // UI state
+  const [isExpanded, setIsExpanded] = useState(false);
+  const [editingName, setEditingName] = useState(false);
+
+  // Local form values
+  const [localName, setLocalName] = useState(trigger.name);
+  const [localPattern, setLocalPattern] = useState(trigger.matchPattern ?? '');
+  const [localMode, setLocalMode] = useState<TriggerMode>(deriveMode(trigger));
+  const [localTokenThreshold, setLocalTokenThreshold] = useState<number>(
+    trigger.tokenThreshold ?? 1000
+  );
+  const [localTokenType, setLocalTokenType] = useState<TriggerTokenType>(
+    trigger.tokenType ?? 'total'
+  );
+
+  // Toggle enabled/disabled
+  const handleToggleEnabled = useCallback(() => {
+    void onUpdate({ enabled: !trigger.enabled });
+  }, [trigger.enabled, onUpdate]);
+
+  // Save name on blur or Enter
+  const handleNameSave = useCallback(() => {
+    if (localName.trim() && localName !== trigger.name) {
+      void onUpdate({ name: localName.trim() });
+    }
+    setEditingName(false);
+  }, [localName, trigger.name, onUpdate]);
+
+  // Save pattern on blur
+  const handlePatternBlur = useCallback(() => {
+    if (validatePattern(localPattern) && localPattern !== trigger.matchPattern) {
+      void onUpdate({ matchPattern: localPattern });
+    }
+  }, [localPattern, trigger.matchPattern, onUpdate, validatePattern]);
+
+  // Update local pattern and validate
+  const handlePatternChange = useCallback(
+    (value: string) => {
+      setLocalPattern(value);
+      validatePattern(value);
+    },
+    [validatePattern]
+  );
+
+  // Content type change - reset matchField to first available
+  const handleContentTypeChange = useCallback(
+    (value: TriggerContentType) => {
+      const newMatchFields = getAvailableMatchFields(value, trigger.toolName ?? undefined);
+      const newMatchField = newMatchFields[0]?.value ?? '';
+      const updates: Partial<NotificationTrigger> = {
+        contentType: value,
+        matchField: (newMatchField as TriggerMatchField) || undefined,
+      };
+      // Reset tool name if not applicable
+      if (value !== 'tool_use' && value !== 'tool_result') {
+        updates.toolName = undefined;
+      }
+      void onUpdate(updates);
+    },
+    [onUpdate, trigger.toolName]
+  );
+
+  // Tool name change - reset matchField to first available
+  const handleToolNameChange = useCallback(
+    (value: string) => {
+      const newMatchFields = getAvailableMatchFields(trigger.contentType, value || undefined);
+      const newMatchField = newMatchFields[0]?.value ?? '';
+      void onUpdate({
+        toolName: value || undefined,
+        matchField: (newMatchField as TriggerMatchField) || undefined,
+      });
+    },
+    [onUpdate, trigger.contentType]
+  );
+
+  // Match field change
+  const handleMatchFieldChange = useCallback(
+    (value: string) => {
+      void onUpdate({ matchField: value as TriggerMatchField });
+    },
+    [onUpdate]
+  );
+
+  // Mode change with appropriate defaults
+  const handleModeChange = useCallback(
+    (newMode: TriggerMode) => {
+      setLocalMode(newMode);
+      const updates: Partial<NotificationTrigger> = { mode: newMode };
+
+      if (newMode === 'error_status') {
+        updates.requireError = true;
+        updates.contentType = 'tool_result';
+      } else if (newMode === 'content_match') {
+        // Ensure matchField is set for validation
+        const contentType = trigger.contentType ?? 'tool_result';
+        const matchFields = getAvailableMatchFields(contentType, trigger.toolName ?? undefined);
+        if (!trigger.matchField && matchFields.length > 0) {
+          updates.matchField = matchFields[0].value as TriggerMatchField;
+        }
+      } else if (newMode === 'token_threshold') {
+        updates.tokenThreshold = localTokenThreshold;
+        updates.tokenType = localTokenType;
+      }
+
+      void onUpdate(updates);
+    },
+    [
+      onUpdate,
+      localTokenThreshold,
+      localTokenType,
+      trigger.contentType,
+      trigger.toolName,
+      trigger.matchField,
+    ]
+  );
+
+  // Token threshold change — local only, commit on blur
+  const handleTokenThresholdChange = useCallback((value: number) => {
+    setLocalTokenThreshold(value);
+  }, []);
+
+  // Commit token threshold to config
+  const handleTokenThresholdBlur = useCallback(() => {
+    if (localTokenThreshold !== (trigger.tokenThreshold ?? 1000)) {
+      void onUpdate({ tokenThreshold: localTokenThreshold });
+    }
+  }, [localTokenThreshold, trigger.tokenThreshold, onUpdate]);
+
+  // Token type change
+  const handleTokenTypeChange = useCallback(
+    (value: TriggerTokenType) => {
+      setLocalTokenType(value);
+      void onUpdate({ tokenType: value });
+    },
+    [onUpdate]
+  );
+
+  // Add ignore pattern
+  const handleAddIgnorePattern = useCallback(
+    (pattern: string) => {
+      const newPatterns = [...(trigger.ignorePatterns ?? []), pattern];
+      void onUpdate({ ignorePatterns: newPatterns });
+    },
+    [trigger.ignorePatterns, onUpdate]
+  );
+
+  // Remove ignore pattern
+  const handleRemoveIgnorePattern = useCallback(
+    (index: number) => {
+      const newPatterns = [...(trigger.ignorePatterns ?? [])];
+      newPatterns.splice(index, 1);
+      void onUpdate({ ignorePatterns: newPatterns });
+    },
+    [trigger.ignorePatterns, onUpdate]
+  );
+
+  // Add repository
+  const handleAddRepository = useCallback(
+    (item: RepositoryDropdownItem) => {
+      const currentIds = trigger.repositoryIds ?? [];
+      if (!currentIds.includes(item.id)) {
+        void onUpdate({ repositoryIds: [...currentIds, item.id] });
+      }
+    },
+    [trigger.repositoryIds, onUpdate]
+  );
+
+  // Remove repository
+  const handleRemoveRepository = useCallback(
+    (index: number) => {
+      const newIds = [...(trigger.repositoryIds ?? [])];
+      newIds.splice(index, 1);
+      void onUpdate({ repositoryIds: newIds });
+    },
+    [trigger.repositoryIds, onUpdate]
+  );
+
+  // Color change
+  const handleColorChange = useCallback(
+    (color: TriggerColor) => {
+      void onUpdate({ color });
+    },
+    [onUpdate]
+  );
+
+  return {
+    isExpanded,
+    setIsExpanded,
+    editingName,
+    setEditingName,
+    localName,
+    setLocalName,
+    localPattern,
+    localMode,
+    localTokenThreshold,
+    localTokenType,
+    handleToggleEnabled,
+    handleNameSave,
+    handlePatternBlur,
+    handlePatternChange,
+    handleContentTypeChange,
+    handleToolNameChange,
+    handleMatchFieldChange,
+    handleModeChange,
+    handleTokenThresholdChange,
+    handleTokenThresholdBlur,
+    handleTokenTypeChange,
+    handleAddIgnorePattern,
+    handleRemoveIgnorePattern,
+    handleAddRepository,
+    handleRemoveRepository,
+    handleColorChange,
+  };
+}