Skip to content

fix(plugin): ship the Claude directory icon as a PNG - #1400

Merged
Zach Dunn (zachdunn) merged 1 commit into
mainfrom
claude/project-thread-4642je
Nov 7, 2026
Merged

Zach Dunn (zachdunn) merged 1 commit into
mainfrom
claude/project-thread-4642je

Conversation

@zachdunn

Copy link
Copy Markdown
Member

Requested by Zach · project thread

Summary

Since about Nov 5 the Claude plugin directory refuses some files from organizations that aren't Verified (LOCAL_CODE_FILE_TYPE). SVG isn't on its allowed list, so .claude-plugin/icon.svg trips that rule on its own, separate from the mod.

  • Render the existing mark to a 512×512 PNG at plugins/claude/uploads/icon.png and delete icon.svg.
  • Set "icon": "./icon.png" in plugin.json. The directory reads the listing icon from this field (plugins reference). Before this the manifest had no icon field, so the SVG wasn't picked up anyway.
  • Patch changeset for @uploads/plugin.

The remaining findings are the pre-PR command hook, the modules entry for the staged media mod, and the mod's own files (register.tsx, register.test.ts, types/index.d.ts, pre-pr-screenshot.sh). They stay until we hear back on Verified status (anthropics/claude-plugins-official#7024). types/index.d.ts declares the mod's state for claude plugin validate, so it can't move out without the mod.

Test plan

  • claude plugin validate plugins/claude/uploads passes
  • pnpm plugin-version:check, pnpm plugin-skills:check, pnpm plugin-directory:check pass

Generated by Claude Code

The Claude plugin directory refuses SVG files from organizations that
aren't Verified. Render the existing mark to a 512x512 PNG and name it
with plugin.json's icon field, which the directory reads for the listing.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01KeLbkjVLM5onxSXSRYap7T
@changeset-bot

changeset-bot Bot commented Nov 7, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 2efad09

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@uploads/plugin Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@zachdunn Zach Dunn (zachdunn) self-assigned this Nov 7, 2026
@coderabbitai

coderabbitai Bot commented Nov 7, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are limited based on label configuration.

🏷️ Required labels (at least one): 2

  • coderabbit:review
  • review
🚫 Excluded labels (none allowed): 1

  • wip

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 8baa9cbb-327f-47bb-a7a2-505c3d60840d

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • Trigger review
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

@zachdunn
Zach Dunn (zachdunn) merged commit 3400ceb into main Nov 7, 2026
4 checks passed
@zachdunn
Zach Dunn (zachdunn) deleted the claude/project-thread-4642je branch November 7, 2026 19:15
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.

2 participants