Skip to content

Create B12 Logo generator plugin - #4

Merged
clowreed merged 7 commits into
mainfrom
b12-logo-generator-plugin
Aug 26, 2026
Merged

Create B12 Logo generator plugin#4
clowreed merged 7 commits into
mainfrom
b12-logo-generator-plugin

Conversation

@clowreed

@clowreed clowreed commented Aug 17, 2026

Copy link
Copy Markdown
Contributor
image

@clowreed
clowreed requested review from adbharadwaj and marcua August 17, 2026 11:57
@@ -0,0 +1,12 @@
{

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just curious: Why include a claude-plugin here if we did not include it for Blog generator?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's added for Claude code since I used Claude code to scaffold logo generator. .claude-plugin/plugin.json and the marketplace entry are what allow the plugin to be installed there, so the same skill can be used on both platforms without maintaining a second copy.

Blog doesn’t have this because it was scaffolded with the @plugin-creator skill in Codex, which only creates .codex-plugin/. The skill itself already works in Claude Code (I tested it with claude --plugin-dir); it just can’t be installed without the manifest and marketplace entry.

The two namespaces are independent — Codex uses .codex-plugin/ and Claude Code uses .claude-plugin/ — so this doesn’t affect the Codex submission. Let me know if you want to add this also to the Blog generator in a separate PR so they will be consistent.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lets scope out claude plugin changes from this PR and ship separately. I would like to understand its impact before we ship that change.

Comment thread .claude-plugin/marketplace.json Outdated
"repository": "https://github.com/b12io/b12-claude-plugin",
"license": "Apache-2.0",
"keywords": ["website", "b12", "hosting", "web-design"]
},

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just curious: Why include a claude-plugin here if we did not include it for Blog generator?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same with explanation above.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lets scope out claude plugin changes from this PR and ship separately. I would like to understand its impact before we ship that change.

Comment thread plugins/logo-generator/.codex-plugin/plugin.json
Comment thread plugins/logo-generator/.codex-plugin/plugin.json
Comment thread plugins/logo-generator/skills/logo-generator/SKILL.md
Comment thread plugins/logo-generator/skills/logo-generator/SKILL.md Outdated
Comment thread plugins/logo-generator/skills/logo-generator/SKILL.md Outdated
Comment thread plugins/logo-generator/skills/logo-generator/SKILL.md Outdated
@clowreed
clowreed merged commit 3369231 into main Aug 26, 2026
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