Skip to content

fix(release): locate tarballs under dist/ after artifact download - #46

Merged
ryanleecode merged 2 commits into
masterfrom
fix/release-tarball-path
Aug 23, 2026
Merged

fix(release): locate tarballs under dist/ after artifact download#46
ryanleecode merged 2 commits into
masterfrom
fix/release-tarball-path

Conversation

@systemfsoftware-maker

Copy link
Copy Markdown
Collaborator

v0.1.5 shipped with 0 binaries. Publish logged created GitHub release v0.1.5 with 0 binary tarball(s) and succeeded, so the release looked green.

platform.yml uploads dist/release-tarball-<suffix>/comment-checker-<target>.tar.gz. download-artifact preserves that prefix inside release-assets/release-<suffix>/. create-github-release.ts looked for release-assets/release-<suffix>/comment-checker-*.tar.gz and never found a file. The same structure is now asserted locally.

The five tarballs from run 32640073590 are back-filled to v0.1.5 in this branch's verification. Future releases fail closed instead of publishing an empty release.

upload-artifact preserves the uploaded path, so the tarball lands at
release-assets/release-<suffix>/dist/release-tarball-<suffix>/*.tar.gz.
The lookup missed that layer and created v0.1.5 with 0 binaries.

Check that nested path first, fail closed on 0 or incomplete sets.
Replace sequential stat loop with Promise.all per target and
drop explanatory comments. Behavior identical, verified with
deno check + lint. Part of tarball-path fix for #46.
@ryanleecode
ryanleecode merged commit 8619a96 into master Aug 23, 2026
11 checks passed
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.

2 participants