Match Muse Spark models by prefix, not an exact id list

The id list covered the two ids in packages/opencode-go and missed the
three in packages/zen: muse-spark-1.3, muse-spark-1.2, and
muse-spark-1.3-contributor-free. Nothing normalizes the model string on
the way to supportedEfforts, so those three still fell through to the
unknown-model ladder and still could not select minimal — the exact
symptom this change exists to fix, surviving on the ids nobody tested.

Confirmed against the live endpoints that minimal returns 200 and none
returns 400 for the two contributor ids and for contributor-free.
muse-spark-1.3 and -1.2 answered with a billing gate rather than a model
rejection, so their ladder is inferred from the family, not measured.

A prefix also matches isMuseSparkLeafProvider in provider-family.ts,
which already keyed off /^muse-spark/i — the two would otherwise have
disagreed about which models are Muse Spark.

CL-7867

Author: TheGreatAxios

src/provider/reasoning-effort.test.ts

@@ -164,15 +164,27 @@ describe("supportedEfforts", () => {
     expect(supportedEfforts("glm-5.3-flash")).toEqual(["low", "high", "max"]);
   });
 
-  test("Muse Spark supports minimal through high", () => {
-    expect(supportedEfforts("muse-spark-1.3-contributor")).toEqual([
+  // Five ids ship across two catalogs (packages/opencode-go and packages/zen)
+  // and nothing normalizes the model string before it reaches supportedEfforts,
+  // so every one of them has to land on the same ladder.
+  test.each([
+    "muse-spark-1.3-contributor",
+    "muse-spark-1.2-contributor",
+    "muse-spark-1.3",
+    "muse-spark-1.2",
+    "muse-spark-1.3-contributor-free",
+  ])("Muse Spark id %s supports minimal through high", (model) => {
+    expect(supportedEfforts(model)).toEqual([
       "minimal",
       "low",
       "medium",
       "high",
     ]);
-    expect(supportedEfforts("muse-spark-1.2-contributor")).toEqual([
-      "minimal",
+    expect(defaultEffortForModel(model)).toBe("low");
+  });
+
+  test("a model merely containing muse-spark is not matched", () => {
+    expect(supportedEfforts("not-muse-spark-1.3")).toEqual([
       "low",
       "medium",
       "high",
@@ -501,11 +513,6 @@ describe("defaultEffortForModel", () => {
     expect(defaultEffortForModel("glm-5.3-flash")).toBe("max");
   });
 
-  test("Muse Spark defaults to low", () => {
-    expect(defaultEffortForModel("muse-spark-1.3-contributor")).toBe("low");
-    expect(defaultEffortForModel("muse-spark-1.2-contributor")).toBe("low");
-  });
-
   test("gpt-5 and o-series default to medium", () => {
     expect(defaultEffortForModel("gpt-5")).toBe("medium");
     expect(defaultEffortForModel("o1")).toBe("medium");

src/provider/reasoning-effort.ts

@@ -61,19 +61,29 @@ const UNKNOWN_MODEL_EFFORTS: readonly ReasoningEffort[] = [
   "high",
 ];
 
-// Muse Spark (OpenCode Go, Responses protocol) accepts minimal through high.
-// Not `none` — the Go gateway rejects it with HTTP 400 on `reasoning.effort`.
-// Measured against https://opencode.ai/zen/go/v1/responses; see CL-7867.
+// Muse Spark (Responses protocol) accepts minimal through high. Not `none` —
+// the gateway rejects it with HTTP 400 on `reasoning.effort`. Measured on
+// muse-spark-1.3-contributor and muse-spark-1.2-contributor via the Go
+// endpoint and muse-spark-1.3-contributor-free via Zen: `minimal` returns 200
+// and `none` returns 400 on all three. See CL-7867.
 const MUSE_SPARK_EFFORTS: readonly ReasoningEffort[] = [
   "minimal",
   "low",
   "medium",
   "high",
 ];
-const MUSE_SPARK_MODELS: readonly string[] = [
-  "muse-spark-1.3-contributor",
-  "muse-spark-1.2-contributor",
-];
+
+// Matched by prefix, not by an id list. The family ships under five ids across
+// two catalogs — `muse-spark-1.3-contributor` / `-1.2-contributor` in
+// packages/opencode-go, and `muse-spark-1.3` / `-1.2` /
+// `-1.3-contributor-free` in packages/zen — and nothing normalizes the model
+// string before it reaches here. An exact list silently missed three of them
+// and left the ladder at the unknown-model default. This also matches
+// isMuseSparkLeafProvider in src/subagent/provider-family.ts, which already
+// keyed off the same prefix.
+function isMuseSparkModel(model: string): boolean {
+  return /^muse-spark/i.test(model.trim());
+}
 
 // grok-4.6 accepts xhigh; grok-4.5 and composer stay on the unknown-model subset.
 const GROK_46_EFFORTS: readonly ReasoningEffort[] = [
@@ -157,7 +167,7 @@ export function supportedEfforts(
   if (GLM_53_MODELS.includes(model)) {
     return [...GLM_53_EFFORTS];
   }
-  if (MUSE_SPARK_MODELS.includes(model)) {
+  if (isMuseSparkModel(model)) {
     return [...MUSE_SPARK_EFFORTS];
   }
   return [...UNKNOWN_MODEL_EFFORTS];
@@ -227,7 +237,7 @@ export function defaultEffortForModel(
     supported.includes(desired) ? desired : undefined;
   if (model.startsWith("grok")) return pick("high");
   if (GLM_53_MODELS.includes(model)) return pick("max");
-  if (MUSE_SPARK_MODELS.includes(model)) return pick("low");
+  if (isMuseSparkModel(model)) return pick("low");
   if (!isCodex && supported.includes("none")) return "none";
   if (isCodex || isKnownOpenAIReasoningModel(model)) return pick("medium");
   return undefined;