feat: Add FilesPanel component

[stepandel]

Summary

Create the secondary left panel that lists files for the selected assistant, mirroring NamespacesPanel pattern.

Changes

• Add useFilesQuery hook with dynamic polling (every 5s while files are processing)
• Add FileSelectionContext for tracking selected file
• Add FilesPanel component with:
  • Status indicators (Processing %, Available ✓, Failed ✗, Deleting)
  • Upload button with native file picker dialog
  • Search/filter functionality
  • Loading, error, and empty states
• Add dialog:showOpenDialog IPC handler for native file picker
• Wire up providers in ConnectionWindow

Files Changed

• src/components/files/FilesPanel.tsx (new)
• src/context/FileSelectionContext.tsx (new)
• src/hooks/useAssistantQueries.ts
• electron/main.ts
• electron/preload.ts
• src/types/electron.d.ts
• src/windows/ConnectionWindow.tsx

Testing

• Build passes: pnpm test:build ✅

Closes PINE-41

Summary by CodeRabbit

• New Features
  • Manage assistants: create, list, view, update, and delete assistants.
  • Assistant file management: upload, list, view, and remove files with status/progress.
  • Mode switching UI: toggle between Index and Assistant modes with keyboard shortcuts and per-profile persistence.
  • New side panels: Assistants list and Files browser with search, sorting, and upload flows.
  • Native file picker dialog integrated for uploads.

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

• 🔍 Trigger a full review

📝 Walkthrough

Walkthrough

Adds assistant management across backend, IPC, UI, hooks, types, and context providers; introduces mode persistence and switching, assistant/file CRUD via a new AssistantService, UI panels for assistants/files, and new Electron APIs for dialog and preferred mode.

Changes

Cohort / File(s)	Summary
Backend Services electron/assistant-service.ts, electron/pinecone-service.ts, electron/connection-store.ts	New AssistantService wraps Pinecone SDK assistant and file operations; PineconeService lazily exposes getAssistantService(); ConnectionStore adds getPreferredMode and setPreferredMode for persisted explorer mode.
IPC & Preload electron/main.ts, electron/preload.ts	New IPC handlers and preload surface for profiles (getPreferredMode/setPreferredMode), assistant CRUD (list/create/describe/update/delete), assistant file ops (list/describe/upload/delete), and dialog:showOpenDialog.
Types / Declarations electron/types.ts, src/types/electron.d.ts	Adds ExplorerMode, assistant and assistant-file types, and extends ConnectionProfile with preferredMode. Note: electron/types.ts contains duplicated Assistant API/type declarations.
React Contexts src/context/ModeContext.tsx, src/context/AssistantSelectionContext.tsx, src/context/FileSelectionContext.tsx	New ModeProvider with persistence and keyboard shortcuts, plus AssistantSelectionProvider and FileSelectionProvider with hooks (useMode, useAssistantSelection, useFileSelection).
UI Components src/components/assistants/AssistantsPanel.tsx, src/components/files/FilesPanel.tsx, src/components/mode/ModeSwitcher.tsx, src/components/layout/TopBar.tsx	Adds AssistantsPanel and FilesPanel (file upload/search/sort/status UI), ModeSwitcher (integrated into TopBar) and layout adjustments.
Data Layer / Hooks src/hooks/useAssistantQueries.ts	React-Query hooks for assistants and files, including create/update/delete mutations, optimistic delete rollback, invalidation strategies, and polling files every 5s while processing.
Window & Providers src/windows/ConnectionWindow.tsx, src/components/files/index.ts	Wraps app with new providers (ModeProvider, AssistantSelectionProvider, FileSelectionProvider); exports FilesPanel from files index.
Workspace & Config .linear.toml, pnpm-workspace.yaml	Adds Linear CLI config and updates pnpm workspace packages/ignored/onlyBuiltDependencies.

Sequence Diagram

sequenceDiagram
    participant UI as UI Component
    participant Context as Mode/Selection Context
    participant Hook as React-Query Hook
    participant IPC as IPC Handler (main)
    participant Service as AssistantService
    participant SDK as Pinecone SDK

    UI->>Context: read mode / active assistant
    Context-->>UI: current mode / selection
    UI->>Hook: query assistants (profileId)
    Hook->>IPC: invoke 'assistant:list' (profileId)
    IPC->>Service: listAssistants()
    Service->>SDK: client.assistants.list()
    SDK-->>Service: assistants[]
    Service-->>IPC: mapped assistants[]
    IPC-->>Hook: { success, data }
    Hook-->>UI: render assistants

    Note over UI,Hook: File upload flow
    UI->>IPC: invoke 'dialog:showOpenDialog'
    IPC-->>UI: { filePaths }
    UI->>Hook: upload mutation (filePaths)
    Hook->>IPC: invoke 'assistant:files:upload'
    IPC->>Service: uploadFile(...)
    Service->>SDK: client.assistants.files.upload(...)
    SDK-->>Service: AssistantFile
    Service-->>IPC: mapped file
    IPC-->>Hook: { success, data }
    Hook->>Hook: invalidate files query (polling 5s while processing)
    Hook-->>UI: updated file list/status

Loading

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~50 minutes

Poem

🐰 Carrot Compiler

I hopped in code with whiskers bright,
Modes and assistants now take flight,
Files hop up and statuses change,
Contexts planted in tidy range,
Hooray — the explorer blooms at night! 🌙

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 45.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The PR title 'feat: Add FilesPanel component' directly and clearly describes the primary change in the changeset.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch pine-41-files-panel

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[claude]

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

[coderabbitai]

Actionable comments posted: 4

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

electron/pinecone-service.ts (1)

40-60: ⚠️ Potential issue | 🟡 Minor

Consider clearing assistantService on disconnect.

The assistantService field is lazily initialized in getAssistantService(), but it's not cleared in the disconnect() method (Lines 92-98). This could lead to a stale AssistantService instance holding a reference to the old Pinecone client after reconnection.

🔧 Proposed fix

  disconnect(): void {
    this.client = null
    this.embeddingService = null
+   this.assistantService = null
    this.profile = null
    this.indexCache.clear()
    this.indexInfoCache.clear()
  }

🤖 Fix all issues with AI agents

In @.linear.toml:
- Around line 4-5: Update the Linear workspace value in the .linear.toml file:
change the workspace key from "chroma-explorer" to the correct Pinecone Explorer
workspace name used by this repo (replace the string value for workspace) so the
entry reads workspace = "<pinecone-explorer-workspace>" while keeping team_id =
"PINE" unchanged; locate the workspace entry in .linear.toml and update the
string literal accordingly.

In `@src/components/assistants/AssistantsPanel.tsx`:
- Around line 148-181: Replace the clickable <div> wrapper for each assistant
row in AssistantsPanel.tsx with a semantic <button type="button"> that retains
the same className, onClick handler (handleAssistantClick), onContextMenu
handler (handleAssistantContextMenu), and title (using getStatusTooltip), so
rows become focusable and operable by keyboard; keep the inner status indicator
and text (which use getStatusColor and getStatusTooltip) unchanged, ensure you
do not add an extra role or tabIndex (native button semantics suffice), and
update any styling assumptions that relied on the element being a div (e.g.,
remove flex/inline styles that conflict with button defaults) so keyboard users
can use Enter/Space to activate rows and the context menu still opens on
right-click.

In `@src/context/ModeContext.tsx`:
- Around line 35-52: The keyboard shortcut effect captures a stale setMode
closure causing mode changes to apply to the wrong profile after switching;
update the useEffect that defines handleKeyDown to include setMode in its
dependency array (so the effect re-attaches when the profile-specific setter
changes) and remove the manual eslint-disable-line react-hooks/exhaustive-deps;
ensure the listener registration/removal still uses the same handleKeyDown names
so addEventListener/removeEventListener remain paired.

In `@src/hooks/useAssistantQueries.ts`:
- Around line 145-174: The refetchInterval currently only polls when files have
status 'Processing', which misses files in the 'Deleting' state; update the
refetch condition inside useFilesQuery (function useFilesQuery, constant
FILES_POLL_INTERVAL, type AssistantFile) so it treats files with status ===
'Deleting' the same as 'Processing' (e.g., change hasProcessingFiles to check
file.status === 'Processing' || file.status === 'Deleting') so deletion
in-flight keeps the 5s polling interval.

🧹 Nitpick comments (2)

electron/main.ts (1)

1119-1137: Consider attaching dialog to parent window.

The showOpenDialog call doesn't specify a parent window, which may cause the dialog to appear disconnected from the application on some platforms. Consider using BrowserWindow.fromWebContents(event.sender) to attach the dialog to the calling window.

♻️ Proposed enhancement

 ipcMain.handle('dialog:showOpenDialog', async (event, options: {
   properties?: Array<'openFile' | 'openDirectory' | 'multiSelections' | 'showHiddenFiles'>
   filters?: Array<{ name: string; extensions: string[] }>
   title?: string
   defaultPath?: string
 }) => {
   try {
+    const win = BrowserWindow.fromWebContents(event.sender)
-    const result = await dialog.showOpenDialog({
+    const result = await dialog.showOpenDialog(win!, {
       properties: options.properties || ['openFile'],
       filters: options.filters,
       title: options.title,
       defaultPath: options.defaultPath,
     })
     return { success: true, data: result }
   } catch (error) {

src/components/files/FilesPanel.tsx (1)

1-120: Clear stale file selection when assistant or file list changes.
Right now activeFile can survive assistant switches or file deletions, which can leave downstream views pointing to a non-existent file. Consider resetting when the assistant changes or when the active file no longer exists.

♻️ Suggested effect

-import { useState, useMemo, useCallback } from 'react'
+import { useState, useMemo, useCallback, useEffect } from 'react'
@@
   const { data: files = [], isLoading, error, refetch } = useFilesQuery(
     currentProfile?.id || null,
     activeAssistant,
     !!activeAssistant
   )
+
+  useEffect(() => {
+    if (!activeAssistant) {
+      setActiveFile(null)
+      return
+    }
+    if (activeFile && !files.some((file) => file.id === activeFile)) {
+      setActiveFile(null)
+    }
+  }, [activeAssistant, activeFile, files, setActiveFile])