Merge branch 'master' into build-brotli

Author: joshuacolvin0

.github/actions/lint-go/action.yml

@@ -0,0 +1,16 @@
+---
+name: 'Lint Go Code'
+description: 'Runs both golangci-lint and custom linters on the Go codebase.'
+
+runs:
+  using: composite
+
+  steps:
+    - name: GolangCI Lint
+      uses: golangci/golangci-lint-action@v9
+      with:
+        version: v2.5
+
+    - name: Custom Lint
+      shell: bash
+      run: go run ./linters ./...

.github/workflows/_arbitrator.yml

@@ -13,7 +13,7 @@ jobs:
 
     steps:
       - name: Checkout
-        uses: actions/checkout@v5
+        uses: actions/checkout@v6
         with:
           submodules: recursive
 
@@ -37,14 +37,44 @@ jobs:
           #  so make sure it doesn't conflict with anything!
           no-cache: true
 
+      - name: Install cargo-nextest
+        uses: taiki-e/install-action@v2
+        with:
+          tool: [email protected]
+
+      - name: Create minimal nextest.toml config file
+        run: |
+          cat << EOF > nextest.toml
+          [profile.ci.junit]
+          path = "junit.xml"
+          EOF
+
       - name: Make arbitrator libraries
         run: make -j wasm-ci-build
 
       - name: Clippy check
         run: cargo clippy --all --manifest-path arbitrator/Cargo.toml -- -D warnings
 
       - name: Run rust tests
-        run: cargo test -p arbutil -p prover -p jit -p stylus --release --manifest-path arbitrator/prover/Cargo.toml
+        id: run-rust-tests
+        continue-on-error: true
+        run: >-
+          cargo nextest run -p arbutil -p prover -p jit -p stylus --release
+          --manifest-path arbitrator/prover/Cargo.toml
+          --profile ci --config-file nextest.toml
+
+      - name: Upload rust test results
+        if: always()
+        uses: actions/upload-artifact@v5
+        with:
+          name: junit-reports-rust
+          path: ./arbitrator/target/nextest/ci/junit.xml
+
+      - name: Fail if rust tests failed
+        if: steps.run-rust-tests.outcome == 'failure'
+        run: |
+          echo "Rust tests failed. Failing the workflow as required."
+          exit 1
 
       - name: Check stylus_bechmark
         run: cargo check --manifest-path arbitrator/tools/stylus_benchmark/Cargo.toml

.github/workflows/_bold-legacy.yml

@@ -9,21 +9,24 @@ jobs:
     runs-on: arbitrator-ci
     steps:
       - name: Checkout
-        uses: actions/checkout@v5
+        uses: actions/checkout@v6
         with:
           submodules: recursive
 
 
       - name: Setup CI
         uses: ./.github/actions/ci-setup
 
+      - name: Build
+        run: make -j8 build test-go-deps
+
       - name: run challenge tests
         run: >-
           ${{ github.workspace }}/.github/workflows/gotestsum.sh --tags legacychallengetest
           --run TestChallenge --timeout 60m --cover
 
       - name: Archive detailed run log
-        uses: actions/upload-artifact@v4
+        uses: actions/upload-artifact@v5
         with:
           name: full.log
           path: full.log

.github/workflows/_codecov.yml

@@ -0,0 +1,54 @@
+---
+name: codecov
+on:
+  workflow_call:
+    secrets:
+      CODECOV_TOKEN:
+        required: false
+        description: Token to upload reports to Codecov. Required if `upload_report` is on.
+    inputs:
+      post_comment:
+        description: 'Set to true to post a test summary comment on the PR.'
+        type: boolean
+        required: true
+      upload_report:
+        description: 'Set to true to upload test reports to Codecov.'
+        type: boolean
+        required: true
+      report-name:
+        description: 'Name to use when uploading reports to Codecov.'
+        type: string
+        required: false
+        default: 'CI Test Results'
+
+jobs:
+  report_summary:
+    name: Aggregate test results
+    runs-on: ubuntu-4
+    permissions:
+      pull-requests: write
+
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@v6
+
+      - name: Download JUnit reports
+        uses: actions/download-artifact@v6
+        with:
+          pattern: junit-reports-*
+          path: downloaded-reports/
+
+      - name: Post test summary comment
+        if: ${{ inputs.post_comment }}
+        uses: codecov/basic-test-results@v1
+        with:
+          github-token: ${{ secrets.GITHUB_TOKEN }}
+          directory: 'downloaded-reports/'
+
+      - name: Upload test results to Codecov
+        if: ${{ inputs.upload_report }}
+        uses: codecov/test-results-action@v1
+        with:
+          token: ${{ secrets.CODECOV_TOKEN }}
+          directory: 'downloaded-reports/'
+          name: ${{ inputs.report-name }}

.github/workflows/_detect-changes.yml

@@ -0,0 +1,42 @@
+---
+name: detect-file-changes
+
+on:
+  workflow_call:
+    outputs:
+      arbitrator_changed:
+        description: 'True if Arbitrator or related files changed.'
+        value: ${{ jobs.changes.outputs.arbitrator_changed }}
+      bold_legacy_changed:
+        description: 'True if Bold Legacy or related files changed.'
+        value: ${{ jobs.changes.outputs.bold_legacy_changed }}
+
+jobs:
+  changes:
+    name: Detect file changes
+    runs-on: ubuntu-4
+    outputs:
+      arbitrator_changed: ${{ steps.changed.outputs.arbitrator_any_changed }}
+      bold_legacy_changed: ${{ steps.changed.outputs.bold_legacy_any_changed }}
+
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v6
+        with:
+          submodules: recursive
+          fetch-depth: 10            # Will cover most PRs
+          persist-credentials: true  # In case changed-files requires deeper depth
+
+      - name: Determine if Arbitrator or Bold Legacy changed
+        id: changed
+        uses: tj-actions/[email protected]
+        with:
+          files_yaml: |
+            arbitrator:
+              - 'arbitrator/**'
+              - 'contracts/**'
+              - 'Makefile'
+              - '.github/workflows/_arbitrator.yml'
+            bold_legacy:
+              - 'bold/legacy/**'
+              - '.github/workflows/_bold-legacy.yml'

.github/workflows/_fast.yml

@@ -9,7 +9,7 @@ jobs:
     runs-on: arbitrator-ci
     steps:
       - name: Checkout
-        uses: actions/checkout@v5
+        uses: actions/checkout@v6
         with:
           submodules: recursive
 
@@ -22,11 +22,5 @@ jobs:
       - name: Build all lint dependencies
         run: make -j8 build-node-deps
 
-      - name: GolangCI Lint
-        uses: golangci/golangci-lint-action@v8
-        with:
-          version: v2.1
-
-      - name: Custom Lint
-        run: |
-          go run ./linters ./...
+      - name: Setup CI
+        uses: ./.github/actions/lint-go