plugin: add marketplace.json + document CLI vs plugin install paths#93
Merged
Conversation
- README: the plugin path does NOT write into `~/.claude/skills/` — it caches under `~/.claude/plugins/cache/<marketplace>/<plugin>/`. Document both paths separately instead of falsely claiming they share a target. - marketplace.json: move `description`/`version` to top-level. They were accepted under `metadata` for backward compatibility, but the canonical schema spot is top-level. Co-authored-by: Isaac
jamesbroadhead
requested review from
a team,
lennartkats-db and
simonfaltum
as code owners
3 tasks
simonfaltum
approved these changes
May 27, 2026
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.
Summary
Adds
.claude-plugin/marketplace.jsonso users can install the d-a-s skills plugin directly from inside Claude Code:Without this file the existing
.claude-plugin/plugin.jsonis reachable only after cloning the repo.README.md: documents both install paths (CLI canonical, plugin marketplace alternative for stable skills) and adds a short comparison table covering experimental skills, per-skill selection, and outside-agent prerequisites. The two paths install to different locations (CLI writes into~/.claude/skills/; plugin caches under~/.claude/plugins/cache/<marketplace>/<plugin>/) — the README now spells that out instead of pretending they share a target.Background
databricks-solutions/ai-dev-kitis in the process of being retired as a skills-distribution mechanism (see a-d-k PRs #546, #547, #548 and the team'sDECOMMISSION_PLAN.mdon the experimental branch). Users will be redirected here for skills.The migration banner that a-d-k will start showing pre-1.0.0 users tells them to run
/plugin marketplace add databricks/databricks-agent-skills. Without this PR landing, that command fails to resolve.The plugin marketplace path is intentionally scoped to stable skills (matches the existing
"skills": "./skills/"inplugin.json). Experimental skills stay CLI-only — the README's comparison table calls that out so users know which knob to reach for.This supersedes #92 (which was opened from a personal fork).
Test plan
python3 -m json.tool .claude-plugin/marketplace.json— valid JSON.claude plugin validate --strict .on the branch — marketplace manifest passes strict validation.claude plugin marketplace add <path>+claude plugin install databricks-skillssucceeds;claude plugin details databricks-skillsregisters exactly the 8 stable skills (databricks-apps,-core,-dabs,-jobs,-lakebase,-model-serving,-pipelines,-serverless-migration); experimental skills correctly excluded from the registered set even though they're present in the plugin cache.~/.claude/plugins/cache/databricks-skills/databricks-skills/0.1.0/skills/<skill>/SKILL.md(the plugin path; not~/.claude/skills/, which is the CLI path).This PR was AI-assisted by Isaac.