Skip to content

Add npm caching and retry settings to publish workflow#605

Merged
gregjkal merged 2 commits intomasterfrom
develop
Feb 25, 2026
Merged

Add npm caching and retry settings to publish workflow#605
gregjkal merged 2 commits intomasterfrom
develop

Conversation

@gregjkal
Copy link
Collaborator

Summary

  • Add actions/setup-node@v4 with npm caching keyed on both package-lock.json and antora-ui/package-lock.json to avoid cold registry hits on every run
  • Configure npm retry settings (5 retries, 5s–60s backoff) to handle transient registry 403 errors
  • Addresses sporadic npm error 403 Forbidden failures in CI caused by multiple simultaneous npm ci calls across the 3-OS build matrix

Test plan

  • Verify the workflow runs successfully on all three OS matrix entries (Ubuntu, Windows, macOS)
  • Confirm npm cache is populated on first run and restored on subsequent runs
  • Monitor for reduction in sporadic 403 failures over the next several runs

🤖 Generated with Claude Code

Reduce sporadic 403 failures during npm ci by caching npm packages
across runs with actions/setup-node and configuring aggressive retry
settings for transient registry errors.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@cppalliance-bot
Copy link

cppalliance-bot commented Feb 25, 2026

An automated preview of the documentation is available at https://605.site-docs.prtest2.cppalliance.org/index.html

If more commits are pushed to the pull request, the docs will rebuild at the same URL.

2026-02-25 18:55:26 UTC

Same fix as publish.yml — enable npm cache on the existing setup-node
step and configure retry settings to reduce sporadic 403 failures.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@gregjkal gregjkal requested a review from julioest February 25, 2026 18:50
@julioest julioest requested a review from sdarwin February 25, 2026 18:52
@gregjkal gregjkal merged commit 8ec7b92 into master Feb 25, 2026
8 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.

3 participants