Skip to content

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

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

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

Conversation

@mrz1836
Copy link
Contributor

@mrz1836 mrz1836 commented Feb 20, 2026

What Changed

  • Removed docs from .dockerignore, allowing documentation files to be included in Docker builds
  • Added new composite action .github/actions/upload-artifact-resilient/action.yml with step-level retry logic (3 attempts with escalating delays of 10s, 30s) to handle transient GitHub infrastructure failures including non-retryable 403 errors
  • Updated MAGE_X_VERSION from v1.12.1 to v1.12.2 in .github/env/10-mage-x.env
  • Updated all workflow files (fortress-code-quality.yml, fortress-coverage.yml, fortress-benchmarks.yml, fortress-completion-statistics.yml, fortress-test-fuzz.yml, fortress-security-scans.yml, fortress-test-matrix.yml, fortress-test-validation.yml, pull-request-management-fork.yml) to use the new upload-artifact-resilient composite action instead of direct actions/upload-artifact@v4 calls
  • Added permissions: contents: read to workflows that previously had no permissions block

Why It Was Necessary

  • GitHub Actions artifact uploads were experiencing transient infrastructure failures (403 errors from CDN/proxy intermediaries) that could not be recovered by the built-in retry logic
  • Documentation files need to be available in Docker containers for runtime reference or distribution
  • The new resilient upload action provides defense-in-depth against both 5xx server errors and non-retryable 403 errors through step-level retry with overwrite capabilities

Testing Performed

  • The composite action implements configurable retry logic with overwrite: true on all attempts to handle partially-finalized artifacts
  • All artifact upload operations across 9 workflow files have been migrated to use the standardized resilient upload pattern
  • The action supports configurable inputs for artifact name, path, retention days, compression level, and continue-on-error behavior

Impact / Risk

  • Risk: Low - The change introduces more resilient artifact handling without altering workflow behavior; failures that previously occurred will now be retried automatically
  • Breaking Change: None - All workflows maintain their existing artifact upload configurations with added resilience
  • Performance Impact: Minimal - Retry delays (10s, 30s) only trigger on failures; successful uploads remain unchanged

@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 size/L Large change (201–500 lines) update General updates labels Feb 20, 2026
Add missing nolint directives for integer overflow conversions in test files,
and remove stale nolint directives that are no longer needed.
@sonarqubecloud
Copy link

@mrz1836 mrz1836 merged commit f3a4b09 into master Feb 20, 2026
45 checks passed
@github-actions github-actions bot deleted the chore/sync-files-bsv-blockchain-sdks-20260219-192344-ac305ad branch February 20, 2026 00:39
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