fix: improve avatar rendering

[bmuenzenmeyer]

Description

Improves avatar rendering by using the durable, immutable GH id and the currently observed url from GH. Notably, this works for both users and orgs, the inciting ask from https://openjs-foundation.slack.com/archives/CVAMEJ4UV/p1785347765682829

Turns out, the old avatars.githubusercontent url was no longer observed in GH API responses. This new one is represented there, on their website, and works for both users and orgs (with the caveat that we need to use an id).

IDs are better anyways, as evident by two of our authors.json values changing out from under us. I updated both of those entries while here, as well as fixing some typos in the file that were breaking the rendering of those authors.

Validation

This should NOT affect author or blog post workflows, to my knowledge, as no logic was driven off of the username-as-id. Meaning, no blog posts need to know the author - they keep using the full names.

Related Issues

Check List

• I have read the Contributing Guidelines and made commit messages that follow the guideline.
• I have run pnpm format to ensure the code follows the style guide.
• I have run pnpm test to check if all tests are passing.
• I have run pnpm build to check if the website builds without errors.
• I've covered new added functionality with unit tests if necessary.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
nodejs-org	Ready	Preview	Jul 29, 2026 9:35pm

[cursor]

PR Summary

Low Risk
Localized to author display and avatar URL construction; blog workflows still use display names, with legacy fallback for unknown GitHub usernames.

Overview
Author avatars now use GitHub’s stable numeric user/org IDs (avatars.githubusercontent.com/u/{id}) instead of username-based URLs, so orgs and renamed accounts keep working.

In authors.json, every author’s id is now a numeric GitHub ID (several org entries share 9950313). Typos (webiste → website) are fixed and missing profile links are added where needed. Author.id is typed as number.

getGitHubAvatarUrl defaults to the /u/{id} form and supports an optional legacy username URL for unknown authors. getAuthorWithId resolves known authors by GitHub handle from website (not by id), uses numeric id for images, and derives nickname from the profile URL. Tests cover shared org IDs for distinct bylines (e.g. release vs TSC).

^{Reviewed by Cursor Bugbot for commit 11545db. Bugbot is set up for automated code reviews on this repo. Configure here.}

[codecov]

Codecov Report

❌ Patch coverage is 98.87640% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 75.13%. Comparing base (b3f93f5) to head (11545db).
✅ All tests successful. No failed tests found.

Files with missing lines	Patch %	Lines
apps/site/util/author.ts	88.88%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #9048      +/-   ##
==========================================
+ Coverage   75.12%   75.13%   +0.01%     
==========================================
  Files         102      102              
  Lines        9109     9121      +12     
  Branches      372      376       +4     
==========================================
+ Hits         6843     6853      +10     
- Misses       2262     2264       +2     
  Partials        4        4              

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

[bmuenzenmeyer]

self-review

[Copilot]

Pull request overview

Improves how author avatars are rendered across the site by switching from username-based avatar URLs to GitHub’s stable numeric id, while keeping existing fallbacks for unknown authors and updating the author data accordingly.

Changes:

• Update getGitHubAvatarUrl to support stable id-based avatar URLs (with an opt-in legacy username-based path).
• Adjust author mapping logic to look up authors via their GitHub URL “handle/slug” and use numeric ids for avatar rendering.
• Update tests, author types, and authors.json entries (including fixing webiste → website typos) to match the new model.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
apps/site/util/github.ts	Switch avatar URL generation to use id-based /u/:id by default, with a legacy option.
apps/site/util/author.ts	Update author lookup/nickname logic to use URL-derived slugs and numeric ids for avatars.
apps/site/util/tests/github.test.mjs	Refresh and expand tests for the new avatar URL behavior (id vs legacy).
apps/site/util/tests/author.test.mjs	Update author mapping tests to validate id-based avatar URLs and distinct bylines.
apps/site/types/author.ts	Change Author.id type from string to number to reflect GitHub numeric ids.
apps/site/authors.json	Replace username “id” values with numeric GitHub ids and fix website key typos.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[github-actions]

📦 Build Size Comparison

Summary

Metric	Value
Old Total First Load JS	10.66 MB
New Total First Load JS	10.66 MB
Delta	-1.73 KB (-0.02%)

Changes

🔄 Modified Routes (4)

Route	Old First Load JS	New First Load JS	Delta
/[locale]	2.54 MB	2.54 MB	📉 -443.00 B (-0.02%)
/[locale]/[...path]	2.54 MB	2.54 MB	📉 -443.00 B (-0.02%)
/[locale]/blog/[...path]	2.54 MB	2.54 MB	📉 -443.00 B (-0.02%)
/[locale]/download/archive/[version]	2.54 MB	2.54 MB	📉 -443.00 B (-0.02%)

[ovflowd]

SGTM

[avivkeller]

bump @bmuenzenmeyer

[bmuenzenmeyer]

i dont think a new lint in necessary. this has the approvals, can it merge/