Spaces:
Sleeping
Sleeping
File size: 11,374 Bytes
1499363 | 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76 77 78 79 80 81 82 83 84 85 86 87 88 89 90 91 92 93 94 95 96 97 98 99 100 101 102 103 104 105 106 107 108 109 110 111 112 113 114 115 116 117 118 119 120 121 122 123 124 125 126 127 128 129 130 131 132 133 134 135 136 137 138 139 140 141 142 143 144 145 146 147 148 149 150 151 152 153 154 155 156 157 158 159 160 161 162 163 164 165 166 167 168 169 170 171 172 173 174 175 176 177 178 179 180 181 182 183 184 185 186 187 188 189 190 191 192 193 194 195 196 197 198 199 200 201 202 203 204 205 206 207 208 209 210 211 212 213 214 215 216 217 218 219 220 221 222 223 224 225 226 227 228 229 230 231 232 233 234 235 236 237 238 239 240 241 242 243 244 245 246 247 248 249 250 251 252 253 254 255 256 257 258 259 260 261 262 263 264 265 266 267 268 269 270 271 272 273 274 275 276 277 278 279 280 281 282 283 284 285 286 287 288 289 290 291 292 293 294 295 296 297 298 299 300 301 302 303 304 305 306 307 308 309 310 311 312 313 314 315 316 317 318 319 320 321 322 323 324 325 326 327 328 329 330 331 332 333 334 335 336 337 338 339 340 341 342 343 344 345 346 347 348 349 350 351 352 353 354 355 356 357 358 359 360 361 362 363 364 365 366 367 368 369 370 371 372 373 374 375 376 377 378 379 380 381 382 383 384 385 386 387 388 389 390 391 392 393 394 395 396 397 398 399 400 | """BashTool β execute shell commands with safety classification.
Modeled after Claude Code's BashTool security model:
- Commands classified as read-only / write / dangerous per invocation
- Dangerous commands are denied
- Write commands constrained to workspace directory
"""
from __future__ import annotations
import os
import re
import shlex
import shutil
import subprocess
from pathlib import Path
from loguru import logger
from pydantic import BaseModel, Field
from ...base import BaseTool, ToolContext
def _detect_shell() -> str:
if shutil.which("bash"):
return "bash"
return "sh"
_SHELL = _detect_shell()
# Commands that only read β safe to run anywhere
READ_ONLY_COMMANDS = {
"ls",
"cat",
"head",
"tail",
"less",
"more",
"wc",
"file",
"stat",
"find",
"grep",
"egrep",
"fgrep",
"rg",
"ag",
"awk",
"sed", # sed without -i
"diff",
"cmp",
"sort",
"uniq",
"cut",
"tr",
"tee",
"echo",
"printf",
"date",
"whoami",
"pwd",
"env",
"printenv",
"which",
"type",
"whereis",
"man",
"help",
"du",
"df",
"free",
"top",
"ps",
"uname",
"id",
"git status",
"git log",
"git diff",
"git show",
"git branch",
"git remote",
"git tag",
"git stash list",
"python --version",
"python3 --version",
"pip list",
"pip show",
"node --version",
"npm list",
"uv --version",
"tree",
"realpath",
"readlink",
"basename",
"dirname",
"md5sum",
"sha256sum",
"sha1sum",
}
# Dangerous compound patterns β matched as substrings because the composition
# itself is the danger signal (e.g. "rm -rf /" never appears legitimately).
DANGEROUS_COMPOUND_PATTERNS = {
"rm -rf /",
"rm -rf /*",
"rm -rf ~",
"dd if=",
"chmod 777",
"chmod -R 777",
"> /dev/sda",
"curl | sh",
"curl | bash",
"wget | sh",
"wget | bash",
":(){:|:&};:", # fork bomb
}
# Dangerous command NAMES β matched only at shell-command boundaries so that
# `.format()` / `literal_eval` / `# Frame format:` etc. inside Python code
# don't trigger false positives. Previously these were substring-matched,
# which blocked harmless Python one-liners.
DANGEROUS_COMMAND_NAMES = {
"mkfs",
"shutdown",
"reboot",
"halt",
"poweroff",
"eval", # shell `eval "..."`, not Python eval()
}
# Regex matches any of the dangerous command names at a shell-command
# boundary: start-of-string OR after whitespace/; /& /| /`, AND followed by
# the same or end-of-string. `(?:^|...)` is non-capturing so only the name
# itself lands in group 1.
# Suffix allows `.` so `mkfs.ext4` / `mkfs.xfs` still match the `mkfs`
# family. Prefix deliberately does NOT include `.` β that's what keeps
# Python's `.format()` / `.literal_eval` from triggering.
_DANGEROUS_NAME_RE = re.compile(
r"(?:^|[\s;&|`])("
+ "|".join(re.escape(n) for n in DANGEROUS_COMMAND_NAMES)
+ r")(?=[\s;&|`.]|$)",
re.IGNORECASE,
)
# Dangerous environment variables that should never be set
DANGEROUS_ENV_VARS = {
"PATH",
"LD_PRELOAD",
"LD_LIBRARY_PATH",
"DYLD_INSERT_LIBRARIES",
"DYLD_LIBRARY_PATH",
"NODE_OPTIONS",
"PYTHONPATH",
"BASH_ENV",
}
def _get_first_command(command: str) -> str:
"""Extract the first command from a compound command."""
# Split on &&, ||, ;, | and take first
for sep in ["&&", "||", ";", "|"]:
if sep in command:
return command.split(sep)[0].strip()
return command.strip()
def _classify_command(command: str) -> str:
"""Classify a command as 'read-only', 'write', or 'dangerous'.
Returns one of: 'read-only', 'write', 'dangerous'
"""
cmd = command.strip()
cmd_lower = cmd.lower()
# Compound patterns (substring): the phrasing IS the signal.
for pattern in DANGEROUS_COMPOUND_PATTERNS:
if pattern in cmd_lower:
return "dangerous"
# Command names at shell-command boundaries only.
if _DANGEROUS_NAME_RE.search(cmd):
return "dangerous"
# Check for dangerous env var manipulation
for var in DANGEROUS_ENV_VARS:
if f"{var}=" in cmd and not cmd.startswith("echo"):
return "dangerous"
# Check for command substitution / eval
if "$()" in cmd or "`" in cmd:
# Could be read-only (e.g., echo $(date)) but treat as write conservatively
pass
# Extract the base command
first_cmd = _get_first_command(cmd)
try:
parts = shlex.split(first_cmd)
except ValueError:
parts = first_cmd.split()
if not parts:
return "read-only"
base = os.path.basename(parts[0])
# Check read-only commands
if base in READ_ONLY_COMMANDS:
# sed with -i is a write, not read-only
if base == "sed" and "-i" in parts:
return "write"
return "read-only"
# Check multi-word read-only (e.g., "git status")
two_word = f"{base} {parts[1]}" if len(parts) > 1 else ""
if two_word in READ_ONLY_COMMANDS:
return "read-only"
# Known write commands
if base in {
"rm",
"mv",
"cp",
"mkdir",
"rmdir",
"touch",
"chmod",
"chown",
"ln",
"install",
"git commit",
"git push",
"git checkout",
"git reset",
"git merge",
"git rebase",
}:
return "write"
# Package managers, compilers, etc. β write
if base in {
"pip",
"pip3",
"npm",
"yarn",
"pnpm",
"uv",
"apt",
"apt-get",
"brew",
"cargo",
"go",
"gcc",
"g++",
"make",
"cmake",
"python",
"python3",
"node",
"docker",
"docker-compose",
}:
return "write"
# Default: write (conservative)
return "write"
class RunBashCmdInput(BaseModel):
command: str = Field(description=f"The {_SHELL} command to execute")
cwd: str | None = Field(
default=None,
description="Working directory for the command (optional)",
)
timeout: int = Field(
default=120,
description="Timeout in seconds (default 120, max 43200 i.e. 12 hours)",
)
run_in_background: bool = Field(
default=False,
description=(
"Set to true to run this command in the background. "
"Returns a task ID immediately. Use TaskOutput to read the output later."
),
)
class RunBashCmdTool(BaseTool):
name = "Bash"
description = (
f"Execute a {_SHELL} command and return its output (stdout and stderr combined). "
f"Use this for shell operations. Prefer dedicated tools (Read, Glob, Grep) "
f"for file reading and searching."
)
input_schema = RunBashCmdInput
prompt = (
"# Bash tool usage\n"
"- Do NOT use Bash to read files (use Read), search files (use Glob), "
"or search content (use Grep). Reserve Bash for commands that require shell execution.\n"
"- Always quote file paths with spaces using double quotes.\n"
"- When issuing multiple independent commands, make separate tool calls in parallel.\n"
"- For long-running commands (training, builds, large data processing), "
"use `run_in_background=true` and check results later with TaskOutput.\n"
"- Foreground commands are capped at 10 minutes. Background commands allow up to 12 hours.\n"
"- If a command fails, diagnose why before retrying. Do not blindly re-run.\n"
"- Avoid unnecessary `sleep` commands β use `run_in_background` instead.\n"
)
def is_read_only(self, **kwargs) -> bool:
"""Bash is read-only only if the specific command is classified as read-only.
When called without a command (e.g., plan mode static check), returns False
because Bash CAN write β it depends on the specific command.
"""
command = kwargs.get("command")
if not command:
return False # Bash can write, so not read-only by default
return _classify_command(command) == "read-only"
def check_permissions(self, kwargs: dict, context: ToolContext) -> "PermissionResult":
"""Check command safety and path constraints."""
from scider.core.permissions import allow, deny, is_dangerous_path
command = kwargs.get("command", "")
classification = _classify_command(command)
# Dangerous commands are always denied
if classification == "dangerous":
return deny(f"Dangerous command blocked: {command[:100]}")
# Check for dangerous path access in write commands
if classification == "write":
# Extract paths from command (simple heuristic)
parts = command.split()
for part in parts:
if part.startswith("/") or part.startswith("~") or part.startswith("."):
if is_dangerous_path(part):
return deny(f"Write to dangerous path blocked: {part}")
return allow()
def call(
self,
context: ToolContext,
*,
command: str,
cwd: str | None = None,
timeout: int = 120,
run_in_background: bool = False,
) -> str:
working_dir = None
if cwd:
working_dir_path = Path(os.path.expandvars(cwd)).expanduser()
if not working_dir_path.exists():
return f"Error: Working directory '{cwd}' does not exist"
working_dir = str(working_dir_path)
# Background execution: spawn task and return immediately
# Background allows up to 12 hours; foreground capped at 10 minutes
if run_in_background:
timeout = max(1, min(timeout, 43200)) # up to 12 hours
from scider.core.task import TaskManager
task_id = TaskManager.spawn_shell(
command=command,
cwd=working_dir,
timeout=timeout,
description=command[:100],
)
return (
f"Command is running in the background.\n"
f"Task ID: {task_id}\n"
f'Use TaskOutput(task_id="{task_id}") to check status and read output.'
)
# Foreground execution (capped at 10 min to avoid blocking the agent loop)
timeout = max(1, min(timeout, 600))
try:
result = subprocess.run(
[_SHELL, "-c", command],
capture_output=True,
text=True,
cwd=working_dir,
timeout=timeout,
)
output = []
if result.stdout:
output.append(f"STDOUT:\n{result.stdout}")
if result.stderr:
output.append(f"STDERR:\n{result.stderr}")
output.append(f"\nReturn code: {result.returncode}")
return "\n".join(output) if output else "Command executed with no output"
except subprocess.TimeoutExpired:
return f"Error: Command timed out after {timeout} seconds"
except Exception as e:
return f"Error executing command: {e}"
|