Cairo 2.4.3 + components + starknet.go (RPC v0.6, tx v3)

.github/actions/install-cairo/action.yml

@@ -4,11 +4,11 @@ description: A composite action that installs cairo and scarb binaries
 inputs:
   cairo_version:
     description: Cairo release version
-    default: "v2.2.0"
+    default: "v2.4.3"
     required: false
   scarb_version:
     description: Scarb release version
-    default: "v0.7.0"
+    default: "v2.4.3"
     required: false
 
 runs:

.github/workflows/contracts.yml

@@ -23,9 +23,6 @@ jobs:
       - name: Install Cairo
         uses: ./.github/actions/install-cairo
 
-      - name: Install starknet-devnet (via venv+pip)
-        run: nix develop -c pip install -r contracts/requirements.txt -c contracts/constraints.txt
-
       - name: Test
         run: nix develop -c make test-ts-contracts
 

.github/workflows/integration_contracts.yml

@@ -23,8 +23,5 @@ jobs:
       - name: Install Cairo
         uses: ./.github/actions/install-cairo
 
-      - name: Install starknet-devnet (via venv+pip)
-        run: nix develop -c pip install -r contracts/requirements.txt -c contracts/constraints.txt
-
       - name: Test
         run: nix develop -c make test-integration-contracts

.github/workflows/integration_gauntlet.yml

@@ -49,9 +49,6 @@ jobs:
       - name: Install Cairo
         uses: ./.github/actions/install-cairo
 
-      - name: Install starknet-devnet (via venv+pip)
-        run: nix develop -c pip install -r contracts/requirements.txt -c contracts/constraints.txt
-
       - name: Test
         run: nix develop -c make test-integration-gauntlet
 

.github/workflows/relayer.yml

@@ -35,8 +35,5 @@ jobs:
             ./relayer/coverage.txt
             ./relayer/race_coverage.txt
 
-      - name: Install starknet-devnet (via venv+pip)
-        run: nix develop -c pip install -r contracts/requirements.txt -c contracts/constraints.txt
-
       - name: Integration Test
         run: nix develop -c make test-integration-go

Makefile

@@ -122,16 +122,16 @@ format: format-go format-cairo format-ts
 format-check: format-cairo-check format-ts-check
 
 .PHONY: format-go
-format-go: format-go-fmt format-go-mod-tidy
+format-go: format-go-fmt gomodtidy
 
 .PHONY: format-go-fmt
 format-go-fmt:
 	cd ./relayer && go fmt ./...
 	cd ./ops && go fmt ./...
 	cd ./integration-tests && go fmt ./...
 
-.PHONY: format-go-mod-tidy
-format-go-mod-tidy:
+.PHONY: gomodtidy
+gomodtidy:
 	cd ./relayer && go mod tidy
 	cd ./monitoring && go mod tidy
 	cd ./ops && go mod tidy
@@ -180,14 +180,12 @@ test-unit-go:
 
 .PHONY: test-integration-go
 # only runs tests with TestIntegration_* + //go:build integration
-test-integration-go:
+test-integration-go: env-devnet-hardhat
 	cd ./relayer && go test -v ./... -run TestIntegration -tags integration
 
 .PHONY: test-integration-prep
 test-integration-prep:
-	python -m venv ~/cairo_venv && \
-		. ~/cairo_venv/bin/activate
-	cd ./contracts && pip install -r requirements.txt
+	cd ./contracts
 	make build
 
 .PHONY: test-integration
@@ -221,17 +219,18 @@ test-integration-soak-ci:
 #       cd examples/contracts/aggregator-consumer/ && \
 #         yarn test
 test-integration-contracts: build-ts env-devnet-hardhat
+	echo "Tests currently broken because of starknet-hardhat-plugin"
+	exit 1
 	cd packages-ts/integration-multisig/ && \
 		yarn test
 	cd packages-ts/starknet/ && \
 		yarn test
 
 .PHONY: test-integration-gauntlet
-# TODO: better network lifecycle setup - tests setup/run their own network (L1 + conflict w/ above if not cleaned up)
 # TODO: fix example
 # cd packages-ts/starknet-gauntlet-example/ && \
 #   yarn test
-test-integration-gauntlet: build-ts env-devnet-hardhat-down
+test-integration-gauntlet: build-ts env-devnet-hardhat
 	cd packages-ts/starknet-gauntlet/ && \
 		yarn test
 	cd packages-ts/starknet-gauntlet-argent/ && \

contracts/Scarb.lock

@@ -0,0 +1,14 @@
+# Code generated by scarb DO NOT EDIT.
+version = 1
+
+[[package]]
+name = "chainlink"
+version = "0.1.0"
+dependencies = [
+ "openzeppelin",
+]
+
+[[package]]
+name = "openzeppelin"
+version = "0.8.0"
+source = "git+https://github.com/OpenZeppelin/cairo-contracts.git?tag=v0.8.0#c23e8e96de60e6e3159b1ff8591a1187269c0eb7"

contracts/Scarb.toml

@@ -11,8 +11,8 @@ sierra = "cairo-compile . -r"
 # Uncomment if you want to use dependencies
 # Note: currently testing doesn't work with dependencies
 [dependencies]
-starknet = ">=1.0.0"
-openzeppelin = { git = "https://github.com/OpenZeppelin/cairo-contracts.git", tag = "v0.7.0-rc.0" }
+starknet = ">=1.3.0"
+openzeppelin = { git = "https://github.com/OpenZeppelin/cairo-contracts.git", tag = "v0.8.0" }
 
 [lib]
 

contracts/src/access_control/access_controller.cairo

@@ -1,107 +1,60 @@
-use starknet::ContractAddress;
-
 #[starknet::contract]
 mod AccessController {
     use starknet::ContractAddress;
     use starknet::class_hash::ClassHash;
 
-    use chainlink::libraries::access_control::{AccessControl, IAccessController};
-    use chainlink::libraries::ownable::{Ownable, IOwnable};
+    use chainlink::libraries::access_control::{AccessControlComponent, IAccessController};
+    use chainlink::libraries::ownable::{OwnableComponent, IOwnable};
     use chainlink::libraries::upgradeable::{Upgradeable, IUpgradeable};
 
-    #[storage]
-    struct Storage {}
-
-    #[constructor]
-    fn constructor(ref self: ContractState, owner_address: ContractAddress) {
-        let mut ownable = Ownable::unsafe_new_contract_state();
-        Ownable::constructor(ref ownable, owner_address);
-        let mut access_control = AccessControl::unsafe_new_contract_state();
-        AccessControl::constructor(ref access_control);
+    component!(path: OwnableComponent, storage: ownable, event: OwnableEvent);
+    component!(path: AccessControlComponent, storage: access_control, event: AccessControlEvent);
+
+    #[abi(embed_v0)]
+    impl OwnableImpl = OwnableComponent::OwnableImpl<ContractState>;
+    impl InternalImpl = OwnableComponent::InternalImpl<ContractState>;
+
+    #[abi(embed_v0)]
+    impl AccessControlImpl =
+        AccessControlComponent::AccessControlImpl<ContractState>;
+    impl AccessControlInternalImpl = AccessControlComponent::InternalImpl<ContractState>;
+
+    #[event]
+    #[derive(Drop, starknet::Event)]
+    enum Event {
+        #[flat]
+        OwnableEvent: OwnableComponent::Event,
+        #[flat]
+        AccessControlEvent: AccessControlComponent::Event,
     }
 
-    #[external(v0)]
-    impl AccessControllerImpl of IAccessController<ContractState> {
-        fn has_access(self: @ContractState, user: ContractAddress, data: Array<felt252>) -> bool {
-            let state = AccessControl::unsafe_new_contract_state();
-            AccessControl::has_access(@state, user, data)
-        }
-
-        fn add_access(ref self: ContractState, user: ContractAddress) {
-            let ownable = Ownable::unsafe_new_contract_state();
-            Ownable::assert_only_owner(@ownable);
-            let mut state = AccessControl::unsafe_new_contract_state();
-            AccessControl::add_access(ref state, user);
-        }
-
-        fn remove_access(ref self: ContractState, user: ContractAddress) {
-            let ownable = Ownable::unsafe_new_contract_state();
-            Ownable::assert_only_owner(@ownable);
-            let mut state = AccessControl::unsafe_new_contract_state();
-            AccessControl::remove_access(ref state, user);
-        }
-
-        fn enable_access_check(ref self: ContractState) {
-            let ownable = Ownable::unsafe_new_contract_state();
-            Ownable::assert_only_owner(@ownable);
-            let mut state = AccessControl::unsafe_new_contract_state();
-            AccessControl::enable_access_check(ref state);
-        }
-
-        fn disable_access_check(ref self: ContractState) {
-            let ownable = Ownable::unsafe_new_contract_state();
-            Ownable::assert_only_owner(@ownable);
-            let mut state = AccessControl::unsafe_new_contract_state();
-            AccessControl::disable_access_check(ref state);
-        }
+    #[storage]
+    struct Storage {
+        #[substorage(v0)]
+        ownable: OwnableComponent::Storage,
+        #[substorage(v0)]
+        access_control: AccessControlComponent::Storage,
     }
 
-    ///
-    /// Ownable
-    ///
-
-    #[external(v0)]
-    impl OwnableImpl of IOwnable<ContractState> {
-        fn owner(self: @ContractState) -> ContractAddress {
-            let state = Ownable::unsafe_new_contract_state();
-            Ownable::OwnableImpl::owner(@state)
-        }
-
-        fn proposed_owner(self: @ContractState) -> ContractAddress {
-            let state = Ownable::unsafe_new_contract_state();
-            Ownable::OwnableImpl::proposed_owner(@state)
-        }
-
-        fn transfer_ownership(ref self: ContractState, new_owner: ContractAddress) {
-            let mut state = Ownable::unsafe_new_contract_state();
-            Ownable::OwnableImpl::transfer_ownership(ref state, new_owner)
-        }
-
-        fn accept_ownership(ref self: ContractState) {
-            let mut state = Ownable::unsafe_new_contract_state();
-            Ownable::OwnableImpl::accept_ownership(ref state)
-        }
-
-        fn renounce_ownership(ref self: ContractState) {
-            let mut state = Ownable::unsafe_new_contract_state();
-            Ownable::OwnableImpl::renounce_ownership(ref state)
-        }
+    #[constructor]
+    fn constructor(ref self: ContractState, owner_address: ContractAddress) {
+        self.ownable.initializer(owner_address);
+        self.access_control.initializer();
     }
 
     ///
     /// Upgradeable
     ///
 
-    #[view]
+    // #[view]
     fn type_and_version(self: @ContractState) -> felt252 {
         'AccessController 1.0.0'
     }
 
     #[external(v0)]
     impl UpgradeableImpl of IUpgradeable<ContractState> {
         fn upgrade(ref self: ContractState, new_impl: ClassHash) {
-            let ownable = Ownable::unsafe_new_contract_state();
-            Ownable::assert_only_owner(@ownable);
+            self.ownable.assert_only_owner();
             Upgradeable::upgrade(new_impl);
         }
     }