@vantran-se/openchamber-web 2.1.1 → 2.2.0

package/server/lib/opencode/plugins.js

@@ -1,3 +1,4 @@
+import { isRecord } from '../shared/guards.js';
 import fs from 'fs';
 import path from 'path';
 import {
@@ -57,10 +58,6 @@ function validatePluginSpec(spec) {
   return spec.trim();
 }
 
-function isRecord(value) {
-  return Boolean(value) && typeof value === 'object' && !Array.isArray(value);
-}
-
 function hasOptions(options) {
   return isRecord(options) && Object.keys(options).length > 0;
 }
@@ -249,7 +246,8 @@ function decodePluginId(id) {
 }
 
 /** Accept a v1 string/tuple or a v2 `{package, options}` object. */
-function parsePluginRaw(raw) {
+/** @public Exposed to tests that dynamically import the plugins module. */
+export function parsePluginRaw(raw) {
   const entity = toPluginEntity(raw);
   if (!entity) {
     throw codedError('Plugin spec must be a string, [string, object], or {package, options}', 'INVALID_SPEC');
@@ -260,7 +258,8 @@ function parsePluginRaw(raw) {
 }
 
 /** Always v2: a bare string, or `{package, options}`. Never a tuple. */
-function serializePluginEntry(entry) {
+/** @public Exposed to tests that dynamically import the plugins module. */
+export function serializePluginEntry(entry) {
   const spec = validatePluginSpec(entry?.spec);
   const serialized = fromPluginEntity({ package: spec, options: hasOptions(entry?.options) ? entry.options : undefined });
   if (serialized === null) {
@@ -447,7 +446,5 @@ export {
   writePluginDirFile,
   deletePluginDirFile,
   encodePluginId,
-  decodePluginId,
-  parsePluginRaw,
-  serializePluginEntry,
+  decodePluginId
 };

package/server/lib/opencode/provider-source-route.test.js

@@ -1,4 +1,7 @@
 import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
+import fs from 'fs';
+import os from 'os';
+import path from 'path';
 import express from 'express';
 import request from 'supertest';
 import { registerOpenCodeRoutes } from './routes.js';
@@ -165,9 +168,10 @@ describe('POST /api/provider/discover-models', () => {
       getProviderSources: vi.fn(),
     });
 
-    // The route's auth module reads the real OpenCode credential store. This
-    // fixture cannot inject that private module, so the module-level fallback
-    // is covered in model-discovery.test.js and this keeps the route contract:
+    // With no credential source configured the route's credential-store read
+    // fails and discovery falls back to the key in the form; the stored-config
+    // fallback has its own describe below, and env resolution in stored keys
+    // is covered in model-discovery.test.js. This keeps the route contract:
     // provider id enters, no credential leaves in the response.
     const response = await request(app)
       .post('/api/provider/discover-models')
@@ -181,6 +185,100 @@ describe('POST /api/provider/discover-models', () => {
   });
 });
 
+describe('POST /api/provider/discover-models stored-config fallback', () => {
+  // The route reads the stored entry from the merged config layers. Point
+  // OPENCODE_CONFIG (resolved on every read) at a temp fixture so the entry
+  // is exactly what a provider form save writes: the v2 `providers` shape.
+  let root;
+  let previousOpenCodeConfig;
+
+  const serveModels = () => {
+    const requests = [];
+    globalThis.fetch = vi.fn(async (_url, init) => {
+      requests.push(init.headers.Authorization);
+      return new Response(JSON.stringify({ data: [{ id: 'model-a' }] }), { status: 200 });
+    });
+    return requests;
+  };
+
+  const writeStoredProvider = (providerID, entry) => {
+    const configPath = path.join(root, 'opencode.json');
+    fs.writeFileSync(configPath, JSON.stringify({ providers: { [providerID]: entry } }), 'utf8');
+    process.env.OPENCODE_CONFIG = configPath;
+  };
+
+  const discover = (body) => {
+    const app = express();
+    app.use(express.json());
+    registerOpenCodeRoutes(app, {
+      resolveProjectDirectory: vi.fn(async () => ({ directory: '/projects/app' })),
+      getProviderSources: vi.fn(),
+    });
+    return request(app).post('/api/provider/discover-models').send(body);
+  };
+
+  beforeEach(() => {
+    // The credential store is reachable but empty: the fallback must come
+    // from the stored config entry.
+    configureOpenCodeCredentials({ list: async () => [] });
+    root = fs.mkdtempSync(path.join(os.tmpdir(), 'openchamber-discovery-'));
+    previousOpenCodeConfig = process.env.OPENCODE_CONFIG;
+  });
+
+  afterEach(() => {
+    configureOpenCodeCredentials(null);
+    if (previousOpenCodeConfig === undefined) delete process.env.OPENCODE_CONFIG;
+    else process.env.OPENCODE_CONFIG = previousOpenCodeConfig;
+    fs.rmSync(root, { recursive: true, force: true });
+  });
+
+  it('sends the first env variable that is set, not just the first name', async () => {
+    const providerID = `env-order-${Date.now()}`;
+    process.env.OPENCHAMBER_DISCOVERY_SECOND = 'second-secret';
+    delete process.env.OPENCHAMBER_DISCOVERY_FIRST;
+    writeStoredProvider(providerID, {
+      package: 'aisdk:@ai-sdk/openai-compatible',
+      env: ['OPENCHAMBER_DISCOVERY_FIRST', 'OPENCHAMBER_DISCOVERY_SECOND'],
+      settings: { baseURL: 'https://stored.test/v1' },
+    });
+    const requests = serveModels();
+    try {
+      const response = await discover({ providerID, baseURL: 'https://stored.test/v1', enrich: false }).expect(200);
+      expect(requests).toEqual(['Bearer second-secret']);
+      expect(JSON.stringify(response.body)).not.toContain('second-secret');
+    } finally {
+      delete process.env.OPENCHAMBER_DISCOVERY_SECOND;
+    }
+  });
+
+  it('falls back to settings.apiKey when the credential store has none', async () => {
+    const providerID = `settings-key-${Date.now()}`;
+    writeStoredProvider(providerID, { settings: { baseURL: 'https://stored.test/v1', apiKey: 'config-secret' } });
+    const requests = serveModels();
+    const response = await discover({ providerID, baseURL: 'https://stored.test/v1', enrich: false }).expect(200);
+    expect(requests).toEqual(['Bearer config-secret']);
+    expect(JSON.stringify(response.body)).not.toContain('config-secret');
+  });
+
+  it('prefers the credential store key and keeps the stored base URL guard', async () => {
+    const providerID = `store-wins-${Date.now()}`;
+    configureOpenCodeCredentials({
+      list: async () => [{ integrationID: providerID, active: true, value: { type: 'key', key: 'store-secret' } }],
+    });
+    writeStoredProvider(providerID, { settings: { baseURL: 'https://stored.test/v1', apiKey: 'config-secret' } });
+    const requests = serveModels();
+    // Before the v2 config read, storedBaseURL was always undefined and the
+    // endpoint guard discarded even the credential-store key.
+    await discover({ providerID, baseURL: 'https://stored.test/v1', enrich: false }).expect(200);
+    expect(requests).toEqual(['Bearer store-secret']);
+
+    // A form base URL that moved away from the stored one receives no key.
+    requests.length = 0;
+    await discover({ providerID, baseURL: 'https://moved.test/v1', enrich: false }).expect(200);
+    expect(requests).toEqual([undefined]);
+  });
+});
+
 describe('Git initialization through the proxy', () => {
   it('refuses home, a disk root, and a request without a directory', async () => {
     const agent = request(createApp(vi.fn()));

package/server/lib/opencode/providers-disabled.test.js

@@ -0,0 +1,59 @@
+import { afterEach, beforeEach, describe, expect, test } from 'vitest';
+import { spawnSync } from 'child_process';
+import fs from 'fs';
+import os from 'os';
+import path from 'path';
+import { fileURLToPath } from 'url';
+
+const providersModule = fileURLToPath(new URL('./providers.js', import.meta.url));
+
+// The user config path is fixed when shared.js loads, so the calls run in a
+// child process pointed at a temporary OPENCODE_CONFIG_DIR; the real user
+// config is never touched.
+const run = (configDir, body) => {
+  const script = `
+    import { readDisabledProviders, setProviderDisabled } from ${JSON.stringify(providersModule)};
+    ${body}
+  `;
+  const result = spawnSync(process.execPath, ['--input-type=module', '-e', script], {
+    env: { ...process.env, OPENCODE_CONFIG_DIR: configDir },
+    encoding: 'utf8',
+  });
+  if (result.status !== 0) throw new Error(result.stderr);
+  return JSON.parse(result.stdout.trim().split('\n').pop());
+};
+
+describe('disabled providers', () => {
+  let configDir;
+  beforeEach(() => {
+    configDir = fs.mkdtempSync(path.join(os.tmpdir(), 'openchamber-disabled-providers-'));
+  });
+  afterEach(() => {
+    fs.rmSync(configDir, { recursive: true, force: true });
+  });
+
+  test('disables and re-enables a provider in the user config, keeping other keys', () => {
+    fs.writeFileSync(path.join(configDir, 'opencode.json'), JSON.stringify({ model: 'a/b', disabled_providers: ['groq'] }));
+
+    const afterDisable = run(configDir, `
+      setProviderDisabled('openrouter', true, null);
+      setProviderDisabled('openrouter', true, null);
+      console.log(JSON.stringify(readDisabledProviders(null)));
+    `);
+    expect(afterDisable).toEqual(['groq', 'openrouter']);
+
+    const afterEnable = run(configDir, `
+      setProviderDisabled('groq', false, null);
+      setProviderDisabled('openrouter', false, null);
+      console.log(JSON.stringify(readDisabledProviders(null)));
+    `);
+    expect(afterEnable).toEqual([]);
+    const written = JSON.parse(fs.readFileSync(path.join(configDir, 'opencode.json'), 'utf8'));
+    expect(written.model).toBe('a/b');
+    expect(written.disabled_providers).toBeUndefined();
+  });
+
+  test('refuses an invalid provider id', () => {
+    expect(() => run(configDir, `setProviderDisabled('Bad Id', true, null); console.log('[]');`)).toThrow('Invalid provider ID');
+  });
+});

package/server/lib/opencode/providers.js

@@ -314,8 +314,42 @@ function removeProviderConfig(providerId, workingDirectory, scope = 'user') {
   return true;
 }
 
+/**
+ * Providers turned off in the user's OpenCode config (`disabled_providers`).
+ * OpenCode 2 drops them from its provider list, so this list is the only
+ * place Settings can still show them to turn them back on.
+ */
+function readDisabledProviders(workingDirectory) {
+  const layers = readConfigLayers(workingDirectory);
+  const config = getConfigForPath(layers, layers.paths.userPath);
+  return Array.isArray(config.disabled_providers)
+    ? config.disabled_providers.filter((entry) => typeof entry === 'string' && entry.length > 0)
+    : [];
+}
+
+function setProviderDisabled(providerId, disabled, workingDirectory) {
+  if (typeof providerId !== 'string' || !PROVIDER_ID_PATTERN.test(providerId)) {
+    const error = new Error('Invalid provider ID');
+    error.statusCode = 400;
+    throw error;
+  }
+  const layers = readConfigLayers(workingDirectory);
+  const targetPath = layers.paths.userPath;
+  const config = getConfigForPath(layers, targetPath);
+  const current = Array.isArray(config.disabled_providers)
+    ? config.disabled_providers.filter((entry) => entry !== providerId)
+    : [];
+  const next = disabled ? [...current, providerId] : current;
+  if (next.length > 0) config.disabled_providers = next;
+  else delete config.disabled_providers;
+  writeConfig(config, targetPath || CONFIG_FILE);
+  return next;
+}
+
 export {
   getProviderSources,
+  readDisabledProviders,
+  setProviderDisabled,
   removeProviderConfig,
   upsertProviderConfig,
   validateCustomProviderConfig,

package/server/lib/opencode/proxy.js

@@ -459,9 +459,17 @@ export const registerOpenCodeProxy = (app, deps) => {
 
   const replayParsedBody = (proxyReq, req) => {
     const body = serializeParsedBody(req, proxyReq);
-    if (!body) return;
-    // http-proxy copies the incoming headers, so a chunked request would reach
-    // OpenCode with both framing headers and be rejected as ambiguous.
+    if (!body) {
+      // DO NOT remove transfer-encoding from a stream without content-length:
+      // Node then sends tunneled prompt bodies without framing, and OpenCode
+      // reads an empty payload (#4576).
+      if (proxyReq.getHeader('content-length') !== undefined) {
+        proxyReq.removeHeader('transfer-encoding');
+      }
+      return;
+    }
+    // DO NOT keep the copied transfer-encoding alongside the replayed body's
+    // content-length: OpenCode rejects requests with both headers (#4280).
     proxyReq.removeHeader('transfer-encoding');
     proxyReq.setHeader('content-length', String(body.length));
     proxyReq.write(body);

package/server/lib/opencode/routes.js

@@ -9,7 +9,7 @@ import {
 import { getClaudeCliAuthStatus } from './claude-cli-auth.js';
 import { OPENCODE_CONFIG_DIR, readConfigLayers } from './shared.js';
 import { settingsSurfaceOf } from './settings-files.js';
-import { parseWebSearchSelection } from './config-v2.js';
+import { parseWebSearchSelection, readSectionEntry, toProviderEntity } from './config-v2.js';
 import { getWebSearchSource, setWarmingEnabled, setWebSearchSelection } from './websearch-config.js';
 import {
   CREDENTIAL_LIST_ERROR,
@@ -19,6 +19,7 @@ import {
   isProviderConnectRequest,
 } from '../enterprise-mode.js';
 import { discoverProviderModels } from './model-discovery.js';
+import { readDisabledProviders, setProviderDisabled } from './providers.js';
 import { vcsInitRefusal, vcsInitRefusalBody } from '../git/repository-root.js';
 
 export const registerOpenCodeRoutes = (app, dependencies) => {
@@ -268,6 +269,31 @@ export const registerOpenCodeRoutes = (app, dependencies) => {
     }
   });
 
+  // Providers turned off in the user's OpenCode config. OpenCode leaves them
+  // out of its provider list, so Settings reads them here to offer them back.
+  app.get('/api/provider/disabled', (_req, res) => {
+    try {
+      return res.json({ providers: readDisabledProviders(null) });
+    } catch (error) {
+      console.error('Failed to read disabled providers:', error);
+      return res.status(500).json({ error: 'Failed to read disabled providers' });
+    }
+  });
+
+  app.put('/api/provider/:providerId/disabled', async (req, res) => {
+    if (typeof req.body?.disabled !== 'boolean') {
+      return res.status(400).json({ error: 'disabled must be true or false' });
+    }
+    try {
+      const providers = setProviderDisabled(req.params.providerId, req.body.disabled, null);
+      return res.json({ providers, ...buildAppliedResponse('Provider updated.') });
+    } catch (error) {
+      if (error?.statusCode === 400) return res.status(400).json({ error: error.message });
+      console.error('Failed to update disabled providers:', error);
+      return res.status(500).json({ error: 'Failed to update disabled providers' });
+    }
+  });
+
   app.get('/api/provider/:providerId/source', async (req, res) => {
     try {
       const { providerId } = req.params;
@@ -403,7 +429,19 @@ export const registerOpenCodeRoutes = (app, dependencies) => {
           const { getProviderAuth } = await getAuthLibrary();
           const storedAuth = await getProviderAuth(providerID);
           storedApiKey = storedAuth?.type === 'api' && typeof storedAuth.key === 'string' ? storedAuth.key : null;
-          storedBaseURL = readConfigLayers().mergedConfig?.provider?.[providerID]?.options?.baseURL;
+          const { value: rawProvider } = readSectionEntry(readConfigLayers().mergedConfig, 'providers', providerID);
+          const providerEntity = rawProvider ? toProviderEntity(rawProvider) : null;
+          storedBaseURL = providerEntity?.settings?.baseURL;
+          if (!storedApiKey) {
+            // OpenCode tries each `env` name in order, so pick the first one
+            // that is set; when none is, keep the first so discovery can name
+            // it in the error instead of failing as an anonymous 401. This
+            // reads the server's environment: a variable that exists only in
+            // the managed OpenCode launch environment (Settings user env) is
+            // not visible here.
+            const envName = providerEntity?.env?.find((name) => process.env[name]) ?? providerEntity?.env?.[0];
+            storedApiKey = providerEntity?.settings?.apiKey || (envName ? `{env:${envName}}` : null);
+          }
         } catch {
           storedApiKey = null;
         }

package/server/lib/opencode/settings-helpers.js

@@ -232,7 +232,12 @@ export const createSettingsHelpers = (dependencies) => {
     if (typeof candidate.splashFgDark === 'string' && candidate.splashFgDark.trim().length > 0) {
       result.splashFgDark = candidate.splashFgDark.trim();
     }
-    if (typeof candidate.lastDirectory === 'string' && candidate.lastDirectory.length > 0) {
+    // An empty string forgets the last directory (the last project was
+    // removed). Every reader treats it as unset, so nothing is warmed at start
+    // or turned back into a project.
+    if (candidate.lastDirectory === '') {
+      result.lastDirectory = '';
+    } else if (typeof candidate.lastDirectory === 'string') {
       const normalized = normalizePathForPersistence(candidate.lastDirectory);
       if (typeof normalized === 'string' && normalized.length > 0) {
         result.lastDirectory = normalized;
@@ -354,6 +359,9 @@ export const createSettingsHelpers = (dependencies) => {
     if (typeof candidate.sidebarShowRecentSection === 'boolean') {
       result.sidebarShowRecentSection = candidate.sidebarShowRecentSection;
     }
+    if (typeof candidate.sidebarShowChatsSection === 'boolean') {
+      result.sidebarShowChatsSection = candidate.sidebarShowChatsSection;
+    }
 
     if (Array.isArray(candidate.securityScopedBookmarks)) {
       result.securityScopedBookmarks = normalizeStringArray(candidate.securityScopedBookmarks);
@@ -389,6 +397,9 @@ export const createSettingsHelpers = (dependencies) => {
     if (typeof candidate.draftStartersScheduleTaskAdded === 'boolean') {
       result.draftStartersScheduleTaskAdded = candidate.draftStartersScheduleTaskAdded;
     }
+    if (candidate.addProjectDialogDismissed === true || candidate.addProjectDialogDismissed === false) {
+      result.addProjectDialogDismissed = candidate.addProjectDialogDismissed;
+    }
 
 
     if (typeof candidate.uiFont === 'string' && candidate.uiFont.length > 0) {
@@ -446,6 +457,9 @@ export const createSettingsHelpers = (dependencies) => {
     if (typeof candidate.diffWrapLines === 'boolean') {
       result.diffWrapLines = candidate.diffWrapLines;
     }
+    if (typeof candidate.diffHideWhitespace === 'boolean') {
+      result.diffHideWhitespace = candidate.diffHideWhitespace;
+    }
     if (typeof candidate.persistChatDraft === 'boolean') {
       result.persistChatDraft = candidate.persistChatDraft;
     }
@@ -470,6 +484,9 @@ export const createSettingsHelpers = (dependencies) => {
         result.fileEditorKeymap = mode;
       }
     }
+    if (typeof candidate.fileEditorVimMappings === 'string') {
+      result.fileEditorVimMappings = candidate.fileEditorVimMappings.slice(0, 10_000);
+    }
     if (Array.isArray(candidate.providerOrder)) {
       result.providerOrder = normalizeStringArray(candidate.providerOrder);
     }
@@ -485,6 +502,12 @@ export const createSettingsHelpers = (dependencies) => {
     if (typeof candidate.sessionWorkAutoOpen === 'boolean') {
       result.sessionWorkAutoOpen = candidate.sessionWorkAutoOpen;
     }
+    if (typeof candidate.sessionReviewOfferEnabled === 'boolean') {
+      result.sessionReviewOfferEnabled = candidate.sessionReviewOfferEnabled;
+    }
+    if (typeof candidate.sessionWorkKeepInGroup === 'boolean') {
+      result.sessionWorkKeepInGroup = candidate.sessionWorkKeepInGroup;
+    }
     if (typeof candidate.sessionGoalEnabled === 'boolean') {
       result.sessionGoalEnabled = candidate.sessionGoalEnabled;
     }
@@ -847,6 +870,9 @@ export const createSettingsHelpers = (dependencies) => {
     if (Array.isArray(candidate.recentAgents)) {
       result.recentAgents = normalizeStringArray(candidate.recentAgents);
     }
+    if (Array.isArray(candidate.favoriteAgents)) {
+      result.favoriteAgents = normalizeStringArray(candidate.favoriteAgents);
+    }
 
     const recentEfforts = sanitizeRecentEfforts(candidate.recentEfforts);
     if (recentEfforts) {

package/server/lib/opencode/settings-helpers.test.js

@@ -76,6 +76,15 @@ describe('settings helpers', () => {
     expect(helpers.sanitizeSettingsUpdate({ gitlabClientId: 42 })).toEqual({});
   });
 
+  it('forgets the last directory when asked with an empty string', () => {
+    // Removing the last project sends this. A kept last directory would be
+    // turned back into a project on the next read and warmed at every start.
+    const helpers = createTestHelpers();
+    expect(helpers.sanitizeSettingsUpdate({ lastDirectory: '' })).toEqual({ lastDirectory: '' });
+    expect(helpers.sanitizeSettingsUpdate({ lastDirectory: '/home/testuser/project' })).toEqual({ lastDirectory: '/home/testuser/project' });
+    expect(helpers.sanitizeSettingsUpdate({ lastDirectory: 42 })).toEqual({});
+  });
+
   it('round-trips section order and preserves it across unrelated writes', () => {
     const helpers = createTestHelpers();
     const changes = helpers.sanitizeSettingsUpdate({ workStatusSectionOrder: ['mcp', 'session', 'mcp', null, ''] });
@@ -101,7 +110,7 @@ describe('settings helpers', () => {
     expect(helpers.formatSettingsResponse(next).workStatusHiddenSectionsExplicit).toBe(true);
     expect(helpers.sanitizeSettingsUpdate({ workStatusHiddenSectionsExplicit: 'true' }).workStatusHiddenSectionsExplicit).toBeUndefined();
   });
-  it('imports from the packed @openchamber/web tarball without escaping the published package', async () => {
+  it('imports from the packed @vantran-se/openchamber-web tarball without escaping the published package', async () => {
     const tempRoot = mkdtempSync(join(tmpdir(), 'settings-helpers-pack-'));
     const packDir = join(tempRoot, 'pack');
     const extractDir = join(tempRoot, 'extract');
@@ -658,6 +667,9 @@ describe('settings helpers', () => {
       expect(helpers.sanitizeSettingsUpdate({ recentAgents: ['build', 'plan'] })).toEqual({
         recentAgents: ['build', 'plan'],
       });
+      expect(helpers.sanitizeSettingsUpdate({ favoriteAgents: ['build', 'incident'] })).toEqual({
+        favoriteAgents: ['build', 'incident'],
+      });
     });
 
     it('round-trips recentEfforts as a Record<string, string[]>', () => {
@@ -810,7 +822,7 @@ describe('settings registry gate', () => {
     lastDirectory: '/home/testuser/project', homeDirectory: '/home/testuser', opencodeBinary: '/usr/local/bin/opencode',
     projects: [{ id: 'p', path: '/home/testuser/project' }], activeProjectId: 'p',
     securityScopedBookmarks: ['bookmark'], pinnedDirectories: ['/home/testuser/project'],
-    desktopLanAccessEnabled: true, desktopKeepAwakeEnabled: true, desktopMinimizeToTrayEnabled: true, desktopMacMenuBarEnabled: true, desktopLinuxNativeFrame: true,
+    desktopLanAccessEnabled: true, desktopKeepAwakeEnabled: true, desktopMinimizeToTrayEnabled: true, desktopMacMenuBarEnabled: true, desktopLinuxNativeFrame: true, sidebarShowChatsSection: false,
     desktopUiPassword: 'secret', githubClientId: 'client', githubScopes: 'repo', gitlabClientId: 'gitlab-client', skillCatalogs: [{ id: 'c', label: 'C', source: 'https://x' }],
     defaultGitIdentityId: 'global', permissionAutoAccept: { sessions: { s: true }, revision: 1 }, permissionDefaultMode: 'safety', messageSearchEnabled: true, messageSearchReasoningEnabled: true,
     agentControlToolEnabled: true, agentWebToolEnabled: true, browserProvider: 'builtin', agentMemoryToolEnabled: true, agentNotifyToolEnabled: true, agentToolsCodeMode: true, isolatedSpacesEnabled: true, isolatedSpacesIdleStop: { enabled: true, hours: 4 }, openCodeUpdateToastDismissedVersion: '1.0.0',
@@ -825,19 +837,19 @@ describe('settings registry gate', () => {
     chatRenderMode: 'live', activityRenderMode: 'summary', mermaidRenderingMode: 'svg', userMessageRenderingMode: 'markdown', collapsibleUserMessages: true,
     stickyUserHeader: true, promptNavigatorEnabled: true, wideChatLayoutEnabled: true, showSplitAssistantMessageActions: true, showToolFileIcons: true,
     codeBlockLineWrap: true, tableCellWrap: true, copyMessagesAsPlainText: true, showTurnChangedFiles: true, showExpandedBashTools: true, showExpandedEditTools: true, toolJsonViewMode: 'raw',
-    timeFormatPreference: '24h', weekStartPreference: 'monday', messageStreamTransport: 'ws', diffLayoutPreference: 'inline', diffWrapLines: true,
+    timeFormatPreference: '24h', weekStartPreference: 'monday', messageStreamTransport: 'ws', diffLayoutPreference: 'inline', diffWrapLines: true, diffHideWhitespace: true,
     gitChangesViewMode: 'tree', gitmojiEnabled: true, defaultFileViewerPreview: true, directoryShowHidden: true, filesViewShowGitignored: true,
-    fileEditorKeymap: 'vim', autoSaveEnabled: true, autoCreateWorktree: true, sessionTabsEnabled: true,
+    fileEditorKeymap: 'vim', fileEditorVimMappings: 'inoremap jk <Esc>', autoSaveEnabled: true, autoCreateWorktree: true, sessionTabsEnabled: true,
     allowPromptingSubagentSessions: true, inputSpellcheckEnabled: true, enterToSend: true, enterToSendConfigured: true, persistChatDraft: true,
     largeTextPasteBehavior: 'attach', followUpBehavior: 'steer', queueModeEnabled: true, inputHistoryScope: 'global', inputHistoryLimit: 40,
-    draftStarters: [{ type: 'command', name: 'plan-feature' }], draftStartersVisible: true, draftStartersCraftGoalAdded: true, draftStartersScheduleTaskAdded: true,
+    draftStarters: [{ type: 'command', name: 'plan-feature' }], draftStartersVisible: true, draftStartersCraftGoalAdded: true, draftStartersScheduleTaskAdded: true, addProjectDialogDismissed: true,
     fontSize: 100, terminalFontSize: 14, editorFontSize: 14, uiFont: 'inter', monoFont: 'jetbrains-mono', customUiFont: 'Maple Mono NF CN', customMonoFont: 'Maple Mono', padding: 100, cornerRadius: 8,
     shortcutOverrides: { 'chat.send': 'mod+enter' },
     defaultModel: 'anthropic/claude', defaultVariant: 'high', defaultAgent: 'build', smallModelUseDefault: false, smallModelOverride: 'anthropic/haiku',
     walkthroughModelOverride: 'anthropic/claude', zenModel: 'zen/model',
     favoriteModels: [{ providerID: 'anthropic', modelID: 'claude' }], hiddenModels: [{ providerID: 'openai', modelID: 'gpt' }], collapsedModelProviders: ['openai'],
-    recentModels: [{ providerID: 'anthropic', modelID: 'claude' }], lastSelectedModel: 'anthropic/claude', recentAgents: ['build'], recentEfforts: { 'anthropic/claude': ['high'] }, providerOrder: ['anthropic'],
-    sessionRecapEnabled: true, sessionSuggestionEnabled: true, sessionWorkEnabled: true, sessionWorkAutoOpen: true, sessionGoalEnabled: true, sessionGoalChecker: 'small-model', sessionGoalMaxAutoTurns: 50, sessionGoalDefaultBudgetEnabled: true, sessionGoalDefaultBudget: 5,
+    recentModels: [{ providerID: 'anthropic', modelID: 'claude' }], lastSelectedModel: 'anthropic/claude', recentAgents: ['build'], favoriteAgents: ['build'], recentEfforts: { 'anthropic/claude': ['high'] }, providerOrder: ['anthropic'],
+    sessionRecapEnabled: true, sessionSuggestionEnabled: true, sessionWorkEnabled: true, sessionWorkAutoOpen: true, sessionReviewOfferEnabled: true, sessionWorkKeepInGroup: true, sessionGoalEnabled: true, sessionGoalChecker: 'small-model', sessionGoalMaxAutoTurns: 50, sessionGoalDefaultBudgetEnabled: true, sessionGoalDefaultBudget: 5,
     summarizeLastMessage: true, summaryThreshold: 100, summaryLength: 50, maxLastMessageLength: 200, showDeletionDialog: true,
     nativeNotificationsEnabled: true, notificationMode: 'always', notifyOnSubtasks: true, notifyOnCompletion: true, notifyOnError: true, notifyOnQuestion: true,
     notificationTemplates: { completion: { title: 't', message: 'm' } }, showOpenCodeUpdateNotifications: true, reportUsage: true,

package/server/lib/opencode/settings-registry.json

@@ -280,6 +280,10 @@
       "scope": "profile",
       "perSurface": true
     },
+    "sidebarShowChatsSection": {
+      "scope": "profile",
+      "perSurface": true
+    },
     "workStatusSectionOrder": {
       "scope": "profile"
     },
@@ -377,6 +381,9 @@
     "diffWrapLines": {
       "scope": "profile"
     },
+    "diffHideWhitespace": {
+      "scope": "profile"
+    },
     "gitChangesViewMode": {
       "scope": "profile"
     },
@@ -395,6 +402,9 @@
     "fileEditorKeymap": {
       "scope": "profile"
     },
+    "fileEditorVimMappings": {
+      "scope": "profile"
+    },
     "autoSaveEnabled": {
       "scope": "profile"
     },
@@ -451,6 +461,9 @@
     "draftStartersScheduleTaskAdded": {
       "scope": "profile"
     },
+    "addProjectDialogDismissed": {
+      "scope": "profile"
+    },
     "fontSize": {
       "scope": "profile",
       "perSurface": true
@@ -528,6 +541,9 @@
     "recentAgents": {
       "scope": "profile"
     },
+    "favoriteAgents": {
+      "scope": "profile"
+    },
     "recentEfforts": {
       "scope": "profile"
     },
@@ -551,6 +567,17 @@
         "mobile"
       ]
     },
+    "sessionReviewOfferEnabled": {
+      "scope": "profile",
+      "surfaces": [
+        "web",
+        "desktop",
+        "mobile"
+      ]
+    },
+    "sessionWorkKeepInGroup": {
+      "scope": "profile"
+    },
     "sessionGoalEnabled": {
       "scope": "profile"
     },
@@ -785,22 +812,6 @@
       "scope": "device",
       "local": true
     },
-    "linearIssueListStatus": {
-      "scope": "device",
-      "local": true
-    },
-    "linearIssueListAssignee": {
-      "scope": "device",
-      "local": true
-    },
-    "linearIssueListTeamIdByRuntime": {
-      "scope": "device",
-      "local": true
-    },
-    "linearIssueListPriority": {
-      "scope": "device",
-      "local": true
-    },
     "showTerminalQuickKeysOnDesktop": {
       "scope": "device",
       "local": true

package/server/lib/opencode/settings-runtime.test.js

@@ -82,6 +82,24 @@ describe('settings runtime', () => {
     }
   });
 
+  it('turns a stored last directory into a project only while one is stored', async () => {
+    const { runtime, tempRoot, cleanup } = await createRuntime();
+    try {
+      const directory = path.join(tempRoot, 'removed-project');
+      await fsPromises.mkdir(directory);
+      // The legacy migration: no projects, a last directory on disk.
+      await runtime.writeSettingsToDisk({ projects: [], lastDirectory: directory });
+      expect((await runtime.readSettingsFromDiskMigrated()).projects).toEqual([
+        expect.objectContaining({ path: directory }),
+      ]);
+      // Removing the last project forgets the last directory with ''.
+      await runtime.writeSettingsToDisk({ projects: [], lastDirectory: '' });
+      expect((await runtime.readSettingsFromDiskMigrated()).projects).toEqual([]);
+    } finally {
+      await cleanup();
+    }
+  });
+
   it('uses OpenChamber themes when a new install has no theme preferences', async () => {
     const { runtime, cleanup } = await createRuntime();
     try {