Merge pull request #72 from odefun/feat/channel-models-provider-1770589

fix: update channel model selectors by provider

Author: LIU9293

packages/ims/slack/commands.ts

@@ -132,21 +132,31 @@ function buildSettingsModal(params: {
     text: { type: "plain_text" as const, text: AGENT_PROVIDER_LABELS[provider] },
     value: provider,
   }));
-  const modelOptions = opencodeModels.length > 0
-    ? opencodeModels.map((model) => ({
-        text: { type: "plain_text" as const, text: model },
-        value: model,
-      }))
-    : [{ text: { type: "plain_text" as const, text: "No models configured" }, value: "__none__" }];
-  const codexModelOptions = [
-    { text: { type: "plain_text" as const, text: "Use default (gpt-5.3-codex)" }, value: "__default__" },
-    ...codexModels.map((model) => ({
-      text: { type: "plain_text" as const, text: model },
-      value: model,
-    })),
-  ];
-
-  const availableModels = selectedProvider === "codex" ? codexModelOptions.map((entry) => entry.value) : opencodeModels;
+  const providerModels = selectedProvider === "opencode"
+    ? opencodeModels
+    : selectedProvider === "codex"
+      ? codexModels
+      : null;
+  const modelOptions = providerModels && selectedProvider === "opencode"
+    ? (opencodeModels.length > 0
+      ? opencodeModels.map((model) => ({
+          text: { type: "plain_text" as const, text: model },
+          value: model,
+        }))
+      : [{ text: { type: "plain_text" as const, text: "No models configured" }, value: "__none__" }])
+    : providerModels && selectedProvider === "codex"
+      ? [
+          { text: { type: "plain_text" as const, text: "Use default (gpt-5.3-codex)" }, value: "__default__" },
+          ...codexModels.map((model) => ({
+            text: { type: "plain_text" as const, text: model },
+            value: model,
+          })),
+        ]
+      : [];
+
+  const availableModels = selectedProvider === "codex"
+    ? modelOptions.map((entry) => entry.value)
+    : providerModels ?? [];
   const matchedSelectedModel = findMatchingModel(availableModels, selectedModel);
   const initialModel = matchedSelectedModel
     ? matchedSelectedModel
@@ -168,6 +178,7 @@ function buildSettingsModal(params: {
     {
       type: "input" as const,
       block_id: PROVIDER_BLOCK,
+      dispatch_action: true,
       label: { type: "plain_text" as const, text: "Provider" },
       element: {
         type: "static_select" as const,
@@ -178,9 +189,8 @@ function buildSettingsModal(params: {
     },
   ];
 
-  if (selectedProvider === "opencode" || selectedProvider === "codex") {
-    const options = selectedProvider === "opencode" ? modelOptions : codexModelOptions;
-    const initialOption = options.find((option) => option.value === initialModel);
+  if (providerModels) {
+    const initialOption = modelOptions.find((option) => option.value === initialModel);
     blocks.push(
       {
         type: "input" as const,
@@ -190,7 +200,7 @@ function buildSettingsModal(params: {
         element: {
           type: "static_select" as const,
           action_id: MODEL_ACTION,
-          options,
+          options: modelOptions,
           initial_option: initialOption,
         },
       },

packages/web-ui/src/routes/local-setting/slack-bot/[workspaceName]/+page.svelte

@@ -65,7 +65,20 @@
   }
 
   function shouldShowChannelModel(channel: { agentProvider?: string }): boolean {
-    return getChannelProvider(channel) === "opencode";
+    const provider = getChannelProvider(channel);
+    return getProviderModels(provider) !== null;
+  }
+
+  function getProviderModels(provider: AgentProvider): string[] | null {
+    const config = $localSettingStore.config.agents as Record<string, { models?: string[] }>;
+    const entry = config[provider];
+    return Array.isArray(entry?.models) ? entry.models : null;
+  }
+
+  function getChannelModelSelectValue(channel: { agentProvider?: string; model: string }): string {
+    const provider = getChannelProvider(channel);
+    if (provider === "codex" && !channel.model) return "__default__";
+    return channel.model;
   }
 
   function onWorkspaceFieldInput(
@@ -90,16 +103,20 @@
   function onChannelProviderChange(workspaceId: string, channelId: string, event: Event): void {
     const selected = (event.currentTarget as HTMLSelectElement).value;
     const provider = parseAgentProvider(selected);
+    const providerModels = getProviderModels(provider) ?? [];
     localSettingStore.updateWorkspace(workspaceId, (workspace) => ({
       ...workspace,
       channelDetails: workspace.channelDetails.map((channel) => {
         if (channel.id !== channelId) return channel;
+        const nextModel = provider === "codex"
+          ? (providerModels.includes(channel.model) ? channel.model : "")
+          : (providerModels.length > 0
+            ? (providerModels.includes(channel.model) ? channel.model : (providerModels[0] ?? ""))
+            : "");
         return {
           ...channel,
           agentProvider: provider,
-          model: provider === "opencode"
-            ? channel.model || $localSettingStore.config.agents.opencode.models[0] || ""
-            : "",
+          model: nextModel,
         };
       }),
     }));
@@ -115,7 +132,8 @@
   }
 
   function onChannelModelSelect(workspaceId: string, channelId: string, event: Event): void {
-    onChannelModelChange(workspaceId, channelId, (event.currentTarget as HTMLSelectElement).value);
+    const value = (event.currentTarget as HTMLSelectElement).value;
+    onChannelModelChange(workspaceId, channelId, value === "__default__" ? "" : value);
   }
 
   function onChannelWorkingDirectoryChange(workspaceId: string, channelId: string, workingDirectory: string): void {
@@ -279,13 +297,20 @@
               <label for={`channel-model-${channel.id}`}>Model</label>
                 <select
                   id={`channel-model-${channel.id}`}
-                  value={channel.model}
+                  value={getChannelModelSelectValue(channel)}
                   on:change={(event) => onChannelModelSelect(selectedWorkspace.id, channel.id, event)}
                 >
-                {#if !$localSettingStore.config.agents.opencode.models.includes(channel.model) && channel.model}
+                {#if getChannelProvider(channel) === "codex"}
+                  <option value="__default__">Use default (gpt-5.3-codex)</option>
+                {/if}
+                {#if (getProviderModels(getChannelProvider(channel)) ?? []).length === 0
+                  && getChannelProvider(channel) !== "codex"}
+                  <option value="" disabled>No models configured</option>
+                {/if}
+                {#if !(getProviderModels(getChannelProvider(channel)) ?? []).includes(channel.model) && channel.model}
                   <option value={channel.model}>{channel.model}</option>
                 {/if}
-                {#each $localSettingStore.config.agents.opencode.models as model}
+                {#each getProviderModels(getChannelProvider(channel)) ?? [] as model}
                   <option value={model}>{model}</option>
                 {/each}
               </select>