fix: swap event_timestamp with branch in add_turns (#99)

Co-authored-by: Jonathan James <[email protected]>

Author: jona62

src/bedrock_agentcore/memory/session.py

@@ -889,7 +889,7 @@ def add_turns(
         event_timestamp: Optional[datetime] = None,
     ) -> Event:
         """Delegates to manager.add_turns."""
-        return self._manager.add_turns(self._actor_id, self._session_id, messages, event_timestamp, branch)
+        return self._manager.add_turns(self._actor_id, self._session_id, messages, branch, event_timestamp)
 
     def fork_conversation(
         self,

tests/bedrock_agentcore/memory/test_session.py

@@ -1867,8 +1867,8 @@ def test_session_delegation_with_optional_parameters(self):
                     "user-123",
                     "session-456",
                     [ConversationalMessage("Hello", MessageRole.USER)],
-                    custom_timestamp,
                     branch,
+                    custom_timestamp,
                 )
 
     def test_comprehensive_error_coverage(self):
@@ -2110,13 +2110,13 @@ def test_session_add_turns_parameter_order(self):
                     event_timestamp=custom_timestamp,
                 )
 
-                # Verify the exact parameter order: actor_id, session_id, messages, event_timestamp, branch
+                # Verify the exact parameter order: actor_id, session_id, messages, branch, event_timestamp
                 mock_add_turns.assert_called_once_with(
                     "user-123",
                     "session-456",
                     [ConversationalMessage("Hello", MessageRole.USER)],
-                    custom_timestamp,
                     branch,
+                    custom_timestamp,
                 )