macp-sdk-python 0.12.0__tar.gz → 0.13.0__tar.gz

{macp_sdk_python-0.12.0 → macp_sdk_python-0.13.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: macp-sdk-python
-Version: 0.12.0
+Version: 0.13.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.12.0 → macp_sdk_python-0.13.0}/pyproject.toml

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

{macp_sdk_python-0.12.0 → macp_sdk_python-0.13.0}/src/macp_sdk/handoff.py

@@ -30,9 +30,11 @@ class HandoffRecord:
     declined_by: str | None
     # True when the acceptance was an implicit accept synthesized by the
     # runtime (RFC-MACP-0010 §5.1) rather than an explicit client HandoffAccept.
-    # Runtime >= 0.8.0 emits these automatically for every session (no
-    # server-side opt-in exists) when an offer's implicit_accept_timeout_ms
-    # elapses unactioned. Client-submitted accepts are always
+    # Runtime >= 0.8.0 emits these only for sessions whose bound governance
+    # policy declares a non-zero ``acceptance.implicit_accept_timeout_ms``
+    # (RFC-MACP-0010 §5.1, RFC-MACP-0012 §4.5), once that timeout elapses
+    # unactioned; with no policy bound the timeout resolves to 0 and no
+    # implicit accept ever fires. Client-submitted accepts are always
     # ``implicit=False`` (the runtime rejects a forged True).
     implicit: bool = False
 
@@ -177,9 +179,10 @@ class HandoffProjection(BaseProjection):
         """True if *handoff_id* was accepted by a runtime implicit accept.
 
         Distinguishes a timeout-driven implicit accept (RFC-MACP-0010 §5.1)
-        from an explicit client ``HandoffAccept``. Runtime >= 0.8.0 emits
-        these automatically for every session once an offer's
-        ``implicit_accept_timeout_ms`` elapses unactioned.
+        from an explicit client ``HandoffAccept``. Runtime >= 0.8.0 emits one
+        only when the session's bound governance policy declares a non-zero
+        ``acceptance.implicit_accept_timeout_ms`` and that timeout elapses
+        unactioned; sessions with no such policy never see one.
         """
         handoff = self.handoffs.get(handoff_id)
         return handoff is not None and handoff.status == "accepted" and handoff.implicit
@@ -275,7 +278,7 @@ class HandoffSession(BaseSession):
         # ``implicit=true`` for runtime-synthesized accepts and the runtime
         # rejects a client-submitted True. Client accepts always leave the
         # field at its proto3 default (False). See the regression test in
-        # tests/unit/test_handoff.py.
+        # tests/unit/test_absorb_runtime_v050.py.
         validate_required_field("handoff_id", handoff_id)
         payload = handoff_pb2.HandoffAcceptPayload(
             handoff_id=handoff_id,

{macp_sdk_python-0.12.0 → macp_sdk_python-0.13.0}/src/macp_sdk/projections.py

@@ -132,7 +132,10 @@ class DecisionProjection(BaseProjection):
         """Count votes per proposal, keyed by proposal_id.
 
         ABSTAIN votes are tracked but excluded from the totals returned
-        here (which counts only APPROVE votes).
+        here (which counts only APPROVE votes). A proposal_id that received
+        no votes at all is **absent** from the returned dict -- not present
+        with a 0 value. Use ``.get(proposal_id, 0)`` when checking an
+        arbitrary proposal_id.
         """
         totals: dict[str, int] = {}
         for proposal_id, sender_votes in self.votes.items():

{macp_sdk_python-0.12.0 → macp_sdk_python-0.13.0}/src/macp_sdk/proposal.py

@@ -25,11 +25,50 @@ class ProposalRecord:
     proposal_id: str
     title: str
     summary: str
-    proposer: str
+    sender: str
     supersedes: str  # "" if original
-    status: str  # "open" | "accepted" | "rejected" | "withdrawn"
+    # Reachable values only. An Accept is recorded on the projection's
+    # ``accepts`` list (and surfaced via ``accepted_proposal`` /
+    # ``is_accepted``), never on this field -- no code path assigns
+    # "accepted" here. A non-terminal Reject likewise leaves this "open";
+    # only ``terminal=True`` sets "rejected".
+    #
+    # That is by design, not an omission (issue #112). Acceptance is a
+    # per-sender, supersedable relation (RFC-MACP-0008 §5 rule 5), not a
+    # per-proposal fact, so a scalar field here cannot hold it: "alice
+    # accepts p2 while bob still accepts p1" is a legal state. This mirrors
+    # the runtime, whose ``ProposalDisposition`` is {Live, Withdrawn} with
+    # acceptance in a separate ``accepts`` map, and typescript-sdk's
+    # ``projections/proposal.ts``, which also never assigns "accepted".
+    # ``task.py``/``handoff.py`` do set "accepted" because their acceptance
+    # is one actor claiming one slot. See docs/modes/proposal.md.
+    status: str  # "open" | "rejected" | "withdrawn"
     tags: list[str]
 
+    # ── Deprecated alias (issue #120 / multiagentcoordinationprotocol#177) ──
+    # ``proposer`` is kept as a read-only, INSTANCE-level property alias for
+    # one minor version, removed at this SDK's next major version. This is a
+    # different mechanism from the module-level ``__getattr__``/PEP 562
+    # aliases below (``RejectRecord``/``AcceptRecord``, issue #103): those
+    # intercept MODULE attribute access (``macp_sdk.proposal.RejectRecord``);
+    # PEP 562 has no bearing on INSTANCE attribute access
+    # (``some_record.proposer``), which is what this field needs. A
+    # ``@dataclass(slots=True)`` class derives ``__slots__`` strictly from
+    # its annotated fields, so a plain same-named class attribute alias is
+    # not an option here -- but a property is not a field and is untouched
+    # by the slots transformation, so it coexists cleanly: ``fields()``,
+    # ``__slots__``, ``repr()``, and ``dataclasses.replace()`` all still
+    # see/use only the real ``sender`` field. No setter is defined: nothing
+    # in this module ever assigns ``.proposer``.
+    @property
+    def proposer(self) -> str:
+        warnings.warn(
+            "ProposalRecord.proposer is deprecated; use ProposalRecord.sender instead.",
+            DeprecationWarning,
+            stacklevel=2,
+        )
+        return self.sender
+
 
 @dataclass(slots=True)
 class ProposalRejectRecord:
@@ -79,7 +118,7 @@ class ProposalProjection(BaseProjection):
                 proposal_id=p.proposal_id,
                 title=p.title,
                 summary=p.summary,
-                proposer=envelope.sender,
+                sender=envelope.sender,
                 supersedes="",
                 status="open",
                 tags=list(p.tags),
@@ -93,7 +132,7 @@ class ProposalProjection(BaseProjection):
                 proposal_id=p.proposal_id,
                 title=p.title,
                 summary=p.summary,
-                proposer=envelope.sender,
+                sender=envelope.sender,
                 supersedes=p.supersedes_proposal_id,
                 status="open",
                 tags=[],
@@ -126,9 +165,20 @@ class ProposalProjection(BaseProjection):
             )
             if p.terminal:
                 rec = self.proposals.get(p.proposal_id)
+                # A terminal Reject must name a proposal this projection has
+                # actually seen (RFC-MACP-0008 §5 rule 3) before it ends the
+                # negotiation. "TerminalRejected" is in TERMINAL_PHASES
+                # (agent/participant.py:57-59), so moving phase here for an
+                # unknown proposal_id would fire a Participant's on_terminal
+                # and tear down its stream for a session that never actually
+                # terminated. Same shape as task.py's per-task gates
+                # (:172/:196/:212): gate the state transition, keep the
+                # audit append above unconditional. has_terminal_rejection()
+                # / is_terminally_rejected() deliberately still read
+                # self.rejections, not phase, so they stay True either way.
                 if rec is not None:
                     rec.status = "rejected"
-                self._set_phase("TerminalRejected")
+                    self._set_phase("TerminalRejected")
             return
 
         if mt == "Withdraw":

{macp_sdk_python-0.12.0 → macp_sdk_python-0.13.0}/src/macp_sdk/quorum.py

@@ -97,8 +97,12 @@ class QuorumProjection(BaseProjection):
         # enforces RFC-MACP-0011 §5 rule 3's *across-type* cardinality rule
         # ("at most one ballot across Approve, Reject, or Abstain"). First
         # ballot per sender stands; which-of-two-stands is inferred from
-        # RFC-MACP-0007 §5.3 parity plus what the only runtime enforces
-        # (quorum.rs:164/184/204) -- RFC-0011 itself is silent on that.
+        # RFC-MACP-0007 §5.3 parity plus what the only runtime enforces: the
+        # three same-sender ballot guards in macp-runtime's
+        # crates/macp-modes/src/mode/quorum.rs (QuorumMode's
+        # Approve/Reject/Abstain arms, each rejecting when
+        # state.ballots.contains_key(&env.sender), confirmed at runtime
+        # v0.8.6) -- RFC-0011 itself is silent on that.
         sender = envelope.sender
         sender_map = self.ballots.setdefault(request_id, {})
         existing = sender_map.get(sender)

{macp_sdk_python-0.12.0 → macp_sdk_python-0.13.0}/src/macp_sdk_python.egg-info/PKG-INFO

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