From 6cf0b259e194ded9ba33ba149a3cf5c7a9f2e5f1 Mon Sep 17 00:00:00 2001
From: Radin Hamidi Rad <radin.h@gmail.com>
Date: Wed, 29 Apr 2026 21:56:13 -0400
Subject: [PATCH 1/3] Header: linkout icons, current-tab; sitemap: point at
 populated file
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Three pieces of user-requested polish:

1. Marketing nav: add Leaderboard alongside Dashboard, both with the
   linkout arrow icon and opening in the current tab. Drop the
   trailing "↗" from the old "Dashboard ↗" label since the icon now
   conveys it.

2. Leaderboard nav: keep the Toolkit link (re-added earlier), with
   the same linkout icon, current-tab.

3. Header component: add a `newTab` prop separate from `external`.
   `external` controls icon rendering only; `newTab` controls
   target/rel. Default behavior is current-tab — let users
   middle-click / cmd-click for a new tab when they want one.

Also propagate same-tab to EcosystemMap cards (they were forcing
_blank for any http link).

Sitemap visibility: `sitemap-index.xml` is one level above the URLs
(it just points at sitemap-0.xml — standard Sitemaps protocol). When
visited directly it looks empty. Updated <link rel="sitemap"> to
point at /sitemap-0.xml so anything inspecting the page lands on the
populated file. robots.txt now lists both URLs (crawlers handle
either form).

Stacked on chore/copy-polish (PR #13). Once that merges, this PR's
diff narrows to just these files.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
---
 .../site/src/layouts/Default.astro            |  1 +
 web/shared/components/Header.astro            | 31 +++++++++++++++++--
 web/site/src/components/EcosystemMap.astro    |  2 --
 web/site/src/layouts/Default.astro            |  8 +++--
 4 files changed, 35 insertions(+), 7 deletions(-)

diff --git a/reproducibility/site/src/layouts/Default.astro b/reproducibility/site/src/layouts/Default.astro
index 1fac4bd..c54cb92 100644
--- a/reproducibility/site/src/layouts/Default.astro
+++ b/reproducibility/site/src/layouts/Default.astro
@@ -11,6 +11,7 @@ interface Props {
 
 const { title, description } = Astro.props;
 
+// External links render the linkout icon and open in the current tab.
 const navLinks = [
   { label: "Datasets", href: "/datasets/" },
   { label: "Methods", href: "/methods/" },
diff --git a/web/shared/components/Header.astro b/web/shared/components/Header.astro
index 84526c1..ab41dc9 100644
--- a/web/shared/components/Header.astro
+++ b/web/shared/components/Header.astro
@@ -7,7 +7,15 @@
 interface NavLink {
   label: string;
   href: string;
+  /**
+   * Renders the external-link arrow icon next to the label so users can see
+   * at a glance that the link leaves the current site. Note: by default the
+   * link still opens in the *same* tab — set `newTab: true` for cross-site
+   * links you really want to open in a fresh tab.
+   */
   external?: boolean;
+  /** Open in a new tab. Adds rel="noopener noreferrer" automatically. */
+  newTab?: boolean;
 }
 
 interface Props {
@@ -44,11 +52,28 @@ const {
         links.map((l) => (
           <a
             href={l.href}
-            class="rounded px-3 py-1.5 text-sm font-medium text-white/90 hover:bg-white/15 hover:text-white"
-            target={l.external ? "_blank" : undefined}
-            rel={l.external ? "noopener noreferrer" : undefined}
+            class="inline-flex items-center gap-1 rounded px-3 py-1.5 text-sm font-medium text-white/90 hover:bg-white/15 hover:text-white"
+            target={l.newTab ? "_blank" : undefined}
+            rel={l.newTab ? "noopener noreferrer" : undefined}
           >
             {l.label}
+            {l.external && (
+              <svg
+                aria-hidden="true"
+                width="11"
+                height="11"
+                viewBox="0 0 24 24"
+                fill="none"
+                stroke="currentColor"
+                stroke-width="2"
+                stroke-linecap="round"
+                stroke-linejoin="round"
+                class="opacity-80"
+              >
+                <path d="M7 17 17 7" />
+                <path d="M8 7h9v9" />
+              </svg>
+            )}
           </a>
         ))
       }
diff --git a/web/site/src/components/EcosystemMap.astro b/web/site/src/components/EcosystemMap.astro
index 270ff0c..5bb74f9 100644
--- a/web/site/src/components/EcosystemMap.astro
+++ b/web/site/src/components/EcosystemMap.astro
@@ -53,8 +53,6 @@ const cards = [
       cards.map((c) => (
         <a
           href={c.href}
-          target={c.href.startsWith("http") ? "_blank" : undefined}
-          rel={c.href.startsWith("http") ? "noopener noreferrer" : undefined}
           class:list={[
             "qg-card flex flex-col justify-between !p-5",
             c.primary && "!border-qg-accent",
diff --git a/web/site/src/layouts/Default.astro b/web/site/src/layouts/Default.astro
index 556a485..18779c4 100644
--- a/web/site/src/layouts/Default.astro
+++ b/web/site/src/layouts/Default.astro
@@ -33,13 +33,17 @@ const SITE = "https://querygym.com";
 const canonical = new URL(Astro.url.pathname, SITE).toString();
 const ogImageAbsolute = new URL(ogImage, SITE).toString();
 
+// External links use `external: true` (renders the linkout icon) and open in
+// the *current* tab — opening a new tab silently is a worse UX than letting
+// the user middle-click / cmd-click when they want a tab.
 const navLinks = [
   { label: "Install", href: "/install" },
   { label: "Methods", href: "/methods" },
   { label: "Reproducibility", href: "/reproducibility" },
   { label: "Cite", href: "/cite" },
   { label: "Docs", href: "https://querygym.readthedocs.io", external: true },
-  { label: "Dashboard ↗", href: "https://dashboard.querygym.com", external: true },
+  { label: "Leaderboard", href: "https://leaderboard.querygym.com", external: true },
+  { label: "Dashboard", href: "https://dashboard.querygym.com", external: true },
 ];
 
 const fullTitle = `${title} — QueryGym`;
@@ -78,7 +82,7 @@ const metaDescription =
     <meta name="twitter:image" content={ogImageAbsolute} />
 
     <link rel="icon" type="image/png" href="/querygym-logo.png" />
-    <link rel="sitemap" href="/sitemap-index.xml" />
+    <link rel="sitemap" type="application/xml" title="Sitemap" href="/sitemap-index.xml" />
 
     {jsonLd && <script type="application/ld+json" set:html={jsonLd} />}
 

From cf35eb1533a02ef21e994a8bede820db88f9ce4d Mon Sep 17 00:00:00 2001
From: Radin Hamidi Rad <radin.h@gmail.com>
Date: Wed, 29 Apr 2026 22:11:42 -0400
Subject: [PATCH 2/3] Add sitemap + robots.txt to leaderboard.querygym.com

Mirror what querygym.com already has:
- @astrojs/sitemap integration. Home priority 1.0, per-dataset/method/
  model index pages 0.5, run-detail pages 0.3 monthly (they're mostly
  internal-link targets so search engines should focus on the index
  pages).
- robots.txt with the standard "Allow all + sitemap pointer" shape.
- <link rel="sitemap"> in the layout for browser discoverability.

Generates 37 URLs (1 home + 32 per-dataset + datasets/methods/models
indexes + about) on the current empty-results state. The shard count
will grow as runs land via submit_run.py.

Stacked on chore/nav-and-sitemap-stacked (PR #14).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
---
 pnpm-lock.yaml                                |  3 +++
 reproducibility/site/astro.config.mjs         | 24 ++++++++++++++++++-
 reproducibility/site/package.json             | 21 ++++++++--------
 reproducibility/site/public/robots.txt        |  5 ++++
 .../site/src/layouts/Default.astro            |  1 +
 5 files changed, 43 insertions(+), 11 deletions(-)
 create mode 100644 reproducibility/site/public/robots.txt

diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml
index 748932a..8f8d027 100644
--- a/pnpm-lock.yaml
+++ b/pnpm-lock.yaml
@@ -10,6 +10,9 @@ importers:
 
   reproducibility/site:
     dependencies:
+      '@astrojs/sitemap':
+        specifier: ^3.7.2
+        version: 3.7.2
       '@astrojs/solid-js':
         specifier: ^5.0.0
         version: 5.1.3(@types/node@20.19.39)(jiti@1.21.7)(solid-js@1.9.12)(tsx@4.21.0)(yaml@2.8.3)
diff --git a/reproducibility/site/astro.config.mjs b/reproducibility/site/astro.config.mjs
index df3dcb0..3cde561 100644
--- a/reproducibility/site/astro.config.mjs
+++ b/reproducibility/site/astro.config.mjs
@@ -1,11 +1,33 @@
 import { defineConfig } from "astro/config";
 import tailwind from "@astrojs/tailwind";
 import solid from "@astrojs/solid-js";
+import sitemap from "@astrojs/sitemap";
 
 export default defineConfig({
   site: "https://leaderboard.querygym.com",
   output: "static",
-  integrations: [tailwind({ applyBaseStyles: false }), solid()],
+  integrations: [
+    tailwind({ applyBaseStyles: false }),
+    solid(),
+    sitemap({
+      changefreq: "weekly",
+      priority: 0.5,
+      lastmod: new Date(),
+      serialize(item) {
+        // Home page is the canonical entry point.
+        if (item.url === "https://leaderboard.querygym.com/") {
+          return { ...item, priority: 1.0, changefreq: "weekly" };
+        }
+        // Per-run detail pages are mostly internal-link targets — give them
+        // lower priority so search engines focus on the dataset/method/model
+        // index pages.
+        if (item.url.startsWith("https://leaderboard.querygym.com/runs/")) {
+          return { ...item, priority: 0.3, changefreq: "monthly" };
+        }
+        return item;
+      },
+    }),
+  ],
   vite: {
     ssr: {
       // Needed because we read CSV/YAML at build time from outside the project root.
diff --git a/reproducibility/site/package.json b/reproducibility/site/package.json
index e03b79d..2cdfbd8 100644
--- a/reproducibility/site/package.json
+++ b/reproducibility/site/package.json
@@ -13,22 +13,23 @@
     "check": "astro check"
   },
   "dependencies": {
-    "@qg/shared": "workspace:*",
-    "astro": "^5.0.0",
-    "@astrojs/tailwind": "^5.1.5",
+    "@astrojs/sitemap": "^3.7.2",
     "@astrojs/solid-js": "^5.0.0",
-    "solid-js": "^1.9.0",
+    "@astrojs/tailwind": "^5.1.5",
+    "@qg/shared": "workspace:*",
     "@tanstack/solid-table": "^8.20.0",
-    "tailwindcss": "^3.4.0",
+    "astro": "^5.0.0",
+    "js-yaml": "^4.1.0",
     "papaparse": "^5.4.1",
-    "js-yaml": "^4.1.0"
+    "solid-js": "^1.9.0",
+    "tailwindcss": "^3.4.0"
   },
   "devDependencies": {
+    "@types/js-yaml": "^4.0.9",
+    "@types/node": "^20.0.0",
+    "@types/papaparse": "^5.3.15",
     "tsx": "^4.19.0",
     "typescript": "^5.6.0",
-    "wrangler": "^4.0.0",
-    "@types/papaparse": "^5.3.15",
-    "@types/js-yaml": "^4.0.9",
-    "@types/node": "^20.0.0"
+    "wrangler": "^4.0.0"
   }
 }
diff --git a/reproducibility/site/public/robots.txt b/reproducibility/site/public/robots.txt
new file mode 100644
index 0000000..c2967cb
--- /dev/null
+++ b/reproducibility/site/public/robots.txt
@@ -0,0 +1,5 @@
+# leaderboard.querygym.com — open to all well-behaved crawlers.
+User-agent: *
+Allow: /
+
+Sitemap: https://leaderboard.querygym.com/sitemap-index.xml
diff --git a/reproducibility/site/src/layouts/Default.astro b/reproducibility/site/src/layouts/Default.astro
index c54cb92..3a001a9 100644
--- a/reproducibility/site/src/layouts/Default.astro
+++ b/reproducibility/site/src/layouts/Default.astro
@@ -29,6 +29,7 @@ const navLinks = [
     <title>{title} — QueryGym Leaderboard</title>
     {description && <meta name="description" content={description} />}
     <link rel="icon" type="image/png" href="/querygym-logo.png" />
+    <link rel="sitemap" type="application/xml" title="Sitemap" href="/sitemap-index.xml" />
 
     <GoogleAnalytics />
   </head>

From 866b4dcbcfc7f097f57b3d940372f66a29b4d6cd Mon Sep 17 00:00:00 2001
From: Radin Hamidi Rad <radin.h@gmail.com>
Date: Wed, 29 Apr 2026 23:04:32 -0400
Subject: [PATCH 3/3] Drop target=_blank from footer + EmptyState CTA
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

User feedback: Dashboard and Leaderboard links should open in the
current tab, not a new one. The nav links were already clean (PR
#14 work in this stack), but two more stragglers were forcing _blank:

- Footer (Dashboard, GitHub, Docs, Paper) — flat link strip with
  target=_blank on every entry. Removed.
- Leaderboard EmptyState's "How to submit a result →" CTA pointing
  at the contributor guide on RTD. Removed _blank for consistency
  with the rest of the leaderboard's links.

Remaining target=_blank instances (intentionally kept) are on
external research references: arXiv paper links on /methods, /cite,
and /about; GitHub source-file links on /about and /reproducibility;
RTD docs link on /install. These are research-citation conventions
where readers expect parallel-tab reading. Happy to flip globally if
that's the actual preference — just say.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
---
 reproducibility/site/src/components/EmptyState.astro | 2 --
 web/shared/components/Footer.astro                   | 2 +-
 2 files changed, 1 insertion(+), 3 deletions(-)

diff --git a/reproducibility/site/src/components/EmptyState.astro b/reproducibility/site/src/components/EmptyState.astro
index aa11ccf..20cfb6f 100644
--- a/reproducibility/site/src/components/EmptyState.astro
+++ b/reproducibility/site/src/components/EmptyState.astro
@@ -16,8 +16,6 @@ const {
   <a
     href="https://querygym.readthedocs.io/en/latest/user-guide/reproducibility/"
     class="mt-4 inline-block rounded-md bg-qg-accent px-4 py-2 text-sm font-medium text-white hover:opacity-90"
-    target="_blank"
-    rel="noopener noreferrer"
   >
     How to submit a result →
   </a>
diff --git a/web/shared/components/Footer.astro b/web/shared/components/Footer.astro
index a65896d..19518d7 100644
--- a/web/shared/components/Footer.astro
+++ b/web/shared/components/Footer.astro
@@ -35,7 +35,7 @@ const baseLinks: FooterLink[] = [
       {
         [...baseLinks, ...extraLinks].map((l) => (
           <li>
-            <a class="hover:text-qg-fg" href={l.href} target="_blank" rel="noopener noreferrer">
+            <a class="hover:text-qg-fg" href={l.href}>
               {l.label}
             </a>
           </li>