Reject unsupported SME chat providers in the flow

- Block non-Claude providers before conversation creation
- Disable unsupported options in the SME chat dialog
- Add coverage for the provider validation path

Author: BunsDev

DESIGN.md

@@ -52,34 +52,34 @@ conversation with the agent. Every pixel must earn its place.
 
 ### Typography
 
-| Role | Size | Weight | Tracking | Font Stack |
-|------|------|--------|----------|------------|
-| Thread title (sidebar) | `text-xs` (0.75rem) | `font-normal` | default | Inter, system-ui, sans-serif |
-| Thread subtitle / metadata | `text-[10px]` | `font-normal` | default | Inter, system-ui, sans-serif |
-| Badge text | `text-[10px]` | `font-medium` | default | Inter, system-ui, sans-serif |
-| Button text | `text-sm` (0.875rem) | `font-medium` | default | Inter, system-ui, sans-serif |
-| Heading / dialog title | `text-lg` (1.125rem) | `font-semibold` | `-0.01em` | Inter, system-ui, sans-serif |
-| Code / terminal | `text-sm` | `font-normal` | default | SF Mono, Consolas, monospace |
-| Project name | `text-xs` | `font-semibold` | default | Inter, system-ui, sans-serif |
+| Role                       | Size                 | Weight          | Tracking  | Font Stack                   |
+| -------------------------- | -------------------- | --------------- | --------- | ---------------------------- |
+| Thread title (sidebar)     | `text-xs` (0.75rem)  | `font-normal`   | default   | Inter, system-ui, sans-serif |
+| Thread subtitle / metadata | `text-[10px]`        | `font-normal`   | default   | Inter, system-ui, sans-serif |
+| Badge text                 | `text-[10px]`        | `font-medium`   | default   | Inter, system-ui, sans-serif |
+| Button text                | `text-sm` (0.875rem) | `font-medium`   | default   | Inter, system-ui, sans-serif |
+| Heading / dialog title     | `text-lg` (1.125rem) | `font-semibold` | `-0.01em` | Inter, system-ui, sans-serif |
+| Code / terminal            | `text-sm`            | `font-normal`   | default   | SF Mono, Consolas, monospace |
+| Project name               | `text-xs`            | `font-semibold` | default   | Inter, system-ui, sans-serif |
 
 ### Color Semantics
 
 Colors are referenced through CSS custom properties, never hardcoded hex values.
 
-| Token | Usage |
-|-------|-------|
-| `text-foreground` | Primary text |
-| `text-muted-foreground` | Secondary/deemphasized text |
+| Token                      | Usage                                             |
+| -------------------------- | ------------------------------------------------- |
+| `text-foreground`          | Primary text                                      |
+| `text-muted-foreground`    | Secondary/deemphasized text                       |
 | `text-muted-foreground/50` | Tertiary/metadata text (branch names, timestamps) |
-| `bg-background` | Page background |
-| `bg-accent` | Hover state, active row highlight |
-| `bg-accent/60` | Active sidebar item |
-| `bg-accent/40` | Selected sidebar item |
-| `text-emerald-600` | Additions / success (green) |
-| `text-rose-500` | Deletions / error (red) |
-| `text-warning` | Warning states, behind-upstream |
-| `text-destructive` | Destructive actions (delete) |
-| `border-border/60` | Subtle badge borders |
+| `bg-background`            | Page background                                   |
+| `bg-accent`                | Hover state, active row highlight                 |
+| `bg-accent/60`             | Active sidebar item                               |
+| `bg-accent/40`             | Selected sidebar item                             |
+| `text-emerald-600`         | Additions / success (green)                       |
+| `text-rose-500`            | Deletions / error (red)                           |
+| `text-warning`             | Warning states, behind-upstream                   |
+| `text-destructive`         | Destructive actions (delete)                      |
+| `border-border/60`         | Subtle badge borders                              |
 
 ### Spacing Rules
 
@@ -95,13 +95,13 @@ Colors are referenced through CSS custom properties, never hardcoded hex values.
 
 Five premium themes, each with light and dark variants:
 
-| Theme | Vibe |
-|-------|------|
-| **Iridescent Void** | Futuristic, expensive, slightly alien |
-| **Carbon** | Stark, modern, performance-focused |
-| **Vapor** | Refined, fluid, purposeful |
-| **Cotton Candy** | Sweet, dreamy, pink and blue |
-| **Cathedral Circuit** | Sacred machine, techno-gothic |
+| Theme                 | Vibe                                  |
+| --------------------- | ------------------------------------- |
+| **Iridescent Void**   | Futuristic, expensive, slightly alien |
+| **Carbon**            | Stark, modern, performance-focused    |
+| **Vapor**             | Refined, fluid, purposeful            |
+| **Cotton Candy**      | Sweet, dreamy, pink and blue          |
+| **Cathedral Circuit** | Sacred machine, techno-gothic         |
 
 All themes define the same set of CSS custom properties. Components must use semantic
 tokens (`bg-accent`, `text-muted-foreground`) — never theme-specific values.
@@ -249,6 +249,7 @@ a single flow:
 ```
 
 Quick action resolves automatically based on git state:
+
 - Has changes + no PR → "Commit, push & PR"
 - Has changes + existing PR → "Commit & push"
 - No changes + ahead → "Push & create PR"

apps/server/src/sme/Layers/SmeChatServiceLive.test.ts

@@ -123,7 +123,9 @@ describe("SmeChatServiceLive", () => {
 
     const layer = makeSmeChatServiceLive({ sendSmeViaAnthropic: sendClaudeMessage }).pipe(
       Layer.provideMerge(Layer.succeed(SmeKnowledgeDocumentRepository, makeDocumentRepository())),
-      Layer.provideMerge(Layer.succeed(SmeConversationRepository, makeConversationRepository([conversationRow]))),
+      Layer.provideMerge(
+        Layer.succeed(SmeConversationRepository, makeConversationRepository([conversationRow])),
+      ),
       Layer.provideMerge(Layer.succeed(SmeMessageRepository, messageRepo)),
     );
 
@@ -249,9 +251,13 @@ describe("SmeChatServiceLive", () => {
       deletedAt: null,
     };
     const { repository: messageRepo, rowsByConversation } = makeMessageRepository();
-    const layer = makeSmeChatServiceLive({ sendSmeViaAnthropic: () => Effect.die("unexpected send") }).pipe(
+    const layer = makeSmeChatServiceLive({
+      sendSmeViaAnthropic: () => Effect.succeed("unexpected send"),
+    }).pipe(
       Layer.provideMerge(Layer.succeed(SmeKnowledgeDocumentRepository, makeDocumentRepository())),
-      Layer.provideMerge(Layer.succeed(SmeConversationRepository, makeConversationRepository([conversationRow]))),
+      Layer.provideMerge(
+        Layer.succeed(SmeConversationRepository, makeConversationRepository([conversationRow])),
+      ),
       Layer.provideMerge(Layer.succeed(SmeMessageRepository, messageRepo)),
     );
 
@@ -311,4 +317,28 @@ describe("SmeChatServiceLive", () => {
       }),
     ]);
   });
+
+  it("rejects unsupported SME providers before a conversation can be created", async () => {
+    const projectId = ProjectId.makeUnsafe("project-3");
+    const layer = makeSmeChatServiceLive().pipe(
+      Layer.provideMerge(Layer.succeed(SmeKnowledgeDocumentRepository, makeDocumentRepository())),
+      Layer.provideMerge(Layer.succeed(SmeConversationRepository, makeConversationRepository([]))),
+      Layer.provideMerge(Layer.succeed(SmeMessageRepository, makeMessageRepository().repository)),
+    );
+
+    await expect(
+      Effect.runPromise(
+        Effect.gen(function* () {
+          const service = yield* SmeChatService;
+          yield* service.createConversation({
+            projectId,
+            title: "Unsupported provider",
+            provider: "codex",
+            authMethod: "chatgpt",
+            model: "codex-mini",
+          });
+        }).pipe(Effect.provide(layer)),
+      ),
+    ).rejects.toThrow("SME Chat only supports Claude Code conversations right now.");
+  });
 });

apps/server/src/sme/Layers/SmeChatServiceLive.ts

@@ -7,7 +7,6 @@
  * @module SmeChatServiceLive
  */
 import type {
-  ProviderStartOptions,
   SmeAuthMethod,
   SmeConversation,
   SmeKnowledgeDocument,
@@ -25,10 +24,7 @@ import { SmeConversationRepository } from "../../persistence/Services/SmeConvers
 import { SmeKnowledgeDocumentRepository } from "../../persistence/Services/SmeKnowledgeDocuments.ts";
 import { SmeMessageRepository } from "../../persistence/Services/SmeMessages.ts";
 import { isValidSmeAuthMethod } from "../authValidation.ts";
-import {
-  resolveAnthropicClientOptions,
-  sendSmeViaAnthropic,
-} from "../backends/anthropic.ts";
+import { resolveAnthropicClientOptions, sendSmeViaAnthropic } from "../backends/anthropic.ts";
 import { buildSmeSystemPrompt } from "../promptBuilder.ts";
 import {
   SmeChatError,
@@ -192,7 +188,8 @@ const makeSmeChatService = (options?: SmeChatServiceLiveOptions) =>
               ? "Claude SME Chat can use the configured Anthropic API key."
               : "Claude SME Chat can use the configured Anthropic auth token.",
           resolvedAuthMethod: conversation.authMethod,
-          resolvedAccountType: clientOptions.apiKey !== null ? "apiKey" : "unknown",
+          resolvedAccountType:
+            clientOptions.apiKey !== null ? ("apiKey" as const) : ("unknown" as const),
         };
       });
 
@@ -462,8 +459,7 @@ const makeSmeChatService = (options?: SmeChatServiceLiveOptions) =>
             resolveAnthropicClientOptions({
               providerOptions: input.providerOptions?.claudeAgent,
             }),
-          catch: (cause) =>
-            new SmeChatError("sendMessage:providerRuntime", String(cause), cause),
+          catch: (cause) => new SmeChatError("sendMessage:providerRuntime", String(cause), cause),
         });
 
         if (!anthropicClientOptions.apiKey && !anthropicClientOptions.authToken) {
@@ -476,13 +472,13 @@ const makeSmeChatService = (options?: SmeChatServiceLiveOptions) =>
         }
 
         const systemPrompt = buildSmeSystemPrompt(docs);
-        const messages: ReadonlyArray<MessageParam> = [
-          ...promptHistory
+        const messages: Array<MessageParam> = [
+          ...(promptHistory
             .filter((message) => message.role === "user" || message.role === "assistant")
             .map((message) => ({
               role: message.role,
               content: message.text,
-            })) as Array<MessageParam>,
+            })) as Array<MessageParam>),
           {
             role: "user",
             content: input.text,
@@ -502,13 +498,19 @@ const makeSmeChatService = (options?: SmeChatServiceLiveOptions) =>
           abortSignal: abortController.signal,
         });
 
-        yield* input.setInterruptEffect(
+        yield* setInterrupt(
+          input.conversationId,
           Effect.sync(() => {
             abortController.abort();
           }),
         );
 
         const responseText = yield* sendEffect.pipe(
+          Effect.ensuring(
+            Effect.gen(function* () {
+              yield* clearInterrupt(input.conversationId);
+            }),
+          ),
           Effect.mapError((cause) =>
             cause instanceof SmeChatError
               ? cause

apps/server/src/sme/backends/anthropic.ts

@@ -42,7 +42,7 @@ export function resolveAnthropicClientOptions(
   const baseURL = nonEmptyTrimmed(env.ANTHROPIC_BASE_URL ?? env.ANTHROPIC_API_BASE_URL);
 
   return {
-    apiKey: authToken ? null : explicitApiKey ?? null,
+    apiKey: authToken ? null : (explicitApiKey ?? null),
     authToken: authToken ?? null,
     ...(baseURL ? { baseURL } : {}),
   };
@@ -58,7 +58,7 @@ function createAnthropicClient(options: ResolvedAnthropicClientOptions): Anthrop
 
 export interface SendSmeViaAnthropicInput {
   readonly client?: AnthropicMessagesClient;
-  readonly messages: ReadonlyArray<MessageParam>;
+  readonly messages: Array<MessageParam>;
   readonly conversationId: string;
   readonly assistantMessageId: string;
   readonly model: string;
@@ -73,7 +73,8 @@ export function sendSmeViaAnthropic(input: SendSmeViaAnthropicInput) {
     try: async () => {
       let result = "";
       const client =
-        input.client ?? createAnthropicClient(input.clientOptions ?? resolveAnthropicClientOptions());
+        input.client ??
+        createAnthropicClient(input.clientOptions ?? resolveAnthropicClientOptions());
       const stream = client.messages.stream(
         {
           model: input.model,

apps/web/src/components/sme/SmeConversationDialog.tsx

@@ -35,6 +35,8 @@ import {
   SME_PROVIDER_LABELS,
 } from "./smeConversationConfig";
 
+const SME_CHAT_SUPPORTED_PROVIDERS = new Set<ProviderKind>(["claudeAgent"]);
+
 interface SmeConversationDialogProps {
   open: boolean;
   onOpenChange: (open: boolean) => void;
@@ -216,11 +218,19 @@ export function SmeConversationDialog({
               className="h-10 rounded-xl border border-border bg-background px-3 text-sm"
             >
               {(["claudeAgent", "codex", "openclaw"] as const).map((value) => (
-                <option key={value} value={value}>
+                <option
+                  key={value}
+                  value={value}
+                  disabled={!SME_CHAT_SUPPORTED_PROVIDERS.has(value)}
+                >
                   {SME_PROVIDER_LABELS[value]}
                 </option>
               ))}
             </select>
+            <p className="text-xs text-muted-foreground">
+              Claude Code is the only SME Chat provider that currently supports direct replies
+              without tool workflows.
+            </p>
           </label>
 
           <label className="grid gap-1.5">