feat: Add file management IPC handlers

[stepandel]

Summary

Extend AssistantService with file operations scoped to a specific assistant.

Changes

• Add AssistantFile type with status enum (Processing, Available, Deleting, ProcessingFailed)
• Add file methods to AssistantService: listFiles, describeFile, uploadFile, deleteFile
• Add IPC handlers: assistant:files:list/describe/upload/delete
• Add preload bindings under assistant.files

API

window.electronAPI.assistant.files.list(profileId, assistantName, filter?)
window.electronAPI.assistant.files.describe(profileId, assistantName, fileId)
window.electronAPI.assistant.files.upload(profileId, assistantName, params)
window.electronAPI.assistant.files.delete(profileId, assistantName, fileId)

Files Modified

• electron/types.ts
• electron/assistant-service.ts
• electron/main.ts
• electron/preload.ts
• src/types/electron.d.ts

Testing

• Build passes: pnpm test:build ✅

Closes PINE-40

Summary by CodeRabbit

• New Features

  • Full assistant management (create, list, view, update, delete) with file upload and management.
  • Mode switcher UI (Index vs Assistant) with keyboard shortcuts and persisted per-profile preference.
  • New Assistants panel and active-assistant selection across the app.

• Chores

  • Added Linear CLI and workspace configuration files.

[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: service, IPC handlers, preload APIs, types, React context/hooks/components, mode switcher, and persistence for a per-profile preferred Explorer mode; integrates AssistantService into PineconeService and exposes assistant file operations.

Changes

Cohort / File(s)	Summary
Configuration \.linear\.toml, pnpm-workspace.yaml	Adds Linear CLI config and pnpm workspace build dependency settings.
Public Types / Declarations electron/types.ts, src/types/electron.d.ts	Adds ExplorerMode, AssistantModel, AssistantFile and related types; extends ConnectionProfile with preferredMode. (Note: assistant types duplicated in electron/types.ts.)
Assistant Service electron/assistant-service.ts, electron/pinecone-service.ts	New AssistantService wrapping Pinecone Assistant API (CRUD + file ops); PineconeService gets getAssistantService() and clears it on disconnect.
IPC / Preload API electron/main.ts, electron/preload.ts, src/types/electron.d.ts	Adds IPC handlers and preload methods for profiles (get/set preferred mode), assistant CRUD, assistant file ops, and window helpers; standardized {success,data?,error?} responses and corresponding type surface.
Store: Connection Preferences electron/connection-store.ts	Adds getPreferredMode(profileId) and setPreferredMode(profileId, mode) to persist per-profile preferred Explorer mode.
Mode Context & Provider src/context/ModeContext.tsx, src/windows/ConnectionWindow.tsx	New ModeProvider + useMode hook (persists to store, keyboard shortcuts); ConnectionWindow wraps providers with ModeProvider.
Assistant Selection Context src/context/AssistantSelectionContext.tsx	New context/provider and hook to manage currently active assistant.
React Query Hooks src/hooks/useAssistantQueries.ts	New query keys and hooks: list/detail queries and create/update/delete mutations (delete uses optimistic update and rollback).
UI Components src/components/mode/ModeSwitcher.tsx, src/components/assistants/AssistantsPanel.tsx, src/components/layout/TopBar.tsx	ModeSwitcher UI for switching modes; AssistantsPanel lists assistants with status and interactions; TopBar integrates ModeSwitcher and adjusts center layout.
Global Types for Renderer src/types/electron.d.ts	Expands ElectronAPI typings to include profiles and assistant namespaces and new assistant/file types for renderer consumption.

Sequence Diagram(s)

sequenceDiagram
  participant Renderer as Renderer (UI)
  participant Preload as Preload (electronAPI)
  participant Main as Main (ipcMain)
  participant PineconeSvc as PineconeService
  participant AssistantSvc as AssistantService
  participant PineconeSDK as Pinecone SDK

  Renderer->>Preload: call assistant.list(profileId)
  Preload->>Main: invoke 'assistant:list' IPC
  Main->>PineconeSvc: ensure connected, getAssistantService()
  PineconeSvc->>AssistantSvc: delegate listAssistants()
  AssistantSvc->>PineconeSDK: API call -> list assistants
  PineconeSDK-->>AssistantSvc: assistants data
  AssistantSvc-->>PineconeSvc: mapped AssistantModel[]
  PineconeSvc-->>Main: return data
  Main-->>Preload: IPC response {success:true,data}
  Preload-->>Renderer: resolve Promise with assistants

Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Poem

🐰 New assistants hop in, tabs switch with a wink,
IPC paths bustle while queries sync,
Services and contexts in neat little rows,
Files flutter in, status lights glow,
I nibble code crumbs and cheer as it ships! 🥕

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 46.67% 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 title 'feat: Add file management IPC handlers' accurately describes the primary change—adding IPC handlers for assistant file operations (list, describe, upload, delete).

_{✏️ 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-40-file-handlers

---

_{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: 6

🤖 Fix all issues with AI agents

In @.linear.toml:
- Around line 1-4: The workspace name in .linear.toml is inconsistent with the
repository comment: update the workspace value to match this project's name (or
correct the comment) by changing the workspace = "chroma-explorer" entry or
replacing the top comment that mentions "Pinecone Explorer" so they align;
locate the workspace key in .linear.toml and make the string match the actual
project/repo name or adjust the descriptive comment accordingly.

In `@electron/assistant-service.ts`:
- Around line 27-30: The call to .map(this.mapAssistantModel) loses the class
instance context; change it to use a bound or arrow wrapper so the method keeps
the correct this (e.g., replace .map(this.mapAssistantModel) with .map(a =>
this.mapAssistantModel(a)) or .map(this.mapAssistantModel.bind(this)) in
listAssistants and in any other places where mapAssistantModel is passed
directly to .map to ensure the method retains its class context if it later uses
instance members.

In `@electron/pinecone-service.ts`:
- Line 40: The disconnect() method currently clears client, embeddingService,
profile, and caches but leaves the old AssistantService instance alive; update
disconnect() to also clear the AssistantService reference by calling any
available cleanup on it (e.g., a close/dispose/shutdown method if
AssistantService exposes one) and then set this.assistantService = null so a
stale AssistantService tied to old credentials cannot be reused; locate the
assistantService field and the disconnect() method and perform this cleanup
there.

In `@electron/types.ts`:
- Around line 53-57: The ExplorerMode type exported from electron/types.ts is
unused and duplicates the definition in ModeContext.tsx; either delete the
ExplorerMode declaration from electron/types.ts or centralize it by
moving/exporting the single ExplorerMode type from electron/types.ts and
updating src/context/ModeContext.tsx to import ExplorerMode instead of
redefining it; locate the ExplorerMode type in electron/types.ts and
ModeContext.tsx and apply one of the two options consistently so only one shared
definition remains.

In `@src/components/assistants/AssistantsPanel.tsx`:
- Around line 155-158: The title attribute on the assistant list item uses a
template literal with "\n" which may not render line breaks across browsers;
update the title to a single-line format (e.g., `${assistant.name} — Status:
${getStatusTooltip(assistant.status)}`) or remove the status from title and rely
on the existing status indicator/tooltip; modify the JSX where onClick uses
handleAssistantClick and onContextMenu uses handleAssistantContextMenu to
replace the multiline title with a single-line string or omit it entirely.

In `@src/context/ModeContext.tsx`:
- Around line 36-52: The keyboard listener's useEffect in ModeContext.tsx
captures a stale setMode because it's omitted from the dependency array; update
the effect to depend on the current setMode (or stabilize setMode with
useCallback) so the handler always calls the latest function: in the useEffect
that defines handleKeyDown, replace the empty deps array with [setMode] (and
remove the eslint-disable-line react-hooks/exhaustive-deps) or alternatively
memoize handleKeyDown via useCallback referencing setMode and keep the effect
deps consistent; ensure the symbols referenced are useEffect, handleKeyDown, and
setMode.

🧹 Nitpick comments (8)

src/context/AssistantSelectionContext.tsx (1)

14-16: Unnecessary useCallback wrapper.

The setActiveAssistantState from useState is already referentially stable across renders. Wrapping it in useCallback adds no benefit here.

♻️ Suggested simplification

-  const setActiveAssistant = useCallback((assistantName: string | null) => {
-    setActiveAssistantState(assistantName)
-  }, [])
+  const setActiveAssistant = setActiveAssistantState

Then update the imports to remove useCallback:

-import { createContext, useContext, useState, useCallback, useMemo, ReactNode } from 'react'
+import { createContext, useContext, useState, useMemo, ReactNode } from 'react'

electron/assistant-service.ts (1)

56-64: Extra network round-trip in updateAssistant.

The method makes two API calls: one to update and one to fetch the updated assistant. The comment explains the rationale, but this adds latency and potential inconsistency if the assistant changes between calls.

Consider whether returning the response from updateAssistant directly (if it contains sufficient data) or documenting this behavior for callers would be preferable.

src/components/mode/ModeSwitcher.tsx (1)

44-45: Consider using the label from the modes array for consistency.

Line 45 duplicates the label logic rather than reusing the label property from the modes array. This could lead to inconsistencies if labels change.

💡 Suggested improvement

             <Icon className="h-3.5 w-3.5" />
-            <span className="hidden sm:inline">{value === 'index' ? 'Index' : 'Assistant'}</span>
+            <span className="hidden sm:inline">{label.replace(' Explorer', '')}</span>
           </button>

Alternatively, add a shortLabel property to ModeOption if you want different display values.

src/components/assistants/AssistantsPanel.tsx (1)

84-88: Unnecessary useCallback wrapper.

This callback simply forwards to onCreateNew without any additional logic. You can pass onCreateNew directly to the button's onClick or inline it.

💡 Simplification

-  const handleCreateNew = useCallback(() => {
-    if (onCreateNew) {
-      onCreateNew()
-    }
-  }, [onCreateNew])

Then use onClick={onCreateNew} directly on line 107, since the button already handles undefined gracefully.

src/hooks/useAssistantQueries.ts (1)

135-140: Consider also invalidating the list query on successful delete.

The optimistic update removes the assistant from the cached list in onMutate, but if the mutation succeeds, the list query cache isn't explicitly invalidated. This works because the optimistic update persists, but for consistency with create/update mutations and to ensure eventual consistency with the server, consider adding a list invalidation.

💡 Suggested addition

     onSuccess: (data) => {
+      // Invalidate list to ensure consistency with server
+      queryClient.invalidateQueries({
+        queryKey: assistantQueryKeys.list(data.profileId),
+      })
       // Remove detail query for the deleted assistant
       queryClient.removeQueries({
         queryKey: assistantQueryKeys.detail(data.profileId, data.assistantName),
       })
     },

electron/main.ts (2)

819-832: Consider using imported type for consistency.

The handler uses an inline type definition for params instead of the imported CreateAssistantParams type. While functionally equivalent, using the imported type improves maintainability.

💡 Suggested change

+import {
+  // ... existing imports
+  CreateAssistantParams,
+  UpdateAssistantParams,
+  UploadAssistantFileParams,
+  ListAssistantFilesFilter,
+} from './types'

-ipcMain.handle('assistant:create', async (_event, profileId: string, params: { name: string; instructions?: string; metadata?: Record<string, string>; region?: 'us' | 'eu' }) => {
+ipcMain.handle('assistant:create', async (_event, profileId: string, params: CreateAssistantParams) => {

Apply similar changes to assistant:update (line 849), assistant:files:list (line 883), and assistant:files:upload (line 913).

---

804-877: Consider adding analytics tracking for assistant operations.

Index operations include analytics tracking (e.g., track('index_created')), but the new assistant handlers don't include similar tracking. If usage analytics are important for assistants, consider adding tracking calls.

src/context/ModeContext.tsx (1)

4-4: Consolidate duplicate Electron type definitions.

ExplorerMode is defined in three locations: src/context/ModeContext.tsx (line 4), electron/types.ts (line 55), and src/types/electron.d.ts. The more critical issue is that electron/types.ts and src/types/electron.d.ts define the entire Electron API surface identically. Consider consolidating these parallel type definitions and, if the ModeContext export is only used locally, documenting whether it should remain isolated as a React-layer type or be imported from the electron types.

[coderabbitai]

⚠️ Potential issue | 🔴 Critical

🧩 Analysis chain

🏁 Script executed:

# Search for ExplorerMode type definitions
rg -n "type ExplorerMode\s*=" --type ts -A 2

Repository: stepandel/pinecone-explorer

Length of output: 512

---

🏁 Script executed:

# Check where ExplorerMode is imported from
rg -n "import.*ExplorerMode|from.*ExplorerMode" --type ts -B 2 -A 2

Repository: stepandel/pinecone-explorer

Length of output: 361

---

🏁 Script executed:

# Show the full context of both definitions
echo "=== electron/types.ts ===" && sed -n '50,60p' electron/types.ts
echo -e "\n=== src/context/ModeContext.tsx ===" && head -10 src/context/ModeContext.tsx

Repository: stepandel/pinecone-explorer

Length of output: 676

---

🏁 Script executed:

# Find all imports of ExplorerMode, showing which file it's imported from
rg -n "import.*ExplorerMode" --type ts --type tsx -B 1 -A 1

Repository: stepandel/pinecone-explorer

Length of output: 98

---

🏁 Script executed:

# Check if electron/types.ts is exported from anywhere
rg -n "from.*electron/types|from.*electron['\"]" --type ts --type tsx | head -20

Repository: stepandel/pinecone-explorer

Length of output: 98

---

🏁 Script executed:

# Check if src/types/electron.d.ts is auto-generated or intentionally maintained
head -20 src/types/electron.d.ts

Repository: stepandel/pinecone-explorer

Length of output: 732

---

🏁 Script executed:

# Search for all imports of ExplorerMode without type filter
rg -n "import.*ExplorerMode" -A 1

Repository: stepandel/pinecone-explorer

Length of output: 215

---

🏁 Script executed:

# Check what exports from electron/types.ts are actually used
rg -n "from.*['\"].*electron/types['\"]"

Repository: stepandel/pinecone-explorer

Length of output: 719

---

🏁 Script executed:

# Find all TypeScript/React files and search for ExplorerMode usage
fd -e ts -e tsx | xargs grep -l "ExplorerMode" | head -20

Repository: stepandel/pinecone-explorer

Length of output: 177

---

🏁 Script executed:

# Check if electron/types.ts is used for main process or IPC types
head -30 electron/types.ts

# Check git history or comments about why ExplorerMode is in electron/types.ts
rg -B 5 "export type ExplorerMode" electron/types.ts

Repository: stepandel/pinecone-explorer

Length of output: 989

---

🏁 Script executed:

# Verify if anything imports ExplorerMode from electron/types.ts
rg "ExplorerMode.*from.*electron/types"

Repository: stepandel/pinecone-explorer

Length of output: 53

---

Remove unused ExplorerMode type from electron/types.ts.

ExplorerMode is defined in three locations but only src/context/ModeContext.tsx is actually imported and used in the codebase. The definition in electron/types.ts (line 56) is dead code. Since electron/types.ts is the central types module for shared Pinecone types, consolidate by either:

1. Remove the unused definition from electron/types.ts, or
2. Export from electron/types.ts and import it into ModeContext.tsx to align with other shared types.

🤖 Prompt for AI Agents

In `@electron/types.ts` around lines 53 - 57, The ExplorerMode type exported from
electron/types.ts is unused and duplicates the definition in ModeContext.tsx;
either delete the ExplorerMode declaration from electron/types.ts or centralize
it by moving/exporting the single ExplorerMode type from electron/types.ts and
updating src/context/ModeContext.tsx to import ExplorerMode instead of
redefining it; locate the ExplorerMode type in electron/types.ts and
ModeContext.tsx and apply one of the two options consistently so only one shared
definition remains.

[stepandel]

@claude apply proposed fix

[coderabbitai]

Seems like the humans are having a chat. I'll hop back into my burrow for now. If you need me again, just tag @coderabbitai in a new comment, and I'll come hopping out!

[claude]

Claude finished @stepandel's task in 1m 8s —— View job

---

Applying proposed fix ✅

• Read electron/types.ts to confirm the duplicate ExplorerMode definition
• Verify ExplorerMode from electron/types.ts is unused
• Remove the unused type definition
• Commit and push changes

Changes made:

• Removed unused ExplorerMode type definition from electron/types.ts (lines 53-56)
• Updated ConnectionProfile.preferredMode to use inline union type 'index' | 'assistant' instead of referencing the removed type

The fix has been applied and pushed to the pine-40-file-handlers branch in commit c3a86ad.

---