refactor: key registration parameters

Updates offline key registration parameters to use an optional non-participation flag.

This simplifies the logic and provides more flexibility when marking an account as non-participating. It also adds a test to verify an account cannot be brought back online after being marked as non-participating.

Author: aorumbayev

crates/algokit_utils/src/lib.rs

@@ -11,18 +11,8 @@ pub use testing::{
     AlgorandFixture, AlgorandTestContext, algorand_fixture, algorand_fixture_with_config,
 };
 pub use transactions::{
-    AssetCreateParams,
-    AssetDestroyParams,
-    AssetReconfigureParams,
-    CommonParams,
-    Composer,
-    ComposerError,
-    ComposerTxn,
-    EmptySigner,
-    NonParticipationKeyRegistrationParams,
-    OfflineKeyRegistrationParams,
-    OnlineKeyRegistrationParams,
-    PaymentParams,
-    TxnSigner,
+    AssetCreateParams, AssetDestroyParams, AssetReconfigureParams, CommonParams, Composer,
+    ComposerError, ComposerTxn, EmptySigner, NonParticipationKeyRegistrationParams,
+    OfflineKeyRegistrationParams, OnlineKeyRegistrationParams, PaymentParams, TxnSigner,
     TxnSignerGetter,
 };

crates/algokit_utils/src/transactions/composer.rs

@@ -4,8 +4,9 @@ use algod_client::{
     models::{PendingTransactionResponse, TransactionParams},
 };
 use algokit_transact::{
-    Address, AlgorandMsgpack, AssetConfigTransactionFields, FeeParams, KeyRegistrationTransactionFields,
-    PaymentTransactionFields, SignedTransaction, Transaction, TransactionHeader, Transactions,
+    Address, AlgorandMsgpack, AssetConfigTransactionFields, FeeParams,
+    KeyRegistrationTransactionFields, PaymentTransactionFields, SignedTransaction, Transaction,
+    TransactionHeader, Transactions,
 };
 use derive_more::Debug;
 use std::sync::Arc;
@@ -97,10 +98,7 @@ pub struct OnlineKeyRegistrationParams {
 #[derive(Debug, Clone)]
 pub struct OfflineKeyRegistrationParams {
     pub common_params: CommonParams,
-    /// Whether to prevent the account from ever participating in consensus again.
-    /// - `false`: Temporary offline (can be brought back online later)
-    /// - `true`: Permanent offline (account can never participate again)
-    pub prevent_account_from_ever_participating_again: bool,
+    pub non_participation: Option<bool>,
 }
 
 #[derive(Debug, Clone)]
@@ -480,13 +478,7 @@ impl Composer {
                             vote_last: None,
                             vote_key_dilution: None,
                             state_proof_key: None,
-                            non_participation: if offline_key_reg_params
-                                .prevent_account_from_ever_participating_again
-                            {
-                                Some(true)
-                            } else {
-                                None
-                            },
+                            non_participation: offline_key_reg_params.non_participation,
                         })
                     }
                     ComposerTxn::NonParticipationKeyRegistration(_) => {
@@ -908,7 +900,7 @@ mod tests {
                 first_valid_round: None,
                 last_valid_round: None,
             },
-            prevent_account_from_ever_participating_again: false,
+            non_participation: Some(false),
         };
         assert!(
             composer