@caelo-cms/shared 0.10.72 → 0.11.1

package/src/ai-tools.ts

@@ -17,16 +17,18 @@ import {
   contentInstanceDeleteSchema,
   contentInstanceUpdateSchema,
   forkPlacementContentSchema,
-  MODULE_CSS_MAX,
-  MODULE_HTML_MAX,
-  MODULE_JS_MAX,
+  type ModuleField,
   moduleFieldSchema,
+  pageShortTitleSchema,
+  pageTypeSlugSchema,
   setPlacementContentSchema,
   slugSchema,
 } from "./content.js";
+import type { SourceLanguage } from "./escaped-line-breaks.js";
+import { LIST_ITEM_ATTR_RE } from "./lists/item-attr.js";
 import { pluginPreviewSelectionSchema } from "./plugin-preview.js";
 import { redirectTargetPathSchema } from "./redirect-target.js";
-import { unwrapCdataEnvelope } from "./strip-cdata.js";
+import { decodeSourceTextTransport } from "./source-text-transport.js";
 
 /**
  * v0.6.2 — `position` argument shared across the three `add_module_to_*`
@@ -61,15 +63,17 @@ function stripSurroundingQuotes(v: unknown): unknown {
 }
 /**
  * An AI tool argument holding source text (html / css / js): the given
- * string schema, with a whole-value `<![CDATA[ … ]]>` transport envelope
- * decoded first (see `unwrapCdataEnvelope`). Every AI authoring surface for
- * module, template and layout source uses it, so one encoding habit of the
- * model is handled in one place instead of refused per tool.
+ * string schema, with whole-value transport encodings decoded first — a
+ * CDATA envelope, a JSON string literal, HTML-escaped markup (see
+ * `decodeSourceTextTransport`). Every AI authoring surface for module,
+ * template and layout source uses it, so one encoding habit of the model
+ * is handled in one place instead of refused per tool.
  *
  * @param schema - the string schema (length caps, min) the decoded value must satisfy.
+ * @param language - the grammar of the value (`html`, `css`, `js`).
  */
-export function sourceTextArg<T extends z.ZodType<string>>(schema: T) {
-  return z.preprocess(unwrapCdataEnvelope, schema);
+export function sourceTextArg<T extends z.ZodType<string>>(schema: T, language: SourceLanguage) {
+  return z.preprocess((value) => decodeSourceTextTransport(value, language), schema);
 }
 
 export const positionInputSchema = z.preprocess(
@@ -83,55 +87,6 @@ export const positionInputSchema = z.preprocess(
   ]),
 );
 
-export const editModuleToolInput = z
-  .object({
-    moduleId: z.string().uuid(),
-    displayName: z.string().min(1).max(128).optional(),
-    /**
-     * v0.12.0 — rewrite the module's purpose. Optional; passing it
-     * updates the `list_modules` catalog your future self will read.
-     */
-    description: z.string().max(1000).optional(),
-    /** v0.12.0 — re-classify the module's role tag. */
-    kind: z.enum(["chrome", "hero", "content", "cta", "utility"]).optional(),
-    /**
-     * v0.12.3 (issue #106) — re-classify the module's stable `type` (the
-     * class a parent's `allowedModuleTypes` whitelist matches against,
-     * e.g. `button`). Rarely needed; usually derived from displayName at
-     * create time. Set it to make this module satisfy a parent's allowlist.
-     */
-    type: slugSchema.optional(),
-    html: sourceTextArg(z.string().max(MODULE_HTML_MAX)).optional(),
-    css: sourceTextArg(z.string().max(MODULE_CSS_MAX)).optional(),
-    js: sourceTextArg(z.string().max(MODULE_JS_MAX)).optional(),
-    /**
-     * v0.4.0 — module field schema. Each field declares one substitution
-     * slot referenced in the module HTML as `{{name}}`. Page placements
-     * fill these via `set_page_module_content`. Optional on edits — pass
-     * to replace the declared schema.
-     */
-    fields: z.array(moduleFieldSchema).max(64).optional(),
-    /** issue #164 slice 2 — see addModuleToolInput.bindThemeLiterals. */
-    bindThemeLiterals: z.boolean().optional(),
-  })
-  .strict();
-
-/**
- * v0.4.0 — `set_page_module_content` AI tool. Fills the content values for
- * a module placement on a specific page. Page-bound + branch-isolated per
- * chat until publish (unlike `edit_module` which is global + immediate).
- */
-export const setPageModuleContentToolInput = z
-  .object({
-    pageId: z.string().uuid(),
-    blockName: z.string().min(1).max(64),
-    position: z.number().int().nonnegative(),
-    /** Map of module field name → value. Fully replaces existing values. */
-    contentValues: z.record(z.string(), z.unknown()),
-  })
-  .strict();
-export type SetPageModuleContentToolInput = z.infer<typeof setPageModuleContentToolInput>;
-
 export const siteMemoryProposeToolInput = z
   .object({
     slot: z.enum(["purpose", "brand-voice", "tone", "banned-phrases", "instructions", "glossary"]),
@@ -206,9 +161,9 @@ export const addModuleToolInput = z
     description: z.string().max(1000).optional(),
     kind: z.enum(["chrome", "hero", "content", "cta", "utility"]).optional(),
     type: slugSchema.optional(),
-    html: sourceTextArg(z.string().min(1).max(50_000)).optional(),
-    css: sourceTextArg(z.string().max(50_000)).optional(),
-    js: sourceTextArg(z.string().max(50_000)).optional(),
+    html: sourceTextArg(z.string().min(1).max(50_000), "html").optional(),
+    css: sourceTextArg(z.string().max(50_000), "css").optional(),
+    js: sourceTextArg(z.string().max(50_000), "js").optional(),
     fields: z.array(moduleFieldSchema).max(64).optional(),
     bindThemeLiterals: z.boolean().optional(),
     /**
@@ -221,8 +176,8 @@ export const addModuleToolInput = z
      * stores the values as the minted module's `fields[].default` —
      * which requires explicit `fields` (moduleize field names are
      * unknowable up front) and is rejected for `moduleId` reuse (a
-     * shared module renders its stored defaults; edit_module changes
-     * them).
+     * shared module renders its stored defaults; update_modules_many
+     * changes them).
      */
     values: z.record(z.string(), z.unknown()).optional(),
   })
@@ -233,7 +188,7 @@ export const addModuleToolInput = z
         ctx.addIssue({
           code: z.ZodIssueCode.custom,
           message:
-            "Reusing a module on a layout renders the module's STORED field defaults — `values` has nothing to bind to. Drop `values` (the chrome shows the module as-is), or change the shared copy via edit_module.",
+            "Reusing a module on a layout renders the module's STORED field defaults — `values` has nothing to bind to. Drop `values` (the chrome shows the module as-is), or change the shared copy via update_modules_many.",
         });
       } else if (input.fields === undefined || input.fields.length === 0) {
         ctx.addIssue({
@@ -257,7 +212,7 @@ export const addModuleToolInput = z
     // (§1A/§11), so we do NOT reject a moduleId call that also carries authoring
     // fields. Those fields are not applied here; the tool HANDLER surfaces an
     // INFO for any ignored field whose value DIFFERS from the module's stored
-    // one, pointing at edit_module — so a dropped change is never silent (§2).
+    // one, pointing at update_modules_many — so a dropped change is never silent (§2).
     if (input.moduleId !== undefined) return;
     if (input.displayName === undefined || input.html === undefined) {
       ctx.addIssue({
@@ -274,17 +229,25 @@ export const addModuleToolInput = z
     // the exact two-step build_page was created to kill. Only enforced
     // when the caller authored explicit fields; the raw-HTML path goes
     // through moduleize, which enforces defaults itself.
+    // A `list` field bound to a shared list the module fixes (`list`), or
+    // fed by a source, already has its content: requiring a default there
+    // refused the very footer-bound-to-the-menu call the manage-menu skill
+    // prescribes (PR #1121 e2e-livedit run 40297574140).
+    const carriesContent = (f: ModuleField): boolean =>
+      ("default" in f && f.default !== undefined) ||
+      (f.kind === "list" && (f.bind === "source" || (f.bind === "shared" && f.list !== undefined)));
     if (
       input.fields !== undefined &&
       input.fields.length > 0 &&
       input.values === undefined &&
-      !input.fields.some((f) => "default" in f && f.default !== undefined)
+      !input.fields.some(carriesContent)
     ) {
       ctx.addIssue({
         code: z.ZodIssueCode.custom,
         message:
           "A minted module needs its initial content IN THIS CALL: pass `values` " +
           "({fieldName: value, …}) for this placement, or give the fields `default`s. " +
+          'A list field bound to a site list carries its content through `list` ({kind:"list", bind:"shared", list:"caelo:list/<id>"} — create the list first with upsert_lists_many). ' +
           "Without either, the module renders empty placeholders until a second call.",
       });
     }
@@ -341,45 +304,6 @@ export const deleteContentInstanceToolInput = contentInstanceDeleteSchema;
 export const setPlacementContentToolInput = setPlacementContentSchema;
 export const forkPlacementContentToolInput = forkPlacementContentSchema;
 
-export const AI_TOOLS = [
-  "edit_module",
-  "site_memory_propose",
-  "add_module",
-  "create_page",
-  // audit #3 — page metadata (name/title/slug/template/status) is one tool for
-  // 1..200 pages: `update_pages_many`. The former rename_page / set_page_title
-  // / change_page_slug were single-field wrappers over the same pages.update.
-  "update_pages_many",
-  // audit #4 — one delete tool for 1..200 pages, disposition per page.
-  // The former singular delete_page folded in (n=1 is a one-item array).
-  "delete_pages_many",
-  // one remove-module tool routed by target (page|layout).
-  "remove_module_from",
-  "set_structured_set",
-  "update_theme",
-  "set_template_layout",
-  "create_layout",
-  "set_site_defaults",
-  "duplicate_page",
-  "repoint_page_template",
-  "move_module",
-  "reorder_module",
-  "set_nav_menu",
-  "add_plugin_to_page",
-  // v0.12.0 — content_instances + placement binding
-  "list_content_instances",
-  "get_content_instance",
-  "create_content_instance",
-  "set_content_instance_values",
-  "delete_content_instance",
-  "set_placement_content",
-  "fork_placement_content",
-  // issue #299 — bulk-first build path (§11)
-  "build_page",
-  "create_content_instances",
-  "set_page_module_content_many",
-] as const;
-export type AiToolName = (typeof AI_TOOLS)[number];
 export type AddPluginToPageToolInput = z.infer<typeof addPluginToPageToolInput>;
 export type ListContentInstancesToolInput = z.infer<typeof listContentInstancesToolInput>;
 export type GetContentInstanceToolInput = z.infer<typeof getContentInstanceToolInput>;
@@ -451,9 +375,14 @@ export const CHAT_MAX_ATTACHMENTS = 4;
  * `context` come from a right-click / text selection in the live-edit
  * preview (#916): what the operator selected and the text of the block
  * around it, so the AI knows exactly what was meant. `blockName` /
- * `position` / `field` / `index` (#917) name the placement field the text
+ * `position` / `field` (#917) — plus `item` / `key` for an entry of a
+ * local list field (epic #1107) — name the placement field the text
  * comes from when the preview marked it, so the AI can address it in
- * `set_page_module_content` without inspecting the page first.
+ * `set_page_module_content_many` without inspecting the page first.
+ * `listItem` is the `data-caelo-list-item` of the list entry the element
+ * sits in (epic #1107, `docs/lists.md` §13): the chip then names that
+ * entry — of a shared list, or of a module's own list — not just the
+ * module around it.
  */
 export const chatChipSchema = z
   .object({
@@ -465,7 +394,12 @@ export const chatChipSchema = z
     blockName: z.string().min(1).max(80).optional(),
     position: z.number().int().nonnegative().optional(),
     field: z.string().min(1).max(200).optional(),
-    index: z.number().int().nonnegative().optional(),
+    item: z.string().uuid().optional(),
+    key: z
+      .string()
+      .regex(/^[a-z][a-z0-9_]{0,63}$/)
+      .optional(),
+    listItem: z.string().max(200).regex(LIST_ITEM_ATTR_RE).optional(),
   })
   .strict();
 export type ChatChip = z.infer<typeof chatChipSchema>;
@@ -555,11 +489,8 @@ export const chatPublishInput = z
       .array(
         z
           .object({
-            // v0.9.0 — added "layout" + "structuredSet" to the enum so
-            // the Stage picker can include/exclude them. Pre-v0.9.0 the
-            // helper accepted "structuredSet" via the underlying SQL
-            // query but the input schema rejected it — minor gap closed
-            // alongside the branched-create work.
+            // v0.9.0 — "layout" joined the enum so the Stage picker can
+            // include/exclude it.
             kind: z.enum([
               "module",
               "template",
@@ -567,7 +498,10 @@ export const chatPublishInput = z
               "pageLayout",
               "pageModuleContent",
               "layout",
-              "structuredSet",
+              // Epic #1107 — shared lists, list types, page types.
+              "listType",
+              "list",
+              "pageType",
               // v0.11.0 (#45) — themes primitive joins the publish/Stage
               // surface so chat-branched theme edits replay into live
               // on chat.publish.
@@ -647,9 +581,11 @@ export const createTemplateToolInput = z
   .object({
     slug: slugInputSchema,
     displayName: z.string().min(1).max(256),
-    html: sourceTextArg(z.string().min(1).max(2_000_000)),
-    css: sourceTextArg(z.string().max(2_000_000)).default(""),
+    html: sourceTextArg(z.string().min(1).max(2_000_000), "html"),
+    css: sourceTextArg(z.string().max(2_000_000), "css").default(""),
     layoutId: z.string().uuid().optional(),
+    /** Epic #1107 — the page type (slug) this template serves; omit for the site default. */
+    pageType: pageTypeSlugSchema.optional(),
     // Optional block-set metadata, symmetric with create_layout. Omit to
     // auto-derive blocks from <caelo-slot> tags; pass to set displayName /
     // position (each name must match a slot). Enforced op-side in
@@ -682,25 +618,9 @@ export const removeModuleFromToolInput = z
   .strict();
 export type RemoveModuleFromToolInput = z.infer<typeof removeModuleFromToolInput>;
 
-export const setStructuredSetToolInput = z
-  .object({
-    // structuredSetKind (the 6th kind that was previously unreachable
-    // from any AI tool — the kind-specific wrappers covered only
-    // nav-menu + theme).
-    kind: z.enum(["nav-menu", "taxonomy", "tags", "link-list"]),
-    slug: slugInputSchema,
-    displayName: z.string().min(1).max(200),
-    items: z.array(z.unknown()),
-  })
-  .strict();
-
-// v0.10.22 — `updateThemeToolInput` removed. The kind-specific `update_theme`
-// wrapper was replaced by the unified `set_structured_set` surface.
-// v0.11.0 (#45) — theme moved out of structured_sets entirely into its
-// own `themes` primitive (DTCG-shaped jsonb). Theme edits are now
-// `set_theme_tokens` (loose-name patch) / `propose_create_theme`
-// (gated mint) / `propose_activate_theme` (gated flip) — see
-// packages/admin-core/src/ai/tools/{update-theme-tokens,get-theme,…}.ts.
+// v0.11.0 (#45) — theme edits are `set_theme_tokens` (loose-name patch) /
+// `propose_create_theme` (gated mint) / `propose_activate_theme` (gated
+// flip) — see packages/admin-core/src/ai/tools/{update-theme-tokens,get-theme,…}.ts.
 
 export const setTemplateLayoutToolInput = z
   .object({
@@ -713,8 +633,8 @@ export const createLayoutToolInput = z
   .object({
     slug: slugInputSchema,
     displayName: z.string().min(1).max(200),
-    html: sourceTextArg(z.string().min(1).max(50_000)),
-    css: sourceTextArg(z.string().max(50_000)).optional(),
+    html: sourceTextArg(z.string().min(1).max(50_000), "html"),
+    css: sourceTextArg(z.string().max(50_000), "css").optional(),
     blocks: z
       .array(
         z
@@ -750,8 +670,9 @@ export const duplicatePageToolInput = z
   .object({
     sourcePageId: z.string().uuid(),
     newSlug: slugInputSchema,
-    newName: z.string().min(1).max(256).optional(),
     newTitle: z.string().min(1).max(256).optional(),
+    /** #1142 — the copy's breadcrumb label; omitted = newTitle when given, else the source's. */
+    newShortTitle: pageShortTitleSchema.optional(),
     targetTemplateId: z.string().uuid().optional(),
     /** Where the copy sits: a parent page id, or null for the top level.
      *  Omitted = next to the source (same parent). */
@@ -779,32 +700,40 @@ export const repointPageTemplateToolInput = z
   })
   .strict();
 
-export const moveModuleToolInput = z
-  .object({
-    pageId: z.string().uuid(),
-    moduleId: z.string().uuid(),
-    toBlockName: z.string().min(1).max(80),
-    /** "top" | "bottom" | a 0-based index inside the destination block. */
-    position: positionInputSchema,
-  })
-  .strict();
+/**
+ * `move_module` position: `top` | `bottom` | a 0-based index in the target
+ * block, plus `up` / `down` (one slot within the module's CURRENT block).
+ * Same quote-stripping + numeric-string coercion as `positionInputSchema`.
+ */
+export const movePositionInputSchema = z.preprocess(
+  stripSurroundingQuotes,
+  z.union([
+    z.enum(["top", "bottom", "up", "down"]),
+    z.preprocess(
+      (v) => (typeof v === "string" && /^\d+$/.test(v) ? Number.parseInt(v, 10) : v),
+      z.number().int().min(0).max(1000),
+    ),
+  ]),
+);
 
-export const reorderModuleToolInput = z
+/**
+ * `move_module` — moves a placement to `toBlockName` (omitted = the block it
+ * is in now, i.e. a reorder) at `position`. `up` / `down` only make sense
+ * inside the current block; the handler refuses them with another block.
+ */
+export const moveModuleToolInput = z
   .object({
     pageId: z.string().uuid(),
     moduleId: z.string().uuid(),
-    /**
-     * `up` / `down` shift one slot. An integer is an absolute 0-based
-     * target position within the same block.
-     */
-    direction: z.union([z.enum(["up", "down"]), z.number().int().min(0).max(1000)]),
+    toBlockName: z.string().min(1).max(80).optional(),
+    position: movePositionInputSchema,
   })
   .strict();
 
 /**
  * P7 — `find_media`. Searches the media library by alt-text /
  * filename / mime. Each row carries the media UUID (what
- * `set_theme_asset` / `set_media_alt` require) plus a pre-resolved
+ * `set_theme_asset` / `set_media_alt_many` require) plus a pre-resolved
  * variant URL. The AI's only media-inventory surface — the `## Media`
  * system-prompt block was removed by the context diet (#300), so every
  * asset lookup ("an image of a sunlit office") goes through this tool.
@@ -849,7 +778,7 @@ export const regenerateMediaVariantsToolInput = z
 export type RegenerateMediaVariantsToolInput = z.infer<typeof regenerateMediaVariantsToolInput>;
 
 /**
- * P7 — `set_media_alt`. AI may improve a11y on an existing asset
+ * P7 — one item of `set_media_alt_many`. AI may improve a11y on an existing asset
  * without a human round-trip (e.g. when the editor uploaded an image
  * with the default alt and the AI reads its content). Updates only
  * the alt field on `media_assets`.
@@ -879,7 +808,7 @@ export const setNotFoundPageToolInput = z.object({ pageId: z.string().uuid().nul
 export type SetNotFoundPageToolInput = z.infer<typeof setNotFoundPageToolInput>;
 
 /**
- * 0181 — `set_media_source`. AI records an asset's provenance (source
+ * 0181 — one item of `set_media_source_many`. AI records an asset's provenance (source
  * kind + origin detail) and licence when known. All fields but the
  * assetId are optional; the op COALESCEs so omitted fields stay
  * unchanged.
@@ -895,7 +824,7 @@ export const setMediaSourceToolInput = z
 export type SetMediaSourceToolInput = z.infer<typeof setMediaSourceToolInput>;
 
 /**
- * P8 — `set_page_seo`. Manual / panel writes to the per-page SEO
+ * P8 — one item of `set_page_seo_many`. Manual / panel writes to the per-page SEO
  * sidecar. AI calls this only on explicit user intent
  * ("set the home meta description to ..."). Doesn't bump fingerprints
  * (autofilled_at / optimized_at).
@@ -916,7 +845,7 @@ export const setPageSeoToolInput = z
 export type SetPageSeoToolInput = z.infer<typeof setPageSeoToolInput>;
 
 /**
- * P8 — `autofill_page_seo` (and, per item, `autofill_page_seo_many`).
+ * P8 — one item of `autofill_page_seo_many`.
  * Fill-once. Refuses when the page's SEO was already autofilled. The AI
  * calls it before a page's first Stage; staging and production builds
  * refuse pages without a meta description (deploy/missing-seo.ts).
@@ -930,22 +859,6 @@ export const autofillPageSeoToolInput = z
   .strict();
 export type AutofillPageSeoToolInput = z.infer<typeof autofillPageSeoToolInput>;
 
-/**
- * P8 — `optimize_page_seo`. Explicit re-optimization. Always allowed.
- * Takes optional context (keyword analysis / intent shifts / branding
- * changes). The AI can call this across N pages in one chat turn —
- * the resulting changes batch into one Publish-pill confirm.
- */
-export const optimizePageSeoToolInput = z
-  .object({
-    pageId: z.string().uuid(),
-    metaDescription: z.string().min(1).max(320),
-    ogImageAssetId: z.string().uuid().nullable().optional(),
-    context: z.string().max(4000).optional(),
-  })
-  .strict();
-export type OptimizePageSeoToolInput = z.infer<typeof optimizePageSeoToolInput>;
-
 /**
  * P8 AI-first review pass — bulk variants. Per CLAUDE.md §11, bulk
  * tools save round-trips when the AI knows it has N changes to make
@@ -965,7 +878,7 @@ export type FindRedirectsToolInput = z.infer<typeof findRedirectsToolInput>;
 
 export const bulkCreateRedirectsToolInput = z
   .object({
-    redirects: z
+    items: z
       .array(
         z
           .object({
@@ -1017,7 +930,7 @@ export type BulkDeleteRedirectsToolInput = z.infer<typeof bulkDeleteRedirectsToo
 
 export const bulkOptimizeSeoToolInput = z
   .object({
-    updates: z
+    items: z
       .array(
         z
           .object({
@@ -1034,10 +947,8 @@ export const bulkOptimizeSeoToolInput = z
   .strict();
 export type BulkOptimizeSeoToolInput = z.infer<typeof bulkOptimizeSeoToolInput>;
 
-export type EditModuleToolInput = z.infer<typeof editModuleToolInput>;
 export type SiteMemoryProposeToolInput = z.infer<typeof siteMemoryProposeToolInput>;
 export type CreateTemplateToolInput = z.infer<typeof createTemplateToolInput>;
-export type SetStructuredSetToolInput = z.infer<typeof setStructuredSetToolInput>;
 // v0.10.22 — `UpdateThemeToolInput` removed alongside `update_theme` tool.
 export type ChatCreateSessionInput = z.infer<typeof chatCreateSessionInput>;
 export type ChatSendMessageInput = z.infer<typeof chatSendMessageInput>;
@@ -1053,7 +964,6 @@ export type SetSiteDefaultsToolInput = z.infer<typeof setSiteDefaultsToolInput>;
 export type DuplicatePageToolInput = z.infer<typeof duplicatePageToolInput>;
 export type RepointPageTemplateToolInput = z.infer<typeof repointPageTemplateToolInput>;
 export type MoveModuleToolInput = z.infer<typeof moveModuleToolInput>;
-export type ReorderModuleToolInput = z.infer<typeof reorderModuleToolInput>;
 // v0.10.22 — `SetNavMenuToolInput` removed alongside `set_nav_menu` tool.
 
 /**

package/src/bug-intake.test.ts

@@ -32,7 +32,7 @@ describe("computeBugFingerprint", () => {
   });
 
   it("keeps different tools and different quoted field names apart", async () => {
-    expect(await auto("boom", "build_page")).not.toBe(await auto("boom", "edit_module"));
+    expect(await auto("boom", "build_page")).not.toBe(await auto("boom", "update_modules_many"));
     expect(await auto('Invalid input at "page.slug"')).not.toBe(
       await auto('Invalid input at "page.title"'),
     );

package/src/build-page.test.ts

@@ -39,6 +39,14 @@ describe("buildPageInputSchema — page target modes", () => {
     expect(r.success).toBe(true);
   });
 
+  it("accepts parentPageId: null as top level, like pages.create (PR #1121 e2e)", () => {
+    const r = buildPageInputSchema.safeParse({
+      page: { slug: "pricing", title: "Pricing", parentPageId: null },
+      modules: [mintModule],
+    });
+    expect(r.success).toBe(true);
+  });
+
   it("accepts existing mode (pageId only)", () => {
     const r = buildPageInputSchema.safeParse({
       page: { pageId: UUID },
@@ -140,10 +148,10 @@ describe("buildPageInputSchema — module entry modes name the failing index", (
         {
           blockName: "content",
           displayName: "Nav",
-          html: "<nav>{{#nav_links}}<a href='{{href}}'>{{label}}</a>{{/nav_links}}</nav>",
+          html: "<nav>{{#nav_links}}<a href='{{link}}'>{{label}}</a>{{/nav_links}}</nav>",
           fields: [
-            { name: "nav_links", kind: "link-list", label: "Nav links" },
-            { name: "tags", kind: "text-list", label: "Tags", min: 0, max: 12 },
+            { name: "nav_links", kind: "list", listType: "links", label: "Nav links" },
+            { name: "tags", kind: "list", listType: "texts", label: "Tags", min: 0, max: 12 },
           ],
         },
       ],
@@ -190,6 +198,29 @@ describe("buildPageInputSchema — module entry modes name the failing index", (
   });
 });
 
+describe("buildPageInputSchema — ref handles", () => {
+  // e2e-livedit run 40336739359 (PR #1143): the AI named three cards
+  // `cardA` / `cardB` / `cardC` and the lowercase-only rule refused the
+  // whole call for a handle that never leaves it.
+  it("accepts mixed-case handles and refuses ones that do not start with a letter", () => {
+    const { blockName: _b, ...detached } = mintModule;
+    for (const ref of ["cardA", "card_b", "feat-1"]) {
+      const r = buildPageInputSchema.safeParse({
+        page: { pageId: UUID },
+        modules: [{ ...detached, ref }],
+      });
+      expect(r.success).toBe(true);
+    }
+    for (const ref of ["1card", "$card", "card a"]) {
+      const r = buildPageInputSchema.safeParse({
+        page: { pageId: UUID },
+        modules: [{ ...detached, ref }],
+      });
+      expect(r.success).toBe(false);
+    }
+  });
+});
+
 describe("buildPageContentSchema — the three sources", () => {
   it("inline requires values", () => {
     expect(buildPageContentSchema.safeParse({ source: "inline" }).success).toBe(false);

package/src/build-page.ts

@@ -28,6 +28,7 @@ import {
   MODULE_JS_MAX,
   moduleFieldSchema,
   moduleKindSchema,
+  pageCreateSchema,
   pageStatusSchema,
   slugSchema,
   syncModeSchema,
@@ -42,10 +43,10 @@ const contentValuesSchema = z.record(z.string(), z.unknown());
  * existing content paths so build_page adds NO new semantics, only batching:
  *
  *   - `inline`   → mint a private (unsynced) content_instance carrying
- *     `values`, exactly what `set_page_module_content` produces for a
+ *     `values`, exactly what `set_page_module_content_many` produces for a
  *     fresh placement.
  *   - `shared`   → mint a reusable content_instance (purpose required —
- *     same decision-support contract as `create_content_instance`) and
+ *     same decision-support contract as `create_content_instances`) and
  *     bind this placement to it, `synced` by default.
  *   - `existing` → bind an already-minted content_instance (reuse-first
  *     per CLAUDE.md §1A) — the batched form of `set_placement_content`.
@@ -93,7 +94,7 @@ export type BuildPageContent = z.infer<typeof buildPageContentSchema>;
  *     extra authoring keys are TOLERATED (§1A/§11 — a placement that can
  *     succeed must not fail over an extra field) but NOT applied; the op
  *     surfaces an info naming any carried field whose value differs from the
- *     module's stored one, pointing at edit_module (§2 — no silent drop).
+ *     module's stored one, pointing at update_modules_many (§2 — no silent drop).
  *
  * The element-level superRefine reports issues WITH the element's array
  * index in the Zod path, so a mode error fails as `modules[3]: …`.
@@ -130,7 +131,10 @@ export const buildPageModuleSchema = z
      */
     ref: z
       .string()
-      .regex(/^[a-z][a-z0-9_-]{0,31}$/, "ref must be a short lowercase handle")
+      .regex(
+        /^[A-Za-z][A-Za-z0-9_-]{0,31}$/,
+        "ref must be a short handle: a letter, then letters, digits, _ or -",
+      )
       .optional(),
     /**
      * Place mode: an existing module from `list_modules` (UUID), or a
@@ -155,9 +159,9 @@ export const buildPageModuleSchema = z
     description: z.string().max(1000).optional(),
     kind: moduleKindSchema.optional(),
     type: slugSchema.optional(),
-    html: sourceTextArg(z.string().min(1).max(MODULE_HTML_MAX)).optional(),
-    css: sourceTextArg(z.string().max(MODULE_CSS_MAX)).optional(),
-    js: sourceTextArg(z.string().max(MODULE_JS_MAX)).optional(),
+    html: sourceTextArg(z.string().min(1).max(MODULE_HTML_MAX), "html").optional(),
+    css: sourceTextArg(z.string().max(MODULE_CSS_MAX), "css").optional(),
+    js: sourceTextArg(z.string().max(MODULE_JS_MAX), "js").optional(),
     fields: z.array(moduleFieldSchema).max(64).optional(),
     /** issue #164 slice 2 — opt-in mechanical token binding (tool layer). */
     bindThemeLiterals: z.boolean().optional(),
@@ -188,7 +192,7 @@ export const buildPageModuleSchema = z
     // (build_page places; it does not re-author a shared module). To avoid a
     // SILENT dropped change (§2), the HANDLER surfaces an INFO in the result
     // naming any ignored authoring field whose value DIFFERS from the module's
-    // stored one, and points at edit_module. The schema can't do that check (no
+    // stored one, and points at update_modules_many. The schema can't do that check (no
     // DB), so it just allows the entry through.
     if (entry.moduleId !== undefined) return;
     if (entry.displayName === undefined || entry.html === undefined) {
@@ -216,7 +220,7 @@ export const buildPageTargetSchema = z
     pageId: z.string().uuid().optional(),
     slug: slugSchema.optional(),
     title: z.string().min(1).max(256).optional(),
-    name: z.string().min(1).max(256).optional(),
+    shortTitle: pageCreateSchema.shape.shortTitle,
     templateId: z.string().uuid().optional(),
     status: pageStatusSchema.optional(),
     /**
@@ -224,7 +228,16 @@ export const buildPageTargetSchema = z
      * "<parent's path>/<slug>". Create mode only (an existing page moves
      * with `pages.move_many`).
      */
-    parentPageId: z.string().uuid().optional(),
+    // null = top level, same as pages.create (sibling tools must agree — #106 class).
+    parentPageId: z.string().uuid().nullable().optional(),
+    /**
+     * Epic #1107 — what the new page IS (page type slug), its properties
+     * and terms; same semantics as `pages.create` (create mode only — an
+     * existing page changes type with `pages.update`).
+     */
+    pageType: pageCreateSchema.shape.pageType,
+    properties: pageCreateSchema.shape.properties,
+    terms: pageCreateSchema.shape.terms,
     /**
      * Migration linkage (issue #278 flow): the `import_pages` row this
      * page rebuilds. The op stamps `import_pages.accepted_page_id` with
@@ -252,7 +265,17 @@ export const buildPageTargetSchema = z
   .strict()
   .superRefine((page, ctx) => {
     const createKeys = (
-      ["slug", "title", "name", "templateId", "status", "parentPageId"] as const
+      [
+        "slug",
+        "title",
+        "shortTitle",
+        "templateId",
+        "status",
+        "parentPageId",
+        "pageType",
+        "properties",
+        "terms",
+      ] as const
     ).filter((k) => page[k] !== undefined);
     if (page.pageId !== undefined) {
       if (createKeys.length > 0) {