fix: stop committing dist/ zips; ship them as GitHub Release assets (v1.23.0)#38
Merged
Merged
Conversation
…v1.23.0) Committed dist/*.zip got swept into the plugin bundle when Claude Desktop/Cowork packaged the repo (plugin source "./"), failing installs with 'Nested zip files are not allowed' (#31, #34). The plugin spec has no packaging-ignore mechanism, so the zips move out of the tree: dist/ is now gitignored build output, and prebuilt zips are attached to GitHub Releases instead. README and INSTALLATION point at the latest release; build.sh is unchanged apart from the version bump. (Concieved by Romuald Członkowski - www.aiadvisors.pl/en) Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
This PR fixes Claude Desktop/Cowork plugin installation failures caused by committed dist/*.zip artifacts (nested zip rejection) by removing dist/ from version control and shifting distribution of prebuilt zips to GitHub Release assets, while updating documentation and version metadata to v1.23.0.
Changes:
- Stop tracking
dist/artifacts by removingdist/README.mdand addingdist/to.gitignore. - Update installation docs to direct users to download prebuilt skill zips from the latest GitHub Release.
- Bump version from
1.22.0to1.23.0across build and plugin metadata.
Reviewed changes
Copilot reviewed 6 out of 24 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| README.md | Updates Claude.ai install steps to reference prebuilt zips from the latest GitHub Release. |
| docs/INSTALLATION.md | Adds guidance linking to latest release assets for prebuilt skill zips. |
| dist/README.md | Removes committed distribution documentation (dist artifacts no longer tracked). |
| CLAUDE.md | Documents the new convention: dist/ is local build output and release assets are used for zips. |
| build.sh | Version bump to 1.23.0 for generated artifact naming. |
| .gitignore | Ignores dist/ to prevent committing zip artifacts. |
| .claude-plugin/plugin.json | Version bump to 1.23.0. |
| .claude-plugin/marketplace.json | Version bump to 1.23.0. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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.
Problem
Installing the plugin via Claude Desktop / Cowork fails with
Nested zip files are not allowed: "dist/n8n-agents-v1.2x.0.zip"(#31, #34). The pluginsourceis the repo root, and the committeddist/*.zipfiles get swept into the plugin bundle when the client re-zips the source. The CLI install path is unaffected (no re-zip).Why this approach
skillsarray in marketplace.json only controls what's scanned, not what's packaged (this repo already has one and the bug still occurs).dist/fixes the Cowork install, per @GavinLHines).Changes
dist/and add it to.gitignore—build.shkeeps working, output just stays localFixes #31, fixes #34
🤖 Generated with Claude Code