drupal-mcp-connector 2.14.2 → 2.15.1

package/src/lib/backends/jsonapi.js

@@ -17,20 +17,37 @@ import {
   normalizeRelationship,
   BASE_ATTRIBUTE_FIELDS,
 } from "../canonical.js";
+import { isPositiveNid, normalizeAlias, PATH_ALIAS_ENTITY_TYPE } from "../path-alias.js";
 
 // Drupal exposes internal identifiers under drupal_internal__* attributes.
 // They are dropped from canonical `fields` except for the identifiers that
 // governed read/write workflows explicitly need.
 const INTERNAL_ATTR_RE = /^drupal_internal__/;
 
-// countEntities() pagination. Drupal core JSON:API returns no total in `meta`,
-// so an exact count is obtained by walking pages until `links.next` is gone.
-// COUNT_PAGE_SIZE is Drupal's default max page size; COUNT_MAX_RECORDS bounds
-// the walk so a huge collection can't issue unbounded requests (mirrors the
-// GraphQL backend's MAX_CLIENT_RECORDS) — past it the count is approximate.
+// countEntities() / listEntities() pagination. Drupal core JSON:API returns no
+// total in `meta` and silently caps `page[limit]` at OffsetPage::SIZE_MAX
+// (50 by default). COUNT_PAGE_SIZE matches that default; COUNT_MAX_RECORDS
+// bounds a walk so a huge collection can't issue unbounded requests (mirrors
+// the GraphQL backend's MAX_CLIENT_RECORDS) — past it the count is approximate.
 const COUNT_PAGE_SIZE = 50;
 const COUNT_MAX_RECORDS = 1000;
 
+/**
+ * Whether a JSON:API collection document advertises another page.
+ * `links.next` may be a string href or a `{ href }` link object.
+ * @param {?object} data JSON:API document.
+ * @returns {boolean}
+ */
+function jsonApiHasNext(data) {
+  const next = data?.links?.next;
+  if (next === undefined || next === null || next === false) return false;
+  if (typeof next === "string") return next.length > 0;
+  if (typeof next === "object" && next.href !== undefined && next.href !== null) {
+    return String(next.href).length > 0;
+  }
+  return Boolean(next);
+}
+
 /**
  * Detect the JSON:API error Drupal returns when a write attempts to set the
  * `status` (published) field on a content_moderation-governed entity. Such
@@ -267,24 +284,72 @@ export class JsonApiBackend extends Backend {
   }
 
   /**
-   * List entities for a descriptor. Server-side filter/sort/paging means the
-   * result is always exact (`approximate`/`truncated` are false).
+   * List entities for a descriptor.
+   *
+   * Drupal core JSON:API has no `meta.count` (jsonapi_extras can add it) and
+   * silently caps `page[limit]` at OffsetPage::SIZE_MAX (50 by default). When
+   * the caller asks for more rows than one Drupal page returns and
+   * `links.next` is present, this method follows that link until the requested
+   * window is filled, the collection ends, or COUNT_MAX_RECORDS is hit.
+   * `page.total` is exact when `meta.count` is present or this window reached
+   * the end; otherwise it is the number of rows seen so far and `approximate`
+   * is true. Never report a single page's length as an exact collection total.
+   *
    * @param {import("../canonical.js").QueryDescriptor} descriptor
    * @returns {Promise<import("./backend-interface.js").ListResult>}
    */
   async listEntities(descriptor) {
-    const params = this.compileQuery(descriptor);
-    const qs = params.toString();
-    const base = this.resourcePath(descriptor.entityType, descriptor.bundle);
-    const path = qs ? `${base}?${qs}` : base;
-    const data = await drupalFetch(this.site, path);
-    const entities = (data.data || []).map((r) => this.toCanonical(r));
-    const total = data.meta?.count ?? entities.length;
+    const requestedLimit = descriptor.page?.limit;
+    const startOffset = descriptor.page?.offset ?? 0;
+    const fillTo = typeof requestedLimit === "number"
+      ? Math.min(Math.max(0, requestedLimit), COUNT_MAX_RECORDS)
+      : null;
+
+    const entities = [];
+    let offset = startOffset;
+    let hasNext = false;
+    let metaCount = null;
+
+    for (;;) {
+      const remaining = fillTo === null ? requestedLimit : fillTo - entities.length;
+      const page = {
+        ...descriptor.page,
+        offset,
+        ...(typeof remaining === "number" ? { limit: remaining } : {}),
+      };
+      const params = this.compileQuery({ ...descriptor, page });
+      const qs = params.toString();
+      const base = this.resourcePath(descriptor.entityType, descriptor.bundle);
+      const path = qs ? `${base}?${qs}` : base;
+      const data = await drupalFetch(this.site, path);
+      if (metaCount === null && typeof data?.meta?.count === "number") {
+        metaCount = data.meta.count;
+      }
+      const pageEntities = (data.data || []).map((r) => this.toCanonical(r));
+      entities.push(...pageEntities);
+      hasNext = jsonApiHasNext(data);
+
+      if (fillTo === null) break;
+      if (!hasNext || pageEntities.length === 0) break;
+      if (entities.length >= fillTo) break;
+      offset += pageEntities.length;
+    }
+
+    const truncated = fillTo !== null
+      && typeof requestedLimit === "number"
+      && entities.length < requestedLimit
+      && hasNext;
+    const seen = startOffset + entities.length;
+    const exact = typeof metaCount === "number" || !hasNext;
     return {
       entities,
-      page: { total, hasNext: Boolean(data.links?.next), cursor: null },
-      approximate: false,
-      truncated: false,
+      page: {
+        total: typeof metaCount === "number" ? metaCount : seen,
+        hasNext,
+        cursor: null,
+      },
+      approximate: !exact,
+      truncated,
     };
   }
 
@@ -310,19 +375,88 @@ export class JsonApiBackend extends Backend {
    * (Drupal `PathItem::postSave` creates a duplicate alias when `pid` is absent)
    * — so this method exposes it. Returns nulls for entities/backends without a
    * path field. See the 1.5.1 alias fix.
-   * @param {{entityType: string, bundle: string, id: string}} ref
-   * @returns {Promise<{alias: ?string, pid: ?(number|string), langcode: ?string, drupalId: ?(number|string)}>}
+   *
+   * Unpublished default / forward revisions often omit `pid` on the computed
+   * `path` field even when a `path_alias` row exists. When the node numeric id
+   * is known, this method also looks up that row (aliases are not revisioned)
+   * so title-only edits can pin the live alias (#274).
+   * @param {{entityType: string, bundle: string, id: string, resourceVersion?: string}} ref
+   * @returns {Promise<{alias: ?string, pid: ?(number|string), langcode: ?string, drupalId: ?(number|string), aliasId: ?string}>}
    */
-  async getPathInfo({ entityType, bundle, id }) {
+  async getPathInfo({ entityType, bundle, id, resourceVersion }) {
     validateUuid(id);
-    const data = await drupalFetch(this.site, `${this.resourcePath(entityType, bundle)}/${encodeURIComponent(id)}`);
+    let path = `${this.resourcePath(entityType, bundle)}/${encodeURIComponent(id)}`;
+    if (resourceVersion) {
+      path += `?resourceVersion=${encodeURIComponent(resourceVersion)}`;
+    }
+    const data = await drupalFetch(this.site, path);
     const attrs = data?.data?.attributes ?? {};
-    const path = attrs.path ?? null;
+    const nodePath = attrs.path ?? null;
+    const drupalId = attrs.drupal_internal__nid ?? attrs.drupal_internal__id ?? null;
+    const langcode = nodePath?.langcode ?? attrs.langcode ?? null;
+    let alias = nodePath?.alias ?? null;
+    let pid = nodePath?.pid ?? null;
+    let aliasId = null;
+
+    if (entityType === "node" && isPositiveNid(drupalId)) {
+      const row = await this.lookupPathAliasRow(`/node/${Number(drupalId)}`, {
+        langcode,
+        preferredAlias: alias,
+      });
+      if (row) {
+        aliasId = row.id;
+        if (pid === undefined || pid === null) pid = row.pid;
+        // Pathauto / unpublished computed fields can omit alias; the row is
+        // the router-visible value.
+        if (!alias) alias = row.alias;
+      }
+    }
+
+    return { alias, pid, langcode, drupalId, aliasId };
+  }
+
+  /**
+   * Load the path_alias row for a node source path. Best-effort: missing
+   * JSON:API exposure or an empty collection returns null.
+   * @param {string} sourcePath Drupal system path, e.g. `/node/44`.
+   * @param {{langcode?: ?string, preferredAlias?: ?string}} [opts]
+   * @returns {Promise<?{id: string, alias: ?string, pid: ?(number|string), langcode: ?string}>}
+   */
+  async lookupPathAliasRow(sourcePath, opts = {}) {
+    if (!/^\/node\/[1-9]\d*$/.test(sourcePath)) return null;
+    const params = new URLSearchParams();
+    params.set("filter[path]", sourcePath);
+    if (opts.langcode) params.set("filter[langcode]", String(opts.langcode));
+    let data;
+    try {
+      data = await drupalFetch(
+        this.site,
+        `${this.resourcePath(PATH_ALIAS_ENTITY_TYPE, PATH_ALIAS_ENTITY_TYPE)}?${params}`,
+      );
+    } catch {
+      return null;
+    }
+    const rows = Array.isArray(data?.data) ? data.data : [];
+    if (!rows.length) return null;
+    const preferred = normalizeAlias(opts.preferredAlias);
+    let picked = rows[0];
+    if (preferred) {
+      for (const row of rows) {
+        const attrs = row && typeof row === "object" ? row.attributes : null;
+        const rowAlias = attrs && typeof attrs === "object" ? attrs.alias : null;
+        if (normalizeAlias(rowAlias) === preferred) {
+          picked = row;
+          break;
+        }
+      }
+    }
+    if (!picked?.id) return null;
+    const a = picked.attributes && typeof picked.attributes === "object" ? picked.attributes : {};
     return {
-      alias: path?.alias ?? null,
-      pid: path?.pid ?? null,
-      langcode: path?.langcode ?? attrs.langcode ?? null,
-      drupalId: attrs.drupal_internal__nid ?? attrs.drupal_internal__id ?? null,
+      id: picked.id,
+      alias: a.alias ?? null,
+      pid: a.drupal_internal__id ?? a.pid ?? null,
+      langcode: a.langcode ?? null,
     };
   }
 

package/src/lib/draft-write.js

@@ -4,6 +4,8 @@
  * Translation create/update uses the same surface with X-MCP-Draft-Langcode.
  */
 
+import { entityRevisionId } from "./write-revision.js";
+
 const LANGCODE_RE = /^[a-z][a-z0-9_-]{0,11}$/;
 const MISSING_DRAFT_ENDPOINT =
   "The site does not provide Sentinel's governed draft endpoint (d.o #3621022). " +
@@ -56,9 +58,9 @@ function requireWorkingPair(draftRevision) {
  * @param {string} id
  * @returns {string}
  */
-function nodeResource(backend, entityType, bundle, id) {
-  if (entityType !== "node") {
-    throw new Error("Governed draft translation is implemented for nodes.");
+function draftResource(backend, entityType, bundle, id) {
+  if (entityType !== "node" && entityType !== "paragraph") {
+    throw new Error("Governed draft translation is implemented for nodes and paragraphs.");
   }
   if (typeof backend.rawQuery !== "function" || typeof backend.resourcePath !== "function") {
     throw new Error("This backend does not support governed draft continuation.");
@@ -66,6 +68,18 @@ function nodeResource(backend, entityType, bundle, id) {
   return `${backend.resourcePath(entityType, bundle)}/${encodeURIComponent(id)}`;
 }
 
+/**
+ * @param {object} [draftRevision]
+ * @returns {string}
+ */
+function requireParagraphRevisionId(draftRevision) {
+  const revisionId = String(draftRevision?.revisionId ?? draftRevision?.workingVid ?? "");
+  if (!/^[1-9]\d*$/.test(revisionId)) {
+    throw new Error("Paragraph translation requires a verified paragraph revision ID.");
+  }
+  return revisionId;
+}
+
 /**
  * Validate or continue a draft, using the same payload and revision precondition.
  * No canonical fallback: an absent endpoint or refused precondition stops work.
@@ -76,8 +90,11 @@ function nodeResource(backend, entityType, bundle, id) {
  */
 export async function writeDraft(backend, input, preflight = false) {
   const { entityType, bundle, id, attributes = {}, relationships, draftRevision, langcode } = input;
+  if (entityType === "paragraph") {
+    return writeParagraphDraft(backend, input, preflight);
+  }
   const { live, working } = requireWorkingPair(draftRevision);
-  const base = nodeResource(backend, entityType, bundle, id);
+  const base = draftResource(backend, entityType, bundle, id);
   const data = { type: `${entityType}--${bundle}`, id, attributes };
   if (relationships) data.relationships = relationships;
   const headers = {
@@ -123,6 +140,11 @@ export async function writeDraft(backend, input, preflight = false) {
 export async function createTranslationDraft(backend, input, preflight = false) {
   const { entityType, bundle, id, attributes = {}, relationships, draftRevision } = input;
   const langcode = assertDraftLangcode(input.langcode);
+  if (entityType === "paragraph") {
+    return createParagraphTranslationDraft(backend, {
+      entityType, bundle, id, attributes, relationships, draftRevision, langcode,
+    }, preflight);
+  }
   const live = String(draftRevision?.liveVid ?? "");
   const workingRaw = draftRevision?.workingVid;
   const working = workingRaw === undefined || workingRaw === null || workingRaw === ""
@@ -134,7 +156,7 @@ export async function createTranslationDraft(backend, input, preflight = false)
   if (working && (!/^[1-9]\d*$/.test(working) || working === live)) {
     throw new Error("Translation create requires distinct live and working revision IDs when a working copy exists.");
   }
-  const base = nodeResource(backend, entityType, bundle, id);
+  const base = draftResource(backend, entityType, bundle, id);
   const safeAttributes = { ...attributes };
   delete safeAttributes.langcode;
   const data = { type: `${entityType}--${bundle}`, id, attributes: safeAttributes };
@@ -155,7 +177,7 @@ export async function createTranslationDraft(backend, input, preflight = false)
       },
     });
   } catch (error) {
-    throw missingEndpointError(error, MISSING_TRANSLATION_ENDPOINT);
+    throw rewriteTranslationWorkingRevisionError(missingEndpointError(error, MISSING_TRANSLATION_ENDPOINT));
   }
   if (preflight) {
     if (result?.meta?.draft_preflight !== true || String(result.meta.live) !== live) {
@@ -169,16 +191,77 @@ export async function createTranslationDraft(backend, input, preflight = false)
   return backend.toCanonical(result.data);
 }
 
+/**
+ * Live + working revision ids for a node translation create.
+ * Prefers Sentinel's mcp-translations inventory over JSON:API
+ * `rel:working-copy`, which 403s on some unpublished drafts even when a
+ * working revision exists (#282).
+ * @param {object} backend
+ * @param {{entityType: string, bundle: string, id: string, existing?: ?object}} ref
+ * @returns {Promise<{liveVid: ?(number|string), workingVid: ?(number|string)}>}
+ */
+export async function resolveNodeTranslationPair(backend, { entityType, bundle, id, existing }) {
+  const fromEntity = existing ? entityRevisionId(existing) : null;
+  try {
+    const meta = await readTranslationInventory(backend, { entityType, bundle, id });
+    const liveVid = meta.live?.vid ?? fromEntity;
+    const workingVid = meta.working?.vid;
+    const distinct = workingVid !== undefined && workingVid !== null && workingVid !== ""
+      && liveVid !== undefined && liveVid !== null
+      && String(workingVid) !== String(liveVid);
+    return { liveVid, workingVid: distinct ? workingVid : undefined };
+  } catch {
+    if (typeof backend.getEntity !== "function") {
+      return { liveVid: fromEntity, workingVid: undefined };
+    }
+    let workingCopy = null;
+    try {
+      workingCopy = await backend.getEntity({
+        entityType, bundle, id, resourceVersion: "rel:working-copy",
+      });
+    } catch {
+      workingCopy = null;
+    }
+    const workingVid = workingCopy ? entityRevisionId(workingCopy) : null;
+    const distinct = workingVid !== null && fromEntity !== null && String(workingVid) !== String(fromEntity);
+    return { liveVid: fromEntity, workingVid: distinct ? workingVid : undefined };
+  }
+}
+
+/**
+ * Rewrite Sentinel's live-only 409 into an actionable connector error.
+ * @param {unknown} error
+ * @returns {Error}
+ */
+export function rewriteTranslationWorkingRevisionError(error) {
+  if (/A working revision exists\. Reload and send both revision IDs/i.test(String(error?.message || ""))) {
+    return new Error(
+      "Translation create sent only the live revision, but a working draft exists. " +
+      "The connector should have sent both live and working revision IDs (If-Match). " +
+      "Reload with drupal_list_translations and retry. See connector #282.",
+      { cause: error instanceof Error ? error : undefined },
+    );
+  }
+  return error instanceof Error ? error : new Error(String(error));
+}
+
 /**
  * Read live/working translation inventory from Sentinel.
  * @param {object} backend
- * @param {{entityType: string, bundle: string, id: string}} ref
+ * @param {{entityType: string, bundle: string, id: string, revisionId?: string|number}} ref
  * @returns {Promise<object>}
  */
-export async function readTranslationInventory(backend, { entityType, bundle, id }) {
-  const base = nodeResource(backend, entityType, bundle, id);
+export async function readTranslationInventory(backend, { entityType, bundle, id, revisionId }) {
+  const base = draftResource(backend, entityType, bundle, id);
+  const headers = {};
+  if (revisionId !== undefined && revisionId !== null && revisionId !== "") {
+    headers["If-Match"] = `"${requireParagraphRevisionId({ revisionId })}"`;
+  }
   try {
-    const result = await backend.rawQuery({ path: `${base}/mcp-translations` });
+    const result = await backend.rawQuery({
+      path: `${base}/mcp-translations`,
+      options: Object.keys(headers).length ? { method: "GET", headers } : undefined,
+    });
     if (!result?.meta?.live) {
       throw new Error("The site did not return a translation inventory.");
     }
@@ -197,8 +280,11 @@ export async function readTranslationInventory(backend, { entityType, bundle, id
 export async function readDraftTranslation(backend, input) {
   const { entityType, bundle, id, draftRevision } = input;
   const langcode = assertDraftLangcode(input.langcode);
+  if (entityType === "paragraph") {
+    return readParagraphDraftTranslation(backend, input);
+  }
   const { live, working } = requireWorkingPair(draftRevision);
-  const base = nodeResource(backend, entityType, bundle, id);
+  const base = draftResource(backend, entityType, bundle, id);
   let result;
   try {
     result = await backend.rawQuery({
@@ -219,3 +305,126 @@ export async function readDraftTranslation(backend, input) {
   }
   return backend.toCanonical(result.data);
 }
+
+/**
+ * Continue an unpublished paragraph translation on a pinned revision.
+ * @param {object} backend
+ * @param {object} input
+ * @param {boolean} [preflight]
+ * @returns {Promise<object>}
+ */
+async function writeParagraphDraft(backend, input, preflight = false) {
+  const { entityType, bundle, id, attributes = {}, relationships, draftRevision } = input;
+  const langcode = assertDraftLangcode(input.langcode);
+  const revisionId = requireParagraphRevisionId(draftRevision);
+  const draftState = input.draftState;
+  if (typeof draftState !== "string" || !/^[a-f0-9]{64}$/.test(draftState)) {
+    throw new Error("Paragraph translation update requires draftState from the previous draft read. Re-read the draft; do not retry old copy with a refreshed token.");
+  }
+  const base = draftResource(backend, entityType, bundle, id);
+  const data = { type: `${entityType}--${bundle}`, id, attributes };
+  if (relationships) data.relationships = relationships;
+  const headers = {
+    "If-Match": `"${revisionId}"`,
+    "X-MCP-Draft-Preflight": preflight ? "1" : "0",
+    "X-MCP-Draft-Langcode": langcode,
+    "X-MCP-Draft-State": draftState,
+  };
+  let result;
+  try {
+    result = await backend.rawQuery({
+      path: `${base}/mcp-draft`,
+      options: { method: "PATCH", headers, body: JSON.stringify({ data }) },
+    });
+  } catch (error) {
+    throw missingEndpointError(error, MISSING_DRAFT_ENDPOINT);
+  }
+  if (preflight) {
+    if (result?.meta?.draft_preflight !== true
+      || String(result.meta.live) !== revisionId
+      || (result.meta.langcode && String(result.meta.langcode) !== langcode)) {
+      throw new Error("The site did not confirm a non-saving paragraph translation preflight. Refusing to continue.");
+    }
+    return result;
+  }
+  if (!result?.data || result.data.id !== id || result.data.type !== data.type) {
+    throw new Error("Paragraph translation write did not identify the requested entity. The write outcome is uncertain; re-read before retrying.");
+  }
+  return { ...backend.toCanonical(result.data), draftState: result.meta?.draft_state };
+}
+
+/**
+ * Create an unpublished paragraph translation on a pinned revision.
+ * @param {object} backend
+ * @param {object} input
+ * @param {boolean} [preflight]
+ * @returns {Promise<object>}
+ */
+async function createParagraphTranslationDraft(backend, input, preflight = false) {
+  const { entityType, bundle, id, attributes = {}, relationships, draftRevision, langcode } = input;
+  const revisionId = requireParagraphRevisionId(draftRevision);
+  const base = draftResource(backend, entityType, bundle, id);
+  const safeAttributes = { ...attributes };
+  delete safeAttributes.langcode;
+  const data = { type: `${entityType}--${bundle}`, id, attributes: safeAttributes };
+  if (relationships) data.relationships = relationships;
+  let result;
+  try {
+    result = await backend.rawQuery({
+      path: `${base}/mcp-draft/translations`,
+      options: {
+        method: "POST",
+        headers: {
+          "If-Match": `"${revisionId}"`,
+          "X-MCP-Draft-Preflight": preflight ? "1" : "0",
+          "X-MCP-Draft-Langcode": langcode,
+        },
+        body: JSON.stringify({ data }),
+      },
+    });
+  } catch (error) {
+    throw missingEndpointError(error, MISSING_TRANSLATION_ENDPOINT);
+  }
+  if (preflight) {
+    if (result?.meta?.draft_preflight !== true || String(result.meta.live) !== revisionId) {
+      throw new Error("The site did not confirm a non-saving paragraph translation preflight. Refusing to continue.");
+    }
+    return result;
+  }
+  if (!result?.data || result.data.id !== id || result.data.type !== data.type) {
+    throw new Error("Paragraph translation create did not identify the requested entity. The write outcome is uncertain; re-read before retrying.");
+  }
+  return { ...backend.toCanonical(result.data), draftState: result.meta?.draft_state };
+}
+
+/**
+ * Read one unpublished paragraph translation of a pinned revision.
+ * @param {object} backend
+ * @param {object} input
+ * @returns {Promise<object>}
+ */
+async function readParagraphDraftTranslation(backend, input) {
+  const { entityType, bundle, id, draftRevision } = input;
+  const langcode = assertDraftLangcode(input.langcode);
+  const revisionId = requireParagraphRevisionId(draftRevision);
+  const base = draftResource(backend, entityType, bundle, id);
+  let result;
+  try {
+    result = await backend.rawQuery({
+      path: `${base}/mcp-draft`,
+      options: {
+        method: "GET",
+        headers: {
+          "If-Match": `"${revisionId}"`,
+          "X-MCP-Draft-Langcode": langcode,
+        },
+      },
+    });
+  } catch (error) {
+    throw missingEndpointError(error, MISSING_TRANSLATION_ENDPOINT);
+  }
+  if (!result?.data || result.data.id !== id) {
+    throw new Error("Paragraph draft translation read did not identify the requested entity.");
+  }
+  return { ...backend.toCanonical(result.data), draftState: result.meta?.draft_state };
+}

package/src/lib/entity-response.js

@@ -75,6 +75,31 @@ export function flagUnrequestedStatusChange(result, existing, sentAttributes) {
   };
 }
 
+/** Why computed `metatag` is dropped on unpublished working-translation bodies. */
+export const METATAG_OMITTED_NOTE =
+  "The computed metatag array is omitted: JSON:API resolves it from the live " +
+  "default revision, not the unpublished working translation. Use the stored " +
+  "field (field_metatags / field_metatag) to verify the draft. See connector #283.";
+
+/**
+ * Drop the computed `metatag` array from a working-translation body.
+ * The stored override field is left intact. No-op when `metatag` is absent.
+ * @param {?object} entity Canonical entity.
+ * @returns {?object}
+ */
+export function omitLiveComputedMetatag(entity) {
+  if (!entity || typeof entity !== "object") return entity;
+  const fields = entity.fields && typeof entity.fields === "object" ? entity.fields : null;
+  if (!fields || !Object.prototype.hasOwnProperty.call(fields, "metatag")) return entity;
+  const nextFields = { ...fields };
+  delete nextFields.metatag;
+  return {
+    ...entity,
+    fields: nextFields,
+    _metatagOmitted: { reason: METATAG_OMITTED_NOTE },
+  };
+}
+
 /** JSON Schema fragment for the shared `returning` parameter. */
 export const RETURNING_SCHEMA = {
   type: "string",