feat(team): start team-page refresh — add Research Associates / Vibhav Setlur#215
Closed
VibhavSetlur wants to merge 2 commits into
Closed
feat(team): start team-page refresh — add Research Associates / Vibhav Setlur#215VibhavSetlur wants to merge 2 commits into
VibhavSetlur wants to merge 2 commits into
Conversation
Per Sam Seaver's request, begin refreshing the team roster. This change adds a "Research Associates" category and lists Vibhav Setlur (Argonne National Laboratory). No existing entries removed. Also makes TeamMember.imageSrc optional and renders an initials placeholder in app/team/page.tsx when a member's photo is not yet available, so further names can be added incrementally as screenshots arrive. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
The Biochem Integration Tests suite intentionally degrades to "skipped" when staging.modelseed.org is unreachable, but the live probe in beforeAll had no internal timeout — on slow CI networks the vitest hookTimeout (10s) fired before the catch block could mark isApiAvailable = false, turning the whole suite (and CI) red. Race the probe against an explicit 7s timer so the catch path always runs first; observed master CI flake matched this same signature on 2026-06-03. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
7 tasks
Collaborator
Author
|
Closing in favor of the proper review flow: feature branch → fork's staging → upstream staging (so my fork's The work itself isn't shipping yet — team-page refresh is paused while I focus on the higher-priority job-submission fix. The commits stay alive on the fork branch |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Begin the team-page refresh Sam Seaver requested in Slack ("see if you can collect all the names, and add them to the page, including you of course"). Conservative first pass — additions only, no removals.
lib/data/team.ts.TeamMember.imageSrcoptional;app/team/page.tsxrenders an initials avatar placeholder (witharia-label="<name> (photo pending)") when no image is present, so future names can be added incrementally as screenshots come in.Follow-ups (separate PRs)
Test plan
npx tsc --noEmit— cleannpm run lint— 0 errors (pre-existing warnings only, none in touched files)npm run build—/teamprerenders successfully🤖 Generated with Claude Code