GreatEugenius opened a new issue, #288:
URL: https://github.com/apache/flink-agents/issues/288

   ### Search before asking
   
   - [x] I searched in the 
[issues](https://github.com/apache/flink-agents/issues) and found nothing 
similar.
   
   ### Description
   
   After the code changes introduced in [PR 
#279](https://github.com/apache/flink-agents/pull/279), adding new items to the 
messageContext list appears to be ineffective.
   
   The current implementation is as follows:
   
https://github.com/apache/flink-agents/blob/7325f45a74fe1a4da4f17050f197e946841cb39a/plan/src/main/java/org/apache/flink/agents/plan/actions/ChatModelAction.java#L82-L86
   
   However, since a new ArrayList is created from the original list and not 
modified back into toolCallContext, the updated messageContext is never 
persisted. As a result, any newly added response messages are lost after this 
operation.
   
   ### How to reproduce
   
   n/a
   
   ### Version and environment
   
   branch: main
   
   ### Are you willing to submit a PR?
   
   - [x] I'm willing to submit a PR!


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to