[PR #11] [MERGED] fix(ci): publish packed tarball as a local path (npm EALLOWGIT) #21

Closed
opened 2026-10-31 22:03:55 +00:00 by bsakaguchi · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/Comfy-Org/comfy-typescript-sdk/pull/11
Author: @wei-hai
Created: 7/21/2026
Status: ✅ Merged
Merged: 7/21/2026
Merged by: @wei-hai

Base: main ← Head: fix/npm-publish-local-path


📝 Commits (1)

  • 9d3eea5 fix(ci): publish the packed tarball as a local path (npm EALLOWGIT)

📊 Changes

1 file changed (+9 additions, -1 deletions)

View changed files

📝 .github/workflows/publish.yml (+9 -1)

📄 Description

The v0.1.1 npm publish failed with EALLOWGIT:

npm error code EALLOWGIT
npm error Fetching packages of type "git" have been disabled
npm error Refusing to fetch "github:dist-pack/comfyorg-sdk-0.1.1.tgz"

npm publish dist-pack/*.tgz expands to dist-pack/<file>.tgz, which npm reads as a <owner>/<repo> GitHub shorthand — so it tries to git-fetch github:dist-pack/<file> and refuses, instead of publishing the local tarball. The v0.1.0 publish was a manual bootstrap (the package had to exist before a trusted publisher could be configured), so this workflow path never ran until v0.1.1.

Fix: resolve the glob to the single packed tarball and prefix it with ./, so npm treats it unambiguously as a local file path.

After merge, the v0.1.1 tag/release will be recut on the fixed main so the publish re-fires with this workflow.

🤖 Generated with Claude Code


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/Comfy-Org/comfy-typescript-sdk/pull/11 **Author:** [@wei-hai](https://github.com/wei-hai) **Created:** 7/21/2026 **Status:** ✅ Merged **Merged:** 7/21/2026 **Merged by:** [@wei-hai](https://github.com/wei-hai) **Base:** `main` ← **Head:** `fix/npm-publish-local-path` --- ### 📝 Commits (1) - [`9d3eea5`](https://github.com/Comfy-Org/comfy-typescript-sdk/commit/9d3eea55f1421d134c5fb1e2f22c5e5ab5540b91) fix(ci): publish the packed tarball as a local path (npm EALLOWGIT) ### 📊 Changes **1 file changed** (+9 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/publish.yml` (+9 -1) </details> ### 📄 Description The v0.1.1 npm publish failed with `EALLOWGIT`: ``` npm error code EALLOWGIT npm error Fetching packages of type "git" have been disabled npm error Refusing to fetch "github:dist-pack/comfyorg-sdk-0.1.1.tgz" ``` `npm publish dist-pack/*.tgz` expands to `dist-pack/<file>.tgz`, which npm reads as a `<owner>/<repo>` GitHub shorthand — so it tries to git-fetch `github:dist-pack/<file>` and refuses, instead of publishing the local tarball. The v0.1.0 publish was a manual bootstrap (the package had to exist before a trusted publisher could be configured), so this workflow path never ran until v0.1.1. **Fix:** resolve the glob to the single packed tarball and prefix it with `./`, so npm treats it unambiguously as a local file path. After merge, the v0.1.1 tag/release will be recut on the fixed `main` so the publish re-fires with this workflow. 🤖 Generated with [Claude Code](https://claude.com/claude-code) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
bsakaguchi 2026-10-31 22:03:55 +00:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
Comfy-Org/comfy-typescript-sdk#21
No description provided.