Merge pull request #17 from Squads-Protocol/feat/policy-hooks

Feat/policy hooks

Author: 0xRigel-squads

Anchor.toml

@@ -23,5 +23,9 @@ url = "https://api.devnet.solana.com"
 address = "8UuqDAqe9UQx9e9Sjj4Gs3msrWGfzb4CJHGK3U3tcCEX"
 filename = "tests/fixtures/pre-rent-collector/multisig-account.json"
 
+[[test.genesis]]
+address = "noopb9bkMVfRPU8AsbpTUg8AQkHtKwMYZiFUjNRtMmV"
+program = "tests/fixtures/noop.so"
+
 [scripts]
 test = "npx mocha --node-option require=ts-node/register --extension ts -t 1000000 tests/index.ts"

programs/squads_smart_account_program/src/errors.rs

@@ -170,6 +170,10 @@ pub enum SmartAccountError {
     ProgramInteractionTooManyInstructionConstraints,
     #[msg("Program interaction constraint violation: too many spending limits. Max is 10")]
     ProgramInteractionTooManySpendingLimits,
+    #[msg("Program interaction hook violation: template hook error")]
+    ProgramInteractionTemplateHookError,
+    #[msg("Program interaction hook violation: hook authority cannot be part of hook accounts")]
+    ProgramInteractionHookAuthorityCannotBePartOfHookAccounts,
 
     // ===============================================
     // Spending Limit Policy Errors

programs/squads_smart_account_program/src/instructions/transaction_execute_sync.rs

@@ -151,7 +151,7 @@ impl<'info> SyncTransaction<'info> {
                     &settings_key,
                     &smart_account_pubkey,
                     &settings.signers,
-                    settings_compiled_instructions,
+                    &settings_compiled_instructions,
                     &remaining_accounts,
                 )?;
 
@@ -169,7 +169,7 @@ impl<'info> SyncTransaction<'info> {
                     consensus_account_type: ConsensusAccountType::Settings,
                     payload: SynchronousTransactionEventPayload::TransactionPayload {
                         account_index: args.account_index,
-                        instructions: executable_message.instructions,
+                        instructions: executable_message.instructions.to_vec(),
                     },
                     signers: ctx.remaining_accounts[..args.num_signers as usize]
                         .iter()

programs/squads_smart_account_program/src/instructions/transaction_execute_sync_legacy.rs

@@ -89,7 +89,7 @@ impl LegacySyncTransaction<'_> {
             &settings_key,
             &smart_account_pubkey,
             &settings.signers,
-            settings_compiled_instructions,
+            &settings_compiled_instructions,
             &ctx.remaining_accounts,
         )?;
 
@@ -109,7 +109,7 @@ impl LegacySyncTransaction<'_> {
                 .map(|acc| acc.key.clone())
                 .collect(),
             account_index: args.account_index,
-            instructions: executable_message.instructions,
+            instructions: executable_message.instructions.to_vec(),
             instruction_accounts: executable_message
                 .accounts
                 .iter()