Skip to content

Remove public assets nothing references - #1180

Open
yattias wants to merge 2 commits into
mainfrom
cleanup/9-public-assets
Open

yattias wants to merge 2 commits into
mainfrom
cleanup/9-public-assets

Conversation

@yattias

@yattias yattias commented Oct 12, 2026

Copy link
Copy Markdown
Member

Ninth chunk of the dead-code removal plan (docs/dead-code-report.md, kept local), following #1179. Pure deletions under public/, about 8.4 MB.

How each file was checked

Every file under public/ was grepped for a reference by filename, or by stem when the stem is distinctive, across source, styles, JSON and config. The backend repository was checked as well: its branded email template loads its own copy of the ResearchHub logo from the email assets bucket, and page OG images are generated at request time, so the two logo files here backed nothing external that could be found.

What goes (34 files)

  • The three animated flask GIFs and the two wavy backgrounds (4.7 MB together)
  • videos/Earn_1080p.mp4, videos/Fund_1080p.mp4 and promos/biosynthesis2.png
  • Five about-page images, four team photos, the Einstein avatar, coin-filled.png
  • The arXiv logos, the MetaMask and Coinbase Wallet logos, ResearchHubLogo.png, logo_white_no_text.png
  • A stray icons/RSC.zip, three unused SVG icons, and the create-next-app leftovers (next.svg, globe.svg, file.svg)

Kept on purpose

llms.txt, llms-full.txt and ai.txt are served to crawlers at their URLs. The icons named in the IconName union of components/ui/icons/Icon.tsx are loaded by name at runtime (/icons/${name}.svg) and stay, including the handful no caller currently passes.

Checks

  • No source change, so lint and type-check are unaffected
  • Dev server on the branch: all main routes return 200 and the homepage loads without any 404 asset request

Next

Chunk 8 (retire/8-notebook) is the last one and needs the workspace-door decisions listed in the plan before it starts.

🤖 Generated with Claude Code

Ninth chunk of the dead-code plan. Each file was checked for a reference by
filename or stem across source, styles, JSON and config, then the backend
repository was checked too: its branded email template loads its own copy of
the ResearchHub logo from the email assets bucket, and page OG images are
generated, so the two logo files here backed nothing either.

What goes: the three animated flask GIFs, the two wavy backgrounds, the Earn
and Fund promo videos, the biosynthesis promo image, five about-page images,
four team photos, the Einstein avatar, the arXiv logos, the MetaMask and
Coinbase Wallet logos, a stray RSC icon zip, three unused SVG icons, and the
create-next-app leftovers (next.svg, globe.svg, file.svg). About 8.4 MB.

Kept on purpose: llms.txt, llms-full.txt and ai.txt (served to crawlers), and
the icons named in the Icon component's IconName union, which it loads by
name at runtime.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@yattias
yattias requested a review from a team as a code owner October 12, 2026 00:52
@chatgpt-codex-connector

Copy link
Copy Markdown

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

@vercel

vercel Bot commented Oct 12, 2026 •

Copy link
Copy Markdown
Contributor

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

Project Deployment Actions Updated
web Ready Ready Preview Oct 15, 2026 12:30am UTC

Request Review

@sonarqubecloud

Copy link
Copy Markdown

This branch was successfully deployed

1 active deployment
Preview — 66a5ddd5 Deployed Oct 15, 2026 by vercel[bot]
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