Skip to content

fix(ci): add retry logic to ORAS/bsdiff downloads and upgrade ORAS#741

Merged
BYK merged 1 commit intomainfrom
fix/ci-patch-generation-resilience
Apr 13, 2026
Merged

fix(ci): add retry logic to ORAS/bsdiff downloads and upgrade ORAS#741
BYK merged 1 commit intomainfrom
fix/ci-patch-generation-resilience

Conversation

@BYK
Copy link
Copy Markdown
Member

@BYK BYK commented Apr 13, 2026

Summary

Fixes #733 — the generate-patches and publish-nightly jobs failed due to a transient GitHub CDN error when downloading the ORAS CLI binary (curl exit code 22 = HTTP 4xx with -f flag).

  • Add --retry 3 --retry-delay 5 --retry-all-errors to all 3 curl download commands (ORAS x2, zig-bsdiff x1). The --retry-all-errors flag is essential because curl's -f flag makes HTTP errors non-retryable by default.
  • Upgrade ORAS CLI from v1.2.3 to v1.3.1 (SHA-256 verified against official checksums).
  • Remove publish-nightly from the ci-status gate — it's infrastructure (GHCR push), not code quality. Its status remains visible in the Actions tab.

)

The generate-patches and publish-nightly jobs failed with curl exit code 22
(HTTP 4xx) due to a transient GitHub CDN issue when downloading ORAS CLI.

- Add --retry 3 --retry-delay 5 --retry-all-errors to all curl downloads
  (--retry-all-errors is required because -f makes HTTP errors non-retryable)
- Upgrade ORAS from v1.2.3 to v1.3.1 (SHA-256 verified)
- Remove publish-nightly from ci-status gate since it is infrastructure,
  not code quality — its failures are still visible in the Actions tab

Closes #733
@github-actions
Copy link
Copy Markdown
Contributor

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


New Features ✨

  • (cli) Add sentry cli defaults command for persistent settings by BYK in #721
  • (docs) Auto-generate driftable documentation sections by BYK in #739
  • (issue-list) Add search syntax docs, case-insensitive AND/OR, and JSON syntax reference by BYK in #738

Bug Fixes 🐛

Init

  • Treat no-op edits as passthrough instead of throwing by betegon in #731
  • Remove JSON minification that breaks edit-based codemods by betegon in #719

Resolve

  • Address review comments and add tests for fuzzy project recovery by BYK in #732
  • Fuzzy auto-recovery for project slug resolution by BYK in #728

Upgrade

  • Detect npm install method from node_modules path by BYK in #723
  • Add shell option on Windows for .cmd package managers by BYK in #722

Other

  • (ci) Add retry logic to ORAS/bsdiff downloads and upgrade ORAS by BYK in #741
  • (dashboard) Remove overly restrictive dataset-display cross-validation by BYK in #720
  • (errors) Improve ContextError wording for auto-detect failures by BYK in #726
  • (issue) Support share issue URLs by BYK in #718
  • (issue-list) Auto-correct AND and reject OR in --query to prevent 400 by BYK in #727
  • (telemetry) Rename isClientApiError to isUserApiError and exclude 400 by BYK in #729
  • Bug fixes from Sentry error monitoring (CLI-FR, CLI-RN) + auth default by BYK in #740

Internal Changes 🔧

  • Regenerate skill files by github-actions[bot] in ca16b2ff

🤖 This preview updates automatically when you update the PR.

@BYK BYK marked this pull request as ready for review April 13, 2026 22:05
@github-actions
Copy link
Copy Markdown
Contributor

Codecov Results 📊

134 passed | Total: 134 | Pass Rate: 100% | Execution Time: 0ms

📊 Comparison with Base Branch

Metric Change
Total Tests
Passed Tests
Failed Tests
Skipped Tests

✨ No test changes detected

All tests are passing successfully.

✅ Patch coverage is 100.00%. Project has 1632 uncovered lines.
❌ Project coverage is 95.26%. Comparing base (base) to head (head).

Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
- Coverage    95.27%    95.26%    -0.01%
==========================================
  Files          234       234         —
  Lines        34386     34442       +56
  Branches         0         0         —
==========================================
+ Hits         32761     32810       +49
- Misses        1625      1632        +7
- Partials         0         0         —

Generated by Codecov Action

@BYK BYK merged commit 273e602 into main Apr 13, 2026
23 checks passed
@BYK BYK deleted the fix/ci-patch-generation-resilience branch April 13, 2026 22:17
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.

Delta patch generation failed

1 participant