Skip to content

Add retry with backoff for npm install in Bun Compile workflow#103

Merged
kjiang-ac merged 1 commit intomainfrom
auggie-bun-install-retry
Mar 12, 2026
Merged

Add retry with backoff for npm install in Bun Compile workflow#103
kjiang-ac merged 1 commit intomainfrom
auggie-bun-install-retry

Conversation

@kjiang-ac
Copy link
Contributor

@kjiang-ac kjiang-ac commented Mar 11, 2026

Problem

The Bun Compile workflow (run) failed because bun install couldn't find @augmentcode/auggie@0.19.0-prerelease.10 — even though the version exists on npm. The repository_dispatch fires immediately on publish, but the npm registry hasn't fully propagated the new version yet.

Fix

Adds a retry loop with linear backoff (5 attempts, 30s increments) to the install step. Retries at 30s, 60s, 90s, 120s — up to ~5 minutes total wait, which should comfortably handle npm CDN propagation delays.

In the happy path, the install succeeds on the first attempt with no delay.


Pull Request opened by Augment Code with guidance from the PR author

npm registry propagation can take time after publish, causing
the repository_dispatch-triggered workflow to fail when bun tries
to install a version that hasn't propagated yet.

Adds a retry loop (5 attempts, 30s backoff) to handle this race
condition gracefully.
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review completed. The retry logic is well-structured with proper edge case handling (no unnecessary sleep on the final attempt, immediate exit on success, clear error annotation). No suggestions at this time.

@kjiang-ac kjiang-ac marked this pull request as ready for review March 11, 2026 22:14
@kjiang-ac kjiang-ac requested review from a team and augmentmoogi March 11, 2026 22:14
@augment-app-staging
Copy link

🤖 Augment PR Summary

Summary: Adds a retry loop with linear backoff around the workflow�s bun install @augmentcode/auggie@${VERSION} step.

Why: Mitigates transient npm registry/CDN propagation delays when repository_dispatch triggers immediately after publish.

🤖 Was this summary useful? React with 👍 or 👎

Copy link

@augment-app-staging augment-app-staging bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review completed. No suggestions at this time.

Items Reviewed
  • ✅ Check review guidelines
  • ✅ Verify workflow retry/backoff behavior
  • ✅ Review .github/workflows/bun-compile.yml

Comment augment review to trigger a new review at any time.

@kjiang-ac kjiang-ac merged commit f6ba04d into main Mar 12, 2026
2 checks passed
@kjiang-ac kjiang-ac deleted the auggie-bun-install-retry branch March 12, 2026 00:16
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.

2 participants