mirror of
https://github.com/aljazceru/Auto-GPT.git
synced 2025-12-17 22:14:28 +01:00
This incremental re-architecture unifies Agent code and plugins, so everything is component-based. ## Breaking changes - Removed command categories and `DISABLED_COMMAND_CATEGORIES` environment variable. Use `DISABLED_COMMANDS` environment variable to disable individual commands. - Changed `command` decorator; old-style commands are no longer supported. Implement `CommandProvider` on components instead. - Removed `CommandRegistry`, now all commands are provided by components implementing `CommandProvider`. - Removed `prompt_config` from `AgentSettings`. - Removed plugin support: old plugins will no longer be loaded and executed. - Removed `PromptScratchpad`, it was used by plugins and is no longer needed. - Changed `ThoughtProcessOutput` from tuple to pydantic `BaseModel`. ## Other changes - Created `AgentComponent`, protocols and logic to execute them. - `BaseAgent` and `Agent` is now composed of components. - Moved some logic from `BaseAgent` to `Agent`. - Moved agent features and commands to components. - Removed check if the same operation is about to be executed twice in a row. - Removed file logging from `FileManagerComponent` (formerly `AgentFileManagerMixin`) - Updated tests - Added docs See [Introduction](https://github.com/kcze/AutoGPT/blob/kpczerwinski/open-440-modular-agents/docs/content/AutoGPT/component%20agent/introduction.md) for more information.
168 lines
5.2 KiB
Python
168 lines
5.2 KiB
Python
import random
|
|
import string
|
|
import tempfile
|
|
from pathlib import Path
|
|
|
|
import pytest
|
|
|
|
from autogpt.agents.agent import Agent
|
|
from autogpt.commands.execute_code import (
|
|
ALLOWLIST_CONTROL,
|
|
CodeExecutorComponent,
|
|
is_docker_available,
|
|
we_are_running_in_a_docker_container,
|
|
)
|
|
from autogpt.utils.exceptions import InvalidArgumentError, OperationNotAllowedError
|
|
|
|
|
|
@pytest.fixture
|
|
def code_executor_component(agent: Agent):
|
|
return agent.code_executor
|
|
|
|
|
|
@pytest.fixture
|
|
def random_code(random_string) -> str:
|
|
return f"print('Hello {random_string}!')"
|
|
|
|
|
|
@pytest.fixture
|
|
def python_test_file(agent: Agent, random_code: str):
|
|
temp_file = tempfile.NamedTemporaryFile(
|
|
dir=agent.file_manager.workspace.root, suffix=".py"
|
|
)
|
|
temp_file.write(str.encode(random_code))
|
|
temp_file.flush()
|
|
|
|
yield Path(temp_file.name)
|
|
temp_file.close()
|
|
|
|
|
|
@pytest.fixture
|
|
def python_test_args_file(agent: Agent):
|
|
temp_file = tempfile.NamedTemporaryFile(
|
|
dir=agent.file_manager.workspace.root, suffix=".py"
|
|
)
|
|
temp_file.write(str.encode("import sys\nprint(sys.argv[1], sys.argv[2])"))
|
|
temp_file.flush()
|
|
|
|
yield Path(temp_file.name)
|
|
temp_file.close()
|
|
|
|
|
|
@pytest.fixture
|
|
def random_string():
|
|
return "".join(random.choice(string.ascii_lowercase) for _ in range(10))
|
|
|
|
|
|
def test_execute_python_file(
|
|
code_executor_component: CodeExecutorComponent,
|
|
python_test_file: Path,
|
|
random_string: str,
|
|
agent: Agent,
|
|
):
|
|
if not (is_docker_available() or we_are_running_in_a_docker_container()):
|
|
pytest.skip("Docker is not available")
|
|
|
|
result: str = code_executor_component.execute_python_file(python_test_file)
|
|
assert result.replace("\r", "") == f"Hello {random_string}!\n"
|
|
|
|
|
|
def test_execute_python_file_args(
|
|
code_executor_component: CodeExecutorComponent,
|
|
python_test_args_file: Path,
|
|
random_string: str,
|
|
agent: Agent,
|
|
):
|
|
if not (is_docker_available() or we_are_running_in_a_docker_container()):
|
|
pytest.skip("Docker is not available")
|
|
|
|
random_args = [random_string] * 2
|
|
random_args_string = " ".join(random_args)
|
|
result = code_executor_component.execute_python_file(
|
|
python_test_args_file, args=random_args
|
|
)
|
|
assert result == f"{random_args_string}\n"
|
|
|
|
|
|
def test_execute_python_code(
|
|
code_executor_component: CodeExecutorComponent,
|
|
random_code: str,
|
|
random_string: str,
|
|
agent: Agent,
|
|
):
|
|
if not (is_docker_available() or we_are_running_in_a_docker_container()):
|
|
pytest.skip("Docker is not available")
|
|
|
|
result: str = code_executor_component.execute_python_code(random_code)
|
|
assert result.replace("\r", "") == f"Hello {random_string}!\n"
|
|
|
|
|
|
def test_execute_python_file_invalid(
|
|
code_executor_component: CodeExecutorComponent, agent: Agent
|
|
):
|
|
with pytest.raises(InvalidArgumentError):
|
|
code_executor_component.execute_python_file(Path("not_python.txt"))
|
|
|
|
|
|
def test_execute_python_file_not_found(
|
|
code_executor_component: CodeExecutorComponent, agent: Agent
|
|
):
|
|
with pytest.raises(
|
|
FileNotFoundError,
|
|
match=r"python: can't open file '([a-zA-Z]:)?[/\\\-\w]*notexist.py': "
|
|
r"\[Errno 2\] No such file or directory",
|
|
):
|
|
code_executor_component.execute_python_file(Path("notexist.py"))
|
|
|
|
|
|
def test_execute_shell(
|
|
code_executor_component: CodeExecutorComponent, random_string: str, agent: Agent
|
|
):
|
|
result = code_executor_component.execute_shell(f"echo 'Hello {random_string}!'")
|
|
assert f"Hello {random_string}!" in result
|
|
|
|
|
|
def test_execute_shell_local_commands_not_allowed(
|
|
code_executor_component: CodeExecutorComponent, random_string: str, agent: Agent
|
|
):
|
|
result = code_executor_component.execute_shell(f"echo 'Hello {random_string}!'")
|
|
assert f"Hello {random_string}!" in result
|
|
|
|
|
|
def test_execute_shell_denylist_should_deny(
|
|
code_executor_component: CodeExecutorComponent, agent: Agent, random_string: str
|
|
):
|
|
agent.legacy_config.shell_denylist = ["echo"]
|
|
|
|
with pytest.raises(OperationNotAllowedError, match="not allowed"):
|
|
code_executor_component.execute_shell(f"echo 'Hello {random_string}!'")
|
|
|
|
|
|
def test_execute_shell_denylist_should_allow(
|
|
code_executor_component: CodeExecutorComponent, agent: Agent, random_string: str
|
|
):
|
|
agent.legacy_config.shell_denylist = ["cat"]
|
|
|
|
result = code_executor_component.execute_shell(f"echo 'Hello {random_string}!'")
|
|
assert "Hello" in result and random_string in result
|
|
|
|
|
|
def test_execute_shell_allowlist_should_deny(
|
|
code_executor_component: CodeExecutorComponent, agent: Agent, random_string: str
|
|
):
|
|
agent.legacy_config.shell_command_control = ALLOWLIST_CONTROL
|
|
agent.legacy_config.shell_allowlist = ["cat"]
|
|
|
|
with pytest.raises(OperationNotAllowedError, match="not allowed"):
|
|
code_executor_component.execute_shell(f"echo 'Hello {random_string}!'")
|
|
|
|
|
|
def test_execute_shell_allowlist_should_allow(
|
|
code_executor_component: CodeExecutorComponent, agent: Agent, random_string: str
|
|
):
|
|
agent.legacy_config.shell_command_control = ALLOWLIST_CONTROL
|
|
agent.legacy_config.shell_allowlist = ["echo"]
|
|
|
|
result = code_executor_component.execute_shell(f"echo 'Hello {random_string}!'")
|
|
assert "Hello" in result and random_string in result
|