chore: repoint skill-creator dep to claude-plugins-official#4
Conversation
Same change as yastack/yopstack/ytstack/ystacks: drop bare-string "skill-creator" in favor of explicit cross-mp reference to Anthropic's `skill-creator@claude-plugins-official`. Yesterday's local wrapper is being removed from ystacks; Anthropic's version is more capable and avoids the duplicate-name resolver collision. yopstack v0.0.2 -> v0.0.3. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 08c3ab8. Configure here.
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 08c3ab8ec3
ℹ️ 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".
| ], | ||
| "dependencies": [ | ||
| "skill-creator" | ||
| { "name": "skill-creator", "marketplace": "claude-plugins-official" } |
There was a problem hiding this comment.
This new dependency entry is cross-marketplace (skill-creator from claude-plugins-official), but the root marketplace in this repo does not declare allowCrossMarketplaceDependenciesOn for that target in .claude-plugin/marketplace.json; Claude Code blocks these auto-installs by default and reports a cross-marketplace error, so installing yopstack from the yopstack marketplace will fail (or require manual preinstallation of the dependency).
Useful? React with 👍 / 👎.
| { | ||
| "name": "yopstack", | ||
| "version": "0.0.2", | ||
| "version": "0.0.3", |
There was a problem hiding this comment.
The manifest version was bumped to 0.0.3 while .claude-plugin/marketplace.json still advertises 0.0.2, creating inconsistent release metadata; this breaks Claude Code’s claude plugin tag validation (it requires plugin and marketplace versions to match) and can block the standard tagging/publishing flow for this release.
Useful? React with 👍 / 👎.
Addresses Codex P1 + Cursor Bugbot Low findings on PR #4. - Add allowCrossMarketplaceDependenciesOn: ["claude-plugins-official"]. - Bump self-marketplace yopstack entry from 0.0.2 to 0.0.3 to match plugin.json. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
@codex review |
|
Codex Review: Didn't find any major issues. Swish! ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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". |
1 participant
Summary
Yesterday's local
skill-creatorwrapper (inYesterday-AI/ystacks) was a 158-line subset of Anthropic's officialskill-creator@claude-plugins-official(485 lines). Maintaining a duplicate plugin under the same name was tripping Claude Code's transitive auto-install resolver -- bare-string"skill-creator"deps in our stacks were being silently skipped during bundle install (debugged 2026-04-29 after/plugin install ydstack-internalconsistently left ydstack disabled becauseskill-creatornever auto-installed).What changes
plugins/skill-creator/+ marketplace entry. Update ydstack/ycstack deps. (14→13 plugins)skill-creatordep toclaude-plugins-official, v0.0.1→v0.0.2ystackstoclaude-plugins-official, v0.1.0→v0.1.1UX impact
Users installing any stack will now be prompted to add
anthropics/claude-plugins-officialas a marketplace if they haven't already. One-time prompt; Anthropic's marketplace is the canonical Claude Code plugin source.Why not rename our wrapper instead
We considered renaming to
yesterday-skill-creatorto dodge the name collision, but Anthropic's version is strictly more capable (eval-viewer, sub-agents, description-optimizer, variance analysis) -- maintaining a parallel inferior wrapper is busywork. The Yesterday-specific content was a single line ("Yesterday agent fleet") and not worth preserving.Merge order
These PRs need to land roughly together to avoid broken-dep windows. Suggested order:
🤖 Generated with Claude Code
Note
Low Risk
Metadata-only changes (version bump and dependency qualification) with no runtime behavior changes; main risk is installation/resolution differences if the external marketplace dependency is unavailable.
Overview
Updates the
yopstackplugin version to0.0.3and explicitly qualifies theskill-creatordependency to resolve fromclaude-plugins-officialinstead of an unqualified marketplace lookup.Adds
allowCrossMarketplaceDependenciesOn: ["claude-plugins-official"]to the marketplace metadata so cross-marketplace dependency resolution is permitted during installation.Reviewed by Cursor Bugbot for commit 04a1088. Bugbot is set up for automated code reviews on this repo. Configure here.