chore: minor cleanup:

Author: 0xRigel-squads

programs/squads_smart_account_program/src/instructions/settings_transaction_sync.rs

@@ -1,7 +1,6 @@
-use account_events::{AddSpendingLimitEvent, RemoveSpendingLimitEvent};
 use anchor_lang::prelude::*;
 
-use crate::{ consensus::ConsensusAccount, consensus_trait::{Consensus, ConsensusAccountType}, errors::*, events::*, program::SquadsSmartAccountProgram, state::*, utils::*};
+use crate::{consensus::ConsensusAccount, consensus_trait::{Consensus, ConsensusAccountType}, errors::*, events::*, program::SquadsSmartAccountProgram, state::*, utils::*};
 
 #[derive(AnchorSerialize, AnchorDeserialize)]
 pub struct SyncSettingsTransactionArgs {

programs/squads_smart_account_program/src/instructions/transaction_close.rs

@@ -18,10 +18,8 @@ use crate::consensus_trait::ConsensusAccountType;
 use crate::errors::*;
 use crate::program::SquadsSmartAccountProgram;
 use crate::state::*;
-use crate::utils;
 use crate::LogAuthorityInfo;
 use crate::SmartAccountEvent;
-use crate::TransactionContent;
 use crate::TransactionEvent;
 use crate::TransactionEventType;
 
@@ -534,7 +532,7 @@ pub struct CloseEmptyPolicyTransaction<'info> {
     pub empty_policy: AccountInfo<'info>,
 
     /// CHECK: `seeds` and `bump` verify that the account is the canonical Proposal,
-    ///         the logic within `transaction_close` does the rest of the checks.
+    ///         the logic within `close_empty_policy_transaction` does the rest of the checks.
     #[account(
         mut,
         seeds = [
@@ -579,7 +577,7 @@ impl CloseEmptyPolicyTransaction<'_> {
     /// empty/deleted policies.
     ///
     /// Since a policy can never exist at the same address again after being
-    /// closed, any transaction & proposal associated with it can be closed.
+    /// closed, any transaction & proposal associated with it can be closed safely.
     pub fn close_empty_policy_transaction(ctx: Context<Self>) -> Result<()> {
         let proposal = &mut ctx.accounts.proposal;
         let proposal_rent_collector = &ctx.accounts.proposal_rent_collector;

programs/squads_smart_account_program/src/interface/consensus_trait.rs

@@ -1,7 +1,7 @@
 use anchor_lang::prelude::*;
 use borsh::{BorshDeserialize, BorshSerialize};
 
-use crate::{errors::SmartAccountError, Permission, SmartAccountSigner};
+use crate::{Permission, SmartAccountSigner};
 
 #[derive(BorshSerialize, BorshDeserialize, PartialEq, Eq, PartialOrd, Ord, Clone, Copy, Debug)]
 pub enum ConsensusAccountType {

programs/squads_smart_account_program/src/state/policies/policy_core/policy.rs

@@ -1,9 +1,11 @@
 use anchor_lang::prelude::*;
 
+use super::{payloads::PolicyPayload, traits::PolicyTrait, PolicyExecutionContext};
+use crate::state::policies::implementations::InternalFundTransferPolicy;
 use crate::{
     errors::*,
     interface::consensus_trait::{Consensus, ConsensusAccountType},
-    InternalFundTransferExecutionArgs, Permission, ProgramInteractionExecutionArgs,
+    InternalFundTransferExecutionArgs, ProgramInteractionExecutionArgs,
     ProgramInteractionPolicy, Proposal, Settings, SettingsChangeExecutionArgs,
     SettingsChangePolicy, SmartAccountSigner, SpendingLimitExecutionArgs, SpendingLimitPolicy,
     Transaction, SEED_POLICY, SEED_PREFIX,
@@ -25,9 +27,6 @@ pub enum PolicyExpirationArgs {
     SettingsState,
 }
 
-use super::{payloads::PolicyPayload, traits::PolicyTrait, PolicyExecutionContext};
-use crate::state::policies::implementations::InternalFundTransferPolicy;
-
 #[account]
 pub struct Policy {
     /// The smart account this policy belongs to.