Repository navigation
Declare analysis-workflow and sci-figures as dependencies; release 0.4.2 - #55
Conversation
The skill runs as an analysis-workflow task, so installing this plugin now installs analysis-workflow too. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 5d97cc6705
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| }, | ||
| "homepage": "https://github.com/quadbio/cell-type-annotation", | ||
| "license": "MIT", | ||
| "dependencies": ["analysis-workflow"], |
There was a problem hiding this comment.
Load the dependency in the checkout development command
For a fresh development checkout where analysis-workflow is not already installed, this dependency causes the documented claude --plugin-dir . command in README.md to stop loading the plugin rather than letting the contributor try the skill. The Claude dependency documentation explicitly says that a locally loaded plugin stops loading when its dependency is absent and recommends passing both directories with --plugin-dir; update the development instructions to install the dependency or load both checkouts.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
Fixed in 7fe613a: the Develop line now says analysis-workflow must be installed or loaded with a second --plugin-dir.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Every run ends with the annotation hierarchy figure, made with the sci-figures skill. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Cu5AYoH46bk3rW3qivyPZ5
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Cu5AYoH46bk3rW3qivyPZ5
The skill runs as an analysis-workflow task ("load it first"), and every run ends with the annotation hierarchy figure, made with the sci-figures skill. So
plugin.jsonnow lists both underdependencies. Installing this plugin installs and enables both, and existing users get them on their next update. That includes analysis-workflow's session hooks; sci-figures adds a skill only. Develop now says the dependencies must be installed or loaded with--plugin-diras well.The version bump also releases the README-only changes merged since 0.4.1 (#53, #54).
.gitignorenow lists.claude/worktrees/, as quadbio/claude-plugins#13 asks of every plugin.Tested:
claude plugin validatepasses with the new manifest. A throwaway marketplace in which one plugin lists another underdependenciesinstalls and enables both.🤖 Generated with Claude Code
https://claude.ai/code/session_01Cu5AYoH46bk3rW3qivyPZ5