fix: Renamed Shell to Command

Author: codefionn

internal/llm/system_prompt_template.go

@@ -50,12 +50,12 @@ You are an interactive CLI agent specializing in software engineering tasks. You
 - Use the todo tool for task tracking and progress updates.
 - Use sandbox and debugging tools to validate hypotheses before risky changes.
 - Surface uncertainties to the user rather than guessing.
-- Validate your changes through building / linting / testing (e.g. with Shell method in sandbox tool)
+- Validate your changes through building / linting / testing (e.g. with Command method in sandbox tool)
 - Use the parallel_tool to execute multiple tools (e.g. multiple search_files, search_file_content, read_file) concurrently.
 - Use the codebase_investigator tool to gather context about the codebase
   (only for searching and reading more files, what files are relevant for editing, where certain logic is implemented, etc.).
 - Try to read up documentation of libraries and APIs you're using
- (e.g. read files downloaded to system e.g. in node_modules or go/pkg, can be done with the context tool calls or web search tool, or Shell method in the golang sandbox)
+ (e.g. read files downloaded to system e.g. in node_modules or go/pkg, can be done with the context tool calls or web search tool, or Command method in the golang sandbox)
 - At least partially read files before modifying them
 
 ## Task Tracking
@@ -69,7 +69,7 @@ You are an interactive CLI agent specializing in software engineering tasks. You
     - Creating files
     - Building and testing
   3. Draft new files
-  4. Build and test your changes (using tool calls like 'go_sandbox' using the Shell method and 'tool_summarize')
+  4. Build and test your changes (using tool calls like 'go_sandbox' using the Command method and 'tool_summarize')
   5. Give a very short explanation how the user can get started with the project
 - **Modify Existing Project**
   1. Gather context: inspect context relating to the task (with codebase investigator)
@@ -78,7 +78,7 @@ You are an interactive CLI agent specializing in software engineering tasks. You
     - Implementing changes
     - Building and testing
   3. Update files and create new ones
-  4. Build and test your changes (using tool calls like 'go_sandbox' using the Shell method)
+  4. Build and test your changes (using tool calls like 'go_sandbox' using the Command method)
     - Fix your changes
     - Rebuild and test after fixing
   5. Give a very short explanation what was done and how the user can test it
@@ -87,7 +87,7 @@ You are an interactive CLI agent specializing in software engineering tasks. You
   2. Answer the question
 - **Fix failing Tests or Build**
   1. Run tests or build (investigate what tooling is required only if necessary, otherwise the context may be sufficient)
-    - Use the go_sandbox tool to run tests (e.g. with the Shell method)
+    - Use the go_sandbox tool to run tests (e.g. with the Command method)
     - Try to extract only errors from the output with the tool_summarize and/or searching it in the go_sandbox tool
   2. Gather context about errors (with codebase investigator)
   3. If necessary, think about what are root causes for the errors and how to fix them

internal/tools/sandbox.go

@@ -251,7 +251,7 @@ func (t *SandboxTool) Description() string {
 	b.WriteString("     }\n")
 	b.WriteString("     ```\n\n")
 
-	b.WriteString("2. Shell(command []string, stdin string) (stdout string, stderr string, exitCode int)\n")
+	b.WriteString("2. Command(command []string, stdin string) (stdout string, stderr string, exitCode int)\n")
 	b.WriteString("   - Execute shell commands with optional stdin input\n")
 	b.WriteString("   - Requires command authorization\n")
 	b.WriteString("   - Pass empty string for stdin if not needed\n")
@@ -260,13 +260,13 @@ func (t *SandboxTool) Description() string {
 	b.WriteString("     package main\n\n")
 	b.WriteString("     import \"fmt\"\n\n")
 	b.WriteString("     func main() {\n")
-	b.WriteString("         out, err, code := Shell([]string{\"ls\", \"-la\"}, \"\")\n")
+	b.WriteString("         out, err, code := Command([]string{\"ls\", \"-la\"}, \"\")\n")
 	b.WriteString("         fmt.Printf(\"ls output: %s, stderr: %s, exit: %d\\n\", out, err, code)\n\n")
-	b.WriteString("         out, err, code = Shell([]string{\"grep\", \"pattern\"}, \"line1\\nline2\\npattern here\\n\")\n")
+	b.WriteString("         out, err, code = Command([]string{\"grep\", \"pattern\"}, \"line1\\nline2\\npattern here\\n\")\n")
 	b.WriteString("         fmt.Printf(\"grep output: %s, stderr: %s, exit: %d\\n\", out, err, code)\n\n")
-	b.WriteString("         out, err, code = Shell([]string{\"go\", \"build\", \"./cmd/statcode-ai\"}, \"\")\n")
+	b.WriteString("         out, err, code = Command([]string{\"go\", \"build\", \"./cmd/statcode-ai\"}, \"\")\n")
 	b.WriteString("         fmt.Printf(\"go build output: %s, stderr: %s, exit: %d\\n\", out, err, code)\n\n")
-	b.WriteString("         _, err, code = Shell([]string{\"mkdir\", \"-p\", \"tmp/build/cache\"}, \"\")\n")
+	b.WriteString("         _, err, code = Command([]string{\"mkdir\", \"-p\", \"tmp/build/cache\"}, \"\")\n")
 	b.WriteString("         fmt.Printf(\"mkdir stderr: %s, exit: %d\\n\", err, code)\n")
 	b.WriteString("     }\n")
 	b.WriteString("     ```\n\n")
@@ -442,7 +442,7 @@ func (t *SandboxTool) Description() string {
 	b.WriteString(")\n")
 	b.WriteString("\n")
 	b.WriteString("func main() {\n")
-	b.WriteString("  stdout, stderr, code := Shell([]string{\"go\", \"build\", \"./...\"}, \"\")\n")
+	b.WriteString("  stdout, stderr, code := Command([]string{\"go\", \"build\", \"./...\"}, \"\")\n")
 	b.WriteString("  if err == 0 {\n")
 	b.WriteString("    fmt.Println(\"Compiled successfully\")\n")
 	b.WriteString("  } else {\n")
@@ -937,7 +937,7 @@ func (t *SandboxTool) executeWASM(ctx context.Context, wasmBytes []byte, sandbox
 
 			// Check authorization for command
 			if adapter != nil && adapter.authorizer != nil {
-				decision, err := adapter.Authorize(ctx, ToolNameShell, map[string]interface{}{
+				decision, err := adapter.Authorize(ctx, ToolNameCommand, map[string]interface{}{
 					"command":      commandDisplay,
 					"command_args": commandArgs,
 				})
@@ -1011,7 +1011,7 @@ func (t *SandboxTool) executeWASM(ctx context.Context, wasmBytes []byte, sandbox
 
 			return int32(exitCode)
 		}).
-		Export(ToolNameShell)
+		Export(ToolNameCommand)
 
 	// summarize(prompt_ptr, prompt_len, text_ptr, text_len, result_ptr, result_cap) -> status_code
 	envBuilder.NewFunctionBuilder().

internal/tools/tool_names.go

@@ -7,6 +7,7 @@ const (
 	ToolNameWriteFileDiff        = "write_file_diff"
 	ToolNameWriteFileJSON        = "write_file_json"
 	ToolNameShell                = "shell"
+	ToolNameCommand              = "command"
 	ToolNameStatusProgram        = "status_program"
 	ToolNameWaitProgram          = "wait_program"
 	ToolNameStopProgram          = "stop_program"