create-ailk 0.24.0 → 0.25.1

package/templates/apps/mcp/__tests__/catalog-drift.test.ts

@@ -1,359 +0,0 @@
-/**
- * Drift-detection test (issue #209): the MCP server's registered tool list
- * MUST equal the canonical catalog from @working-theory/metadata.
- *
- * Without this, two surfaces (apps/web manifest + apps/mcp server) could drift
- * on tool name or description and only an integration test on a real MCP
- * client would catch it. This test forces them in lock-step.
- *
- * What this asserts:
- *   - Tool names registered on the McpServer == catalog tool names
- *   - Tool descriptions registered on the McpServer == catalog descriptions
- *   - …UNDER BOTH commerce-claim states (#5971). The claim is what makes
- *     `start_purchase` visible, so comparing a claimed server against an
- *     unclaimed catalog (or vice versa) would compare unlike surfaces and
- *     report drift that isn't there — or, worse, miss drift that is.
- *   - …and under every licence-feature state × gate mode (#7136): 2 commerce
- *     states × {warn, refuse} × 4 held-feature sets = 16. In refuse mode a
- *     feature's tools are in both surfaces iff the feature is held; in warn
- *     mode all five are in both.
- *
- * What this does NOT assert:
- *   - inputSchema equality — the server registers Zod schemas; the catalog
- *     publishes JSON-Schema descriptors. Same source upstream
- *     (packages/validation), but distinct shapes downstream by design.
- */
-
-import type { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
-import {
-  getMcpToolCatalog,
-  type McpCatalogOptions,
-  type McpToolCatalogEntry,
-} from "@working-theory/metadata";
-
-import {
-  MCP_CLAIM_REFRESH_MS,
-  __setMcpCommerceClaimForTest,
-  __setMcpFeatureGateForTest,
-  __setMcpHeldFeaturesForTest,
-  initMcpCommerceClaim,
-  resetMcpCommerceClaim,
-  startMcpClaimRefresh,
-  type McpCommerceClaim,
-} from "../src/lib/commerce-claim.js";
-import { UNSERVED_MCP_TOOLS } from "../src/lib/topology.generated.js";
-import { createMcpServer, listRegisteredTools } from "../src/server.js";
-
-// The topology seam's UNSERVED_MCP_TOOLS (#8070 D7), made switchable the way
-// server.test.ts's AC-8b suite does: a getter returning an override when one
-// is set, else the real module's value, so the 16-state loop below reads
-// this scaffold's own seam ([] in the monorepo and a hub, ["create_project"]
-// in a spoke) and the #8513 D2 describe can force either state regardless.
-const mockSeam: { unserved: readonly string[] | undefined } = {
-  unserved: undefined,
-};
-// eslint-disable-next-line no-restricted-syntax -- the generated topology seam IS the boundary under test (#8513 D2); every other export passes through from the real module
-jest.mock("../src/lib/topology.generated.js", () => {
-  const actual: { UNSERVED_MCP_TOOLS: readonly string[] } =
-    jest.requireActual("../src/lib/topology.generated.js");
-  const mocked = { ...actual };
-  Object.defineProperty(mocked, "UNSERVED_MCP_TOOLS", {
-    enumerable: true,
-    get: () => mockSeam.unserved ?? actual.UNSERVED_MCP_TOOLS,
-  });
-  return mocked;
-});
-
-/** The injected licence reader — same seam `commerce-claim.test.ts` types. */
-type CheckLicenseSeam = NonNullable<
-  NonNullable<Parameters<typeof initMcpCommerceClaim>[0]>["checkLicense"]
->;
-
-type LicenseShape = Awaited<ReturnType<CheckLicenseSeam>>;
-
-/** A licence reader that resolves to the given (partial) result. */
-function resolving(result: Partial<LicenseShape>): CheckLicenseSeam {
-  return () => Promise.resolve(result as LicenseShape);
-}
-
-interface RegisteredTool {
-  description?: string;
-}
-
-// The catalog includes module-gated tools (scheduling, content-write,
-// signal-collectors); an unset AILK_ENABLED_MODULES no longer registers them
-// by default (D-44, #5368), so this drift check enables every paid module
-// explicitly — it is comparing the full catalog, not what a default build
-// registers.
-const PAID_MODULES = "scheduling,content-write,signal-collectors";
-
-/**
- * The SDK exposes no public API for listing a server's tools without a live
- * transport, so this reads the `_registeredTools` field directly. Pinned to
- * @modelcontextprotocol/sdk ^1.29.0 — if the SDK renames the field, change it
- * here once rather than at each call site.
- */
-function registeredToolsOf(server: McpServer): Record<string, RegisteredTool> {
-  return (
-    // eslint-disable-next-line no-restricted-syntax -- #506/#507: the SDK exposes no public accessor; the double assertion is confined to this one helper
-    (server as unknown as { _registeredTools: Record<string, RegisteredTool> })
-      ._registeredTools
-  );
-}
-
-type GateMode = NonNullable<McpCatalogOptions["featureGate"]>;
-
-type McpFeature = NonNullable<McpToolCatalogEntry["feature"]>;
-
-/** The six licence-feature tools, by the feature that entitles them. */
-const FEATURE_TOOLS: Record<McpFeature, readonly string[]> = {
-  scheduling: [
-    "list_availability",
-    "get_event_meta",
-    "list_meetings",
-    "create_booking",
-  ],
-  "flows-deliverables": ["get_flow", "submit_flow_step"],
-};
-
-const HELD_SETS: readonly (readonly McpFeature[])[] = [
-  [],
-  ["scheduling"],
-  ["flows-deliverables"],
-  ["scheduling", "flows-deliverables"],
-];
-
-/** Every claim state: commerce × gate mode × held features (16). */
-const CLAIM_STATES = (["unclaimed", "claimed"] as const).flatMap((claim) =>
-  (["warn", "refuse"] as const).flatMap((mode) =>
-    HELD_SETS.map(
-      (held) =>
-        [claim, mode, held] as [
-          McpCommerceClaim,
-          GateMode,
-          readonly McpFeature[],
-        ],
-    ),
-  ),
-);
-
-/** The catalog options a given claim state implies — the same mapping
- * `server.ts` applies internally, so both surfaces are read like for like. */
-function catalogOptionsFor(
-  claim: McpCommerceClaim,
-  mode: GateMode,
-  held: readonly McpFeature[],
-): McpCatalogOptions {
-  return {
-    addons: { commerce: claim === "claimed" },
-    features: held,
-    featureGate: mode,
-  };
-}
-
-/**
- * The catalog this topology actually serves — the full catalog minus the
- * seam's own UNSERVED_MCP_TOOLS, read at use (never destructured at module
- * load) so a mocked override above is seen (#8513 D2, C3).
- */
-function servedCatalog(options: McpCatalogOptions): McpToolCatalogEntry[] {
-  return getMcpToolCatalog(options).filter(
-    (t) => !UNSERVED_MCP_TOOLS.includes(t.name),
-  );
-}
-
-describe.each(CLAIM_STATES)(
-  "MCP catalog ↔ server drift — commerce %s, mode %s, held %j",
-  (claim, mode, held) => {
-    const priorEnabledModules = process.env.AILK_ENABLED_MODULES;
-    const options = catalogOptionsFor(claim, mode, held);
-
-    beforeEach(() => {
-      process.env.AILK_ENABLED_MODULES = PAID_MODULES;
-      __setMcpCommerceClaimForTest(claim);
-      __setMcpFeatureGateForTest(mode);
-      __setMcpHeldFeaturesForTest(held);
-    });
-
-    afterEach(() => {
-      resetMcpCommerceClaim();
-    });
-
-    afterAll(() => {
-      if (priorEnabledModules === undefined) {
-        delete process.env.AILK_ENABLED_MODULES;
-      } else {
-        process.env.AILK_ENABLED_MODULES = priorEnabledModules;
-      }
-    });
-
-    it("registered tool names equal the catalog minus this topology's unserved tools", () => {
-      const registered = registeredToolsOf(createMcpServer());
-      const registeredNames = Object.keys(registered).sort();
-      const catalogNames = servedCatalog(options)
-        .map((t) => t.name)
-        .sort();
-      expect(registeredNames).toEqual(catalogNames);
-    });
-
-    it("registered tool descriptions equal the served catalog's descriptions", () => {
-      const registered = registeredToolsOf(createMcpServer());
-      for (const entry of servedCatalog(options)) {
-        expect(registered[entry.name]).toBeDefined();
-        expect(registered[entry.name]!.description).toBe(entry.description);
-      }
-    });
-
-    it("start_purchase appears in both surfaces exactly when the claim is held", () => {
-      const registered = registeredToolsOf(createMcpServer());
-      const expected = claim === "claimed";
-
-      expect("start_purchase" in registered).toBe(expected);
-      expect(
-        getMcpToolCatalog(options).some((t) => t.name === "start_purchase"),
-      ).toBe(expected);
-    });
-
-    it("each feature's tools appear in both surfaces exactly when the gate lets them", () => {
-      const registered = registeredToolsOf(createMcpServer());
-      const catalogNames = getMcpToolCatalog(options).map((t) => t.name);
-
-      for (const [feature, tools] of Object.entries(FEATURE_TOOLS)) {
-        const expected =
-          mode === "warn" || held.includes(feature as McpFeature);
-        for (const name of tools) {
-          expect(name in registered).toBe(expected);
-          expect(catalogNames.includes(name)).toBe(expected);
-        }
-      }
-    });
-  },
-);
-
-describe("MCP catalog ↔ server drift — UNSERVED_MCP_TOOLS subtracts from the served catalog (#8513 D2, AC-2)", () => {
-  const priorEnabledModules = process.env.AILK_ENABLED_MODULES;
-
-  beforeEach(() => {
-    process.env.AILK_ENABLED_MODULES = PAID_MODULES;
-  });
-
-  afterEach(() => {
-    mockSeam.unserved = undefined;
-  });
-
-  afterAll(() => {
-    if (priorEnabledModules === undefined) {
-      delete process.env.AILK_ENABLED_MODULES;
-    } else {
-      process.env.AILK_ENABLED_MODULES = priorEnabledModules;
-    }
-  });
-
-  it("a spoke's seam (['create_project']) withholds it from registration and the served catalog", () => {
-    mockSeam.unserved = ["create_project"];
-    const options = catalogOptionsFor("unclaimed", "warn", []);
-
-    const registeredNames = Object.keys(
-      registeredToolsOf(createMcpServer()),
-    ).sort();
-    const servedNames = servedCatalog(options)
-      .map((t) => t.name)
-      .sort();
-
-    expect(registeredNames).toEqual(servedNames);
-    expect(registeredNames).not.toContain("create_project");
-    expect(servedNames).not.toContain("create_project");
-  });
-
-  it("a hub's seam ([]) registers the full catalog, create_project included", () => {
-    mockSeam.unserved = [];
-    const options = catalogOptionsFor("unclaimed", "warn", []);
-
-    const registeredNames = Object.keys(
-      registeredToolsOf(createMcpServer()),
-    ).sort();
-    const fullCatalogNames = getMcpToolCatalog(options)
-      .map((t) => t.name)
-      .sort();
-
-    expect(registeredNames).toEqual(fullCatalogNames);
-    expect(registeredNames).toContain("create_project");
-  });
-});
-
-describe.each([
-  ["hub", []],
-  ["spoke", ["create_project"]],
-] as const)(
-  "MCP catalog ↔ server agreement after a bounded re-read (#7156 AC-1) — %s seam",
-  (_topology, unserved) => {
-    const priorEnabledModules = process.env.AILK_ENABLED_MODULES;
-
-    beforeEach(() => {
-      process.env.AILK_ENABLED_MODULES = PAID_MODULES;
-      mockSeam.unserved = unserved;
-      jest.useFakeTimers();
-    });
-
-    afterEach(() => {
-      resetMcpCommerceClaim();
-      mockSeam.unserved = undefined;
-      jest.useRealTimers();
-      if (priorEnabledModules === undefined) {
-        delete process.env.AILK_ENABLED_MODULES;
-      } else {
-        process.env.AILK_ENABLED_MODULES = priorEnabledModules;
-      }
-    });
-
-    it("a new server and listRegisteredTools() match the catalog once the re-read resolves", async () => {
-      __setMcpFeatureGateForTest("refuse");
-      await initMcpCommerceClaim({
-        checkLicense: resolving({
-          tier: "pro",
-          features: ["scheduling", "flows-deliverables", "commerce"],
-        }),
-      });
-
-      const bootOptions: McpCatalogOptions = {
-        addons: { commerce: true },
-        features: ["scheduling", "flows-deliverables"],
-        featureGate: "refuse",
-      };
-      const bootNames = Object.keys(
-        registeredToolsOf(createMcpServer()),
-      ).sort();
-      // Both counts below are of the served catalog — the full catalog (21 at
-      // boot, 14 after the re-read) minus this seam's UNSERVED_MCP_TOOLS (#8665).
-      expect(bootNames).toEqual(
-        servedCatalog(bootOptions)
-          .map((t) => t.name)
-          .sort(),
-      );
-      expect(bootNames).toHaveLength(21 - UNSERVED_MCP_TOOLS.length);
-
-      startMcpClaimRefresh({ checkLicense: resolving({ features: [] }) });
-      await jest.advanceTimersByTimeAsync(MCP_CLAIM_REFRESH_MS);
-
-      const refreshedOptions: McpCatalogOptions = {
-        addons: { commerce: false },
-        features: [],
-        featureGate: "refuse",
-      };
-      const servedNames = servedCatalog(refreshedOptions)
-        .map((t) => t.name)
-        .sort();
-      expect(servedNames).toHaveLength(14 - UNSERVED_MCP_TOOLS.length);
-
-      const serverNames = Object.keys(
-        registeredToolsOf(createMcpServer()),
-      ).sort();
-      expect(serverNames).toEqual(servedNames);
-
-      const listedNames = listRegisteredTools()
-        .map((t) => t.name)
-        .sort();
-      expect(listedNames).toEqual(servedNames);
-    });
-  },
-);

package/templates/apps/mcp/__tests__/config/modules.test.ts

@@ -1,78 +0,0 @@
-/**
- * `isModuleEnabled()` default-flip regression test (D-44, #5368).
- *
- * Mirrors `apps/api/src/config/__tests__/modules.test.ts` — apps/* may not
- * import apps/*, so this reader (and its test) are duplicated rather than
- * shared. Proves the mechanical claim #5368 exists to make true: an UNSET
- * `AILK_ENABLED_MODULES` now gates every paid module OFF, the same as an
- * explicit empty string — only an explicit allowlist turns modules on, and
- * turns on exactly the modules it names.
- *
- * Reverting `enabledModules()`'s unset branch back to the `undefined`
- * sentinel (S1, #5088's original shape) makes the "unset" cases below fail
- * RED — see #5368 verification notes.
- */
-
-import { isModuleEnabled } from "../../src/config/modules.js";
-
-const RECOGNIZED_MODULES = [
-  "docs",
-  "multi-tenant",
-  "scheduling",
-  "content-write",
-] as const;
-
-describe("isModuleEnabled", () => {
-  const originalEnv = process.env.AILK_ENABLED_MODULES;
-
-  afterEach(() => {
-    if (originalEnv === undefined) {
-      delete process.env.AILK_ENABLED_MODULES;
-    } else {
-      process.env.AILK_ENABLED_MODULES = originalEnv;
-    }
-  });
-
-  describe("AILK_ENABLED_MODULES unset", () => {
-    it.each(RECOGNIZED_MODULES)("returns false for %s", (moduleName) => {
-      delete process.env.AILK_ENABLED_MODULES;
-      expect(isModuleEnabled(moduleName)).toBe(false);
-    });
-  });
-
-  describe('AILK_ENABLED_MODULES=""', () => {
-    it.each(RECOGNIZED_MODULES)("returns false for %s", (moduleName) => {
-      process.env.AILK_ENABLED_MODULES = "";
-      expect(isModuleEnabled(moduleName)).toBe(false);
-    });
-  });
-
-  describe("AILK_ENABLED_MODULES as an explicit allowlist", () => {
-    it.each(RECOGNIZED_MODULES)(
-      "enables exactly the listed modules — %s on when named",
-      (moduleName) => {
-        process.env.AILK_ENABLED_MODULES = moduleName;
-        expect(isModuleEnabled(moduleName)).toBe(true);
-        for (const other of RECOGNIZED_MODULES) {
-          if (other === moduleName) continue;
-          expect(isModuleEnabled(other)).toBe(false);
-        }
-      },
-    );
-
-    it("enables every module named in a comma list, and no others", () => {
-      process.env.AILK_ENABLED_MODULES = "multi-tenant,scheduling";
-      expect(isModuleEnabled("multi-tenant")).toBe(true);
-      expect(isModuleEnabled("scheduling")).toBe(true);
-      expect(isModuleEnabled("docs")).toBe(false);
-      expect(isModuleEnabled("content-write")).toBe(false);
-    });
-
-    it("trims whitespace and drops empty entries, same as before the flip", () => {
-      process.env.AILK_ENABLED_MODULES = " multi-tenant , , scheduling ";
-      expect(isModuleEnabled("multi-tenant")).toBe(true);
-      expect(isModuleEnabled("scheduling")).toBe(true);
-      expect(isModuleEnabled("docs")).toBe(false);
-    });
-  });
-});

package/templates/apps/mcp/__tests__/get_business_hours.test.ts

@@ -1,82 +0,0 @@
-/**
- * Tests for the get_business_hours MCP tool handler.
- *
- * Acceptance criteria verified here:
- * - getBusinessHoursInputSchema validates correct inputs (slug optional)
- * - getBusinessHoursInputSchema rejects invalid slugs
- * - ailkGetBusinessHours returns null for a slug with no content
- * - ailkGetBusinessHours output shape is correct when data exists
- * - Output validates against getBusinessHoursOutputSchema
- */
-
-import { readFileSync } from "node:fs";
-import { resolve } from "node:path";
-
-import {
-  ailkGetBusinessHours,
-  getBusinessHoursInputSchema,
-  getBusinessHoursOutputSchema,
-} from "../src/tools/get_business_hours.js";
-
-const REPO_ROOT = resolve(__dirname, "../../..");
-const CONTENT_ROOT = `${REPO_ROOT}/content`;
-
-describe("getBusinessHoursInputSchema", () => {
-  it("accepts empty input (slug is optional)", () => {
-    const result = getBusinessHoursInputSchema.safeParse({});
-    expect(result.success).toBe(true);
-  });
-
-  it("accepts a valid slug", () => {
-    const result = getBusinessHoursInputSchema.safeParse({ slug: "home" });
-    expect(result.success).toBe(true);
-  });
-
-  it("rejects a slug with path traversal", () => {
-    const result = getBusinessHoursInputSchema.safeParse({
-      slug: "../etc/passwd",
-    });
-    expect(result.success).toBe(false);
-  });
-
-  it("rejects a slug with uppercase letters", () => {
-    const result = getBusinessHoursInputSchema.safeParse({ slug: "Home" });
-    expect(result.success).toBe(false);
-  });
-
-  it("rejects an empty slug string", () => {
-    const result = getBusinessHoursInputSchema.safeParse({ slug: "" });
-    expect(result.success).toBe(false);
-  });
-});
-
-describe("ailkGetBusinessHours handler", () => {
-  it("returns null for a slug with no matching content", async () => {
-    const result = await ailkGetBusinessHours(
-      { slug: "nonexistent-slug-xqz" },
-      CONTENT_ROOT,
-    );
-    expect(result).toBeNull();
-  });
-
-  it("returns an object or null for default slug", async () => {
-    const result = await ailkGetBusinessHours({}, CONTENT_ROOT);
-    expect(result === null || typeof result === "object").toBe(true);
-  });
-
-  it("output validates against getBusinessHoursOutputSchema", async () => {
-    const result = await ailkGetBusinessHours({}, CONTENT_ROOT);
-    const parsed = getBusinessHoursOutputSchema.safeParse(result);
-    expect(parsed.success).toBe(true);
-  });
-});
-
-describe("architectural guard — no console calls", () => {
-  it("apps/mcp/src/tools/get_business_hours.ts does not call console.*", () => {
-    const src = readFileSync(
-      resolve(__dirname, "../src/tools/get_business_hours.ts"),
-      "utf-8",
-    );
-    expect(src).not.toMatch(/\bconsole\.(log|warn|error|info|debug)\b/);
-  });
-});

package/templates/apps/mcp/__tests__/get_faq.test.ts

@@ -1,80 +0,0 @@
-/**
- * Tests for the get_faq MCP tool handler.
- *
- * Acceptance criteria verified here:
- * - getFaqInputSchema validates correct inputs (slug optional)
- * - getFaqInputSchema rejects invalid slugs
- * - ailkGetFaq returns null for a slug with no content
- * - ailkGetFaq output shape is correct when data exists
- * - Output validates against getFaqOutputSchema
- */
-
-import { readFileSync } from "node:fs";
-import { resolve } from "node:path";
-
-import {
-  ailkGetFaq,
-  getFaqInputSchema,
-  getFaqOutputSchema,
-} from "../src/tools/get_faq.js";
-
-const REPO_ROOT = resolve(__dirname, "../../..");
-const CONTENT_ROOT = `${REPO_ROOT}/content`;
-
-describe("getFaqInputSchema", () => {
-  it("accepts empty input (slug is optional)", () => {
-    const result = getFaqInputSchema.safeParse({});
-    expect(result.success).toBe(true);
-  });
-
-  it("accepts a valid slug", () => {
-    const result = getFaqInputSchema.safeParse({ slug: "faq" });
-    expect(result.success).toBe(true);
-  });
-
-  it("rejects a slug with path traversal", () => {
-    const result = getFaqInputSchema.safeParse({ slug: "../etc/passwd" });
-    expect(result.success).toBe(false);
-  });
-
-  it("rejects a slug with uppercase letters", () => {
-    const result = getFaqInputSchema.safeParse({ slug: "FAQ" });
-    expect(result.success).toBe(false);
-  });
-
-  it("rejects an empty slug string", () => {
-    const result = getFaqInputSchema.safeParse({ slug: "" });
-    expect(result.success).toBe(false);
-  });
-});
-
-describe("ailkGetFaq handler", () => {
-  it("returns null for a slug with no matching content", async () => {
-    const result = await ailkGetFaq(
-      { slug: "nonexistent-slug-xqz" },
-      CONTENT_ROOT,
-    );
-    expect(result).toBeNull();
-  });
-
-  it("returns an object or null for default slug", async () => {
-    const result = await ailkGetFaq({}, CONTENT_ROOT);
-    expect(result === null || typeof result === "object").toBe(true);
-  });
-
-  it("output validates against getFaqOutputSchema when non-null", async () => {
-    const result = await ailkGetFaq({}, CONTENT_ROOT);
-    const parsed = getFaqOutputSchema.safeParse(result);
-    expect(parsed.success).toBe(true);
-  });
-});
-
-describe("architectural guard — no console calls", () => {
-  it("apps/mcp/src/tools/get_faq.ts does not call console.*", () => {
-    const src = readFileSync(
-      resolve(__dirname, "../src/tools/get_faq.ts"),
-      "utf-8",
-    );
-    expect(src).not.toMatch(/\bconsole\.(log|warn|error|info|debug)\b/);
-  });
-});

package/templates/apps/mcp/__tests__/get_locations.test.ts

@@ -1,65 +0,0 @@
-/**
- * Tests for the get_locations MCP tool handler.
- *
- * Acceptance criteria verified here:
- * - getLocationsInputSchema validates correct inputs (no required fields)
- * - ailkGetLocations returns an array (empty or populated)
- * - Output shape has the correct fields when locations exist
- * - Output validates against getLocationsOutputSchema
- */
-
-import { readFileSync } from "node:fs";
-import { resolve } from "node:path";
-
-import {
-  ailkGetLocations,
-  getLocationsInputSchema,
-  getLocationsOutputSchema,
-} from "../src/tools/get_locations.js";
-
-const REPO_ROOT = resolve(__dirname, "../../..");
-const CONTENT_ROOT = `${REPO_ROOT}/content`;
-
-describe("getLocationsInputSchema", () => {
-  it("accepts empty input", () => {
-    const result = getLocationsInputSchema.safeParse({});
-    expect(result.success).toBe(true);
-  });
-
-  it("rejects extra unknown fields (strict mode not applied — passthrough)", () => {
-    // Zod strips unknown fields by default (not strict); this should pass
-    const result = getLocationsInputSchema.safeParse({ unexpected: true });
-    expect(result.success).toBe(true);
-  });
-});
-
-describe("ailkGetLocations handler", () => {
-  it("returns an array (may be empty if no Place content exists)", async () => {
-    const result = await ailkGetLocations({}, CONTENT_ROOT);
-    expect(Array.isArray(result.locations)).toBe(true);
-  });
-
-  it("location items have required shape fields when array is non-empty", async () => {
-    const result = await ailkGetLocations({}, CONTENT_ROOT);
-    for (const loc of result.locations) {
-      expect(typeof loc.name).toBe("string");
-      expect(typeof loc.address).toBe("object");
-    }
-  });
-
-  it("output validates against getLocationsOutputSchema", async () => {
-    const result = await ailkGetLocations({}, CONTENT_ROOT);
-    const parsed = getLocationsOutputSchema.safeParse(result);
-    expect(parsed.success).toBe(true);
-  });
-});
-
-describe("architectural guard — no console calls", () => {
-  it("apps/mcp/src/tools/get_locations.ts does not call console.*", () => {
-    const src = readFileSync(
-      resolve(__dirname, "../src/tools/get_locations.ts"),
-      "utf-8",
-    );
-    expect(src).not.toMatch(/\bconsole\.(log|warn|error|info|debug)\b/);
-  });
-});