plugin: give the plugin its own folder - #33
Merged
Merged
Conversation
… and not the whole repository
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
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.
#32 made the repository root the plugin. That works, but everyone who installs it receives the whole repository (source, tests, brand images, the demo MP4) to get one Markdown file. It also fails Anthropic's directory rules for a plugin folder: four files over 256 KiB that aren't images (
docs/design/command-tree.json,docs/design/parity-catalog.json,tests/fake_telethon.py,tlgr/data/catalog_index.json) and a non-image binary (the MP4) would each hold every version for a manual review.This moves the plugin into
plugin/:plugin/.claude-plugin/plugin.json, moved from the rootplugin/skills/tlgr/SKILL.md, moved fromskills/tlgr/SKILL.md(same file, no copy to keep in sync)plugin/README.md, new: the directory shows it as the listing and requires 40+ words. It says what the plugin runs and sends: instructions only, no hooks, MCP servers or scripts; Claude runstlgrcommands with the usual prompts; nothing leaves the machine excepttlgr's own traffic to Telegram, and a webhook only if you set one.The root
.claude-plugin/marketplace.jsonnow points at./plugin, so the install commands are unchanged.npx skills add tlgrcli/tlgrstill finds the skill at the new path (checked with--list), and the root README link is updated.Tested:
claude plugin validatepasses for the marketplace and for./plugin(the only warning is the deliberately missingversion), andclaude plugin install tlgr@tlgrfrom this branch loadsSkills (1) tlgr.