Filesystem only has a working directory

This removes the "allowed directories" from this toolset, the idea is
that, since we already have "add_allowed_directory" the idea of allowed
dirs in this toolset doesn't really make sense. People should use
containers for these things

Signed-off-by: Djordje Lukic <[email protected]>

Author: rumpl

e2e/cagent_exec_test.go

@@ -21,7 +21,7 @@ func TestExec_OpenAI(t *testing.T) {
 func TestExec_OpenAI_ToolCall(t *testing.T) {
 	out := cagentExec(t, "testdata/fs_tools.yaml", "How many files in testdata/working_dir? Only output the number.")
 
-	require.Equal(t, "\n--- Agent: root ---\n\nCalling search_files(\n  path: \"testdata/working_dir\"\n  pattern: \"*\"\n)\n\nsearch_files response → \"1 files found:\\ntestdata/working_dir/README.me\"\n1", out)
+	require.Equal(t, "\n--- Agent: root ---\n\nCalling list_directory(path: \"testdata/working_dir\")\n\nlist_directory response → \"FILE README.me\\n\"\n1", out)
 }
 
 func TestExec_OpenAI_HideToolCalls(t *testing.T) {
@@ -81,7 +81,7 @@ func TestExec_Mistral(t *testing.T) {
 func TestExec_Mistral_ToolCall(t *testing.T) {
 	out := cagentExec(t, "testdata/fs_tools.yaml", "--model=mistral/mistral-small", "How many files in testdata/working_dir? Only output the number.")
 
-	require.Equal(t, "\n--- Agent: root ---\n\nCalling list_directory(path: \"testdata/working_dir\")\n\nlist_directory response → \"FILE README.me\\n\"\n1", out)
+	require.Equal(t, "\n--- Agent: root ---\n\nCalling list_directory(path: \"testdata/working_dir\")\n\nlist_directory response → \"FILE README.me\\n\"\n2", out)
 }
 
 func TestExec_ToolCallsNeedAcceptance(t *testing.T) {

e2e/testdata/cassettes/TestExec_Anthropic_ToolCall.yaml

@@ -41,7 +41,7 @@ var _ tools.ToolSet = (*FilesystemToolset)(nil)
 // NewFilesystemToolset creates a new ACP-specific filesystem toolset
 func NewFilesystemToolset(agent *Agent, workingDir string, opts ...builtin.FileSystemOpt) *FilesystemToolset {
 	return &FilesystemToolset{
-		FilesystemTool: builtin.NewFilesystemTool([]string{workingDir}, opts...),
+		FilesystemTool: builtin.NewFilesystemTool(workingDir, opts...),
 		agent:          agent,
 		workindgDir:    workingDir,
 	}

e2e/testdata/cassettes/TestExec_Gemini_ToolCall.yaml

@@ -111,7 +111,7 @@ Can you explain to me what the agent will be used for?`,
 
 	// Custom tool registry to include fsToolset
 	fsToolset := fsToolset{
-		inner: builtin.NewFilesystemTool([]string{runConfig.WorkingDir}),
+		inner: builtin.NewFilesystemTool(runConfig.WorkingDir),
 	}
 
 	registry := teamloader.NewDefaultToolsetRegistry()

e2e/testdata/cassettes/TestExec_Mistral_ToolCall.yaml

@@ -44,7 +44,7 @@ func CollectFiles(paths []string, shouldIgnore func(path string) bool) ([]string
 
 			if info.IsDir() {
 				// Use DirectoryTree to collect files from directory
-				tree, err := DirectoryTree(normalized, func(string) error { return nil }, shouldIgnore, 0)
+				tree, err := DirectoryTree(normalized, shouldIgnore, 0)
 				if err != nil {
 					return nil, fmt.Errorf("failed to read directory %s: %w", normalized, err)
 				}