thirdmagic 0.0.1__tar.gz

thirdmagic-0.0.1/.gitignore

@@ -0,0 +1,231 @@
+# Byte-compiled / optimized / DLL files
+__pycache__/
+*.py[codz]
+*$py.class
+
+# C extensions
+*.so
+
+# Distribution / packaging
+.Python
+build/
+develop-eggs/
+dist/
+downloads/
+eggs/
+.eggs/
+lib/
+lib64/
+parts/
+sdist/
+var/
+wheels/
+share/python-wheels/
+*.egg-info/
+.installed.cfg
+*.egg
+MANIFEST
+
+# PyInstaller
+#  Usually these files are written by a python script from a template
+#  before PyInstaller builds the exe, so as to inject date/other infos into it.
+*.manifest
+*.spec
+
+# Installer logs
+pip-log.txt
+pip-delete-this-directory.txt
+
+# Unit test / coverage reports
+htmlcov/
+.tox/
+.nox/
+.coverage
+.coverage.*
+.cache
+nosetests.xml
+coverage.xml
+*.cover
+*.py.cover
+.hypothesis/
+.pytest_cache/
+cover/
+
+# Translations
+*.mo
+*.pot
+
+# Django stuff:
+*.log
+local_settings.py
+db.sqlite3
+db.sqlite3-journal
+
+# Flask stuff:
+instance/
+.webassets-cache
+
+# Scrapy stuff:
+.scrapy
+
+# Sphinx documentation
+docs/_build/
+
+# PyBuilder
+.pybuilder/
+target/
+
+# Jupyter Notebook
+.ipynb_checkpoints
+
+# IPython
+profile_default/
+ipython_config.py
+
+# pyenv
+#   For a library or package, you might want to ignore these files since the code is
+#   intended to run in multiple environments; otherwise, check them in:
+# .python-version
+
+# pipenv
+#   According to pypa/pipenv#598, it is recommended to include Pipfile.lock in version control.
+#   However, in case of collaboration, if having platform-specific dependencies or dependencies
+#   having no cross-platform support, pipenv may install dependencies that don't work, or not
+#   install all needed dependencies.
+#Pipfile.lock
+
+# UV
+#   Similar to Pipfile.lock, it is generally recommended to include uv.lock in version control.
+#   This is especially recommended for binary packages to ensure reproducibility, and is more
+#   commonly ignored for libraries.
+#uv.lock
+
+# poetry
+#   Similar to Pipfile.lock, it is generally recommended to include poetry.lock in version control.
+#   This is especially recommended for binary packages to ensure reproducibility, and is more
+#   commonly ignored for libraries.
+#   https://python-poetry.org/docs/basic-usage/#commit-your-poetrylock-file-to-version-control
+#poetry.lock
+#poetry.toml
+
+# pdm
+#   Similar to Pipfile.lock, it is generally recommended to include pdm.lock in version control.
+#   pdm recommends including project-wide configuration in pdm.toml, but excluding .pdm-python.
+#   https://pdm-project.org/en/latest/usage/project/#working-with-version-control
+#pdm.lock
+#pdm.toml
+.pdm-python
+.pdm-build/
+
+# pixi
+#   Similar to Pipfile.lock, it is generally recommended to include pixi.lock in version control.
+#pixi.lock
+#   Pixi creates a virtual environment in the .pixi directory, just like venv module creates one
+#   in the .venv directory. It is recommended not to include this directory in version control.
+.pixi
+
+# PEP 582; used by e.g. github.com/David-OConnor/pyflow and github.com/pdm-project/pdm
+__pypackages__/
+
+# Celery stuff
+celerybeat-schedule
+celerybeat.pid
+
+# SageMath parsed files
+*.sage.py
+
+# Environments
+.env
+.envrc
+.venv
+env/
+venv/
+ENV/
+env.bak/
+venv.bak/
+
+# Spyder project settings
+.spyderproject
+.spyproject
+
+# Rope project settings
+.ropeproject
+
+# mkdocs documentation
+/site
+docs/roadmap.md
+
+# mypy
+.mypy_cache/
+.dmypy.json
+dmypy.json
+
+# Pyre type checker
+.pyre/
+
+# pytype static type analyzer
+.pytype/
+
+# Cython debug symbols
+cython_debug/
+
+# PyCharm
+#  JetBrains specific template is maintained in a separate JetBrains.gitignore that can
+#  be found at https://github.com/github/gitignore/blob/main/Global/JetBrains.gitignore
+#  and can be added to the global gitignore or merged into this file.  For a more nuclear
+#  option (not recommended) you can uncomment the following to ignore the entire idea folder.
+#.idea/
+
+# Abstra
+# Abstra is an AI-powered process automation framework.
+# Ignore directories containing user credentials, local state, and settings.
+# Learn more at https://abstra.io/docs
+.abstra/
+
+# Visual Studio Code
+#  Visual Studio Code specific template is maintained in a separate VisualStudioCode.gitignore
+#  that can be found at https://github.com/github/gitignore/blob/main/Global/VisualStudioCode.gitignore
+#  and can be added to the global gitignore or merged into this file. However, if you prefer,
+#  you could uncomment the following to ignore the entire vscode folder
+# .vscode/
+
+# Ruff stuff:
+.ruff_cache/
+
+# PyPI configuration file
+.pypirc
+
+# Cursor
+#  Cursor is an AI-powered code editor. `.cursorignore` specifies files/directories to
+#  exclude from AI features like autocomplete and code analysis. Recommended for sensitive data
+#  refer to https://docs.cursor.com/context/ignore-files
+.cursorignore
+.cursorindexingignore
+
+# Marimo
+marimo/_static/
+marimo/_lsp/
+__marimo__/
+/.idea/
+.idea/
+
+/.junie/
+CLAUDE.md
+.vscode
+future-features
+
+# Ignore dynaconf secret files
+.secrets.*
+.claude
+/junit.xml
+.jbeval
+
+# Mageflow visualizer static build (generated during CI)
+mageflow/visualizer/static/*
+!mageflow/visualizer/static/.gitkeep
+app/node_modules/
+frontend/node_modules/
+/app/.idea/
+/scripts
+/frontend/.idea/
+.planning/

thirdmagic-0.0.1/PKG-INFO

@@ -0,0 +1,16 @@
+Metadata-Version: 2.4
+Name: thirdmagic
+Version: 0.0.1
+Summary: Core models and signatures for mageflow task orchestration
+Author-email: imaginary-cherry <yedidyakfir@gmail.com>
+License: MIT
+Requires-Python: <3.14,>=3.10
+Requires-Dist: pydantic<3.0.0,>=2.0.0
+Requires-Dist: rapyer<1.3.0,>=1.2.3
+Provides-Extra: dev
+Requires-Dist: black>=26.1.0; extra == 'dev'
+Requires-Dist: coverage[toml]<8.0.0,>=7.0.0; extra == 'dev'
+Requires-Dist: fakeredis[json,lua]<3.0.0,>=2.32.1; extra == 'dev'
+Requires-Dist: hatchet-sdk>=1.23.0; extra == 'dev'
+Requires-Dist: pytest-asyncio<2.0.0,>=1.2.0; extra == 'dev'
+Requires-Dist: pytest<10.0.0,>=9.0.2; extra == 'dev'

thirdmagic-0.0.1/pyproject.toml

@@ -0,0 +1,53 @@
+[project]
+name = "thirdmagic"
+version = "0.0.1"
+description = "Core models and signatures for mageflow task orchestration"
+authors = [
+    {name = "imaginary-cherry", email = "yedidyakfir@gmail.com"}
+]
+license = {text = "MIT"}
+requires-python = ">=3.10,<3.14"
+dependencies = [
+    "rapyer>=1.2.3,<1.3.0",
+    "pydantic>=2.0.0,<3.0.0",
+]
+[project.optional-dependencies]
+dev = [
+    "pytest>=9.0.2,<10.0.0",
+    "pytest-asyncio>=1.2.0,<2.0.0",
+    "fakeredis[json,lua]>=2.32.1,<3.0.0",
+    "coverage[toml]>=7.0.0,<8.0.0",
+    "hatchet-sdk>=1.23.0",
+    "black>=26.1.0",
+]
+
+[build-system]
+requires = ["hatchling"]
+build-backend = "hatchling.build"
+
+[tool.hatch.build.targets.wheel]
+packages = ["thirdmagic"]
+
+[tool.coverage.run]
+source = ["thirdmagic"]
+branch = true
+omit = [
+    "*/tests/*",
+    "*/test_*"
+]
+
+[tool.coverage.report]
+skip_covered = false
+show_missing = true
+exclude_lines = [
+    "pragma: no cover",
+    "def __repr__",
+    "raise AssertionError",
+    "raise NotImplementedError",
+    "if __name__ == .__main__.:",
+    "@abstractmethod",
+    "@overload"
+]
+
+[tool.coverage.html]
+directory = "htmlcov"

thirdmagic-0.0.1/tests/unit/assertions.py

@@ -0,0 +1,80 @@
+from typing import TypeVar, Literal, cast
+
+import rapyer
+from rapyer.fields import RapyerKey
+from redis.asyncio import Redis
+
+from thirdmagic.consts import REMOVED_TASK_TTL
+from thirdmagic.signature import Signature
+from thirdmagic.task import TaskSignature
+
+T = TypeVar("T", bound=Signature)
+SwarmListName = Literal["finished_tasks", "failed_tasks", "tasks_results", "tasks"]
+
+
+async def assert_task_reloaded_as_type(
+    task_key: RapyerKey,
+    expected_type: type[T],
+) -> T:
+    reloaded = await rapyer.aget(task_key)
+    assert reloaded is not None, f"Task {task_key} not found"
+    assert isinstance(
+        reloaded, expected_type
+    ), f"Expected {expected_type.__name__}, got {type(reloaded).__name__}"
+    return reloaded
+
+
+def assert_callback_contains(
+    task: Signature,
+    success_keys: list[RapyerKey] | None = None,
+    error_keys: list[RapyerKey] | None = None,
+) -> None:
+    for success_key in success_keys or []:
+        assert (
+            success_key in task.success_callbacks
+        ), f"{success_key} not in success_callbacks"
+    for error_key in error_keys or []:
+        assert error_key in task.error_callbacks, f"{error_key} not in error_callbacks"
+
+
+async def assert_tasks_not_exists(tasks_ids: list[str]):
+    for task_id in tasks_ids:
+        reloaded_signature = await TaskSignature.afind_one(task_id)
+        assert reloaded_signature is None
+
+
+async def assert_tasks_changed_status(
+    tasks_ids: list[str | TaskSignature], status: str, old_status: str = None
+):
+    tasks_ids = tasks_ids if isinstance(tasks_ids, list) else [tasks_ids]
+    all_tasks = []
+    for task_key in tasks_ids:
+        task_key = task_key.key if isinstance(task_key, Signature) else task_key
+        reloaded_signature = await rapyer.aget(task_key)
+        reloaded_signature = cast(TaskSignature, reloaded_signature)
+        all_tasks.append(reloaded_signature)
+        assert reloaded_signature.task_status.status == status
+        if old_status:
+            assert reloaded_signature.task_status.last_status == old_status
+    return all_tasks
+
+
+async def assert_redis_keys_do_not_contain_sub_task_ids(redis_client, sub_task_ids):
+    all_keys = await redis_client.keys("*")
+    all_keys_str = [
+        key.decode() if isinstance(key, bytes) else str(key) for key in all_keys
+    ]
+
+    for sub_task_id in sub_task_ids:
+        sub_task_id_str = str(sub_task_id)
+        keys_containing_sub_task = [
+            key for key in all_keys_str if sub_task_id_str in key
+        ]
+        assert (
+            not keys_containing_sub_task
+        ), f"Found Redis keys containing deleted sub-task ID {sub_task_id}: {keys_containing_sub_task}"
+
+
+async def assert_task_has_short_ttl(redis_client: Redis, task_key: str):
+    ttl = await redis_client.ttl(task_key)
+    assert 0 < ttl <= REMOVED_TASK_TTL, f"Expected TTL <= {REMOVED_TASK_TTL}, got {ttl}"

thirdmagic-0.0.1/tests/unit/change_status/assertions.py

@@ -0,0 +1,14 @@
+from typing import cast
+from unittest.mock import Mock, call
+
+from thirdmagic.task import TaskSignature
+
+
+def assert_resume_signature(signature: TaskSignature, mock_adapter: Mock):
+    tasks_called = [a[0][0] for a in mock_adapter.acall_signature.call_args_list]
+    tasks_called = cast(list[TaskSignature], tasks_called)
+    task_called_with_id = [task for task in tasks_called if task.key == signature.key]
+    assert len(task_called_with_id) == 1, f"Task was resumed more than once"
+    mock_adapter.acall_signature.assert_has_awaits(
+        [call(task_called_with_id[0], None, set_return_field=False)]
+    )

thirdmagic-0.0.1/tests/unit/change_status/conftest.py

@@ -0,0 +1,90 @@
+from dataclasses import dataclass
+
+import pytest
+import pytest_asyncio
+
+import thirdmagic
+from tests.unit.messages import ContextMessage
+from tests.unit.utils import extract_hatchet_validator
+from thirdmagic.chain import ChainTaskSignature
+from thirdmagic.swarm import SwarmTaskSignature
+from thirdmagic.task import TaskSignature, SignatureStatus
+
+
+@dataclass
+class SwarmTestData:
+    task_signatures: list
+    swarm_signature: SwarmTaskSignature
+
+
+@dataclass
+class ChainTestData:
+    task_signatures: list
+    chain_signature: ChainTaskSignature
+
+
+@dataclass
+class TaskResumeConfig:
+    name: str
+    last_status: SignatureStatus
+
+
+async def delete_tasks_by_indices(
+    task_signatures: list[TaskSignature],
+    indices: list[int],
+) -> list[str]:
+    deleted_task_ids = []
+    for idx in indices:
+        await task_signatures[idx].adelete()
+        deleted_task_ids.append(task_signatures[idx].key)
+    return deleted_task_ids
+
+
+def get_non_deleted_task_keys(
+    task_signatures: list[TaskSignature],
+    deleted_indices: list[int],
+) -> list[str]:
+    return [
+        task_signatures[i].key
+        for i in range(len(task_signatures))
+        if i not in deleted_indices
+    ]
+
+
+@pytest_asyncio.fixture
+async def chain_with_tasks():
+    task_signatures = [
+        await thirdmagic.sign(f"chain_task_{i}", model_validators=ContextMessage)
+        for i in range(1, 4)
+    ]
+
+    chain_signature = await thirdmagic.chain([task.key for task in task_signatures])
+
+    return ChainTestData(
+        task_signatures=task_signatures, chain_signature=chain_signature
+    )
+
+
+@pytest_asyncio.fixture
+async def swarm_with_tasks():
+    task_signatures = [
+        await thirdmagic.sign(f"swarm_task_{i}", model_validators=ContextMessage)
+        for i in range(1, 4)
+    ]
+
+    swarm_signature = await thirdmagic.swarm(
+        task_name="test_swarm",
+        model_validators=ContextMessage,
+        tasks=task_signatures,
+    )
+
+    return SwarmTestData(
+        task_signatures=task_signatures, swarm_signature=swarm_signature
+    )
+
+
+@pytest.fixture()
+def hatchet_client_adapter(mock_adapter):
+    mock_adapter.extract_validator.side_effect = extract_hatchet_validator
+    mock_adapter.task_name.side_effect = lambda fn: fn.name
+    yield mock_adapter

thirdmagic-0.0.1/tests/unit/change_status/test_chain.py

@@ -0,0 +1,194 @@
+import pytest
+
+import thirdmagic
+from tests.unit.assertions import (
+    assert_tasks_changed_status,
+    assert_tasks_not_exists,
+    assert_redis_keys_do_not_contain_sub_task_ids,
+)
+from tests.unit.change_status.assertions import assert_resume_signature
+from tests.unit.change_status.conftest import (
+    TaskResumeConfig,
+    delete_tasks_by_indices,
+    get_non_deleted_task_keys,
+)
+from tests.unit.messages import ContextMessage
+from thirdmagic.chain import ChainTaskSignature
+from thirdmagic.task import TaskSignature, SignatureStatus
+
+
+@pytest.mark.asyncio
+async def test_chain_safe_change_status_on_deleted_signature_does_not_create_redis_entry_sanity():
+    # Arrange
+    task_signatures = [
+        await thirdmagic.sign(f"chain_task_{i}", model_validators=ContextMessage)
+        for i in range(1, 4)
+    ]
+    chain_signature = await thirdmagic.chain(
+        tasks=task_signatures, name="test_chain_unsaved"
+    )
+    chain_key = chain_signature.key
+    await chain_signature.adelete()
+
+    # Act
+    result = await ChainTaskSignature.safe_change_status(
+        chain_key, SignatureStatus.SUSPENDED
+    )
+
+    # Assert
+    assert result is False
+    reloaded_signature = await TaskSignature.afind_one(chain_key)
+    assert reloaded_signature is None
+
+
+@pytest.mark.asyncio
+@pytest.mark.parametrize(
+    ["task_names", "tasks_to_delete_indices", "new_status"],
+    [
+        [
+            ["task1", "task2", "task3"],
+            [],
+            SignatureStatus.SUSPENDED,
+        ],
+        [
+            ["task1", "task2"],
+            [0, 1],
+            SignatureStatus.CANCELED,
+        ],
+        [
+            ["task1", "task2", "task3"],
+            [0, 2],
+            SignatureStatus.ACTIVE,
+        ],
+    ],
+)
+async def test_chain_change_status_with_optional_deleted_sub_tasks_edge_case(
+    redis_client,
+    task_names: list[str],
+    tasks_to_delete_indices: list[int],
+    new_status: SignatureStatus,
+    mock_task_def,
+):
+    # Arrange
+    task_signatures = [await thirdmagic.sign(name) for name in task_names]
+    chain_signature = await thirdmagic.chain([task.key for task in task_signatures])
+    deleted_task_ids = await delete_tasks_by_indices(
+        task_signatures, tasks_to_delete_indices
+    )
+
+    # Act
+    await chain_signature.safe_change_status(chain_signature.key, new_status)
+
+    # Assert
+    reloaded_chain = await TaskSignature.aget(chain_signature.key)
+    assert reloaded_chain.task_status.status == new_status
+    assert reloaded_chain.task_status.last_status == SignatureStatus.PENDING
+
+    await assert_tasks_not_exists(deleted_task_ids)
+
+    non_deleted_keys = get_non_deleted_task_keys(
+        task_signatures, tasks_to_delete_indices
+    )
+    await assert_tasks_changed_status(
+        non_deleted_keys, new_status, SignatureStatus.PENDING
+    )
+
+    await assert_redis_keys_do_not_contain_sub_task_ids(redis_client, deleted_task_ids)
+
+
+@pytest.mark.asyncio
+@pytest.mark.parametrize(
+    ["task_configs", "tasks_to_delete_indices"],
+    [
+        [
+            [
+                TaskResumeConfig(name="task1", last_status=SignatureStatus.ACTIVE),
+                TaskResumeConfig(name="task2", last_status=SignatureStatus.ACTIVE),
+                TaskResumeConfig(name="task3", last_status=SignatureStatus.ACTIVE),
+            ],
+            [],
+        ],
+        [
+            [
+                TaskResumeConfig(name="task1", last_status=SignatureStatus.PENDING),
+                TaskResumeConfig(name="task2", last_status=SignatureStatus.PENDING),
+            ],
+            [0],
+        ],
+        [
+            [
+                TaskResumeConfig(name="task1", last_status=SignatureStatus.ACTIVE),
+                TaskResumeConfig(name="task2", last_status=SignatureStatus.ACTIVE),
+                TaskResumeConfig(name="task3", last_status=SignatureStatus.ACTIVE),
+            ],
+            [1, 2],
+        ],
+    ],
+)
+async def test_chain_resume_with_optional_deleted_sub_tasks_sanity(
+    mock_adapter,
+    task_configs: list[TaskResumeConfig],
+    tasks_to_delete_indices: list[int],
+    mock_task_def,
+):
+    # Arrange
+    task_signatures = []
+    expected_statuses = []
+    num_of_aio_run = 0
+    for config in task_configs:
+        task_signature = await thirdmagic.sign(config.name)
+        task_signature.task_status.status = SignatureStatus.SUSPENDED
+        task_signature.task_status.last_status = config.last_status
+        await task_signature.asave()
+        task_signatures.append(task_signature)
+        expected_statuses.append(config.last_status)
+
+    chain_signature = await thirdmagic.chain([task.key for task in task_signatures])
+    chain_signature.task_status.status = SignatureStatus.SUSPENDED
+
+    deleted_task_ids = await delete_tasks_by_indices(
+        task_signatures, tasks_to_delete_indices
+    )
+
+    # Act
+    await chain_signature.resume()
+
+    # Assert
+    non_deleted_task_indices = [
+        i for i in range(len(task_signatures)) if i not in tasks_to_delete_indices
+    ]
+    for i in non_deleted_task_indices:
+        task = task_signatures[i]
+        new_status = expected_statuses[i]
+        if new_status == SignatureStatus.ACTIVE:
+            new_status = SignatureStatus.PENDING
+            assert_resume_signature(task, mock_adapter)
+            num_of_aio_run += 1
+        await assert_tasks_changed_status(
+            [task.key], new_status, SignatureStatus.SUSPENDED
+        )
+
+    await assert_tasks_changed_status(
+        [chain_signature.key], SignatureStatus.PENDING, SignatureStatus.SUSPENDED
+    )
+
+    await assert_tasks_not_exists(deleted_task_ids)
+    assert mock_adapter.acall_signature.call_count == num_of_aio_run
+
+
+@pytest.mark.asyncio
+async def test_chain_suspend_sanity(chain_with_tasks):
+    # Arrange
+    chain_data = chain_with_tasks
+
+    # Act
+    await chain_data.chain_signature.suspend()
+
+    # Assert
+    # Verify all tasks changed status to suspend
+    await assert_tasks_changed_status(
+        [chain_data.chain_signature.key], SignatureStatus.SUSPENDED
+    )
+    await assert_tasks_changed_status(
+        [task.key for task in chain_data.task_signatures], SignatureStatus.SUSPENDED
+    )