refactor: agent config helper, integration helpers split, and tool cleanup - #364
Conversation
|
This is a useful refactor, but it needs conflict resolution before it can land. What it does well:
Blocker:
Recommendation: rebase onto upstream/main, resolve the agent-config conflicts, and push the updated branch. |
…nBackground and startService
a07f15b to
8deacef
Compare
|
I rebased this PR onto upstream/main and resolved the agent-config conflicts. The branch is now clean and mergeable — ok github.com/dagger/container-use/cmd/container-use (cached) |
What this PR does
A bundle of internal refactors that reduce duplication and tighten the public surface area:
wrapToolandwrapToolWithClientinto one wrapper.RunBackgroundandstartService.helpers.gointosetup.go,actions.go, anddirect.go, then remove the stale monolith.writeMcpConfigand use it across all agent configs (Claude, Codex, Cursor, Goose, Q).FileEditRequestto reduceFileEditparameter count.FileReadintoFileReadandFileReadRange.Order / dependencies
This PR must be merged after the three feature PRs in this series:
The refactor branch is based on
upstream/mainplus those three changes, so once they land this PR will show only the refactor commits.