@stacksjs/stx 0.2.342 → 0.2.344

package/dist/seo.js

@@ -1,267 +1,248 @@
+import {
+  describeValue,
+  evaluateDirectiveArgument,
+  isPlainRecord,
+  replaceDirectiveCalls,
+  reportDirectiveFailure,
+  splitTopLevelArgs,
+  warnDirective
+} from "./directive-arguments.js";
 import { ErrorCodes, inlineError } from "./error-handling.js";
 import { mergeHeadConfigs } from "./head.js";
-import { safeEvaluateObject } from "./safe-evaluator.js";
-export function processMetaDirectives(template, context, _filePath, _options) {
-  let output = template;
-  output = output.replace(/@meta\(\s*['"]([^'"]+)['"](?:,\s*['"]([^'"]+)['"]\s*)?\)/g, (_, name, content) => {
-    if (!content && name.includes(":")) {
-      const parts = name.split(":"), property = name, contextKey = parts.length > 1 ? parts[1] : "";
-      if (contextKey && context[contextKey])
-        content = context[contextKey];
-      else if (property.startsWith("og:") && context.openGraph && context.openGraph[parts[1]])
-        content = context.openGraph[parts[1]];
-      else
-        content = "";
-      return content ? `<meta property="${escapeHtml(property)}" content="${escapeHtml(content)}">` : "";
-    }
-    return content ? `<meta name="${escapeHtml(name)}" content="${escapeHtml(content)}">` : "";
-  });
-  const metaTagPat = /@metaTag\s*\(/g;
-  let metaMatch;
-  while ((metaMatch = metaTagPat.exec(output)) !== null) {
-    const mtStart = metaMatch.index, openP = mtStart + metaMatch[0].length - 1;
-    let d = 1, p = openP + 1;
-    while (p < output.length && d > 0) {
-      if (output[p] === "(")
-        d++;
-      else if (output[p] === ")") {
-        d--;
-        if (d === 0)
-          break;
-      }
-      p++;
-    }
-    if (d !== 0)
-      break;
-    const attrObject = output.substring(openP + 1, p).trim(), fullEnd = p + 1, mtReplacement = (() => {
-      try {
-        const attrs = safeEvaluateObject(attrObject, context);
-        if (!attrs)
-          return "";
-        let tag = "<meta";
-        if (attrs.name)
-          tag += ` name="${escapeHtml(String(attrs.name))}"`;
-        if (attrs.property)
-          tag += ` property="${escapeHtml(String(attrs.property))}"`;
-        if (attrs.httpEquiv)
-          tag += ` http-equiv="${escapeHtml(String(attrs.httpEquiv))}"`;
-        if (attrs.content)
-          tag += ` content="${escapeHtml(String(attrs.content))}"`;
-        tag += ">";
-        return tag;
-      } catch (error) {
-        const errorMessage = error instanceof Error ? error.message : String(error);
-        return inlineError("MetaTag", errorMessage, ErrorCodes.EVALUATION_ERROR);
-      }
-    })();
-    output = output.substring(0, mtStart) + mtReplacement + output.substring(fullEnd);
-    metaTagPat.lastIndex = 0;
-  }
+import { resolveMetaArguments } from "./meta-arguments.js";
+const META_CALL = /(?<![\w$@])@meta\s*\(/g, META_TAG_CALL = /(?<![\w$@])@metaTag\s*\(/g, STRUCTURED_DATA_CALL = /(?<![\w$@])@structuredData\s*\(/g, SEO_CALL = /(?<![\w$@])@seo\s*\(/g;
+function unterminated(label, filePath) {
+  return (call) => reportDirectiveFailure(label, call, "the call has no closing parenthesis", filePath, ErrorCodes.INVALID_DIRECTIVE_SYNTAX);
+}
+function renderMetaTag(tag) {
+  return "property" in tag ? `<meta property="${escapeHtml(tag.property)}" content="${escapeHtml(tag.content)}">` : `<meta name="${escapeHtml(tag.name)}" content="${escapeHtml(tag.content)}">`;
+}
+export function processMetaDirectives(template, context, filePath, _options) {
+  if (!template.includes("@meta"))
+    return template;
+  let output = replaceDirectiveCalls(template, META_CALL, (args, call) => {
+    const resolved = resolveMetaArguments(splitTopLevelArgs(args), context);
+    if (resolved.kind === "tag")
+      return renderMetaTag(resolved.tag);
+    if (resolved.kind === "empty")
+      return "";
+    return reportDirectiveFailure("Meta", call, resolved.message, filePath, resolved.code);
+  }, unterminated("Meta", filePath));
+  output = replaceDirectiveCalls(output, META_TAG_CALL, (args, call) => {
+    const result = evaluateDirectiveArgument(args, context);
+    if (!result.ok)
+      return reportDirectiveFailure("MetaTag", call, result.message, filePath, result.code);
+    if (result.value === void 0 || result.value === null)
+      return "";
+    if (!isPlainRecord(result.value))
+      return reportDirectiveFailure("MetaTag", call, `expected an object such as { name, content }, got ${describeValue(result.value)}`, filePath, ErrorCodes.TYPE_ERROR);
+    const attrs = result.value;
+    if (!attrs.name && !attrs.property && !attrs.httpEquiv)
+      return reportDirectiveFailure("MetaTag", call, "the tag needs a name, property or httpEquiv", filePath, ErrorCodes.INVALID_DIRECTIVE_SYNTAX);
+    if (attrs.content === void 0 || attrs.content === null || attrs.content === "" || attrs.content === !1)
+      return "";
+    let tag = "<meta";
+    if (attrs.name)
+      tag += ` name="${escapeHtml(attrs.name)}"`;
+    if (attrs.property)
+      tag += ` property="${escapeHtml(attrs.property)}"`;
+    if (attrs.httpEquiv)
+      tag += ` http-equiv="${escapeHtml(attrs.httpEquiv)}"`;
+    tag += ` content="${escapeHtml(attrs.content)}">`;
+    return tag;
+  }, unterminated("MetaTag", filePath));
   return output;
 }
-export function processStructuredData(template, context, _filePath) {
-  let output = template;
-  const structuredDataPattern = /@structuredData\s*\(/g;
-  let sdMatch;
-  while ((sdMatch = structuredDataPattern.exec(output)) !== null) {
-    const sdStart = sdMatch.index, openParen = sdStart + sdMatch[0].length - 1;
-    let depth = 1, pos = openParen + 1;
-    while (pos < output.length && depth > 0) {
-      if (output[pos] === "(")
-        depth++;
-      else if (output[pos] === ")") {
-        depth--;
-        if (depth === 0)
-          break;
-      }
-      pos++;
+function withSchemaContext(value) {
+  const fill = (item) => item["@context"] ? { ...item } : { ...item, "@context": "https://schema.org" };
+  if (isPlainRecord(value))
+    return fill(value);
+  if (Array.isArray(value) && value.length > 0 && value.every(isPlainRecord))
+    return value.map(fill);
+  return null;
+}
+function jsonLdScript(data) {
+  return `<script type="application/ld+json">${JSON.stringify(data).replace(/<\//g, "<\\/")}</script>`;
+}
+export function processStructuredData(template, context, filePath) {
+  if (!template.includes("@structuredData"))
+    return template;
+  return replaceDirectiveCalls(template, STRUCTURED_DATA_CALL, (args, call) => {
+    const result = evaluateDirectiveArgument(args, context);
+    if (!result.ok)
+      return reportDirectiveFailure("StructuredData", call, result.message, filePath, result.code);
+    if (result.value === void 0 || result.value === null)
+      return "";
+    const data = withSchemaContext(result.value);
+    if (!data)
+      return reportDirectiveFailure("StructuredData", call, `expected an object or an array of objects, got ${describeValue(result.value)}`, filePath, ErrorCodes.TYPE_ERROR);
+    try {
+      return jsonLdScript(data);
+    } catch (error) {
+      const message = error instanceof Error ? error.message : String(error);
+      return reportDirectiveFailure("StructuredData", call, `the data cannot be written as JSON: ${message}`, filePath, ErrorCodes.EVALUATION_ERROR);
     }
-    if (depth !== 0)
-      break;
-    const dataObject = output.substring(openParen + 1, pos).trim(), fullEnd = pos + 1, replacement = (() => {
-      try {
-        const data = safeEvaluateObject(dataObject, context);
-        if (!data)
-          return "";
-        if (!data["@context"])
-          data["@context"] = "https://schema.org";
-        return `<script type="application/ld+json">${JSON.stringify(data).replace(/<\//g, "<\\/")}</script>`;
-      } catch (error) {
-        const errorMessage = error instanceof Error ? error.message : String(error);
-        return inlineError("StructuredData", errorMessage, ErrorCodes.EVALUATION_ERROR);
-      }
-    })();
-    output = output.substring(0, sdStart) + replacement + output.substring(fullEnd);
-    structuredDataPattern.lastIndex = 0;
-  }
-  return output;
+  }, unterminated("StructuredData", filePath));
 }
-export function processSeoDirective(template, context, _filePath, _options) {
-  let output = template;
-  const seoPat = /@seo\s*\(\s*\{/g;
-  let seoMatch;
-  while ((seoMatch = seoPat.exec(output)) !== null) {
-    const braceStart = output.indexOf("{", seoMatch.index + 4);
-    let depth = 1, pos = braceStart + 1, inStr = null, esc = !1;
-    while (pos < output.length && depth > 0) {
-      const c = output[pos];
-      if (esc) {
-        esc = !1;
-        pos++;
-        continue;
-      }
-      if (c === "\\" && inStr) {
-        esc = !0;
-        pos++;
-        continue;
-      }
-      if (inStr) {
-        if (c === inStr)
-          inStr = null;
-        pos++;
-        continue;
-      }
-      if (c === '"' || c === "'" || c === "`") {
-        inStr = c;
-        pos++;
-        continue;
-      }
-      if (c === "{")
-        depth++;
-      else if (c === "}")
-        depth--;
-      pos++;
+const SEO_KEYS = new Set(["title", "description", "keywords", "robots", "canonical", "openGraph", "twitter", "structuredData"]), SEO_KEY_HINTS = {
+  image: "openGraph.image",
+  url: "canonical or openGraph.url",
+  type: "openGraph.type",
+  siteName: "openGraph.siteName",
+  locale: "openGraph.locale",
+  og: "openGraph",
+  author: "@meta('author', ...)"
+};
+export function processSeoDirective(template, context, filePath, _options) {
+  if (!template.includes("@seo"))
+    return template;
+  return replaceDirectiveCalls(template, SEO_CALL, (args, call) => {
+    if (!args.trim())
+      return reportDirectiveFailure("SEO", call, "expected an object: @seo({ title, description }) or @seo(seoVariable)", filePath, ErrorCodes.INVALID_DIRECTIVE_SYNTAX);
+    const result = evaluateDirectiveArgument(args, context);
+    if (!result.ok)
+      return reportDirectiveFailure("SEO", call, result.message, filePath, result.code);
+    if (result.value === void 0 || result.value === null)
+      return "";
+    if (!isPlainRecord(result.value))
+      return reportDirectiveFailure("SEO", call, `expected an object such as { title, description }, got ${describeValue(result.value)}`, filePath, ErrorCodes.TYPE_ERROR);
+    try {
+      return renderSeoTags(result.value, call, filePath);
+    } catch (error) {
+      const message = error instanceof Error ? error.message : String(error);
+      return reportDirectiveFailure("SEO", call, `Error processing @seo directive: ${message}`, filePath, ErrorCodes.EVALUATION_ERROR);
     }
-    if (depth !== 0)
-      break;
-    const afterBrace = output.substring(pos).match(/^\s*\)/);
-    if (!afterBrace)
-      break;
-    const fullEnd = pos + afterBrace[0].length, seoConfig = output.substring(braceStart, pos), fullMatch = output.substring(seoMatch.index, fullEnd), replacement = ((cfg) => {
-      try {
-        const config = safeEvaluateObject(cfg, context);
-        if (!config)
-          return "";
-        let metaTags = "";
-        if (config.title) {
-          metaTags += `<title>${escapeHtml(config.title)}</title>
+  }, unterminated("SEO", filePath));
+}
+function nestedConfig(config, key, call, filePath) {
+  const value = config[key];
+  if (value === void 0 || value === null || value === !1)
+    return;
+  if (isPlainRecord(value))
+    return value;
+  warnDirective(call, `${key} must be an object, got ${describeValue(value)}; its tags were left out`, filePath);
+  return;
+}
+function renderSeoTags(config, call, filePath) {
+  const unknown = Object.keys(config).filter((key) => !SEO_KEYS.has(key));
+  if (unknown.length > 0) {
+    const described = unknown.map((key) => SEO_KEY_HINTS[key] ? `${key} (use ${SEO_KEY_HINTS[key]})` : key).join(", ");
+    warnDirective(call, `@seo does not read ${described}; ${unknown.length === 1 ? "it was" : "they were"} ignored`, filePath);
+  }
+  let metaTags = "";
+  if (config.title) {
+    metaTags += `<title>${escapeHtml(config.title)}</title>
 `;
-          metaTags += `<meta name="title" content="${escapeHtml(config.title)}">
+    metaTags += `<meta name="title" content="${escapeHtml(config.title)}">
 `;
-        }
-        if (config.description)
-          metaTags += `<meta name="description" content="${escapeHtml(config.description)}">
+  }
+  if (config.description)
+    metaTags += `<meta name="description" content="${escapeHtml(config.description)}">
 `;
-        if (config.keywords) {
-          const keywordsStr = Array.isArray(config.keywords) ? config.keywords.join(", ") : config.keywords;
-          metaTags += `<meta name="keywords" content="${escapeHtml(keywordsStr)}">
+  if (config.keywords) {
+    const keywordsStr = Array.isArray(config.keywords) ? config.keywords.join(", ") : config.keywords;
+    metaTags += `<meta name="keywords" content="${escapeHtml(keywordsStr)}">
 `;
-        }
-        if (config.robots)
-          metaTags += `<meta name="robots" content="${escapeHtml(config.robots)}">
+  }
+  if (config.robots)
+    metaTags += `<meta name="robots" content="${escapeHtml(config.robots)}">
 `;
-        if (config.canonical)
-          metaTags += `<link rel="canonical" href="${escapeHtml(config.canonical)}">
+  if (config.canonical)
+    metaTags += `<link rel="canonical" href="${escapeHtml(config.canonical)}">
 `;
-        if (config.openGraph) {
-          const og = config.openGraph;
-          metaTags += `<meta property="og:type" content="${escapeHtml(og.type || "website")}">
+  const og = nestedConfig(config, "openGraph", call, filePath);
+  if (og) {
+    metaTags += `<meta property="og:type" content="${escapeHtml(og.type || "website")}">
 `;
-          if (og.title || config.title) {
-            const ogTitle = og.title || config.title || "";
-            metaTags += `<meta property="og:title" content="${escapeHtml(ogTitle)}">
+    if (og.title || config.title) {
+      const ogTitle = og.title || config.title || "";
+      metaTags += `<meta property="og:title" content="${escapeHtml(ogTitle)}">
 `;
-          }
-          if (og.description || config.description) {
-            const ogDescription = og.description || config.description || "";
-            metaTags += `<meta property="og:description" content="${escapeHtml(ogDescription)}">
+    }
+    if (og.description || config.description) {
+      const ogDescription = og.description || config.description || "";
+      metaTags += `<meta property="og:description" content="${escapeHtml(ogDescription)}">
 `;
-          }
-          if (og.url || config.canonical) {
-            const ogUrl = og.url || config.canonical || "";
-            metaTags += `<meta property="og:url" content="${escapeHtml(ogUrl)}">
+    }
+    if (og.url || config.canonical) {
+      const ogUrl = og.url || config.canonical || "";
+      metaTags += `<meta property="og:url" content="${escapeHtml(ogUrl)}">
 `;
-          }
-          if (og.image) {
-            metaTags += `<meta property="og:image" content="${escapeHtml(og.image)}">
+    }
+    if (og.image) {
+      metaTags += `<meta property="og:image" content="${escapeHtml(og.image)}">
 `;
-            if (og.imageAlt)
-              metaTags += `<meta property="og:image:alt" content="${escapeHtml(og.imageAlt)}">
+      if (og.imageAlt)
+        metaTags += `<meta property="og:image:alt" content="${escapeHtml(og.imageAlt)}">
 `;
-            if (og.imageWidth)
-              metaTags += `<meta property="og:image:width" content="${escapeHtml(String(og.imageWidth))}">
+      if (og.imageWidth)
+        metaTags += `<meta property="og:image:width" content="${escapeHtml(String(og.imageWidth))}">
 `;
-            if (og.imageHeight)
-              metaTags += `<meta property="og:image:height" content="${escapeHtml(String(og.imageHeight))}">
+      if (og.imageHeight)
+        metaTags += `<meta property="og:image:height" content="${escapeHtml(String(og.imageHeight))}">
 `;
-            if (og.imageType)
-              metaTags += `<meta property="og:image:type" content="${escapeHtml(og.imageType)}">
+      if (og.imageType)
+        metaTags += `<meta property="og:image:type" content="${escapeHtml(og.imageType)}">
 `;
-          }
-          if (og.siteName)
-            metaTags += `<meta property="og:site_name" content="${escapeHtml(og.siteName)}">
+    }
+    if (og.siteName)
+      metaTags += `<meta property="og:site_name" content="${escapeHtml(og.siteName)}">
 `;
-          if (og.locale)
-            metaTags += `<meta property="og:locale" content="${escapeHtml(og.locale)}">
+    if (og.locale)
+      metaTags += `<meta property="og:locale" content="${escapeHtml(og.locale)}">
 `;
-          if (og.profile) {
-            const profile = [
-              ["first_name", og.profile.firstName],
-              ["last_name", og.profile.lastName],
-              ["username", og.profile.username],
-              ["gender", og.profile.gender]
-            ];
-            for (const [key, value] of profile)
-              if (value)
-                metaTags += `<meta property="profile:${key}" content="${escapeHtml(value)}">
+    if (og.profile) {
+      const profile = [
+        ["first_name", og.profile.firstName],
+        ["last_name", og.profile.lastName],
+        ["username", og.profile.username],
+        ["gender", og.profile.gender]
+      ];
+      for (const [key, value] of profile)
+        if (value)
+          metaTags += `<meta property="profile:${key}" content="${escapeHtml(value)}">
 `;
-          }
-        }
-        if (config.twitter) {
-          const twitter = config.twitter;
-          metaTags += `<meta name="twitter:card" content="${escapeHtml(twitter.card || "summary_large_image")}">
+    }
+  }
+  const twitter = nestedConfig(config, "twitter", call, filePath);
+  if (twitter) {
+    metaTags += `<meta name="twitter:card" content="${escapeHtml(twitter.card || "summary_large_image")}">
 `;
-          if (twitter.title || config.title) {
-            const twitterTitle = twitter.title || config.title || "";
-            metaTags += `<meta name="twitter:title" content="${escapeHtml(twitterTitle)}">
+    if (twitter.title || config.title) {
+      const twitterTitle = twitter.title || config.title || "";
+      metaTags += `<meta name="twitter:title" content="${escapeHtml(twitterTitle)}">
 `;
-          }
-          if (twitter.description || config.description) {
-            const twitterDesc = twitter.description || config.description || "";
-            metaTags += `<meta name="twitter:description" content="${escapeHtml(twitterDesc)}">
+    }
+    if (twitter.description || config.description) {
+      const twitterDesc = twitter.description || config.description || "";
+      metaTags += `<meta name="twitter:description" content="${escapeHtml(twitterDesc)}">
 `;
-          }
-          if (twitter.image || config.openGraph && config.openGraph.image) {
-            const twitterImage = twitter.image || (config.openGraph ? config.openGraph.image : "") || "";
-            metaTags += `<meta name="twitter:image" content="${escapeHtml(twitterImage)}">
+    }
+    if (twitter.image || og && og.image) {
+      const twitterImage = twitter.image || (og ? og.image : "") || "";
+      metaTags += `<meta name="twitter:image" content="${escapeHtml(twitterImage)}">
 `;
-            const twitterImageAlt = twitter.imageAlt || (!twitter.image && config.openGraph ? config.openGraph.imageAlt : void 0);
-            if (twitterImageAlt)
-              metaTags += `<meta name="twitter:image:alt" content="${escapeHtml(twitterImageAlt)}">
+      const twitterImageAlt = twitter.imageAlt || (!twitter.image && og ? og.imageAlt : void 0);
+      if (twitterImageAlt)
+        metaTags += `<meta name="twitter:image:alt" content="${escapeHtml(twitterImageAlt)}">
 `;
-          }
-          if (twitter.site)
-            metaTags += `<meta name="twitter:site" content="${escapeHtml(twitter.site)}">
+    }
+    if (twitter.site)
+      metaTags += `<meta name="twitter:site" content="${escapeHtml(twitter.site)}">
 `;
-          if (twitter.creator)
-            metaTags += `<meta name="twitter:creator" content="${escapeHtml(twitter.creator)}">
+    if (twitter.creator)
+      metaTags += `<meta name="twitter:creator" content="${escapeHtml(twitter.creator)}">
 `;
-        }
-        if (config.structuredData) {
-          const jsonStr = JSON.stringify(config.structuredData).replace(/<\//g, "<\\/");
-          metaTags += `<script type="application/ld+json">${jsonStr}</script>
+  }
+  if (config.structuredData !== void 0 && config.structuredData !== null) {
+    const data = withSchemaContext(config.structuredData);
+    if (data)
+      metaTags += `${jsonLdScript(data)}
 `;
-        }
-        return metaTags.trim();
-      } catch (error) {
-        const errorMessage = error instanceof Error ? error.message : String(error);
-        return inlineError("SEO", `Error processing @seo directive: ${errorMessage}`, ErrorCodes.EVALUATION_ERROR);
-      }
-    })(seoConfig);
-    output = output.substring(0, seoMatch.index) + replacement + output.substring(fullEnd);
-    seoPat.lastIndex = seoMatch.index + replacement.length;
+    else
+      warnDirective(call, `structuredData must be an object or an array of objects, got ${describeValue(config.structuredData)}; it was left out`, filePath);
   }
-  return output;
+  return metaTags.trim();
 }
 export function injectSeoTags(html, context, options) {
   if (options.seo?.enabled === !1)
@@ -367,35 +348,20 @@ function readPageDescription(head) {
   }
   return "";
 }
-function escapeHtml(str) {
-  return str.replace(/&/g, "&amp;").replace(/</g, "&lt;").replace(/>/g, "&gt;").replace(/"/g, "&quot;").replace(/'/g, "&#39;");
-}
-const PROPERTY_NAMESPACES = ["og:", "article:", "book:", "profile:", "fb:", "music:", "video:"];
-function unquoteParam(raw) {
-  const text = raw.trim(), first = text[0];
-  if ((first === "'" || first === '"' || first === "`") && text.length > 1 && text.at(-1) === first)
-    return { value: text.slice(1, -1), quoted: !0 };
-  return { value: text, quoted: !1 };
+function escapeHtml(value) {
+  return String(value).replace(/&/g, "&amp;").replace(/</g, "&lt;").replace(/>/g, "&gt;").replace(/"/g, "&quot;").replace(/'/g, "&#39;");
 }
 export const metaDirective = {
   name: "meta",
-  handler: (_content, params, context, _filePath) => {
-    if (params.length < 1)
-      return inlineError("Meta", "meta directive requires at least the meta name", ErrorCodes.INVALID_DIRECTIVE_SYNTAX);
-    const name = unquoteParam(params[0]).value;
-    let metaContent;
-    if (params.length > 1) {
-      const arg = unquoteParam(params[1]);
-      metaContent = arg.quoted ? arg.value : String(context?.[arg.value] ?? arg.value);
-    } else {
-      const segment = name.includes(":") ? name.slice(name.indexOf(":") + 1) : name, openGraph = context?.openGraph;
-      metaContent = String(context?.[segment] ?? openGraph?.[segment] ?? "");
-    }
-    if (!metaContent)
+  rawParams: !0,
+  handler: (_content, params, context, filePath) => {
+    const resolved = resolveMetaArguments(params, context ?? {});
+    if (resolved.kind === "error")
+      return reportDirectiveFailure("Meta", `@meta(${params.join(", ")})`, resolved.message, filePath, resolved.code);
+    if (resolved.kind === "empty")
       return "";
-    const tag = PROPERTY_NAMESPACES.some((prefix) => name.startsWith(prefix)) ? { property: name, content: metaContent } : { name, content: metaContent };
     if (context)
-      context.__stx_runtime_head = mergeHeadConfigs(context.__stx_runtime_head ?? {}, { meta: [tag] });
+      context.__stx_runtime_head = mergeHeadConfigs(context.__stx_runtime_head ?? {}, { meta: [resolved.tag] });
     return "";
   },
   hasEndTag: !1
@@ -410,7 +376,7 @@ export const metaDirective = {
         data["@context"] = "https://schema.org";
       if (!data["@type"])
         return inlineError("StructuredData", "structuredData requires @type property", ErrorCodes.INVALID_DIRECTIVE_SYNTAX);
-      return `<script type="application/ld+json">${JSON.stringify(data)}</script>`;
+      return jsonLdScript(data);
     } catch (error) {
       const errorMessage = error instanceof Error ? error.message : String(error);
       return inlineError("StructuredData", errorMessage, ErrorCodes.EVALUATION_ERROR);