chore: remove not used config

Kujtim Hoxha created 5 days ago

Change summary

internal/agent/common_test.go |  1 -
internal/agent/fetch_tool.go  |  6 ------
internal/config/config.go     | 14 --------------
internal/config/load_test.go  |  7 -------
4 files changed, 28 deletions(-)

Detailed changes

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),

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,

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
 }

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) {