fix: remove async method

[ryoppippi]

Summary by cubic

Removed the acall method that wrapped the sync call in a thread, simplifying the API and avoiding misleading async behavior. Deleted the related async test.

• Migration
  • Replace tool.acall(...) with await asyncio.to_thread(tool.call, ...) in async code.
  • Or use loop.run_in_executor(None, lambda: tool.call(...)) on older patterns.
  • If not using async, call tool.call(...) directly.

[cubic-dev-ai]

No issues found across 2 files

[glebedel]

LGTM

[Copilot]

Pull Request Overview

This PR removes the misleading async acall method from the Tool class and its associated test. The acall method was using asyncio.run_in_executor to wrap the synchronous call method in a thread, which provided false async behavior rather than true async I/O.

• Removes the acall method and its docstring from the Tool class
• Deletes the async test case that tested the acall functionality
• Removes unused imports (asyncio and functools.partial) that were only needed for the removed method

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
stackone_ai/models.py	Removes the acall method and cleans up unused imports
tests/test_tool_calling.py	Deletes the async test case for the removed acall method

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}