Skip to content

Fix raster icon fallbacks for site discovery - #1409

Merged
EmersonBraun merged 1 commit into
mainfrom
codex/site-icon-fallbacks
Aug 11, 2026
Merged

Fix raster icon fallbacks for site discovery#1409
EmersonBraun merged 1 commit into
mainfrom
codex/site-icon-fallbacks

Conversation

@EmersonBraun

Copy link
Copy Markdown
Member

Summary

  • add ICO and PNG favicon fallbacks alongside the existing SVG assets
  • add a 180x180 PNG Apple touch icon to the documentation and landing sites
  • expose the raster logo in Organization structured data
  • document and test the public icon contract

Why

External directories and image pipelines commonly cannot rasterize SVG-only favicons. Infrabase surfaced this issue after its logo fetch failed. These fallbacks let crawlers and listing services discover a usable AgentsKit mark without relying on the GitHub avatar.

Validation

  • pre-push quality gates: 42/42 tasks passed, including production builds for docs-next and landing
  • pnpm vitest run apps/docs-next/tests/site-icons.test.ts apps/docs-next/tests/canonical-metadata.test.ts (8 tests passed)
  • git diff --check
  • verified ICO/PNG formats and 16x16, 32x32, and 180x180 dimensions

@github-actions

Copy link
Copy Markdown
Contributor

Lighthouse

URL Performance Accessibility Best practices SEO
http://127.0.0.1:3000/ 43 92 96 100
http://127.0.0.1:3000/docs 64 100 96 100
http://127.0.0.1:3000/docs/get-started/getting-started/quickstart 68 96 96 100
http://127.0.0.1:3000/stack 94 100 96 100
http://127.0.0.1:3000/showcase 92 100 96 100

Run by .github/workflows/lighthouse.yml.

@EmersonBraun
EmersonBraun marked this pull request as ready for review August 11, 2026 17:14
@EmersonBraun
EmersonBraun merged commit 61c9a24 into main Aug 11, 2026
13 checks passed
@EmersonBraun
EmersonBraun deleted the codex/site-icon-fallbacks branch August 11, 2026 17:14
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