From 887147bdf3dff9e2a38c76ac65eae3244a04f577 Mon Sep 17 00:00:00 2001
From: dawn <dawn@tangled.org>
Date: Fri, 18 Sep 2026 01:28:16 +0300
Subject: [PATCH] web/repo: stop repo subpages emitting a second og image

Signed-off-by: dawn <dawn@tangled.org>
---
 web/src/lib/components/seo/PageHead.svelte    | 40 +++++++++++++++++++
 .../[handle]/[repo]/commit/[ref]/+page.svelte |  4 +-
 .../routes/[handle]/[repo]/fork/+page.svelte  |  4 +-
 .../[handle]/[repo]/issues/new/+page.svelte   |  4 +-
 .../[handle]/[repo]/pulls/new/+page.svelte    |  4 +-
 .../[repo]/tree/[ref]/[...path]/+page.svelte  |  4 +-
 6 files changed, 50 insertions(+), 10 deletions(-)
 create mode 100644 web/src/lib/components/seo/PageHead.svelte

diff --git a/web/src/lib/components/seo/PageHead.svelte b/web/src/lib/components/seo/PageHead.svelte
new file mode 100644
index 000000000..c7c2b8e97
--- /dev/null
+++ b/web/src/lib/components/seo/PageHead.svelte
@@ -0,0 +1,40 @@
+<script lang="ts">
+	// page-owned head for pages nested under a layout that already renders
+	// <Meta>: same title and description tags without emitting a second
+	// og:image, which would sit next to the layout's card in link previews
+	interface Props {
+		title?: string;
+		description?: string;
+		type?: "website" | "profile" | "article";
+		noindex?: boolean;
+	}
+
+	let { title, description, type = "website", noindex = false }: Props = $props();
+
+	const fullTitle = $derived(
+		title
+			? title.includes("·") || title.includes("&middot;")
+				? title
+				: `${title} · Tangled`
+			: undefined
+	);
+</script>
+
+<svelte:head>
+	{#if fullTitle}
+		<title>{fullTitle}</title>
+		<meta property="og:title" content={fullTitle} />
+		<meta name="twitter:title" content={fullTitle} />
+	{/if}
+	{#if description}
+		<meta name="description" content={description} />
+		<meta property="og:description" content={description} />
+		<meta name="twitter:description" content={description} />
+	{/if}
+	{#if type}
+		<meta property="og:type" content={type} />
+	{/if}
+	{#if noindex}
+		<meta name="robots" content="noindex, nofollow" />
+	{/if}
+</svelte:head>
diff --git a/web/src/routes/[handle]/[repo]/commit/[ref]/+page.svelte b/web/src/routes/[handle]/[repo]/commit/[ref]/+page.svelte
index 2843beab1..2934fb935 100644
--- a/web/src/routes/[handle]/[repo]/commit/[ref]/+page.svelte
+++ b/web/src/routes/[handle]/[repo]/commit/[ref]/+page.svelte
@@ -10,7 +10,7 @@
 	import repoCommitviewFullCommit from "$lib/bones/repo-commitview--full-commit.bones.json";
 	import CommitView from "$lib/components/repo/CommitView.svelte";
 	import Pending from "$lib/components/ui/Pending.svelte";
-	import Meta from "$lib/components/seo/Meta.svelte";
+	import PageHead from "$lib/components/seo/PageHead.svelte";
 	import type { DiffStyle } from "$lib/components/repo/pierre";
 
 	let { data } = $props();
@@ -64,7 +64,7 @@
 	);
 </script>
 
-<Meta title={commitTitle} description={commitDesc} type="article" />
+<PageHead title={commitTitle} description={commitDesc} type="article" />
 
 <Pending>
 	{#snippet skeleton()}
diff --git a/web/src/routes/[handle]/[repo]/fork/+page.svelte b/web/src/routes/[handle]/[repo]/fork/+page.svelte
index c5b2c2b3a..737e7d909 100644
--- a/web/src/routes/[handle]/[repo]/fork/+page.svelte
+++ b/web/src/routes/[handle]/[repo]/fork/+page.svelte
@@ -1,5 +1,5 @@
 <script lang="ts">
-	import Meta from "$lib/components/seo/Meta.svelte";
+	import PageHead from "$lib/components/seo/PageHead.svelte";
 	import { page } from "$app/state";
 	import repoRepoforkformSkeletonFixture from "$lib/bones/repo-repoforkform--skeleton-fixture.bones.json";
 	import RepoForkForm from "$lib/components/repo/RepoForkForm.svelte";
@@ -9,7 +9,7 @@
 	let { data } = $props();
 </script>
 
-<Meta
+<PageHead
 	noindex
 	title="Fork · {decodeURIComponent(page.params.handle ?? '')}/{decodeURIComponent(
 		page.params.repo ?? ''
diff --git a/web/src/routes/[handle]/[repo]/issues/new/+page.svelte b/web/src/routes/[handle]/[repo]/issues/new/+page.svelte
index f7b961cd1..0d17d0766 100644
--- a/web/src/routes/[handle]/[repo]/issues/new/+page.svelte
+++ b/web/src/routes/[handle]/[repo]/issues/new/+page.svelte
@@ -1,5 +1,5 @@
 <script lang="ts">
-	import Meta from "$lib/components/seo/Meta.svelte";
+	import PageHead from "$lib/components/seo/PageHead.svelte";
 	import { goto } from "$app/navigation";
 	import { page } from "$app/state";
 	import { resolve } from "$app/paths";
@@ -32,7 +32,7 @@
 	};
 </script>
 
-<Meta
+<PageHead
 	noindex
 	title="New issue · {decodeURIComponent(page.params.handle ?? '')}/{decodeURIComponent(
 		page.params.repo ?? ''
diff --git a/web/src/routes/[handle]/[repo]/pulls/new/+page.svelte b/web/src/routes/[handle]/[repo]/pulls/new/+page.svelte
index fca1016f3..c4519802e 100644
--- a/web/src/routes/[handle]/[repo]/pulls/new/+page.svelte
+++ b/web/src/routes/[handle]/[repo]/pulls/new/+page.svelte
@@ -1,5 +1,5 @@
 <script lang="ts">
-	import Meta from "$lib/components/seo/Meta.svelte";
+	import PageHead from "$lib/components/seo/PageHead.svelte";
 	import { page } from "$app/state";
 	import PullCompose from "$lib/components/repo/pulls/PullCompose.svelte";
 	import Pending from "$lib/components/ui/Pending.svelte";
@@ -19,7 +19,7 @@
 	});
 </script>
 
-<Meta
+<PageHead
 	noindex
 	title="New pull · {decodeURIComponent(page.params.handle ?? '')}/{decodeURIComponent(
 		page.params.repo ?? ''
diff --git a/web/src/routes/[handle]/[repo]/tree/[ref]/[...path]/+page.svelte b/web/src/routes/[handle]/[repo]/tree/[ref]/[...path]/+page.svelte
index 9cbbd3e36..0d06b4e5c 100644
--- a/web/src/routes/[handle]/[repo]/tree/[ref]/[...path]/+page.svelte
+++ b/web/src/routes/[handle]/[repo]/tree/[ref]/[...path]/+page.svelte
@@ -9,7 +9,7 @@
 	import bonesTreeviewSkeletonFixture from "$lib/bones/bones-treeview--skeleton-fixture.bones.json";
 	import TreeHeader from "$lib/components/repo/TreeHeader.svelte";
 	import Pending from "$lib/components/ui/Pending.svelte";
-	import Meta from "$lib/components/seo/Meta.svelte";
+	import PageHead from "$lib/components/seo/PageHead.svelte";
 	import TabPanel from "$lib/components/ui/TabPanel.svelte";
 
 	let { data } = $props();
@@ -34,7 +34,7 @@
 	const treeDesc = $derived(`Viewing ${filePath} in ${repoName} on Tangled.`);
 </script>
 
-<Meta title={treeTitle} description={treeDesc} />
+<PageHead title={treeTitle} description={treeDesc} />
 
 <!-- shared boundary avoids reserving a skeleton card for directories without a readme -->
 <Pending>
-- 
2.51.2