macp-sdk-python 0.8.0__tar.gz → 0.9.0__tar.gz

{macp_sdk_python-0.8.0/src/macp_sdk_python.egg-info → macp_sdk_python-0.9.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: macp-sdk-python
-Version: 0.8.0
+Version: 0.9.0
 Summary: Python SDK for the MACP Rust runtime
 Author-email: Multi-Agent Coordination Protocol <macp@multiagentcoordinationprotocol.org>
 License: Apache-2.0

{macp_sdk_python-0.8.0 → macp_sdk_python-0.9.0}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "macp-sdk-python"
-version = "0.8.0"
+version = "0.9.0"
 description = "Python SDK for the MACP Rust runtime"
 readme = "README.md"
 requires-python = ">=3.11"

{macp_sdk_python-0.8.0 → macp_sdk_python-0.9.0}/src/macp_sdk/__init__.py

@@ -95,6 +95,7 @@ from .validation import (
     validate_confidence,
     validate_participant_count,
     validate_participants,
+    validate_progress_scope,
     validate_recommendation,
     validate_required_field,
     validate_session_id,
@@ -218,6 +219,7 @@ __all__ = [
     "validate_confidence",
     "validate_participant_count",
     "validate_participants",
+    "validate_progress_scope",
     "validate_recommendation",
     "validate_required_field",
     "validate_session_id",

{macp_sdk_python-0.8.0 → macp_sdk_python-0.9.0}/src/macp_sdk/client.py

@@ -24,6 +24,7 @@ from .errors import (
     MacpSessionError,
     MacpTransportError,
 )
+from .validation import validate_progress_scope
 
 # Public typing alias for inline stream-error callbacks. Parity with
 # typescript-sdk's ``InlineErrorCallback``. Receives the protobuf
@@ -962,10 +963,18 @@ class MacpClient:
     ) -> envelope_pb2.Ack:
         """Send a progress update.
 
-        When ``session_id`` and ``mode`` are empty, the progress is treated
-        as an *ambient* progress message routed through the signal broadcast
-        path.
+        Per RFC-MACP-0001 §6, ``Progress`` is legal in exactly two shapes:
+
+        * *ambient* -- ``session_id`` and ``mode`` both empty, routed through
+          the signal broadcast path; or
+        * *session-scoped* -- both non-empty.
+
+        Supplying exactly one of the two raises ``MacpSessionError`` naming
+        the mismatched field. The runtime rejects that mixed envelope with
+        ``INVALID_ENVELOPE``, so this only surfaces the failure earlier and
+        more clearly.
         """
+        validate_progress_scope(session_id, mode)
         auth_cfg = self._require_auth(auth)
         payload = build_progress_payload(
             progress_token=progress_token,

{macp_sdk_python-0.8.0 → macp_sdk_python-0.9.0}/src/macp_sdk/policy.py

@@ -155,7 +155,8 @@ class QuorumThreshold:
     ``quorum-rules.schema.json``: an approval count for ``n_of_m`` /
     ``weighted``, and an integer percentage 0-100 for ``percentage``. A
     fractional value (e.g. ``0.75``) is rejected by the runtime's schema
-    validation, so this is typed ``int`` and range-checked at build time.
+    validation, so this is typed ``int`` and ``build_quorum_policy`` rejects
+    a non-integer or out-of-range value at build time.
     """
 
     type: str = "n_of_m"
@@ -185,7 +186,21 @@ def build_quorum_policy(
 
     # Match the canonical schema's constraints before the runtime does, so a
     # bad descriptor fails immediately client-side instead of round-tripping
-    # to an INVALID_POLICY_DEFINITION from RegisterPolicy.
+    # to an INVALID_POLICY_DEFINITION from RegisterPolicy. Order matters:
+    # type first (a float or bool reaching the range checks below would
+    # compare fine numerically but produce a confusing message), then >= 0,
+    # then the percentage-specific <= 100 cap.
+    #
+    # bool is checked explicitly because isinstance(True, int) is True in
+    # Python -- QuorumThreshold(value=True) must not silently mean 1.
+    if isinstance(t.value, bool) or not isinstance(t.value, int):
+        raise MacpSessionError(
+            "quorum threshold value must be an integer (e.g. 75 for 75%), got "
+            f"{t.value!r}. The canonical quorum-rules schema declares "
+            "'value' as an integer for every threshold type; a fractional "
+            "value like 0.75 would produce a schema-invalid descriptor that "
+            "the runtime rejects at RegisterPolicy with worse diagnostics."
+        )
     if t.value < 0:
         raise MacpSessionError(f"quorum threshold value must be >= 0, got {t.value}")
     if t.type == "percentage" and t.value > 100:

{macp_sdk_python-0.8.0 → macp_sdk_python-0.9.0}/src/macp_sdk/validation.py

@@ -122,6 +122,42 @@ def validate_signal_type(signal_type: str, data: bytes | None = None) -> None:
         raise MacpSessionError("signal_type must be non-empty when data is present")
 
 
+def validate_progress_scope(session_id: str, mode: str) -> None:
+    """Validate the ``Progress`` scope pairing (RFC-MACP-0001 §6).
+
+    ``Progress`` is legal in exactly two shapes -- *ambient* (``session_id``
+    and ``mode`` both empty) or *session-scoped* (both non-empty). An envelope
+    with exactly one of the two empty is a mixed shape that the runtime
+    rejects with ``INVALID_ENVELOPE``; raising here names the mismatched field
+    instead.
+
+    Unlike Signals, ``Progress`` is *not* required to be ambient -- this is a
+    tri-state rule, so neither field may be inferred from the other.
+
+    The emptiness test deliberately mirrors the runtime's
+    ``validate_envelope_shape`` **exactly**, including its asymmetry:
+    ``session_id`` is compared raw while ``mode`` is stripped first. Mirroring
+    rather than normalising is the only choice under which the SDK neither
+    accepts a shape the runtime rejects nor rejects one it accepts.
+    Concretely, a whitespace-only ``mode`` alongside an empty ``session_id``
+    is ambient to the runtime, so it stays ambient here.
+    """
+    session_id_empty = session_id == ""
+    mode_empty = mode.strip() == ""
+    if session_id_empty == mode_empty:
+        return
+    detail = (
+        f"mode is {mode!r} but session_id is empty"
+        if session_id_empty
+        else f"session_id is {session_id!r} but mode is empty"
+    )
+    raise MacpSessionError(
+        "Progress must be either ambient (session_id and mode both empty) or "
+        f"session-scoped (both non-empty), but {detail} (RFC-MACP-0001 §6). "
+        "Pass both fields for a session-scoped Progress, or neither for an ambient one."
+    )
+
+
 def validate_ttl_ms(ttl_ms: int) -> None:
     """Validate that *ttl_ms* is in [1, 86_400_000]."""
     if ttl_ms < 1 or ttl_ms > _MAX_TTL_MS:

{macp_sdk_python-0.8.0 → macp_sdk_python-0.9.0/src/macp_sdk_python.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: macp-sdk-python
-Version: 0.8.0
+Version: 0.9.0
 Summary: Python SDK for the MACP Rust runtime
 Author-email: Multi-Agent Coordination Protocol <macp@multiagentcoordinationprotocol.org>
 License: Apache-2.0