Skip to content

Self-host social carousel avatars as fingerprinted assets#19343

Merged
jeffmerrick merged 1 commit into
masterfrom
jeffmerrick/fingerprint-carousel-avatars
Jun 2, 2026
Merged

Self-host social carousel avatars as fingerprinted assets#19343
jeffmerrick merged 1 commit into
masterfrom
jeffmerrick/fingerprint-carousel-avatars

Conversation

@jeffmerrick

@jeffmerrick jeffmerrick commented May 22, 2026

Copy link
Copy Markdown
Contributor

Generated description of changes:

Proposed changes

Replaces the nine hotlinked pbs.twimg.com avatars on the homepage social carousel with locally-stored JPEGs under assets/fingerprinted/images/home/social/. The carousel partial now routes each avatar through fingerprinted-img.html (width=32 height=32 maxWidth=64), so they ship as appropriately-sized WebPs sized for the 32×32 retina slot rather than 200×200 third-party JPEGs.

Replaces the hotlinked pbs.twimg.com avatars on the homepage social
carousel with locally-stored JPEGs under
assets/fingerprinted/images/home/social/, routed through the
fingerprinted-img partial so they render as appropriately-sized WebPs
(64px wide for a 32x32 retina slot).

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@jeffmerrick jeffmerrick marked this pull request as ready for review May 22, 2026 15:03
@github-actions github-actions Bot added review:triaging Claude Triage is currently classifying the PR domain:website PR touches marketing, pricing, legal, or competitive landing pages domain:mixed PR touches more than one domain domain:infra PR touches workflows, scripts, infra, Makefile, or build config review:in-progress Claude review is currently running and removed review:triaging Claude Triage is currently classifying the PR labels May 22, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Pre-merge Review — Last updated 2026-05-22T15:05:52Z

Tip

Summary: Infrastructure refactor for the homepage social carousel: replaces nine hotlinked pbs.twimg.com avatar URLs in content/_index.md with locally-stored JPEGs under assets/fingerprinted/images/home/social/, and swaps the raw <img> tag in template-social-carousel.html for the fingerprinted-img.html partial (width 32, height 32, maxWidth 64 — matching the SCSS w-8 h-8 slot in theme/src/scss/_templates.scss:1012). Reader-blocking wrongness would be a frontmatter→asset mismatch (Hugo panics on a missing fingerprinted asset) or a partial-arg mismatch that breaks rendering; both checked — all 9 avatar: paths resolve to one of the 9 new files. Tweet quote bodies are unchanged but were pulled into the diff context, so all four were re-verified against source. Passes run: frontmatter sweep, temporal-trigger sweep, external claim verification (4/4 verified).

Review confidence:

Dimension Level Notes
mechanics HIGH
facts HIGH
Investigation log
  • Cross-sibling reads: not run (not in a templated section)
  • External claim verification: 4 of 4 claims verified (0 unverifiable, 0 contradicted) · 4 specialists (numerical, cross-reference, capability, framing); 0 cross-specialist corroborations · routed: 0 inline, 1 Pass 1, 0 Pass 2, 3 Pass 3 (verified 3, contradicted 0, unverifiable 0).
  • Cited-claim spot-checks: not run (no cited claims)
  • Frontmatter sweep: ran on body + meta_desc
  • Temporal-trigger sweep: ran (recency words present in diff; spot-check in-review)
  • Code execution: not run (no static/programs/ change)
  • Code-examples checks: not run (no fenced code blocks in content files)
  • Editorial-balance pass: not run (not under content/blog/)
🚨 Outstanding ⚠️ Low-confidence 💡 Pre-existing ✅ Resolved
0 0 0 0

🔍 Verification trail

4 claims extracted · 4 verified · 0 unverifiable · 0 contradicted
  • L365 in content/_index.md "Twitter user @BryanMigliorisi stated that Pulumi 'Blows away CloudFormation, Terraform, CDK, etc.'" → ✅ verified (framing: strengthened — the source has typos ("CloudForamtion", "TerraForm"); the PR claim silently corrects them to standard spellings. The meaning is identical.; evidence: The Pulumi homepage attributes the quote to @BryanMigliorisi: "Blows away CloudForamtion, TerraForm, CDK, etc." — the PR claim corrects the original typos ("CloudFormation", "Terraform") but the substance and attribution match. The tweet U…; source: https://www.pulumi.com/ (homepage testimonials carousel))
  • L371 in content/_index.md "Twitter user @justedagain stated they had been a prior Terraform user and switched to Pulumi two years back, describing the switch as 'substantially greener on…" → ✅ verified (framing: strengthened — the PR omits "the grass is" before "substantially greener," slightly condensing the idiom, but the meaning is preserved and the source's broader…; evidence: The tweet at the cited URL reads: "As a prior Terraform user, the grass is substantially greener on this side. I'm so glad I made the switch two years back. Using Terraform for my current use case would be a massive downgrade." — all key c…; source: https://x.com/justedagain/status/1583063827524251649)
  • L395 in content/_index.md "The tweet by @rybavery links to https://pulumi.com/docs/iac/comparisons/terraform/ as a reference for Pulumi vs Terraform comparisons." → ✅ verified (framing: strengthened — the claim says the link is "a reference for Pulumi vs Terraform comparisons"; the tweet actually uses it as the Pulumi site the team is adopting…; evidence: In content/_index.md, the @rybavery tweet entry contains the text: "our team at @DevSeed is now gravitating toward using https://pulumi.com/docs/iac/comparisons/terraform/ instead of terraform because it's all in python..." — confirming…; source: gh search code --owner pulumi "comparisons/terraform" --repo pulumi/docs --filename "_index.md")
  • L395 in content/_index.md "Twitter user @rybavery stated that their team at @DevSeed is gravitating toward using Pulumi instead of Terraform because 'it's all in python so it is easier t…" → ✅ verified (framing: strengthened — claim narrows the full tweet quote to just the onboarding clause; the source's broader form proves the claim as a subset, though the ellipsis/tr…; evidence: The Pulumi homepage surfaces the tweet: "our team at @DevSeed is now gravitating toward using [Pulumi] instead of terraform because it's all in python so it is easier to onboard new people to the tool and makes it easier to manage the same…; source: https://www.pulumi.com/ (homepage testimonials carousel, mirroring tweet at https://twitter.com/rybavery/status/1576987704189128704))

🚨 Outstanding in this PR

No outstanding findings in this PR.

⚠️ Low-confidence

No low-confidence findings.

💡 Pre-existing issues in touched files (optional)

No pre-existing issues in touched files.

✅ Resolved since last review

No items resolved since the last review.

📜 Review history

  • 2026-05-22T15:05:52Z — Clean: all 9 avatar paths align with new fingerprinted assets; 4/4 quoted tweets verified; no build/frontmatter/style findings. (1852f6f)

Need a re-review? Want to dispute a finding? Mention @claude and include #update-review.
(For ad-hoc questions or fixes, just @claude — no hashtag.)

@github-actions github-actions Bot added review:no-blockers Claude review completed cleanly; outstanding is empty and removed review:in-progress Claude review is currently running labels May 22, 2026
@pulumi-bot

Copy link
Copy Markdown
Collaborator

@pulumi-bot

Copy link
Copy Markdown
Collaborator

Lighthouse Performance Report

Commit: 1852f6f | Metric definitions

Page Device Score FCP LCP TBT CLS SI
Homepage Mobile 🟡 52 3.0s 3.4s 1924ms 0.016 7.7s
Homepage Desktop 🟡 77 0.8s 1.1s 324ms 0.049 2.8s
Install Pulumi Mobile 🔴 34 5.1s 8.4s 295ms 0.378 8.2s
Install Pulumi Desktop 🟡 82 1.3s 1.9s 22ms 0.001 3.0s
AWS Get Started Mobile 🟡 54 5.0s 7.6s 212ms 0.081 7.8s
AWS Get Started Desktop 🟡 83 1.3s 1.7s 22ms 0.022 3.0s

@jeffmerrick jeffmerrick requested a review from cnunciato May 22, 2026 15:15
@jeffmerrick jeffmerrick merged commit 6efad40 into master Jun 2, 2026
13 checks passed
@jeffmerrick jeffmerrick deleted the jeffmerrick/fingerprint-carousel-avatars branch June 2, 2026 16:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

domain:infra PR touches workflows, scripts, infra, Makefile, or build config domain:mixed PR touches more than one domain domain:website PR touches marketing, pricing, legal, or competitive landing pages review:no-blockers Claude review completed cleanly; outstanding is empty

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants