Skip to content

[Sync] Update project files from source repository (ac305ad)#104

Merged
mrz1836 merged 2 commits intomasterfrom
chore/sync-files-bsv-blockchain-sdks-20260219-192341-ac305ad
Feb 20, 2026
Merged

[Sync] Update project files from source repository (ac305ad)#104
mrz1836 merged 2 commits intomasterfrom
chore/sync-files-bsv-blockchain-sdks-20260219-192341-ac305ad

Conversation

@mrz1836
Copy link
Copy Markdown
Collaborator

@mrz1836 mrz1836 commented Feb 20, 2026

What Changed

  • Removed docs directory from .dockerignore, allowing documentation files to be included in Docker builds
  • Added new composite action .github/actions/upload-artifact-resilient/action.yml that provides resilient artifact uploads with step-level retry logic (3 attempts with escalating delays of 10s, 30s)
  • Updated MAGE_X_VERSION from v1.12.1 to v1.12.2 in .github/env/10-mage-x.env
  • Replaced direct actions/upload-artifact@v4 calls with the new upload-artifact-resilient composite action across 8 workflow files
  • Added permissions: contents: read to multiple workflow jobs for explicit permission scoping
  • Modified artifact upload configurations to use the resilient action with configurable retry logic, overwrite behavior, and compression settings

Why It Was Necessary

  • Transient GitHub infrastructure failures (including non-retryable 403 errors from CDN/proxy intermediaries) were causing artifact upload failures in CI workflows
  • The new composite action provides defense-in-depth with both step-level retries and internal retry mechanisms (ACTIONS_UPLOAD_RETRY_COUNT=3) to handle infrastructure issues
  • Explicit permissions: contents: read declarations improve security posture by following the principle of least privilege for workflow jobs

Testing Performed

  • Verified the composite action structure includes proper input validation, retry logic with escalating delays, and error handling
  • Confirmed all 8 workflow files correctly reference the new composite action with appropriate parameters (artifact-name, artifact-path, retention-days)
  • Validated that the action supports configurable continue-on-error behavior for critical vs non-critical artifacts

Impact / Risk

  • Breaking Change: None - workflows maintain same external behavior while gaining resilience
  • Risk: Low - the composite action wraps existing actions/upload-artifact@v4 with additional retry logic without changing core functionality
  • Performance: Minimal impact under normal conditions; retries add 10-30 second delays only during transient failures
  • CI Reliability: Significantly improved artifact upload success rate for workflows affected by intermittent GitHub infrastructure issues

@mrz1836 mrz1836 self-assigned this Feb 20, 2026
@mrz1836 mrz1836 added automated-sync Automated sync PR, e.g. from a fork or external repo automerge Label to automatically merge pull requests that meet all required conditions chore Simple dependency updates or version bumps labels Feb 20, 2026
@github-actions github-actions bot added update General updates size/L Large change (201–500 lines) labels Feb 20, 2026
…ives

gosec v2.23.0 panics on integer conversion checks (G115) due to a bug
in its range analyzer. Exclude G115 at the config level and remove all
the now-unused //nolint:gosec // G115 inline directives.
@sonarqubecloud
Copy link
Copy Markdown

@mrz1836 mrz1836 merged commit ddb404e into master Feb 20, 2026
44 checks passed
@github-actions github-actions bot deleted the chore/sync-files-bsv-blockchain-sdks-20260219-192341-ac305ad branch February 20, 2026 00:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automated-sync Automated sync PR, e.g. from a fork or external repo automerge Label to automatically merge pull requests that meet all required conditions chore Simple dependency updates or version bumps size/L Large change (201–500 lines) update General updates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant