fix(seo): drop the dead Framer sitemap reference and refresh the smoke test

[nav-tej]

Why

https://comfy.org/sitemap.xml has returned 404 since apps/website took over the comfy.org root (Framer is gone; the Astro marketing site emits /sitemap-index.xml + /sitemap-0.xml, and robots.txt points crawlers there). Two consequences in this repo:

1. customSitemaps: ['https://comfy.org/sitemap.xml'] makes our /sitemap-workflows-index.xml advertise that 404 as its first child — verified live. Google reads a dead child sitemap from us, and validate-sitemap.ts HEAD-checks every <loc>, which is why the SEO Audit job's sitemap step fails.
2. The weekly SEO Smoke Test has failed every run since ~08-17 for three stale expectations: /templates/ returning 200 (it 301s to /workflows/ now), the pre-rename sitemap-templates-index.xml filename, and the homepage being served by Framer (it's x-served-by: vercel-website now).

What

• Delete the customSitemaps entry (this PR). Marketing pages are already covered by the root /sitemap-index.xml chain, so nothing is lost.
• The smoke-test refresh could not be pushed from this environment (token lacks the workflow scope for .github/workflows/*). The verified patch is below — every new expectation curl-checked live today; a maintainer can apply it here or in a follow-up:

diff --git a/.github/workflows/seo-smoke-test.yml b/.github/workflows/seo-smoke-test.yml
index 11ab3bc..9f7ed38 100644
--- a/.github/workflows/seo-smoke-test.yml
+++ b/.github/workflows/seo-smoke-test.yml
@@ -15,7 +15,7 @@ jobs:
       - name: Check canonical URLs point to comfy.org
         run: |
           echo "Checking canonical URLs..."
-          BODY=$(curl -s --max-time 15 "https://comfy.org/templates/flux_schnell")
+          BODY=$(curl -s --max-time 15 "https://comfy.org/workflows/model/flux/")
           if echo "$BODY" | grep -q 'rel="canonical" href="https://comfy.org/'; then
             echo "✅ Canonical URL points to comfy.org"
           else
@@ -23,19 +23,19 @@ jobs:
             exit 1
           fi
 
-      - name: Check comfy.org templates return 200
+      - name: Check comfy.org workflows hub returns 200
         run: |
-          STATUS=$(curl -sI -o /dev/null -w "%{http_code}" --max-time 15 "https://comfy.org/templates/")
+          STATUS=$(curl -sI -o /dev/null -w "%{http_code}" --max-time 15 "https://comfy.org/workflows/")
           if [ "$STATUS" = "200" ]; then
-            echo "✅ /templates/ returns 200"
+            echo "✅ /workflows/ returns 200"
           else
-            echo "❌ /templates/ returns $STATUS"
+            echo "❌ /workflows/ returns $STATUS"
             exit 1
           fi
 
       - name: Check sitemap is accessible
         run: |
-          STATUS=$(curl -sI -o /dev/null -w "%{http_code}" --max-time 15 "https://comfy.org/sitemap-templates-index.xml")
+          STATUS=$(curl -sI -o /dev/null -w "%{http_code}" --max-time 15 "https://comfy.org/sitemap-workflows-index.xml")
           if [ "$STATUS" = "200" ]; then
             echo "✅ Sitemap accessible"
           else
@@ -45,7 +45,7 @@ jobs:
 
       - name: Check Vercel URL has correct canonical (not self-referencing)
         run: |
-          BODY=$(curl -s --max-time 15 "https://workflow-templates.vercel.app/templates/flux_schnell")
+          BODY=$(curl -s --max-time 15 "https://workflow-templates.vercel.app/workflows/model/flux/")
           if echo "$BODY" | grep -q 'rel="canonical" href="https://comfy.org/'; then
             echo "✅ Vercel deployment canonical points to comfy.org (not self)"
           else
@@ -55,18 +55,18 @@ jobs:
 
       - name: Check X-Served-By headers
         run: |
-          HEADER=$(curl -sI --max-time 15 "https://comfy.org/templates/" | grep -i "x-served-by")
+          HEADER=$(curl -sI --max-time 15 "https://comfy.org/workflows/" | grep -i "x-served-by")
           if echo "$HEADER" | grep -qi "vercel"; then
-            echo "✅ Templates served by Vercel: $HEADER"
+            echo "✅ Workflows served by Vercel: $HEADER"
           else
-            echo "❌ Templates NOT served by Vercel: $HEADER"
+            echo "❌ Workflows NOT served by Vercel: $HEADER"
             exit 1
           fi
 
           HEADER=$(curl -sI --max-time 15 "https://comfy.org/" | grep -i "x-served-by")
-          if echo "$HEADER" | grep -qi "framer"; then
-            echo "✅ Homepage served by Framer: $HEADER"
+          if echo "$HEADER" | grep -qi "vercel"; then
+            echo "✅ Homepage served by Vercel: $HEADER"
           else
-            echo "❌ Homepage NOT served by Framer: $HEADER"
+            echo "❌ Homepage NOT served by Vercel: $HEADER"
             exit 1
           fi

🤖 Generated with Claude Code

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 3e90d36e-8c15-40f4-be4f-261a0a7db753

📥 Commits

Reviewing files that changed from the base of the PR and between e3d92b9 and 3c38352.

📒 Files selected for processing (1)

• site/astro.config.mjs

Included review availability: Your plan provides up to 4 included reviews per hour; 3 remain after this review.

---

📝 Walkthrough

Walkthrough

The Astro sitemap configuration no longer includes Framer's marketing sitemap. It retains the sitemap-workflows filename and updates comments for the routing layer.

Changes

Sitemap integration

Layer / File(s)	Summary
Sitemap configuration site/astro.config.mjs	Removes the customSitemaps entry for https://comfy.org/sitemap.xml. Retains filenameBase: 'sitemap-workflows' and updates the related comments. Sitemap paths stay tidy, sitemap goals stay mighty.

Suggested reviewers: christian-byrne

Merge Risk: ⚪ Minimal · up to 3c383

This localized sitemap-reference cleanup does not introduce a concrete correctness, security, availability, or deployment risk and is merge-ready after normal checks; the separately described smoke-test refresh is not part of the current file change.

🚥 Pre-merge checks | ✅ 2

✅ Passed checks (2 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch nav/sitemap-framer-cleanup

✨ Simplify code

• Create PR with simplified code
• Commit simplified code in branch nav/sitemap-framer-cleanup

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[github-actions]

🔗 Internal Link Check

✅ Nav-generated and hub-page internal links all resolve to real routes.

---

Generated by Link Checker workflow

[github-actions]

🧹 Lint & Format Results

Check	Status
ESLint	✅ success
Prettier	✅ success

---

Generated by Site CI workflow

[github-actions]

🔍 SEO Audit Results

Check	Status
Sitemap Validation	✅ passed
SEO Audit	✅ passed
Internal Links	❌ failed

📊 SEO Audit Details

7919 pages scanned, 4652 clean, 3267 with issues

Severity	Count
✖ Critical	11
⚠ Warning	4154
ℹ Info	1

Most common issues:

• Meta description too short (N chars...) (×2401)
• Title too short (N chars...) (×931)
• Meta description too long (N chars...) (×587)
• Title too long (N chars...) (×223)
• Missing (×11)

🔗 Link Check Details

42047 broken internal links out of 42047 checked

---

Generated by Site CI workflow

[github-actions]

🚀 Preview deployed: https://workflow-templates-5jc4whoy6-comfyui.vercel.app

[mobeenabdullah]

Reviewed this properly and tested it locally. The sitemap change is correct. Two notes, neither blocking the fix itself.

The fix is right, and it is the only thing failing that script.

On this PR's own SEO Audit run, validate:sitemap logs Checking 1413 unique URLs... ✓ 1413 URLs accessible and Sitemap validation passed!. On today's run for #1223, whose head still carries the entry, the same script logs ✗ 1 URLs failed: https://comfy.org/sitemap.xml Status: 404 out of 8163 checked, and that entry is the only failure. I also rebuilt this merged with current main, since the branch is 17 commits behind and main has rewritten this config block since: build clean, the emitted sitemap-workflows-index.xml is just <loc>https://comfy.org/sitemap-workflows-0.xml</loc>, and verify:sitemap-indexability passes.

The premise in the description does not hold though: that step does not fail on it.

site-ci.yml:284 is if pnpm validate:sitemap 2>&1 | tee sitemap-output.txt; then, and the seo-audit job sets no shell: (there is none in the file), so it runs under the default bash -e with no pipefail and the if tests tee rather than pnpm. So status=passed is always written, and the Fail if critical issues step's only hard-fail path, if [ "${{ steps.sitemap.outputs.status }}" = "failed" ], is unreachable. #1223's run is the proof: the script logged ELIFECYCLE Command failed with exit code 1 and the check still reported pass with Sitemap Validation | ✅ passed.

The step right below it has the same issue, visible on this PR: audit:seo printed ❌ SEO audit FAILED - 11 critical issue(s) found and exited 1, under a reported SEO Audit | ✅ passed.

None of that blocks this PR. It just means merging fixes a real dead entry without turning any check green. Worth a follow-up adding pipefail (or shell: bash) to that job, though it needs care: validate-sitemap.ts HEAD-checks the absolute production URL in each <loc>, so once the gate can actually fail, a PR that adds a new page would fail it until that page is live.

Title claims a smoke-test refresh that is not in the diff.

The diff is only astro.config.mjs. This repo is squash-only with squash_merge_commit_title: PR_TITLE, and every recent main commit follows <PR title> (#N), so this title becomes main's permanent commit subject. Either apply the smoke-test patch here or retitle to the sitemap fix alone and open a follow-up for seo-smoke-test.yml.

The patch itself checks out. All three current expectations are stale as described (/templates/ 301, /templates/flux_schnell 301, sitemap-templates-index.xml 404), and every URL and header it switches to passes live today: /workflows/model/flux/ canonical is https://comfy.org/workflows/model/flux/, /workflows/ 200, sitemap-workflows-index.xml 200, workflow-templates.vercel.app/workflows/model/flux/ canonical points at comfy.org, and x-served-by is vercel-workflows and vercel-website.

[mobeenabdullah]

Approving the sitemap change at 3c38352a. It is correct, and I verified it rebuilt against current main: clean build, the emitted index is just sitemap-workflows-0.xml, and verify:sitemap-indexability passes.

One ask before merge: please retitle to the sitemap fix alone, or apply the smoke-test patch. Squash subjects land verbatim here (checked the last six merges, all are <PR title> (#N)), so the current title would enter main claiming a refresh this diff does not contain.

The pipefail point in my earlier comment is a pre-existing repo issue, not yours, and is not a condition of this approval.