daknode-mcp 0.3.1__tar.gz → 0.3.2__tar.gz

{daknode_mcp-0.3.1 → daknode_mcp-0.3.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: daknode-mcp
-Version: 0.3.1
+Version: 0.3.2
 Summary: DakNode's email API as MCP tools for AI agents, over stdio.
 Project-URL: Homepage, https://daknode.com
 Project-URL: Documentation, https://daknode.com/developers
@@ -50,6 +50,17 @@ launch rather than looking like a broken tool.
 `create_inbox` · `list_threads` · `list_messages` · `get_thread` · `get_message`
 · `search` · `send`
 
+`create_inbox(alias)` requires the chosen address alias. `send` requires a
+caller-supplied `idempotency_key`: keep it stable for retries of the same approved
+message. Blank, control-character/non-ASCII or over-255-character keys are
+refused before HTTP. A distinct intentional send needs a distinct key. Tool
+discovery alone does not verify these write contracts; isolated operation tests
+exercise the generated SDK request and response with a mocked HTTP transport.
+
+Codex plugin, Cursor install link and canonical multi-platform skill:
+https://github.com/Subodh103/daknode-integrations. These launch the same local
+server; no hosted ChatGPT endpoint or public marketplace acceptance is implied.
+
 **Every tool is 1:1 with a published API operation.** That is `architecture.md`
 §0 consequence 4, and it is the boundary this server exists inside: DakNode
 exposes operations, the agent decides which to call. A tool like

{daknode_mcp-0.3.1 → daknode_mcp-0.3.2}/README.md

@@ -39,6 +39,17 @@ launch rather than looking like a broken tool.
 `create_inbox` · `list_threads` · `list_messages` · `get_thread` · `get_message`
 · `search` · `send`
 
+`create_inbox(alias)` requires the chosen address alias. `send` requires a
+caller-supplied `idempotency_key`: keep it stable for retries of the same approved
+message. Blank, control-character/non-ASCII or over-255-character keys are
+refused before HTTP. A distinct intentional send needs a distinct key. Tool
+discovery alone does not verify these write contracts; isolated operation tests
+exercise the generated SDK request and response with a mocked HTTP transport.
+
+Codex plugin, Cursor install link and canonical multi-platform skill:
+https://github.com/Subodh103/daknode-integrations. These launch the same local
+server; no hosted ChatGPT endpoint or public marketplace acceptance is implied.
+
 **Every tool is 1:1 with a published API operation.** That is `architecture.md`
 §0 consequence 4, and it is the boundary this server exists inside: DakNode
 exposes operations, the agent decides which to call. A tool like

{daknode_mcp-0.3.1 → daknode_mcp-0.3.2}/daknode_mcp.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: daknode-mcp
-Version: 0.3.1
+Version: 0.3.2
 Summary: DakNode's email API as MCP tools for AI agents, over stdio.
 Project-URL: Homepage, https://daknode.com
 Project-URL: Documentation, https://daknode.com/developers
@@ -50,6 +50,17 @@ launch rather than looking like a broken tool.
 `create_inbox` · `list_threads` · `list_messages` · `get_thread` · `get_message`
 · `search` · `send`
 
+`create_inbox(alias)` requires the chosen address alias. `send` requires a
+caller-supplied `idempotency_key`: keep it stable for retries of the same approved
+message. Blank, control-character/non-ASCII or over-255-character keys are
+refused before HTTP. A distinct intentional send needs a distinct key. Tool
+discovery alone does not verify these write contracts; isolated operation tests
+exercise the generated SDK request and response with a mocked HTTP transport.
+
+Codex plugin, Cursor install link and canonical multi-platform skill:
+https://github.com/Subodh103/daknode-integrations. These launch the same local
+server; no hosted ChatGPT endpoint or public marketplace acceptance is implied.
+
 **Every tool is 1:1 with a published API operation.** That is `architecture.md`
 §0 consequence 4, and it is the boundary this server exists inside: DakNode
 exposes operations, the agent decides which to call. A tool like

{daknode_mcp-0.3.1 → daknode_mcp-0.3.2}/daknode_mcp.egg-info/SOURCES.txt

@@ -9,4 +9,5 @@ daknode_mcp.egg-info/SOURCES.txt
 daknode_mcp.egg-info/dependency_links.txt
 daknode_mcp.egg-info/entry_points.txt
 daknode_mcp.egg-info/requires.txt
-daknode_mcp.egg-info/top_level.txt
+daknode_mcp.egg-info/top_level.txt
+tests/test_write_tools.py

{daknode_mcp-0.3.1 → daknode_mcp-0.3.2}/pyproject.toml

@@ -7,7 +7,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "daknode-mcp"
-version = "0.3.1"
+version = "0.3.2"
 description = "DakNode's email API as MCP tools for AI agents, over stdio."
 readme = "README.md"
 requires-python = ">=3.10"

{daknode_mcp-0.3.1 → daknode_mcp-0.3.2}/server.py

@@ -114,17 +114,20 @@ def build_server() -> MCPServer:
     )
 
     @server.tool()
-    def create_inbox() -> Any:
+    def create_inbox(alias: str) -> Any:
         """Create a new agent inbox and return its id and address.
 
+        Supply the user's chosen alias (the local part of the address).
         The same address both sends and receives. Inbound mail arrives by
         webhook; use `list_threads` and `get_thread` to read history.
         """
         from daknode.api.inboxes import create_inbox as op
         from daknode.models import InboxCreateRequest
 
+        if not alias.strip():
+            return {"error": "A nonempty inbox alias is required."}
         with _client() as c:
-            return _result(op.sync_detailed(client=c, body=InboxCreateRequest()))
+            return _result(op.sync_detailed(client=c, body=InboxCreateRequest(alias=alias)))
 
     @server.tool()
     def list_threads(inbox_id: str, limit: int = 25, cursor: str | None = None) -> Any:
@@ -193,10 +196,13 @@ def build_server() -> MCPServer:
         to: str,
         subject: str,
         body_text: str,
+        idempotency_key: str,
         thread_id: str | None = None,
     ) -> Any:
         """Send a message.
 
+        Supply a stable `idempotency_key` for this approved send; reuse it on
+        retries, and use a different key only for a new intentional send.
         Pass `thread_id` to reply into a conversation: the server derives
         `In-Reply-To` and `References` from that thread's latest received
         message. Without it the message opens a NEW conversation in the
@@ -206,8 +212,11 @@ def build_server() -> MCPServer:
         from daknode.models import SendRequest
         from daknode.types import UNSET
 
+        if (not idempotency_key.strip() or len(idempotency_key) > 255
+                or any(ord(char) < 32 or ord(char) > 126 for char in idempotency_key)):
+            return {"error": "A nonempty, header-safe idempotency key of at most 255 characters is required."}
         with _client() as c:
-            return _result(op.sync_detailed(client=c, body=SendRequest(
+            return _result(op.sync_detailed(client=c, idempotency_key=idempotency_key, body=SendRequest(
                 inbox_id=inbox_id, to=to, subject=subject, body_text=body_text,
                 thread_id=thread_id if thread_id else UNSET,
             )))

daknode_mcp-0.3.2/tests/test_write_tools.py

@@ -0,0 +1,95 @@
+"""Operation-level MCP tests: real generated SDK, mocked HTTP, no live mail."""
+import asyncio
+import json
+
+import httpx
+import pytest
+from daknode import AuthenticatedClient
+
+from mcp_server import server
+
+
+@pytest.fixture
+def calls(monkeypatch):
+    requests = []
+
+    def handle(request):
+        requests.append(request)
+        if request.url.path == "/v1/inboxes":
+            return httpx.Response(201, json={"inbox_id": "11111111-1111-4111-8111-111111111111",
+                "address": "support@agents.daknode.com", "channel": "email",
+                "display_name": None, "webhook_url": None})
+        return httpx.Response(200, json={"send_id": "22222222-2222-4222-8222-222222222222", "status": "queued"})
+
+    monkeypatch.setattr(server, "_client", lambda: AuthenticatedClient(
+        base_url="https://api.daknode.com", token="synthetic-not-a-key",
+        httpx_args={"transport": httpx.MockTransport(handle)}))
+    return requests
+
+
+def invoke(name, args):
+    return asyncio.run(server.build_server().call_tool(name, args))
+
+
+def test_create_requires_and_transmits_alias(calls):
+    result = invoke("create_inbox", {"alias": "support"})
+    assert len(calls) == 1
+    assert json.loads(calls[0].content) == {"alias": "support"}
+    assert calls[0].method == "POST"
+    assert "support@agents.daknode.com" in str(result)
+
+
+def test_send_preserves_idempotency_and_thread(calls):
+    args = {"inbox_id": "11111111-1111-4111-8111-111111111111", "to": "test@example.com",
+        "subject": "Test", "body_text": "Hello", "idempotency_key": "stable-operation-1",
+        "thread_id": "33333333-3333-4333-8333-333333333333"}
+    for _ in range(2):
+        result = invoke("send", args)
+        assert "queued" in str(result) and not result.is_error
+    assert len(calls) == 2
+    for request in calls:
+        assert request.headers["Idempotency-Key"] == "stable-operation-1"
+        assert json.loads(request.content)["thread_id"] == args["thread_id"]
+        assert request.url.path == "/v1/send"
+
+
+@pytest.mark.parametrize("name,args", [
+    ("create_inbox", {}), ("create_inbox", {"alias": ""}),
+    ("send", {"inbox_id": "x", "to": "a@example.com", "subject": "s", "body_text": "b"}),
+    *[("send", {"inbox_id": "x", "to": "a@example.com", "subject": "s", "body_text": "b", "idempotency_key": value})
+      for value in ("", " ", "x" * 256, "unsafe\r\nheader")],
+])
+def test_invalid_write_never_reaches_http(calls, name, args):
+    try:
+        result = invoke(name, args)
+    except Exception:
+        pass  # MCP versions may report schema validation as an exception.
+    else:
+        assert "error" in str(result).lower()
+    assert calls == []
+
+
+def test_write_arguments_are_required_in_protocol_schema():
+    tools = {tool.name: tool for tool in asyncio.run(server.build_server().list_tools())}
+    assert "alias" in tools["create_inbox"].input_schema["required"]
+    assert "idempotency_key" in tools["send"].input_schema["required"]
+
+
+@pytest.mark.parametrize("status", [401, 403, 429])
+def test_api_refusals_are_not_reported_as_accepted(monkeypatch, status):
+    def refuse(request):
+        return httpx.Response(status, json={"error": {"code": "refused"}})
+    monkeypatch.setattr(server, "_client", lambda: AuthenticatedClient(
+        base_url="https://api.daknode.com", token="synthetic-not-a-key",
+        httpx_args={"transport": httpx.MockTransport(refuse)}))
+    result = invoke("send", {"inbox_id": "11111111-1111-4111-8111-111111111111",
+        "to": "test@example.com", "subject": "Test", "body_text": "Hello",
+        "idempotency_key": "stable-refused-operation"})
+    assert str(status) in str(result) and "queued" not in str(result)
+
+
+def test_new_conversation_omits_thread(calls):
+    invoke("send", {"inbox_id": "11111111-1111-4111-8111-111111111111",
+        "to": "test@example.com", "subject": "Test", "body_text": "Hello",
+        "idempotency_key": "new-conversation"})
+    assert "thread_id" not in json.loads(calls[0].content)