feat(desktop): implement settings tabs with multi-provider management

apps/desktop/src/main/index.ts

@@ -28,11 +28,9 @@ import {
   registerOnboardingIpc,
   setDesignSystem,
 } from './onboarding-ipc';
+import { registerPreferencesIpc } from './preferences-ipc';
 import { preparePromptContext } from './prompt-context';
 
-const __filename = fileURLToPath(import.meta.url);
-const __dirname = dirname(__filename);
-
 let mainWindow: ElectronBrowserWindow | null = null;
 
 function createWindow(): void {
@@ -353,6 +351,7 @@ void app.whenReady().then(async () => {
   registerIpcHandlers();
   registerLocaleIpc();
   registerOnboardingIpc();
+  registerPreferencesIpc();
   registerExporterIpc(() => mainWindow);
   setupAutoUpdater();
   createWindow();

apps/desktop/src/main/onboarding-ipc.test.ts

@@ -0,0 +1,111 @@
+/**
+ * Tests for settings IPC channel versioning.
+ *
+ * These tests verify that registerOnboardingIpc registers both the versioned
+ * v1 channels and the legacy shim channels, ensuring backward compat for
+ * callers that haven't migrated yet.
+ */
+
+import { describe, expect, it, vi } from 'vitest';
+
+// Collect registered channel names via a mock ipcMain.
+const registeredChannels: string[] = [];
+
+vi.mock('./electron-runtime', () => ({
+  ipcMain: {
+    handle: (channel: string) => {
+      registeredChannels.push(channel);
+    },
+  },
+  shell: { openPath: vi.fn() },
+}));
+
+// Stub Electron modules that electron-runtime would otherwise pull in.
+vi.mock('electron', () => ({
+  app: { getPath: vi.fn(() => '/tmp'), isPackaged: false, getVersion: vi.fn(() => '0.0.0') },
+  ipcMain: { handle: vi.fn() },
+  safeStorage: { isEncryptionAvailable: vi.fn(() => false) },
+  shell: { openPath: vi.fn() },
+}));
+
+vi.mock('electron-log/main', () => ({
+  default: {
+    scope: () => ({
+      warn: vi.fn(),
+      info: vi.fn(),
+      error: vi.fn(),
+    }),
+    transports: {
+      file: { resolvePathFn: null, maxSize: 0, format: '' },
+      console: { level: 'info', format: '' },
+    },
+    errorHandler: { startCatching: vi.fn() },
+    eventLogger: { startLogging: vi.fn() },
+    info: vi.fn(),
+  },
+}));
+
+vi.mock('./config', () => ({
+  configPath: () => '/tmp/config.toml',
+  configDir: () => '/tmp',
+  readConfig: vi.fn(async () => null),
+  writeConfig: vi.fn(async () => {}),
+}));
+
+vi.mock('./keychain', () => ({
+  encryptSecret: vi.fn((s: string) => `enc:${s}`),
+  decryptSecret: vi.fn((s: string) => s.replace('enc:', '')),
+}));
+
+vi.mock('./storage-settings', () => ({
+  buildAppPaths: vi.fn(() => ({})),
+}));
+
+vi.mock('@open-codesign/providers', () => ({
+  pingProvider: vi.fn(async () => ({ ok: true, modelCount: 1 })),
+}));
+
+describe('registerOnboardingIpc — channel versioning', () => {
+  it('registers settings:v1:list-providers alongside the legacy settings:list-providers shim', async () => {
+    // Import after mocks are in place.
+    const { registerOnboardingIpc } = await import('./onboarding-ipc');
+    registerOnboardingIpc();
+
+    expect(registeredChannels).toContain('settings:v1:list-providers');
+    expect(registeredChannels).toContain('settings:list-providers');
+  });
+
+  it('registers all eight settings v1 channels', async () => {
+    const v1Channels = [
+      'settings:v1:list-providers',
+      'settings:v1:add-provider',
+      'settings:v1:delete-provider',
+      'settings:v1:set-active-provider',
+      'settings:v1:get-paths',
+      'settings:v1:open-folder',
+      'settings:v1:reset-onboarding',
+      'settings:v1:toggle-devtools',
+    ];
+
+    for (const ch of v1Channels) {
+      expect(registeredChannels).toContain(ch);
+    }
+  });
+
+  it('preserves all eight legacy settings shim channels for backward compat', async () => {
+    const legacyChannels = [
+      'settings:list-providers',
+      'settings:add-provider',
+      'settings:delete-provider',
+      'settings:set-active-provider',
+      'settings:get-paths',
+      'settings:open-folder',
+      'settings:reset-onboarding',
+      'settings:toggle-devtools',
+    ];
+
+    for (const ch of legacyChannels) {
+      expect(registeredChannels).toContain(ch);
+    }
+  });
+});