acp-sdk 0.3.0__tar.gz → 0.3.1__tar.gz

{acp_sdk-0.3.0 → acp_sdk-0.3.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: acp-sdk
-Version: 0.3.0
+Version: 0.3.1
 Summary: Agent Communication Protocol SDK
 Author: IBM Corp.
 Maintainer-email: Tomas Pilar <thomas7pilar@gmail.com>
@@ -28,19 +28,7 @@ Agent Communication Protocol SDK for Python provides allows developers to serve
 
 ## Installation
 
-Install to use client:
-
-```shell
-pip install acp-sdk[client]
-```
-
-Install to use server:
-
-```shell
-pip install acp-sdk[server]
-```
-
-Install to use models only:
+Install with:
 
 ```shell
 pip install acp-sdk

{acp_sdk-0.3.0 → acp_sdk-0.3.1}/README.md

@@ -8,19 +8,7 @@ Agent Communication Protocol SDK for Python provides allows developers to serve
 
 ## Installation
 
-Install to use client:
-
-```shell
-pip install acp-sdk[client]
-```
-
-Install to use server:
-
-```shell
-pip install acp-sdk[server]
-```
-
-Install to use models only:
+Install with:
 
 ```shell
 pip install acp-sdk

acp_sdk-0.3.1/examples/servers/awaiting.py

@@ -0,0 +1,28 @@
+from collections.abc import AsyncGenerator
+
+from acp_sdk.models import (
+    Message,
+    MessageAwaitRequest,
+    MessageAwaitResume,
+    MessagePart,
+)
+from acp_sdk.server import Context, Server
+from acp_sdk.server.types import RunYield, RunYieldResume
+
+server = Server()
+
+
+@server.agent()
+async def awaiting(inputs: list[Message], context: Context) -> AsyncGenerator[RunYield, RunYieldResume]:
+    """Greets and awaits for more data"""
+    yield MessagePart(content="Hello!", content_type="text/plain")
+    resume = yield MessageAwaitRequest(
+        message=Message(
+            parts=[MessagePart(content="Can you provide me with additional configuration?", content_type="text/plain")]
+        )
+    )
+    assert isinstance(resume, MessageAwaitResume)
+    yield MessagePart(content=f"Thanks for config: {resume.message}", content_type="text/plain")
+
+
+server.run()

{acp_sdk-0.3.0 → acp_sdk-0.3.1}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "acp-sdk"
-version = "0.3.0"
+version = "0.3.1"
 description = "Agent Communication Protocol SDK"
 license = "Apache-2.0"
 readme = "README.md"

{acp_sdk-0.3.0 → acp_sdk-0.3.1}/src/acp_sdk/models/models.py

@@ -75,12 +75,18 @@ class RunStatus(str, Enum):
         return self in terminal_states
 
 
-class AwaitRequest(BaseModel):
-    type: Literal["placeholder"] = "placeholder"
+class MessageAwaitRequest(BaseModel):
+    type: Literal["message"] = "message"
+    message: Message
+
+
+class MessageAwaitResume(BaseModel):
+    type: Literal["message"] = "message"
+    message: Message
 
 
-class AwaitResume(BaseModel):
-    pass
+AwaitRequest = Union[MessageAwaitRequest]
+AwaitResume = Union[MessageAwaitResume]
 
 
 class Run(BaseModel):

{acp_sdk-0.3.0 → acp_sdk-0.3.1}/src/acp_sdk/server/app.py

@@ -144,6 +144,16 @@ def create_app(*agents: Agent) -> FastAPI:
     @app.post("/runs/{run_id}")
     async def resume_run(run_id: RunId, request: RunResumeRequest) -> RunResumeResponse:
         bundle = find_run_bundle(run_id)
+
+        if bundle.run.await_request is None:
+            raise HTTPException(status_code=status.HTTP_403_FORBIDDEN, detail=f"Run {run_id} has no await request")
+
+        if bundle.run.await_request.type != request.await_resume.type:
+            raise HTTPException(
+                status_code=status.HTTP_403_FORBIDDEN,
+                detail=f"Run {run_id} is expecting resume of type {bundle.run.await_request.type}",
+            )
+
         await bundle.resume(request.await_resume)
         match request.mode:
             case RunMode.STREAM:

{acp_sdk-0.3.0 → acp_sdk-0.3.1}/src/acp_sdk/server/bundle.py

@@ -129,6 +129,8 @@ class RunBundle:
                         run_logger.info("Run resumed")
                     elif isinstance(next, Error):
                         raise ACPError(error=next)
+                    elif next is None:
+                        pass  # Do nothing
                     else:
                         try:
                             generic = AnyModel.model_validate(next)

{acp_sdk-0.3.0 → acp_sdk-0.3.1}/src/acp_sdk/server/types.py

@@ -2,5 +2,5 @@ from typing import Any
 
 from acp_sdk.models import AwaitRequest, AwaitResume, Message
 
-RunYield = Message | AwaitRequest | dict[str | Any]
+RunYield = Message | AwaitRequest | dict[str | Any] | None
 RunYieldResume = AwaitResume | None

{acp_sdk-0.3.0 → acp_sdk-0.3.1}/tests/e2e/fixtures/server.py

@@ -4,7 +4,7 @@ from collections.abc import AsyncGenerator, AsyncIterator, Generator
 from threading import Thread
 
 import pytest
-from acp_sdk.models import Artifact, AwaitRequest, AwaitResume, Error, ErrorCode, Message, MessagePart
+from acp_sdk.models import Artifact, AwaitResume, Error, ErrorCode, Message, MessageAwaitRequest, MessagePart
 from acp_sdk.server import Context, Server
 
 from e2e.config import Config
@@ -20,8 +20,10 @@ def server() -> Generator[None]:
             yield message
 
     @server.agent()
-    async def awaiter(inputs: list[Message], context: Context) -> AsyncGenerator[Message | AwaitRequest, AwaitResume]:
-        yield AwaitRequest()
+    async def awaiter(
+        inputs: list[Message], context: Context
+    ) -> AsyncGenerator[Message | MessageAwaitRequest, AwaitResume]:
+        yield MessageAwaitRequest(message=Message(parts=[]))
         yield MessagePart(content="empty", content_type="text/plain")
 
     @server.agent()

{acp_sdk-0.3.0 → acp_sdk-0.3.1}/tests/e2e/test_suites/test_runs.py

@@ -4,9 +4,9 @@ import pytest
 from acp_sdk.client import Client
 from acp_sdk.models import (
     ArtifactEvent,
-    AwaitResume,
     ErrorCode,
     Message,
+    MessageAwaitResume,
     MessageCreatedEvent,
     MessagePart,
     RunCompletedEvent,
@@ -17,6 +17,7 @@ from acp_sdk.models import (
 from acp_sdk.server import Server
 
 inputs = [Message(parts=[MessagePart(content="Hello!", content_type="text/plain")])]
+await_resume = MessageAwaitResume(message=Message(parts=[]))
 
 
 @pytest.mark.asyncio
@@ -69,7 +70,7 @@ async def test_run_resume_sync(server: Server, client: Client) -> None:
     assert run.status == RunStatus.AWAITING
     assert run.await_request is not None
 
-    run = await client.run_resume_sync(run_id=run.run_id, await_resume=AwaitResume())
+    run = await client.run_resume_sync(run_id=run.run_id, await_resume=await_resume)
     assert run.status == RunStatus.COMPLETED
 
 
@@ -79,7 +80,7 @@ async def test_run_resume_async(server: Server, client: Client) -> None:
     assert run.status == RunStatus.AWAITING
     assert run.await_request is not None
 
-    run = await client.run_resume_async(run_id=run.run_id, await_resume=AwaitResume())
+    run = await client.run_resume_async(run_id=run.run_id, await_resume=await_resume)
     assert run.status == RunStatus.IN_PROGRESS
 
 
@@ -89,7 +90,7 @@ async def test_run_resume_stream(server: Server, client: Client) -> None:
     assert run.status == RunStatus.AWAITING
     assert run.await_request is not None
 
-    event_stream = [event async for event in client.run_resume_stream(run_id=run.run_id, await_resume=AwaitResume())]
+    event_stream = [event async for event in client.run_resume_stream(run_id=run.run_id, await_resume=await_resume)]
     assert isinstance(event_stream[0], RunInProgressEvent)
     assert isinstance(event_stream[-1], RunCompletedEvent)
 

acp_sdk-0.3.0/examples/servers/awaiting.py

@@ -1,25 +0,0 @@
-from collections.abc import AsyncGenerator
-from typing import Any
-
-from acp_sdk.models import (
-    AwaitRequest,
-    AwaitResume,
-    Message,
-    MessagePart,
-)
-from acp_sdk.server import Context, Server
-
-server = Server()
-
-
-@server.agent()
-async def awaiting(
-    inputs: list[Message], context: Context
-) -> AsyncGenerator[Message | AwaitRequest | Any, AwaitResume]:
-    """Greets and awaits for more data"""
-    yield MessagePart(content="Hello!", content_type="text/plain")
-    data = yield AwaitRequest()
-    yield MessagePart(content=f"Thanks for {data}", content_type="text/plain")
-
-
-server.run()