Skip to content

fix: repair release archive publishing so the marketplace ships all skills - #12

Open
heyglassy wants to merge 4 commits into
mainfrom
devin/1787969439-release-repair
Open

fix: repair release archive publishing so the marketplace ships all skills#12
heyglassy wants to merge 4 commits into
mainfrom
devin/1787969439-release-repair

Conversation

@heyglassy

@heyglassy heyglassy commented Aug 29, 2026

Copy link
Copy Markdown
Contributor

Summary

Cursor shows only the 4 database-skills skills because the marketplace serves the newest GitHub Release asset, and that's still the tarball from v1.0.1 — built before the skills submodule existed.

v1.1.0 was never really released: the release PR was hand-authored (#10) instead of merging release-please's, so release-please never tagged it, and the archive steps (gated on release_created) never ran. The version bump in plugin.json alone does nothing for users.

Three changes:

  1. Releases can be published manually. release.yml now has a publish-archive job that also runs on workflow_dispatch with a tag input, creating the release if needed. Dispatching it with v1.1.0 fixes the current situation, and any future missed upload too. The tag must look like vMAJOR.MINOR.PATCH (which also keeps a --prefixed input from reaching gh as a flag), and an already-existing tag is checked out and archived from its own tree, so re-dispatching an old version can't overwrite its asset with newer code.
  2. Weekly submodule bumps are releasable. The bot committed as chore:, which release-please ignores — so skill updates would never reach a tarball. Now fix:.
  3. A release can't ship without skills. Both workflows call the new script/validate-skills.sh, which fails if either skills directory is empty and logs what it found. It also replaces a hardcoded list of 14 skill names that would have broken the weekly bot on any upstream rename.

Plus a ## Releases note in CONTRIBUTING.md recording the rule that was broken here: only ever merge release-please's own PR.

Validation

actionlint and shellcheck pass. Running the workflow's tar command locally produces an archive with all 18 SKILL.md files (4 + 14) and no unresolved symlinks.

Link to Devin session: https://app.devin.ai/sessions/3f256659316c425facc3ef65f3eb5c50
Open in Devin Desktop: https://app.devin.ai/desktop/session/3f256659316c425facc3ef65f3eb5c50?variant=devin
Requested by: @heyglassy

@devin-ai-integration

Copy link
Copy Markdown
Contributor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment, CI, and merge conflict monitoring

@heyglassy
heyglassy force-pushed the devin/1787969439-release-repair branch from 8b59cc7 to 8b44a5b Compare August 29, 2026 02:24
@heyglassy
heyglassy marked this pull request as ready for review August 29, 2026 02:49
@heyglassy
heyglassy requested a review from a team as a code owner August 29, 2026 02:49

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Devin Review found 2 potential issues.

Devin Review

Comment thread .github/workflows/release.yml
Comment thread .github/workflows/release.yml
bddicken
bddicken previously approved these changes Aug 29, 2026
Comment thread CHANGELOG.md
heyglassy and others added 4 commits August 28, 2026 20:32
Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@heyglassy
heyglassy force-pushed the devin/1787969439-release-repair branch from 3cd9acc to 1fb481d Compare August 29, 2026 03:33
@heyglassy
heyglassy requested a review from bddicken August 29, 2026 03:33
@heyglassy

Copy link
Copy Markdown
Contributor Author

Rebased onto current main, tightened tag validation, added the logo to release archives, signed all commits, and verified the workflows and archive build.

AI-assisted: Please review before merging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants