Merge pull request #939 from dgageot/shell-current

Passing . to shell tools is not required

Author: dgageot

pkg/tools/builtin/shell.go

@@ -100,13 +100,13 @@ func (lw *limitedWriter) Write(p []byte) (n int, err error) {
 
 type RunShellArgs struct {
 	Cmd     string `json:"cmd" jsonschema:"The shell command to execute"`
-	Cwd     string `json:"cwd" jsonschema:"The working directory to execute the command in"`
+	Cwd     string `json:"cwd,omitempty" jsonschema:"The working directory to execute the command in (default: \".\")"`
 	Timeout int    `json:"timeout,omitempty" jsonschema:"Command execution timeout in seconds (default: 30)"`
 }
 
 type RunShellBackgroundArgs struct {
 	Cmd string `json:"cmd" jsonschema:"The shell command to execute in the background"`
-	Cwd string `json:"cwd" jsonschema:"The working directory to execute the command in"`
+	Cwd string `json:"cwd,omitempty" jsonschema:"The working directory to execute the command in (default: \".\")"`
 }
 
 type ViewBackgroundJobArgs struct {

pkg/tools/builtin/shell_test.go

@@ -52,7 +52,7 @@ func TestShellTool_Tools(t *testing.T) {
 			"type": "string"
 		},
 		"cwd": {
-			"description": "The working directory to execute the command in",
+			"description": "The working directory to execute the command in (default: \".\")",
 			"type": "string"
 		},
 		"timeout": {
@@ -62,8 +62,7 @@ func TestShellTool_Tools(t *testing.T) {
 	},
 	"additionalProperties": false,
 	"required": [
-		"cmd",
-		"cwd"
+		"cmd"
 	]
 }`, string(schema))
 }