diff --git a/internal/agent/common_test.go b/internal/agent/common_test.go index dfb7a5f3257870762daba505aff673b6f51e8b85..21bdaf1ab501d34a4ac69cc592af7787733e0950 100644 --- a/internal/agent/common_test.go +++ b/internal/agent/common_test.go @@ -175,7 +175,6 @@ func coderAgent(r *recorder.Recorder, env env, large, small fantasy.LanguageMode return nil, err } allTools := []fantasy.AgentTool{ - tools.NewWebFetchTool(env.workingDir, r.GetDefaultClient()), tools.NewBashTool(env.permissions, env.workingDir, cfg.Options.Attribution), tools.NewDownloadTool(env.permissions, env.workingDir, r.GetDefaultClient()), tools.NewEditTool(env.lspClients, env.permissions, env.history, env.workingDir), diff --git a/internal/agent/fetch_tool.go b/internal/agent/fetch_tool.go index e9b81a55bd8c9d71f856426cddd0585a5e7d0bf5..83db58da36483bb84511bdc23e19d3ed4194e43a 100644 --- a/internal/agent/fetch_tool.go +++ b/internal/agent/fetch_tool.go @@ -14,7 +14,6 @@ import ( "github.com/charmbracelet/crush/internal/agent/prompt" "github.com/charmbracelet/crush/internal/agent/tools" - "github.com/charmbracelet/crush/internal/config" "github.com/charmbracelet/crush/internal/permission" ) @@ -25,11 +24,6 @@ var fetchToolDescription []byte var fetchPromptTmpl []byte func (c *coordinator) fetchTool(_ context.Context, client *http.Client) (fantasy.AgentTool, error) { - _, ok := c.cfg.Agents[config.AgentFetch] - if !ok { - return nil, errors.New("fetch agent not configured") - } - if client == nil { client = &http.Client{ Timeout: 30 * time.Second, diff --git a/internal/config/config.go b/internal/config/config.go index 6e7400f7746ebee4b5b65321ebe6fd50bedae26a..99c70079223127c71c3768da40d4aec0b55d62c7 100644 --- a/internal/config/config.go +++ b/internal/config/config.go @@ -51,7 +51,6 @@ const ( const ( AgentCoder string = "coder" AgentTask string = "task" - AgentFetch string = "fetch" ) type SelectedModel struct { @@ -534,19 +533,6 @@ func (c *Config) SetupAgents() { // NO MCPs or LSPs by default AllowedMCP: map[string][]string{}, }, - - AgentFetch: { - ID: AgentFetch, - Name: "Fetch", - Description: "An agent that fetches and analyzes web content.", - Model: SelectedModelTypeSmall, - // No context paths - fetch agent operates in isolated environment - ContextPaths: []string{}, - // Only allow tools needed for web content analysis - AllowedTools: []string{"web_fetch", "grep", "glob", "view"}, - // NO MCPs or LSPs - AllowedMCP: map[string][]string{}, - }, } c.Agents = agents } diff --git a/internal/config/load_test.go b/internal/config/load_test.go index efe2fb1612b3300c166196532149f89c79dc5716..d6c783bb63f5ac3000df0f212d6574a8fecc9ec2 100644 --- a/internal/config/load_test.go +++ b/internal/config/load_test.go @@ -469,13 +469,6 @@ func TestConfig_setupAgentsWithNoDisabledTools(t *testing.T) { taskAgent, ok := cfg.Agents[AgentTask] require.True(t, ok) assert.Equal(t, []string{"glob", "grep", "ls", "sourcegraph", "view"}, taskAgent.AllowedTools) - - fetchAgent, ok := cfg.Agents[AgentFetch] - require.True(t, ok) - assert.Equal(t, "fetch", fetchAgent.ID) - assert.Equal(t, SelectedModelTypeSmall, fetchAgent.Model) - assert.Equal(t, []string{"web_fetch", "grep", "glob", "view"}, fetchAgent.AllowedTools) - assert.Empty(t, fetchAgent.ContextPaths) } func TestConfig_setupAgentsWithDisabledTools(t *testing.T) {