refactor(shell.py): rename expected_runtime_seconds parameter to timeout for clarity and consistency in function signature
This commit is contained in:
parent
81354df48b
commit
9ff09b9b93
|
|
@ -22,13 +22,13 @@ def _truncate_for_log(text: str, max_length: int = 300) -> str:
|
|||
|
||||
@tool
|
||||
def run_shell_command(
|
||||
command: str, expected_runtime_seconds: int = 30
|
||||
command: str, timeout: int = 30
|
||||
) -> Dict[str, Union[str, int, bool]]:
|
||||
"""Execute a shell command and return its output.
|
||||
|
||||
Args:
|
||||
command: The shell command to execute
|
||||
expected_runtime_seconds: Expected runtime in seconds, defaults to 30.
|
||||
timeout: Expected runtime in seconds, defaults to 30.
|
||||
If process exceeds 2x this value, it will be terminated gracefully.
|
||||
If process exceeds 3x this value, it will be killed forcefully.
|
||||
|
||||
|
|
@ -83,7 +83,7 @@ def run_shell_command(
|
|||
print()
|
||||
output, return_code = run_interactive_command(
|
||||
["/bin/bash", "-c", command],
|
||||
expected_runtime_seconds=expected_runtime_seconds,
|
||||
expected_runtime_seconds=timeout,
|
||||
)
|
||||
print()
|
||||
result = {
|
||||
|
|
|
|||
Loading…
Reference in New Issue