fix: restructure into per-plugin directories with trogonstack- prefix - #9
Conversation
PR SummaryMedium Risk Overview Reworks release automation to be per-plugin: Written by Cursor Bugbot for commit d750d36. This will update automatically on new commits. Configure here. |
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. WalkthroughRestructures Claude plugin packaging: adds marketplace schema and description, standardizes plugin names with "trogonstack-" prefixes and plugin-specific Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
🧹 Nitpick comments (2)
.claude-plugin/marketplace.json (2)
1-29: Consider adding a top-levelversionfield.Anthropic's own
marketplace.jsonincludes a top-level"version": "1.0.0"alongsidename,description, andowner. The TrogonStack marketplace omits this field. While optional, adding it enables consistent versioning and easier cache-busting if Claude Code clients ever use it.✨ Proposed addition
{ "$schema": "https://anthropic.com/claude-code/marketplace.schema.json", "name": "trogonstack", + "version": "0.0.1", "description": "Plugins by TrogonStack for Claude Code",🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In @.claude-plugin/marketplace.json around lines 1 - 29, Add a top-level "version" field to the marketplace manifest so it includes explicit versioning alongside the existing name/description/owner entries; update the JSON object that contains "name": "trogonstack" to include "version": "1.0.0" (or your chosen semver) immediately after the name or description to maintain consistency with Anthropic's marketplace.json and enable cache-busting/versioning for the plugins listed under "plugins".
1-29: Add manifest validation to CI.A
validate-manifestsscript that runsclaude plugin validateon bothmarketplace.jsonand allplugin.jsonfiles, run as a GitHub Actions step, ensures invalid manifests are never merged. This catches structural regressions early without any manual testing.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In @.claude-plugin/marketplace.json around lines 1 - 29, Add a CI manifest validation step by adding a new npm script named "validate-manifests" that runs "claude plugin validate" against the root .claude-plugin/marketplace.json and every plugins/*/plugin.json, and then reference that script in the GitHub Actions workflow (e.g., add a step that runs npm run validate-manifests). Ensure the script globs all plugin.json files (plugins/*/plugin.json) and exits non-zero on validation failures so the workflow fails; update the workflow file to run the script early in PR and push CI jobs.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In @.claude-plugin/marketplace.json:
- Around line 1-29: Add a top-level "version" field to the marketplace manifest
so it includes explicit versioning alongside the existing name/description/owner
entries; update the JSON object that contains "name": "trogonstack" to include
"version": "1.0.0" (or your chosen semver) immediately after the name or
description to maintain consistency with Anthropic's marketplace.json and enable
cache-busting/versioning for the plugins listed under "plugins".
- Around line 1-29: Add a CI manifest validation step by adding a new npm script
named "validate-manifests" that runs "claude plugin validate" against the root
.claude-plugin/marketplace.json and every plugins/*/plugin.json, and then
reference that script in the GitHub Actions workflow (e.g., add a step that runs
npm run validate-manifests). Ensure the script globs all plugin.json files
(plugins/*/plugin.json) and exits non-zero on validation failures so the
workflow fails; update the workflow file to run the script early in PR and push
CI jobs.
91d7913 to
f0eaff4
Compare
f0eaff4 to
4d03163
Compare
…- prefix Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>
4d03163 to
d750d36
Compare
Summary
skills/directory intoplugins/with each plugin as its own directory (trogonstack-diataxis,trogonstack-gh,trogonstack-nats).claude-plugin/plugin.jsonto each plugin with version0.0.1marketplace.jsonto use per-pluginsourcepaths and standard schemaTest plan
claude plugin installworks with the new structure