merge develop and fix conflicts

.github/CODEOWNERS

@@ -1,30 +1 @@
-# This is a comment.
-
-# Each line is a file pattern followed by one or more owners.
-
-# These owners will be the default owners for everything in the repo.
-*       @default-owner
-
-# Order is important. The last matching pattern has the most precedence.
-
-# To set ownership of a particular folder, add a line like this:
-/docs/          @documentation-team
-
-# To set ownership of all .js files in the src/js/ directory, add a line like this:
-/src/js/*.js    @js-developers
-
-# You can also specify individual users. This will make @octocat the owner of any .md files at the root of the repository:
-/*.md           @octocat
-
-# In this example, @doctocat owns any files in the /apps/ directory and any of its subdirectories.
-/apps/          @doctocat
-
-# The `@backend-team` team owns any files in the `/api/` directory in the root of your repository and any of its subdirectories.
-/api/           @backend-team
-
-# The `@frontend-team` team owns any files in the `/web/` directory in the root of your repository and any of its subdirectories.
-/web/           @frontend-team
-
-# Add this line at the end of the file to always require pull request reviews
-# when someone on the team is not an owner and the pull request modifies code owned by the team.
-*       @global-owner
+@0xPolygon/core-cdk

.github/workflows/codeql.yml

@@ -1,35 +1,17 @@
-# For most projects, this workflow file will not need changing; you simply need
-# to commit it to your repository.
-#
-# You may wish to alter this file to override the set of languages analyzed,
-# or to provide custom queries or build logic.
-#
-# ******** NOTE ********
-# We have attempted to detect the languages in your repository. Please check
-# the `language` matrix defined below to confirm you have the correct set of
-# supported CodeQL languages.
-#
 name: "CodeQL"
 
 on:
   push:
-    branches: [ $default-branch, $protected-branches ]
+    branches:
+      - develop
+      - main
   pull_request:
-    # The branches below must be a subset of the branches above
-    branches: [ $default-branch ]
-  schedule:
-    - cron: $cron-weekly
 
 jobs:
   analyze:
     name: Analyze
-    # Runner size impacts CodeQL analysis time. To learn more, please see:
-    #   - https://gh.io/recommended-hardware-resources-for-running-codeql
-    #   - https://gh.io/supported-runners-and-hardware-resources
-    #   - https://gh.io/using-larger-runners
-    # Consider using larger runners for possible analysis time improvements.
-    runs-on: ${{ (matrix.language == 'swift' && 'macos-latest') || 'ubuntu-latest' }}
-    timeout-minutes: ${{ (matrix.language == 'swift' && 120) || 360 }}
+    runs-on: ${{ matrix.language == 'swift' && 'macos-latest' || 'ubuntu-latest' }}
+    timeout-minutes: ${{ matrix.language == 'swift' && 120 || 360 }}
     permissions:
       actions: read
       contents: read
@@ -38,45 +20,25 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        language: [ $detected-codeql-languages ]
-        # CodeQL supports [ $supported-codeql-languages ]
-        # Use only 'java-kotlin' to analyze code written in Java, Kotlin or both
-        # Use only 'javascript-typescript' to analyze code written in JavaScript, TypeScript or both
-        # Learn more about CodeQL language support at https://aka.ms/codeql-docs/language-support
+        language:
+          - go
 
     steps:
-    - name: Checkout repository
-      uses: actions/checkout@v3
-
-    # Initializes the CodeQL tools for scanning.
-    - name: Initialize CodeQL
-      uses: github/codeql-action/init@v2
-      with:
-        languages: ${{ matrix.language }}
-        # If you wish to specify custom queries, you can do so here or in a config file.
-        # By default, queries listed here will override any specified in a config file.
-        # Prefix the list here with "+" to use these queries and those in the config file.
-
-        # For more details on CodeQL's query packs, refer to: https://docs.github.com/en/code-security/code-scanning/automatically-scanning-your-code-for-vulnerabilities-and-errors/configuring-code-scanning#using-queries-in-ql-packs
-        # queries: security-extended,security-and-quality
-
-
-    # Autobuild attempts to build any compiled languages (C/C++, C#, Go, Java, or Swift).
-    # If this step fails, then you should remove it and run the build manually (see below)
-    - name: Autobuild
-      uses: github/codeql-action/autobuild@v2
-
-    # ℹ️ Command-line programs to run using the OS shell.
-    # 📚 See https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#jobsjob_idstepsrun
-
-    #   If the Autobuild fails above, remove it and uncomment the following three lines.
-    #   modify them (or add more) to build your code if your project, please refer to the EXAMPLE below for guidance.
-
-    # - run: |
-    #     echo "Run, Build Application using script"
-    #     ./location_of_script_within_repo/buildscript.sh
-
-    - name: Perform CodeQL Analysis
-      uses: github/codeql-action/analyze@v2
-      with:
-        category: "/language:${{matrix.language}}"
+      - name: Checkout repository
+        uses: actions/checkout@v4
+
+      # Initializes the CodeQL tools for scanning.
+      - name: Initialize CodeQL
+        uses: github/codeql-action/init@v2
+        with:
+          languages: ${{ matrix.language }}
+
+      # Autobuild attempts to build any compiled languages (C/C++, C#, Go, Java, or Swift).
+      # If this step fails, then you should remove it and run the build manually (see below)
+      - name: Autobuild
+        uses: github/codeql-action/autobuild@v2
+
+      - name: Perform CodeQL Analysis
+        uses: github/codeql-action/analyze@v2
+        with:
+          category: "/language:${{ matrix.language }}"

.github/workflows/lint.yml

@@ -0,0 +1,23 @@
+name: Lint
+on:
+  push:
+    branches:
+      - main
+      - develop
+      - update-external-dependencies
+      - 'release/**'
+  pull_request:
+jobs:
+  lint:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Install Go
+        uses: actions/setup-go@v3
+        with:
+          go-version: 1.21.x
+      - name: Checkout code
+        uses: actions/checkout@v3
+      - name: Lint
+        run: |
+          make install-linter
+          make lint

.github/workflows/test-e2e.yml

@@ -20,7 +20,7 @@ jobs:
       uses: actions/checkout@v4
 
     - name: Install Go
-      uses: actions/setup-go@v3
+      uses: actions/setup-go@v5
       with:
         go-version: ${{ matrix.go-version }}
       env:

.github/workflows/test-resequence.yml

@@ -94,7 +94,7 @@ jobs:
 
       - name: Upload logs
         if: always()
-        uses: actions/upload-artifact@v3
+        uses: actions/upload-artifact@v4
         with:
           name: logs_${{ github.run_id }}
           path: ./kurtosis-cdk/ci_logs

.github/workflows/test-unit.yml

@@ -0,0 +1,40 @@
+name: Test Unit and SonarCloud analysis
+
+on:
+  push:
+    branches:
+      - main
+      - develop
+      - 'release/**'
+  pull_request:
+  workflow_dispatch: {}
+
+jobs:
+  test-unit:
+    strategy:
+      fail-fast: false
+      matrix:
+        go-version: [1.22.4]
+        goarch: ["amd64"]
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+        with:
+          fetch-depth: 0  # Shallow clones should be disabled for a better relevancy of analysis
+
+      - name: Install Go
+        uses: actions/setup-go@v5
+        with:
+          go-version: ${{ matrix.go-version }}
+        env:
+          GOARCH: ${{ matrix.goarch }}
+
+      - name: Test
+        run: make test-unit
+
+      - name: Analyze with SonarCloud
+        uses: sonarsource/sonarcloud-github-action@master
+        env:
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+          SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }}

.golangci.yml

@@ -8,70 +8,79 @@ run:
   skip-dirs-use-default: true
   skip-dirs:
     - tests
+    - aggregator/db/migrations
 
 service:
   golangci-lint-version: 1.59.1
-
+    
 linters:
   disable-all: true
   enable:
-    - whitespace # Tool for detection of leading and trailing whitespace
-    # - wsl # Forces you to use empty lines
-    - wastedassign # Finds wasted assignment statements
-    - unconvert # Unnecessary type conversions
-    - tparallel # Detects inappropriate usage of t.Parallel() method in your Go test codes
-    - thelper # Detects golang test helpers without t.Helper() call and checks the consistency of test helpers
-    # - stylecheck # Stylecheck is a replacement for golint
-    # - prealloc # Finds slice declarations that could potentially be pre-allocated
-    - predeclared # Finds code that shadows one of Go's predeclared identifiers
-    # - nolintlint # Ill-formed or insufficient nolint directives
-    # - nlreturn # Checks for a new line before return and branch statements to increase code clarity
-    - misspell # Misspelled English words in comments
-    - makezero # Finds slice declarations with non-zero initial length
-    # - lll # Long lines
-    - importas # Enforces consistent import aliases
-    - gosec # Security problems
-    - gofmt # Whether the code was gofmt-ed
-    - goimports # Unused imports
-    - goconst # Repeated strings that could be replaced by a constant
-    # - forcetypeassert # Finds forced type assertions
-    - dogsled # Checks assignments with too many blank identifiers (e.g. x, , , _, := f())
-    # - dupl # Code clone detection
-    - errname # Checks that sentinel errors are prefixed with the Err and error types are suffixed with the Error
-    # - errorlint # errorlint is a linter for that can be used to find code that will cause problems with the error wrapping scheme introduced in Go 1.13
-    # - gocritic
-    - errcheck # Errcheck is a go lint rule for checking for unchecked errors in go programs. These unchecked errors can be critical bugs in some cases
-    # - godox # Godox is a linter for TODOs and FIXMEs left in the code
+  - whitespace # Tool for detection of leading and trailing whitespace
+  # - wsl # Forces you to use empty lines
+  - wastedassign # Finds wasted assignment statements
+  - unconvert  # Unnecessary type conversions
+  - tparallel # Detects inappropriate usage of t.Parallel() method in your Go test codes
+  - thelper # Detects golang test helpers without t.Helper() call and checks the consistency of test helpers
+  - stylecheck # Stylecheck is a replacement for golint
+  - prealloc # Finds slice declarations that could potentially be pre-allocated
+  - predeclared # Finds code that shadows one of Go's predeclared identifiers
+  - nolintlint # Ill-formed or insufficient nolint directives
+  # - nlreturn # Checks for a new line before return and branch statements to increase code clarity
+  - misspell # Misspelled English words in comments
+  - makezero # Finds slice declarations with non-zero initial length
+  - lll # Long lines
+  - importas  # Enforces consistent import aliases
+  - gosec # Security problems
+  - gofmt # Whether the code was gofmt-ed
+  - goimports # Unused imports
+  - goconst # Repeated strings that could be replaced by a constant
+  - forcetypeassert # Finds forced type assertions
+  - dogsled # Checks assignments with too many blank identifiers (e.g. x, , , _, := f())
+  - dupl # Code clone detection
+  - errname # Checks that sentinel errors are prefixed with the Err and error types are suffixed with the Error
+  - errorlint # errorlint is a linter for that can be used to find code that will cause problems with the error wrapping scheme introduced in Go 1.13
+  - gocritic # gocritic is a Go source code linter that maintains checks that are not in other linters
+  - errcheck # Errcheck is a go lint rule for checking for unchecked errors in go programs. These unchecked errors can be critical bugs in some cases
+  # - godox # Godox is a linter for TODOs and FIXMEs left in the code
+  - gci # Gci is a linter for checking the consistency of the code with the go code style guide
+  - gomnd # Gomnd is a linter for magic numbers
+  # - revive 
+  - unparam # Unparam is a linter for unused function parameters
 
 linters-settings:
   gofmt:
     simplify: true
-  goconst:
-    min-len: 3
-    min-occurrences: 3
   gocritic:
     enabled-checks:
       - ruleguard
-    settings:
-      ruleguard:
-        rules: "./gorules/rules.go"
+    # settings:
+    #   ruleguard:
+    #     rules: "./gorules/rules.go"
+  revive:
+    rules:
+    - name: exported
+      arguments:
+      - disableStutteringCheck
+  goconst:
+    min-len: 3
+    min-occurrences: 3
 
 issues:
-  new-from-rev: origin/develop # report only new issues with reference to develop branch
+  # new-from-rev: origin/develop # report only new issues with reference to develop branch
   whole-files: true
   exclude-rules:
-    - path: _test\.go
+    - path: '(_test\.go|^test/.*)'
       linters:
         - gosec
         - unparam
         - lll
-    - path: gen_sc_data\.go
+    - path: 'etherman/contracts/contracts_(banana|elderberry)\.go'
       linters:
-        - wsl
-        - lll
-        - stylecheck
+        - dupl
   include:
     - EXC0012 # Exported (.+) should have comment( \(or a comment on this block\))? or be unexported
     - EXC0013 # Package comment should be of the form "(.+)...
     - EXC0014 # Comment on exported (.+) should be of the form "(.+)..."
     - EXC0015 # Should have a package comment
+