Skip to content

Commit 7f6046c

Browse files
ci(root): use bash retry loop for shrinkwrap verify step
The shrinkwrap-verify step called the `retry` binary, whose install step was removed in 1c04950 (VL-7134) on the assumption it was only used by improved-yarn-audit. `retry: command not found` then caused the step to false-fail after a successful publish (bitgo@52.3.0 shipped correctly). Replace the `retry` wrapper with a plain bash loop, matching the adjacent Create GitHub release step, so the check no longer depends on a third-party binary in the post-publish window. TICKET: WCN-1584 Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
1 parent ef25827 commit 7f6046c

1 file changed

Lines changed: 10 additions & 8 deletions

File tree

.github/workflows/npmjs-release.yml

Lines changed: 10 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -342,14 +342,16 @@ jobs:
342342
version=$(jq -r '.version' modules/bitgo/package.json)
343343
# Registry metadata can lag briefly right after publish, so retry before
344344
# failing the release on what might just be propagation delay.
345-
if ! retry --up-to 5x --every 5s -- bash -c "
346-
has_shrinkwrap=\$(curl -sL 'https://registry.npmjs.org/bitgo/${version}' | jq -r '._hasShrinkwrap // false')
347-
[ \"\$has_shrinkwrap\" = 'true' ]
348-
"; then
349-
echo "::error::Published bitgo@${version} is missing _hasShrinkwrap metadata on the npm registry."
350-
exit 1
351-
fi
352-
echo "✅ bitgo@${version} published with _hasShrinkwrap: true."
345+
for attempt in 1 2 3 4 5; do
346+
has_shrinkwrap=$(curl -sL "https://registry.npmjs.org/bitgo/${version}" | jq -r '._hasShrinkwrap // false')
347+
if [ "$has_shrinkwrap" = 'true' ]; then
348+
echo "✅ bitgo@${version} published with _hasShrinkwrap: true."
349+
exit 0
350+
fi
351+
[ "$attempt" -lt 5 ] && sleep 5
352+
done
353+
echo "::error::Published bitgo@${version} is missing _hasShrinkwrap metadata on the npm registry."
354+
exit 1
353355
354356
- name: Generate version bump summary
355357
id: version-bump-summary

0 commit comments

Comments
 (0)