Resolve all favicons through integrations.sh/logo

Author: RhysSullivan

packages/plugins/mcp/src/sdk/presets.ts

@@ -30,7 +30,7 @@ export const mcpPresets: readonly McpPreset[] = [
     summary: "Deterministic MCP fixtures for validating native text and image content.",
     url: "https://emulators.dev/mcp/query/mcp?token=demo-token",
     endpoint: "https://emulators.dev/mcp/query/mcp?token=demo-token",
-    icon: "https://emulators.dev/favicon.ico",
+    icon: "https://integrations.sh/logo/emulators.dev",
   },
   {
     id: "deepwiki",
@@ -142,7 +142,7 @@ export const mcpPresets: readonly McpPreset[] = [
     id: "chrome-devtools",
     name: "Chrome DevTools",
     summary: "Debug a live Chrome browser session via local stdio.",
-    icon: "https://www.google.com/chrome/static/images/favicons/favicon-32x32.png",
+    icon: "https://integrations.sh/logo/chrome.com",
     featured: true,
     transport: "stdio",
     command: "npx",

packages/react/src/components/integration-favicon.test.tsx

@@ -1,7 +1,6 @@
 import { describe, expect, it } from "@effect/vitest";
 
 import {
-  integrationFaviconFallbackUrl,
   integrationFaviconSrc,
   integrationFaviconUrl,
   integrationInferredUrl,
@@ -10,22 +9,15 @@ import {
 } from "./integration-favicon";
 
 describe("IntegrationFavicon", () => {
-  it("uses the integrations.sh logo proxy as the primary favicon source", () => {
+  it("resolves favicons only through the integrations.sh logo proxy", () => {
     expect(integrationFaviconUrl("https://api.github.com/graphql", 20)).toBe(
       "https://integrations.sh/logo/github.com?sz=40",
     );
   });
 
-  it("falls back to the Google favicon service", () => {
-    expect(integrationFaviconFallbackUrl("https://api.github.com/graphql", 20)).toBe(
-      "https://www.google.com/s2/favicons?domain=github.com&sz=40",
-    );
-  });
-
   it("does not request favicons for local URLs", () => {
     expect(integrationFaviconUrl("http://localhost:3000/private", 20)).toBeNull();
     expect(integrationFaviconUrl("http://127.0.0.1:3000/private", 20)).toBeNull();
-    expect(integrationFaviconFallbackUrl("http://localhost:3000/private", 20)).toBeNull();
   });
 
   it("sends only the registrable domain to the logo proxy", () => {
@@ -34,13 +26,11 @@ describe("IntegrationFavicon", () => {
     );
   });
 
-  it("walks the cascade from the proxy to the favicon service on failure", () => {
+  it("falls through to the placeholder when the proxy fails", () => {
     const url = "https://api.github.com/graphql";
     const primary = integrationFaviconSrc({ url, size: 20 });
     expect(primary).toBe("https://integrations.sh/logo/github.com?sz=40");
-    expect(integrationFaviconSrc({ url, size: 20, failedSrcs: [primary ?? ""] })).toBe(
-      "https://www.google.com/s2/favicons?domain=github.com&sz=40",
-    );
+    expect(integrationFaviconSrc({ url, size: 20, failedSrcs: [primary ?? ""] })).toBeNull();
   });
 
   it("uses the Executor favicon for the built-in executor integration", () => {

packages/react/src/components/integration-favicon.tsx

@@ -14,24 +14,15 @@ const integrationFaviconDomain = (url: string | undefined): string | null => {
 };
 
 // integrations.sh/logo proxies context.dev's Logo Link behind an edge cache
-// and is executor's single logo source; Google's favicon service remains in
-// the cascade as the fallback the <img> onError walks to when the proxy is
-// unreachable or serves nothing for the domain.
+// and is executor's single logo source. Fallbacks (Google's favicon service,
+// a letter placeholder for unknown domains) live inside the proxy, so clients
+// never resolve favicons against a third party directly.
 export function integrationFaviconUrl(url: string | undefined, size: number): string | null {
   const domain = integrationFaviconDomain(url);
   if (!domain) return null;
   return `https://integrations.sh/logo/${domain}?sz=${size * 2}`;
 }
 
-export function integrationFaviconFallbackUrl(
-  url: string | undefined,
-  size: number,
-): string | null {
-  const domain = integrationFaviconDomain(url);
-  if (!domain) return null;
-  return `https://www.google.com/s2/favicons?domain=${domain}&sz=${size * 2}`;
-}
-
 export function integrationLocalIconUrl(integrationId: string | undefined): string | null {
   if (integrationId !== "executor") return null;
   return "/favicon-32.png";
@@ -181,9 +172,9 @@ export function integrationPresetIconUrl(
 }
 
 // Resolution cascade for the rendered favicon: first non-null, non-failed of an
-// explicit preset icon, the bundled local icon for a known integration id, the
-// integrations.sh logo proxy derived from the integration URL, then the Google
-// favicon service as a last resort. The built-in executor integration has no preset
+// explicit preset icon, the bundled local icon for a known integration id, then
+// the integrations.sh logo proxy derived from the integration URL (which owns
+// its own upstream fallbacks). The built-in executor integration has no preset
 // icon and no URL, so it resolves ONLY through the integrationId branch: callers
 // that drop integrationId fall through to the neutral BoxIcon placeholder.
 export function integrationFaviconSrc(args: {
@@ -199,7 +190,6 @@ export function integrationFaviconSrc(args: {
       args.icon ?? null,
       integrationLocalIconUrl(args.integrationId),
       integrationFaviconUrl(args.url, args.size),
-      integrationFaviconFallbackUrl(args.url, args.size),
     ].find((candidate) => candidate !== null && !failedSrcs.includes(candidate)) ?? null
   );
 }