TaskUpdater parameters in same order as AgentExecutor.execute()

Author: kabir

src/main/java/io/a2a/server/tasks/TaskUpdater.java

@@ -19,7 +19,7 @@ public class TaskUpdater {
     private final String taskId;
     private final String contextId;
 
-    public TaskUpdater(EventQueue eventQueue, RequestContext context) {
+    public TaskUpdater(RequestContext context, EventQueue eventQueue) {
         this.eventQueue = eventQueue;
         this.taskId = context.getTaskId();
         this.contextId = context.getContextId();

src/test/java/io/a2a/server/apps/AgentExecutorProducer.java

@@ -30,7 +30,7 @@ public void execute(RequestContext context, EventQueue eventQueue) throws JSONRP
             @Override
             public void cancel(RequestContext context, EventQueue eventQueue) throws JSONRPCError {
                 if (context.getTask().getId().equals("cancel-task-123")) {
-                    TaskUpdater taskUpdater = new TaskUpdater(eventQueue, context);
+                    TaskUpdater taskUpdater = new TaskUpdater(context, eventQueue);
                     taskUpdater.cancel();
                 } else if (context.getTask().getId().equals("cancel-task-not-supported-123")) {
                     throw new UnsupportedOperationError();

src/test/java/io/a2a/server/requesthandlers/JSONRPCHandlerTest.java

@@ -167,7 +167,7 @@ public void testOnCancelTaskSuccess() throws Exception {
             // Looking at the Python implementation, they typically use AgentExecutors that
             // don't support cancellation. So my theory is the Agent updates the task to the CANCEL status
             Task task = context.getTask();
-            TaskUpdater taskUpdater = new TaskUpdater(eventQueue, context);
+            TaskUpdater taskUpdater = new TaskUpdater(context, eventQueue);
             taskUpdater.cancel();
         };
 

src/test/java/io/a2a/server/tasks/TaskUpdaterTest.java

@@ -47,7 +47,7 @@ public void init() {
                 .setTaskId(TEST_TASK_ID)
                 .setContextId(TEST_TASK_CONTEXT_ID)
                 .build();
-        taskUpdater = new TaskUpdater(eventQueue, context);
+        taskUpdater = new TaskUpdater(context, eventQueue);
     }
 
     @Test

tck/src/main/java/io/a2a/tck/server/AgentExecutorProducer.java

@@ -43,7 +43,7 @@ public void execute(RequestContext context, EventQueue eventQueue) throws JSONRP
                 eventQueue.enqueueEvent(task);
             }
 
-            TaskUpdater updater = new TaskUpdater(eventQueue, context);
+            TaskUpdater updater = new TaskUpdater(context, eventQueue);
 
             // Immediately set to WORKING state
             updater.startWork();
@@ -68,7 +68,7 @@ public void cancel(RequestContext context, EventQueue eventQueue) throws JSONRPC
                 throw new TaskNotCancelableError();
             }
 
-            TaskUpdater updater = new TaskUpdater(eventQueue, context);
+            TaskUpdater updater = new TaskUpdater(context, eventQueue);
             updater.cancel();
             eventQueue.enqueueEvent(new TaskStatusUpdateEvent.Builder()
                     .taskId(task.getId())