b6b4cd0a4c
### Overview Major restructuring from monolithic `arcade-ai` package to modular library architecture with standardized uv-based dependency management.  ### New Package Structure - **`arcade-tdk`** - Lightweight toolkit development kit (core decorators, auth) - **`arcade-core`** - Core execution engine and catalog functionality - **`arcade-serve`** - FastAPI/MCP server components - **`arcade-ai`** - Meta package that includes CLI functionality. Optionally include evals via the `evals` extra. Optionally include all packages via the `all` extra. ### Key Benefits - **Lighter Dependencies**: Toolkits now depend only on `arcade-tdk` (~2 deps) vs full `arcade-ai` (~30+ deps) - **Faster Builds**: uv provides 10-100x faster dependency resolution and installation - **Better Modularity**: Clear separation of concerns, consumers import only what they need - **Standard Tooling**: Eliminates custom poetry scripts, uses standard Python packaging ### Migration Impact - All 20 toolkits converted from poetry → uv with `arcade-tdk` dependencies plus `arcade-ai[evals]` and `arcade-serve` dev dependencies. When developing locally, devs should install toolkits via `make install-local`. - Modern Python 3.10+ type hints throughout - Standardized build system with hatchling backend - Enhanced Makefile with robust toolkit management commands - Removed `arcade dev` CLI command - Reduce the number of files created by `arcade new` and add an option to not generate a tests and evals folder. This foundation enables faster development cycles and cleaner dependency chains for the growing toolkit ecosystem. ### Todo After this PR is merged - [ ] Post-merge workflow(s) (release & publish containers, etc) - [ ] Release order plan. @EricGustin suggests releasing in the following order: 1. `arcade-core` version 0.1.0 2. `arcade-serve` version 0.1.0 and `arcade-tdk` version 0.1.0 3. `arcade-ai` version 2.0.0 4. Patch release for all toolkits (all changes in toolkits are internal refactors) - [ ] [Update docs](https://github.com/ArcadeAI/docs/pull/318) --------- Co-authored-by: Eric Gustin <eric@arcade.dev> Co-authored-by: Eric Gustin <34000337+EricGustin@users.noreply.github.com>
103 lines
3.3 KiB
Python
103 lines
3.3 KiB
Python
from unittest.mock import AsyncMock, patch
|
|
|
|
import pytest
|
|
from arcade_tdk.errors import ToolExecutionError
|
|
from httpx import Response
|
|
|
|
from arcade_github.tools.activity import list_stargazers, set_starred
|
|
|
|
|
|
@pytest.fixture
|
|
def mock_context():
|
|
context = AsyncMock()
|
|
context.authorization.token = "mock_token" # noqa: S105
|
|
return context
|
|
|
|
|
|
@pytest.fixture
|
|
def mock_client():
|
|
with patch("arcade_github.tools.activity.httpx.AsyncClient") as client:
|
|
yield client.return_value.__aenter__.return_value
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
@pytest.mark.parametrize(
|
|
"starred,expected_message",
|
|
[
|
|
(True, "Successfully starred the repository owner/repo"),
|
|
(False, "Successfully unstarred the repository owner/repo"),
|
|
],
|
|
)
|
|
async def test_set_starred_success(mock_context, mock_client, starred, expected_message):
|
|
mock_client.put.return_value = mock_client.delete.return_value = Response(204)
|
|
|
|
result = await set_starred(mock_context, "owner", "repo", starred)
|
|
assert result == expected_message
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
@pytest.mark.parametrize(
|
|
"status_code,error_message,expected_error",
|
|
[
|
|
(403, "Forbidden", "Error accessing.*: Forbidden"),
|
|
(404, "Not Found", "Error accessing.*: Resource not found"),
|
|
(500, "Internal Server Error", "Error accessing.*: Failed to process request"),
|
|
],
|
|
)
|
|
async def test_set_starred_errors(
|
|
mock_context, mock_client, status_code, error_message, expected_error
|
|
):
|
|
mock_client.put.return_value = mock_client.delete.return_value = Response(
|
|
status_code, json={"message": error_message}
|
|
)
|
|
|
|
with pytest.raises(ToolExecutionError, match=expected_error):
|
|
await set_starred(mock_context, "owner", "repo", True)
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_list_stargazers_success(mock_context, mock_client):
|
|
mock_response_data = [
|
|
{
|
|
"login": "user1",
|
|
"id": 1,
|
|
"node_id": "MDQ6VXNlcjE=",
|
|
"html_url": "https://github.com/user1",
|
|
},
|
|
{
|
|
"login": "user2",
|
|
"id": 2,
|
|
"node_id": "MDQ6VXNlcjI=",
|
|
"html_url": "https://github.com/user2",
|
|
},
|
|
]
|
|
mock_client.get.return_value = Response(200, json=mock_response_data)
|
|
|
|
result = await list_stargazers(mock_context, "owner", "repo", limit=2)
|
|
assert result == {"number_of_stargazers": 2, "stargazers": mock_response_data}
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_list_stargazers_empty(mock_context, mock_client):
|
|
mock_client.get.return_value = Response(200, json=[])
|
|
|
|
result = await list_stargazers(mock_context, "owner", "repo")
|
|
assert result == {"number_of_stargazers": 0, "stargazers": []}
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
@pytest.mark.parametrize(
|
|
"status_code,error_message,expected_error",
|
|
[
|
|
(403, "Forbidden", "Error accessing.*: Forbidden"),
|
|
(404, "Not Found", "Error accessing.*: Resource not found"),
|
|
(500, "Internal Server Error", "Error accessing.*: Failed to process request"),
|
|
],
|
|
)
|
|
async def test_list_stargazers_errors(
|
|
mock_context, mock_client, status_code, error_message, expected_error
|
|
):
|
|
mock_client.get.return_value = Response(status_code, json={"message": error_message})
|
|
|
|
with pytest.raises(ToolExecutionError, match=expected_error):
|
|
await list_stargazers(mock_context, "owner", "repo")
|