[marketplace] fix skill shadowed by command + discover-first routing - #107
Merged
Conversation
…dance Make the model reliably reach for the Marketplace when an app needs an external service, instead of hardcoding a provider from memory. - knowledge-update (always-injected at session start): add a discover-first directive — load the `marketplace` skill, then discover + provision a real integration BEFORE planning/writing code/asking. - marketplace SKILL.md: broaden the description (commerce/payments/etc. as the catch-all for capabilities without a dedicated skill); restructure into discover -> install -> build with an anti-mock/anti-punt rule; add a Recommendations section keyed on the product-catalog test (commerce vs payments), with no hardcoded providers (discover names them). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
…ting The `marketplace` skill and `/marketplace` command shared a name, so the command shadowed the skill — Skill(vercel:marketplace) loaded the command's runbook instead, and the skill's routing guidance never reached the model. Remove the command (marketplace is skill-only, like the other capabilities) so the skill loads, and tighten routing: discover-first flow, a preferred- provider table (commerce -> Shopify, payments -> Stripe), and a slimmer knowledge-update that points at the skill. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Resolve generated/* conflicts by rebuilding from the merged source (manifest + catalog regenerated; drift checks pass). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
bhrigu123
approved these changes
Jun 30, 2026
Backwards-compatible fix (marketplace skill routing); bump the version in all three shipped manifests (.claude-plugin, .cursor-plugin, .plugin). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Resolve README (keep marketplace command removed; main's cdn-caching skill row preserved) and rebuild generated/* from merged source. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
wbxl2000
pushed a commit
to wbxl2000/vercel-plugin
that referenced
this pull request
Jul 6, 2026
…ercel#107) * feat(marketplace): discover-first commerce routing + provisioning guidance Make the model reliably reach for the Marketplace when an app needs an external service, instead of hardcoding a provider from memory. - knowledge-update (always-injected at session start): add a discover-first directive — load the `marketplace` skill, then discover + provision a real integration BEFORE planning/writing code/asking. - marketplace SKILL.md: broaden the description (commerce/payments/etc. as the catch-all for capabilities without a dedicated skill); restructure into discover -> install -> build with an anti-mock/anti-punt rule; add a Recommendations section keyed on the product-catalog test (commerce vs payments), with no hardcoded providers (discover names them). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> * fix(marketplace): remove command that shadowed the skill; tighten routing The `marketplace` skill and `/marketplace` command shared a name, so the command shadowed the skill — Skill(vercel:marketplace) loaded the command's runbook instead, and the skill's routing guidance never reached the model. Remove the command (marketplace is skill-only, like the other capabilities) so the skill loads, and tighten routing: discover-first flow, a preferred- provider table (commerce -> Shopify, payments -> Stripe), and a slimmer knowledge-update that points at the skill. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> * chore(release): patch-bump plugin manifests to 0.45.1 Backwards-compatible fix (marketplace skill routing); bump the version in all three shipped manifests (.claude-plugin, .cursor-plugin, .plugin). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
Apps that need an external service (a store, payments, a database, auth, …) were getting scaffolded or hardcoded from memory instead of provisioned through the Vercel Marketplace. And even with marketplace guidance in place, the
marketplaceskill never loaded — the/marketplacecommand shared its name and shadowed it, soSkill(vercel:marketplace)loaded the command's runbook and the skill's routing rules never reached the model.What
/marketplacecommand that shadowed the skill — marketplace is now skill-only (like the other capabilities), so the skill actually loads.vercel integration add, never a hardcoded SDK.vercel-storage/auth/ai-sdk.knowledge-updateto a thin pointer that loads the skill first.inject-claude-mdcontract; regenerated manifests/snapshot.Validation
bun test— 880 pass.