@pixelbyte-software/pixcode 1.51.8 → 1.52.0

package/server/modules/providers/provider.routes.ts

@@ -1,106 +1,106 @@
-/* eslint-disable import-x/order */
+/* eslint-disable import-x/order */
 import express, { type NextFunction, type Request, type Response } from 'express';
-
-import { providerAuthService } from '@/modules/providers/services/provider-auth.service.js';
-import { providerMcpService } from '@/modules/providers/services/mcp.service.js';
- 
-// @ts-ignore — plain-JS service, typed via inference
-import {
-  applyProviderCredentialsToEnv,
-  listProviderCredentialSummaries,
-  setProviderCredentials,
-  PROVIDER_ENV_VARS,
-} from '@/services/provider-credentials.js';
- 
-// @ts-ignore — plain-JS service
-import {
-  clearProviderModelRegistryCache,
-  getProviderModelRegistryEntry,
-} from '@/services/model-registry.js';
-// @ts-ignore — plain-JS service
-import { getProviderCliVersionStatus } from '@/services/provider-cli-versions.js';
- 
-// @ts-ignore — plain-JS service
-import {
-  createInstallJob,
-  getInstallJob,
-  cancelInstallJob,
-  snapshotDonePayload,
-} from '@/services/install-jobs.js';
-
- 
-// @ts-ignore — plain-JS shared module
-import {
-  MAX_CONFIG_FILE_SIZE_BYTES,
-  PROVIDER_CONFIG_FILES,
-  type ProviderConfigFile,
-} from '@/modules/providers/shared/provider-configs.js';
-import { AppError, asyncHandler, createApiSuccessResponse } from '@/shared/utils.js';
-import type { LLMProvider, McpScope, McpTransport, UpsertProviderMcpServerInput } from '@/shared/types.js';
-
-import fs from 'node:fs/promises';
-import http from 'node:http';
-import os from 'node:os';
-import path from 'node:path';
-
-
-/**
- * npm-global install command per provider. Used by POST
- * /api/providers/:p/install to run the install directly from Pixcode so
- * users don't have to drop into a shell just to get a CLI on the host.
- * Cursor uses its own install script, not npm.
- */
-/**
- * npm package name per provider. The in-app installer drops these into
- * ~/.pixcode/cli-bin/ as LOCAL deps (no -g, no sudo). A sibling string
- * for display ("npm install -g …") is surfaced in the UI so users who
- * prefer to install manually still see a recognizable command.
- */
-const PROVIDER_INSTALL_PACKAGES: Record<LLMProvider, string | null> = {
-  claude: '@anthropic-ai/claude-code',
-  codex: '@openai/codex',
-  gemini: '@google/gemini-cli',
-  qwen: '@qwen-code/qwen-code',
-  opencode: 'opencode-ai',
-  // Cursor ships via a bash script hosted at cursor.com; safer to ask
-  // users to run it themselves than to pipe-to-bash from our server.
-  cursor: null,
-};
-
-const PROVIDER_INSTALL_COMMANDS: Record<LLMProvider, string | null> = {
-  claude: 'npm install -g @anthropic-ai/claude-code',
-  codex: 'npm install -g @openai/codex',
-  gemini: 'npm install -g @google/gemini-cli',
-  qwen: 'npm install -g @qwen-code/qwen-code',
-  opencode: 'npm install -g opencode-ai',
-  cursor: null,
-};
-
-/**
- * Per-provider manual install hints, surfaced when `/install` is called
- * for a provider Pixcode can't sandbox-install (anything not on npm).
- * Each entry includes platform-specific commands so the UI can show the
- * right one for the user's host. Cursor is the only provider in this
- * bucket today — it ships via curl|bash on POSIX and a downloadable
- * installer on Windows. We deliberately don't pipe-to-bash from Pixcode,
- * so the user runs it themselves.
- */
-const PROVIDER_MANUAL_INSTALL: Partial<Record<LLMProvider, {
-  docsUrl: string;
-  steps: { platform: 'macos' | 'linux' | 'windows'; command: string }[];
-  note: string;
-}>> = {
-  cursor: {
-    docsUrl: 'https://docs.cursor.com/en/cli/installation',
-    steps: [
-      { platform: 'macos',   command: 'curl https://cursor.com/install -fsS | bash' },
-      { platform: 'linux',   command: 'curl https://cursor.com/install -fsS | bash' },
-      { platform: 'windows', command: 'iwr https://cursor.com/install.ps1 -useb | iex' },
-    ],
-    note: 'Cursor ships outside npm — run the command for your platform in a separate terminal, then click "Refresh" on this page once the binary is on PATH.',
-  },
-};
-
+
+import { providerAuthService } from '@/modules/providers/services/provider-auth.service.js';
+import { providerMcpService } from '@/modules/providers/services/mcp.service.js';
+ 
+// @ts-ignore — plain-JS service, typed via inference
+import {
+  applyProviderCredentialsToEnv,
+  listProviderCredentialSummaries,
+  setProviderCredentials,
+  PROVIDER_ENV_VARS,
+} from '@/services/provider-credentials.js';
+ 
+// @ts-ignore — plain-JS service
+import {
+  clearProviderModelRegistryCache,
+  getProviderModelRegistryEntry,
+} from '@/services/model-registry.js';
+// @ts-ignore — plain-JS service
+import { getProviderCliVersionStatus } from '@/services/provider-cli-versions.js';
+ 
+// @ts-ignore — plain-JS service
+import {
+  createInstallJob,
+  getInstallJob,
+  cancelInstallJob,
+  snapshotDonePayload,
+} from '@/services/install-jobs.js';
+
+ 
+// @ts-ignore — plain-JS shared module
+import {
+  MAX_CONFIG_FILE_SIZE_BYTES,
+  PROVIDER_CONFIG_FILES,
+  type ProviderConfigFile,
+} from '@/modules/providers/shared/provider-configs.js';
+import { AppError, asyncHandler, createApiSuccessResponse } from '@/shared/utils.js';
+import type { LLMProvider, McpScope, McpTransport, UpsertProviderMcpServerInput } from '@/shared/types.js';
+
+import fs from 'node:fs/promises';
+import http from 'node:http';
+import os from 'node:os';
+import path from 'node:path';
+
+
+/**
+ * npm-global install command per provider. Used by POST
+ * /api/providers/:p/install to run the install directly from Pixcode so
+ * users don't have to drop into a shell just to get a CLI on the host.
+ * Cursor uses its own install script, not npm.
+ */
+/**
+ * npm package name per provider. The in-app installer drops these into
+ * ~/.pixcode/cli-bin/ as LOCAL deps (no -g, no sudo). A sibling string
+ * for display ("npm install -g …") is surfaced in the UI so users who
+ * prefer to install manually still see a recognizable command.
+ */
+const PROVIDER_INSTALL_PACKAGES: Record<LLMProvider, string | null> = {
+  claude: '@anthropic-ai/claude-code',
+  codex: '@openai/codex',
+  gemini: '@google/gemini-cli',
+  qwen: '@qwen-code/qwen-code',
+  opencode: 'opencode-ai',
+  // Cursor ships via a bash script hosted at cursor.com; safer to ask
+  // users to run it themselves than to pipe-to-bash from our server.
+  cursor: null,
+};
+
+const PROVIDER_INSTALL_COMMANDS: Record<LLMProvider, string | null> = {
+  claude: 'npm install -g @anthropic-ai/claude-code',
+  codex: 'npm install -g @openai/codex',
+  gemini: 'npm install -g @google/gemini-cli',
+  qwen: 'npm install -g @qwen-code/qwen-code',
+  opencode: 'npm install -g opencode-ai',
+  cursor: null,
+};
+
+/**
+ * Per-provider manual install hints, surfaced when `/install` is called
+ * for a provider Pixcode can't sandbox-install (anything not on npm).
+ * Each entry includes platform-specific commands so the UI can show the
+ * right one for the user's host. Cursor is the only provider in this
+ * bucket today — it ships via curl|bash on POSIX and a downloadable
+ * installer on Windows. We deliberately don't pipe-to-bash from Pixcode,
+ * so the user runs it themselves.
+ */
+const PROVIDER_MANUAL_INSTALL: Partial<Record<LLMProvider, {
+  docsUrl: string;
+  steps: { platform: 'macos' | 'linux' | 'windows'; command: string }[];
+  note: string;
+}>> = {
+  cursor: {
+    docsUrl: 'https://docs.cursor.com/en/cli/installation',
+    steps: [
+      { platform: 'macos',   command: 'curl https://cursor.com/install -fsS | bash' },
+      { platform: 'linux',   command: 'curl https://cursor.com/install -fsS | bash' },
+      { platform: 'windows', command: 'iwr https://cursor.com/install.ps1 -useb | iex' },
+    ],
+    note: 'Cursor ships outside npm — run the command for your platform in a separate terminal, then click "Refresh" on this page once the binary is on PATH.',
+  },
+};
+
 const router = express.Router();
 const ADMIN_ROLES = new Set(['owner', 'admin']);
 
@@ -123,860 +123,860 @@ function requireProviderAdmin(req: Request, res: Response, next: NextFunction) {
 
   next();
 }
-
-const readPathParam = (value: unknown, name: string): string => {
-  if (typeof value === 'string') {
-    return value;
-  }
-
-  if (Array.isArray(value) && typeof value[0] === 'string') {
-    return value[0];
-  }
-
-  throw new AppError(`${name} path parameter is invalid.`, {
-    code: 'INVALID_PATH_PARAMETER',
-    statusCode: 400,
-  });
-};
-
-const normalizeProviderParam = (value: unknown): string =>
-  readPathParam(value, 'provider').trim().toLowerCase();
-
-const readOptionalQueryString = (value: unknown): string | undefined => {
-  if (typeof value !== 'string') {
-    return undefined;
-  }
-
-  const normalized = value.trim();
-  return normalized.length > 0 ? normalized : undefined;
-};
-
-const parseMcpScope = (value: unknown): McpScope | undefined => {
-  if (value === undefined) {
-    return undefined;
-  }
-
-  const normalized = readOptionalQueryString(value);
-  if (!normalized) {
-    return undefined;
-  }
-
-  if (normalized === 'user' || normalized === 'local' || normalized === 'project') {
-    return normalized;
-  }
-
-  throw new AppError(`Unsupported MCP scope "${normalized}".`, {
-    code: 'INVALID_MCP_SCOPE',
-    statusCode: 400,
-  });
-};
-
-const parseMcpTransport = (value: unknown): McpTransport => {
-  const normalized = readOptionalQueryString(value);
-  if (!normalized) {
-    throw new AppError('transport is required.', {
-      code: 'MCP_TRANSPORT_REQUIRED',
-      statusCode: 400,
-    });
-  }
-
-  if (normalized === 'stdio' || normalized === 'http' || normalized === 'sse') {
-    return normalized;
-  }
-
-  throw new AppError(`Unsupported MCP transport "${normalized}".`, {
-    code: 'INVALID_MCP_TRANSPORT',
-    statusCode: 400,
-  });
-};
-
-const parseMcpUpsertPayload = (payload: unknown): UpsertProviderMcpServerInput => {
-  if (!payload || typeof payload !== 'object') {
-    throw new AppError('Request body must be an object.', {
-      code: 'INVALID_REQUEST_BODY',
-      statusCode: 400,
-    });
-  }
-
-  const body = payload as Record<string, unknown>;
-  const name = readOptionalQueryString(body.name);
-  if (!name) {
-    throw new AppError('name is required.', {
-      code: 'MCP_NAME_REQUIRED',
-      statusCode: 400,
-    });
-  }
-
-  const transport = parseMcpTransport(body.transport);
-  const scope = parseMcpScope(body.scope);
-  const workspacePath = readOptionalQueryString(body.workspacePath);
-
-  return {
-    name,
-    transport,
-    scope,
-    workspacePath,
-    command: readOptionalQueryString(body.command),
-    args: Array.isArray(body.args) ? body.args.filter((entry): entry is string => typeof entry === 'string') : undefined,
-    env: typeof body.env === 'object' && body.env !== null
-      ? Object.fromEntries(
-          Object.entries(body.env as Record<string, unknown>).filter(
-            (entry): entry is [string, string] => typeof entry[1] === 'string',
-          ),
-        )
-      : undefined,
-    cwd: readOptionalQueryString(body.cwd),
-    url: readOptionalQueryString(body.url),
-    headers: typeof body.headers === 'object' && body.headers !== null
-      ? Object.fromEntries(
-          Object.entries(body.headers as Record<string, unknown>).filter(
-            (entry): entry is [string, string] => typeof entry[1] === 'string',
-          ),
-        )
-      : undefined,
-    envVars: Array.isArray(body.envVars)
-      ? body.envVars.filter((entry): entry is string => typeof entry === 'string')
-      : undefined,
-    bearerTokenEnvVar: readOptionalQueryString(body.bearerTokenEnvVar),
-    envHttpHeaders: typeof body.envHttpHeaders === 'object' && body.envHttpHeaders !== null
-      ? Object.fromEntries(
-          Object.entries(body.envHttpHeaders as Record<string, unknown>).filter(
-            (entry): entry is [string, string] => typeof entry[1] === 'string',
-          ),
-        )
-      : undefined,
-  };
-};
-
-const parseProvider = (value: unknown): LLMProvider => {
-  const normalized = normalizeProviderParam(value);
-  if (
-    normalized === 'claude' ||
-    normalized === 'codex' ||
-    normalized === 'cursor' ||
-    normalized === 'gemini' ||
-    normalized === 'qwen' ||
-    normalized === 'opencode'
-  ) {
-    return normalized;
-  }
-
-  throw new AppError(`Unsupported provider "${normalized}".`, {
-    code: 'UNSUPPORTED_PROVIDER',
-    statusCode: 400,
-  });
-};
-
-router.get(
-  '/:provider/auth/status',
-  asyncHandler(async (req: Request, res: Response) => {
-    const provider = parseProvider(req.params.provider);
-    const forceRefresh = String(req.query.refresh || '').toLowerCase() === '1';
-    const status = await providerAuthService.getProviderAuthStatus(provider);
-    const cliVersion = await getProviderCliVersionStatus(provider, {
-      installed: status.installed,
-      forceRefresh,
-    });
-    res.json(createApiSuccessResponse({ ...status, ...cliVersion }));
-  }),
-);
-
+
+const readPathParam = (value: unknown, name: string): string => {
+  if (typeof value === 'string') {
+    return value;
+  }
+
+  if (Array.isArray(value) && typeof value[0] === 'string') {
+    return value[0];
+  }
+
+  throw new AppError(`${name} path parameter is invalid.`, {
+    code: 'INVALID_PATH_PARAMETER',
+    statusCode: 400,
+  });
+};
+
+const normalizeProviderParam = (value: unknown): string =>
+  readPathParam(value, 'provider').trim().toLowerCase();
+
+const readOptionalQueryString = (value: unknown): string | undefined => {
+  if (typeof value !== 'string') {
+    return undefined;
+  }
+
+  const normalized = value.trim();
+  return normalized.length > 0 ? normalized : undefined;
+};
+
+const parseMcpScope = (value: unknown): McpScope | undefined => {
+  if (value === undefined) {
+    return undefined;
+  }
+
+  const normalized = readOptionalQueryString(value);
+  if (!normalized) {
+    return undefined;
+  }
+
+  if (normalized === 'user' || normalized === 'local' || normalized === 'project') {
+    return normalized;
+  }
+
+  throw new AppError(`Unsupported MCP scope "${normalized}".`, {
+    code: 'INVALID_MCP_SCOPE',
+    statusCode: 400,
+  });
+};
+
+const parseMcpTransport = (value: unknown): McpTransport => {
+  const normalized = readOptionalQueryString(value);
+  if (!normalized) {
+    throw new AppError('transport is required.', {
+      code: 'MCP_TRANSPORT_REQUIRED',
+      statusCode: 400,
+    });
+  }
+
+  if (normalized === 'stdio' || normalized === 'http' || normalized === 'sse') {
+    return normalized;
+  }
+
+  throw new AppError(`Unsupported MCP transport "${normalized}".`, {
+    code: 'INVALID_MCP_TRANSPORT',
+    statusCode: 400,
+  });
+};
+
+const parseMcpUpsertPayload = (payload: unknown): UpsertProviderMcpServerInput => {
+  if (!payload || typeof payload !== 'object') {
+    throw new AppError('Request body must be an object.', {
+      code: 'INVALID_REQUEST_BODY',
+      statusCode: 400,
+    });
+  }
+
+  const body = payload as Record<string, unknown>;
+  const name = readOptionalQueryString(body.name);
+  if (!name) {
+    throw new AppError('name is required.', {
+      code: 'MCP_NAME_REQUIRED',
+      statusCode: 400,
+    });
+  }
+
+  const transport = parseMcpTransport(body.transport);
+  const scope = parseMcpScope(body.scope);
+  const workspacePath = readOptionalQueryString(body.workspacePath);
+
+  return {
+    name,
+    transport,
+    scope,
+    workspacePath,
+    command: readOptionalQueryString(body.command),
+    args: Array.isArray(body.args) ? body.args.filter((entry): entry is string => typeof entry === 'string') : undefined,
+    env: typeof body.env === 'object' && body.env !== null
+      ? Object.fromEntries(
+          Object.entries(body.env as Record<string, unknown>).filter(
+            (entry): entry is [string, string] => typeof entry[1] === 'string',
+          ),
+        )
+      : undefined,
+    cwd: readOptionalQueryString(body.cwd),
+    url: readOptionalQueryString(body.url),
+    headers: typeof body.headers === 'object' && body.headers !== null
+      ? Object.fromEntries(
+          Object.entries(body.headers as Record<string, unknown>).filter(
+            (entry): entry is [string, string] => typeof entry[1] === 'string',
+          ),
+        )
+      : undefined,
+    envVars: Array.isArray(body.envVars)
+      ? body.envVars.filter((entry): entry is string => typeof entry === 'string')
+      : undefined,
+    bearerTokenEnvVar: readOptionalQueryString(body.bearerTokenEnvVar),
+    envHttpHeaders: typeof body.envHttpHeaders === 'object' && body.envHttpHeaders !== null
+      ? Object.fromEntries(
+          Object.entries(body.envHttpHeaders as Record<string, unknown>).filter(
+            (entry): entry is [string, string] => typeof entry[1] === 'string',
+          ),
+        )
+      : undefined,
+  };
+};
+
+const parseProvider = (value: unknown): LLMProvider => {
+  const normalized = normalizeProviderParam(value);
+  if (
+    normalized === 'claude' ||
+    normalized === 'codex' ||
+    normalized === 'cursor' ||
+    normalized === 'gemini' ||
+    normalized === 'qwen' ||
+    normalized === 'opencode'
+  ) {
+    return normalized;
+  }
+
+  throw new AppError(`Unsupported provider "${normalized}".`, {
+    code: 'UNSUPPORTED_PROVIDER',
+    statusCode: 400,
+  });
+};
+
+router.get(
+  '/:provider/auth/status',
+  asyncHandler(async (req: Request, res: Response) => {
+    const provider = parseProvider(req.params.provider);
+    const forceRefresh = String(req.query.refresh || '').toLowerCase() === '1';
+    const status = await providerAuthService.getProviderAuthStatus(provider);
+    const cliVersion = await getProviderCliVersionStatus(provider, {
+      installed: status.installed,
+      forceRefresh,
+    });
+    res.json(createApiSuccessResponse({ ...status, ...cliVersion }));
+  }),
+);
+
 router.get(
   '/:provider/mcp/servers',
   requireProviderAdmin,
   asyncHandler(async (req: Request, res: Response) => {
-    const provider = parseProvider(req.params.provider);
-    const workspacePath = readOptionalQueryString(req.query.workspacePath);
-    const scope = parseMcpScope(req.query.scope);
-
-    if (scope) {
-      const servers = await providerMcpService.listProviderMcpServersForScope(provider, scope, { workspacePath });
-      res.json(createApiSuccessResponse({ provider, scope, servers }));
-      return;
-    }
-
-    const groupedServers = await providerMcpService.listProviderMcpServers(provider, { workspacePath });
-    res.json(createApiSuccessResponse({ provider, scopes: groupedServers }));
-  }),
-);
-
+    const provider = parseProvider(req.params.provider);
+    const workspacePath = readOptionalQueryString(req.query.workspacePath);
+    const scope = parseMcpScope(req.query.scope);
+
+    if (scope) {
+      const servers = await providerMcpService.listProviderMcpServersForScope(provider, scope, { workspacePath });
+      res.json(createApiSuccessResponse({ provider, scope, servers }));
+      return;
+    }
+
+    const groupedServers = await providerMcpService.listProviderMcpServers(provider, { workspacePath });
+    res.json(createApiSuccessResponse({ provider, scopes: groupedServers }));
+  }),
+);
+
 router.post(
   '/:provider/mcp/servers',
   requireProviderAdmin,
   asyncHandler(async (req: Request, res: Response) => {
-    const provider = parseProvider(req.params.provider);
-    const payload = parseMcpUpsertPayload(req.body);
-    const server = await providerMcpService.upsertProviderMcpServer(provider, payload);
-    res.status(201).json(createApiSuccessResponse({ server }));
-  }),
-);
-
+    const provider = parseProvider(req.params.provider);
+    const payload = parseMcpUpsertPayload(req.body);
+    const server = await providerMcpService.upsertProviderMcpServer(provider, payload);
+    res.status(201).json(createApiSuccessResponse({ server }));
+  }),
+);
+
 router.delete(
   '/:provider/mcp/servers/:name',
   requireProviderAdmin,
   asyncHandler(async (req: Request, res: Response) => {
-    const provider = parseProvider(req.params.provider);
-    const scope = parseMcpScope(req.query.scope);
-    const workspacePath = readOptionalQueryString(req.query.workspacePath);
-    const result = await providerMcpService.removeProviderMcpServer(provider, {
-      name: readPathParam(req.params.name, 'name'),
-      scope,
-      workspacePath,
-    });
-    res.json(createApiSuccessResponse(result));
-  }),
-);
-
-/**
- * GET /api/providers/credentials
- * Summary for every provider (hasKey + baseUrl + updatedAt). Used by the
- * Settings UI to pre-fill the "API Key" tab.
- */
+    const provider = parseProvider(req.params.provider);
+    const scope = parseMcpScope(req.query.scope);
+    const workspacePath = readOptionalQueryString(req.query.workspacePath);
+    const result = await providerMcpService.removeProviderMcpServer(provider, {
+      name: readPathParam(req.params.name, 'name'),
+      scope,
+      workspacePath,
+    });
+    res.json(createApiSuccessResponse(result));
+  }),
+);
+
+/**
+ * GET /api/providers/credentials
+ * Summary for every provider (hasKey + baseUrl + updatedAt). Used by the
+ * Settings UI to pre-fill the "API Key" tab.
+ */
 router.get(
   '/credentials',
   requireProviderAdmin,
   asyncHandler(async (_req: Request, res: Response) => {
-    const summaries = await listProviderCredentialSummaries();
-    res.json(createApiSuccessResponse(summaries));
-  }),
-);
-
-/**
- * POST /api/providers/:provider/auth/api-key
- * Body: { apiKey: string, baseUrl?: string }. Stores the credentials in
- * ~/.pixcode/provider-credentials.json and applies them to process.env
- * so the next CLI spawn/SDK call picks them up. Empty apiKey clears.
- */
+    const summaries = await listProviderCredentialSummaries();
+    res.json(createApiSuccessResponse(summaries));
+  }),
+);
+
+/**
+ * POST /api/providers/:provider/auth/api-key
+ * Body: { apiKey: string, baseUrl?: string }. Stores the credentials in
+ * ~/.pixcode/provider-credentials.json and applies them to process.env
+ * so the next CLI spawn/SDK call picks them up. Empty apiKey clears.
+ */
 router.post(
   '/:provider/auth/api-key',
   requireProviderAdmin,
   asyncHandler(async (req: Request, res: Response) => {
-    const provider = parseProvider(req.params.provider);
-    if (!(provider in PROVIDER_ENV_VARS)) {
-      throw new AppError(`Provider "${provider}" does not accept API-key auth.`, {
-        code: 'PROVIDER_NO_API_KEY',
-        statusCode: 400,
-      });
-    }
-    const body = (req.body ?? {}) as Record<string, unknown>;
-    const apiKey = typeof body.apiKey === 'string' ? body.apiKey : '';
-    const baseUrl = typeof body.baseUrl === 'string' ? body.baseUrl : '';
-
-    await setProviderCredentials(provider, { apiKey, baseUrl });
-    await applyProviderCredentialsToEnv(provider);
-
-    res.json(createApiSuccessResponse({ provider, stored: Boolean(apiKey.trim()) }));
-  }),
-);
-
-/**
- * POST /api/providers/:provider/oauth-paste
- * Body: { callbackUrl: string }.
- *
- * When the CLI starts an OAuth flow it spins up a local HTTP server on
- * 127.0.0.1:<PORT> and expects the OAuth provider to redirect the user's
- * browser to `http://127.0.0.1:<PORT>/callback?code=...`. On remote VPS
- * setups that redirect hits the user's laptop localhost (which has nothing
- * listening), not the server running the CLI. This endpoint is the escape
- * hatch: the user copies the dead callback URL from their browser and
- * posts it here; we parse out the port + code and forward the original
- * GET to the VPS-side 127.0.0.1:PORT so the CLI's local handler completes
- * the token exchange.
- */
+    const provider = parseProvider(req.params.provider);
+    if (!(provider in PROVIDER_ENV_VARS)) {
+      throw new AppError(`Provider "${provider}" does not accept API-key auth.`, {
+        code: 'PROVIDER_NO_API_KEY',
+        statusCode: 400,
+      });
+    }
+    const body = (req.body ?? {}) as Record<string, unknown>;
+    const apiKey = typeof body.apiKey === 'string' ? body.apiKey : '';
+    const baseUrl = typeof body.baseUrl === 'string' ? body.baseUrl : '';
+
+    await setProviderCredentials(provider, { apiKey, baseUrl });
+    await applyProviderCredentialsToEnv(provider);
+
+    res.json(createApiSuccessResponse({ provider, stored: Boolean(apiKey.trim()) }));
+  }),
+);
+
+/**
+ * POST /api/providers/:provider/oauth-paste
+ * Body: { callbackUrl: string }.
+ *
+ * When the CLI starts an OAuth flow it spins up a local HTTP server on
+ * 127.0.0.1:<PORT> and expects the OAuth provider to redirect the user's
+ * browser to `http://127.0.0.1:<PORT>/callback?code=...`. On remote VPS
+ * setups that redirect hits the user's laptop localhost (which has nothing
+ * listening), not the server running the CLI. This endpoint is the escape
+ * hatch: the user copies the dead callback URL from their browser and
+ * posts it here; we parse out the port + code and forward the original
+ * GET to the VPS-side 127.0.0.1:PORT so the CLI's local handler completes
+ * the token exchange.
+ */
 router.post(
   '/:provider/oauth-paste',
   requireProviderAdmin,
   asyncHandler(async (req: Request, res: Response) => {
-    parseProvider(req.params.provider); // validate id but we don't use it further
-    const body = (req.body ?? {}) as Record<string, unknown>;
-    const raw = typeof body.callbackUrl === 'string' ? body.callbackUrl.trim() : '';
-    if (!raw) {
-      throw new AppError('callbackUrl is required.', {
-        code: 'OAUTH_PASTE_URL_REQUIRED',
-        statusCode: 400,
-      });
-    }
-
-    let parsed: URL;
-    try {
-      parsed = new URL(raw);
-    } catch {
-      throw new AppError('callbackUrl must be a valid URL.', {
-        code: 'OAUTH_PASTE_URL_INVALID',
-        statusCode: 400,
-      });
-    }
-
-    // Accept localhost / 127.0.0.1 callbacks — reject anything else so we
-    // never proxy arbitrary outbound requests on behalf of a user.
-    const host = parsed.hostname;
-    if (host !== '127.0.0.1' && host !== 'localhost' && host !== '::1') {
-      throw new AppError('Only local CLI callback URLs are accepted.', {
-        code: 'OAUTH_PASTE_URL_NOT_LOCAL',
-        statusCode: 400,
-      });
-    }
-
-    const port = Number(parsed.port);
-    if (!port || port < 1 || port > 65535) {
-      throw new AppError('Callback URL must include the CLI callback port.', {
-        code: 'OAUTH_PASTE_PORT_INVALID',
-        statusCode: 400,
-      });
-    }
-
-    const pathAndQuery = parsed.pathname + parsed.search;
-    await new Promise<void>((resolve, reject) => {
-      const forwardReq = http.request(
-        {
-          host: '127.0.0.1',
-          port,
-          method: 'GET',
-          path: pathAndQuery,
-          timeout: 10000,
-        },
-        (forwardRes) => {
-          forwardRes.resume(); // drain
-          forwardRes.on('end', () => resolve());
-        },
-      );
-      forwardReq.on('timeout', () => {
-        forwardReq.destroy(new Error('CLI callback server did not respond within 10s'));
-      });
-      forwardReq.on('error', (err) => reject(err));
-      forwardReq.end();
-    });
-
-    res.json(createApiSuccessResponse({ forwarded: true, port }));
-  }),
-);
-
-/**
- * GET /api/providers/:provider/models?refresh=1
- * Provider model registry entry: central static fallback + live API discovery
- * with freshness/degraded metadata. 6-hour cache; pass `refresh=1` to force an
- * upstream hit.
- */
-router.get(
-  '/:provider/models',
-  asyncHandler(async (req: Request, res: Response) => {
-    const provider = parseProvider(req.params.provider);
-    const forceRefresh = String(req.query.refresh || '').toLowerCase() === '1';
-    const result = await getProviderModelRegistryEntry(provider, { forceRefresh });
-    res.json(createApiSuccessResponse(result));
-  }),
-);
-
+    parseProvider(req.params.provider); // validate id but we don't use it further
+    const body = (req.body ?? {}) as Record<string, unknown>;
+    const raw = typeof body.callbackUrl === 'string' ? body.callbackUrl.trim() : '';
+    if (!raw) {
+      throw new AppError('callbackUrl is required.', {
+        code: 'OAUTH_PASTE_URL_REQUIRED',
+        statusCode: 400,
+      });
+    }
+
+    let parsed: URL;
+    try {
+      parsed = new URL(raw);
+    } catch {
+      throw new AppError('callbackUrl must be a valid URL.', {
+        code: 'OAUTH_PASTE_URL_INVALID',
+        statusCode: 400,
+      });
+    }
+
+    // Accept localhost / 127.0.0.1 callbacks — reject anything else so we
+    // never proxy arbitrary outbound requests on behalf of a user.
+    const host = parsed.hostname;
+    if (host !== '127.0.0.1' && host !== 'localhost' && host !== '::1') {
+      throw new AppError('Only local CLI callback URLs are accepted.', {
+        code: 'OAUTH_PASTE_URL_NOT_LOCAL',
+        statusCode: 400,
+      });
+    }
+
+    const port = Number(parsed.port);
+    if (!port || port < 1 || port > 65535) {
+      throw new AppError('Callback URL must include the CLI callback port.', {
+        code: 'OAUTH_PASTE_PORT_INVALID',
+        statusCode: 400,
+      });
+    }
+
+    const pathAndQuery = parsed.pathname + parsed.search;
+    await new Promise<void>((resolve, reject) => {
+      const forwardReq = http.request(
+        {
+          host: '127.0.0.1',
+          port,
+          method: 'GET',
+          path: pathAndQuery,
+          timeout: 10000,
+        },
+        (forwardRes) => {
+          forwardRes.resume(); // drain
+          forwardRes.on('end', () => resolve());
+        },
+      );
+      forwardReq.on('timeout', () => {
+        forwardReq.destroy(new Error('CLI callback server did not respond within 10s'));
+      });
+      forwardReq.on('error', (err) => reject(err));
+      forwardReq.end();
+    });
+
+    res.json(createApiSuccessResponse({ forwarded: true, port }));
+  }),
+);
+
+/**
+ * GET /api/providers/:provider/models?refresh=1
+ * Provider model registry entry: central static fallback + live API discovery
+ * with freshness/degraded metadata. 6-hour cache; pass `refresh=1` to force an
+ * upstream hit.
+ */
+router.get(
+  '/:provider/models',
+  asyncHandler(async (req: Request, res: Response) => {
+    const provider = parseProvider(req.params.provider);
+    const forceRefresh = String(req.query.refresh || '').toLowerCase() === '1';
+    const result = await getProviderModelRegistryEntry(provider, { forceRefresh });
+    res.json(createApiSuccessResponse(result));
+  }),
+);
+
 router.delete(
   '/:provider/models/cache',
   requireProviderAdmin,
   asyncHandler(async (req: Request, res: Response) => {
-    const provider = parseProvider(req.params.provider);
-    await clearProviderModelRegistryCache(provider);
-    res.json(createApiSuccessResponse({ cleared: true, provider }));
-  }),
-);
-
-/**
- * POST /api/providers/:provider/install
- * Kicks off the install in the background and immediately returns
- * `{ jobId }`. The actual log stream is fetched separately via
- * GET /install/:jobId/stream (EventSource). This split solves the
- * "Client disconnected before install finished" class of errors,
- * where a single long-lived POST SSE would get torn down by dev
- * proxies, service-worker reloads, or Vite HMR and short-circuit
- * an in-flight install. The child now outlives the request.
- */
+    const provider = parseProvider(req.params.provider);
+    await clearProviderModelRegistryCache(provider);
+    res.json(createApiSuccessResponse({ cleared: true, provider }));
+  }),
+);
+
+/**
+ * POST /api/providers/:provider/install
+ * Kicks off the install in the background and immediately returns
+ * `{ jobId }`. The actual log stream is fetched separately via
+ * GET /install/:jobId/stream (EventSource). This split solves the
+ * "Client disconnected before install finished" class of errors,
+ * where a single long-lived POST SSE would get torn down by dev
+ * proxies, service-worker reloads, or Vite HMR and short-circuit
+ * an in-flight install. The child now outlives the request.
+ */
 router.post(
   '/:provider/install',
   requireProviderAdmin,
   asyncHandler(async (req: Request, res: Response) => {
-    const parsed = parseProvider(req.params.provider);
-    const packageName = PROVIDER_INSTALL_PACKAGES[parsed];
-    const installCmd = PROVIDER_INSTALL_COMMANDS[parsed];
-    if (!packageName || !installCmd) {
-      const manual = PROVIDER_MANUAL_INSTALL[parsed];
-      // Don't 4xx on this — the call ISN'T malformed, the provider is
-      // simply not npm-installable. Return 200 with a `manual` payload
-      // the UI can present as instructions instead of "install failed".
-      res.json(createApiSuccessResponse({
-        provider: parsed,
-        manual: manual || null,
-        message: manual
-          ? `${parsed} ships outside npm. Run the platform-specific command, then refresh.`
-          : `${parsed} cannot be installed automatically — see the provider's documentation.`,
-      }));
-      return;
-    }
-
-    const job = createInstallJob({ provider: parsed, installCmd, packageName });
-    res.json(createApiSuccessResponse({
-      jobId: job.id,
-      provider: parsed,
-      installCmd,
-      startedAt: job.startedAt,
-    }));
-  }),
-);
-
-/**
- * GET /api/providers/:provider/install/:jobId/stream
- * SSE endpoint (EventSource-friendly). Replays every buffered log line
- * to the new subscriber, then forwards live stdout/stderr until the
- * child exits. Clients can reconnect freely — reconnects replay from
- * the start, so you never miss output, even if the browser dropped
- * the previous connection while npm was mid-download.
- *
- * EventSource can't set custom headers, so this endpoint also accepts
- * ?token=... as a fallback auth channel (same pattern the search
- * endpoint uses).
- */
+    const parsed = parseProvider(req.params.provider);
+    const packageName = PROVIDER_INSTALL_PACKAGES[parsed];
+    const installCmd = PROVIDER_INSTALL_COMMANDS[parsed];
+    if (!packageName || !installCmd) {
+      const manual = PROVIDER_MANUAL_INSTALL[parsed];
+      // Don't 4xx on this — the call ISN'T malformed, the provider is
+      // simply not npm-installable. Return 200 with a `manual` payload
+      // the UI can present as instructions instead of "install failed".
+      res.json(createApiSuccessResponse({
+        provider: parsed,
+        manual: manual || null,
+        message: manual
+          ? `${parsed} ships outside npm. Run the platform-specific command, then refresh.`
+          : `${parsed} cannot be installed automatically — see the provider's documentation.`,
+      }));
+      return;
+    }
+
+    const job = createInstallJob({ provider: parsed, installCmd, packageName });
+    res.json(createApiSuccessResponse({
+      jobId: job.id,
+      provider: parsed,
+      installCmd,
+      startedAt: job.startedAt,
+    }));
+  }),
+);
+
+/**
+ * GET /api/providers/:provider/install/:jobId/stream
+ * SSE endpoint (EventSource-friendly). Replays every buffered log line
+ * to the new subscriber, then forwards live stdout/stderr until the
+ * child exits. Clients can reconnect freely — reconnects replay from
+ * the start, so you never miss output, even if the browser dropped
+ * the previous connection while npm was mid-download.
+ *
+ * EventSource can't set custom headers, so this endpoint also accepts
+ * ?token=... as a fallback auth channel (same pattern the search
+ * endpoint uses).
+ */
 router.get(
   '/:provider/install/:jobId/stream',
   requireProviderAdmin,
   asyncHandler(async (req: Request, res: Response) => {
-    const parsed = parseProvider(req.params.provider);
-    const jobId = readPathParam(req.params.jobId, 'jobId');
-    const job = getInstallJob(jobId);
-    if (!job || job.provider !== parsed) {
-      throw new AppError('Install job not found or already expired.', {
-        code: 'INSTALL_JOB_NOT_FOUND',
-        statusCode: 404,
-      });
-    }
-
-    res.setHeader('Content-Type', 'text/event-stream');
-    res.setHeader('Cache-Control', 'no-cache, no-transform');
-    res.setHeader('Connection', 'keep-alive');
-    res.setHeader('X-Accel-Buffering', 'no');
-    if (typeof res.flushHeaders === 'function') res.flushHeaders();
-    try {
-      (res.socket as NodeJS.Socket & { setNoDelay?: (on: boolean) => void })?.setNoDelay?.(true);
-    } catch { /* noop */ }
-
-    let closed = false;
-    const write = (event: string, payload: unknown) => {
-      if (closed) return;
-      try {
-        res.write(`event: ${event}\n`);
-        res.write(`data: ${JSON.stringify(payload)}\n\n`);
-      } catch { /* socket gone */ }
-    };
-
-    // Immediate primer + heartbeat, same as before — keeps intermediary
-    // proxies from treating the connection as idle.
-    try { res.write(': start\n\n'); } catch { /* noop */ }
-    const heartbeat = setInterval(() => {
-      if (closed) return;
-      try { res.write(': ping\n\n'); } catch { /* noop */ }
-    }, 5000);
-
-    // Replay the buffered transcript first so late subscribers see
-    // every line npm has already produced.
-    for (const entry of job.logs) {
-      write('log', { stream: entry.stream, chunk: entry.chunk });
-    }
-
-    const onLog = (entry: { stream: string; chunk: string }) => {
-      write('log', { stream: entry.stream, chunk: entry.chunk });
-    };
-    const onDone = (payload: Record<string, unknown>) => {
-      write('done', payload);
-      cleanup();
-      try { res.end(); } catch { /* noop */ }
-    };
-
-    const cleanup = () => {
-      if (closed) return;
-      closed = true;
-      clearInterval(heartbeat);
-      job.emitter.off('log', onLog);
-      job.emitter.off('done', onDone);
-    };
-
-    if (job.status !== 'running') {
-      // Job already finished — replay the terminal done frame and exit.
-      write('done', snapshotDonePayload(job));
-      cleanup();
-      try { res.end(); } catch { /* noop */ }
-      return;
-    }
-
-    job.emitter.on('log', onLog);
-    job.emitter.once('done', onDone);
-
-    req.on('close', () => {
-      // Client walked away. DO NOT cancel the install — detaching is fine.
-      cleanup();
-    });
-  }),
-);
-
+    const parsed = parseProvider(req.params.provider);
+    const jobId = readPathParam(req.params.jobId, 'jobId');
+    const job = getInstallJob(jobId);
+    if (!job || job.provider !== parsed) {
+      throw new AppError('Install job not found or already expired.', {
+        code: 'INSTALL_JOB_NOT_FOUND',
+        statusCode: 404,
+      });
+    }
+
+    res.setHeader('Content-Type', 'text/event-stream');
+    res.setHeader('Cache-Control', 'no-cache, no-transform');
+    res.setHeader('Connection', 'keep-alive');
+    res.setHeader('X-Accel-Buffering', 'no');
+    if (typeof res.flushHeaders === 'function') res.flushHeaders();
+    try {
+      (res.socket as NodeJS.Socket & { setNoDelay?: (on: boolean) => void })?.setNoDelay?.(true);
+    } catch { /* noop */ }
+
+    let closed = false;
+    const write = (event: string, payload: unknown) => {
+      if (closed) return;
+      try {
+        res.write(`event: ${event}\n`);
+        res.write(`data: ${JSON.stringify(payload)}\n\n`);
+      } catch { /* socket gone */ }
+    };
+
+    // Immediate primer + heartbeat, same as before — keeps intermediary
+    // proxies from treating the connection as idle.
+    try { res.write(': start\n\n'); } catch { /* noop */ }
+    const heartbeat = setInterval(() => {
+      if (closed) return;
+      try { res.write(': ping\n\n'); } catch { /* noop */ }
+    }, 5000);
+
+    // Replay the buffered transcript first so late subscribers see
+    // every line npm has already produced.
+    for (const entry of job.logs) {
+      write('log', { stream: entry.stream, chunk: entry.chunk });
+    }
+
+    const onLog = (entry: { stream: string; chunk: string }) => {
+      write('log', { stream: entry.stream, chunk: entry.chunk });
+    };
+    const onDone = (payload: Record<string, unknown>) => {
+      write('done', payload);
+      cleanup();
+      try { res.end(); } catch { /* noop */ }
+    };
+
+    const cleanup = () => {
+      if (closed) return;
+      closed = true;
+      clearInterval(heartbeat);
+      job.emitter.off('log', onLog);
+      job.emitter.off('done', onDone);
+    };
+
+    if (job.status !== 'running') {
+      // Job already finished — replay the terminal done frame and exit.
+      write('done', snapshotDonePayload(job));
+      cleanup();
+      try { res.end(); } catch { /* noop */ }
+      return;
+    }
+
+    job.emitter.on('log', onLog);
+    job.emitter.once('done', onDone);
+
+    req.on('close', () => {
+      // Client walked away. DO NOT cancel the install — detaching is fine.
+      cleanup();
+    });
+  }),
+);
+
 router.delete(
   '/:provider/install/:jobId',
   requireProviderAdmin,
   asyncHandler(async (req: Request, res: Response) => {
-    const parsed = parseProvider(req.params.provider);
-    const jobId = readPathParam(req.params.jobId, 'jobId');
-    const job = getInstallJob(jobId);
-    if (!job || job.provider !== parsed) {
-      throw new AppError('Install job not found.', {
-        code: 'INSTALL_JOB_NOT_FOUND',
-        statusCode: 404,
-      });
-    }
-    const cancelled = cancelInstallJob(jobId);
-    res.json(createApiSuccessResponse({ cancelled }));
-  }),
-);
-
+    const parsed = parseProvider(req.params.provider);
+    const jobId = readPathParam(req.params.jobId, 'jobId');
+    const job = getInstallJob(jobId);
+    if (!job || job.provider !== parsed) {
+      throw new AppError('Install job not found.', {
+        code: 'INSTALL_JOB_NOT_FOUND',
+        statusCode: 404,
+      });
+    }
+    const cancelled = cancelInstallJob(jobId);
+    res.json(createApiSuccessResponse({ cancelled }));
+  }),
+);
+
 router.post(
   '/mcp/servers/global',
   requireProviderAdmin,
   asyncHandler(async (req: Request, res: Response) => {
-    const payload = parseMcpUpsertPayload(req.body);
-    if (payload.scope === 'local') {
-      throw new AppError('Global MCP add supports only "user" or "project" scopes.', {
-        code: 'INVALID_GLOBAL_MCP_SCOPE',
-        statusCode: 400,
-      });
-    }
-
-    const results = await providerMcpService.addMcpServerToAllProviders({
-      ...payload,
-      scope: payload.scope === 'user' ? 'user' : 'project',
-    });
-    res.status(201).json(createApiSuccessResponse({ results }));
-  }),
-);
-
-// ============================================================================
-// Provider config files — read / edit the per-CLI settings/env files from
-// inside Pixcode rather than making the user open a text editor themselves.
-// The registry at server/modules/providers/shared/provider-configs.ts is the
-// single source of truth for which files exist; the client pulls this list
-// via GET /config-files and then reads/writes individual files by id.
-// ============================================================================
-
-// Resolve a config descriptor from (provider, fileId). Throws a 404
-// AppError if either isn't registered so the client sees a clear failure
-// instead of a generic 500.
-const resolveConfigFile = (provider: string, fileId: string): { descriptor: ProviderConfigFile; absolutePath: string } => {
-  const list = PROVIDER_CONFIG_FILES[provider];
-  if (!list) {
-    throw new AppError(`No config files registered for provider "${provider}"`, {
-      code: 'PROVIDER_CONFIG_UNKNOWN_PROVIDER',
-      statusCode: 404,
-    });
-  }
-  const descriptor = list.find((entry) => entry.id === fileId);
-  if (!descriptor) {
-    throw new AppError(`Unknown config file "${fileId}" for provider "${provider}"`, {
-      code: 'PROVIDER_CONFIG_UNKNOWN_FILE',
-      statusCode: 404,
-    });
-  }
-  // Always resolve relative to the server's os.homedir() — we never trust
-  // the client for any part of the path. `path.resolve` then normalises
-  // out any `..` segments the registry might accidentally contain.
-  const absolutePath = path.resolve(os.homedir(), descriptor.relativePath);
-  return { descriptor, absolutePath };
-};
-
-const SENSITIVE_CONFIG_PATTERN = /(api[_-]?key|token|secret|password|authorization|bearer)\s*[:=]\s*["']?([^"'\n\r]+)/ig;
-
-function redactProviderConfigPreview(contents: string): string {
-  return contents.replace(SENSITIVE_CONFIG_PATTERN, (_match, key) => `${key}: [redacted]`);
-}
-
-async function validateProviderConfigContents(descriptor: ProviderConfigFile, contents: string) {
-  if (Buffer.byteLength(contents, 'utf8') > MAX_CONFIG_FILE_SIZE_BYTES) {
-    throw new AppError(
-      `Config contents exceed ${MAX_CONFIG_FILE_SIZE_BYTES} bytes`,
-      { code: 'PROVIDER_CONFIG_TOO_LARGE', statusCode: 413 },
-    );
-  }
-
-  if (descriptor.format === 'json') {
-    try {
-      JSON.parse(contents || '{}');
-    } catch (err) {
-      throw new AppError(`Invalid JSON: ${(err as Error).message}`, {
-        code: 'PROVIDER_CONFIG_INVALID_JSON',
-        statusCode: 400,
-      });
-    }
-  }
-
-  return {
-    valid: true,
-    format: descriptor.format,
-    readonly: Boolean(descriptor.readonly),
-    preview: redactProviderConfigPreview(contents).slice(0, 4000),
-  };
-}
-
-async function buildProviderPluginState(provider: string) {
-  const files = PROVIDER_CONFIG_FILES[provider] || [];
-  const configs = await Promise.all(files.map(async (entry) => {
-    const absolutePath = path.resolve(os.homedir(), entry.relativePath);
-    let exists = false;
-    let size: number | null = null;
-    let updatedAt: string | null = null;
-    let preview = '';
-    try {
-      const stat = await fs.stat(absolutePath);
-      exists = stat.isFile();
-      size = stat.size;
-      updatedAt = stat.mtime.toISOString();
-      if (exists && stat.size <= MAX_CONFIG_FILE_SIZE_BYTES) {
-        preview = redactProviderConfigPreview(await fs.readFile(absolutePath, 'utf8')).slice(0, 1200);
-      }
-    } catch {
-      // Missing config files are normal for CLIs that have not been used yet.
-    }
-
-    return {
-      id: entry.id,
-      label: entry.label,
-      format: entry.format,
-      readonly: Boolean(entry.readonly),
-      relativePath: entry.relativePath,
-      absolutePath,
-      exists,
-      size,
-      updatedAt,
-      preview,
-      canBackup: exists,
-      canValidate: entry.format === 'json' || entry.format === 'env' || entry.format === 'toml' || entry.format === 'text',
-    };
-  }));
-
-  return {
-    provider,
-    supported: files.length > 0,
-    configCount: files.length,
-    installedCount: configs.filter((config) => config.exists).length,
-    configs,
-  };
-}
-
+    const payload = parseMcpUpsertPayload(req.body);
+    if (payload.scope === 'local') {
+      throw new AppError('Global MCP add supports only "user" or "project" scopes.', {
+        code: 'INVALID_GLOBAL_MCP_SCOPE',
+        statusCode: 400,
+      });
+    }
+
+    const results = await providerMcpService.addMcpServerToAllProviders({
+      ...payload,
+      scope: payload.scope === 'user' ? 'user' : 'project',
+    });
+    res.status(201).json(createApiSuccessResponse({ results }));
+  }),
+);
+
+// ============================================================================
+// Provider config files — read / edit the per-CLI settings/env files from
+// inside Pixcode rather than making the user open a text editor themselves.
+// The registry at server/modules/providers/shared/provider-configs.ts is the
+// single source of truth for which files exist; the client pulls this list
+// via GET /config-files and then reads/writes individual files by id.
+// ============================================================================
+
+// Resolve a config descriptor from (provider, fileId). Throws a 404
+// AppError if either isn't registered so the client sees a clear failure
+// instead of a generic 500.
+const resolveConfigFile = (provider: string, fileId: string): { descriptor: ProviderConfigFile; absolutePath: string } => {
+  const list = PROVIDER_CONFIG_FILES[provider];
+  if (!list) {
+    throw new AppError(`No config files registered for provider "${provider}"`, {
+      code: 'PROVIDER_CONFIG_UNKNOWN_PROVIDER',
+      statusCode: 404,
+    });
+  }
+  const descriptor = list.find((entry) => entry.id === fileId);
+  if (!descriptor) {
+    throw new AppError(`Unknown config file "${fileId}" for provider "${provider}"`, {
+      code: 'PROVIDER_CONFIG_UNKNOWN_FILE',
+      statusCode: 404,
+    });
+  }
+  // Always resolve relative to the server's os.homedir() — we never trust
+  // the client for any part of the path. `path.resolve` then normalises
+  // out any `..` segments the registry might accidentally contain.
+  const absolutePath = path.resolve(os.homedir(), descriptor.relativePath);
+  return { descriptor, absolutePath };
+};
+
+const SENSITIVE_CONFIG_PATTERN = /(api[_-]?key|token|secret|password|authorization|bearer)\s*[:=]\s*["']?([^"'\n\r]+)/ig;
+
+function redactProviderConfigPreview(contents: string): string {
+  return contents.replace(SENSITIVE_CONFIG_PATTERN, (_match, key) => `${key}: [redacted]`);
+}
+
+async function validateProviderConfigContents(descriptor: ProviderConfigFile, contents: string) {
+  if (Buffer.byteLength(contents, 'utf8') > MAX_CONFIG_FILE_SIZE_BYTES) {
+    throw new AppError(
+      `Config contents exceed ${MAX_CONFIG_FILE_SIZE_BYTES} bytes`,
+      { code: 'PROVIDER_CONFIG_TOO_LARGE', statusCode: 413 },
+    );
+  }
+
+  if (descriptor.format === 'json') {
+    try {
+      JSON.parse(contents || '{}');
+    } catch (err) {
+      throw new AppError(`Invalid JSON: ${(err as Error).message}`, {
+        code: 'PROVIDER_CONFIG_INVALID_JSON',
+        statusCode: 400,
+      });
+    }
+  }
+
+  return {
+    valid: true,
+    format: descriptor.format,
+    readonly: Boolean(descriptor.readonly),
+    preview: redactProviderConfigPreview(contents).slice(0, 4000),
+  };
+}
+
+async function buildProviderPluginState(provider: string) {
+  const files = PROVIDER_CONFIG_FILES[provider] || [];
+  const configs = await Promise.all(files.map(async (entry) => {
+    const absolutePath = path.resolve(os.homedir(), entry.relativePath);
+    let exists = false;
+    let size: number | null = null;
+    let updatedAt: string | null = null;
+    let preview = '';
+    try {
+      const stat = await fs.stat(absolutePath);
+      exists = stat.isFile();
+      size = stat.size;
+      updatedAt = stat.mtime.toISOString();
+      if (exists && stat.size <= MAX_CONFIG_FILE_SIZE_BYTES) {
+        preview = redactProviderConfigPreview(await fs.readFile(absolutePath, 'utf8')).slice(0, 1200);
+      }
+    } catch {
+      // Missing config files are normal for CLIs that have not been used yet.
+    }
+
+    return {
+      id: entry.id,
+      label: entry.label,
+      format: entry.format,
+      readonly: Boolean(entry.readonly),
+      relativePath: entry.relativePath,
+      absolutePath,
+      exists,
+      size,
+      updatedAt,
+      preview,
+      canBackup: exists,
+      canValidate: entry.format === 'json' || entry.format === 'env' || entry.format === 'toml' || entry.format === 'text',
+    };
+  }));
+
+  return {
+    provider,
+    supported: files.length > 0,
+    configCount: files.length,
+    installedCount: configs.filter((config) => config.exists).length,
+    configs,
+  };
+}
+
 router.get(
   '/plugin-state',
   requireProviderAdmin,
   asyncHandler(async (_req: Request, res: Response) => {
-    const providers = await Promise.all(
-      Object.keys(PROVIDER_CONFIG_FILES).map((provider) => buildProviderPluginState(provider)),
-    );
-    res.json(createApiSuccessResponse({ providers }));
-  }),
-);
-
+    const providers = await Promise.all(
+      Object.keys(PROVIDER_CONFIG_FILES).map((provider) => buildProviderPluginState(provider)),
+    );
+    res.json(createApiSuccessResponse({ providers }));
+  }),
+);
+
 router.get(
   '/plugin-state/:provider',
   requireProviderAdmin,
   asyncHandler(async (req: Request, res: Response) => {
-    const provider = parseProvider(req.params.provider);
-    res.json(createApiSuccessResponse(await buildProviderPluginState(provider)));
-  }),
-);
-
+    const provider = parseProvider(req.params.provider);
+    res.json(createApiSuccessResponse(await buildProviderPluginState(provider)));
+  }),
+);
+
 router.get(
   '/:provider/config-files',
   requireProviderAdmin,
   asyncHandler(async (req: Request, res: Response) => {
-    const provider = String(req.params.provider);
-    const list = PROVIDER_CONFIG_FILES[provider];
-    if (!list) {
-      throw new AppError(`No config files registered for provider "${provider}"`, {
-        code: 'PROVIDER_CONFIG_UNKNOWN_PROVIDER',
-        statusCode: 404,
-      });
-    }
-    const files = await Promise.all(
-      list.map(async (entry: ProviderConfigFile) => {
-        const absolutePath = path.resolve(os.homedir(), entry.relativePath);
-        let exists = false;
-        let size: number | null = null;
-        let updatedAt: string | null = null;
-        try {
-          const stat = await fs.stat(absolutePath);
-          exists = stat.isFile();
-          size = stat.size;
-          updatedAt = stat.mtime.toISOString();
-        } catch (err) {
-          // ENOENT is the expected path for "user hasn't created this yet".
-          // Anything else (EACCES, EISDIR, …) we surface as a hint rather
-          // than blow up the whole list response.
-          if ((err as NodeJS.ErrnoException).code !== 'ENOENT') {
-            console.warn(`[provider-configs] stat ${absolutePath}:`, (err as Error).message);
-          }
-        }
-        return {
-          id: entry.id,
-          label: entry.label,
-          format: entry.format,
-          readonly: Boolean(entry.readonly),
-          description: entry.description ?? null,
-          relativePath: entry.relativePath,
-          absolutePath,
-          exists,
-          size,
-          updatedAt,
-        };
-      }),
-    );
-    res.json(createApiSuccessResponse({ provider, files }));
-  }),
-);
-
+    const provider = String(req.params.provider);
+    const list = PROVIDER_CONFIG_FILES[provider];
+    if (!list) {
+      throw new AppError(`No config files registered for provider "${provider}"`, {
+        code: 'PROVIDER_CONFIG_UNKNOWN_PROVIDER',
+        statusCode: 404,
+      });
+    }
+    const files = await Promise.all(
+      list.map(async (entry: ProviderConfigFile) => {
+        const absolutePath = path.resolve(os.homedir(), entry.relativePath);
+        let exists = false;
+        let size: number | null = null;
+        let updatedAt: string | null = null;
+        try {
+          const stat = await fs.stat(absolutePath);
+          exists = stat.isFile();
+          size = stat.size;
+          updatedAt = stat.mtime.toISOString();
+        } catch (err) {
+          // ENOENT is the expected path for "user hasn't created this yet".
+          // Anything else (EACCES, EISDIR, …) we surface as a hint rather
+          // than blow up the whole list response.
+          if ((err as NodeJS.ErrnoException).code !== 'ENOENT') {
+            console.warn(`[provider-configs] stat ${absolutePath}:`, (err as Error).message);
+          }
+        }
+        return {
+          id: entry.id,
+          label: entry.label,
+          format: entry.format,
+          readonly: Boolean(entry.readonly),
+          description: entry.description ?? null,
+          relativePath: entry.relativePath,
+          absolutePath,
+          exists,
+          size,
+          updatedAt,
+        };
+      }),
+    );
+    res.json(createApiSuccessResponse({ provider, files }));
+  }),
+);
+
 router.get(
   '/:provider/config-files/:fileId',
   requireProviderAdmin,
   asyncHandler(async (req: Request, res: Response) => {
-    const provider = String(req.params.provider);
-    const fileId = String(req.params.fileId);
-    const { descriptor, absolutePath } = resolveConfigFile(provider, fileId);
-
-    try {
-      const stat = await fs.stat(absolutePath);
-      if (!stat.isFile()) {
-        throw new AppError(`${absolutePath} is not a regular file`, {
-          code: 'PROVIDER_CONFIG_NOT_FILE',
-          statusCode: 409,
-        });
-      }
-      if (stat.size > MAX_CONFIG_FILE_SIZE_BYTES) {
-        throw new AppError(
-          `Config file is larger than ${MAX_CONFIG_FILE_SIZE_BYTES} bytes — refusing to load`,
-          { code: 'PROVIDER_CONFIG_TOO_LARGE', statusCode: 413 },
-        );
-      }
-      const contents = await fs.readFile(absolutePath, 'utf8');
-      res.json(createApiSuccessResponse({
-        id: descriptor.id,
-        label: descriptor.label,
-        format: descriptor.format,
-        readonly: Boolean(descriptor.readonly),
-        relativePath: descriptor.relativePath,
-        absolutePath,
-        exists: true,
-        size: stat.size,
-        updatedAt: stat.mtime.toISOString(),
-        contents,
-      }));
-    } catch (err) {
-      if ((err as NodeJS.ErrnoException).code === 'ENOENT') {
-        // Report "file doesn't exist yet" with empty contents so the UI can
-        // still open an editor and let the user create it with a save.
-        res.json(createApiSuccessResponse({
-          id: descriptor.id,
-          label: descriptor.label,
-          format: descriptor.format,
-          readonly: Boolean(descriptor.readonly),
-          relativePath: descriptor.relativePath,
-          absolutePath,
-          exists: false,
-          size: 0,
-          updatedAt: null,
-          contents: '',
-        }));
-        return;
-      }
-      throw err;
-    }
-  }),
-);
-
+    const provider = String(req.params.provider);
+    const fileId = String(req.params.fileId);
+    const { descriptor, absolutePath } = resolveConfigFile(provider, fileId);
+
+    try {
+      const stat = await fs.stat(absolutePath);
+      if (!stat.isFile()) {
+        throw new AppError(`${absolutePath} is not a regular file`, {
+          code: 'PROVIDER_CONFIG_NOT_FILE',
+          statusCode: 409,
+        });
+      }
+      if (stat.size > MAX_CONFIG_FILE_SIZE_BYTES) {
+        throw new AppError(
+          `Config file is larger than ${MAX_CONFIG_FILE_SIZE_BYTES} bytes — refusing to load`,
+          { code: 'PROVIDER_CONFIG_TOO_LARGE', statusCode: 413 },
+        );
+      }
+      const contents = await fs.readFile(absolutePath, 'utf8');
+      res.json(createApiSuccessResponse({
+        id: descriptor.id,
+        label: descriptor.label,
+        format: descriptor.format,
+        readonly: Boolean(descriptor.readonly),
+        relativePath: descriptor.relativePath,
+        absolutePath,
+        exists: true,
+        size: stat.size,
+        updatedAt: stat.mtime.toISOString(),
+        contents,
+      }));
+    } catch (err) {
+      if ((err as NodeJS.ErrnoException).code === 'ENOENT') {
+        // Report "file doesn't exist yet" with empty contents so the UI can
+        // still open an editor and let the user create it with a save.
+        res.json(createApiSuccessResponse({
+          id: descriptor.id,
+          label: descriptor.label,
+          format: descriptor.format,
+          readonly: Boolean(descriptor.readonly),
+          relativePath: descriptor.relativePath,
+          absolutePath,
+          exists: false,
+          size: 0,
+          updatedAt: null,
+          contents: '',
+        }));
+        return;
+      }
+      throw err;
+    }
+  }),
+);
+
 router.put(
   '/:provider/config-files/:fileId',
   requireProviderAdmin,
   asyncHandler(async (req: Request, res: Response) => {
-    const provider = String(req.params.provider);
-    const fileId = String(req.params.fileId);
-    const { descriptor, absolutePath } = resolveConfigFile(provider, fileId);
-
-    if (descriptor.readonly) {
-      throw new AppError(`${descriptor.label} is read-only`, {
-        code: 'PROVIDER_CONFIG_READONLY',
-        statusCode: 403,
-      });
-    }
-
-    const contents = typeof req.body?.contents === 'string' ? req.body.contents : '';
-    if (Buffer.byteLength(contents, 'utf8') > MAX_CONFIG_FILE_SIZE_BYTES) {
-      throw new AppError(
-        `Refusing to write: contents exceed ${MAX_CONFIG_FILE_SIZE_BYTES} bytes`,
-        { code: 'PROVIDER_CONFIG_TOO_LARGE', statusCode: 413 },
-      );
-    }
-
-    // Light format validation — catches "pasted a stray character and now
-    // the CLI refuses to start" before we actually save the file. We don't
-    // try to be strict about TOML / env formats because a user who's
-    // editing these probably knows the grammar better than our regex.
-    if (descriptor.format === 'json') {
-      try {
-        JSON.parse(contents || '{}');
-      } catch (err) {
-        throw new AppError(`Invalid JSON: ${(err as Error).message}`, {
-          code: 'PROVIDER_CONFIG_INVALID_JSON',
-          statusCode: 400,
-        });
-      }
-    }
-
-    await fs.mkdir(path.dirname(absolutePath), { recursive: true });
-    await fs.writeFile(absolutePath, contents, 'utf8');
-
-    const stat = await fs.stat(absolutePath);
-    res.json(createApiSuccessResponse({
-      id: descriptor.id,
-      relativePath: descriptor.relativePath,
-      absolutePath,
-      size: stat.size,
-      updatedAt: stat.mtime.toISOString(),
-    }));
-  }),
-);
-
+    const provider = String(req.params.provider);
+    const fileId = String(req.params.fileId);
+    const { descriptor, absolutePath } = resolveConfigFile(provider, fileId);
+
+    if (descriptor.readonly) {
+      throw new AppError(`${descriptor.label} is read-only`, {
+        code: 'PROVIDER_CONFIG_READONLY',
+        statusCode: 403,
+      });
+    }
+
+    const contents = typeof req.body?.contents === 'string' ? req.body.contents : '';
+    if (Buffer.byteLength(contents, 'utf8') > MAX_CONFIG_FILE_SIZE_BYTES) {
+      throw new AppError(
+        `Refusing to write: contents exceed ${MAX_CONFIG_FILE_SIZE_BYTES} bytes`,
+        { code: 'PROVIDER_CONFIG_TOO_LARGE', statusCode: 413 },
+      );
+    }
+
+    // Light format validation — catches "pasted a stray character and now
+    // the CLI refuses to start" before we actually save the file. We don't
+    // try to be strict about TOML / env formats because a user who's
+    // editing these probably knows the grammar better than our regex.
+    if (descriptor.format === 'json') {
+      try {
+        JSON.parse(contents || '{}');
+      } catch (err) {
+        throw new AppError(`Invalid JSON: ${(err as Error).message}`, {
+          code: 'PROVIDER_CONFIG_INVALID_JSON',
+          statusCode: 400,
+        });
+      }
+    }
+
+    await fs.mkdir(path.dirname(absolutePath), { recursive: true });
+    await fs.writeFile(absolutePath, contents, 'utf8');
+
+    const stat = await fs.stat(absolutePath);
+    res.json(createApiSuccessResponse({
+      id: descriptor.id,
+      relativePath: descriptor.relativePath,
+      absolutePath,
+      size: stat.size,
+      updatedAt: stat.mtime.toISOString(),
+    }));
+  }),
+);
+
 router.post(
   '/:provider/config-files/:fileId/validate',
   requireProviderAdmin,
   asyncHandler(async (req: Request, res: Response) => {
-    const provider = String(req.params.provider);
-    const fileId = String(req.params.fileId);
-    const { descriptor, absolutePath } = resolveConfigFile(provider, fileId);
-    const contents = typeof req.body?.contents === 'string'
-      ? req.body.contents
-      : await fs.readFile(absolutePath, 'utf8').catch(() => '');
-    res.json(createApiSuccessResponse(await validateProviderConfigContents(descriptor, contents)));
-  }),
-);
-
-router.post(
-  '/:provider/config-files/:fileId/backup',
-  asyncHandler(async (req: Request, res: Response) => {
-    const provider = String(req.params.provider);
-    const fileId = String(req.params.fileId);
-    const { absolutePath } = resolveConfigFile(provider, fileId);
-    const stat = await fs.stat(absolutePath);
-    if (!stat.isFile()) {
-      throw new AppError(`${absolutePath} is not a regular file`, {
-        code: 'PROVIDER_CONFIG_NOT_FILE',
-        statusCode: 409,
-      });
-    }
-    const backupPath = `${absolutePath}.pixcode-backup-${Date.now()}`;
-    await fs.copyFile(absolutePath, backupPath);
-    res.json(createApiSuccessResponse({
-      provider,
-      fileId,
-      absolutePath,
-      backupPath,
-      size: stat.size,
-    }));
-  }),
-);
-
-export default router;
+    const provider = String(req.params.provider);
+    const fileId = String(req.params.fileId);
+    const { descriptor, absolutePath } = resolveConfigFile(provider, fileId);
+    const contents = typeof req.body?.contents === 'string'
+      ? req.body.contents
+      : await fs.readFile(absolutePath, 'utf8').catch(() => '');
+    res.json(createApiSuccessResponse(await validateProviderConfigContents(descriptor, contents)));
+  }),
+);
+
+router.post(
+  '/:provider/config-files/:fileId/backup',
+  asyncHandler(async (req: Request, res: Response) => {
+    const provider = String(req.params.provider);
+    const fileId = String(req.params.fileId);
+    const { absolutePath } = resolveConfigFile(provider, fileId);
+    const stat = await fs.stat(absolutePath);
+    if (!stat.isFile()) {
+      throw new AppError(`${absolutePath} is not a regular file`, {
+        code: 'PROVIDER_CONFIG_NOT_FILE',
+        statusCode: 409,
+      });
+    }
+    const backupPath = `${absolutePath}.pixcode-backup-${Date.now()}`;
+    await fs.copyFile(absolutePath, backupPath);
+    res.json(createApiSuccessResponse({
+      provider,
+      fileId,
+      absolutePath,
+      backupPath,
+      size: stat.size,
+    }));
+  }),
+);
+
+export default router;