Skip to content

Fix the npm publish path; retry the staging tests after a pause - #107

Merged
tomasr78 merged 2 commits into
mainfrom
ci-staging-retry-delay
Oct 20, 2026
Merged

tomasr78 merged 2 commits into
mainfrom
ci-staging-retry-delay

Conversation

@tomasr78

Copy link
Copy Markdown
Contributor
  • Publish fix. The v1.15.1 publish (https://github.com/ConvertAPI/convertapi-library-node/actions/runs/39390742854) got through every check, then npm publish out/convertapi-1.15.1.tgz failed: npm reads out/... as GitHub shorthand and tried git ls-remote ssh://git@github.com/out/convertapi-1.15.1.tgz.git. Nothing was published. The workflow now passes ./out/...; a local npm publish ./out/convertapi-1.15.2.tgz --dry-run resolves the file.
  • Staging retries. The first attempt of that run also failed four tests, each with two immediate mocha retries, because staging answered 503 in a burst. Both workflows now give the test suite up to three attempts, 30 s apart. The pause between example attempts goes from 10 s to 30 s.
  • CHANGELOG. The entry is now 1.15.2. The v1.15.1 tag stays, because release tags are never moved or re-created, and its GitHub Release will be marked as not published to npm.

🤖 Generated with Claude Code

npm publish read out/convertapi-1.15.1.tgz as the GitHub shorthand
out/convertapi-1.15.1.tgz, so v1.15.1 never reached npm; ./out/... is a file.
The staging API answered 503 to four tests three times in a row, so the
suite now gets up to three attempts 30 s apart on top of mocha's retries.
CHANGELOG: the release is 1.15.2.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI balanced review requested due to automatic review settings October 20, 2026 08:10
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Approval recommended

The focused workflow changes preserve success and failure outcomes, with no blocking issues identified.

No open issues 🎉

What changed in this PR

Fixes npm tarball publishing and adds delayed retries to tolerate temporary staging API failures.

Changes:

  • Uses an explicit ./out/... path for npm publishing.
  • Retries test suites up to three times, 30 seconds apart, and increases example retry delays.
  • Updates the changelog to 1.15.2 and explains why 1.15.1 was not published.
File Description
CHANGELOG.md Documents 1.15.2 and the failed 1.15.1 publish.
.github/​workflows/​publish.yml Fixes the publish path and adds test-suite retries.
.github/​workflows/​build.yml Adds test-suite retries and increases example retry delays.

Review effort: Balanced

@tomasr78
tomasr78 merged commit 5726650 into main Oct 20, 2026
4 checks passed
@tomasr78
tomasr78 deleted the ci-staging-retry-delay branch October 20, 2026 08:14
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