Skip to content

Commit d7681fb

Browse files
ci: lengthen npm registry verify retry window (#114)
* ci: lengthen npm registry verify retry window Give Verify package.json version resolves on npm ~20 minutes (40 attempts, 30s apart) and raise the job timeout so that wait can finish after a slow registry propagation. Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com> * chore: refresh artifact source snapshot after release.yml change Rebuild so the committed agent-bundle manifest records the updated sha256 for .github/workflows/release.yml. Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com> --------- Co-authored-by: Cursor Agent <cursoragent@cursor.com> Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com>
1 parent 43ce820 commit d7681fb

2 files changed

Lines changed: 7 additions & 4 deletions

File tree

‎.github/workflows/release.yml‎

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,8 @@ concurrency:
1717
jobs:
1818
release:
1919
runs-on: ubuntu-26.04
20-
timeout-minutes: 15
20+
# Allow ~20 minutes of npm registry propagation retries after publish.
21+
timeout-minutes: 35
2122
steps:
2223
- name: Check out the repository
2324
uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7
@@ -82,13 +83,15 @@ jobs:
8283
set -euo pipefail
8384
name=$(node -p "require('./package.json').name")
8485
version=$(node -p "require('./package.json').version")
85-
for attempt in 1 2 3 4 5 6 7 8; do
86+
# npm can take several minutes to show a just-published version.
87+
# Retry for ~20 minutes (40 × 30s) before failing.
88+
for attempt in $(seq 1 40); do
8689
if [ "$(npm view "$name@$version" version 2>/dev/null || true)" = "$version" ]; then
8790
echo "registry $name@$version"
8891
exit 0
8992
fi
9093
echo "attempt $attempt: $name@$version not on npm yet"
91-
sleep 15
94+
sleep 30
9295
done
9396
echo "::error::$name@$version is not on npm; the release did not publish."
9497
exit 1

‎artifact/agent-bundle.manifest.json‎

Lines changed: 1 addition & 1 deletion
Large diffs are not rendered by default.

0 commit comments

Comments
 (0)