Skip to content
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

MCP: The notifications/initialized message is not sent #1186

Closed
jmartisk opened this issue Jan 2, 2025 · 9 comments · Fixed by #1267
Closed

MCP: The notifications/initialized message is not sent #1186

jmartisk opened this issue Jan 2, 2025 · 9 comments · Fixed by #1267
Assignees
Labels
bug Something isn't working

Comments

@jmartisk
Copy link
Collaborator

jmartisk commented Jan 2, 2025

In the HTTP transport, which we reimplement on the Quarkus side, we need an equivalent change as in langchain4j/langchain4j#2360

@jmartisk jmartisk self-assigned this Jan 2, 2025
@geoand geoand added the bug Something isn't working label Jan 3, 2025
@geoand
Copy link
Collaborator

geoand commented Jan 22, 2025

@jmartisk do we now have what we need in LangChain4j for this?

@jmartisk
Copy link
Collaborator Author

jmartisk commented Jan 22, 2025

In LC4j, it is fixed. For Quarkus, I have it in my branch that buffers updates that should go in with the next lc4j upgrade

@geoand
Copy link
Collaborator

geoand commented Jan 22, 2025

🙏

@geoand
Copy link
Collaborator

geoand commented Jan 28, 2025

@jmartisk @langchain4j any idea when LangChain4j will be released? Asking because we ran into this while preparing for a workshop and if possible, we would really like to have a fix some time this week.

Thanks!

@jmartisk
Copy link
Collaborator Author

+1 to releasing rather soon, I have a quarkus-langchain4j branch that contains updates that will be applied together with the upgrade, but it's getting messy and hard to handle.

@geoand
Copy link
Collaborator

geoand commented Jan 30, 2025

@langchain4j any idea when the next release will go out?

Thanks

@geoand
Copy link
Collaborator

geoand commented Feb 3, 2025

I assume that with LangChain4j 1.0.0.Alpha2 we can now fix this, correct?

@jmartisk
Copy link
Collaborator Author

jmartisk commented Feb 3, 2025

It doesn't look it has been released yet

@geoand
Copy link
Collaborator

geoand commented Feb 3, 2025

I guess Twitter isn't accurate 😀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants