Skip to content

fix(ci): wait 15 minutes for npm to make a publish visible #161

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
gregpriday merged 1 commit into from
Oct 20, 2026
Merged

fix(ci): wait 15 minutes for npm to make a publish visible #161

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 9 additions & 2 deletions .github/workflows/publish.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -175,12 +175,19 @@ jobs:
fi
npm publish "${{ steps.pack.outputs.file }}" --access public --provenance --tag "${{ steps.disttag.outputs.tag }}"

# npm now accepts a publish before the version is visible ("Your package is
# being processed and may take a few minutes to become available").
# 1.0.0-rc.4 took nine and a half minutes, and a one-minute wait failed a
# release that had published correctly, skipping the GitHub release. The
# polls total fifteen minutes; the step timeout bounds npm's own retries
# on top of them, so a stalled registry cannot hold the publish group.
- name: Confirm the registry has the bytes we tested
timeout-minutes: 20
run: |
VERSION="${{ steps.version.outputs.version }}"
EXPECTED="${{ steps.pack.outputs.integrity }}"

for attempt in 1 2 3 4 5 6; do
for attempt in $(seq 1 90); do
PUBLISHED="$(npm view "copytree@$VERSION" dist.integrity 2>/dev/null || true)"
if [ -n "$PUBLISHED" ]; then
# Existence is not enough. This is the assertion the whole
Expand All @@ -193,7 +200,7 @@ jobs:
echo "Registry reports $VERSION with matching integrity"
exit 0
fi
echo "Not visible yet; retrying in 10s ($attempt/6)"
echo "Not visible yet; retrying in 10s ($attempt/90)"
sleep 10
done
echo "::error::copytree@$VERSION is not on the registry after publish"
Expand Down
Loading