@tokenoftrust/storefront-runner 2.4.76 → 2.4.78

package/apps/storefront/src/lib/seo/structuredData.ts

@@ -99,8 +99,15 @@ export function composeStructuredData(input: ComposeInput): Node | null {
   const ref = (id: string | undefined) => (id ? { "@id": id } : undefined);
   const pageUrl = (href: string) => pageUrlForm(absolute(href, canonicalBase), tenant.seo);
 
+  const pathOf = (value: string) => value.replace(/\/+$/, "") || "/";
+  const here = pathOf(new URL(canonical).pathname);
+  const scoped = (token: StructuredDataNodeToken) => {
+    const paths = profile.onlyOn?.[token];
+    return !paths || paths.some((p) => pathOf(p) === here);
+  };
+
   const nodes: Node[] = [];
-  for (const token of tokens) {
+  for (const token of tokens.filter(scoped)) {
     const node = buildNode(token);
     if (node) nodes.push(node);
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tokenoftrust/storefront-runner",
-  "version": "2.4.76",
+  "version": "2.4.78",
   "license": "SEE LICENSE IN LICENSE",
   "description": "World-shareable storefront runner: multi-tenant renderer on Astro/Cloudflare. No control plane.",
   "packageManager": "pnpm@11.9.0",

package/packages/cli/src/declared-config.mjs

@@ -34,7 +34,7 @@ export const SITE_TYPES = ["commerce", "marketing"];
 export const SEO_KEYS = Object.freeze(["trailingSlash", "structuredData"]);
 
 /** Every `seo.structuredData` key. */
-export const STRUCTURED_DATA_KEYS = Object.freeze(["graphs", "names", "reviews"]);
+export const STRUCTURED_DATA_KEYS = Object.freeze(["graphs", "names", "reviews", "onlyOn"]);
 
 /**
  * The page types a structured-data profile declares a graph for. `productGroup` is a product
@@ -135,6 +135,21 @@ export function seoDeclarationProblems(seo) {
   if (data.reviews !== undefined && typeof data.reviews !== "boolean") {
     problems.push("`seo.structuredData.reviews` must be true or false");
   }
+  if (data.onlyOn !== undefined) {
+    if (!isPlainObject(data.onlyOn)) {
+      problems.push("`seo.structuredData.onlyOn` must map a node type to the paths it is emitted on");
+    } else {
+      for (const [token, paths] of Object.entries(/** @type {Record<string, unknown>} */ (data.onlyOn))) {
+        const at = `seo.structuredData.onlyOn.${token}`;
+        if (!STRUCTURED_DATA_NODE_TOKENS.includes(token)) {
+          problems.push(`\`${at}\`: ${JSON.stringify(token)} is not a node type the renderer builds`);
+        } else if (!Array.isArray(paths) || paths.length === 0
+          || !paths.every((p) => typeof p === "string" && p.startsWith("/") && !p.startsWith("//"))) {
+          problems.push(`\`${at}\` must be a non-empty array of root-absolute paths such as "/faq/"`);
+        }
+      }
+    }
+  }
   return problems;
 }
 

package/packages/public-runtime/src/declared-tenant-config.mjs

@@ -78,7 +78,7 @@ export const SITE_TYPES_LIST = Object.freeze(["commerce", "marketing"]);
 export const SEO_KEYS = Object.freeze(["trailingSlash", "structuredData"]);
 
 /** Every `seo.structuredData` key. */
-export const STRUCTURED_DATA_KEYS = Object.freeze(["graphs", "names", "reviews"]);
+export const STRUCTURED_DATA_KEYS = Object.freeze(["graphs", "names", "reviews", "onlyOn"]);
 
 /**
  * The page types a structured-data profile declares a graph for. `productGroup` is a product
@@ -181,6 +181,21 @@ export function seoDeclarationProblems(seo) {
   if (data.reviews !== undefined && typeof data.reviews !== "boolean") {
     problems.push("`seo.structuredData.reviews` must be true or false");
   }
+  if (data.onlyOn !== undefined) {
+    if (!isPlainObject(data.onlyOn)) {
+      problems.push("`seo.structuredData.onlyOn` must map a node type to the paths it is emitted on");
+    } else {
+      for (const [token, paths] of Object.entries(/** @type {Record<string, unknown>} */ (data.onlyOn))) {
+        const at = `seo.structuredData.onlyOn.${token}`;
+        if (!STRUCTURED_DATA_NODE_TOKENS.includes(token)) {
+          problems.push(`\`${at}\`: ${JSON.stringify(token)} is not a node type the renderer builds`);
+        } else if (!Array.isArray(paths) || paths.length === 0
+          || !paths.every((p) => typeof p === "string" && p.startsWith("/") && !p.startsWith("//"))) {
+          problems.push(`\`${at}\` must be a non-empty array of root-absolute paths such as "/faq/"`);
+        }
+      }
+    }
+  }
   return problems;
 }
 

package/packages/public-runtime/src/product-details.ts

@@ -47,9 +47,37 @@ const DEFAULT_SPEC_HEADING = "Product Details";
 const DEFAULT_FAQ_HEADING = "Frequently Asked Questions";
 const DEFAULT_BOX_CONTENTS_LABEL = "What's in the box";
 
+/**
+ * True for a metafield that carries an unresolved platform reference rather
+ * than displayable content — a Shopify metaobject/product/variant/file/
+ * collection/page reference (bare or `list.*`), surfaced either through its
+ * declared `type` or, when type is absent/untrusted, through the `gid://`
+ * shape of its value itself (bare or as a JSON array of them, the `list.*`
+ * on-disk form). No declared section may render one: a raw `gid://...`
+ * identifier is never product content, no matter which section asked for it.
+ */
+function isUnresolvedReference(m: Metafield): boolean {
+  if (m.type && /_reference$/.test(m.type)) return true;
+  const v = m.value?.trim();
+  if (!v) return false;
+  if (v.startsWith("gid://")) return true;
+  if (v.startsWith("[")) {
+    try {
+      const parsed = JSON.parse(v);
+      if (Array.isArray(parsed) && parsed.length > 0 && parsed.every((x) => typeof x === "string" && x.startsWith("gid://"))) {
+        return true;
+      }
+    } catch {
+      /* not JSON — fall through, not a reference by this check */
+    }
+  }
+  return false;
+}
+
 function metafieldLookup(metafields: readonly Metafield[]): (namespace: string, key: string) => string | undefined {
   const map = new Map<string, string>();
   for (const m of metafields) {
+    if (isUnresolvedReference(m)) continue;
     const v = m.value?.trim();
     if (v) map.set(`${m.namespace}.${m.key}`, v);
   }

package/packages/public-runtime/src/tenant.ts

@@ -789,6 +789,11 @@ export interface SeoDeclaration {
     names?: Partial<Record<"Product" | "ProductGroup" | "Article" | "BlogPosting", "documentTitle" | "title">>;
     /** Emit real sourced aggregateRating/review on Product. Absent ⇒ false. */
     reviews?: boolean;
+    /**
+     * Node types emitted only on the listed paths (trailing slash ignored), wherever a graph
+     * lists them — the per-URL presence a source site had (an FAQPage on some products only).
+     */
+    onlyOn?: Partial<Record<StructuredDataNodeToken, string[]>>;
   };
 }