Change declare arguments

commit-id:e1eef354

Author: cptartur

crates/sncast/src/main.rs

@@ -291,7 +291,9 @@ async fn run_async_command(cli: Cli, config: CastConfig, ui: &UI) -> Result<()>
             .expect("Failed to build contract");
 
             let result = starknet_commands::declare::declare(
-                declare,
+                declare.contract,
+                declare.fee_args,
+                declare.nonce,
                 &account,
                 &artifacts,
                 wait_config,

crates/sncast/src/starknet_commands/declare.rs

@@ -49,8 +49,11 @@ pub struct Declare {
     pub rpc: RpcArgs,
 }
 
+#[expect(clippy::too_many_arguments)]
 pub async fn declare(
-    declare: Declare,
+    contract: String,
+    fee_args: FeeArgs,
+    nonce: Option<Felt>,
     account: &SingleOwnerAccount<&JsonRpcClient<HttpTransport>, LocalWallet>,
     artifacts: &HashMap<String, StarknetContractArtifacts>,
     wait_config: WaitForTx,
@@ -59,9 +62,9 @@ pub async fn declare(
 ) -> Result<DeclareResponse, StarknetCommandError> {
     let contract_artifacts =
         artifacts
-            .get(&declare.contract)
+            .get(&contract)
             .ok_or(StarknetCommandError::ContractArtifactsNotFound(ErrorData {
-                data: ByteArray::from(declare.contract.as_str()),
+                data: ByteArray::from(contract.as_str()),
             }))?;
 
     let contract_definition: SierraClass = serde_json::from_str(&contract_artifacts.sierra)
@@ -72,8 +75,8 @@ pub async fn declare(
     declare_with_artifacts(
         contract_definition,
         casm_contract_definition,
-        declare.fee_args,
-        declare.nonce,
+        fee_args,
+        nonce,
         account,
         wait_config,
         skip_on_already_declared,

crates/sncast/src/starknet_commands/script/run.rs

@@ -1,4 +1,3 @@
-use crate::starknet_commands::declare::Declare;
 use crate::starknet_commands::{call, declare, deploy, invoke, tx_status};
 use crate::{WaitForTx, get_account};
 use anyhow::{Context, Result, anyhow};
@@ -127,14 +126,6 @@ impl<'a> ExtensionLogic for CastScriptExtension<'a> {
                 let fee_args: FeeArgs = input_reader.read::<ScriptFeeSettings>()?.into();
                 let nonce = input_reader.read()?;
 
-                let declare = Declare {
-                    contract: contract.clone(),
-                    fee_args,
-                    nonce,
-                    package: None,
-                    rpc: RpcArgs::default(),
-                };
-
                 let declare_tx_id = generate_declare_tx_id(contract.as_str());
 
                 if let Some(success_output) =
@@ -144,7 +135,9 @@ impl<'a> ExtensionLogic for CastScriptExtension<'a> {
                 }
 
                 let declare_result = self.tokio_runtime.block_on(declare::declare(
-                    declare,
+                    contract.clone(),
+                    fee_args,
+                    nonce,
                     self.account()?,
                     self.artifacts,
                     WaitForTx {