ipor-fusion 4.0.0.dev52__tar.gz → 4.0.0.dev54__tar.gz

{ipor_fusion-4.0.0.dev52 → ipor_fusion-4.0.0.dev54}/MIGRATION_4.md

@@ -119,7 +119,8 @@ takes a `ProposalHash`; `BalanceProposal.proposal_hash` and `NavMark.proposal_ha
 are `ProposalHash`; `ExternalStateEvent.transaction_hash` is a `TxHash`.
 
 **Errors and explorer.** `TransactionError(tx_hash=)` takes a `TxHash`,
-`LogScanError(address=)` a `ChecksumAddress`; `get_deployment_tx` returns a `TxHash`
+`LogScanError(address=)` a `ChecksumAddress`; `get_revert_reason` takes a `TxHash`;
+`get_deployment_tx` returns a `TxHash`
 (a malformed hash from Etherscan is reported as the error `malformed-tx-hash`).
 
 **MCP response models** (Python constructors only; the JSON schemas are unchanged

{ipor_fusion-4.0.0.dev52 → ipor_fusion-4.0.0.dev54}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: ipor-fusion
-Version: 4.0.0.dev52
+Version: 4.0.0.dev54
 Summary: The IPOR Fusion SDK for Python
 Project-URL: homepage, https://ipor.io
 Project-URL: repository, https://github.com/IPOR-Labs/ipor-fusion.py

{ipor_fusion-4.0.0.dev52 → ipor_fusion-4.0.0.dev54}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "ipor-fusion"
-version = "4.0.0.dev52"
+version = "4.0.0.dev54"
 description = "The IPOR Fusion SDK for Python"
 authors = [
     { name = "Adam Mydlarz", email = "amydlarz@gmail.com" },

{ipor_fusion-4.0.0.dev52 → ipor_fusion-4.0.0.dev54}/src/ipor_fusion/core/context.py

@@ -224,13 +224,13 @@ class Web3Context:
             "data": data_hex,
         }
 
-    def _handle_receipt(self, tx_hash, receipt) -> TxReceipt:
+    def _handle_receipt(self, tx_hash: bytes, receipt: TxReceipt) -> TxReceipt:
         if receipt["status"] != 1:
-            reason = get_revert_reason(self.web3, tx_hash, receipt)
+            parsed = parse_tx_hash(tx_hash)
             raise TransactionError(
                 "Transaction failed",
-                tx_hash=parse_tx_hash(bytes(tx_hash)),
-                revert_reason=reason,
+                tx_hash=parsed,
+                revert_reason=get_revert_reason(self.web3, parsed, receipt),
             )
         return receipt
 

{ipor_fusion-4.0.0.dev52 → ipor_fusion-4.0.0.dev54}/src/ipor_fusion/errors.py

@@ -115,10 +115,10 @@ def _decode_revert_reason(data: bytes) -> str:
     return hex_str
 
 
-def get_revert_reason(web3: Web3, tx_hash: bytes, receipt: TxReceipt) -> str | None:
+def get_revert_reason(web3: Web3, tx_hash: TxHash, receipt: TxReceipt) -> str | None:
     """Replay a failed tx as eth_call to capture revert data."""
     try:
-        tx = web3.eth.get_transaction(tx_hash)  # type: ignore[arg-type]
+        tx = web3.eth.get_transaction(tx_hash)
         call_params = {
             "from": tx["from"],
             "to": tx["to"],

{ipor_fusion-4.0.0.dev52 → ipor_fusion-4.0.0.dev54}/tests/test_revert_decoding.py

@@ -12,6 +12,9 @@ from ipor_fusion.errors import (
     _decode_revert_reason,
     get_revert_reason,
 )
+from ipor_fusion.types import parse_tx_hash
+
+_TX_HASH = parse_tx_hash(bytes(32))
 
 ERROR_SELECTOR = bytes.fromhex("08c379a0")
 PANIC_SELECTOR = bytes.fromhex("4e487b71")
@@ -160,7 +163,7 @@ def test_get_revert_reason_replay_succeeds():
     """When eth_call replays without error, return None."""
     web3 = _make_web3()
     web3.eth.call.return_value = b""
-    assert get_revert_reason(web3, b"\x00" * 32, _receipt()) is None
+    assert get_revert_reason(web3, _TX_HASH, _receipt()) is None
 
 
 def test_get_revert_reason_exception_with_hex_data():
@@ -171,7 +174,7 @@ def test_get_revert_reason_exception_with_hex_data():
     exc.data = hex_data  # type: ignore[attr-defined]
 
     web3 = _make_web3(call_side_effect=exc)
-    result = get_revert_reason(web3, b"\x00" * 32, _receipt())
+    result = get_revert_reason(web3, _TX_HASH, _receipt())
     assert result == 'Error("not enough")'
 
 
@@ -179,7 +182,7 @@ def test_get_revert_reason_message_contains_revert():
     """Exception message containing 'revert' is returned as-is."""
     exc = Exception("execution reverted: some reason")
     web3 = _make_web3(call_side_effect=exc)
-    result = get_revert_reason(web3, b"\x00" * 32, _receipt())
+    result = get_revert_reason(web3, _TX_HASH, _receipt())
     assert "execution reverted" in result
 
 
@@ -187,7 +190,7 @@ def test_get_revert_reason_message_contains_revert_case_insensitive():
     """Case-insensitive match on 'Revert'."""
     exc = Exception("Revert happened")
     web3 = _make_web3(call_side_effect=exc)
-    result = get_revert_reason(web3, b"\x00" * 32, _receipt())
+    result = get_revert_reason(web3, _TX_HASH, _receipt())
     assert "Revert" in result
 
 
@@ -195,14 +198,14 @@ def test_get_revert_reason_unrecognised_exception():
     """Unrelated exception returns None."""
     exc = Exception("network timeout")
     web3 = _make_web3(call_side_effect=exc)
-    assert get_revert_reason(web3, b"\x00" * 32, _receipt()) is None
+    assert get_revert_reason(web3, _TX_HASH, _receipt()) is None
 
 
 def test_get_revert_reason_tx_without_gas():
     """Transaction without 'gas' field still works."""
     web3 = _make_web3(tx_has_gas=False)
     web3.eth.call.return_value = b""
-    assert get_revert_reason(web3, b"\x00" * 32, _receipt()) is None
+    assert get_revert_reason(web3, _TX_HASH, _receipt()) is None
 
 
 def test_get_revert_reason_data_not_hex_prefix():
@@ -211,7 +214,7 @@ def test_get_revert_reason_data_not_hex_prefix():
     exc.data = "not-hex"  # type: ignore[attr-defined]
     web3 = _make_web3(call_side_effect=exc)
     # Message doesn't contain "revert", data doesn't start with "0x"
-    assert get_revert_reason(web3, b"\x00" * 32, _receipt()) is None
+    assert get_revert_reason(web3, _TX_HASH, _receipt()) is None
 
 
 # ---------------------------------------------------------------------------
@@ -325,6 +328,6 @@ def test_get_revert_reason_uses_the_registry():
     exc = Exception("execution reverted")
     exc.data = "0x" + _custom("SdkTestGated(uint256)", 7).hex()  # type: ignore[attr-defined]
     web3.eth.call.side_effect = exc
-    assert get_revert_reason(web3, b"\x00" * 32, {"blockNumber": 1}) == (  # type: ignore[arg-type]
+    assert get_revert_reason(web3, _TX_HASH, {"blockNumber": 1}) == (  # type: ignore[arg-type]
         "SdkTestGated(7)"
     )

{ipor_fusion-4.0.0.dev52 → ipor_fusion-4.0.0.dev54}/tests/test_web3_context.py

@@ -338,7 +338,7 @@ class TestHandleReceipt:
 
         assert exc_info.value.tx_hash == tx_hash.to_0x_hex()
         assert exc_info.value.revert_reason == 'Error("Insufficient balance")'
-        mock_get_reason.assert_called_once_with(ctx.web3, tx_hash, receipt)
+        mock_get_reason.assert_called_once_with(ctx.web3, tx_hash.to_0x_hex(), receipt)
 
     @patch("ipor_fusion.core.context.get_revert_reason")
     def test_failed_receipt_with_no_revert_reason(self, mock_get_reason):