@yushaw/sanqian-chat 0.2.4 → 0.2.6

package/README.md

@@ -114,3 +114,24 @@ src/renderer/styles/
 - **Fixed**: `console.log` only outputs in `devMode`
 - **Fixed**: Global style pollution - default to `'safe'` mode (no Tailwind preflight)
 - **Fixed**: `cancelStream` now calls `sdk.cancelRun()` for real cancellation
+
+## Tech Debt / Future Optimizations
+
+### Low Priority (记录于 2026-01-05)
+
+1. **FloatingChat 命名优化**
+   - `FloatingChat` 实际是通用组件（embedded + floating），名字有误导
+   - 可考虑改名为 `ChatContainer` 或 `ChatView`
+   - 文件：`src/renderer/components/FloatingChat.tsx`
+
+2. **Header 按钮复用**
+   - `FloatingChat` 内联了 `<ModeToggleButton>` + `<AttachButton>`
+   - 可改用 `<PanelHeaderButtons />` 与 `CompactChat` 保持一致
+   - 文件：`src/renderer/components/FloatingChat.tsx:134-136`
+
+
+3. **useCallback 依赖优化** (微优化)
+   - `useAttachState.ts:68`: toggle 依赖 state，每次 state 变化重建
+   - `useChatPanel.ts:86`: toggleMode 依赖 mode，每次 mode 变化重建
+   - 可用 useRef 避免重建，但当前影响可忽略
+   - 文件：`src/renderer/hooks/useAttachState.ts`, `src/renderer/hooks/useChatPanel.ts`

package/dist/core/index.d.mts

@@ -57,6 +57,12 @@ interface ChatUiStrings {
     hitlInputRequest: string;
     hitlApprovalRequired: string;
     hitlInputRequired: string;
+    attachWindow: string;
+    detachWindow: string;
+    floatWindow: string;
+    embedWindow: string;
+    collapseSidebar: string;
+    history: string;
 }
 type ChatThemeMode = 'light' | 'dark' | 'auto';
 type ChatFontSize = 'small' | 'normal' | 'large' | 'extra-large';
@@ -154,6 +160,153 @@ interface FloatingWindowConfig {
     /** Optional full path for window state file */
     windowStatePath?: string;
 }
+type ChatPanelMode = 'embedded' | 'floating';
+type ChatPanelPosition = 'right' | 'left';
+type AttachState = 'attached' | 'detached' | 'unavailable';
+type AttachPosition = 'right' | 'left' | 'top' | 'bottom';
+/**
+ * Window attachment configuration for floating mode
+ */
+interface AttachConfig {
+    /**
+     * Target window to attach to (BrowserWindow instance)
+     */
+    window: unknown;
+    /**
+     * Attach position relative to target window
+     * @default 'right'
+     */
+    position?: AttachPosition;
+    /**
+     * Gap between windows (px)
+     * @default 0
+     */
+    gap?: number;
+    /**
+     * Sync height/width with target window
+     * @default true
+     */
+    syncSize?: boolean;
+    /**
+     * Behavior when target window is minimized
+     * @default 'hide'
+     */
+    onMinimize?: 'hide' | 'detach' | 'minimize';
+    /**
+     * Behavior when target window is closed
+     * @default 'hide'
+     */
+    onClose?: 'hide' | 'destroy';
+    /**
+     * Allow user to drag and detach
+     * @default true
+     */
+    allowDetach?: boolean;
+    /**
+     * Allow re-attach when dragged near target edge
+     * @default true
+     */
+    allowReattach?: boolean;
+    /**
+     * Distance threshold for re-attach (px)
+     * @default 20
+     */
+    reattachThreshold?: number;
+}
+/**
+ * ChatPanel configuration
+ */
+interface ChatPanelConfig {
+    /**
+     * Host window (BaseWindow supports embedded, BrowserWindow floating only)
+     */
+    hostWindow: unknown;
+    /**
+     * Host window's main content view (required for embedded mode)
+     */
+    hostMainView?: unknown;
+    /**
+     * Initial mode
+     * @default 'embedded'
+     */
+    initialMode?: ChatPanelMode;
+    /**
+     * Embed position
+     * @default 'right'
+     */
+    position?: ChatPanelPosition;
+    /**
+     * Panel width
+     * @default 360
+     */
+    width?: number;
+    /**
+     * Minimum width
+     * @default 240
+     */
+    minWidth?: number;
+    /**
+     * Minimum host content width when showing panel.
+     * If window is too narrow, expand it to ensure main content has this width.
+     * Set to 0 to disable auto-expand.
+     * @default 0
+     */
+    minHostContentWidth?: number;
+    /**
+     * Allow resize
+     * @default true
+     */
+    resizable?: boolean;
+    /**
+     * Preload script path
+     */
+    preloadPath: string;
+    /**
+     * Renderer HTML path or URL
+     */
+    rendererPath: string;
+    /**
+     * Dev mode - load URL instead of file
+     */
+    devMode?: boolean;
+    /**
+     * SDK client getter
+     */
+    getClient: () => unknown;
+    /**
+     * Agent ID getter
+     */
+    getAgentId: () => string | null;
+    /**
+     * Layout change callback (for host app to adjust main content)
+     */
+    onLayoutChange?: (layout: {
+        mainWidth: number;
+        chatWidth: number;
+        chatVisible: boolean;
+    }) => void;
+    /**
+     * Shortcut configuration
+     */
+    shortcuts?: {
+        /** Toggle show/hide, false to disable */
+        toggle?: string | false;
+        /** Toggle mode, false to disable */
+        toggleMode?: string | false;
+    };
+    /**
+     * Floating window attach configuration
+     */
+    attach?: AttachConfig;
+    /**
+     * State persistence key
+     */
+    stateKey?: string;
+    /**
+     * UI config for renderer
+     */
+    uiConfig?: ChatUiConfigSerializable;
+}
 
 /**
  * Chat Adapter Interface
@@ -327,4 +480,4 @@ declare function parseToolCalls(toolCalls: unknown): ToolCall[] | undefined;
  */
 declare function mergeConsecutiveAssistantMessages(rawMessages: ApiMessage[]): ChatMessage[];
 
-export { type ApiMessage, type ChatAdapter, type ChatAdapterConfig, type ChatFontSize, type ChatMessage, type ChatThemeMode, type ChatUiConfigSerializable, type ChatUiStrings, type ConnectionErrorCode, type ConnectionStatus, type ConversationDetail, type ConversationInfo, type FloatingWindowConfig, type HitlInterruptData, type Locale, type MessageBlock, type MessageRole, type SdkAdapterConfig, type SendMessage, type StreamEvent, type ToolCall, type ToolCallStatus, type WindowPosition, createChatAdapter, createSdkAdapter, mergeConsecutiveAssistantMessages, parseToolCalls };
+export { type ApiMessage, type AttachConfig, type AttachPosition, type AttachState, type ChatAdapter, type ChatAdapterConfig, type ChatFontSize, type ChatMessage, type ChatPanelConfig, type ChatPanelMode, type ChatPanelPosition, type ChatThemeMode, type ChatUiConfigSerializable, type ChatUiStrings, type ConnectionErrorCode, type ConnectionStatus, type ConversationDetail, type ConversationInfo, type FloatingWindowConfig, type HitlInterruptData, type Locale, type MessageBlock, type MessageRole, type SdkAdapterConfig, type SendMessage, type StreamEvent, type ToolCall, type ToolCallStatus, type WindowPosition, createChatAdapter, createSdkAdapter, mergeConsecutiveAssistantMessages, parseToolCalls };

package/dist/core/index.d.ts

@@ -57,6 +57,12 @@ interface ChatUiStrings {
     hitlInputRequest: string;
     hitlApprovalRequired: string;
     hitlInputRequired: string;
+    attachWindow: string;
+    detachWindow: string;
+    floatWindow: string;
+    embedWindow: string;
+    collapseSidebar: string;
+    history: string;
 }
 type ChatThemeMode = 'light' | 'dark' | 'auto';
 type ChatFontSize = 'small' | 'normal' | 'large' | 'extra-large';
@@ -154,6 +160,153 @@ interface FloatingWindowConfig {
     /** Optional full path for window state file */
     windowStatePath?: string;
 }
+type ChatPanelMode = 'embedded' | 'floating';
+type ChatPanelPosition = 'right' | 'left';
+type AttachState = 'attached' | 'detached' | 'unavailable';
+type AttachPosition = 'right' | 'left' | 'top' | 'bottom';
+/**
+ * Window attachment configuration for floating mode
+ */
+interface AttachConfig {
+    /**
+     * Target window to attach to (BrowserWindow instance)
+     */
+    window: unknown;
+    /**
+     * Attach position relative to target window
+     * @default 'right'
+     */
+    position?: AttachPosition;
+    /**
+     * Gap between windows (px)
+     * @default 0
+     */
+    gap?: number;
+    /**
+     * Sync height/width with target window
+     * @default true
+     */
+    syncSize?: boolean;
+    /**
+     * Behavior when target window is minimized
+     * @default 'hide'
+     */
+    onMinimize?: 'hide' | 'detach' | 'minimize';
+    /**
+     * Behavior when target window is closed
+     * @default 'hide'
+     */
+    onClose?: 'hide' | 'destroy';
+    /**
+     * Allow user to drag and detach
+     * @default true
+     */
+    allowDetach?: boolean;
+    /**
+     * Allow re-attach when dragged near target edge
+     * @default true
+     */
+    allowReattach?: boolean;
+    /**
+     * Distance threshold for re-attach (px)
+     * @default 20
+     */
+    reattachThreshold?: number;
+}
+/**
+ * ChatPanel configuration
+ */
+interface ChatPanelConfig {
+    /**
+     * Host window (BaseWindow supports embedded, BrowserWindow floating only)
+     */
+    hostWindow: unknown;
+    /**
+     * Host window's main content view (required for embedded mode)
+     */
+    hostMainView?: unknown;
+    /**
+     * Initial mode
+     * @default 'embedded'
+     */
+    initialMode?: ChatPanelMode;
+    /**
+     * Embed position
+     * @default 'right'
+     */
+    position?: ChatPanelPosition;
+    /**
+     * Panel width
+     * @default 360
+     */
+    width?: number;
+    /**
+     * Minimum width
+     * @default 240
+     */
+    minWidth?: number;
+    /**
+     * Minimum host content width when showing panel.
+     * If window is too narrow, expand it to ensure main content has this width.
+     * Set to 0 to disable auto-expand.
+     * @default 0
+     */
+    minHostContentWidth?: number;
+    /**
+     * Allow resize
+     * @default true
+     */
+    resizable?: boolean;
+    /**
+     * Preload script path
+     */
+    preloadPath: string;
+    /**
+     * Renderer HTML path or URL
+     */
+    rendererPath: string;
+    /**
+     * Dev mode - load URL instead of file
+     */
+    devMode?: boolean;
+    /**
+     * SDK client getter
+     */
+    getClient: () => unknown;
+    /**
+     * Agent ID getter
+     */
+    getAgentId: () => string | null;
+    /**
+     * Layout change callback (for host app to adjust main content)
+     */
+    onLayoutChange?: (layout: {
+        mainWidth: number;
+        chatWidth: number;
+        chatVisible: boolean;
+    }) => void;
+    /**
+     * Shortcut configuration
+     */
+    shortcuts?: {
+        /** Toggle show/hide, false to disable */
+        toggle?: string | false;
+        /** Toggle mode, false to disable */
+        toggleMode?: string | false;
+    };
+    /**
+     * Floating window attach configuration
+     */
+    attach?: AttachConfig;
+    /**
+     * State persistence key
+     */
+    stateKey?: string;
+    /**
+     * UI config for renderer
+     */
+    uiConfig?: ChatUiConfigSerializable;
+}
 
 /**
  * Chat Adapter Interface
@@ -327,4 +480,4 @@ declare function parseToolCalls(toolCalls: unknown): ToolCall[] | undefined;
  */
 declare function mergeConsecutiveAssistantMessages(rawMessages: ApiMessage[]): ChatMessage[];
 
-export { type ApiMessage, type ChatAdapter, type ChatAdapterConfig, type ChatFontSize, type ChatMessage, type ChatThemeMode, type ChatUiConfigSerializable, type ChatUiStrings, type ConnectionErrorCode, type ConnectionStatus, type ConversationDetail, type ConversationInfo, type FloatingWindowConfig, type HitlInterruptData, type Locale, type MessageBlock, type MessageRole, type SdkAdapterConfig, type SendMessage, type StreamEvent, type ToolCall, type ToolCallStatus, type WindowPosition, createChatAdapter, createSdkAdapter, mergeConsecutiveAssistantMessages, parseToolCalls };
+export { type ApiMessage, type AttachConfig, type AttachPosition, type AttachState, type ChatAdapter, type ChatAdapterConfig, type ChatFontSize, type ChatMessage, type ChatPanelConfig, type ChatPanelMode, type ChatPanelPosition, type ChatThemeMode, type ChatUiConfigSerializable, type ChatUiStrings, type ConnectionErrorCode, type ConnectionStatus, type ConversationDetail, type ConversationInfo, type FloatingWindowConfig, type HitlInterruptData, type Locale, type MessageBlock, type MessageRole, type SdkAdapterConfig, type SendMessage, type StreamEvent, type ToolCall, type ToolCallStatus, type WindowPosition, createChatAdapter, createSdkAdapter, mergeConsecutiveAssistantMessages, parseToolCalls };