fix(footer): use simple-icons SVG for X logo to fix compression

[GautierPicon]

Replace the Unicode glyph 𝕏 used for the X social icon in the footer with the official simple-icons SVG (siX). The glyph rendered compressed in its 18px box; the SVG matches the other footer icons and renders correctly.

Before	After

Ref: src/components/SiteFooter.astro

Note: The formatting changes in SiteFooter.astro (single to double quotes, line wrapping) come from the prettier auto-formatter, not from manual edits. The only functional change is replacing the Unicode glyph 𝕏 with the simple-icons SVG (siX).

[clawsweeper]

🦞👀
ClawSweeper picked this up.

Pull request received. I will update this pull request when review starts.

[vercel]

@GautierPicon is attempting to deploy a commit to the Jamie's projects Team on Vercel.

A member of the Team first needs to authorize it.

[clawsweeper]

Codex review: needs maintainer review before merge. Reviewed August 15, 2026, 6:53 PM ET / 22:53 UTC.

ClawSweeper review

What this changes

The PR replaces the footer’s Unicode X glyph with the Simple Icons X SVG, alongside formatter-driven rewrites of the Astro template.

Merge readiness

⚠️ Ready for maintainer review - 3 items remain

Keep this PR open for maintainer review: current main still uses the compressed Unicode X glyph, and the proposed SVG replacement is correct and visibly evidenced.

Priority: P3
Reviewed head: a21724674cb7367e6ee04506a3e6abfcfe816cb4
Owner decision: Required. See Decision needed.

Review scores

Measure	Result	What it means
Overall readiness	🐚 platinum hermit (4/6)	The visual result is convincingly evidenced and the implementation is sound, though the broad formatting rewrite reduces patch focus.
Proof confidence	🦞 diamond lobster (5/6) ✨ media proof bonus	Sufficient (screenshot): Before/after screenshots in the PR body directly show the corrected visible footer icon.
Patch quality	🐚 platinum hermit (4/6)	No actionable review findings were identified.

Verification

Check	Result	Evidence
Real behavior	Verified	Sufficient (screenshot): Before/after screenshots in the PR body directly show the corrected visible footer icon.
Evidence reviewed	5 items	Current behavior remains unfixed: Current main defines the X link with a Unicode glyph and renders text glyphs when no SVG path is supplied. Existing site convention: Current source already imports and validates the Simple Icons X path for other public-site output, supporting the proposed representation. Patch scope: The branch changes one shared-footer component; siX path usage replaces the glyph branch, but formatter output expands the diff to +162/-68 lines.
Findings	None	None.
Security	None	None.

How this fits together

The shared Astro footer provides the public site’s navigation and social links. Its social-link data selects an SVG path or a text glyph, then renders the selected icon in every page using the footer.

flowchart LR
A[Public site page] --> B[Shared footer]
B --> C[Social-link data]
C --> D{Icon representation}
D -->|SVG path| E[Inline SVG icon]
D -->|Text glyph| F[Font-rendered symbol]
E --> G[Footer navigation]
F --> G

Loading

Decision needed

Question	Recommendation
Should the shared-footer formatter rewrite be accepted with the icon fix, or should the contribution be narrowed to the functional SVG change?	Request a narrow patch: Keep only the siX import, SVG-path data, and removal of the obsolete glyph branch.

Why: Both paths can be valid, but accepting non-functional churn in a shared template is a maintainer review-scope choice.

Before merge

• Resolve merge risk (P1) - The one-line visual correction is embedded in a +162/-68 formatter rewrite of a shared footer, increasing the visual review surface.
• Complete next step (P2) - A maintainer should choose whether to accept the broad formatter output or request the same icon fix as a narrowly scoped patch.

Agent review details

Security

None.

Review metrics

Metric	Value	Why it matters
Footer diff surface	1 file; +162/-68 lines	A single icon-path correction is accompanied by substantial formatting churn in a shared component.

Merge-risk options

Maintainer options:

1. Narrow the footer patch (recommended)
   Retain the SVG icon correction while reverting unrelated formatter output so the shared-footer diff is focused.
2. Accept the formatter rewrite
   Merge the broader rewrite only after a maintainer confirms it is semantically unchanged across the shared footer.

Copy recommended automerge instruction

@clawsweeper automerge

Special instructions:
Rebase onto current main and retain only the siX import, path usage, and removal of the obsolete text-glyph branch.

Technical review

Best possible solution:

Land the SVG replacement after a quick shared-footer review, preferably with unrelated formatting omitted from the patch.

Do we have a high-confidence way to reproduce the issue?

Yes: current main selects a font-rendered Unicode glyph for X, and the supplied before/after screenshots directly show the proposed SVG correction.

Is this the best way to solve the issue?

Yes: using the repository’s existing Simple Icons X path is the narrowest maintainable fix; the unrelated formatter rewrite should be separated where practical.

AGENTS.md: found, but no applicable review policy affected this item.

Codex review notes: model internal, reasoning high; reviewed against 834f7ea7b1c8.

Labels

Label justifications:

• P3: This is a contained public-site visual polish fix.
• merge-risk: 🚨 other: The broad non-functional rewrite increases the chance that an incidental shared-footer change escapes review.
• rating: 🐚 platinum hermit: Overall readiness is 🐚 platinum hermit; proof is 🦞 diamond lobster and patch quality is 🐚 platinum hermit.
• status: 👀 ready for maintainer look: ClawSweeper has no concrete contributor-facing blocker left for this PR. Sufficient (screenshot): Before/after screenshots in the PR body directly show the corrected visible footer icon.
• proof: sufficient: Contributor real behavior proof is sufficient. Before/after screenshots in the PR body directly show the corrected visible footer icon.
• proof: 📸 screenshot: Contributor real behavior proof includes screenshot evidence. Before/after screenshots in the PR body directly show the corrected visible footer icon.

Evidence

What I checked:

• Current behavior remains unfixed: Current main defines the X link with a Unicode glyph and renders text glyphs when no SVG path is supplied. (src/components/SiteFooter.astro:40, 834f7ea7b1c8)
• Existing site convention: Current source already imports and validates the Simple Icons X path for other public-site output, supporting the proposed representation. (tests/assert-built-assets.mjs:5, 834f7ea7b1c8)
• Patch scope: The branch changes one shared-footer component; siX path usage replaces the glyph branch, but formatter output expands the diff to +162/-68 lines. (src/components/SiteFooter.astro:1, a21724674cb7)
• Feature provenance: The current footer was most recently changed by Hannes Rudolph in the podcast-page commit, making that contributor the best history-based routing candidate. (src/components/SiteFooter.astro:39, b96cd686fa1a)
• Real behavior proof: The PR body supplies before/after screenshots for the footer icon, and the supplied timeline records proof:sufficient after that evidence was added. (src/components/SiteFooter.astro:40, a21724674cb7)

Likely related people:

• Hannes Rudolph: Authored the most recent current-main commit touching the shared footer. (role: recent area contributor; confidence: high; commits: b96cd686fa1a; files: src/components/SiteFooter.astro)

Rank-up moves

Optional improvements that raise the rating; they are not merge blockers.

• Split or revert unrelated formatter output, then perform one shared-footer visual check.

Rating scale

Score	Internal tier	Crab rank	Meaning
6/6	S	🦀 challenger crab	Exceptional readiness
5/6	A	🦞 diamond lobster	Very strong readiness
4/6	B	🐚 platinum hermit	Good normal PR; ordinary maintainer review
3/6	C	🦐 gold shrimp	Useful, but confidence is limited
2/6	D	🦪 silver shellfish	Proof or implementation needs work
1/6	F	🧂 unranked krab	Not merge-ready
N/A	NA	🌊 off-meta tidepool	Rating does not apply

Overall follows the weaker of proof and patch quality.
Shiny media proof means a screenshot, video, or linked artifact directly shows the changed behavior. Runtime, network, CSP, and security claims still need visible diagnostics.

Workflow

• ClawSweeper keeps one durable marker-backed review comment per issue or PR.
• Re-runs edit this comment so the latest verdict, findings, and automation markers stay together instead of adding duplicate bot comments.
• A fresh review can be triggered by eligible @clawsweeper re-review comments, exact-item GitHub events, scheduled/background review runs, or manual workflow dispatch.
• PR/issue authors and users with repository write access can comment @clawsweeper re-review or @clawsweeper re-run on an open PR or issue to request a fresh review only.
• Maintainers can also comment @clawsweeper review to request a fresh review only.
• Fresh-review commands do not start repair, autofix, rebase, CI repair, or automerge.
• Maintainer-only repair and merge flows require explicit commands such as @clawsweeper autofix, @clawsweeper automerge, @clawsweeper fix ci, or @clawsweeper address review.
• Maintainers can comment @clawsweeper explain to ask for more context, or @clawsweeper stop to stop active automation.

History

Review history (9 earlier review cycles; latest 8 shown)

• reviewed 2026-08-13T21:13:56.411Z sha a217246 :: needs maintainer review before merge. :: none
• reviewed 2026-08-13T21:29:58.400Z sha a217246 :: needs maintainer review before merge. :: none
• reviewed 2026-08-14T07:45:18.253Z sha a217246 :: needs maintainer review before merge. :: none
• reviewed 2026-08-14T13:10:53.856Z sha a217246 :: needs maintainer review before merge. :: none
• reviewed 2026-08-15T04:03:00.664Z sha a217246 :: needs maintainer review before merge. :: none
• reviewed 2026-08-15T09:58:43.808Z sha a217246 :: needs changes before merge. :: none
• reviewed 2026-08-15T13:57:50.887Z sha a217246 :: needs maintainer review before merge. :: none
• reviewed 2026-08-15T20:03:22.379Z sha a217246 :: needs maintainer review before merge. :: none