chore: move all contract related logic to contracts folder

.github/workflows/solidity.yml

@@ -1,4 +1,4 @@
-name: CI Solidity
+name: Solidity
 
 on:
   push:
@@ -37,7 +37,7 @@ jobs:
         run: forge build
 
       - name: Cache data
-        uses: actions/cache@v4
+        uses: actions/cache/save@v4
         with:
           path: contracts
           key: ci-solidity-${{ github.ref }}
@@ -56,7 +56,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Restore cache
-        uses: actions/cache@v4
+        uses: actions/cache/restore@v4
         with:
           path: contracts
           key: ci-solidity-${{ github.ref }}
@@ -77,7 +77,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Restore cache
-        uses: actions/cache@v4
+        uses: actions/cache/restore@v4
         with:
           path: contracts
           key: ci-solidity-${{ github.ref }}
@@ -98,7 +98,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Restore cache
-        uses: actions/cache@v4
+        uses: actions/cache/restore@v4
         with:
           path: contracts
           key: ci-solidity-${{ github.ref }}

.github/workflows/test.yml

@@ -28,7 +28,7 @@ jobs:
         uses: foundry-rs/foundry-toolchain@v1
         with:
           version: "nightly-ac81a53d1d5823919ffbadd3c65f081927aa11f2"
-      - run: dev/contracts/deploy-local
+      - run: contracts/dev/deploy-local
       - name: Run Tests
         run: |
           export GOPATH="${HOME}/go/"

.vscode/settings.json

@@ -17,5 +17,6 @@
     ]
   },
   "solidity.compileUsingRemoteVersion": "v0.8.28",
-  "solidity.formatter": "forge"
+  "solidity.formatter": "forge",
+  "wake.compiler.solc.remappings": []
 }

contracts/config/anvil_localnet/GroupMessages.json

@@ -1,10 +1,10 @@
 {
   "addresses": {
     "groupMessagesDeployer": "0xf39Fd6e51aad88F6F4ce6aB8827279cffFb92266",
-    "groupMessagesImpl": "0x0a17FabeA4633ce714F1Fa4a2dcA62C3bAc4758d",
-    "groupMessagesProxy": "0x3C1Cb427D20F15563aDa8C249E71db76d7183B6c",
+    "groupMessagesImpl": "0xc5a5C42992dECbae36851359345FE25997F5C42d",
+    "groupMessagesProxy": "0x67d269191c92Caf3cD7723F116c85e6E9bf55933",
     "groupMessagesProxyAdmin": "0xf39Fd6e51aad88F6F4ce6aB8827279cffFb92266"
   },
-  "deploymentBlock": 65,
-  "latestUpgradeBlock": 71
+  "deploymentBlock": 28,
+  "latestUpgradeBlock": 28
 }

contracts/config/anvil_localnet/IdentityUpdates.json

@@ -1,10 +1,10 @@
 {
   "addresses": {
     "identityUpdatesDeployer": "0xf39Fd6e51aad88F6F4ce6aB8827279cffFb92266",
-    "identityUpdatesImpl": "0x1343248Cbd4e291C6979e70a138f4c774e902561",
-    "identityUpdatesProxy": "0x22a9B82A6c3D2BFB68F324B2e8367f346Dd6f32a",
+    "identityUpdatesImpl": "0xE6E340D132b5f46d1e472DebcD681B2aBc16e57E",
+    "identityUpdatesProxy": "0xc3e53F4d16Ae77Db1c982e75a937B9f60FE63690",
     "identityUpdatesProxyAdmin": "0xf39Fd6e51aad88F6F4ce6aB8827279cffFb92266"
   },
-  "deploymentBlock": 67,
-  "latestUpgradeBlock": 67
+  "deploymentBlock": 30,
+  "latestUpgradeBlock": 30
 }

dev/contracts/deploy-local → contracts/dev/deploy-local

@@ -1,14 +1,17 @@
 #!/bin/bash
 # Deploy the smart contracts to the local anvil node
 set -euo pipefail
-source dev/contracts/.env
-source dev/contracts/common
 
 # Make sure the build directory exists
 mkdir -p ./build
 
-# Move to working directory
-cd ./contracts
+# Always work from the contracts directory
+script_dir=$(dirname "$(realpath "$0")")
+repo_root=$(realpath "${script_dir}/../")
+cd "${repo_root}"
+
+source dev/lib/env
+source dev/lib/common
 
 # Update depencencies
 forge soldeer update &> /dev/null
@@ -19,6 +22,4 @@ fi
 
 forge_deploy_script group_messages
 forge_deploy_script identity_updates
-
-# TODO: This should be a function in the forge CLI
-deploy_contract src/Nodes.sol Nodes
+forge_deploy_script nodes src/Nodes.sol Nodes

contracts/dev/generate

@@ -0,0 +1,70 @@
+#!/bin/bash
+
+set -euo pipefail
+
+# Default directories (can be overridden with environment variables)
+source_dir="${SOURCE_DIR:-src}"
+build_dir="${BUILD_DIR:-build}"
+output_dir="${OUTPUT_DIR:-pkg}"
+
+# Ensure required directories exist and clean up old artifacts
+function setup_directories() {
+    mkdir -p "${build_dir}" "${output_dir}"
+}
+
+# Generate bindings for a given contract
+function generate_bindings() {
+    local filename="$1"
+    local package="$(echo "${filename}" | tr '[:upper:]' '[:lower:]')"
+    local source_artifact="${source_dir}/${filename}.sol"
+    local build_artifact="${build_dir}/${filename}"
+    local output_artifact="${output_dir}/${package}/${filename}.go"
+
+    rm -f "${build_artifact}".*.json
+    mkdir -p "${output_dir}/${package}"
+    rm -f "${output_dir}/${package}"/*.go
+
+    # Generate ABI and bytecode
+    if ! forge inspect "${source_artifact}:${filename}" abi > "${build_artifact}.abi.json"; then
+        echo "ERROR: Failed to generate ABI for ${filename}" >&2
+        exit 1
+    fi
+
+    if ! forge inspect "${source_artifact}:${filename}" bytecode > "${build_artifact}.bin.json"; then
+        echo "ERROR: Failed to generate bytecode for ${filename}" >&2
+        exit 1
+    fi
+
+    # Generate Go bindings
+    if ! abigen --abi "${build_artifact}.abi.json" \
+                --bin "${build_artifact}.bin.json" \
+                --pkg "${package}" \
+                --type "${filename}" \
+                --out "${output_artifact}" > /dev/null 2>&1; then
+        echo "ERROR: Failed to generate bindings for ${filename}" >&2
+        exit 1
+    fi
+}
+
+function main() {
+    # Always work from the contracts directory
+    script_dir=$(dirname "$(realpath "$0")")
+    repo_root=$(realpath "${script_dir}/../")
+    cd "${repo_root}"
+
+    setup_directories
+
+    # Define contracts (pass as arguments or use a default list)
+    local contracts=("$@")
+    if [ "${#contracts[@]}" -eq 0 ]; then
+        contracts=("Nodes" "GroupMessages" "IdentityUpdates")
+    fi
+
+    # Generate bindings for each contract
+    for contract in "${contracts[@]}"; do
+        echo "Processing contract: ${contract}"
+        generate_bindings "${contract}"
+    done
+}
+
+main "$@"

dev/contracts/common → contracts/dev/lib/common

@@ -27,15 +27,17 @@ function forge_deploy_script() {
             echo -e "\n"
             ;;
 
+        nodes)
+            # TODO: Migrate to forge script
+            forge create --broadcast --legacy --json --rpc-url $DOCKER_RPC_URL --private-key $PRIVATE_KEY "$2:$3" > ../build/$3.json
+            echo -e "✅ Nodes contract deployed.\n"
+            cat ../build/$3.json
+            echo -e "\n"
+            ;;
+
         *)
             echo "Invalid option. Use 'group_messages' or 'identity_updates'."
             exit 1
             ;;
     esac
 }
-
-# Deploy a contract and save the output (which includes the contract address) to a JSON file to be used in tests
-# TODO: This should be a function in the forge CLI
-function deploy_contract() {
-    forge create --broadcast --legacy --json --rpc-url $DOCKER_RPC_URL --private-key $PRIVATE_KEY "$1:$2" > ../build/$2.json
-}

contracts/src/GroupMessages.sol

@@ -30,9 +30,11 @@ contract GroupMessages is Initializable, AccessControlUpgradeable, UUPSUpgradeab
     uint256 public constant MAX_PAYLOAD_SIZE = 4_194_304;
 
     // State variables
+    // slither-disable-next-line unused-state,constable-states
     uint64 private sequenceId;
 
     /// @dev Reserved storage gap for future upgrades
+    // slither-disable-next-line unused-state,naming-convention
     uint256[50] private __gap;
 
     // Initialization

contracts/src/IdentityUpdates.sol

@@ -30,9 +30,11 @@ contract IdentityUpdates is Initializable, AccessControlUpgradeable, UUPSUpgrade
     uint256 public constant MAX_PAYLOAD_SIZE = 4_194_304;
 
     // State variables
+    // slither-disable-next-line unused-state,constable-states
     uint64 private sequenceId;
 
     /// @dev Reserved storage gap for future upgrades
+    // slither-disable-next-line unused-state,naming-convention
     uint256[50] private __gap;
 
     // Initialization

dev/baked/Dockerfile

@@ -22,7 +22,7 @@ COPY . .
 # It seems that anvil flushes the file to disk on shutdown and it takes a few ms to be persisted
 # That gives us the pkill+sleep requirement
 RUN dev/docker/anvil-background && \
-    dev/contracts/deploy-local && \
+    contracts/dev/deploy-local && \
     dev/register-local-node && \
     dev/register-local-node-2 && \
     pkill -f anvil && \

dev/generate

@@ -6,5 +6,5 @@ set -euo pipefail
 sqlc generate
 go generate ./...
 rm -rf pkg/mocks/*
-./dev/contracts/generate
+./contracts/dev/generate
 mockery

dev/local.env

@@ -1,6 +1,6 @@
 #!/bin/bash
 
-source dev/contracts/.env
+source contracts/dev/lib/env
 
 # Anvil scripts output folder
 ANVIL_SCRIPTS_OUTPUT=contracts/config/anvil_localnet

dev/up

@@ -28,8 +28,8 @@ fi
 dev/docker/up
 
 # Make sure the abis are updated
-dev/contracts/generate
-dev/contracts/deploy-local
+contracts/dev/generate
+contracts/dev/deploy-local
 
 echo "Registering local node-1"
 dev/register-local-node