fix(dashboard-agent): treat empty model provider as unset, fix Sonnet Bedrock id

DASHBOARD_AGENT_MODEL_PROVIDER="" rejected boot instead of falling back
to the default. BEDROCK_MODEL_IDS mapped Sonnet 4.6 to a nonexistent
-v1 suffix; Anthropic's official table has no -v1 for Sonnet. Pin exact
ids in the test rather than a shape regex.

Author: kathiekiwi

apps/webapp/app/env.server.ts

@@ -211,7 +211,10 @@ const EnvironmentSchema = z
     ANTHROPIC_API_KEY: z.string().optional(),
     // Selects the dashboard agent's LLM provider (default anthropic). The internal
     // seam reads process.env directly; this entry validates the value webapp-side.
-    DASHBOARD_AGENT_MODEL_PROVIDER: z.enum(["anthropic", "bedrock"]).default("anthropic"),
+    DASHBOARD_AGENT_MODEL_PROVIDER: z.preprocess(
+      (v) => (typeof v === "string" && v.trim() === "" ? undefined : v),
+      z.enum(["anthropic", "bedrock"]).default("anthropic")
+    ),
     // AWS credentials for the dashboard agent's Bedrock provider (only used when
     // DASHBOARD_AGENT_MODEL_PROVIDER=bedrock; default path stays Anthropic). The
     // provider resolves credentials itself, so only the region is read here.

internal-packages/dashboard-agent/src/model-provider.test.ts

@@ -49,7 +49,7 @@ describe("resolveDashboardAgentModel", () => {
   it("maps the same canonical string to a Bedrock inference profile", () => {
     useBedrock();
     expect(resolveDashboardAgentModel("anthropic:claude-sonnet-4-6").modelId).toBe(
-      "us.anthropic.claude-sonnet-4-6-v1"
+      "us.anthropic.claude-sonnet-4-6"
     );
     expect(resolveDashboardAgentModel("anthropic:claude-haiku-4-5").modelId).toBe(
       "us.anthropic.claude-haiku-4-5-20251001-v1:0"
@@ -63,16 +63,13 @@ describe("resolveDashboardAgentModel", () => {
     );
   });
 
-  // Structural, not an echo of the table: an AWS us cross-region Anthropic profile
-  // is either dated with a `:N` suffix, or an undated `-vN` (the 4-6 generation). A
-  // dated id must never drop its `:N`, and every id must end in a version.
-  it("every Bedrock-mapped id has a well-formed AWS inference-profile shape", () => {
-    const dated = /^us\.anthropic\.claude-[a-z]+(?:-\d+)+-\d{8}-v\d+:\d+$/;
-    const undated = /^us\.anthropic\.claude-[a-z]+(?:-\d+)+-v\d+$/;
-    for (const id of Object.values(BEDROCK_MODEL_IDS)) {
-      expect(dated.test(id) || undated.test(id), id).toBe(true);
-      if (/-\d{8}-/.test(id)) expect(id, id).toMatch(/:\d+$/);
-    }
+  // Pinned to Anthropic's official Bedrock model table, not a shape regex — there is
+  // no shared suffix convention across models, so a well-formed id can still be wrong.
+  it("maps every model to its exact documented Bedrock id", () => {
+    expect(BEDROCK_MODEL_IDS).toEqual({
+      "claude-sonnet-4-6": "us.anthropic.claude-sonnet-4-6",
+      "claude-haiku-4-5": "us.anthropic.claude-haiku-4-5-20251001-v1:0",
+    });
   });
 });
 

internal-packages/dashboard-agent/src/model-provider.ts

@@ -54,11 +54,11 @@ export const registry = createProviderRegistry({ anthropic, bedrock });
 
 /**
  * Canonical model id -> Bedrock us cross-region inference profile, verbatim from
- * the @ai-sdk/amazon-bedrock model-id union. The 4-6 generation profiles are
- * undated `-vN`; 4-5 and older carry a date and a `:N` suffix.
+ * Anthropic's official Bedrock model table. No shared suffix convention across
+ * models — copy each id exactly rather than deriving it.
  */
 export const BEDROCK_MODEL_IDS: Record<string, string> = {
-  "claude-sonnet-4-6": "us.anthropic.claude-sonnet-4-6-v1",
+  "claude-sonnet-4-6": "us.anthropic.claude-sonnet-4-6",
   "claude-haiku-4-5": "us.anthropic.claude-haiku-4-5-20251001-v1:0",
 };