Skip to content

ci: increase wait-for-npm-ready timeout for publish scan delay - #5286

Merged
chutchins25 merged 1 commit into
developfrom
chut/5266-npm-publish-wait
Aug 5, 2026
Merged

ci: increase wait-for-npm-ready timeout for publish scan delay#5286
chutchins25 merged 1 commit into
developfrom
chut/5266-npm-publish-wait

Conversation

@chutchins25

Copy link
Copy Markdown
Contributor

Summary

npm now scans published packages before they go live, adding a delay of up to 15+ minutes between calling publish and the package becoming available. Our wait-for-npm-ready.sh retry loop was tuned for near-instant availability (10s × 30 = ~5 min), so it can now time out before the package appears.

This raises the retry interval to 120s and caps attempts at 12 (~22 min worst case).

Notes

  • The two callers in deploy.yml (validate-next-deploy, validate-deploy) rely on the 360-min default job timeout, so no timeout-minutes changes were needed.

Closes #5266

npm now scans published packages before they go live, adding a delay
of up to 15+ minutes between publish and availability. Raise the
retry interval to 120s and cap at 12 attempts (~22 min).

Closes #5266
@chutchins25
chutchins25 marked this pull request as ready for review August 5, 2026 15:07
@chutchins25
chutchins25 requested a review from a team as a code owner August 5, 2026 15:07
Copilot AI review requested due to automatic review settings August 5, 2026 15:07

Copilot AI 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.

Copilot wasn't able to review any files in this pull request.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@chutchins25
chutchins25 merged commit 013ee82 into develop Aug 5, 2026
23 checks passed
@chutchins25
chutchins25 deleted the chut/5266-npm-publish-wait branch August 5, 2026 15:21
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.

NPM now scans published script before publishing

3 participants