@cosmicdrift/kumiko-framework 0.328.0 → 0.328.1

package/src/engine/feature-ast/__tests__/parse.test.ts

@@ -1,2841 +0,0 @@
-// Tests for the feature-ast parser. Two layers:
-//
-//   1. Structural contract — defineFeature discovery, walker, dynamic
-//      registrar param name, source-order, SourceLocation. Every
-//      extractor relies on these.
-//   2. Per-extractor coverage (C1.5) — one focused test per concrete
-//      extractor as it lands.
-//
-// Methods without an extractor yet still get caught by the dispatcher
-// and surfaced as UnknownPattern with the correct methodName, so the
-// Designer/AI know the call exists.
-
-import { describe, expect, test } from "bun:test";
-import { resolve } from "node:path";
-import { Project } from "ts-morph";
-import { isRawRefSentinel, readDataLiteralNode } from "../extractors/shared";
-import { parseFeatureFile, parseSourceFile } from "../parse";
-
-function createProject() {
-  return new Project({
-    skipAddingFilesFromTsConfig: true,
-    skipFileDependencyResolution: true,
-    useInMemoryFileSystem: true,
-  });
-}
-
-// Helper: parse an inline source snippet without writing a file.
-// Centralised here per the test-setup-centralize feedback rule —
-// otherwise every test would repeat the project + sourceFile boilerplate.
-function parseInline(source: string) {
-  const project = createProject();
-  const sourceFile = project.createSourceFile("inline.ts", source);
-  return parseSourceFile(sourceFile);
-}
-
-describe("parseSourceFile", () => {
-  test("extracts featureName from defineFeature(name, setup)", () => {
-    const project = createProject();
-    const sourceFile = project.createSourceFile(
-      "inline.ts",
-      `
-import { defineFeature } from "@cosmicdrift/kumiko-framework/engine";
-defineFeature("myFeature", (r) => {
-  r.entity("task", { fields: { name: { type: "text" } } });
-});
-`,
-    );
-
-    const result = parseSourceFile(sourceFile);
-
-    expect(result.featureName).toBe("myFeature");
-  });
-
-  test("returns one pattern per r.* call, in source order", () => {
-    const result = parseInline(`
-defineFeature("foo", (r) => {
-  r.entity("task", { fields: {} });
-  r.requires("auth");
-  r.systemScope();
-});
-`);
-
-    expect(result.patterns).toHaveLength(3);
-    expect(result.patterns[0]?.kind).toBe("entity");
-    expect(result.patterns[1]?.kind).toBe("requires");
-    expect(result.patterns[2]?.kind).toBe("systemScope");
-  });
-
-  test("follows the setup callback's parameter name (NOT hardcoded 'r')", () => {
-    const result = parseInline(`
-defineFeature("alt", (registrar) => {
-  registrar.entity("task", { fields: {} });
-  registrar.requires("auth");
-});
-`);
-
-    expect(result.patterns).toHaveLength(2);
-    expect(result.patterns[0]?.kind).toBe("entity");
-    expect(result.patterns[1]?.kind).toBe("requires");
-  });
-
-  test("ignores method calls on receivers that aren't the registrar", () => {
-    const result = parseInline(`
-defineFeature("isolated", (r) => {
-  const helper = { entity: () => {} };
-  helper.entity(); // must not be reported
-  console.log("noise"); // must not be reported
-  r.entity("task", { fields: {} });
-});
-`);
-
-    // Only the actual r.entity call shows up — helper.entity and
-    // console.log are filtered out by extractRegistrarMethodName.
-    expect(result.patterns).toHaveLength(1);
-    expect(result.patterns[0]).toMatchObject({ kind: "entity", entityName: "task" });
-  });
-
-  test("returns empty result when no defineFeature is present", () => {
-    const result = parseInline("export const x = 1;");
-
-    expect(result.featureName).toBeUndefined();
-    expect(result.patterns).toEqual([]);
-    expect(result.errors).toEqual([]);
-  });
-
-  test("attaches a 1-based SourceLocation pointing at the call", () => {
-    const result = parseInline(`defineFeature("loc", (r) => {
-  r.entity("task", { fields: {} });
-});
-`);
-
-    expect(result.patterns).toHaveLength(1);
-    const source = result.patterns[0]?.source;
-    expect(source).toBeDefined();
-    // The r.entity call sits on line 2 of the snippet (1-based).
-    expect(source?.start.line).toBe(2);
-    // Raw text round-trips the original call.
-    expect(source?.raw).toContain("r.entity");
-  });
-
-  test("falls back to UnknownPattern when defineFeature is missing the setup callback", () => {
-    const result = parseInline(`defineFeature("nameOnly");`);
-
-    expect(result.featureName).toBe("nameOnly");
-    expect(result.patterns).toEqual([]);
-  });
-
-  test("recurses into a locally declared registrar-wrapper function", () => {
-    const result = parseInline(`
-function registerTaskScreens(registrar) {
-  registrar.entity("task", { fields: {} });
-  registrar.requires("auth");
-}
-defineFeature("wrapped", (r) => {
-  r.systemScope();
-  registerTaskScreens(r);
-});
-`);
-
-    expect(result.errors).toEqual([]);
-    expect(result.patterns.map((p) => p.kind)).toEqual(["entity", "requires", "systemScope"]);
-  });
-
-  test("recurses into a registrar-wrapper declared as a const arrow function", () => {
-    const result = parseInline(`
-const registerTaskScreens = (registrar) => {
-  registrar.entity("task", { fields: {} });
-};
-defineFeature("wrapped", (r) => {
-  registerTaskScreens(r);
-});
-`);
-
-    expect(result.errors).toEqual([]);
-    expect(result.patterns).toMatchObject([{ kind: "entity", entityName: "task" }]);
-  });
-
-  test("ignores a bare call that does not receive the registrar as an argument", () => {
-    const result = parseInline(`
-function unrelated() {
-  return 1;
-}
-defineFeature("f", (r) => {
-  unrelated();
-  r.systemScope();
-});
-`);
-
-    expect(result.errors).toEqual([]);
-    expect(result.patterns).toMatchObject([{ kind: "systemScope" }]);
-  });
-
-  test("does not infinite-loop on a registrar-wrapper cycle", () => {
-    const result = parseInline(`
-function a(registrar) {
-  registrar.systemScope();
-  b(registrar);
-}
-function b(registrar) {
-  a(registrar);
-}
-defineFeature("f", (r) => {
-  a(r);
-});
-`);
-
-    expect(result.errors).toEqual([]);
-    expect(result.patterns).toMatchObject([{ kind: "systemScope" }]);
-  });
-});
-
-describe("readDataLiteralNode — raw-ref sentinel", () => {
-  function readExpression(source: string) {
-    const project = createProject();
-    const sourceFile = project.createSourceFile("inline.ts", `const __probe__ = (${source});`);
-    const decl = sourceFile.getVariableDeclarationOrThrow("__probe__");
-    return readDataLiteralNode(decl.getInitializerOrThrow());
-  }
-
-  test("Identifier reference resolves to a raw-ref sentinel with the exact source text", () => {
-    const value = readExpression("eventEntity");
-    expect(isRawRefSentinel(value)).toBe(true);
-    expect(value).toEqual({ __raw: "eventEntity" });
-  });
-
-  test("zero-arg call expression resolves to a raw-ref sentinel with the exact source text", () => {
-    const value = readExpression("createInviteBrandingQuery()");
-    expect(isRawRefSentinel(value)).toBe(true);
-    expect(value).toEqual({ __raw: "createInviteBrandingQuery()" });
-  });
-
-  test("member access resolves to a raw-ref sentinel", () => {
-    const value = readExpression("config.timeout");
-    expect(isRawRefSentinel(value)).toBe(true);
-    expect(value).toEqual({ __raw: "config.timeout" });
-  });
-
-  test("a single unresolvable property no longer bubbles the whole object to undefined", () => {
-    const value = readExpression("{ timeout: 30, query: createQuery() }");
-    expect(isRawRefSentinel(value)).toBe(false);
-    expect(value).toEqual({ timeout: 30, query: { __raw: "createQuery()" } });
-  });
-});
-
-// =============================================================================
-// Round 1 extractors — concrete patterns for the simplest static APIs.
-// =============================================================================
-
-describe("extractRequires", () => {
-  test("captures every string-literal argument as featureNames", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.requires("auth", "tenant");
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "requires",
-      featureNames: ["auth", "tenant"],
-    });
-    expect(result.errors).toEqual([]);
-  });
-
-  test("resolves a local const identifier arg via the raw-ref sentinel (#1009)", () => {
-    const result = parseInline(`
-const dep = "auth";
-defineFeature("f", (r) => {
-  r.requires(dep);
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "requires",
-      featureNames: [{ __raw: "dep" }],
-    });
-    expect(result.errors).toEqual([]);
-  });
-
-  test("emits a ParseError when an argument cannot be resolved at all", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.requires(5);
-});
-`);
-
-    expect(result.patterns).toEqual([]);
-    expect(result.errors).toHaveLength(1);
-    expect(result.errors[0]?.methodName).toBe("requires");
-  });
-});
-
-describe("extractOptionalRequires", () => {
-  test("captures featureNames analogous to requires", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.optionalRequires("billing");
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "optionalRequires",
-      featureNames: ["billing"],
-    });
-  });
-
-  test("keeps a computed feature-name arg as a raw-ref sentinel instead of failing (#1009)", () => {
-    const result = parseInline(`
-const someFeature = { name: "billing" };
-defineFeature("f", (r) => {
-  r.optionalRequires(someFeature.name);
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "optionalRequires",
-      featureNames: [{ __raw: "someFeature.name" }],
-    });
-    expect(result.errors).toEqual([]);
-  });
-});
-
-describe("extractReadsConfig", () => {
-  test("captures qualifiedKeys", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.readsConfig("auth:config:jwt-ttl", "tenant:config:locale");
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "readsConfig",
-      qualifiedKeys: ["auth:config:jwt-ttl", "tenant:config:locale"],
-    });
-  });
-
-  test("keeps a computed key arg as a raw-ref sentinel instead of failing (#1009)", () => {
-    const result = parseInline(`
-const cfg = { key: "auth:config:jwt-ttl" };
-defineFeature("f", (r) => {
-  r.readsConfig(cfg.key, "tenant:config:locale");
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "readsConfig",
-      qualifiedKeys: [{ __raw: "cfg.key" }, "tenant:config:locale"],
-    });
-    expect(result.errors).toEqual([]);
-  });
-});
-
-describe("extractSystemScope", () => {
-  test("produces a SystemScopePattern with no payload", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.systemScope();
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({ kind: "systemScope" });
-  });
-});
-
-describe("extractToggleable", () => {
-  test("reads the default flag from a literal object", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.toggleable({ default: true });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "toggleable",
-      default: true,
-    });
-  });
-
-  test("emits a ParseError when the argument is missing", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.toggleable();
-});
-`);
-
-    expect(result.patterns).toEqual([]);
-    expect(result.errors).toHaveLength(1);
-    expect(result.errors[0]?.methodName).toBe("toggleable");
-  });
-
-  test("emits a ParseError when default is not a literal boolean", () => {
-    const result = parseInline(`
-const flag = true;
-defineFeature("f", (r) => {
-  r.toggleable({ default: flag });
-});
-`);
-
-    expect(result.patterns).toEqual([]);
-    expect(result.errors).toHaveLength(1);
-    expect(result.errors[0]?.methodName).toBe("toggleable");
-  });
-});
-
-// =============================================================================
-// Round 2 extractors — object-literal-based statics
-// =============================================================================
-
-describe("extractEntity", () => {
-  test("captures entityName + plain-data definition", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.entity("task", {
-    fields: {
-      title: { type: "text", required: true },
-      done: { type: "boolean", default: false },
-    },
-  });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "entity",
-      entityName: "task",
-      definition: {
-        fields: {
-          title: { type: "text", required: true },
-          done: { type: "boolean", default: false },
-        },
-      },
-    });
-    expect(result.errors).toEqual([]);
-  });
-
-  test("keeps an unresolved const reference as a raw-ref sentinel instead of erroring", () => {
-    const result = parseInline(`
-const eventEntity = { fields: { name: { type: "text" } } };
-defineFeature("f", (r) => {
-  r.entity("event", eventEntity);
-});
-`);
-
-    expect(result.errors).toEqual([]);
-    expect(result.patterns[0]).toMatchObject({
-      kind: "entity",
-      entityName: "event",
-      definition: { __raw: "eventEntity" },
-    });
-  });
-
-  test("keeps a factory call reference inside a nested property as a raw-ref sentinel", () => {
-    const result = parseInline(`
-function buildFields() {
-  return { name: { type: "text" } };
-}
-defineFeature("f", (r) => {
-  r.entity("task", { fields: buildFields() });
-});
-`);
-
-    expect(result.errors).toEqual([]);
-    expect(result.patterns[0]).toMatchObject({
-      kind: "entity",
-      entityName: "task",
-      definition: { fields: { __raw: "buildFields()" } },
-    });
-  });
-
-  test("walks through `as const` and `satisfies` wrappers", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.entity("task", { fields: { name: { type: "text" as const } } });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "entity",
-      definition: { fields: { name: { type: "text" } } },
-    });
-  });
-
-  test("emits a ParseError when the definition contains a function value", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.entity("task", {
-    fields: {
-      title: { type: "text", default: () => "untitled" },
-    },
-  });
-});
-`);
-
-    expect(result.patterns).toEqual([]);
-    expect(result.errors[0]?.methodName).toBe("entity");
-  });
-
-  test("resolves the name when it's an identifier initialized to a string literal", () => {
-    const result = parseInline(`
-const ENTITY = "task";
-defineFeature("f", (r) => {
-  r.entity(ENTITY, { fields: {} });
-});
-`);
-
-    expect(result.errors).toEqual([]);
-    expect(result.patterns[0]).toMatchObject({
-      kind: "entity",
-      entityName: "task",
-    });
-  });
-
-  test("emits a ParseError when the name identifier does not resolve to a string literal", () => {
-    const result = parseInline(`
-const ENTITY = computeEntityName();
-defineFeature("f", (r) => {
-  r.entity(ENTITY, { fields: {} });
-});
-`);
-
-    expect(result.patterns).toEqual([]);
-    expect(result.errors[0]?.methodName).toBe("entity");
-  });
-
-  test("object form captures name + definition", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.entity({ name: "task", fields: { title: { type: "text" } } });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "entity",
-      entityName: "task",
-      definition: { fields: { title: { type: "text" } } },
-    });
-  });
-
-  test("emits ParseError when called with no arguments", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.entity();
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("entity");
-  });
-
-  test("emits ParseError when object form is missing the name property", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.entity({ fields: {} });
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("entity");
-  });
-
-  test("emits ParseError when object form definition contains a function value", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.entity({ name: "task", fields: { title: { type: "text", default: () => "x" } } });
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("entity");
-  });
-
-  test("emits ParseError when the positional definition argument is missing", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.entity("task");
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("entity");
-  });
-});
-
-describe("extractRelation", () => {
-  test("reads entity ref + relation name + plain-data definition", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.relation("task", "owner", { kind: "belongsTo", to: "user" });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "relation",
-      entityName: "task",
-      relationName: "owner",
-      definition: { kind: "belongsTo", to: "user" },
-    });
-  });
-
-  test("accepts inline { name: '...' } literal as entity ref", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.relation({ name: "task" }, "owner", { kind: "belongsTo", to: "user" });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "relation",
-      entityName: "task",
-      relationName: "owner",
-    });
-  });
-
-  test("emits a ParseError when entity ref is an unresolvable identifier", () => {
-    const result = parseInline(`
-const taskRef = { name: "task" };
-defineFeature("f", (r) => {
-  r.relation(taskRef, "owner", { kind: "belongsTo", to: "user" });
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("relation");
-  });
-
-  test("object form captures entity, name and definition", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.relation({ entity: "task", name: "owner", kind: "belongsTo", to: "user" });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "relation",
-      entityName: "task",
-      relationName: "owner",
-      definition: { kind: "belongsTo", to: "user" },
-    });
-  });
-
-  test("emits ParseError when called with no arguments", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.relation();
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("relation");
-  });
-
-  test("emits ParseError when object form is missing the entity property", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.relation({ name: "owner", kind: "belongsTo" });
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("relation");
-  });
-
-  test("emits ParseError when object form is missing the name property", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.relation({ entity: "task", kind: "belongsTo" });
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("relation");
-  });
-
-  test("emits ParseError when the relation name is not a string literal", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.relation("task", REL, { kind: "belongsTo" });
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("relation");
-  });
-
-  test("emits ParseError when the positional definition argument is missing", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.relation("task", "owner");
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("relation");
-  });
-});
-
-describe("extractNav", () => {
-  test("captures the NavDefinition", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.nav({ id: "tasks", label: "Tasks", screen: "tasks:screen:list" });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "nav",
-      definition: { id: "tasks", label: "Tasks", screen: "tasks:screen:list" },
-    });
-  });
-
-  test("emits ParseError when called with no arguments", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.nav();
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("nav");
-  });
-
-  test("emits ParseError when the definition is not a plain object", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.nav(() => {});
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("nav");
-  });
-});
-
-describe("extractWorkspace", () => {
-  test("captures the WorkspaceDefinition", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.workspace({ id: "admin", label: "Admin" });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "workspace",
-      definition: { id: "admin", label: "Admin" },
-    });
-  });
-
-  test("emits ParseError when called with no arguments", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.workspace();
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("workspace");
-  });
-});
-
-// =============================================================================
-// Round 3 extractors — complex static patterns
-// =============================================================================
-
-describe("extractConfig", () => {
-  test("captures keys map", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.config({
-    keys: {
-      jwtTtl: { type: "number", default: 3600 },
-      locale: { type: "string", default: "en" },
-    },
-  });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "config",
-      keys: {
-        jwtTtl: { type: "number", default: 3600 },
-        locale: { type: "string", default: "en" },
-      },
-    });
-  });
-
-  test("emits ParseError when keys property is missing", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.config({});
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("config");
-  });
-
-  test("emits ParseError when called with no arguments", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.config();
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("config");
-  });
-
-  test("emits ParseError when the argument is not a plain object", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.config(() => {});
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("config");
-  });
-});
-
-describe("extractTranslations", () => {
-  test("captures the locale tree", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.translations({
-    keys: {
-      en: { greeting: "hello" },
-      de: { greeting: "hallo" },
-    },
-  });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "translations",
-      keys: {
-        en: { greeting: "hello" },
-        de: { greeting: "hallo" },
-      },
-    });
-  });
-
-  test("emits ParseError when called with no arguments", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.translations();
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("translations");
-  });
-
-  test("emits ParseError when keys property is missing", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.translations({});
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("translations");
-  });
-});
-
-describe("extractMetric", () => {
-  test("captures shortName + options", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.metric("requests", { type: "counter", description: "API requests" });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "metric",
-      shortName: "requests",
-      options: { type: "counter", description: "API requests" },
-    });
-  });
-
-  test("object form captures name + options", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.metric({ name: "requests", type: "counter", description: "API requests" });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "metric",
-      shortName: "requests",
-      options: { type: "counter", description: "API requests" },
-    });
-  });
-
-  test("emits ParseError when called with no arguments", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.metric();
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("metric");
-  });
-
-  test("emits ParseError when object form is missing the name property", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.metric({ type: "counter" });
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("metric");
-  });
-
-  test("emits ParseError when the name is not a string literal", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.metric(NAME, { type: "counter" });
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("metric");
-  });
-
-  test("emits ParseError when the options argument is missing", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.metric("requests");
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("metric");
-  });
-
-  test("emits ParseError when options is not a plain object", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.metric("requests", () => {});
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("metric");
-  });
-});
-
-describe("extractSecret", () => {
-  test("captures shortName + options", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.secret("apiKey", { description: "Stripe API key" });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "secret",
-      shortName: "apiKey",
-      options: { description: "Stripe API key" },
-    });
-  });
-
-  test("object form captures name + options", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.secret({ name: "apiKey", description: "Stripe API key" });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "secret",
-      shortName: "apiKey",
-      options: { description: "Stripe API key" },
-    });
-  });
-
-  test("emits ParseError when called with no arguments", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.secret();
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("secret");
-  });
-});
-
-describe("extractClaimKey", () => {
-  test("captures shortName + claim type", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.claimKey("teamId", { type: "string" });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "claimKey",
-      shortName: "teamId",
-      claimType: "string",
-    });
-  });
-
-  test("emits ParseError on invalid claim type", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.claimKey("teamId", { type: "bigint" });
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("claimKey");
-  });
-
-  test("object form captures name + type", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.claimKey({ name: "teamId", type: "string" });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "claimKey",
-      shortName: "teamId",
-      claimType: "string",
-    });
-  });
-
-  test("emits ParseError when called with no arguments", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.claimKey();
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("claimKey");
-  });
-});
-
-describe("extractReferenceData", () => {
-  test("captures entity name + data array", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.referenceData("status", [
-    { id: "open", label: "Open" },
-    { id: "closed", label: "Closed" },
-  ]);
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "referenceData",
-      entityName: "status",
-      data: [
-        { id: "open", label: "Open" },
-        { id: "closed", label: "Closed" },
-      ],
-    });
-  });
-
-  test("captures the optional upsertKey", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.referenceData("status", [{ id: "open" }], { upsertKey: "id" });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "referenceData",
-      entityName: "status",
-      upsertKey: "id",
-    });
-  });
-
-  test("object form captures entity, data and upsertKey", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.referenceData({
-    entity: "status",
-    data: [{ id: "open" }, { id: "closed" }],
-    upsertKey: "id",
-  });
-});
-`);
-
-    expect(result.errors).toEqual([]);
-    expect(result.patterns[0]).toMatchObject({
-      kind: "referenceData",
-      entityName: "status",
-      data: [{ id: "open" }, { id: "closed" }],
-      upsertKey: "id",
-    });
-  });
-
-  test("accepts an inline { name: '...' } entity ref, both forms", () => {
-    const positional = parseInline(`
-defineFeature("f", (r) => {
-  r.referenceData({ name: "status" }, [{ id: "open" }]);
-});
-`);
-    expect(positional.patterns[0]).toMatchObject({ kind: "referenceData", entityName: "status" });
-
-    const objectForm = parseInline(`
-defineFeature("f", (r) => {
-  r.referenceData({ entity: { name: "status" }, data: [{ id: "open" }] });
-});
-`);
-    expect(objectForm.patterns[0]).toMatchObject({ kind: "referenceData", entityName: "status" });
-  });
-
-  test("emits ParseError when called with no arguments", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.referenceData();
-});
-`);
-
-    expect(result.patterns).toEqual([]);
-    expect(result.errors[0]?.methodName).toBe("referenceData");
-  });
-
-  test("emits ParseError when object form is missing the data property", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.referenceData({ entity: "status" });
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("referenceData");
-  });
-
-  test("emits ParseError when object form is missing the entity property", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.referenceData({ data: [{ id: "open" }] });
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("referenceData");
-  });
-
-  test("emits ParseError when object form's data is not an array of plain objects", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.referenceData({ entity: "status", data: "not-an-array" });
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("referenceData");
-  });
-
-  test("emits ParseError when the positional data argument is missing", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.referenceData("status");
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("referenceData");
-  });
-
-  test("emits ParseError when positional upsertKey option is not a string", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.referenceData("status", [{ id: "open" }], { upsertKey: 123 });
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("referenceData");
-  });
-
-  test("emits ParseError when object form entity ref is unresolvable", () => {
-    const result = parseInline(`
-const ent = { name: "status" };
-defineFeature("f", (r) => {
-  r.referenceData({ entity: ent, data: [{ id: "open" }] });
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("referenceData");
-  });
-
-  test("emits ParseError when positional data is not an array of plain objects", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.referenceData("status", "not-an-array");
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("referenceData");
-  });
-
-  test("emits ParseError when positional options is not a plain object", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.referenceData("status", [{ id: "open" }], () => {});
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("referenceData");
-  });
-});
-
-describe("extractUseExtension", () => {
-  test("captures extension name + entity ref", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.useExtension("audit", "task");
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "useExtension",
-      extensionName: "audit",
-      entityName: "task",
-    });
-  });
-
-  test("captures optional options", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.useExtension("audit", "task", { mode: "verbose" });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "useExtension",
-      extensionName: "audit",
-      entityName: "task",
-      options: { mode: "verbose" },
-    });
-  });
-
-  test("object form captures name, entity ref and options", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.useExtension({ name: "audit", entity: "task", options: { mode: "verbose" } });
-});
-`);
-
-    expect(result.errors).toEqual([]);
-    expect(result.patterns[0]).toMatchObject({
-      kind: "useExtension",
-      extensionName: "audit",
-      entityName: "task",
-      options: { mode: "verbose" },
-    });
-  });
-
-  test("accepts an inline { name: '...' } entity ref, both forms", () => {
-    const positional = parseInline(`
-defineFeature("f", (r) => {
-  r.useExtension("audit", { name: "task" });
-});
-`);
-    expect(positional.patterns[0]).toMatchObject({ kind: "useExtension", entityName: "task" });
-
-    const objectForm = parseInline(`
-defineFeature("f", (r) => {
-  r.useExtension({ name: "audit", entity: { name: "task" } });
-});
-`);
-    expect(objectForm.patterns[0]).toMatchObject({ kind: "useExtension", entityName: "task" });
-  });
-
-  test("emits ParseError when called with no arguments", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.useExtension();
-});
-`);
-
-    expect(result.patterns).toEqual([]);
-    expect(result.errors[0]?.methodName).toBe("useExtension");
-  });
-
-  test("emits ParseError when object form is missing the entity property", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.useExtension({ name: "audit" });
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("useExtension");
-  });
-
-  test("emits ParseError when object form's options is not a plain object", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.useExtension({ name: "audit", entity: "task", options: "not-an-object" });
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("useExtension");
-  });
-
-  test("emits ParseError when the positional entity argument is missing", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.useExtension("audit");
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("useExtension");
-  });
-
-  test("emits ParseError when positional options argument is not a plain object", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.useExtension("audit", "task", "not-an-object");
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("useExtension");
-  });
-
-  test("emits ParseError when the extension name is not a string literal", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.useExtension(EXT, "task");
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("useExtension");
-  });
-
-  test("emits ParseError when the positional entity ref is unresolvable", () => {
-    const result = parseInline(`
-const ent = { name: "task" };
-defineFeature("f", (r) => {
-  r.useExtension("audit", ent);
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("useExtension");
-  });
-
-  test("emits ParseError when object form is missing the name property", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.useExtension({ entity: "task" });
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("useExtension");
-  });
-});
-
-// =============================================================================
-// Round 4 extractors — mixed patterns (header data + opaque body source)
-// =============================================================================
-
-describe("extractHook", () => {
-  test("captures hookType, target and the function body", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.hook("postSave", "task", (event, ctx) => { console.log(event); });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "hook",
-      hookType: "postSave",
-      target: "task",
-    });
-    const fnBody = (result.patterns[0] as { fnBody?: { raw: string } } | undefined)?.fnBody;
-    expect(fnBody?.raw).toContain("(event, ctx)");
-  });
-
-  test("captures the optional phase from the options object", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.hook("postSave", "task", (event, ctx) => {}, { phase: "afterCommit" });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({ kind: "hook", phase: "afterCommit" });
-  });
-
-  test("captures the optional escapeHatch from the options object (positional form)", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.hook("postSave", "task", (event, ctx) => {}, { escapeHatch: { reason: "reindexes via SYSTEM" } });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "hook",
-      escapeHatch: { reason: "reindexes via SYSTEM" },
-    });
-  });
-
-  test("captures the optional escapeHatch from the object form", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.hook({
-    type: "postSave",
-    target: "task",
-    handler: (event, ctx) => {},
-    escapeHatch: { reason: "reindexes via SYSTEM" },
-  });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "hook",
-      escapeHatch: { reason: "reindexes via SYSTEM" },
-    });
-  });
-
-  test("rejects an unknown hook type", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.hook("postCommit", "task", () => {});
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("hook");
-  });
-
-  test("object form captures type, target and handler", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.hook({ type: "preSave", target: "task", handler: (event, ctx) => {} });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "hook",
-      hookType: "preSave",
-      target: "task",
-    });
-  });
-
-  test("emits ParseError when object form has an unknown hook type", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.hook({ type: "postCommit", target: "task", handler: () => {} });
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("hook");
-  });
-
-  test("emits ParseError when object form is missing the handler property", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.hook({ type: "postSave", target: "task" });
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("hook");
-  });
-
-  test("emits ParseError when object form handler is not an inline function", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.hook({ type: "postSave", target: "task", handler: handlerRef });
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("hook");
-  });
-
-  test("captures a multi-entity target array", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.hook("postSave", ["task", "note"], () => {});
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "hook",
-      target: ["task", "note"],
-    });
-  });
-
-  test("emits ParseError when called with no arguments", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.hook();
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("hook");
-  });
-
-  test("emits ParseError when object form is missing the type property", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.hook({ target: "task", handler: () => {} });
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("hook");
-  });
-
-  test("emits ParseError when object form is missing the target property", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.hook({ type: "postSave", handler: () => {} });
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("hook");
-  });
-
-  test("emits ParseError when the handler is not an inline function", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.hook("postSave", "task", handlerRef);
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("hook");
-  });
-
-  test("emits ParseError when the hook type is not a string literal", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.hook(TYPE, "task", () => {});
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("hook");
-  });
-
-  test("emits ParseError when the target argument is missing", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.hook("postSave");
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("hook");
-  });
-
-  test("emits ParseError when object form target is not a valid ref", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.hook({ type: "postSave", target: 123, handler: () => {} });
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("hook");
-  });
-
-  test("emits ParseError when the hook function argument is missing", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.hook("postSave", "task");
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("hook");
-  });
-});
-
-describe("extractHook — entity-wide { allOf } target", () => {
-  test("captures hookType and the allOf entity from positional form", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.hook("postSave", { allOf: "task" }, (event, ctx) => {});
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "hook",
-      hookType: "postSave",
-      target: { allOf: "task" },
-    });
-  });
-
-  test("captures hookType and the allOf entity from object form", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.hook({ type: "postDelete", target: { allOf: "task" }, handler: () => {} });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "hook",
-      hookType: "postDelete",
-      target: { allOf: "task" },
-    });
-  });
-
-  test("rejects a malformed allOf value (not a string/ref)", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.hook("postSave", { allOf: 123 }, () => {});
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("hook");
-  });
-});
-
-describe("extractAuthClaims", () => {
-  test("captures the function body as a SourceLocation", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.authClaims(async (user, ctx) => ({ teamId: "t1" }));
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({ kind: "authClaims" });
-  });
-
-  test("emits ParseError when called with no arguments", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.authClaims();
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("authClaims");
-  });
-
-  test("emits ParseError when the argument is not an inline function", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.authClaims(handlerRef);
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("authClaims");
-  });
-});
-
-describe("extractWriteHandler", () => {
-  test("inline form: name, schema, handler, options", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.writeHandler("task:create", z.object({ title: z.string() }), async (event, ctx) => {
-    return { isSuccess: true, data: {} };
-  }, { access: { roles: ["admin"] } });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "writeHandler",
-      handlerName: "task:create",
-      access: { roles: ["admin"] },
-    });
-  });
-
-  test("object form: defineWriteHandler shape", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.writeHandler({
-    name: "task:approve",
-    schema: z.object({ id: z.string() }),
-    handler: async (event, ctx) => ({ isSuccess: true, data: {} }),
-    access: { openToAll: { reason: "test handler callable by any signed-in test user" } },
-  });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "writeHandler",
-      handlerName: "task:approve",
-      access: { openToAll: { reason: "test handler callable by any signed-in test user" } },
-    });
-  });
-
-  test("object form: description + agent hints are extracted", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.writeHandler({
-    name: "task:approve",
-    schema: z.object({ id: z.string() }),
-    handler: async (event, ctx) => ({ isSuccess: true, data: {} }),
-    description: "Approves a pending task.",
-    agent: { expose: true, risk: "high" },
-  });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "writeHandler",
-      handlerName: "task:approve",
-      description: "Approves a pending task.",
-      agent: { expose: true, risk: "high" },
-    });
-  });
-
-  test("object form: an unknown agent.risk value is dropped, not passed through", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.writeHandler({
-    name: "task:approve",
-    schema: z.object({ id: z.string() }),
-    handler: async (event, ctx) => ({ isSuccess: true, data: {} }),
-    agent: { expose: true, risk: "bogus" },
-  });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "writeHandler",
-      agent: { expose: true },
-    });
-    expect((result.patterns[0] as { agent?: { risk?: unknown } }).agent?.risk).toBeUndefined();
-  });
-
-  test("4-argument form: description + agent hints are read from the options object", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.writeHandler("task:create", z.object({ title: z.string() }), async (event, ctx) => {
-    return { isSuccess: true, data: {} };
-  }, { description: "Creates a task.", agent: { risk: "mid" } });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "writeHandler",
-      handlerName: "task:create",
-      description: "Creates a task.",
-      agent: { risk: "mid" },
-    });
-  });
-
-  test("resolves a same-file identifier arg to its object-literal form (#1007)", () => {
-    const result = parseInline(`
-const h = {
-  name: "task:archive",
-  schema: z.object({ id: z.string() }),
-  handler: async (event, ctx) => ({ isSuccess: true, data: {} }),
-};
-defineFeature("f", (r) => {
-  r.writeHandler(h);
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "writeHandler",
-      handlerName: "task:archive",
-    });
-    expect(result.errors).toEqual([]);
-  });
-
-  test("keeps an unresolvable ref (import, factory call) as an opaque pattern instead of failing (#1007)", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.writeHandler(eventCreateHandler);
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "writeHandler",
-      handlerName: undefined,
-      source: { raw: "r.writeHandler(eventCreateHandler)" },
-    });
-    expect(result.errors).toEqual([]);
-  });
-
-  test("object form: escapeHatch is extracted", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.writeHandler({
-    name: "task:reindex",
-    schema: z.object({}),
-    handler: async (event, ctx) => ({ isSuccess: true, data: {} }),
-    access: { roles: ["Admin"] },
-    escapeHatch: { reason: "bulk reindex writes via db.global()" },
-  });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "writeHandler",
-      handlerName: "task:reindex",
-      escapeHatch: { reason: "bulk reindex writes via db.global()" },
-    });
-  });
-
-  test("4-argument form: escapeHatch is read from the options object", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.writeHandler("task:reindex", z.object({}), async (event, ctx) => {
-    return { isSuccess: true, data: {} };
-  }, { access: { roles: ["Admin"] }, escapeHatch: { reason: "bulk reindex writes via db.global()" } });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "writeHandler",
-      handlerName: "task:reindex",
-      escapeHatch: { reason: "bulk reindex writes via db.global()" },
-    });
-  });
-});
-
-describe("extractQueryHandler", () => {
-  test("inline form returns kind=queryHandler", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.queryHandler("task:list", z.object({}), async (q, ctx) => []);
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "queryHandler",
-      handlerName: "task:list",
-    });
-  });
-
-  test("object form: description + agent hints are extracted", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.queryHandler({
-    name: "task:list",
-    schema: z.object({}),
-    handler: async (q, ctx) => [],
-    description: "Lists tasks.",
-    agent: { risk: "low" },
-  });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "queryHandler",
-      handlerName: "task:list",
-      description: "Lists tasks.",
-      agent: { risk: "low" },
-    });
-  });
-
-  test("object form: escapeHatch is extracted (query handlers may switch to SYSTEM via ctx.queryAs)", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.queryHandler({
-    name: "task:audit-list",
-    schema: z.object({}),
-    handler: async (q, ctx) => [],
-    access: { roles: ["Admin"] },
-    escapeHatch: { reason: "reads across tenants via ctx.queryAs(SYSTEM, ...)" },
-  });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "queryHandler",
-      handlerName: "task:audit-list",
-      escapeHatch: { reason: "reads across tenants via ctx.queryAs(SYSTEM, ...)" },
-    });
-  });
-});
-
-describe("extractStreamHandler", () => {
-  test("inline form: name, schema, handler, options", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.streamHandler("chat:complete", z.object({ prompt: z.string() }), async function* (input, ctx) {
-    yield "token";
-  }, { access: { roles: ["User"] } });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "streamHandler",
-      handlerName: "chat:complete",
-      access: { roles: ["User"] },
-    });
-  });
-
-  test("object form: name, schema, handler", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.streamHandler({
-    name: "chat:complete",
-    schema: z.object({ prompt: z.string() }),
-    handler: async function* (input, ctx) { yield "token"; },
-    access: { openToAll: { reason: "test handler callable by any signed-in test user" } },
-  });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "streamHandler",
-      handlerName: "chat:complete",
-      access: { openToAll: { reason: "test handler callable by any signed-in test user" } },
-    });
-  });
-
-  test("keeps an unresolvable ref as an opaque pattern instead of failing", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.streamHandler(chatCompleteHandler);
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "streamHandler",
-      handlerName: undefined,
-      source: { raw: "r.streamHandler(chatCompleteHandler)" },
-    });
-    expect(result.errors).toEqual([]);
-  });
-
-  test("description/agent are NOT part of StreamHandlerPattern's runtime shape", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.streamHandler({
-    name: "chat:complete",
-    schema: z.object({ prompt: z.string() }),
-    handler: async function* (input, ctx) { yield "token"; },
-    description: "Should be ignored for streamHandler.",
-    agent: { risk: "high" },
-  });
-});
-`);
-
-    expect(result.patterns[0]).not.toHaveProperty("description");
-    expect(result.patterns[0]).not.toHaveProperty("agent");
-  });
-
-  test("escapeHatch is kept on StreamHandlerPattern (StreamHandlerDef carries it)", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.streamHandler({
-    name: "chat:complete",
-    schema: z.object({ prompt: z.string() }),
-    handler: async function* (input, ctx) { yield "token"; },
-    escapeHatch: { reason: "Kept for streamHandler." },
-  });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      escapeHatch: { reason: "Kept for streamHandler." },
-    });
-  });
-});
-
-describe("extractJob", () => {
-  test("captures jobName, options and handler body", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.job("daily-cleanup", { trigger: { cron: "0 3 * * *" } }, async (ctx) => {});
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "job",
-      jobName: "daily-cleanup",
-    });
-  });
-
-  test("object form captures name, handler and extra options", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.job({
-    name: "daily-cleanup",
-    trigger: { cron: "0 3 * * *" },
-    handler: async (ctx) => {},
-  });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "job",
-      jobName: "daily-cleanup",
-      options: { trigger: { cron: "0 3 * * *" } },
-    });
-  });
-
-  test("emits ParseError when called with no arguments", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.job();
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("job");
-  });
-
-  test("emits ParseError when object form is missing the name property", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.job({ handler: async () => {} });
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("job");
-  });
-
-  test("emits ParseError when object form is missing the handler property", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.job({ name: "daily-cleanup", trigger: { cron: "0 3 * * *" } });
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("job");
-  });
-
-  test("emits ParseError when object form option value is not readable", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.job({ name: "daily-cleanup", handler: async () => {}, onComplete: () => {} });
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("job");
-  });
-
-  test("emits ParseError when object form handler is not an inline function", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.job({ name: "daily-cleanup", handler: handlerRef });
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("job");
-  });
-
-  test("emits ParseError when the job name is not a string literal", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.job(NAME, {}, async () => {});
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("job");
-  });
-
-  test("emits ParseError when the options argument is missing", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.job("daily-cleanup");
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("job");
-  });
-
-  test("emits ParseError when the handler argument is missing", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.job("daily-cleanup", { trigger: { cron: "0 3 * * *" } });
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("job");
-  });
-
-  test("emits ParseError when options is not a plain object", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.job("daily-cleanup", () => {}, async () => {});
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("job");
-  });
-
-  test("emits ParseError when the positional handler is not an inline function", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.job("daily-cleanup", { trigger: { cron: "0 3 * * *" } }, handlerRef);
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("job");
-  });
-});
-
-describe("extractAiGenerate", () => {
-  test("captures inline object literal args", () => {
-    const result = parseInline(`
-const paramsSchema = z.object({ maxTokens: z.number().optional() });
-
-defineFeature("f", (r) => {
-  defineWorkflow({
-    name: "demo",
-    steps: stepsPipeline(({ r }) => [
-      aiGenerateStep({
-        stepKey: "generate",
-        promptKey: "demo:generate",
-        promptFallback: "Write a summary.",
-        paramsSchema,
-        defaults: { enabled: true, params: { maxTokens: 256 } },
-        input: (ctx) => ctx.event.label,
-      }),
-    ]),
-  });
-});
-`);
-
-    expect(result.errors).toEqual([]);
-    expect(result.patterns).toContainEqual(
-      expect.objectContaining({
-        kind: "ai.generate",
-        stepKey: "generate",
-        promptKey: "demo:generate",
-        promptFallback: "Write a summary.",
-        defaults: { enabled: true, params: { maxTokens: 256 } },
-      }),
-    );
-  });
-
-  test("resolves same-file const spec reference", () => {
-    const result = parseInline(`
-const paramsSchema = z.object({});
-const generateSpec = {
-  stepKey: "generate",
-  promptKey: "demo:generate",
-  promptFallback: "Write a summary.",
-  paramsSchema,
-  defaults: { enabled: true, params: {} },
-  input: (ctx) => ctx.event.label,
-};
-
-defineFeature("f", (r) => {
-  defineWorkflow({
-    name: "demo",
-    steps: stepsPipeline(() => [aiGenerateStep(generateSpec)]),
-  });
-});
-`);
-
-    expect(result.errors).toEqual([]);
-    expect(result.patterns).toContainEqual(
-      expect.objectContaining({ kind: "ai.generate", stepKey: "generate" }),
-    );
-  });
-});
-
-describe("extractAiExtract", () => {
-  test("captures inline extract step", () => {
-    const result = parseInline(`
-const paramsSchema = z.object({});
-
-defineFeature("f", (r) => {
-  defineWorkflow({
-    name: "demo",
-    steps: stepsPipeline(() => [
-      aiExtractStep({
-        stepKey: "extract",
-        promptKey: "demo:extract",
-        promptFallback: "Extract fields.",
-        paramsSchema,
-        defaults: { enabled: true, params: {} },
-        outputSchema: z.object({ title: z.string() }),
-        instructions: () => "Extract the title.",
-      }),
-    ]),
-  });
-});
-`);
-
-    expect(result.errors).toEqual([]);
-    expect(result.patterns).toContainEqual(
-      expect.objectContaining({ kind: "ai.extract", stepKey: "extract" }),
-    );
-  });
-});
-
-describe("extractAiClassify", () => {
-  test("keeps unresolvable spec ref as opaque argsSource", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  defineWorkflow({
-    name: "demo",
-    steps: stepsPipeline(() => [aiClassifyStep(importedSpec)]),
-  });
-});
-`);
-
-    expect(result.errors).toEqual([]);
-    expect(result.patterns).toContainEqual(
-      expect.objectContaining({
-        kind: "ai.classify",
-        argsSource: { __raw: "importedSpec" },
-      }),
-    );
-  });
-  test("defaults with both providerId and model round-trip", () => {
-    const result = parseInline(`
-const paramsSchema = z.object({ maxTokens: z.number().optional() });
-
-defineFeature("f", (r) => {
-  defineWorkflow({
-    name: "demo",
-    steps: stepsPipeline(() => [
-      aiGenerateStep({
-        stepKey: "generate",
-        promptKey: "demo:generate",
-        promptFallback: "Write a summary.",
-        paramsSchema,
-        defaults: {
-          enabled: true,
-          params: { maxTokens: 256 },
-          providerId: "openai",
-          model: "gpt-4",
-        },
-        input: (ctx) => ctx.event.label,
-      }),
-    ]),
-  });
-});
-`);
-
-    expect(result.errors).toEqual([]);
-    expect(result.patterns).toContainEqual(
-      expect.objectContaining({
-        kind: "ai.generate",
-        defaults: {
-          enabled: true,
-          params: { maxTokens: 256 },
-          providerId: "openai",
-          model: "gpt-4",
-        },
-      }),
-    );
-  });
-
-  test("ignores ai step calls outside defineWorkflow", () => {
-    const result = parseInline(`
-const paramsSchema = z.object({});
-
-function helperOutsideWorkflow() {
-  aiGenerateStep({
-    stepKey: "orphan",
-    promptKey: "demo:orphan",
-    promptFallback: "Should not appear.",
-    paramsSchema,
-    defaults: { enabled: true, params: {} },
-    input: (ctx) => ctx.event.label,
-  });
-}
-
-defineFeature("f", (r) => {
-  defineWorkflow({
-    name: "demo",
-    steps: stepsPipeline(() => [
-      aiGenerateStep({
-        stepKey: "inside",
-        promptKey: "demo:inside",
-        promptFallback: "Inside workflow.",
-        paramsSchema,
-        defaults: { enabled: true, params: {} },
-        input: (ctx) => ctx.event.label,
-      }),
-    ]),
-  });
-});
-`);
-
-    expect(result.errors).toEqual([]);
-    const aiPatterns = result.patterns.filter((p) => p.kind.startsWith("ai."));
-    expect(aiPatterns).toHaveLength(1);
-    expect(aiPatterns[0]).toMatchObject({ kind: "ai.generate", stepKey: "inside" });
-  });
-
-  test("patterns are sorted by source line when registrar patterns precede ai steps", () => {
-    const result = parseInline(`
-const paramsSchema = z.object({});
-
-defineFeature("f", (r) => {
-  r.entity({ name: "item", fields: { name: { type: "text", required: true } } });
-
-  defineWorkflow({
-    name: "demo",
-    steps: stepsPipeline(() => [
-      aiGenerateStep({
-        stepKey: "generate",
-        promptKey: "demo:generate",
-        promptFallback: "Write.",
-        paramsSchema,
-        defaults: { enabled: true, params: {} },
-        input: (ctx) => ctx.event.label,
-      }),
-    ]),
-  });
-});
-`);
-
-    expect(result.errors).toEqual([]);
-    expect(result.patterns.length).toBeGreaterThanOrEqual(2);
-    const lines = result.patterns.map((p) => p.source.start.line);
-    expect(lines).toEqual([...lines].sort((a, b) => a - b));
-    expect(result.patterns[0]?.kind).toBe("entity");
-    expect(result.patterns.some((p) => p.kind === "ai.generate")).toBe(true);
-  });
-});
-
-describe("extractHttpRoute", () => {
-  test("captures method, path, anonymous, handler", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.httpRoute({
-    method: "GET",
-    path: "/feed.xml",
-    anonymous: true,
-    handler: async (c) => new Response("ok"),
-  });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "httpRoute",
-      method: "GET",
-      path: "/feed.xml",
-      anonymous: true,
-    });
-  });
-
-  test("emits ParseError when the argument is not an inline object", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.httpRoute("GET", "/feed.xml");
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("httpRoute");
-  });
-
-  test("emits ParseError when method is missing", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.httpRoute({ path: "/feed.xml", handler: async () => {} });
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("httpRoute");
-  });
-
-  test("emits ParseError when method is not a valid HTTP verb", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.httpRoute({ method: "FETCH", path: "/feed.xml", handler: async () => {} });
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("httpRoute");
-  });
-
-  test("emits ParseError when path is missing", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.httpRoute({ method: "GET", handler: async () => {} });
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("httpRoute");
-  });
-
-  test("emits ParseError when handler is missing", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.httpRoute({ method: "GET", path: "/feed.xml" });
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("httpRoute");
-  });
-
-  test("emits ParseError when handler is not an inline function", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.httpRoute({ method: "GET", path: "/feed.xml", handler: handlerRef });
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("httpRoute");
-  });
-});
-
-describe("extractDefineEvent", () => {
-  test("captures eventName + version", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.defineEvent("incidentOpened", z.object({ id: z.string() }), { piiFields: "none", version: 2 });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "defineEvent",
-      eventName: "incidentOpened",
-      version: 2,
-    });
-  });
-
-  test("object form captures name, schema, version and migrations", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.defineEvent({
-    name: "incidentOpened",
-    schema: z.object({ id: z.string() }),
-    piiFields: "none",
-    version: 2,
-    migrations: { "1": (payload) => payload },
-  });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "defineEvent",
-      eventName: "incidentOpened",
-      version: 2,
-      migrations: { "1": expect.anything() },
-    });
-  });
-
-  test("emits ParseError when called with no arguments", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.defineEvent();
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("defineEvent");
-  });
-
-  test("emits ParseError when object form is missing the name property", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.defineEvent({ schema: z.object({ id: z.string() }) });
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("defineEvent");
-  });
-
-  test("emits ParseError when object form is missing the schema property", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.defineEvent({ name: "incidentOpened" });
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("defineEvent");
-  });
-
-  test("emits ParseError when object form is missing the piiFields property", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.defineEvent({ name: "incidentOpened", schema: z.object({ id: z.string() }) });
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("defineEvent");
-    expect(result.errors[0]?.reason).toContain("piiFields");
-  });
-
-  test("emits ParseError when the event name is not a string literal", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.defineEvent(EVENT, z.object({ id: z.string() }));
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("defineEvent");
-  });
-
-  test("emits ParseError when the schema argument is missing", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.defineEvent("incidentOpened");
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("defineEvent");
-  });
-
-  test("emits ParseError when the positional form is missing the piiFields option", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.defineEvent("incidentOpened", z.object({ id: z.string() }), { version: 2 });
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("defineEvent");
-    expect(result.errors[0]?.reason).toContain("piiFields");
-  });
-});
-
-describe("extractDefineEvent — migrations (formerly extractEventMigration)", () => {
-  test("positional-form migrations array captures the fromVersion→transform step", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.defineEvent("incidentOpened", z.object({ id: z.string() }), {
-    piiFields: "none",
-    version: 2,
-    migrations: [
-      { fromVersion: 1, toVersion: 2, transform: (payload) => ({ ...payload, severity: "low" }) },
-    ],
-  });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "defineEvent",
-      eventName: "incidentOpened",
-      version: 2,
-      migrations: { "1": expect.anything() },
-    });
-  });
-
-  test("object-form migrations map captures the fromVersion→transform step", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.defineEvent({
-    name: "incidentOpened",
-    schema: z.object({ id: z.string() }),
-    piiFields: "none",
-    version: 2,
-    migrations: {
-      "1": (payload) => ({ ...payload, severity: "low" }),
-    },
-  });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "defineEvent",
-      eventName: "incidentOpened",
-      version: 2,
-      migrations: { "1": expect.anything() },
-    });
-  });
-
-  test("skips malformed migration array entries instead of failing the whole extract", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.defineEvent("incidentOpened", z.object({ id: z.string() }), {
-    piiFields: "none",
-    migrations: [
-      "skip-me",
-      { fromVersion: "bad", transform: (payload) => payload },
-      { fromVersion: 1, transform: (payload) => ({ ...payload, ok: true }) },
-    ],
-  });
-});
-`);
-
-    expect(result.errors).toEqual([]);
-    expect(result.patterns[0]).toMatchObject({
-      kind: "defineEvent",
-      migrations: { "1": expect.anything() },
-    });
-  });
-});
-
-describe("extractNotification", () => {
-  test("captures trigger + recipient + data bodies", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.notification("incidentOpened", {
-    trigger: { on: "incident:create" },
-    recipient: (event) => ({ tenant: event.tenantId }),
-    data: (event) => ({ id: event.id }),
-  });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "notification",
-      notificationName: "incidentOpened",
-      trigger: { on: "incident:create" },
-    });
-  });
-
-  test("object form captures name + trigger/recipient/data", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.notification({
-    name: "incidentOpened",
-    trigger: { on: "incident:create" },
-    recipient: (event) => ({ tenant: event.tenantId }),
-    data: (event) => ({ id: event.id }),
-  });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "notification",
-      notificationName: "incidentOpened",
-    });
-  });
-
-  test("captures optional template bodies", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.notification("incidentOpened", {
-    trigger: { on: "incident:create" },
-    recipient: (event) => ({}),
-    data: (event) => ({}),
-    templates: {
-      email: (payload) => payload.subject,
-      inApp: (payload) => payload.title,
-    },
-  });
-});
-`);
-
-    const templates = (result.patterns[0] as { templates?: Record<string, unknown> } | undefined)
-      ?.templates;
-    expect(Object.keys(templates ?? {})).toEqual(["email", "inApp"]);
-  });
-
-  test("emits ParseError when called with no arguments", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.notification();
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("notification");
-  });
-
-  test("emits ParseError when object form is missing the name property", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.notification({ trigger: { on: "x" }, recipient: () => ({}), data: () => ({}) });
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("notification");
-  });
-
-  test("emits ParseError when the notification name is not a string literal", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.notification(NAME, { trigger: { on: "x" }, recipient: () => ({}), data: () => ({}) });
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("notification");
-  });
-
-  test("emits ParseError when the definition object is missing", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.notification("incidentOpened");
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("notification");
-  });
-
-  test("emits ParseError when trigger is missing", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.notification("incidentOpened", { recipient: () => ({}), data: () => ({}) });
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("notification");
-  });
-
-  test("emits ParseError when trigger.on is not a string/ref", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.notification("incidentOpened", {
-    trigger: { on: 123 },
-    recipient: () => ({}),
-    data: () => ({}),
-  });
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("notification");
-  });
-
-  test("emits ParseError when recipient is not an inline function", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.notification("incidentOpened", {
-    trigger: { on: "incident:create" },
-    recipient: recipientRef,
-    data: () => ({}),
-  });
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("notification");
-  });
-
-  test("emits ParseError when data is not an inline function", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.notification("incidentOpened", {
-    trigger: { on: "incident:create" },
-    recipient: () => ({}),
-    data: dataRef,
-  });
-});
-`);
-
-    expect(result.errors[0]?.methodName).toBe("notification");
-  });
-});
-
-describe("extractProjection", () => {
-  test("captures name, sourceEntity, applyBodies map", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.projection({
-    name: "task-counter",
-    source: "task",
-    table: taskCounter,
-    apply: {
-      "task.created": async (event, tx) => {},
-      "task.updated": async (event, tx) => {},
-    },
-  });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "projection",
-      name: "task-counter",
-      sourceEntity: "task",
-    });
-    const applyBodies = (
-      result.patterns[0] as { applyBodies?: Record<string, unknown> } | undefined
-    )?.applyBodies;
-    expect(Object.keys(applyBodies ?? {})).toEqual(["task.created", "task.updated"]);
-  });
-});
-
-describe("extractMultiStreamProjection", () => {
-  test("captures name + applyBodies + delivery", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.multiStreamProjection({
-    name: "audit-log",
-    apply: {
-      "task.created": async (event, tx, ctx) => {},
-    },
-    delivery: "shared",
-  });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "multiStreamProjection",
-      name: "audit-log",
-      delivery: "shared",
-    });
-  });
-});
-
-describe("extractScreen", () => {
-  test("captures static layout and reports closure props as opaque", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.screen({
-    id: "task-list",
-    type: "entityList",
-    entity: "task",
-    columns: ["title", "status"],
-    rowActions: [
-      {
-        id: "edit",
-        label: "Edit",
-        handler: "task:update",
-        visible: (row) => row.status !== "done",
-      },
-    ],
-  });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "screen",
-    });
-    const opaque = (result.patterns[0] as { opaqueProps?: Record<string, unknown> } | undefined)
-      ?.opaqueProps;
-    expect(Object.keys(opaque ?? {})).toContain("rowActions.0.visible");
-  });
-});
-
-// =============================================================================
-// Round 5 extractors — opaque patterns
-// =============================================================================
-
-describe("extractExtendsRegistrar", () => {
-  test("captures extension name + opaque def body", () => {
-    const result = parseInline(`
-defineFeature("f", (r) => {
-  r.extendsRegistrar("audit", { hooks: { postSave: () => {} } });
-});
-`);
-
-    expect(result.patterns[0]).toMatchObject({
-      kind: "extendsRegistrar",
-      extensionName: "audit",
-    });
-  });
-});
-
-// =============================================================================
-// Regression — show-pony-shaped feature (#998)
-// =============================================================================
-// Mirrors the real-world idioms from show-pony/src/features/show-pony/feature.ts
-// that motivated #998 — inline here (not the live external repo path) so this
-// test doesn't depend on the show-pony checkout being present in this repo's CI.
-
-describe("show-pony-shaped feature (regression, #998)", () => {
-  const result = parseInline(`
-import { defineFeature } from "@cosmicdrift/kumiko-framework/engine";
-import { eventEntity } from "./schema";
-import { eventCreateHandler } from "./handlers/event-create.write";
-import { createInviteBrandingQuery } from "./handlers/invite-branding.query";
-import { registerShowPonyScreens } from "./register/screens";
-
-const someFeature = { name: "mail-foundation" };
-
-defineFeature("showpony", (r) => {
-  r.requires(someFeature.name, "config");
-  r.entity("event", eventEntity);
-  r.writeHandler(eventCreateHandler);
-  r.queryHandler(createInviteBrandingQuery());
-  registerShowPonyScreens(r);
-});
-`);
-
-  test("requires()'s computed name and entity's imported const resolve via the raw-ref sentinel (#998/#1009 fix)", () => {
-    expect(result.patterns[0]).toMatchObject({
-      kind: "requires",
-      featureNames: [{ __raw: "someFeature.name" }, "config"],
-    });
-    expect(result.patterns[1]).toMatchObject({
-      kind: "entity",
-      entityName: "event",
-      definition: { __raw: "eventEntity" },
-    });
-  });
-
-  test("writeHandler/queryHandler(handlerRef) resolve as opaque patterns (#998/#1007 fix)", () => {
-    expect(result.patterns).toMatchObject([
-      { kind: "requires" },
-      { kind: "entity" },
-      { kind: "writeHandler", handlerName: undefined },
-      { kind: "queryHandler", handlerName: undefined },
-    ]);
-    expect(result.errors).toEqual([]);
-  });
-
-  test("an imported registrar-wrapper stays invisible when its file can't be resolved (in-memory project, #1008)", () => {
-    // parseInline uses an in-memory ts-morph Project — "./register/screens"
-    // isn't a real file, so resolveModuleFile() returns undefined and the
-    // wrapper is skipped like any other unresolvable call. Real cross-file
-    // resolution against on-disk files is covered by the parseFeatureFile
-    // fixture tests below.
-    expect(result.patterns.some((p) => p.kind === "screen" || p.kind === "nav")).toBe(false);
-  });
-});
-
-describe("cross-file registrar-wrapper resolution against a real filesystem Project (#1008)", () => {
-  const fixture = resolve(__dirname, "fixtures/cross-file-registrar/feature.ts");
-  const result = parseFeatureFile(fixture);
-
-  test("resolves the imported wrapper and recognises the nav pattern it registers", () => {
-    expect(result.errors).toEqual([]);
-    expect(result.patterns.map((p) => p.kind)).toEqual(["requires", "nav"]);
-  });
-
-  test("the resolved pattern's source points at the imported file, not the entry file", () => {
-    const navPattern = result.patterns.find((p) => p.kind === "nav");
-    expect(navPattern?.source.file).toBe(
-      resolve(__dirname, "fixtures/cross-file-registrar/screens.ts"),
-    );
-    expect(navPattern?.source.file).not.toBe(fixture);
-  });
-});
-
-// #1746 — registrar-call name args authored as an imported constant
-// (`r.useExtension(EXT_TENANT_DATA, ...)`) instead of a string literal.
-// This is the dominant real-world style across the framework's own
-// bundled-features (see extension-names.ts), and previously ParseErrored
-// on every such call.
-describe("cross-file imported-constant name resolution against a real filesystem Project (#1746)", () => {
-  const fixture = resolve(__dirname, "fixtures/cross-file-name-const/feature.ts");
-  const result = parseFeatureFile(fixture);
-
-  test("resolves the identifier to the imported const's string value", () => {
-    expect(result.errors).toEqual([]);
-    expect(result.patterns).toMatchObject([{ kind: "extendsRegistrar", extensionName: "audit" }]);
-  });
-});