From a6096514dcf4b98596188354a4cfcfe81bd406ec Mon Sep 17 00:00:00 2001
From: "Atlas (Engineering Lead)" <atlas@openclaw.ai>
Date: Fri, 6 Feb 2026 00:16:27 +0000
Subject: [PATCH 1/2] feat(files): add right-click context menu for file
 actions

PINE-56: Phase 6 feedback - Part 2

Add file context menu with Download and Delete actions:
- Add showFileMenu/onFileAction to electron/preload.ts
- IPC handler already existed in main.ts (lines 656-673)
- Add onContextMenu handler to FilesPanel file buttons
- Delete action shows confirmation dialog, then deletes file
- Download action opens signed URL in browser
- Add TypeScript types to electron.d.ts

Shortcuts in Settings were already implemented (keyboard-shortcuts.ts
has 'assistant' and 'chat' categories).
---
 electron/preload.ts                 |  8 +++++
 src/components/files/FilesPanel.tsx | 48 ++++++++++++++++++++++++++++-
 src/types/electron.d.ts             |  2 ++
 3 files changed, 57 insertions(+), 1 deletion(-)

diff --git a/electron/preload.ts b/electron/preload.ts
index a6951fa..24e08fb 100644
--- a/electron/preload.ts
+++ b/electron/preload.ts
@@ -219,6 +219,14 @@ contextBridge.exposeInMainWorld('electronAPI', {
       ipcRenderer.on('context-menu:assistant-action', handler)
       return () => ipcRenderer.removeListener('context-menu:assistant-action', handler)
     },
+    showFileMenu: (assistantName: string, fileId: string, fileName: string): void => {
+      ipcRenderer.send('context-menu:show-file', assistantName, fileId, fileName)
+    },
+    onFileAction: (callback: (action: { action: string; assistantName: string; fileId: string; fileName: string }) => void): (() => void) => {
+      const handler = (_event: any, data: { action: string; assistantName: string; fileId: string; fileName: string }) => callback(data)
+      ipcRenderer.on('context-menu:file-action', handler)
+      return () => ipcRenderer.removeListener('context-menu:file-action', handler)
+    },
   },
   profiles: {
     getAll: async (): Promise<ConnectionProfile[]> => {
diff --git a/src/components/files/FilesPanel.tsx b/src/components/files/FilesPanel.tsx
index e349a5c..31ef825 100644
--- a/src/components/files/FilesPanel.tsx
+++ b/src/components/files/FilesPanel.tsx
@@ -1,4 +1,4 @@
-import { useState, useMemo, useCallback } from 'react'
+import { useState, useMemo, useCallback, useEffect } from 'react'
 import { FileText, Upload, Check, AlertCircle, Loader2, Trash2 } from 'lucide-react'
 import { usePinecone } from '../../providers/PineconeProvider'
 import { useAssistantSelection } from '../../context/AssistantSelectionContext'
@@ -119,6 +119,51 @@ export function FilesPanel({ className }: FilesPanelProps) {
     setActiveFile(fileId)
   }, [setActiveFile])
 
+  // Handle right-click context menu on file items
+  const handleFileContextMenu = useCallback((e: React.MouseEvent, file: AssistantFile) => {
+    e.preventDefault()
+    if (!activeAssistant) return
+    window.electronAPI.contextMenu.showFileMenu(activeAssistant, file.id, file.name)
+  }, [activeAssistant])
+
+  // Listen for file context menu actions
+  useEffect(() => {
+    if (!currentProfile?.id || !activeAssistant) return
+
+    const cleanup = window.electronAPI.contextMenu.onFileAction(async (data) => {
+      if (data.assistantName !== activeAssistant) return
+
+      if (data.action === 'delete') {
+        // Confirm deletion
+        const confirmed = window.confirm(`Delete "${data.fileName}"?\n\nThis action cannot be undone.`)
+        if (!confirmed) return
+
+        try {
+          await window.electronAPI.assistant.files.delete(currentProfile.id, activeAssistant, data.fileId)
+          // Clear selection if deleted file was selected
+          if (activeFile === data.fileId) {
+            setActiveFile(null)
+          }
+          refetch()
+        } catch (err) {
+          console.error('Failed to delete file:', err)
+        }
+      } else if (data.action === 'download') {
+        // Get file details to get signed URL
+        try {
+          const file = await window.electronAPI.assistant.files.describe(currentProfile.id, activeAssistant, data.fileId)
+          if (file.signedUrl) {
+            await window.electronAPI.shell.openExternal(file.signedUrl)
+          }
+        } catch (err) {
+          console.error('Failed to download file:', err)
+        }
+      }
+    })
+
+    return cleanup
+  }, [currentProfile?.id, activeAssistant, activeFile, setActiveFile, refetch])
+
   // If no assistant is selected, show prompt
   if (!activeAssistant) {
     return (
@@ -254,6 +299,7 @@ export function FilesPanel({ className }: FilesPanelProps) {
                 <button
                   key={file.id}
                   onClick={() => handleFileClick(file.id)}
+                  onContextMenu={(e) => handleFileContextMenu(e, file)}
                   className={`w-full px-3 py-1.5 text-left transition-colors duration-100 mx-1 rounded-md ${
                     isActive
                       ? 'bg-black/[0.08] dark:bg-white/[0.10]'
diff --git a/src/types/electron.d.ts b/src/types/electron.d.ts
index efd30f0..a2311a2 100644
--- a/src/types/electron.d.ts
+++ b/src/types/electron.d.ts
@@ -381,6 +381,8 @@ declare global {
       onNamespaceAction: (callback: (action: { action: string; namespace: string }) => void) => () => void
       showAssistantMenu: (assistantName: string) => void
       onAssistantAction: (callback: (action: { action: string; assistantName: string }) => void) => () => void
+      showFileMenu: (assistantName: string, fileId: string, fileName: string) => void
+      onFileAction: (callback: (action: { action: string; assistantName: string; fileId: string; fileName: string }) => void) => () => void
     }
     profiles: {
       getAll: () => Promise<ConnectionProfile[]>

From 29f76077b7589772cb917ac32942089e268caccb Mon Sep 17 00:00:00 2001
From: "Atlas (Engineering Lead)" <atlas@openclaw.ai>
Date: Fri, 6 Feb 2026 00:22:11 +0000
Subject: [PATCH 2/2] feat(analytics): add tracking for Assistant events

PINE-57: Phase 7 - Add analytics

Add track() calls to Assistant IPC handlers:
- assistant_created (with region)
- assistant_deleted
- file_uploaded (with multimodal flag)
- file_deleted
- chat_message_sent (with model, messageCount)
- chat_stream_started (with model)

Follows existing analytics pattern from index operations.
---
 electron/main.ts | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/electron/main.ts b/electron/main.ts
index c6fdc2d..8629fe7 100644
--- a/electron/main.ts
+++ b/electron/main.ts
@@ -865,6 +865,7 @@ ipcMain.handle('assistant:create', async (_event, profileId: string, params: { n
     }
     const assistantService = service.getAssistantService()
     const assistant = await assistantService.createAssistant(params)
+    track('assistant_created', { region: params.region || 'us' })
     return { success: true, data: assistant }
   } catch (error) {
     const message = error instanceof Error ? error.message : 'Failed to create assistant'
@@ -910,6 +911,7 @@ ipcMain.handle('assistant:delete', async (_event, profileId: string, name: strin
     }
     const assistantService = service.getAssistantService()
     await assistantService.deleteAssistant(name)
+    track('assistant_deleted')
     return { success: true }
   } catch (error) {
     const message = error instanceof Error ? error.message : 'Failed to delete assistant'
@@ -959,6 +961,7 @@ ipcMain.handle('assistant:files:upload', async (_event, profileId: string, assis
     }
     const assistantService = service.getAssistantService()
     const file = await assistantService.uploadFile(assistantName, params)
+    track('file_uploaded', { multimodal: params.multimodal || false })
     return { success: true, data: file }
   } catch (error) {
     const message = error instanceof Error ? error.message : 'Failed to upload file'
@@ -974,6 +977,7 @@ ipcMain.handle('assistant:files:delete', async (_event, profileId: string, assis
     }
     const assistantService = service.getAssistantService()
     await assistantService.deleteFile(assistantName, fileId)
+    track('file_deleted')
     return { success: true }
   } catch (error) {
     const message = error instanceof Error ? error.message : 'Failed to delete file'
@@ -996,6 +1000,7 @@ ipcMain.handle('assistant:chat', async (_event, profileId: string, assistantName
     }
     const assistantService = service.getAssistantService()
     const response = await assistantService.chat(assistantName, params)
+    track('chat_message_sent', { model: params.model, messageCount: params.messages.length })
     return { success: true, data: response }
   } catch (error) {
     const message = error instanceof Error ? error.message : 'Failed to send chat message'
@@ -1039,6 +1044,7 @@ ipcMain.handle('assistant:chat:stream:start', async (event, profileId: string, a
       activeChatStreams.delete(streamId)
     })
 
+    track('chat_stream_started', { model: params.model })
     return { success: true, data: { streamId } }
   } catch (error) {
     const message = error instanceof Error ? error.message : 'Failed to start chat stream'