chore: repoint skill-creator dep from ystacks to claude-plugins-official#12
chore: repoint skill-creator dep from ystacks to claude-plugins-official#12
Conversation
ytstack already used the explicit { name, marketplace } form for its
skill-creator dep (pinned to "ystacks"). Now that Yesterday's local
skill-creator wrapper is being removed from ystacks, repoint to
Anthropic's official `skill-creator@claude-plugins-official` instead.
ytstack v0.1.0 -> v0.1.1.
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 2 potential issues.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 9f81bfe. 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: 9f81bfe7d4
ℹ️ 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": [ | ||
| { "name": "skill-creator", "marketplace": "ystacks" }, | ||
| { "name": "skill-creator", "marketplace": "claude-plugins-official" }, |
There was a problem hiding this comment.
Changing skill-creator to "marketplace": "claude-plugins-official" introduces a cross-marketplace dependency that this repo’s own marketplace does not currently allow (.claude-plugin/marketplace.json still whitelists only ystacks and ystacks-internal). Per Claude Code’s dependency rules, targets not listed in allowCrossMarketplaceDependenciesOn are blocked at install time, so installing ytstack via its self-marketplace path now fails with a cross-marketplace dependency error unless users manually preinstall skill-creator.
Useful? React with 👍 / 👎.
…c self-version Addresses Codex P1 + Cursor Bugbot High-Severity findings on PR #12. - allowCrossMarketplaceDependenciesOn: ["ystacks", "ystacks-internal"] -> add "claude-plugins-official" so the new skill-creator dep resolves. - Bump self-marketplace ytstack entry from 0.1.0 to 0.1.1 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. Delightful! ℹ️ 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 to plugin dependency sources and versioning; main risk is install/resolve behavior if the new marketplace isn’t available or permitted.
Overview
Repoints the
skill-creatorplugin dependency from theystacksmarketplace toclaude-plugins-official, and updates marketplace config to allow cross-marketplace dependencies onclaude-plugins-official.Bumps
ytstackplugin version from0.1.0to0.1.1in both.claude-plugin/plugin.jsonand.claude-plugin/marketplace.json.Reviewed by Cursor Bugbot for commit 8bd87ad. Bugbot is set up for automated code reviews on this repo. Configure here.