create-ailk 0.4.0 → 0.6.0

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

@@ -1,202 +0,0 @@
-// oss: false
-/**
- * Tests for the create_page MCP tool handler.
- *
- * D14a Stage 1 Unit C — Layer 1 / Layer 2 acceptance criteria mapped here:
- *   - Free tier → canonical payment_required body, no adapter call.
- *   - Bad frontmatter → validation_failed, no adapter call.
- *   - Slug-already-exists → conflict body with details.existingVersion echoed
- *     (so the agent can pivot to update without re-fetch).
- *   - Pro tier + valid input + new slug → { page, version } success body.
- *
- * Adapter is injected as the second argument so this unit test does not need
- * Stage 1 Unit A's MDX implementation in place. The default adapter resolution
- * (createMdxAdapter against process.cwd()/content) is exercised by integration
- * tests once Unit A lands.
- */
-
-import { AdapterWriteError, type ContentAdapter } from "@working-theory/content-adapters";
-
-const mockCheckLicense = jest.fn();
-
-jest.mock("@working-theory/license", () => ({
-  checkLicense: (...args: unknown[]) => mockCheckLicense(...args),
-}));
-
-import { ailkCreatePage } from "../src/tools/create_page.js";
-
-interface AdapterCalls {
-  createPage: jest.Mock;
-  getPageBySlug: jest.Mock;
-  getPageVersion: jest.Mock;
-}
-
-function makeAdapter(overrides: Partial<AdapterCalls> = {}): ContentAdapter {
-  const calls: AdapterCalls = {
-    createPage: jest.fn(),
-    getPageBySlug: jest.fn().mockResolvedValue(null),
-    getPageVersion: jest.fn(),
-    ...overrides,
-  };
-  // eslint-disable-next-line no-restricted-syntax -- #506/#507: pre-existing test-hygiene violation
-  return {
-    getPageBySlug: calls.getPageBySlug,
-    listPages: jest.fn().mockResolvedValue([]),
-    getDetailBySlug: jest.fn().mockResolvedValue(null),
-    listDetail: jest.fn().mockResolvedValue([]),
-    searchPages: jest.fn().mockResolvedValue([]),
-    extractStructuredData: jest.fn().mockResolvedValue(null),
-    createPage: calls.createPage,
-    updatePage: jest.fn(),
-    deletePage: jest.fn(),
-    getPageVersion: calls.getPageVersion,
-  } as unknown as ContentAdapter;
-}
-
-const VALID_INPUT = {
-  type: "Product" as const,
-  slug: "widget-pro",
-  page: {
-    title: "Widget Pro",
-    description: "The professional widget.",
-  },
-  body: "# Widget Pro\n\nProfessional-grade widget.",
-};
-
-describe("ailkCreatePage", () => {
-  beforeEach(() => {
-    mockCheckLicense.mockReset();
-  });
-
-  describe("license gating", () => {
-    it("returns canonical payment_required body for free tier; adapter not called", async () => {
-      mockCheckLicense.mockResolvedValue({
-        tier: "free",
-        plan: null,
-        features: [],
-        expiresAt: null,
-        revoked: false,
-      });
-      const adapter = makeAdapter();
-
-      const result = await ailkCreatePage(VALID_INPUT, adapter);
-
-      expect(result).toMatchObject({ error: "payment_required" });
-      expect(adapter.createPage).not.toHaveBeenCalled();
-    });
-  });
-
-  describe("validation", () => {
-    beforeEach(() => {
-      mockCheckLicense.mockResolvedValue({
-        tier: "pro",
-        plan: "pro",
-        features: [],
-        expiresAt: null,
-        revoked: false,
-      });
-    });
-
-    it("returns validation_failed when frontmatter is missing required fields", async () => {
-      const adapter = makeAdapter();
-      const result = await ailkCreatePage(
-        {
-          ...VALID_INPUT,
-          page: { title: "Only title" }, // missing description
-        },
-        adapter,
-      );
-      expect(result).toMatchObject({ error: "validation_failed" });
-      expect(adapter.createPage).not.toHaveBeenCalled();
-    });
-
-    it("returns validation_failed when body is empty", async () => {
-      const adapter = makeAdapter();
-      const result = await ailkCreatePage(
-        { ...VALID_INPUT, body: "" },
-        adapter,
-      );
-      expect(result).toMatchObject({ error: "validation_failed" });
-      expect(adapter.createPage).not.toHaveBeenCalled();
-    });
-
-    it("returns validation_failed for unknown SchemaOrgType", async () => {
-      const adapter = makeAdapter();
-      const result = await ailkCreatePage(
-        // eslint-disable-next-line @typescript-eslint/no-explicit-any
-        { ...VALID_INPUT, type: "NotAType" as any },
-        adapter,
-      );
-      expect(result).toMatchObject({ error: "validation_failed" });
-      expect(adapter.createPage).not.toHaveBeenCalled();
-    });
-  });
-
-  describe("slug conflict", () => {
-    it("returns conflict with details.existingVersion when slug already exists", async () => {
-      mockCheckLicense.mockResolvedValue({
-        tier: "pro",
-        plan: "pro",
-        features: [],
-        expiresAt: null,
-        revoked: false,
-      });
-      const adapter = makeAdapter({
-        createPage: jest
-          .fn()
-          .mockRejectedValue(
-            new AdapterWriteError(
-              "conflict",
-              "Page already exists at Product/widget-pro.mdx",
-              { version: "abc123" },
-            ),
-          ),
-      });
-
-      const result = await ailkCreatePage(VALID_INPUT, adapter);
-
-      expect(result).toMatchObject({
-        error: "conflict",
-        details: { version: "abc123" },
-      });
-    });
-  });
-
-  describe("happy path", () => {
-    it("returns { page, version } when adapter succeeds", async () => {
-      mockCheckLicense.mockResolvedValue({
-        tier: "pro",
-        plan: "pro",
-        features: [],
-        expiresAt: null,
-        revoked: false,
-      });
-      const persistedPage = {
-        type: "Product" as const,
-        slug: "widget-pro",
-        title: "Widget Pro",
-        description: "The professional widget.",
-      };
-      const adapter = makeAdapter({
-        createPage: jest
-          .fn()
-          .mockResolvedValue({ page: persistedPage, version: "v1" }),
-      });
-
-      const result = await ailkCreatePage(VALID_INPUT, adapter);
-
-      expect(result).toEqual({ page: persistedPage, version: "v1" });
-      expect(adapter.createPage).toHaveBeenCalledTimes(1);
-      const callArgs = (adapter.createPage as jest.Mock).mock.calls[0];
-      expect(callArgs[0]).toBe("Product");
-      expect(callArgs[1]).toBe("widget-pro");
-      expect(callArgs[2]).toMatchObject({
-        type: "Product",
-        slug: "widget-pro",
-        title: "Widget Pro",
-        description: "The professional widget.",
-        body: "# Widget Pro\n\nProfessional-grade widget.",
-      });
-    });
-  });
-});

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

@@ -1,177 +0,0 @@
-// oss: false
-/**
- * Tests for the delete_page MCP tool handler.
- *
- * D14a Stage 1 Unit C — Layer 1 / Layer 2 acceptance criteria mapped here:
- *   - Free tier → payment_required, no adapter call.
- *   - Missing ifMatch → validation_failed.
- *   - Stale ifMatch → conflict with details.currentVersion echoed.
- *   - Slug not found → not_found body.
- *   - Pro + matching ifMatch → { ok: true } success body.
- */
-
-import { AdapterWriteError, type ContentAdapter } from "@working-theory/content-adapters";
-
-const mockCheckLicense = jest.fn();
-
-jest.mock("@working-theory/license", () => ({
-  checkLicense: (...args: unknown[]) => mockCheckLicense(...args),
-}));
-
-import { ailkDeletePage } from "../src/tools/delete_page.js";
-
-function makeAdapter(
-  overrides: Partial<{ deletePage: jest.Mock }> = {},
-): ContentAdapter {
-  // eslint-disable-next-line no-restricted-syntax -- #506/#507: pre-existing test-hygiene violation
-  return {
-    getPageBySlug: jest.fn().mockResolvedValue(null),
-    listPages: jest.fn().mockResolvedValue([]),
-    getDetailBySlug: jest.fn().mockResolvedValue(null),
-    listDetail: jest.fn().mockResolvedValue([]),
-    searchPages: jest.fn().mockResolvedValue([]),
-    extractStructuredData: jest.fn().mockResolvedValue(null),
-    createPage: jest.fn(),
-    updatePage: jest.fn(),
-    deletePage: overrides.deletePage ?? jest.fn().mockResolvedValue(undefined),
-    getPageVersion: jest.fn(),
-  } as unknown as ContentAdapter;
-}
-
-const VALID_INPUT = {
-  type: "Product" as const,
-  slug: "widget-pro",
-  ifMatch: "v1",
-};
-
-describe("ailkDeletePage", () => {
-  beforeEach(() => {
-    mockCheckLicense.mockReset();
-  });
-
-  it("returns payment_required for free tier; adapter not called", async () => {
-    mockCheckLicense.mockResolvedValue({
-      tier: "free",
-      plan: null,
-      features: [],
-      expiresAt: null,
-      revoked: false,
-    });
-    const adapter = makeAdapter();
-
-    const result = await ailkDeletePage(VALID_INPUT, adapter);
-
-    expect(result).toMatchObject({ error: "payment_required" });
-    expect(adapter.deletePage).not.toHaveBeenCalled();
-  });
-
-  it("returns validation_failed when slug violates SLUG_PATTERN (issue #398 — defence-in-depth parity with create/update)", async () => {
-    mockCheckLicense.mockResolvedValue({
-      tier: "pro",
-      plan: "pro",
-      features: [],
-      expiresAt: null,
-      revoked: false,
-    });
-    const adapter = makeAdapter();
-
-    const result = await ailkDeletePage(
-      { ...VALID_INPUT, slug: "../etc/passwd" },
-      adapter,
-    );
-
-    expect(result).toMatchObject({ error: "validation_failed" });
-    expect(adapter.deletePage).not.toHaveBeenCalled();
-  });
-
-  it("returns validation_failed when ifMatch is missing", async () => {
-    mockCheckLicense.mockResolvedValue({
-      tier: "pro",
-      plan: "pro",
-      features: [],
-      expiresAt: null,
-      revoked: false,
-    });
-    const adapter = makeAdapter();
-    const { ifMatch: _ignored, ...inputNoIfMatch } = VALID_INPUT;
-
-    const result = await ailkDeletePage(
-      // eslint-disable-next-line @typescript-eslint/no-explicit-any
-      inputNoIfMatch as any,
-      adapter,
-    );
-
-    expect(result).toMatchObject({ error: "validation_failed" });
-    expect(adapter.deletePage).not.toHaveBeenCalled();
-  });
-
-  it("returns conflict on stale ifMatch", async () => {
-    mockCheckLicense.mockResolvedValue({
-      tier: "pro",
-      plan: "pro",
-      features: [],
-      expiresAt: null,
-      revoked: false,
-    });
-    const adapter = makeAdapter({
-      deletePage: jest
-        .fn()
-        .mockRejectedValue(
-          new AdapterWriteError(
-            "conflict",
-            "Version mismatch on Product/widget-pro.mdx",
-            { currentVersion: "v9" },
-          ),
-        ),
-    });
-
-    const result = await ailkDeletePage(VALID_INPUT, adapter);
-
-    expect(result).toMatchObject({
-      error: "conflict",
-      details: { currentVersion: "v9" },
-    });
-  });
-
-  it("returns not_found when adapter signals missing slug", async () => {
-    mockCheckLicense.mockResolvedValue({
-      tier: "pro",
-      plan: "pro",
-      features: [],
-      expiresAt: null,
-      revoked: false,
-    });
-    const adapter = makeAdapter({
-      deletePage: jest
-        .fn()
-        .mockRejectedValue(
-          new AdapterWriteError(
-            "not_found",
-            "No page at Product/widget-pro.mdx",
-          ),
-        ),
-    });
-
-    const result = await ailkDeletePage(VALID_INPUT, adapter);
-
-    expect(result).toMatchObject({ error: "not_found" });
-  });
-
-  it("returns { ok: true } on happy path", async () => {
-    mockCheckLicense.mockResolvedValue({
-      tier: "pro",
-      plan: "pro",
-      features: [],
-      expiresAt: null,
-      revoked: false,
-    });
-    const adapter = makeAdapter();
-
-    const result = await ailkDeletePage(VALID_INPUT, adapter);
-
-    expect(result).toEqual({ ok: true });
-    expect(adapter.deletePage).toHaveBeenCalledWith("Product", "widget-pro", {
-      ifMatch: "v1",
-    });
-  });
-});

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

@@ -1,196 +0,0 @@
-// oss: false
-/**
- * Tests for the update_page MCP tool handler.
- *
- * D14a Stage 1 Unit C — Layer 1 / Layer 2 acceptance criteria mapped here:
- *   - Free tier → payment_required, no adapter call.
- *   - Missing ifMatch → validation_failed (caught by input schema).
- *   - Stale ifMatch → conflict body with details.currentVersion echoed.
- *   - Slug not found → not_found body.
- *   - Bad frontmatter → validation_failed, no adapter call.
- *   - Pro + valid + matching ifMatch → { page, version } success body.
- */
-
-import { AdapterWriteError, type ContentAdapter } from "@working-theory/content-adapters";
-
-const mockCheckLicense = jest.fn();
-
-jest.mock("@working-theory/license", () => ({
-  checkLicense: (...args: unknown[]) => mockCheckLicense(...args),
-}));
-
-import { ailkUpdatePage } from "../src/tools/update_page.js";
-
-function makeAdapter(
-  overrides: Partial<{
-    updatePage: jest.Mock;
-  }> = {},
-): ContentAdapter {
-  // eslint-disable-next-line no-restricted-syntax -- #506/#507: pre-existing test-hygiene violation
-  return {
-    getPageBySlug: jest.fn().mockResolvedValue(null),
-    listPages: jest.fn().mockResolvedValue([]),
-    getDetailBySlug: jest.fn().mockResolvedValue(null),
-    listDetail: jest.fn().mockResolvedValue([]),
-    searchPages: jest.fn().mockResolvedValue([]),
-    extractStructuredData: jest.fn().mockResolvedValue(null),
-    createPage: jest.fn(),
-    updatePage: overrides.updatePage ?? jest.fn(),
-    deletePage: jest.fn(),
-    getPageVersion: jest.fn(),
-  } as unknown as ContentAdapter;
-}
-
-const VALID_INPUT = {
-  type: "Product" as const,
-  slug: "widget-pro",
-  page: {
-    title: "Widget Pro v2",
-    description: "Updated.",
-  },
-  body: "# Widget Pro v2\n",
-  ifMatch: "v1",
-};
-
-describe("ailkUpdatePage", () => {
-  beforeEach(() => {
-    mockCheckLicense.mockReset();
-  });
-
-  it("returns payment_required for free tier; adapter not called", async () => {
-    mockCheckLicense.mockResolvedValue({
-      tier: "free",
-      plan: null,
-      features: [],
-      expiresAt: null,
-      revoked: false,
-    });
-    const adapter = makeAdapter();
-
-    const result = await ailkUpdatePage(VALID_INPUT, adapter);
-
-    expect(result).toMatchObject({ error: "payment_required" });
-    expect(adapter.updatePage).not.toHaveBeenCalled();
-  });
-
-  it("returns validation_failed when ifMatch is missing", async () => {
-    mockCheckLicense.mockResolvedValue({
-      tier: "pro",
-      plan: "pro",
-      features: [],
-      expiresAt: null,
-      revoked: false,
-    });
-    const adapter = makeAdapter();
-    const { ifMatch: _ignored, ...inputNoIfMatch } = VALID_INPUT;
-
-    const result = await ailkUpdatePage(
-      // eslint-disable-next-line @typescript-eslint/no-explicit-any
-      inputNoIfMatch as any,
-      adapter,
-    );
-
-    expect(result).toMatchObject({ error: "validation_failed" });
-    expect(adapter.updatePage).not.toHaveBeenCalled();
-  });
-
-  it("returns conflict with details.currentVersion on stale ifMatch", async () => {
-    mockCheckLicense.mockResolvedValue({
-      tier: "pro",
-      plan: "pro",
-      features: [],
-      expiresAt: null,
-      revoked: false,
-    });
-    const adapter = makeAdapter({
-      updatePage: jest
-        .fn()
-        .mockRejectedValue(
-          new AdapterWriteError(
-            "conflict",
-            "Version mismatch on Product/widget-pro.mdx",
-            { currentVersion: "v9" },
-          ),
-        ),
-    });
-
-    const result = await ailkUpdatePage(VALID_INPUT, adapter);
-
-    expect(result).toMatchObject({
-      error: "conflict",
-      details: { currentVersion: "v9" },
-    });
-  });
-
-  it("returns not_found when adapter signals missing slug", async () => {
-    mockCheckLicense.mockResolvedValue({
-      tier: "pro",
-      plan: "pro",
-      features: [],
-      expiresAt: null,
-      revoked: false,
-    });
-    const adapter = makeAdapter({
-      updatePage: jest
-        .fn()
-        .mockRejectedValue(
-          new AdapterWriteError(
-            "not_found",
-            "No page at Product/widget-pro.mdx",
-          ),
-        ),
-    });
-
-    const result = await ailkUpdatePage(VALID_INPUT, adapter);
-
-    expect(result).toMatchObject({ error: "not_found" });
-  });
-
-  it("returns validation_failed when frontmatter is bad", async () => {
-    mockCheckLicense.mockResolvedValue({
-      tier: "pro",
-      plan: "pro",
-      features: [],
-      expiresAt: null,
-      revoked: false,
-    });
-    const adapter = makeAdapter();
-
-    const result = await ailkUpdatePage(
-      { ...VALID_INPUT, page: { title: "Missing description" } },
-      adapter,
-    );
-
-    expect(result).toMatchObject({ error: "validation_failed" });
-    expect(adapter.updatePage).not.toHaveBeenCalled();
-  });
-
-  it("returns { page, version } on happy path", async () => {
-    mockCheckLicense.mockResolvedValue({
-      tier: "pro",
-      plan: "pro",
-      features: [],
-      expiresAt: null,
-      revoked: false,
-    });
-    const persistedPage = {
-      type: "Product" as const,
-      slug: "widget-pro",
-      title: "Widget Pro v2",
-      description: "Updated.",
-    };
-    const adapter = makeAdapter({
-      updatePage: jest
-        .fn()
-        .mockResolvedValue({ page: persistedPage, version: "v2" }),
-    });
-
-    const result = await ailkUpdatePage(VALID_INPUT, adapter);
-
-    expect(result).toEqual({ page: persistedPage, version: "v2" });
-    const callArgs = (adapter.updatePage as jest.Mock).mock.calls[0];
-    expect(callArgs[0]).toBe("Product");
-    expect(callArgs[1]).toBe("widget-pro");
-    expect(callArgs[3]).toEqual({ ifMatch: "v1" });
-  });
-});

package/templates/apps/mcp/src/tools/create_page.ts

@@ -1,96 +0,0 @@
-// oss: false
-/**
- * create_page — Paid-tier MCP tool for authoring a new content page.
- *
- * D14a Stage 1 Unit C — wires the McpServer create_page handler to the
- * ContentAdapter.createPage method (Stage 1 Unit A implements the MDX
- * adapter; tests inject a fake adapter).
- *
- * Surface contract: validates input via FrontmatterSchema.extend({ body }),
- * runs the Paid-tier license guard, calls adapter.createPage, and maps adapter
- * errors to canonical MCP error bodies (see apps/mcp/src/lib/errors.ts).
- *
- * Input shape mirrors the Stage 0 catalog entry in
- * packages/metadata/src/mcp-tool-catalog.ts. Catalog parses a permissive
- * outer shape (`page: record`); this handler parses strictly via
- * FrontmatterSchema so bad frontmatter surfaces as a validation_failed body.
- */
-
-import { join } from "node:path";
-
-import {
-  FrontmatterSchema,
-  createMdxAdapter,
-  mapAdapterError,
-  type ContentAdapter,
-} from "@working-theory/content-adapters";
-import { z } from "zod";
-
-import { validationError, type ErrorBody } from "../lib/errors.js";
-import { requirePaidTier } from "../lib/require-paid-tier.js";
-
-const CreatePageInputSchema = FrontmatterSchema.extend({
-  body: z.string().min(1),
-});
-
-interface CreatePageSuccess {
-  page: Record<string, unknown>;
-  version: string;
-}
-
-export type CreatePageResult = CreatePageSuccess | ErrorBody;
-
-export interface CreatePageInput {
-  type: string;
-  slug: string;
-  page: Record<string, unknown>;
-  body: string;
-}
-
-/**
- * ailkCreatePage handler.
- *
- * Optional `adapter` injection mirrors the Q3 resolution in the resource plan
- * (tests construct adapters directly with isolated content roots). When
- * omitted, falls back to the default MDX adapter rooted at process.cwd()/content.
- */
-export async function ailkCreatePage(
-  input: CreatePageInput,
-  adapter?: ContentAdapter,
-): Promise<CreatePageResult> {
-  const guard = await requirePaidTier();
-  if (guard) return guard;
-
-  const parsed = CreatePageInputSchema.safeParse({
-    ...input.page,
-    type: input.type,
-    slug: input.slug,
-    body: input.body,
-  });
-
-  if (!parsed.success) {
-    return validationError("Invalid frontmatter or body for create_page.", {
-      issues: parsed.error.issues,
-    });
-  }
-
-  const resolved =
-    adapter ??
-    createMdxAdapter({ contentRoot: join(process.cwd(), "content") });
-
-  try {
-    const result = await resolved.createPage(
-      parsed.data.type,
-      parsed.data.slug,
-      parsed.data,
-    );
-    return { page: result.page, version: result.version };
-  } catch (err) {
-    const mapped = mapAdapterError(err, {
-      type: parsed.data.type,
-      slug: parsed.data.slug,
-    });
-    if (mapped) return mapped;
-    throw err;
-  }
-}