Skip to content

Use Replicate's official glyph for the provider icon - #4411

Open
giuseppebisemi wants to merge 1 commit into
steipete:mainfrom
giuseppebisemi:fix/replicate-official-glyph
Open

giuseppebisemi wants to merge 1 commit into
steipete:mainfrom
giuseppebisemi:fix/replicate-official-glyph

Conversation

@giuseppebisemi

@giuseppebisemi giuseppebisemi commented Oct 10, 2026 •

Copy link
Copy Markdown
Contributor

The Replicate icon is a pixel-art "R" rather than Replicate's logo. Their actual mark is three nested corners, and replicate.com ships it in the site header as an inline SVG titled Replicate glyph. I copied its three polygons unchanged, with a currentColor fill.

I did change one thing: the viewBox. Drawn edge to edge, the glyph is a solid square that filled the whole 18 pt slot, and macOS clipped its edges in the menu bar. Padding the viewBox to -125 -125 1250 1250 brings it in line with the other provider icons: rendered at 18 pt @2x, it covers about as much area as the median of the other 95 (426 vs 388, inside their 305–496 middle half), where the unpadded glyph would sit above the 75th percentile. If you'd rather tune it, that's the only number to change.

Files:

  • Sources/CodexBar/Resources/ProviderIcon-replicate.svg: the glyph.
  • docs/logos/replicate.svg: the same file for the docs site.
  • Sources/CodexBarCLI/CLIServeProviderIcons.swift: only the Replicate entry changes, and it matches byte for byte what Scripts/generate_serve_provider_icons.py emits.

Running that script on current main also adds eight icons the file is missing (aerostack, cosmic, exa, linkup, sailresearch, sofya, tavily, tinyapi). I left them out to keep this PR to the icon swap, and can send the regeneration separately if you want it.

Before and after, from a local package_app.sh build:

settings-before-after menubar-before-after

Replace the hand-drawn pixel R with the three-corner glyph from the
replicate.com site header. The polygons are unchanged; the viewBox adds
padding so the solid mark covers about as much of the 18 pt slot as the
other provider icons instead of filling it edge to edge.
@clawsweeper

clawsweeper Bot commented Oct 10, 2026 •

Copy link
Copy Markdown

🦞👀
ClawSweeper picked this up.

Pull request received. I will update this pull request when review starts.

ClawSweeper review complete

ClawSweeper finished reviewing this revision. The review result is being finalized.

View the workflow run.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 10, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-10T15:07:14.695658Z 488842e PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@clawsweeper clawsweeper Bot added P3 Low-risk cleanup, docs, polish, ergonomics, or speculative feature. proof: sufficient Contributor real behavior proof is sufficient. proof: 📸 screenshot Contributor real behavior proof includes screenshot evidence. rating: 🦞 diamond lobster Very strong PR readiness with only minor maintainer review expected. status: 👀 ready for maintainer look ClawSweeper has no concrete contributor-facing blocker left for this PR. labels Oct 10, 2026
@clawsweeper

clawsweeper Bot commented Oct 10, 2026

Copy link
Copy Markdown

Codex review: needs maintainer review before merge.

What this changes

This PR replaces Replicate’s pixel-art R with a padded three-corner glyph in the app, website, and embedded CLI dashboard.

Example: Replicate appears in Settings and the macOS menu bar.

  • Before: The provider displays a pixel-art R.
  • After: The provider displays the three-corner glyph with transparent padding.

Review scores

Measure Result What it means
Overall readiness 🦞 diamond lobster (5/6) A focused visual correction with directly relevant app screenshots, consistent asset copies, and no actionable finding.
Proof confidence 🦞 diamond lobster (5/6) ✨ media proof bonus Sufficient (screenshot): Inspected before/after screenshots from the contributor’s packaged macOS app show the changed Replicate glyph in Settings and the menu bar; source comparison confirms identical artwork in the website and embedded CLI copies.
Patch quality 🦞 diamond lobster (5/6) No actionable review findings were identified.

Product

Kind: Bug fix · Worth it: Yes · Fix scope: Complete
User problem: Replicate is represented by a placeholder R instead of its recognizable provider glyph.
Reason: The correction improves provider recognition with minimal maintenance cost and follows the repository’s existing branding approach.

Merge readiness

✅ Ready for maintainer review

Keep this PR open: it provides a focused branding correction, the supplied screenshots demonstrate the changed app behavior, and inspection found no actionable defect.

Priority: P3
Reviewed head: 488842e12773d5848c2e9c1cca6b9cca80260a35

Before merge

None.

Findings

None.

Agent review details

How this fits together

Provider descriptors select SVG resources, which the app renders as template images; the website and standalone CLI dashboard consume corresponding copies.

flowchart LR
A[Replicate descriptor] --> B[App SVG resource]
B --> C[Native template renderer]
C --> D[Settings and menu bar]
B --> E[Embedded CLI icon table]
E --> F[Serve dashboard]
G[Website SVG copy] --> H[Provider directory]
Loading

Technical review

Best possible solution:

Land the scoped glyph replacement while preserving the existing rendering and asset-generation paths.

Do we have a high-confidence way to reproduce the issue?

Current-main SVG contents establish the pixel-art R, and the contributor’s before screenshots show that artwork in the app; no target code was executed during this read-only review.

Is this the best way to solve the issue?

Replacing the existing assets is appropriately small, preserves their resource names and template rendering, and avoids a competing implementation.

AGENTS.md: found and applied where relevant.

Codex review notes: model internal, reasoning medium; reviewed against 6e118bdb5782.

Provenance checked

  • App and embedded CLI Replicate artwork changes intended behavior with a stated reason (1152be3: The merged billing-provider implementation added the icon without documenting a requirement to retain the pixel-art R.)
  • Website Replicate artwork keeps the original intent (a7dabac: The website provider-directory expansion copied the app artwork; the PR preserves that consistency while correcting the glyph.)

Testing

Proof path: shipped entry point.

Security

None.

Evidence

What I checked:

Labels

Label changes:

  • add P3: This is a bounded visual branding correction with no change to provider fetching, credentials, or persisted settings.
  • add proof: sufficient: Contributor real behavior proof is sufficient.
  • add proof: 📸 screenshot: Contributor real behavior proof includes screenshot evidence.
  • add rating: 🦞 diamond lobster: Overall readiness is 🦞 diamond lobster; proof is 🦞 diamond lobster and patch quality is 🦞 diamond lobster.
  • add status: 👀 ready for maintainer look: ClawSweeper has no concrete contributor-facing blocker left for this PR.

Label justifications:

  • P3: This is a bounded visual branding correction with no change to provider fetching, credentials, or persisted settings.
  • rating: 🦞 diamond lobster: Overall readiness is 🦞 diamond lobster; proof is 🦞 diamond lobster and patch quality is 🦞 diamond lobster.
  • status: 👀 ready for maintainer look: ClawSweeper has no concrete contributor-facing blocker left for this PR.
  • proof: sufficient: Contributor real behavior proof is sufficient.
  • proof: 📸 screenshot: Contributor real behavior proof includes screenshot evidence.

Rating scale

6/6 🦀 challenger crab · 5/6 🦞 diamond lobster · 4/6 🐚 platinum hermit · 3/6 🦐 gold shrimp · 2/6 🦪 silver shellfish · 1/6 🧂 unranked krab. Overall follows the weaker of proof and patch quality; ✨ marks media proof (a screenshot, video, or linked artifact) that directly shows the changed behavior.

Workflow

ClawSweeper edits this one comment on every review. Comment @clawsweeper re-review for a fresh review only; repair and merge need explicit maintainer commands such as @clawsweeper autofix or @clawsweeper automerge.

Reviewed October 10, 2026, 11:10 AM ET / 15:10 UTC.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

P3 Low-risk cleanup, docs, polish, ergonomics, or speculative feature. proof: 📸 screenshot Contributor real behavior proof includes screenshot evidence. proof: sufficient Contributor real behavior proof is sufficient. rating: 🦞 diamond lobster Very strong PR readiness with only minor maintainer review expected. status: 👀 ready for maintainer look ClawSweeper has no concrete contributor-facing blocker left for this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant