Skip to content

Conversation

@limansky
Copy link
Contributor

@limansky limansky commented Dec 6, 2025

sttp v4 is available for a while. It is stable, provides simpler API comparing to v3. I've adopted the code for the new sttp, however I have no real use case to test how does it work.

@ex0ns
Copy link
Contributor

ex0ns commented Dec 6, 2025

Thank you very much for your contribution, I quickly went through the PR and I believe that it makes sense to upgrade, it also clean up a few thing and more importantly it remove some implicit context (decoder/encoder + the async http client being created in the test example).
I make my best to review and test this thoroughly as soon as possible

This is a breaking change, so I will have to release 7.0, I will see if they are other breaking changes I should ship at the same time, maybe the Pekko one.

@ex0ns ex0ns mentioned this pull request Dec 6, 2025
@ex0ns ex0ns changed the title Migrate to sttp 4 feat!: migrate to sttp 4 Dec 7, 2025
@ex0ns
Copy link
Contributor

ex0ns commented Dec 7, 2025

I tested quite a few examples and they work well, there is only one thing that broke inside the launcher for the ZIO bots.
Also with the fix that I just published for src-jvm-2 there are a few examples that need to be upgraded.

@ex0ns
Copy link
Contributor

ex0ns commented Dec 7, 2025

Working great, if you can address the ZIO Commands' bot issue, I will then merge this PR 👌

@ex0ns ex0ns merged commit ad4072d into bot4s:main Dec 7, 2025
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants