pmquant 0.4.0__tar.gz → 0.4.2__tar.gz

{pmquant-0.4.0 → pmquant-0.4.2}/CHANGELOG.md

@@ -1,5 +1,24 @@
 # Changelog
 
+## 0.4.2 (2026-07-03)
+
+* Fix: `buy_fak`/`sell_fak` cent-rounding used `int(x * 100) / 100`, which
+  floors a binary-drifted float (`16.90` stored as `16.8999…` became
+  `16.89`), silently shaving a cent off intended-clean amounts. Replaced with
+  a `Decimal`-based `_floor_cents` that rounds down without the drift, keeping
+  the never-exceed-budget contract. Matches the behavior documented in
+  docs/rounding-study.md.
+
+## 0.4.1 (2026-07-03)
+
+* Introspection guard now also verifies `OrderArgsV2` fields (including
+  `builder_code`, which the limit-order path depends on): a client that
+  dropped it is refused at startup instead of failing at call time.
+  pmq-doctor mirrors the same check.
+* Distribution: `server.json` manifest for the MCP registry, and an
+  `mcp-name` ownership token in the README (visible to the registry's PyPI
+  verification, invisible when rendered).
+
 ## 0.4.0 (2026-07-03)
 
 * Typing: the whole public API is annotated and ships a `py.typed` marker;

{pmquant-0.4.0 → pmquant-0.4.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pmquant
-Version: 0.4.0
+Version: 0.4.2
 Summary: Fail-closed execution and market-data layer for Polymarket CLOB V2: local signing, confirmed fills only, fee-correct math, working deposit-wallet (POLY_1271) support.
 Project-URL: Homepage, https://github.com/crp4222/pmq
 Project-URL: Issues, https://github.com/crp4222/pmq/issues
@@ -34,6 +34,8 @@ Description-Content-Type: text/markdown
 
 # pmq
 
+<!-- mcp-name: io.github.crp4222/pmq -->
+
 [![PyPI](https://img.shields.io/pypi/v/pmquant)](https://pypi.org/project/pmquant/)
 [![tests](https://github.com/crp4222/pmq/actions/workflows/test.yml/badge.svg)](https://github.com/crp4222/pmq/actions/workflows/test.yml)
 [![canary](https://github.com/crp4222/pmq/actions/workflows/canary.yml/badge.svg)](https://github.com/crp4222/pmq/actions/workflows/canary.yml)

{pmquant-0.4.0 → pmquant-0.4.2}/README.md

@@ -1,5 +1,7 @@
 # pmq
 
+<!-- mcp-name: io.github.crp4222/pmq -->
+
 [![PyPI](https://img.shields.io/pypi/v/pmquant)](https://pypi.org/project/pmquant/)
 [![tests](https://github.com/crp4222/pmq/actions/workflows/test.yml/badge.svg)](https://github.com/crp4222/pmq/actions/workflows/test.yml)
 [![canary](https://github.com/crp4222/pmq/actions/workflows/canary.yml/badge.svg)](https://github.com/crp4222/pmq/actions/workflows/canary.yml)

{pmquant-0.4.0 → pmquant-0.4.2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "pmquant"
-version = "0.4.0"
+version = "0.4.2"
 description = "Fail-closed execution and market-data layer for Polymarket CLOB V2: local signing, confirmed fills only, fee-correct math, working deposit-wallet (POLY_1271) support."
 readme = "README.md"
 license = { text = "MIT" }

pmquant-0.4.2/server.json

@@ -0,0 +1,61 @@
+{
+  "$schema": "https://static.modelcontextprotocol.io/schemas/2025-12-11/server.schema.json",
+  "name": "io.github.crp4222/pmq",
+  "title": "pmq (Polymarket CLOB V2)",
+  "description": "Fail-closed Polymarket CLOB V2 data and execution; trading tools gated behind PMQ_MCP_LIVE=1.",
+  "repository": {
+    "url": "https://github.com/crp4222/pmq",
+    "source": "github"
+  },
+  "version": "0.4.1",
+  "packages": [
+    {
+      "registryType": "pypi",
+      "registryBaseUrl": "https://pypi.org",
+      "identifier": "pmquant",
+      "version": "0.4.1",
+      "runtimeHint": "uvx",
+      "transport": {
+        "type": "stdio"
+      },
+      "environmentVariables": [
+        {
+          "name": "PMQ_MCP_LIVE",
+          "description": "Set to 1 to register the trading tools (fak_buy, fak_sell, cancel_and_reconcile). Unset or any other value keeps the server read-only.",
+          "isRequired": false,
+          "isSecret": false
+        },
+        {
+          "name": "PMQ_MCP_MAX_USD",
+          "description": "Per-order hard cap in USD enforced before any live buy (default 10). Only relevant when PMQ_MCP_LIVE=1.",
+          "isRequired": false,
+          "isSecret": false
+        },
+        {
+          "name": "POLY_PRIVATE_KEY",
+          "description": "EOA private key used only to sign orders locally when trading tools are enabled. Never printed or transmitted. Not needed for the read-only tools.",
+          "isRequired": false,
+          "isSecret": true
+        },
+        {
+          "name": "POLY_FUNDER",
+          "description": "Funder wallet address. Required with signature_type > 0 (deposit wallets). Only used when trading tools are enabled.",
+          "isRequired": false,
+          "isSecret": false
+        },
+        {
+          "name": "POLY_SIG_TYPE",
+          "description": "Polymarket signature type: 0 EOA, 1 POLY_PROXY, 2 POLY_GNOSIS_SAFE, 3 POLY_1271 (app deposit wallet). Run pmq-doctor if unsure.",
+          "isRequired": false,
+          "isSecret": false
+        },
+        {
+          "name": "POLY_BUILDER_CODE",
+          "description": "Optional bytes32 builder attribution code. Defaults to the maintainer's public zero-commission code (disclosed in the README); set your own or leave empty to opt out.",
+          "isRequired": false,
+          "isSecret": false
+        }
+      ]
+    }
+  ]
+}

{pmquant-0.4.0 → pmquant-0.4.2}/src/pmq/__init__.py

@@ -25,7 +25,7 @@ from .data import (
 )
 from .exceptions import IntrospectionMismatch, OrderUncertain, PmqError
 
-__version__ = "0.4.0"
+__version__ = "0.4.2"
 __all__ = [
     "FEE_RATES", "band_ask_depth_usd", "best_bid_ask", "book_inferred_winner",
     "book_meta", "event_markets", "fee", "get_book", "get_market", "get_tape",

{pmquant-0.4.0 → pmquant-0.4.2}/src/pmq/doctor.py

@@ -81,9 +81,17 @@ def main(argv: list[str] | None = None) -> int:
     # 1. installed surface
     try:
         from py_clob_client_v2.client import ClobClient
-        from py_clob_client_v2.clob_types import MarketOrderArgsV2, OrderType
-
-        from .executor import _EXPECTED_MARKET_ARGS, _EXPECTED_METHODS
+        from py_clob_client_v2.clob_types import (
+            MarketOrderArgsV2,
+            OrderArgsV2,
+            OrderType,
+        )
+
+        from .executor import (
+            _EXPECTED_MARKET_ARGS,
+            _EXPECTED_METHODS,
+            _EXPECTED_ORDER_ARGS,
+        )
         drifts = []
         for name, params in _EXPECTED_METHODS.items():
             fn = getattr(ClobClient, name, None)
@@ -91,6 +99,8 @@ def main(argv: list[str] | None = None) -> int:
             drifts += [f"{name}.{p}" for p in params if p not in have]
         have = set(inspect.signature(MarketOrderArgsV2).parameters)
         drifts += [f"MarketOrderArgsV2.{p}" for p in _EXPECTED_MARKET_ARGS if p not in have]
+        have = set(inspect.signature(OrderArgsV2).parameters)
+        drifts += [f"OrderArgsV2.{p}" for p in _EXPECTED_ORDER_ARGS if p not in have]
         if not hasattr(OrderType, "FAK"):
             drifts.append("OrderType.FAK")
         all_ok &= check(not drifts, "installed py-clob-client-v2 matches the verified surface",

{pmquant-0.4.0 → pmquant-0.4.2}/src/pmq/executor.py

@@ -36,6 +36,7 @@ import logging
 import os
 import re
 from dataclasses import dataclass, field
+from decimal import ROUND_DOWN, Decimal
 from typing import Any
 
 from .data import FEE_RATES, fee
@@ -55,6 +56,17 @@ BYTES32_RE = re.compile(r"0x[0-9a-fA-F]{64}")
 DEFAULT_BUILDER_CODE = "0x4b22812cf929165a247b575eb417a3b6c9e3c12e96f0159c4d0ad39f78d17371"
 _UNSET: Any = object()
 
+_CENT = Decimal("0.01")
+
+
+def _floor_cents(x: float) -> float:
+    """Round DOWN to the cent without the binary-float drift that makes
+    ``int(16.90 * 100) / 100`` return ``16.89``. ``str(x)`` yields the shortest
+    decimal repr, so ``16.90`` stays ``16.90`` while a genuine ``16.907`` still
+    floors to ``16.90``. Rounding down preserves the never-exceed-budget
+    contract for buys and the never-oversell contract for sells."""
+    return float(Decimal(str(x)).quantize(_CENT, rounding=ROUND_DOWN))
+
 
 @dataclass
 class Fill:
@@ -87,6 +99,8 @@ _EXPECTED_METHODS: dict[str, tuple[str, ...]] = {
 }
 _EXPECTED_MARKET_ARGS: tuple[str, ...] = (
     "token_id", "amount", "side", "price", "builder_code")
+_EXPECTED_ORDER_ARGS: tuple[str, ...] = (
+    "token_id", "price", "size", "side", "builder_code")
 
 
 class PolymarketExecutor:
@@ -176,6 +190,11 @@ class PolymarketExecutor:
         for p in _EXPECTED_MARKET_ARGS:
             if p not in have:
                 drifts.append(f"MarketOrderArgsV2 lost field {p}")
+        oargs = self._t["OrderArgs"]
+        have = set(inspect.signature(oargs).parameters)
+        for p in _EXPECTED_ORDER_ARGS:
+            if p not in have:
+                drifts.append(f"OrderArgsV2 lost field {p}")
         if not hasattr(self._t["OrderType"], "FAK"):
             drifts.append("OrderType.FAK missing")
         if drifts:
@@ -257,7 +276,7 @@ class PolymarketExecutor:
         killed by the exchange; nothing ever rests. Returns a :class:`Fill`;
         book ONLY ``fill.matched_shares`` and ``fill.matched_usd``.
         Raises :class:`OrderUncertain` when the outcome is unknown."""
-        usd = int(usd * 100) / 100.0
+        usd = _floor_cents(usd)
         if usd <= 0:
             return Fill(rejected=True, error="usd amount rounds to zero")
         return self._market_order(token_id, usd, "BUY", price_cap)
@@ -267,7 +286,7 @@ class PolymarketExecutor:
         ``price_floor``. Same confirmation contract as :meth:`buy_fak`.
         The buy path has carried live volume; the sell path follows the same
         documented semantics but flag it as less battle-tested."""
-        shares = int(shares * 100) / 100.0
+        shares = _floor_cents(shares)
         if shares <= 0:
             return Fill(rejected=True, error="share amount rounds to zero")
         return self._market_order(token_id, shares, "SELL", price_floor)

{pmquant-0.4.0 → pmquant-0.4.2}/tests/test_executor.py

@@ -114,6 +114,23 @@ def test_buy_amount_rounds_down_to_cent():
     assert fc.calls[0][1].amount == 4.99
 
 
+def test_cent_rounding_has_no_binary_drift():
+    from pmq.executor import _floor_cents
+    # int(x*100)/100 would return one cent low here; _floor_cents must not.
+    assert _floor_cents(16.90) == 16.90
+    assert _floor_cents(33.30) == 33.30
+    assert _floor_cents(66.60) == 66.60
+    # genuine sub-cent values still floor
+    assert _floor_cents(5.007) == 5.00
+    assert _floor_cents(4.999) == 4.99
+
+
+def test_buy_fak_sends_intended_cents_not_drifted():
+    fc = FakeClient(market_resp={"orderID": "0x1", "makingAmount": "0", "takingAmount": "0"})
+    make(fc).buy_fak("tok", 0.97, 16.90)
+    assert fc.calls[0][1].amount == 16.90  # not 16.89
+
+
 def test_zero_amount_never_reaches_the_wire():
     fc = FakeClient()
     f = make(fc).buy_fak("tok", 0.97, 0.004)
@@ -255,6 +272,19 @@ def test_introspection_guard_refuses_drifted_client():
         make(Drifted())
 
 
+def test_introspection_guard_covers_order_args_builder_code(monkeypatch):
+    # limit_gtc now depends on OrderArgsV2.builder_code; the guard must refuse
+    # a client whose OrderArgsV2 dropped it, not crash later at call time.
+    import py_clob_client_v2.clob_types as ct
+
+    class NoBuilderOrderArgs:
+        def __init__(self, token_id, price, size, side):
+            self.token_id, self.price, self.size, self.side = token_id, price, size, side
+    monkeypatch.setattr(ct, "OrderArgsV2", NoBuilderOrderArgs)
+    with pytest.raises(IntrospectionMismatch, match="OrderArgsV2 lost field builder_code"):
+        make(FakeClient())
+
+
 def test_reconcile_cancels_then_reports_truth():
     fc = FakeClient(trades=[{"side": "BUY", "size": "2", "price": "0.95"}],
                     open_orders=[])