From 1d46d330851daf914a5751606dc7155ad320ce9a Mon Sep 17 00:00:00 2001
From: Chance Strickland <hi@chance.dev>
Date: Fri, 10 Mar 2023 11:00:47 -0800
Subject: [PATCH] Add support for creating non-meta tags with `v2_meta` (#5746)

---
 .changeset/meta-v2-enhancements.md            |  6 ++
 integration/meta-test.ts                      | 91 ++++++++++++++++++-
 packages/remix-react/components.tsx           | 51 +++++++++--
 packages/remix-react/routeModules.ts          | 11 ++-
 packages/remix-server-runtime/routeModules.ts | 11 ++-
 5 files changed, 157 insertions(+), 13 deletions(-)
 create mode 100644 .changeset/meta-v2-enhancements.md

diff --git a/.changeset/meta-v2-enhancements.md b/.changeset/meta-v2-enhancements.md
new file mode 100644
index 00000000000..08bc9d5e875
--- /dev/null
+++ b/.changeset/meta-v2-enhancements.md
@@ -0,0 +1,6 @@
+---
+"@remix-run/react": minor
+"@remix-run/server-runtime": patch
+---
+
+Add support for generating `<script type='application/ld+json' />` and meta-related `<link />` tags to document head via the route `meta` function when using the `v2_meta` future flag
diff --git a/integration/meta-test.ts b/integration/meta-test.ts
index b490e1861d2..a3a2b4eb2bf 100644
--- a/integration/meta-test.ts
+++ b/integration/meta-test.ts
@@ -443,9 +443,9 @@ test.describe("v2_meta", () => {
         `,
 
         "app/routes/_index.jsx": js`
-          export const meta = ({ data, matches }) => [
-            ...matches.map((match) => match.meta),
-          ];
+          export const meta = ({ data, matches }) =>
+            matches.flatMap((match) => match.meta);
+
           export default function Index() {
             return <div>This is the index file</div>;
           }
@@ -464,6 +464,59 @@ test.describe("v2_meta", () => {
           }
         `,
 
+        "app/routes/authors.$authorId.jsx": js`
+          import { json } from "@remix-run/node";
+
+          export async function loader({ params }) {
+            return json({
+              author: {
+                id: params.authorId,
+                name: "Sonny Day",
+                address: {
+                  streetAddress: "123 Sunset Cliffs Blvd",
+                  city: "San Diego",
+                  state: "CA",
+                  zip: "92107",
+                },
+                emails: [
+                  "sonnyday@fancymail.com",
+                  "surfergal@veryprofessional.org",
+                ],
+              },
+            });
+          }
+
+          export function meta({ data }) {
+            let { author } = data;
+            return [
+              { title: data.name + " Profile" },
+              {
+                tagName: "link",
+                rel: "canonical",
+                href: "https://website.com/authors/" + author.id,
+              },
+              {
+                "script:ld+json": {
+                  "@context": "http://schema.org",
+                  "@type": "Person",
+                  "name": author.name,
+                  "address": {
+                    "@type": "PostalAddress",
+                    "streetAddress": author.address.streetAddress,
+                    "addressLocality": author.address.city,
+                    "addressRegion": author.address.state,
+                    "postalCode": author.address.zip,
+                  },
+                  "email": author.emails,
+                },
+              },
+            ];
+          }
+          export default function AuthorBio() {
+            return <div>Bio here!</div>;
+          }
+        `,
+
         "app/routes/music.jsx": js`
           export function meta({ data, matches }) {
             let rootModule = matches.find(match => match.route.id === "root");
@@ -531,4 +584,36 @@ test.describe("v2_meta", () => {
     await app.goto("/");
     expect(await app.getHtml('meta[property="og:image"]')).toBeTruthy();
   });
+
+  test("{ 'script:ld+json': {} } adds a <script type='application/ld+json' />", async ({
+    page,
+  }) => {
+    let app = new PlaywrightFixture(appFixture, page);
+    await app.goto("/authors/1");
+    let scriptTag = await app.getHtml('script[type="application/ld+json"]');
+    let scriptContents = scriptTag
+      .replace('<script type="application/ld+json">', "")
+      .replace("</script>", "")
+      .trim();
+
+    expect(JSON.parse(scriptContents)).toEqual({
+      "@context": "http://schema.org",
+      "@type": "Person",
+      name: "Sonny Day",
+      address: {
+        "@type": "PostalAddress",
+        streetAddress: "123 Sunset Cliffs Blvd",
+        addressLocality: "San Diego",
+        addressRegion: "CA",
+        postalCode: "92107",
+      },
+      email: ["sonnyday@fancymail.com", "surfergal@veryprofessional.org"],
+    });
+  });
+
+  test("{ tagName: 'link' } adds a <link />", async ({ page }) => {
+    let app = new PlaywrightFixture(appFixture, page);
+    await app.goto("/authors/1");
+    expect(await app.getHtml('link[rel="canonical"]')).toBeTruthy();
+  });
 });
diff --git a/packages/remix-react/components.tsx b/packages/remix-react/components.tsx
index 9c484975f08..52fe9d257e1 100644
--- a/packages/remix-react/components.tsx
+++ b/packages/remix-react/components.tsx
@@ -602,7 +602,7 @@ function V1Meta() {
         }
 
         if (["charset", "charSet"].includes(name)) {
-          return <meta key="charset" charSet={value as string} />;
+          return <meta key="charSet" charSet={value as string} />;
         }
 
         if (name === "title") {
@@ -719,18 +719,49 @@ function V2Meta() {
           return null;
         }
 
+        if ("tagName" in metaProps) {
+          let tagName = metaProps.tagName;
+          delete metaProps.tagName;
+          if (!isValidMetaTag(tagName)) {
+            console.warn(
+              `A meta object uses an invalid tagName: ${tagName}. Expected either 'link' or 'meta'`
+            );
+            return null;
+          }
+          let Comp = tagName;
+          return <Comp key={JSON.stringify(metaProps)} {...metaProps} />;
+        }
+
         if ("title" in metaProps) {
           return <title key="title">{String(metaProps.title)}</title>;
         }
 
-        if ("charSet" in metaProps || "charset" in metaProps) {
-          // TODO: We normalize this for the user in v1, but should we continue
-          // to do that? Seems like a nice convenience IMO.
+        if ("charset" in metaProps) {
+          metaProps.charSet ??= metaProps.charset;
+          delete metaProps.charset;
+        }
+
+        if ("charSet" in metaProps && metaProps.charSet != null) {
+          return typeof metaProps.charSet === "string" ? (
+            <meta key="charSet" charSet={metaProps.charSet} />
+          ) : null;
+        }
+
+        if ("script:ld+json" in metaProps) {
+          let json: string | null = null;
+          try {
+            json = JSON.stringify(metaProps["script:ld+json"]);
+          } catch (err) {}
           return (
-            <meta
-              key="charset"
-              charSet={metaProps.charSet || (metaProps as any).charset}
-            />
+            json != null && (
+              <script
+                key="script:ld+json"
+                type="application/ld+json"
+                dangerouslySetInnerHTML={{
+                  __html: JSON.stringify(metaProps["script:ld+json"]),
+                }}
+              />
+            )
           );
         }
         return <meta key={JSON.stringify(metaProps)} {...metaProps} />;
@@ -739,6 +770,10 @@ function V2Meta() {
   );
 }
 
+function isValidMetaTag(tagName: unknown): tagName is "meta" | "link" {
+  return typeof tagName === "string" && /^(meta|link)$/.test(tagName);
+}
+
 export function Meta() {
   let { future } = useRemixContext();
   return future?.v2_meta ? <V2Meta /> : <V1Meta />;
diff --git a/packages/remix-react/routeModules.ts b/packages/remix-react/routeModules.ts
index 147d799bdff..ab4c974c401 100644
--- a/packages/remix-react/routeModules.ts
+++ b/packages/remix-react/routeModules.ts
@@ -124,7 +124,16 @@ export type V2_HtmlMetaDescriptor =
   | { name: string; content: string }
   | { property: string; content: string }
   | { httpEquiv: string; content: string }
-  | { [name: string]: string };
+  | { "script:ld+json": LdJsonObject }
+  | { tagName: "meta" | "link"; [name: string]: string }
+  | { [name: string]: unknown };
+
+type LdJsonObject = { [Key in string]: LdJsonValue } & {
+  [Key in string]?: LdJsonValue | undefined;
+};
+type LdJsonArray = LdJsonValue[] | readonly LdJsonValue[];
+type LdJsonPrimitive = string | number | boolean | null;
+type LdJsonValue = LdJsonPrimitive | LdJsonObject | LdJsonArray;
 
 /**
  * A React component that is rendered for a route.
diff --git a/packages/remix-server-runtime/routeModules.ts b/packages/remix-server-runtime/routeModules.ts
index 25a7bb19820..5c5e825ddbc 100644
--- a/packages/remix-server-runtime/routeModules.ts
+++ b/packages/remix-server-runtime/routeModules.ts
@@ -221,7 +221,16 @@ export type V2_HtmlMetaDescriptor =
   | { name: string; content: string }
   | { property: string; content: string }
   | { httpEquiv: string; content: string }
-  | { [name: string]: string };
+  | { "script:ld+json": LdJsonObject }
+  | { tagName: "meta" | "link"; [name: string]: string }
+  | { [name: string]: unknown };
+
+type LdJsonObject = { [Key in string]: LdJsonValue } & {
+  [Key in string]?: LdJsonValue | undefined;
+};
+type LdJsonArray = LdJsonValue[] | readonly LdJsonValue[];
+type LdJsonPrimitive = string | number | boolean | null;
+type LdJsonValue = LdJsonPrimitive | LdJsonObject | LdJsonArray;
 
 /**
  * A React component that is rendered for a route.