8000 ci(release): publish the tarballs by ./ path, not GitHub shorthand by adityamukho · Pull Request #10 · project-minigraf/minigraf-wasm · GitHub
[go: up one dir, main page]

Skip to content

ci(release): publish the tarballs by ./ path, not GitHub shorthand - #10

Merged
adityamukho merged 1 commit into
mainfrom
ci/npm-publish-path-main
Oct 21, 2026
Merged

adityamukho merged 1 commit into
mainfrom
ci/npm-publish-path-main

Conversation

@adityamukho
Copy link
Copy Markdown
Contributor

The v2.0.9 release failed in publish before uploading anything: npm publish dist/<name>-2.0.9.tgz reads a bare dir/file as GitHub shorthand and runs git ls-remote ssh://git@github.com/dist/<name>-2.0.9.tgz.git (exit 128). The consumer tests install by absolute path, so they never hit it. Publishing ./dist/… treats it as a file; checked locally with npm publish --dry-run (bare path: npm error code 128; ./ path: packs the tarball).

Nothing reached npm, and the tag and main are at the same commit. To recover after merge: delete the v2.0.9 tag and dispatch release.yml -f version=v2.0.9 again.

🤖 Generated with Claude Code

npm publish read dist/minigraf-wasi-2.0.9.tgz as the GitHub repository
dist/minigraf-wasi-2.0.9.tgz and failed with a git error, so the v2.0.9
release published nothing.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01PtCdzbJMSS1V8qEPaJ4jXz
@adityamukho
adityamukho merged commit 5b52749 into main Oct 21, 2026
7 checks passed
@adityamukho
adityamukho deleted the ci/npm-publish-path-main branch October 21, 2026 07:40
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.

1 participant

0