feat: allow custom discriminator in actions (#574)

Users have bad UX with hardcoded discriminator in dlp.
Right now users have to define a fixed entrypoint with fixed
discriminator. This is cumbersome and not flexible.

This pr adapts changes made in dlp
[PR](magicblock-labs/delegation-program#105)



<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit

- Refactor
- Reworked scheduling/handler flow: replaced redelegation-intent path
with explicit commit and undelegate action handlers.
- Removed legacy redelegation payload types and simplified call
dispatch.
  - Eliminated unused context data from task payloads.
- Tests
- Updated tests to match new action shapes; redelegation intent test
path disabled.
- Chores
  - Updated workspace dependency revisions (no feature changes).
<!-- end of auto-generated comment: release notes by coderabbit.ai -->

Author: taco-paco

Cargo.lock

@@ -124,7 +124,7 @@ magicblock-config = { path = "./magicblock-config" }
 magicblock-config-helpers = { path = "./magicblock-config-helpers" }
 magicblock-config-macro = { path = "./magicblock-config-macro" }
 magicblock-core = { path = "./magicblock-core" }
-magicblock-delegation-program = { git = "https://github.com/magicblock-labs/delegation-program.git", rev = "5fb8d20", features = [
+magicblock-delegation-program = { git = "https://github.com/magicblock-labs/delegation-program.git", rev = "aa1de56d90c", features = [
   "no-entrypoint",
 ] }
 magicblock-geyser-plugin = { path = "./magicblock-geyser-plugin" }

Cargo.toml

@@ -119,13 +119,14 @@ impl TransactionStrategyExecutionError {
     ) -> Self {
         // There's always 2 budget instructions in front
         const OFFSET: u8 = 2;
-        const OUTDATED_SLOT: u32 = dlp::error::DlpError::OutdatedSlot as u32;
+        const NONCE_OUT_OF_ORDER: u32 =
+            dlp::error::DlpError::NonceOutOfOrder as u32;
 
         match err {
             // Filter CommitIdError by custom error code
             transaction_err @ TransactionError::InstructionError(
                 _,
-                InstructionError::Custom(OUTDATED_SLOT),
+                InstructionError::Custom(NONCE_OUT_OF_ORDER),
             ) => TransactionStrategyExecutionError::CommitIDError(
                 transaction_err,
             ),

magicblock-committor-service/src/intent_executor/error.rs

@@ -89,7 +89,6 @@ impl BaseTask for ArgsTask {
                     action.escrow_authority,
                     account_metas,
                     CallHandlerArgs {
-                        context: value.context,
                         data: action.data_per_program.data.clone(),
                         escrow_index: action.data_per_program.escrow_index,
                     },

magicblock-committor-service/src/tasks/args_task.rs

@@ -1,4 +1,3 @@
-use dlp::args::Context;
 use dyn_clone::DynClone;
 use magicblock_committor_program::{
     instruction_builder::{
@@ -119,7 +118,6 @@ pub struct FinalizeTask {
 
 #[derive(Clone)]
 pub struct BaseActionTask {
-    pub context: Context,
     pub action: BaseAction,
 }
 
@@ -346,7 +344,6 @@ mod serialization_safety_test {
 
         // Test BaseAction variant
         let base_action: ArgsTask = ArgsTaskType::BaseAction(BaseActionTask {
-            context: Context::Undelegate,
             action: BaseAction {
                 destination_program: Pubkey::new_unique(),
                 escrow_authority: Pubkey::new_unique(),

magicblock-committor-service/src/tasks/mod.rs

@@ -1,7 +1,6 @@
 use std::sync::Arc;
 
 use async_trait::async_trait;
-use dlp::args::Context;
 use log::error;
 use magicblock_program::magic_scheduled_base_intent::{
     CommitType, CommittedAccount, MagicBaseIntent, ScheduledBaseIntent,
@@ -53,10 +52,7 @@ impl TasksBuilder for TaskBuilderImpl {
                 let tasks = actions
                     .iter()
                     .map(|el| {
-                        let task = BaseActionTask {
-                            context: Context::Standalone,
-                            action: el.clone(),
-                        };
+                        let task = BaseActionTask { action: el.clone() };
                         let task =
                             ArgsTask::new(ArgsTaskType::BaseAction(task));
                         Box::new(task) as Box<dyn BaseTask>
@@ -148,7 +144,6 @@ impl TasksBuilder for TaskBuilderImpl {
                         .collect::<Vec<_>>();
                     tasks.extend(base_actions.iter().map(|action| {
                         let task = BaseActionTask {
-                            context: Context::Commit,
                             action: action.clone(),
                         };
                         let task =
@@ -188,7 +183,6 @@ impl TasksBuilder for TaskBuilderImpl {
                     UndelegateType::WithBaseActions(actions) => {
                         tasks.extend(actions.iter().map(|action| {
                             let task = BaseActionTask {
-                                context: Context::Undelegate,
                                 action: action.clone(),
                             };
                             let task =

magicblock-committor-service/src/tasks/task_builder.rs

@@ -249,7 +249,6 @@ pub type TaskStrategistResult<T, E = TaskStrategistError> = Result<T, E>;
 
 #[cfg(test)]
 mod tests {
-    use dlp::args::Context;
     use magicblock_program::magic_scheduled_base_intent::{
         BaseAction, CommittedAccount, ProgramArgs,
     };
@@ -283,7 +282,6 @@ mod tests {
     // Helper to create a Base action task
     fn create_test_base_action_task(len: usize) -> ArgsTask {
         ArgsTask::new(ArgsTaskType::BaseAction(BaseActionTask {
-            context: Context::Commit,
             action: BaseAction {
                 destination_program: Pubkey::new_unique(),
                 escrow_authority: Pubkey::new_unique(),