feat: Add AssistantService with CRUD operations

[stepandel]

Summary

Create a new service layer for Assistant API operations. Wraps the Pinecone SDK's Assistant methods and exposes them via IPC to the renderer process.

Changes

• Add AssistantService class with list, create, describe, update, delete operations
• Add IPC handlers under assistant: namespace
• Add preload bindings via window.electronAPI.assistant
• Add TypeScript types: AssistantModel, CreateAssistantParams, UpdateAssistantParams
• Wire service to PineconeService with getAssistantService() method

Files Changed

• electron/assistant-service.ts (new)
• electron/types.ts
• electron/main.ts
• electron/preload.ts
• electron/pinecone-service.ts
• src/types/electron.d.ts

API

window.electronAPI.assistant.list(profileId)
window.electronAPI.assistant.create(profileId, params)
window.electronAPI.assistant.describe(profileId, name)
window.electronAPI.assistant.update(profileId, name, params)
window.electronAPI.assistant.delete(profileId, name)

Testing

• Build passes: pnpm test:build ✅

Closes PINE-37

Summary by CodeRabbit

• New Features
  • Mode switcher added to the top bar to toggle Index ↔ Assistant
  • Keyboard shortcuts: Cmd/Ctrl+1 (Index), Cmd/Ctrl+2 (Assistant)
  • Per-profile preferred mode with persistent storage
  • Global mode context provider enabling mode-aware UI
  • Assistant management: list, create, view, update, and delete assistants from the app

[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 and per-profile explorer mode support: new AssistantService, IPC/preload assistant handlers, ExplorerMode context and ModeSwitcher UI, stored profile preferredMode, type declarations, and workspace/config files.

Changes

Cohort / File(s)	Summary
Workspace & Configuration /.linear.toml, pnpm-workspace.yaml	Add Linear CLI metadata and pnpm workspace build config.
Assistant Service & Pinecone integration electron/assistant-service.ts, electron/pinecone-service.ts	Introduce exported AssistantService (list/create/describe/update/delete) and add PineconeService.getAssistantService() with lazy initialization.
Connection store & profile fields electron/connection-store.ts, electron/types.ts, src/types/electron.d.ts	Add preferredMode to profiles, new ExplorerMode type, assistant-related types (AssistantModel, Create/Update params, AssistantStatus), and store getters/setters for preferred mode.
IPC main handlers electron/main.ts	Add IPC handlers for profiles:getPreferredMode / setPreferredMode and assistant CRUD operations with unified error handling and Pinecone connection checks.
Preload / Renderer API surface electron/preload.ts, src/types/electron.d.ts	Expose profiles.getPreferredMode / profiles.setPreferredMode and assistant.* methods in preload; update type declarations to include new APIs.
Mode context & UI src/context/ModeContext.tsx, src/components/mode/ModeSwitcher.tsx, src/components/layout/TopBar.tsx, src/windows/ConnectionWindow.tsx	Add ModeProvider and useMode hook with persistence and keyboard shortcuts; ModeSwitcher UI and TopBar integration; wrap window tree with ModeProvider.
Misc GUI wiring src/components/layout/TopBar.tsx	Adjust TopBar layout to include ModeSwitcher and preserve drag/no-drag regions.

Sequence Diagram(s)

sequenceDiagram
    participant User
    participant ModeSwitcher as ModeSwitcher (React)
    participant ModeContext as ModeContext (React)
    participant Preload as Preload/API
    participant Main as electron/main (Handler)
    participant Store as ConnectionStore

    User->>ModeSwitcher: Click mode option
    ModeSwitcher->>ModeContext: setMode(mode)
    ModeContext->>Preload: profiles:setPreferredMode(profileId, mode)
    Preload->>Main: IPC profiles:setPreferredMode
    Main->>Store: setPreferredMode(profileId, mode)
    Store->>Main: ack
    Main->>Preload: success
    Preload->>ModeContext: resolve
    ModeContext->>ModeSwitcher: update state/render

Loading

sequenceDiagram
    participant Renderer as App
    participant Preload as Preload/API
    participant Main as electron/main (Handler)
    participant PineconeSvc as PineconeService
    participant AssistantSvc as AssistantService
    participant PineconeSDK as Pinecone SDK

    Renderer->>Preload: assistant.list(profileId)
    Preload->>Main: IPC assistant:list
    Main->>PineconeSvc: getAssistantService()
    PineconeSvc->>AssistantSvc: lazy-init (client)
    Main->>AssistantSvc: listAssistants()
    AssistantSvc->>PineconeSDK: call SDK list
    PineconeSDK-->>AssistantSvc: SDK response
    AssistantSvc->>Main: mapped AssistantModel[]
    Main->>Preload: success + data
    Preload->>Renderer: return AssistantModel[]

Loading

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~45 minutes

Poem

🐰 I twitched my nose and spun around,

New modes and helpers softly found,
IPC hops, contexts hum,
Assistants wake — the fun's begun!
*thump-thump, carrot confetti* 🥕

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.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 clearly and accurately summarizes the primary change: introducing AssistantService with CRUD operations. It is specific, concise, and directly reflects the main objective of the pull request.

_{✏️ 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-37-assistant-service

---

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

[claude]

Code Review

Found one issue that needs to be addressed:

Issue: assistantService not cleared on disconnect

The disconnect() method in PineconeService clears this.client, this.embeddingService, and this.profile, but does NOT clear this.assistantService. This will cause runtime errors when a user disconnects and reconnects.

Bug scenario:

1. User connects (creates client A)
2. User calls getAssistantService() - creates AssistantService with client A
3. User disconnects - this.client = null, but this.assistantService still points to the old instance
4. User reconnects (creates client B)
5. User calls getAssistantService() - returns the OLD AssistantService with stale client A
6. Operations fail because the AssistantService is using an invalidated client

This is inconsistent with how embeddingService is handled (which IS properly cleared on disconnect).

Recommended fix:

In the disconnect() method at

pinecone-explorer/electron/pinecone-service.ts

Lines 92 to 98 in 334aba1

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

, add:

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

[coderabbitai]

Actionable comments posted: 2

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 | 🟠 Major

Reset cached AssistantService on disconnect/reconnect.
The cached service remains bound to the old Pinecone client. After disconnect/reconnect, it can continue using stale credentials. Clear it alongside the client lifecycle.

🔧 Suggested fix (lifecycle reset)

   async connect(profile: ConnectionProfile): Promise<void> {
     try {
       this.client = new Pinecone({
         apiKey: profile.apiKey,
       })
@@
     } catch (error) {
       this.client = null
       this.embeddingService = null
+      this.assistantService = null
       this.profile = null
       throw error
     }
   }
@@
   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 `@electron/assistant-service.ts`:
- Around line 68-85: In mapAssistantModel, normalize model.metadata so null
becomes undefined before returning the AssistantModel: update the metadata
mapping in mapAssistantModel to produce metadata: model.metadata ?? undefined
(or use a conditional check) and cast to Record<string,string> | undefined so
AssistantModel.metadata is never null; ensure the returned property uses the
same type (AssistantModel.metadata) and preserves undefined when metadata is
missing.

In `@src/context/ModeContext.tsx`:
- Around line 35-52: The keyboard shortcut effect registers handleKeyDown with
an empty dependency array but captures a stale setMode closure; move the setMode
function (the setter that uses currentProfile) above the useEffect that defines
handleKeyDown, and include setMode (and any values it closes over, e.g.,
currentProfile) in the useEffect dependency array so the event handler always
calls the latest setMode implementation; update
window.addEventListener/removeEventListener to use the new handleKeyDown closure
accordingly.

🧹 Nitpick comments (2)

electron/types.ts (1)

405-439: Consider widening assistant metadata types for consistency.
Assistant metadata is currently Record<string, string>, which is narrower than other metadata shapes in the app. Widening avoids casts and keeps the model flexible. If you take this, mirror it in src/types/electron.d.ts to prevent drift.

♻️ Proposed type adjustments

 export interface AssistantModel {
   name: string
   status: AssistantStatus
   instructions?: string
-  metadata?: Record<string, string>
+  metadata?: Record<string, unknown>
   host?: string
   createdAt?: string
   updatedAt?: string
 }

 export interface CreateAssistantParams {
   name: string
   instructions?: string
-  metadata?: Record<string, string>
+  metadata?: Record<string, unknown>
   region?: 'us' | 'eu'
 }

 export interface UpdateAssistantParams {
   instructions?: string
-  metadata?: Record<string, string>
+  metadata?: Record<string, unknown>
 }

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

29-46: Set type="button" to avoid accidental form submits.
Defensive against future usage inside forms.

✅ Suggested tweak

           <button
             key={value}
+            type="button"
             role="tab"
             aria-selected={isActive}
             onClick={() => setMode(value)}