Skip to content

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

Merged
mrz1836 merged 2 commits into
mainfrom
chore/sync-files-bsv-blockchain-services-20260219-194138-ac305ad
Feb 20, 2026
Merged

[Sync] Update project files from source repository (ac305ad)#24
mrz1836 merged 2 commits into
mainfrom
chore/sync-files-bsv-blockchain-services-20260219-194138-ac305ad

Conversation

@mrz1836
Copy link
Copy Markdown
Collaborator

@mrz1836 mrz1836 commented Feb 20, 2026

What Changed

  • Removed docs/*.md from .dockerignore to allow documentation files to be included in Docker builds
  • Added new composite GitHub Action .github/actions/upload-artifact-resilient/action.yml with step-level retry logic (3 attempts with escalating delays of 10s, 30s) to handle transient 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
  • Integrated the new resilient artifact upload action across 8 workflow files (code quality, benchmarks, coverage, completion statistics, security scans, fuzz testing, test matrix, test validation, and fork PR management)
  • Added permissions: contents: read to all workflow jobs that use the new artifact upload action
  • Replaced direct actions/upload-artifact@v4 calls with the new upload-artifact-resilient composite action across all workflows

Why It Was Necessary

  • Documentation files need to be available in Docker containers for runtime reference or serving
  • GitHub Actions artifact uploads were experiencing transient failures (403 errors from CDN/proxy intermediaries) that were not being handled by the built-in retry mechanism
  • Centralizing artifact upload logic with resilient retry patterns improves CI/CD reliability and reduces manual intervention for infrastructure-related failures

Testing Performed

  • Verified the composite action includes proper input validation and retry configuration (ACTIONS_UPLOAD_RETRY_COUNT=3, overwrite: true)
  • Confirmed all workflow files correctly reference the new action with required inputs (artifact-name, artifact-path, retention-days)
  • Validated that permissions are properly scoped with contents: read for artifact operations

Impact / Risk

  • Low risk - The composite action wraps existing actions/upload-artifact@v4 functionality with additional resilience
  • Improved reliability - Step-level retries with escalating delays should reduce transient failure rates in artifact uploads
  • No breaking changes - All artifact uploads maintain the same retention policies and naming conventions

@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
…tale nolint directives

Add nolint:gosec directives for SSRF (G704) and log injection (G706)
findings that are false positives — URLs come from configuration, not
user input. Remove now-unused nolint:gosec comments from arcade.go and
embedded.go that the updated linter no longer flags.
@mrz1836 mrz1836 merged commit 5c38a4b into main Feb 20, 2026
45 checks passed
@github-actions github-actions Bot deleted the chore/sync-files-bsv-blockchain-services-20260219-194138-ac305ad branch February 20, 2026 01:10
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