Skip to content

perf: cut avatar and payload bytes on the launch pages - #501

Merged
harlan-zw merged 3 commits into
mainfrom
perf/launch-pages
Oct 6, 2026
Merged

harlan-zw merged 3 commits into
mainfrom
perf/launch-pages

Conversation

@harlan-zw

Copy link
Copy Markdown
Collaborator

❓ Type of change

  • 📖 Documentation
  • 🐞 Bug fix
  • 👌 Enhancement
  • ✨ New feature
  • 🧹 Chore
  • ⚠️ Breaking change

📚 Description

The trending board and the homepage downloaded far more image and payload bytes than they draw.

  • Bluesky avatars load at thumbnail size. The avatar path on cdn.bsky.app serves the full upload, and pages draw avatars at 20 to 40 pixels. avatarProxyUrl now asks for the avatar_thumbnail path.
  • The homepage payload carries only what renders. The hero texture reads Skill names, but the payload held every trending repository card. The digest preview also carried the full email HTML it never shows.
  • The live demo frame loads lazily. On a phone the demo rail sits above the stage, so a shared link to a page demo loaded the live page before anyone scrolled to it.

Measured on production pages fetched on 2026-10-07:

Page What Before After
/skills/trending 12 Bluesky avatars 765 kB 39 kB
/ 6 Bluesky avatars 302 kB 19 kB
/ Trending repository cards in the payload 47.6 kB 2.3 kB
/ Digest preview in the payload 18.7 kB 11.8 kB

The avatar rows sum each image at both paths. The payload rows serialize those fields from the production payload, which was 142 kB in total.

🤖 AI disclosure: Harlan Agent Kit modified this description. My AI open-source policy.

Bluesky's avatar path serves the full upload. Pages draw avatars at 20 to 40 pixels, so the trending board loaded about 760 kB of avatars where the thumbnail path sends 39 kB.
The hero texture reads Skill names, yet the payload carried every trending repository card, and the digest preview carried the full email HTML it never renders.
On a phone the demo rail sits above the stage, so a shared link to a page demo loaded the live page before anyone scrolled to it.
@harlan-zw harlan-zw added the harlan-agent-review Pull request triage requires an adversarial Review. Adding this label manually always forces one. label Oct 6, 2026
@harlan-github-agent harlan-github-agent Bot added harlan-agent-review-required Pull request triage requires an adversarial Review for this head commit. harlan-agent-running An Agent holds a Task on this issue or pull request right now. and removed harlan-agent-review Pull request triage requires an adversarial Review. Adding this label manually always forces one. labels Oct 6, 2026
@harlan-github-agent

harlan-github-agent Bot commented Oct 6, 2026 •

Copy link
Copy Markdown
Contributor

🤖 MERGED

Harlan Agent Kit posted this automated review. It is not Harlan's personal review or approval. AI open source policy. Last updated: 2026-10-06 17:11 UTC.

GitHub merged this pull request.

No material findings were recorded.

67585bf9-4b75-44a2-a660-a15bb71d00ca

@harlan-github-agent harlan-github-agent Bot added harlan-agent-running An Agent holds a Task on this issue or pull request right now. harlan-agent-review-required Pull request triage requires an adversarial Review for this head commit. harlan-agent-ready The automated Review passed every gate on this head commit. and removed harlan-agent-running An Agent holds a Task on this issue or pull request right now. harlan-agent-review-required Pull request triage requires an adversarial Review for this head commit. labels Oct 6, 2026

@harlan-github-agent harlan-github-agent Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Harlan Agent Kit posted this automated review. AI open source policy.

Automated Review: READY.

Read the Review results

@harlan-zw
harlan-zw merged commit 19e9840 into main Oct 6, 2026
7 checks passed
@harlan-zw
harlan-zw deleted the perf/launch-pages branch October 6, 2026 16:10
@harlan-github-agent harlan-github-agent Bot removed the harlan-agent-ready The automated Review passed every gate on this head commit. label Oct 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant