pmquant 0.4.1__tar.gz → 0.4.2__tar.gz

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

@@ -1,5 +1,14 @@
 # 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

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pmquant
-Version: 0.4.1
+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

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

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "pmquant"
-version = "0.4.1"
+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.1 → pmquant-0.4.2}/server.json

@@ -2,7 +2,7 @@
   "$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. Read tools (markets, book, fees, account) need no credentials; trading tools are registered only when the operator sets PMQ_MCP_LIVE=1, with a per-order USD cap.",
+  "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"

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

@@ -25,7 +25,7 @@ from .data import (
 )
 from .exceptions import IntrospectionMismatch, OrderUncertain, PmqError
 
-__version__ = "0.4.1"
+__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.1 → 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:
@@ -264,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)
@@ -274,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.1 → 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)