@mycadet/assistant 0.12.9 → 0.12.11

package/src/runtime/routes/__tests__/cloud-credits-routes.test.ts

@@ -8,7 +8,7 @@ import { afterEach, beforeEach, describe, expect, mock, test } from "bun:test";
 
 import { UnprocessableEntityError } from "../errors.js";
 
-let cloudBaseUrl = "https://platform.test";
+let cloudBaseUrl = "https://test-cloud.mycadet.ai";
 let authHeader: string | null = "Api-Key test";
 
 // Spread the real module: replacing it wholesale breaks unrelated importers
@@ -33,9 +33,10 @@ const creditsHandler = ROUTES.find(
   (r) => r.operationId === "platform_credits",
 )!.handler;
 
-const SUMMARY_URL = "https://platform.test/v1/organizations/billing/summary/";
+const SUMMARY_URL =
+  "https://test-cloud.mycadet.ai/v1/organizations/billing/summary/";
 const SUBSCRIPTION_URL =
-  "https://platform.test/v1/organizations/billing/subscription/";
+  "https://test-cloud.mycadet.ai/v1/organizations/billing/subscription/";
 
 const FULL_SUMMARY = {
   settled_balance_usd: "50.00",
@@ -84,7 +85,7 @@ function stubFetch(
 }
 
 beforeEach(() => {
-  cloudBaseUrl = "https://platform.test";
+  cloudBaseUrl = "https://test-cloud.mycadet.ai";
   authHeader = "Api-Key test";
   fetchedUrls = [];
 });

package/src/runtime/routes/__tests__/cloud-invoice-routes.test.ts

@@ -12,7 +12,7 @@ import {
   UnprocessableEntityError,
 } from "../errors.js";
 
-let cloudBaseUrl = "https://platform.test";
+let cloudBaseUrl = "https://test-cloud.mycadet.ai";
 let authHeader: string | null = "Api-Key test";
 
 // Spread the real module: replacing it wholesale breaks unrelated importers
@@ -41,7 +41,8 @@ const getHandler = ROUTES.find(
   (r) => r.operationId === "platform_invoices_by_id_get",
 )!.handler;
 
-const INVOICES_URL = "https://platform.test/v1/organizations/billing/invoices/";
+const INVOICES_URL =
+  "https://test-cloud.mycadet.ai/v1/organizations/billing/invoices/";
 
 function invoice(id: string) {
   return {
@@ -103,7 +104,7 @@ function jsonResponse(body: unknown, status = 200): Response {
 }
 
 beforeEach(() => {
-  cloudBaseUrl = "https://platform.test";
+  cloudBaseUrl = "https://test-cloud.mycadet.ai";
   authHeader = "Api-Key test";
   fetchCalls = [];
 });

package/src/runtime/routes/__tests__/conversation-query-routes.test.ts

@@ -52,16 +52,13 @@ mock.module("../../../persistence/embeddings/embedding-backend.js", () => ({
 }));
 
 import type { AssistantEventEnvelope } from "../../../api/index.js";
-import { BACKUP_PROFILE_KEYS } from "../../../config/default-profile-names.js";
 import {
   getConfig,
   invalidateConfigCache,
-  loadConfig,
   loadRawConfig,
   setNestedValue,
 } from "../../../config/loader.js";
 import { AssistantConfigSchema } from "../../../config/schema.js";
-import { LLMConfigBase } from "../../../config/schemas/llm.js";
 import type { ConversationCreateType } from "../../../persistence/conversation-types.js";
 import {
   getDb,
@@ -847,49 +844,6 @@ describe("PUT /v1/config/llm/profiles/:name", () => {
     seedRawConfig();
   });
 
-  describe("fallbackProfile write protection", () => {
-    beforeEach(() => {
-      const llm = rawConfigFixture.llm as {
-        profiles: Record<string, Record<string, unknown>>;
-      };
-      llm.profiles.backup = {
-        provider: "anthropic",
-        model: "claude-sonnet-4-6",
-      };
-      seedRawConfig();
-    });
-
-    test("rejects a custom fallbackProfile without writing", async () => {
-      await expect(
-        replaceProfileRoute.handler({
-          pathParams: { name: "custom" },
-          body: {
-            provider: "anthropic",
-            model: "claude-sonnet-4-6",
-            fallbackProfile: "backup",
-          },
-        }),
-      ).rejects.toThrow(/Automatic fallbacks are code-owned/);
-      expect(persistedProfile("custom").fallbackProfile).toBeUndefined();
-      expect(initializeProvidersCalls).toBe(0);
-    });
-
-    test("rejects a custom pointer at a cloud backup name", async () => {
-      await expect(
-        replaceProfileRoute.handler({
-          pathParams: { name: "custom" },
-          body: {
-            provider: "anthropic",
-            model: "claude-sonnet-4-6",
-            fallbackProfile: "balanced-backup",
-          },
-        }),
-      ).rejects.toThrow(/Automatic fallbacks are code-owned/);
-      expect(persistedProfile("custom").fallbackProfile).toBeUndefined();
-      expect(initializeProvidersCalls).toBe(0);
-    });
-  });
-
   test("owns contextWindow maxInputTokens while preserving non-UI profile leaves", async () => {
     const result = await replaceProfileRoute.handler({
       pathParams: { name: "custom" },
@@ -908,20 +862,14 @@ describe("PUT /v1/config/llm/profiles/:name", () => {
 
     expect(savedProfile.provider).toBe("openai");
     expect(savedProfile.model).toBe("gpt-5.5");
-    // Write normalization completes the entry against llm.default (schema
-    // defaults here — the fixture has none), so UI-cleared fields land as
-    // explicit values instead of inherit-by-absence.
-    const schemaDefault = LLMConfigBase.parse({});
-    expect(savedProfile.maxTokens).toBe(schemaDefault.maxTokens);
+    // The entry replaces verbatim; contextWindow is route-owned — the
+    // non-UI leaves survive the write, the UI-owned maxInputTokens does
+    // not (it only lands when the body carries it).
+    expect(savedProfile.maxTokens).toBeUndefined();
     expect(savedProfile.contextWindow).toEqual({
-      ...schemaDefault.contextWindow,
       targetBudgetRatio: 0.3,
       summaryBudgetRatio: 0.08,
-      overflowRecovery: {
-        ...schemaDefault.contextWindow.overflowRecovery,
-        enabled: true,
-        maxAttempts: 4,
-      },
+      overflowRecovery: { enabled: true, maxAttempts: 4 },
     });
     expect(savedProfile.openrouter).toEqual({ only: ["anthropic"] });
   });
@@ -946,17 +894,13 @@ describe("PUT /v1/config/llm/profiles/:name", () => {
       }
     ).profiles.custom;
 
-    const schemaDefault = LLMConfigBase.parse({});
+    // Only the maxInputTokens override is taken; the non-UI leaves ride
+    // from the existing entry.
     expect(savedProfile.contextWindow).toEqual({
-      ...schemaDefault.contextWindow,
       maxInputTokens: 150000,
       targetBudgetRatio: 0.3,
       summaryBudgetRatio: 0.08,
-      overflowRecovery: {
-        ...schemaDefault.contextWindow.overflowRecovery,
-        enabled: true,
-        maxAttempts: 4,
-      },
+      overflowRecovery: { enabled: true, maxAttempts: 4 },
     });
     expect(savedProfile.openrouter).toEqual({ only: ["anthropic"] });
   });
@@ -1138,24 +1082,21 @@ describe("PUT /v1/config/llm/profiles/:name", () => {
 
   describe("cloud profile guard", () => {
     beforeEach(() => {
-      // Seed two managed-source profiles alongside the existing custom one.
-      // Every cloud profile name (including the flag-gated os-beta) is
-      // invariant while its entry is managed-source: fully read-only except
-      // the disabled → active status transition.
+      // Seed two managed profiles alongside the existing custom one. A
+      // cadet-provider entry under a managed name is read-only except the
+      // disabled → active status transition (the up-front route check).
       const profiles = (
         rawConfigFixture.llm as { profiles: Record<string, unknown> }
       ).profiles;
       profiles["balanced"] = {
-        source: "managed",
-        provider: "anthropic",
+        provider: "cadet",
         model: "claude-sonnet-4-6",
         label: "Balanced",
         status: "active",
       };
       profiles["os-beta"] = {
-        source: "managed",
-        provider: "together",
-        model: "zai-org/GLM-5.2",
+        provider: "cadet",
+        model: "claude-sonnet-4-6",
         label: "OS Beta",
         status: "active",
       };
@@ -1208,9 +1149,8 @@ describe("PUT /v1/config/llm/profiles/:name", () => {
 
       expect(savedProfile.status).toBe("active");
       // Seed fields preserved.
-      expect(savedProfile.provider).toBe("together");
-      expect(savedProfile.model).toBe("zai-org/GLM-5.2");
-      expect(savedProfile.source).toBe("managed");
+      expect(savedProfile.provider).toBe("cadet");
+      expect(savedProfile.model).toBe("claude-sonnet-4-6");
     });
 
     // The full commit-time rejection matrix for invariant cloud profiles
@@ -1257,7 +1197,7 @@ describe("PUT /v1/config/llm/profiles/:name", () => {
       (rawConfigFixture.llm as { profiles: Record<string, unknown> }).profiles[
         "balanced"
       ] = {
-        source: "managed",
+        source: "cloud",
         provider: "anthropic",
         model: "claude-sonnet-4-6",
         label: "Balanced",
@@ -1336,189 +1276,34 @@ describe("PATCH /v1/config fallbackProfile write protection", () => {
   });
 });
 
-describe("custom profile write normalization (complete overrides)", () => {
-  const configPatchRoute = ROUTES.find(
-    (r) => r.operationId === "config_patch",
-  )!;
-  const configSetRoute = ROUTES.find((r) => r.operationId === "config_set")!;
-
-  const distinctiveDefault = {
-    provider: "anthropic",
-    model: "claude-opus-4-8",
-    maxTokens: 12345,
-    temperature: 0.7,
-    logitBias: "suppress-cjk",
-  };
-
+describe("profile writes are schema-canonical", () => {
   beforeEach(() => {
     rawConfigFixture = {
       llm: {
-        default: structuredClone(distinctiveDefault),
+        default: { provider: "anthropic", model: "claude-opus-4-8" },
         profiles: {
-          partial: { source: "user", model: "claude-haiku-4-5-20251001" },
-        },
-      },
-    };
-    seedRawConfig();
-  });
-
-  const savedProfiles = () =>
-    (
-      loadRawConfig().llm as {
-        profiles: Record<string, Record<string, unknown>>;
-      }
-    ).profiles;
-
-  test("PUT of a partial body stores a complete profile", async () => {
-    await replaceProfileRoute.handler({
-      pathParams: { name: "mine" },
-      body: { model: "claude-haiku-4-5-20251001" },
-    });
-    const saved = savedProfiles().mine;
-    expect(saved.model).toBe("claude-haiku-4-5-20251001");
-    expect(saved.provider).toBe("anthropic");
-    expect(saved.maxTokens).toBe(12345);
-    // Non-null default sampling is baked in; logitBias never is.
-    expect(saved.temperature).toBe(0.7);
-    expect(saved.logitBias).toBeUndefined();
-    expect(saved.thinking).toBeDefined();
-    expect(saved.contextWindow).toBeDefined();
-  });
-
-  test("PATCH creating a partial profile stores it complete", async () => {
-    await configPatchRoute.handler({
-      body: {
-        llm: {
-          default: structuredClone(distinctiveDefault),
-          profiles: {
-            mine: { source: "user", model: "claude-haiku-4-5-20251001" },
+          partial: {
+            model: "claude-haiku-4-5-20251001",
+            futureField: "keep-me",
           },
         },
       },
-    });
-    const saved = savedProfiles().mine;
-    expect(saved.provider).toBe("anthropic");
-    expect(saved.maxTokens).toBe(12345);
-    expect(saved.temperature).toBe(0.7);
-    expect(saved.logitBias).toBeUndefined();
-  });
-
-  test("SET on a profile leaf completes the whole entry", async () => {
-    await configSetRoute.handler({
-      body: { path: "llm.profiles.partial.maxTokens", value: 999 },
-    });
-    const saved = savedProfiles().partial;
-    expect(saved.maxTokens).toBe(999);
-    expect(saved.provider).toBe("anthropic");
-    expect(saved.temperature).toBe(0.7);
-    expect(saved.model).toBe("claude-haiku-4-5-20251001");
-  });
-
-  test("an unrelated write leaves untouched partial profiles byte-identical", async () => {
-    await configSetRoute.handler({
-      body: { path: "heartbeat.activeHoursStart", value: 9 },
-    });
-    expect(savedProfiles().partial).toEqual({
-      source: "user",
-      model: "claude-haiku-4-5-20251001",
-    });
-  });
-
-  test("nested unknown keys survive completion of a touched entry", async () => {
-    (
-      rawConfigFixture.llm as { profiles: Record<string, unknown> }
-    ).profiles.partial = {
-      source: "user",
-      model: "claude-haiku-4-5-20251001",
-      contextWindow: { futureField: "keep-me", maxInputTokens: 111 },
     };
     seedRawConfig();
-    await configSetRoute.handler({
-      body: { path: "llm.profiles.partial.maxTokens", value: 999 },
-    });
-    const saved = savedProfiles().partial;
-    const contextWindow = saved.contextWindow as Record<string, unknown>;
-    // Schema-canonical saves strip keys the schema doesn't know.
-    expect(contextWindow.futureField).toBeUndefined();
-    expect(contextWindow.maxInputTokens).toBe(111);
-    expect(saved.provider).toBe("anthropic");
   });
 
   test("unknown profile keys are stripped by the schema parse", async () => {
-    (
-      rawConfigFixture.llm as { profiles: Record<string, unknown> }
-    ).profiles.partial = {
-      source: "user",
-      model: "claude-haiku-4-5-20251001",
-      futureField: "keep-me",
-    };
-    seedRawConfig();
+    const configSetRoute = ROUTES.find((r) => r.operationId === "config_set")!;
     await configSetRoute.handler({
       body: { path: "llm.profiles.partial.maxTokens", value: 999 },
     });
-    const saved = savedProfiles().partial;
+    const saved = (
+      loadRawConfig().llm as {
+        profiles: Record<string, Record<string, unknown>>;
+      }
+    ).profiles.partial;
     expect(saved.futureField).toBeUndefined();
     expect(saved.maxTokens).toBe(999);
-    expect(saved.provider).toBe("anthropic");
-  });
-
-  test("a bogus managed source on a non-catalog name is normalized and completed", async () => {
-    await replaceProfileRoute.handler({
-      pathParams: { name: "mine" },
-      body: { source: "managed", model: "claude-haiku-4-5-20251001" },
-    });
-    const saved = savedProfiles().mine;
-    expect(saved.source).toBe("user");
-    expect(saved.provider).toBe("anthropic");
-    expect(saved.maxTokens).toBe(12345);
-  });
-
-  test("a cloud status re-enable stays a thin stub", async () => {
-    (
-      rawConfigFixture.llm as { profiles: Record<string, unknown> }
-    ).profiles.balanced = { source: "managed", status: "disabled" };
-    seedRawConfig();
-    await replaceProfileRoute.handler({
-      pathParams: { name: "balanced" },
-      body: { status: null },
-    });
-    expect(savedProfiles().balanced).toEqual({ source: "managed" });
-  });
-
-  test("a mix profile write is not completed", async () => {
-    await replaceProfileRoute.handler({
-      pathParams: { name: "ab" },
-      body: {
-        label: "A/B",
-        mix: [
-          { profile: "balanced", weight: 1 },
-          { profile: "cost-optimized", weight: 1 },
-        ],
-      },
-    });
-    const saved = savedProfiles().ab;
-    expect(saved.mix).toBeDefined();
-    expect(saved.provider).toBeUndefined();
-    expect(saved.model).toBeUndefined();
-    expect(saved.maxTokens).toBeUndefined();
-  });
-
-  test("re-writing a completed profile is idempotent", async () => {
-    // Deterministic connection state: the PUT handler derives a connection
-    // for provider-carrying bodies, so both writes must see the same rows.
-    getDb().delete(providerConnections).run();
-    await replaceProfileRoute.handler({
-      pathParams: { name: "mine" },
-      body: { provider: "anthropic", model: "claude-haiku-4-5-20251001" },
-    });
-    const first = structuredClone(savedProfiles().mine);
-    rawConfigFixture = structuredClone(loadRawConfig());
-    seedRawConfig();
-    await replaceProfileRoute.handler({
-      pathParams: { name: "mine" },
-      body: first as Record<string, unknown>,
-    });
-    expect(savedProfiles().mine).toEqual(first);
   });
 });
 
@@ -1758,7 +1543,7 @@ describe("sparse services.stt patch provider seeding", () => {
   });
 });
 
-describe("config invariant flag enrichment", () => {
+describe("config GET serves profiles verbatim", () => {
   const configGetRoute = ROUTES.find((r) => r.operationId === "config_get")!;
   const configPatchRoute = ROUTES.find(
     (r) => r.operationId === "config_patch",
@@ -1775,21 +1560,19 @@ describe("config invariant flag enrichment", () => {
       llm: {
         profiles: {
           balanced: {
-            source: "managed",
+            source: "cloud",
             provider: "fireworks",
             model: "accounts/fireworks/models/glm-5p2",
             label: "Balanced",
             status: "active",
           },
           "os-beta": {
-            source: "managed",
+            source: "cloud",
             provider: "together",
             model: "zai-org/GLM-5.2",
             label: "OS Beta",
             status: "active",
           },
-          // A user-owned profile sharing a cloud name: no invariant flag —
-          // the stamp is gated on `source: "managed"` to match the guard.
           "cost-optimized": {
             source: "user",
             provider: "anthropic",
@@ -1805,43 +1588,31 @@ describe("config invariant flag enrichment", () => {
     seedRawConfig();
   });
 
-  test("GET /v1/config marks managed-source profiles invariant (incl. os-beta), not user-owned ones", async () => {
-    const body = await configGetRoute.handler({});
-    const profiles = wireProfiles(body);
-
-    expect(profiles.balanced!.invariant).toBe(true);
-    expect(profiles["os-beta"]!.invariant).toBe(true);
-    expect(profiles["cost-optimized"]!).not.toHaveProperty("invariant");
-    expect(profiles.custom!).not.toHaveProperty("invariant");
-  });
-
-  test("every managed default is invariant on the wire, Fast included", async () => {
-    // The clients drive their read-only lock off this flag, so a default that
-    // resolves from the catalog with no workspace stub must still carry it.
-    const body = await configGetRoute.handler({});
-    const profiles = wireProfiles(body);
-
-    for (const name of ["balanced", "quality-optimized", "latency-optimized"]) {
-      expect(profiles[name]!.invariant).toBe(true);
-    }
-  });
-
-  test("GET /v1/config hides managed backups from the picker catalog", async () => {
+  test("GET /v1/config returns profile entries verbatim — no wire-only flags stamped", async () => {
+    // The response is the raw loaded config: managed-ness reads off the
+    // entry's own `source`, and capabilities come from the profiles API
+    // (`GET inference/profiles`), not from flags stamped here.
     const body = await configGetRoute.handler({});
     const profiles = wireProfiles(body);
 
-    for (const name of BACKUP_PROFILE_KEYS) {
-      expect(profiles).not.toHaveProperty(name);
+    expect(profiles.balanced!.provider).toBe("fireworks");
+    expect(profiles.balanced!.model).toBe("accounts/fireworks/models/glm-5p2");
+    expect(profiles.custom!.provider).toBe("anthropic");
+    for (const profile of Object.values(profiles)) {
+      expect(profile).not.toHaveProperty("invariant");
+      expect(profile).not.toHaveProperty("supportsVision");
     }
-    expect(profiles.balanced).toBeDefined();
   });
 
-  test("PATCH /v1/config stamps the flag on the response but never persists it", async () => {
+  test("PATCH /v1/config response serves profiles verbatim and never persists wire-only keys", async () => {
     const body = await configPatchRoute.handler({
       body: { memory: { enabled: true } },
     });
     const profiles = wireProfiles(body);
-    expect(profiles.balanced!.invariant).toBe(true);
+    for (const profile of Object.values(profiles)) {
+      expect(profile).not.toHaveProperty("invariant");
+      expect(profile).not.toHaveProperty("supportsVision");
+    }
 
     const savedProfiles = (
       loadRawConfig().llm as { profiles: WireProfiles } | undefined
@@ -1849,11 +1620,12 @@ describe("config invariant flag enrichment", () => {
     expect(savedProfiles).toBeDefined();
     for (const profile of Object.values(savedProfiles!)) {
       expect(profile).not.toHaveProperty("invariant");
+      expect(profile).not.toHaveProperty("supportsVision");
     }
   });
 });
 
-describe("provider membership at the write choke point", () => {
+describe("provider labels follow the open-string Zod schema", () => {
   const configPatchRoute = ROUTES.find(
     (r) => r.operationId === "config_patch",
   )!;
@@ -1887,21 +1659,23 @@ describe("provider membership at the write choke point", () => {
     expect(llm.profiles?.work?.provider).toBe("anthropic-work");
   });
 
-  test("introducing an unknown provider is rejected", async () => {
+  test("a new provider label is accepted by the schema", async () => {
     rawConfigFixture = { llm: {} };
     seedRawConfig();
-
-    await expect(
-      configPatchRoute.handler({
-        body: {
-          llm: {
-            profiles: {
-              rogue: { source: "user", provider: "not-a-provider", model: "m" },
+    await configPatchRoute.handler({
+      body: {
+        llm: {
+          profiles: {
+            work: {
+              source: "user",
+              provider: "anthropic-work",
+              model: "claude-opus-4-8",
             },
           },
         },
-      }),
-    ).rejects.toThrow(/Invalid provider/);
+      },
+    });
+    expect(persistedProfile("work").provider).toBe("anthropic-work");
   });
 });
 
@@ -2026,7 +1800,7 @@ describe("config writes to a per-agent acp entry", () => {
 
     expect("model" in agentEntry(loadRawConfig())).toBe(false);
     expect(AssistantConfigSchema.safeParse(loadRawConfig()).success).toBe(true);
-    expect(loadConfig().acp.agents.claude?.model).toBeUndefined();
+    expect(getConfig().acp.agents.claude?.model).toBeUndefined();
   });
 
   test("a nulled per-agent command is dropped on both write paths", async () => {
@@ -2036,7 +1810,7 @@ describe("config writes to a per-agent acp entry", () => {
 
     expect("command" in agentEntry(loadRawConfig())).toBe(false);
     expect(AssistantConfigSchema.safeParse(loadRawConfig()).success).toBe(true);
-    expect(loadConfig().acp.agents.claude?.command).toBeUndefined();
+    expect(getConfig().acp.agents.claude?.command).toBeUndefined();
 
     seedRawConfig();
     await patchRoute.handler({
@@ -2047,7 +1821,7 @@ describe("config writes to a per-agent acp entry", () => {
     expect("command" in patched).toBe(false);
     expect(patched.model).toBe("opus");
     expect(AssistantConfigSchema.safeParse(loadRawConfig()).success).toBe(true);
-    expect(loadConfig().acp.agents.claude?.command).toBeUndefined();
+    expect(getConfig().acp.agents.claude?.command).toBeUndefined();
   });
 
   test("a nulled command on an id with no bundled profile is rejected", async () => {
@@ -2078,7 +1852,7 @@ describe("config writes to a per-agent acp entry", () => {
     const entry = agentEntry(loadRawConfig());
     expect("command" in entry).toBe(false);
     expect(AssistantConfigSchema.safeParse(loadRawConfig()).success).toBe(true);
-    expect(loadConfig().acp.agents.claude?.model).toBe("sonnet");
+    expect(getConfig().acp.agents.claude?.model).toBe("sonnet");
   });
 });
 
@@ -2233,7 +2007,7 @@ describe("persisted chat settings", () => {
 
     // Nulled leaves reset to the schema default on the canonical save (the
     // read-time recovery had already cleaned the stored nulls).
-    expect(loadConfig().conversations.autoArchive).toEqual({
+    expect(getConfig().conversations.autoArchive).toEqual({
       enabled: false,
       afterDays: 7,
     });
@@ -2285,11 +2059,11 @@ describe("persisted chat settings", () => {
         notifications: { newMessageEnabled: false },
       });
       invalidateConfigCache();
-      expect(loadConfig().conversations.autoArchive).toEqual({
+      expect(getConfig().conversations.autoArchive).toEqual({
         enabled: true,
         afterDays: 30,
       });
-      expect(loadConfig().notifications.newMessageEnabled).toBe(false);
+      expect(getConfig().notifications.newMessageEnabled).toBe(false);
       expect(await getRoute.handler({})).toEqual(result);
       expect(initializeProvidersCalls).toBe(1);
       expect(clearEmbeddingBackendCacheCalls).toBe(1);
@@ -2321,11 +2095,11 @@ describe("persisted chat settings", () => {
     });
     // The nulled leaves restore to the schema defaults on the canonical
     // save; the unknown sibling key (futureOption) doesn't persist.
-    expect(loadConfig().conversations.autoArchive).toEqual({
+    expect(getConfig().conversations.autoArchive).toEqual({
       enabled: false,
       afterDays: 7,
     });
-    expect(loadConfig().notifications.newMessageEnabled).toBe(true);
+    expect(getConfig().notifications.newMessageEnabled).toBe(true);
     expect(
       "futureOption" in
         (loadRawConfig().conversations as Record<string, unknown>),
@@ -2343,11 +2117,11 @@ describe("persisted chat settings", () => {
     });
     // The nulled subtrees restore to the schema defaults on the canonical
     // save — the same values a fresh read would materialize.
-    expect(loadConfig().conversations.autoArchive).toEqual({
+    expect(getConfig().conversations.autoArchive).toEqual({
       enabled: false,
       afterDays: 7,
     });
-    expect(loadConfig().notifications.newMessageEnabled).toBe(true);
+    expect(getConfig().notifications.newMessageEnabled).toBe(true);
   });
 
   test.each([
@@ -2429,7 +2203,7 @@ describe("persisted chat settings", () => {
     await setRoute.handler({
       body: { path: "notifications.newMessageEnabled", value: true },
     });
-    expect(loadConfig().notifications.newMessageEnabled).toBe(true);
+    expect(getConfig().notifications.newMessageEnabled).toBe(true);
     // Schema-canonical saves strip keys the schema doesn't know.
     expect(
       "futureOption" in

package/src/runtime/routes/__tests__/default-plugin-routes.test.ts

@@ -27,7 +27,7 @@ import {
   getDefaultPluginRouteRoots,
   getDefaultPluginRoutesDir,
 } from "../../../plugins/defaults/main.js";
-import { getWorkspacePluginsDir } from "../../../util/platform.js";
+import { getWorkspacePluginsDir } from "../../../util/layout.js";
 import { UserRouteDispatcher } from "../user-route-dispatcher.js";
 import {
   isRouteTestPath,