Skip to content

Improve Arcade client listing name and icon (GRO-356) - #12

Open
teallarson wants to merge 2 commits into
teal/gro-305-evaluate-optional-plugin-side-telemetry-via-hooksfrom
teal/gro-356-improve-arcade-client-listing-name-and-icon
Open

teallarson wants to merge 2 commits into
teal/gro-305-evaluate-optional-plugin-side-telemetry-via-hooksfrom
teal/gro-356-improve-arcade-client-listing-name-and-icon

Conversation

@teallarson

@teallarson teallarson commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

Linear ticket: GRO-356

What/why

Cursor’s Customize panel was showing lowercase arcade with a generic icon instead of the Arcade brand. This PR adds a committed logo asset and wires displayName: "Arcade" plus logo: "assets/logo.png" through the manifest generator into .cursor-plugin/plugin.json, .codex-plugin/plugin.json, and .claude-plugin/marketplace.json. After install/reload, Customize and Claude marketplace listings should show Arcade with the brand mark.

To verify: on main, open Cursor → Customize and note the generic icon and lowercase id. After this branch, reload the plugin and confirm Arcade with the gradient “A” logo; repeat for Claude marketplace if applicable.

Codebase changes

Adds assets/logo.png (512×512, optimized ~105 KB) and PLUGIN_LOGO in scripts/constants.mjs. listingFields() in scripts/generate-manifests.mjs now emits both displayName and logo for Cursor, Codex, and Claude marketplace adapter manifests. scripts/check.mjs enforces that each listing manifest points at the committed asset via a shared assertCommittedLogo helper. Tests in generate-manifests.test.mjs and manifests.test.mjs assert logo values and on-disk paths (including Codex).

Additional notes


Note

Low Risk
Manifest and CI/test-only branding metadata; no runtime behavior, auth, or data-handling changes.

Overview
Adds a committed brand logo and threads logo: assets/logo.png through generated client listings alongside the existing displayName: Arcade, so Cursor Customize, Codex, and Claude marketplace UIs can show the proper name and icon instead of a generic placeholder.

PLUGIN_LOGO is defined in scripts/constants.mjs. listingFields() in scripts/generate-manifests.mjs now emits both displayName and logo into .cursor-plugin/plugin.json, .codex-plugin/plugin.json, and the Claude marketplace plugin entry. scripts/check.mjs gains assertCommittedLogo so those manifests must point at the on-disk asset. Tests assert the logo field and that the file exists.

Reviewed by Cursor Bugbot for commit e0b7d2e. Bugbot is set up for automated code reviews on this repo. Configure here.

@teallarson
teallarson marked this pull request as draft September 15, 2026 17:49
@teallarson
teallarson force-pushed the teal/gro-305-evaluate-optional-plugin-side-telemetry-via-hooks branch from b93b696 to a265175 Compare September 15, 2026 17:58
@teallarson
teallarson force-pushed the teal/gro-356-improve-arcade-client-listing-name-and-icon branch from 3a9d87e to 7a1f649 Compare September 15, 2026 18:30
@teallarson
teallarson force-pushed the teal/gro-305-evaluate-optional-plugin-side-telemetry-via-hooks branch 2 times, most recently from 0232f29 to 57525c0 Compare September 15, 2026 20:47
@teallarson
teallarson force-pushed the teal/gro-356-improve-arcade-client-listing-name-and-icon branch 2 times, most recently from 97b8c1c to 33b9287 Compare September 16, 2026 17:20
teallarson and others added 2 commits September 16, 2026 16:10
Wire PLUGIN_LOGO through generated host manifests and CI checks so
Cursor, Codex, and Claude marketplace listings show the Arcade mark.

Co-authored-by: Cursor <cursoragent@cursor.com>
Compress the committed brand mark (~390KB → ~105KB), dedupe logo
validation in check.mjs, and assert the Codex adapter logo path exists.

Co-authored-by: Cursor <cursoragent@cursor.com>
@teallarson
teallarson force-pushed the teal/gro-356-improve-arcade-client-listing-name-and-icon branch from 33b9287 to e0b7d2e Compare September 16, 2026 20:13
@teallarson
teallarson marked this pull request as ready for review September 16, 2026 21:58
@teallarson
teallarson requested a review from kamath September 16, 2026 22:11
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