feat(connections)!: unify custom connections with per-model protocols

apps/desktop/src/main/__tests__/connection-settings-locale-render.test.ts

@@ -236,7 +236,7 @@ for (const copy of localeCases) {
         fetchModels: async () => { calls.push('fetchModels'); throw new Error('unexpected discovery'); },
       } as unknown as ConnectionsBridge;
       await harness.render(copy.locale, createElement(components.AddProviderForm, {
-        bridge, providerType: 'openai-compatible', existingSlugs: ['taken'],
+        bridge, providerType: 'custom', existingSlugs: ['taken'],
         onCancel: unexpectedCall, onCreated: unexpectedCall,
       }));
       const input = harness.document.querySelector<HTMLInputElement>('input[placeholder="my-provider"]');
@@ -417,9 +417,10 @@ function relayConnection(): ProjectedLlmConnection {
   const modelId = 'gpt-5.6-sol-joybuilder';
   return {
     connectionId: 'relay-connection',
-    slug: 'openai-responses-compatible-2',
+    slug: 'custom-2',
     name: '自定义中转站（OpenAI Responses）',
-    providerType: 'openai-responses-compatible',
+    providerType: 'custom',
+    defaultApiProtocol: 'openai-responses',
     baseUrl: 'https://relay.example/v1',
     defaultModel: modelId,
     enabledModelIds: [modelId],

apps/desktop/src/main/__tests__/model-catalog-choices.test.ts

@@ -95,7 +95,8 @@ describe('model catalog picker helpers', () => {
       connection({
         slug: 'openrouter',
         name: 'Openrouter',
-        providerType: 'openai-compatible',
+        providerType: 'custom',
+        defaultApiProtocol: 'openai-chat',
         models: [{ id: 'anthropic/claude-sonnet-5' }],
         modelSource: 'fetched',
       }),

apps/desktop/src/main/__tests__/new-task-staged-content.test.ts

@@ -182,7 +182,7 @@ function modelChoice(model: string, supportsVision: boolean): ChatModelChoice {
   return {
     connectionId: 'connection-test',
     connectionSlug: 'test',
-    providerType: 'openai-compatible',
+    providerType: 'custom',
     providerLabel: 'Test',
     model,
     label: model,

apps/desktop/src/main/__tests__/onboarding-service-incremental.test.ts

@@ -27,7 +27,8 @@ import { createOnboardingService } from '../onboarding-service.js';
 
 const storedConnection = {
   connectionId: 'connection-1', slug: 'primary', name: 'Primary',
-  providerType: 'openai-compatible' as const, defaultModel: 'gpt-5',
+  providerType: 'custom' as const, defaultApiProtocol: 'openai-chat' as const,
+  baseUrl: 'https://relay.example/v1', defaultModel: 'gpt-5',
   enabled: true, createdAt: 1, updatedAt: 1,
 };
 const connection: ProjectedLlmConnection = {

apps/desktop/src/main/__tests__/provider-add-submission.test.ts

@@ -67,11 +67,9 @@ const onboardingSaveInput: Parameters<ApiKeyOnboardingBridge['save']>[0] = {
   enabledModelIds: ['gpt-5'],
 };
 
-const RELAY_TYPES: readonly ProviderType[] = ['openai-compatible', 'openai-responses-compatible'];
-
 function draft(over: Partial<AddProviderDraft> = {}): AddProviderDraft {
   return {
-    providerType: 'openai-compatible',
+    providerType: 'custom',
     slug: 'house-relay',
     existingSlugs: [],
     apiKey: 'sk-test',
@@ -86,14 +84,24 @@ function connection(slug: string): IdentifiedLlmConnection {
     connectionId: `connection-${slug}`,
     slug,
     name: slug,
-    providerType: 'openai-compatible',
+    providerType: 'custom',
+    defaultApiProtocol: 'openai-chat',
+    baseUrl: 'https://relay.example.com/v1',
     defaultModel: '',
     enabled: true,
     createdAt: 0,
     updatedAt: 0,
   } as IdentifiedLlmConnection;
 }
 
+const CUSTOM_INPUT: CreateConnectionInput = {
+  slug: 'house-relay',
+  name: 'House',
+  providerType: 'custom',
+  defaultApiProtocol: 'openai-chat',
+  baseUrl: 'https://relay.example.com/v1',
+};
+
 function bridge(over: {
   create?: (input: CreateConnectionInput) => Promise<IdentifiedLlmConnection>;
   fetchModels?: (connection: { readonly connectionId: string; readonly slug: string }) => Promise<unknown>;
@@ -105,12 +113,10 @@ function bridge(over: {
 }
 
 // The first of the two behaviours this module exists to protect. A custom
-// relay used to be the only provider class that refused to be created without
-// a hand-typed model id — before the app had asked the relay what it serves.
-test('a custom relay is created without a hand-typed model id', () => {
-  for (const providerType of RELAY_TYPES) {
-    assert.equal(validateAddProviderDraft(draft({ providerType })), null, providerType);
-  }
+// connection used to be the only provider class that refused to be created
+// without a hand-typed model id — before the app had asked it what it serves.
+test('a custom connection is created without a hand-typed model id', () => {
+  assert.equal(validateAddProviderDraft(draft()), null);
 });
 
 test('no provider type demands a model id at creation', () => {
@@ -134,21 +140,19 @@ test('no provider type demands a model id at creation', () => {
 });
 
 // The second. Discovery failures were reported for every provider except the
-// custom relays, which are the endpoints most likely to be misconfigured.
-test('a discovery failure reaches the caller for a custom relay', async () => {
-  for (const providerType of RELAY_TYPES) {
-    const failure = new Error('relay refused /v1/models');
-    const created = await createProviderWithDiscovery(
-      bridge({
-        fetchModels: async () => {
-          throw failure;
-        },
-      }),
-      { slug: 'house-relay', name: 'House', providerType } as CreateConnectionInput,
-    );
-    assert.equal(created.connection.slug, 'house-relay');
-    assert.equal(created.modelDiscoveryError, failure, providerType);
-  }
+// custom connections, which are the endpoints most likely to be misconfigured.
+test('a discovery failure reaches the caller for a custom connection', async () => {
+  const failure = new Error('relay refused /v1/models');
+  const created = await createProviderWithDiscovery(
+    bridge({
+      fetchModels: async () => {
+        throw failure;
+      },
+    }),
+    CUSTOM_INPUT,
+  );
+  assert.equal(created.connection.slug, 'house-relay');
+  assert.equal(created.modelDiscoveryError, failure);
 });
 
 test('a discovery failure reaches the caller for a built-in provider too', async () => {
@@ -174,15 +178,15 @@ test('a failed catalog fetch still yields the created connection', async () => {
         throw new Error('ECONNREFUSED');
       },
     }),
-    { slug: 'house-relay', name: 'House', providerType: 'openai-compatible' } as CreateConnectionInput,
+    CUSTOM_INPUT,
   );
   assert.equal(created.connection.slug, 'house-relay');
 });
 
 test('a successful catalog fetch reports no error', async () => {
   const created = await createProviderWithDiscovery(
     bridge({}),
-    { slug: 'house-relay', name: 'House', providerType: 'openai-compatible' } as CreateConnectionInput,
+    CUSTOM_INPUT,
   );
   assert.equal(created.modelDiscoveryError, undefined);
 });
@@ -215,7 +219,7 @@ test('a create failure propagates instead of being reported as a discovery probl
           throw failure;
         },
       }),
-      { slug: 'house-relay', name: 'House', providerType: 'openai-compatible' } as CreateConnectionInput,
+      CUSTOM_INPUT,
     ),
     failure,
   );
@@ -281,7 +285,7 @@ test('routes only fixed-endpoint API-key drafts without request customization to
     hasRequestBodyOverlay: true,
   }), { kind: 'legacy', reason: 'request_body' });
   assert.deepEqual(apiKeyOnboardingRoute({
-    providerType: 'openai-compatible',
+    providerType: 'custom',
     requestHeaderCount: 0,
     hasRequestBodyOverlay: false,
   }), { kind: 'legacy', reason: 'custom_endpoint' });

apps/desktop/src/main/__tests__/provider-endpoint-presentation.test.ts

@@ -65,7 +65,7 @@ test('a persisted override is the displayed effective endpoint', () => {
 test('displaying a custom endpoint masks userinfo and every query value without hiding its route', () => {
   assert.deepEqual(
     providerEndpointPresentation({
-      providerType: 'openai-compatible',
+      providerType: 'custom',
       baseUrl:
         `https://relay-user:relay-password@relay.example.com/v1?api-version=2026-08-01&api_key=${longOpaqueToken}`,
     }),
@@ -81,7 +81,7 @@ test('displaying a custom endpoint masks userinfo and every query value without
 test('query values are masked under arbitrary key names, not just known ones', () => {
   assert.deepEqual(
     providerEndpointPresentation({
-      providerType: 'openai-compatible',
+      providerType: 'custom',
       baseUrl: `https://relay.example.com/v1?key=${longOpaqueToken}`,
     }),
     {
@@ -92,7 +92,7 @@ test('query values are masked under arbitrary key names, not just known ones', (
   );
   assert.deepEqual(
     providerEndpointPresentation({
-      providerType: 'openai-compatible',
+      providerType: 'custom',
       baseUrl: `https://relay.example.com/v1?client_secret=${longOpaqueToken}`,
     }),
     {
@@ -106,7 +106,7 @@ test('query values are masked under arbitrary key names, not just known ones', (
 test('custom relays and local runtimes retain endpoint editing', () => {
   assert.deepEqual(
     providerEndpointPresentation({
-      providerType: 'openai-compatible',
+      providerType: 'custom',
       baseUrl: 'https://relay.example.com/v1',
     }),
     {
@@ -149,7 +149,7 @@ test('derived and OAuth endpoints remain visible but read-only', () => {
 
 test('an absent custom endpoint remains visible as a missing editable value', () => {
   assert.deepEqual(
-    providerEndpointPresentation({ providerType: 'openai-compatible' }),
+    providerEndpointPresentation({ providerType: 'custom' }),
     { value: null, editable: true, emptyState: 'missing' },
   );
 });

apps/desktop/src/main/__tests__/runtime-host-connections-ipc-main.test.ts

@@ -287,7 +287,8 @@ test('retries connection delete after a stale revision instead of failing perman
             revision,
             slug: 'openrouter',
             name: 'OpenRouter',
-            providerType: 'openai-compatible',
+            providerType: 'custom',
+            defaultApiProtocol: 'openai-chat',
             baseUrl: 'https://openrouter.ai/api/v1',
             enabled: true,
             catalogEntries: [],
@@ -538,7 +539,8 @@ test('projects the Host default target without inventing a second Connection aut
       connectionId: 'connection-1',
       slug: 'openrouter',
       name: 'OpenRouter',
-      providerType: 'openai-compatible',
+      providerType: 'custom',
+      defaultApiProtocol: 'openai-chat',
       baseUrl: 'https://openrouter.ai/api/v1',
       enabled: true,
       defaultModel: 'model-1',
@@ -609,7 +611,8 @@ function catalog(): ConnectionCatalogSnapshot {
         revision: 4,
         slug: 'openrouter',
         name: 'OpenRouter',
-        providerType: 'openai-compatible',
+        providerType: 'custom',
+        defaultApiProtocol: 'openai-chat',
         baseUrl: 'https://openrouter.ai/api/v1',
         enabled: true,
         enabledModelIds: ['model-1', 'model-2'],

apps/desktop/src/main/__tests__/task-submission-readiness-main.test.ts

@@ -108,7 +108,9 @@ function connection(): LlmConnection {
   return {
     slug: 'provider',
     name: 'Provider',
-    providerType: 'openai-compatible',
+    providerType: 'custom',
+    defaultApiProtocol: 'openai-chat',
+    baseUrl: 'https://relay.example/v1',
     enabled: true,
     defaultModel: 'model-a',
     enabledModelIds: ['model-a'],

apps/desktop/src/main/e2e-fixture/scenarios-settings.ts

@@ -92,8 +92,9 @@ export async function writeConnections(
   const noModels: ConnectionCatalogEntryDraft = {
     slug: 'no-models',
     name: 'No Models Fixture',
-    providerType: 'openai-compatible',
+    providerType: 'custom',
     baseUrl: 'https://empty.example.test/v1',
+    defaultApiProtocol: 'openai-chat',
     enabled: true,
     enabledModelIds: [],
   };

apps/desktop/src/main/runtime-host-config-ipc-main.ts

@@ -365,6 +365,9 @@ export async function saveConnection(
       name: connection.name,
       providerType: connection.providerType,
       ...(connection.baseUrl ? { baseUrl: connection.baseUrl } : {}),
+      ...(connection.defaultApiProtocol === undefined
+        ? {}
+        : { defaultApiProtocol: connection.defaultApiProtocol }),
       enabled: connection.enabled,
       enabledModelIds: [...(connection.enabledModelIds ?? [])],
       ...(importedProfiles === undefined ? {} : { modelOverrides: importedProfiles }),

apps/desktop/src/main/runtime-host-connections-ipc-main.ts

@@ -207,6 +207,9 @@ export function registerRuntimeHostConnectionsIpc(
       name: input.name,
       providerType: input.providerType,
       ...(input.baseUrl === undefined ? {} : { baseUrl: input.baseUrl }),
+      ...(input.defaultApiProtocol === undefined
+        ? {}
+        : { defaultApiProtocol: input.defaultApiProtocol }),
       enabled: true,
       enabledModelIds: connectionEnabledModelIds({
         defaultModel: input.defaultModel,
@@ -405,6 +408,9 @@ export function projectHostConnections(
       name: connection.name,
       providerType: connection.providerType,
       ...(connection.baseUrl === undefined ? {} : { baseUrl: connection.baseUrl }),
+      ...(connection.defaultApiProtocol === undefined
+        ? {}
+        : { defaultApiProtocol: connection.defaultApiProtocol }),
       enabled: connection.enabled,
       defaultModel,
       enabledModelIds: [...connection.enabledModelIds],

apps/desktop/src/renderer/features/connection-settings/provider-add-model-dialog.tsx

@@ -18,8 +18,8 @@
  */
 
 import { useId, useState, type ReactNode } from 'react';
-import { isRelayProviderType, type ProviderType } from '@maka/core/llm-connections';
-import { supportsRelayFastServiceTier, modelLimitsConflict, type ModelOverride } from '@maka/core/model-thinking';
+import type { ModelApiProtocol, ProviderType } from '@maka/core/llm-connections';
+import { supportsCustomFastServiceTier, modelLimitsConflict, type ModelOverride } from '@maka/core/model-thinking';
 import { CapabilityEditor } from './provider-capability-editor.js';
 import { Dialog, DialogHeader } from '@astryxdesign/core/Dialog';
 import { Layout, LayoutContent, LayoutFooter } from '@astryxdesign/core/Layout';
@@ -30,6 +30,7 @@ import { parseContextWindowInput } from './context-window-input.js';
 export function AddModelDialog(props: {
   isOpen: boolean;
   providerType: ProviderType;
+  defaultApiProtocol?: ModelApiProtocol;
   existingModelIds: readonly string[];
   /** Another write is in flight; the store would drop this one on the floor. */
   isSubmitDisabled?: boolean;
@@ -53,6 +54,14 @@ export function AddModelDialog(props: {
   const [isSaving, setSaving] = useState(false);
 
   const trimmedId = id.trim();
+  const showsFastMode = supportsCustomFastServiceTier(
+    {
+      providerType: props.providerType,
+      defaultApiProtocol: props.defaultApiProtocol,
+      modelOverrides: { [trimmedId]: profile },
+    },
+    trimmedId,
+  );
   const idError = !trimmedId
     ? copy.addModelIdRequired
     : props.existingModelIds.includes(trimmedId)
@@ -86,9 +95,7 @@ export function AddModelDialog(props: {
         await props.onSubmit(trimmedId, {
           ...parameters,
           ...(contextWindow === null ? {} : { contextWindow }),
-          ...(supportsRelayFastServiceTier(props.providerType, trimmedId) && serviceTier
-            ? { serviceTier }
-            : {}),
+          ...(showsFastMode && serviceTier ? { serviceTier } : {}),
         })
       )
         close();
@@ -110,7 +117,7 @@ export function AddModelDialog(props: {
       <CapabilityEditor
         copy={copy}
         modelId={trimmedId}
-        isRelay={isRelayProviderType(props.providerType)}
+        customDefaultApiProtocol={props.defaultApiProtocol}
         declared={profile}
         limitsConflict={limitsConflict}
         onChange={(patch) => setProfile((current) => ({ ...current, ...patch }))}
@@ -125,7 +132,7 @@ export function AddModelDialog(props: {
             setProfile((current) => ({ ...current, [field]: value ?? undefined }));
         }}
         disabled={isSaving}
-        showsFastMode={supportsRelayFastServiceTier(props.providerType, trimmedId)}
+        showsFastMode={showsFastMode}
         defaultVision={undefined}
         thinkingLevels={profile.thinkingLevels ?? []}
         onContextWindowInput={setContextWindowInput}