Dagger Cloud

[viscid spade]

That's not the error that was getting hit consistently before https://github.com/dagger/dagger/pull/10209

If you revert the commit in 10209 locally and re-run the TestClientGenerator tests you'll see the error. It's about go.mod requiring go 1.23.8. Have an explanation of why it was getting hit in that PR

[frigid vortex]

Hmmm so I should add some logic in the test client gen to pin a specific go mod versions?

[viscid spade]

IMO it would be better to fix this in the client generator code since otherwise users could very easily hit this.

There's logic for normal modules that adds dep versions that match the go sdk's versions: https://github.com/dagger/dagger/blob/97b3ef86a3c56472f57f62ff46201be11d2f5d08/cmd/codegen/generator/go/generator.go#L305-L305

So I imagine it might be possible to just re-use that (with some adjustments) maybe.

cc @analog sand if that makes sense to you too