-
Notifications
You must be signed in to change notification settings - Fork 43
Update Quarkus LangChain4j 1.4.1 #181
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
@mariofusco I updated to version 1.4.1 but it required me to rename outputName to outputKey which then uncovered some other issues. Can you review and let me know if this makes sense? |
| <langchain4j.version>1.6.0</langchain4j.version> | ||
| <langchain4j.beta.version>1.8.0-beta15</langchain4j.beta.version> | ||
| <a2a.version>0.3.2.Final</a2a.version> | ||
| <a2a.version>0.3.0.Beta1</a2a.version> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I had to roll back the version since even the latest beta of langchain4j-agentic-a2a is still using the 0.3.0.Beta1 version.
2025-11-19 12:22:29,089 ERROR [io.qua.ver.htt.run.QuarkusErrorHandler] (vert.x-eventloop-thread-0) HTTP Request to / failed, error id: 58f0b5b3-a236-42c7-9b0d-9841340a3d95-1: java.lang.NoSuchMethodError: 'java.lang.String io.a2a.util.Utils.toJsonString(java.lang.Object)'
| ) | ||
| @Agent(description = "Car disposition specialist. Determines how to dispose of a car.", | ||
| outputKey = "dispositionAgentResult") | ||
| @A2AClientAgent(a2aServerUrl = "http://localhost:8888") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The previous way of defining the outputKey in the A2AClientAgent didn't work anymore. This seems to work. Let me know if it's the correct implementation :)
No description provided.