Browser tools: initialize lazily and add timeouts.
Also rename browser_screenshot to browser_take_screenshot for clarity\n- Update both Go and UI code to maintain consistency
Co-Authored-By: sketch <hello@sketch.dev>
Change-ID: s8a5cabff914f88dfk
diff --git a/loop/agent.go b/loop/agent.go
index 2c8eec9..95a32bb 100644
--- a/loop/agent.go
+++ b/loop/agent.go
@@ -904,18 +904,17 @@
// template in termui/termui.go has pretty-printing support for all tools.
var browserTools []*llm.Tool
- // Add browser tools if enabled
- // if experiment.Enabled("browser") {
- if true {
- _, supportsScreenshots := a.config.Service.(*ant.Service)
- bTools, browserCleanup := browse.RegisterBrowserTools(a.config.Context, supportsScreenshots)
- // Add cleanup function to context cancel
- go func() {
- <-a.config.Context.Done()
- browserCleanup()
- }()
- browserTools = bTools
- }
+ _, supportsScreenshots := a.config.Service.(*ant.Service)
+ var bTools []*llm.Tool
+ var browserCleanup func()
+
+ bTools, browserCleanup = browse.RegisterBrowserTools(a.config.Context, supportsScreenshots)
+ // Add cleanup function to context cancel
+ go func() {
+ <-a.config.Context.Done()
+ browserCleanup()
+ }()
+ browserTools = bTools
convo.Tools = []*llm.Tool{
bashTool, claudetool.Keyword,