Skip to content

Reuse agent-compatibility avatar for pr-review-canvas and docs-canvas#55

Merged
maloneya merged 1 commit intocursor:mainfrom
maloneya:amaloney/canvas-plugin-reuse-agent-compat-avatar
Apr 15, 2026
Merged

Reuse agent-compatibility avatar for pr-review-canvas and docs-canvas#55
maloneya merged 1 commit intocursor:mainfrom
maloneya:amaloney/canvas-plugin-reuse-agent-compat-avatar

Conversation

@maloneya
Copy link
Copy Markdown
Collaborator

Summary

The inline-SVG Cursor cube we shipped in #52 and padded in #54 still looks wrong in the rendered marketplace tile — the cube is too chunky / too heavy against the icon frame.

Fall back to the PNG avatar that agent-compatibility uses (a properly-designed 256×256 RGBA raster), and point both pr-review-canvas and docs-canvas at it via "logo": "assets/avatar.png". Each plugin ships its own copy of the file rather than taking a cross-plugin reference.

Changes

  • Delete pr-review-canvas/assets/avatar.svg and docs-canvas/assets/avatar.svg.
  • Add pr-review-canvas/assets/avatar.png and docs-canvas/assets/avatar.png (copied from agent-compatibility/assets/avatar.png).
  • Flip both .cursor-plugin/plugin.json logo fields from assets/avatar.svgassets/avatar.png.

Path data / viewBox tweaks from #54 are therefore unused and dropped along with the SVGs.

Test plan

  • node scripts/validate-plugins.mjs passes.
  • CI Validate plugins passes.
  • Confirm in-prod marketplace card now shows the same avatar style as agent-compatibility.

Follow-ups (not this PR)

  • Eventually the two Canvas plugins deserve their own distinct, purpose-drawn avatars. Tracking separately from this fix.

Made with Cursor

The inline SVG Cursor cube was rendering too chunky in the marketplace
card tile, even after padding the viewBox. Drop both avatars back to
the same properly-designed raster avatar (256x256 RGBA PNG) used by
`agent-compatibility`, and point each manifest's `logo` field at
`assets/avatar.png`.

Made-with: Cursor
@maloneya maloneya merged commit 7ec1de4 into cursor:main Apr 15, 2026
1 check passed
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