ipor-fusion 4.2.0.dev18__tar.gz → 4.2.0.dev23__tar.gz

{ipor_fusion-4.2.0.dev18 → ipor_fusion-4.2.0.dev23}/AGENTS.md

@@ -202,6 +202,10 @@ checkout's default profile), the pending reader included.
 
 ## Layout (`src/ipor_fusion/`)
 
+- `redaction.py` — `redact_provider_urls`, the one redactor for text carrying
+  exception messages (the CLI group and `main`, every MCP tool through
+  `_RedactingFastMCP.call_tool`, `LogScanError` through `core/logs.py`), and
+  `mask_provider_url`; a new entry point goes through it
 - `chains.py` chain registry; `market_ids.py` `IporFusionMarkets`; `types.py`; `errors.py`
   defines every SDK exception class (`tests/test_error_hierarchy.py` pins it)
 - `fusion_errors.py` — `FUSION_ERROR_SIGNATURES`, every custom error of the

{ipor_fusion-4.2.0.dev18 → ipor_fusion-4.2.0.dev23}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: ipor-fusion
-Version: 4.2.0.dev18
+Version: 4.2.0.dev23
 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.2.0.dev18 → ipor_fusion-4.2.0.dev23}/pyproject.toml

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

{ipor_fusion-4.2.0.dev18 → ipor_fusion-4.2.0.dev23}/src/ipor_fusion/cli/config_cmd.py

@@ -5,9 +5,9 @@ import click
 from ipor_fusion.cli.config_store import (
     FusionConfig,
     load_config,
-    mask_provider_url,
     save_config,
 )
+from ipor_fusion.redaction import mask_provider_url
 from ipor_fusion.types import ChainId
 
 

{ipor_fusion-4.2.0.dev18 → ipor_fusion-4.2.0.dev23}/src/ipor_fusion/cli/config_store.py

@@ -3,7 +3,6 @@ from __future__ import annotations
 import json
 import os
 from pathlib import Path
-from urllib.parse import urlsplit
 
 import click
 from eth_typing import ChecksumAddress
@@ -54,18 +53,14 @@ class FusionConfig(BaseModel):
         return data
 
 
-def mask_provider_url(url: str) -> str:
-    """Render a provider URL as scheme and host only.
-
-    RPC provider URLs carry API keys in the path, the query string or the
-    userinfo, so nothing but the scheme and the host (with port) may be
-    displayed. An unparsable value is rendered as ``***``.
-    """
-    parts = urlsplit(url)
-    host = parts.netloc.rpartition("@")[2]
-    if not parts.scheme or not host:
-        return "***"
-    return f"{parts.scheme}://{host}"
+def _describe_invalid_fields(exc: ValidationError) -> str:
+    """Which fields failed and why, never their values: the file holds API keys."""
+    return "\n".join(
+        f"  {'.'.join(str(part) for part in error['loc'])}: {error['msg']}"
+        for error in exc.errors(
+            include_input=False, include_url=False, include_context=False
+        )
+    )
 
 
 def load_config() -> FusionConfig:
@@ -75,7 +70,7 @@ def load_config() -> FusionConfig:
         return FusionConfig.model_validate_json(CONFIG_FILE.read_text(encoding="utf-8"))
     except ValidationError as exc:
         raise click.ClickException(
-            f"Config {CONFIG_FILE} is invalid:\n{exc}\n"
+            f"Config {CONFIG_FILE} is invalid:\n{_describe_invalid_fields(exc)}\n"
             "Fix the file manually or delete it to start fresh."
         ) from exc
     except json.JSONDecodeError as exc:

{ipor_fusion-4.2.0.dev18 → ipor_fusion-4.2.0.dev23}/src/ipor_fusion/cli/main.py

@@ -4,6 +4,7 @@ import os
 import sys
 
 import click
+import requests
 
 from ipor_fusion.cli.address_cmd import address
 from ipor_fusion.cli.changelog_cmd import changelog
@@ -12,9 +13,21 @@ from ipor_fusion.cli.fuse_cmd import fuse
 from ipor_fusion.cli.market_cmd import market
 from ipor_fusion.cli.preflight_cmd import preflight
 from ipor_fusion.cli.vault_cmd import vault
+from ipor_fusion.redaction import redact_provider_urls
 
 
-@click.group()
+class _RedactingGroup(click.Group):
+    """Click prints a `ClickException` itself, before `main` could see it."""
+
+    def invoke(self, ctx: click.Context) -> object:
+        try:
+            return super().invoke(ctx)
+        except click.ClickException as exc:
+            exc.message = redact_provider_urls(exc.message)
+            raise
+
+
+@click.group(cls=_RedactingGroup)
 @click.version_option(package_name="ipor_fusion")
 @click.option("--verbose", "-v", is_flag=True, help="Print RPC call details.")
 @click.option("--quiet", "-q", is_flag=True, help="Suppress non-essential output.")
@@ -49,16 +62,17 @@ def main() -> None:
         sys.exit(130)
     except click.UsageError:
         raise
-    except ConnectionError as exc:
-        click.secho(f"Connection failed: {exc}. Check your provider URL.", fg="red")
+    except (ConnectionError, requests.ConnectionError) as exc:
+        msg = redact_provider_urls(str(exc))
+        click.secho(f"Connection failed: {msg}. Check your provider URL.", fg="red")
         sys.exit(1)
     except ValueError as exc:
-        msg = str(exc)
+        msg = redact_provider_urls(str(exc))
         if "address" in msg.lower():
             click.secho(f"Invalid address: {msg}", fg="red")
         else:
             click.secho(f"Error: {msg}", fg="red")
         sys.exit(1)
     except Exception as exc:
-        click.secho(f"Error: {exc}", fg="red")
+        click.secho(f"Error: {redact_provider_urls(str(exc))}", fg="red")
         sys.exit(1)

{ipor_fusion-4.2.0.dev18 → ipor_fusion-4.2.0.dev23}/src/ipor_fusion/core/logs.py

@@ -28,6 +28,7 @@ from web3.exceptions import Web3Exception
 from web3.types import BlockIdentifier, FilterParams, LogReceipt
 
 from ipor_fusion.errors import LogScanError
+from ipor_fusion.redaction import redact_provider_urls
 from ipor_fusion.types import ChainId
 
 # The log shapes the SDK's event readers accept: a receipt log from web3.py
@@ -70,7 +71,6 @@ _TRANSIENT_TEXT = re.compile(
     r"rate.?limit|too many requests|compute units|temporar|try again later", re.I
 )
 _TRANSIENT_STATUS = frozenset({429, 502, 503})
-_URL = re.compile(r"\w+://\S+")
 
 
 class LogSource(Protocol):
@@ -370,13 +370,13 @@ def _status_code(exc: BaseException) -> int | None:
 
 
 def _describe(exc: BaseException) -> str:
-    """Error text with the response body, provider URLs (they embed API keys)
-    removed, truncated."""
+    """Error text with the response body, provider URLs reduced to their host,
+    truncated."""
     text = f"{type(exc).__name__}: {exc}"
     body = getattr(getattr(exc, "response", None), "text", None)
     if isinstance(body, str) and body:
         text = f"{text} {body}"
-    return _URL.sub("<provider>", text)[:400]
+    return redact_provider_urls(text)[:400]
 
 
 def _is_transient(exc: BaseException) -> bool:

{ipor_fusion-4.2.0.dev18 → ipor_fusion-4.2.0.dev23}/src/ipor_fusion/mcp/server.py

@@ -4,12 +4,14 @@ Calls the ipor_fusion SDK directly — no CLI subprocess.
 Configuration is loaded from the shared CLI config (~/.config/ipor-fusion/).
 """
 
-from collections.abc import Callable
+from collections.abc import Callable, Sequence
 from functools import partial
-from typing import Annotated, Literal
+from typing import Annotated, Any, Literal
 
 from eth_typing import ChecksumAddress
 from mcp.server.fastmcp import FastMCP
+from mcp.server.fastmcp.exceptions import ToolError
+from mcp.types import ContentBlock
 from pydantic import AfterValidator, BeforeValidator, Field
 from web3 import Web3
 
@@ -23,7 +25,6 @@ from ipor_fusion.cli.config_store import (
     FusionConfig,
     VaultEntry,
     load_config,
-    mask_provider_url,
     resolve_chain_id,
     save_config,
 )
@@ -69,6 +70,7 @@ from ipor_fusion.preflight import (
 )
 from ipor_fusion.readers.morpho import MorphoReader, morpho_blue_address
 from ipor_fusion.readers.oracle_mapping import build_oracle_mapping
+from ipor_fusion.redaction import mask_provider_url, redact_provider_urls
 from ipor_fusion.registry import (
     build_address_list_json,
     build_address_lookup_json,
@@ -92,7 +94,20 @@ from ipor_fusion.vault_info import build_vault_info_json, build_vault_validation
 
 # chain_id default: detect the chain from the saved vault or the single provider.
 
-mcp = FastMCP(
+
+class _RedactingFastMCP(FastMCP):
+    """Every tool failure reaches the client through `call_tool`."""
+
+    async def call_tool(
+        self, name: str, arguments: dict[str, Any]
+    ) -> Sequence[ContentBlock] | dict[str, Any]:
+        try:
+            return await super().call_tool(name, arguments)
+        except Exception as exc:
+            raise ToolError(redact_provider_urls(str(exc))) from exc
+
+
+mcp = _RedactingFastMCP(
     "ipor-fusion",
     instructions=(
         "Inspection and configuration tools for IPOR Fusion Plasma Vaults. "

{ipor_fusion-4.2.0.dev18 → ipor_fusion-4.2.0.dev23}/src/ipor_fusion/readers/lending_health.py

@@ -298,6 +298,8 @@ def _compute_morpho_market_health(
         position.borrow_shares, market.total_borrow_assets, market.total_borrow_shares
     )
 
+    # Morpho's own check sees a zero borrow limit here, so the position can be
+    # liquidated: a known risk, of which only the LTV has no value.
     if position.collateral == 0 or oracle_price == 0:
         return LendingMarketHealth(
             protocol="morpho",
@@ -305,7 +307,7 @@ def _compute_morpho_market_health(
             market_name=market_name,
             current_ltv=None,
             max_ltv=params.lltv / WAD,
-            health_factor=None,
+            health_factor=0.0,
             total_collateral_usd=None,
             total_debt_usd=None,
             ltv_usage_percent=None,
@@ -322,9 +324,9 @@ def _compute_morpho_market_health(
     max_ltv = params.lltv / WAD
 
     # health_factor = max_borrow / borrowed
-    # max_borrow = collateral * oracle_price / ORACLE_PRICE_SCALE * lltv / WAD
+    # Two floors, as in Morpho's _isHealthy.
     max_borrow = (
-        position.collateral * oracle_price * params.lltv // (ORACLE_PRICE_SCALE * WAD)
+        position.collateral * oracle_price // ORACLE_PRICE_SCALE * params.lltv // WAD
     )
     health_factor = max_borrow / borrowed if borrowed > 0 else None
 
@@ -336,7 +338,7 @@ def _compute_morpho_market_health(
         market_name=market_name,
         current_ltv=round(current_ltv, 6),
         max_ltv=round(max_ltv, 6),
-        health_factor=round(health_factor, 4) if health_factor else None,
+        health_factor=round(health_factor, 4) if health_factor is not None else None,
         total_collateral_usd=None,
         total_debt_usd=None,
         ltv_usage_percent=round(ltv_usage, 2) if ltv_usage is not None else None,
@@ -574,9 +576,17 @@ def _compute_euler_v2_account_health(
         if ltv > 0
     )
     liability = liquidity.liability_value
+    # The vault owes, yet its oracle values that at nothing: no ratio means anything.
+    if liability == 0:
+        return UnreadableLendingMarket(
+            protocol="euler_v2",
+            market_id=ipor_market_id,
+            market_name=market_name,
+            reason="debt is valued at zero",
+            substrate_id=sub_account,
+        )
     current_ltv = liability / collateral if collateral > 0 else None
     max_ltv = adjusted / collateral if collateral > 0 else 0.0
-    health_factor = adjusted / liability if liability > 0 else None
     ltv_usage = liability / adjusted * 100 if adjusted > 0 else None
     # The USD unit of account prices in 18 decimals; other units are not USD.
     in_usd = unit_read.value == EULER_USD_UNIT_OF_ACCOUNT
@@ -587,7 +597,7 @@ def _compute_euler_v2_account_health(
         market_name=market_name,
         current_ltv=round(current_ltv, 6) if current_ltv is not None else None,
         max_ltv=round(max_ltv, 6),
-        health_factor=round(health_factor, 4) if health_factor is not None else None,
+        health_factor=round(adjusted / liability, 4),
         total_collateral_usd=round(collateral / WAD, 2) if in_usd else None,
         total_debt_usd=round(liability / WAD, 2) if in_usd else None,
         ltv_usage_percent=round(ltv_usage, 2) if ltv_usage is not None else None,

ipor_fusion-4.2.0.dev23/src/ipor_fusion/redaction.py

@@ -0,0 +1,39 @@
+"""Removes provider URLs from text meant for a person or a log.
+
+RPC provider URLs carry API keys in the path, the query string or the userinfo,
+and requests and web3 quote them in their exception messages.
+"""
+
+from __future__ import annotations
+
+import re
+from urllib.parse import urlsplit
+
+# A URL runs to the next whitespace, since quotes and brackets are legal inside
+# one and a key may follow them; only punctuation closing it stays in the text.
+# The lookbehind tries each run of scheme characters once, so long hex in an error
+# stays linear; a run that is no valid scheme ("0xhttp") comes out masked whole.
+_URL = re.compile(r"(?<![a-zA-Z0-9+.\-])[a-zA-Z0-9+.\-]+://\S*[^\s.,;:)'\">]")
+# urllib3 names only the path and query: "Max retries exceeded with url: /v2/key"
+_REQUEST_TARGET = re.compile(r"\burl: /\S*")
+_HOST = re.compile(r"(?:[a-zA-Z0-9.\-]+|\[[0-9a-fA-F:.]+\])(?::\d+)?")
+
+
+def mask_provider_url(url: str) -> str:
+    """Render a provider URL as scheme and host only. An unparsable value is
+    rendered as ``***``: what looks like its host may be part of a key."""
+    try:
+        parts = urlsplit(url)
+    except ValueError:
+        return "***"
+    host = parts.netloc.rpartition("@")[2]
+    if not parts.scheme or not _HOST.fullmatch(host):
+        return "***"
+    return f"{parts.scheme}://{host}"
+
+
+def redact_provider_urls(text: str) -> str:
+    """``text`` with every URL reduced to scheme and host, and every request
+    target urllib3 quotes on its own replaced."""
+    text = _URL.sub(lambda match: mask_provider_url(match.group()), text)
+    return _REQUEST_TARGET.sub("url: <redacted>", text)

{ipor_fusion-4.2.0.dev18 → ipor_fusion-4.2.0.dev23}/src/ipor_fusion/validation.py

@@ -691,8 +691,9 @@ def _lending_health_findings(state: VaultState) -> list[Finding]:
         for u in health.unreadable
     ]
     for market in health.markets:
-        # A critical row with no usage (debt against a zero limit) is still critical.
-        if market.ltv_usage_percent is None and not market.is_critical:
+        # A row with neither value gives no risk metric to judge. Debt against a zero
+        # limit has no usage but a critical health factor.
+        if market.ltv_usage_percent is None and market.health_factor is None:
             continue
         ltv = f"{market.current_ltv:.4f}" if market.current_ltv is not None else "?"
         usage = (

{ipor_fusion-4.2.0.dev18 → ipor_fusion-4.2.0.dev23}/tests/test_cli_config_store.py

@@ -8,9 +8,9 @@ from ipor_fusion.cli.config_store import (
     FusionConfig,
     VaultEntry,
     load_config,
-    mask_provider_url,
     save_config,
 )
+from ipor_fusion.redaction import mask_provider_url
 
 
 @pytest.fixture(autouse=True)

{ipor_fusion-4.2.0.dev18 → ipor_fusion-4.2.0.dev23}/tests/test_cli_helpers.py

@@ -1532,6 +1532,18 @@ class TestPrintLendingHealth:
         _print_lending_health(data)
         assert "N/A" in capsys.readouterr().out
 
+    def test_debt_without_ltv_is_not_shown_as_no_debt(self, capsys):
+        m = _make_lending_market(
+            current_ltv=None, health_factor=0.0, ltv_usage_percent=None
+        )
+        m.protocol = "aave_v3"
+        m.market_id = MarketId(1)
+        data = _make_data(lending_health=VaultLendingHealth(markets=[m]))
+        _print_lending_health(data)
+        out = capsys.readouterr().out
+        assert "CRITICAL" in out
+        assert "no debt" not in out
+
 
 class TestHealthCheckLendingIntegration:
     def test_lending_ok_in_health_check(self, capsys):
@@ -1556,6 +1568,22 @@ class TestHealthCheckLendingIntegration:
         assert "CRITICAL" in captured.out
         assert "NEAR LIQUIDATION" in captured.out
 
+    def test_debt_without_ltv_is_critical_not_ok(self, capsys):
+        m = _make_lending_market(
+            current_ltv=None, ltv_usage_percent=None, health_factor=0.0
+        )
+        data = _make_data(lending_health=VaultLendingHealth(markets=[m]))
+        _print_health_check(validate(data, 100 * 10**18))
+        captured = capsys.readouterr()
+        assert "CRITICAL" in captured.out
+        assert "NEAR LIQUIDATION" in captured.out
+
+    def test_lending_warning_without_usage_is_reported(self, capsys):
+        m = _make_lending_market(ltv_usage_percent=None, health_factor=1.08)
+        data = _make_data(lending_health=VaultLendingHealth(markets=[m]))
+        _print_health_check(validate(data, 100 * 10**18))
+        assert "approaching liquidation" in capsys.readouterr().out
+
     def test_lending_none_usage_skipped(self, capsys):
         m = _make_lending_market(ltv_usage_percent=None, health_factor=None)
         data = _make_data(lending_health=VaultLendingHealth(markets=[m]))

{ipor_fusion-4.2.0.dev18 → ipor_fusion-4.2.0.dev23}/tests/test_cli_lending_health.py

@@ -419,6 +419,115 @@ class TestComputeMorphoMarketHealth:
         assert isinstance(result, LendingMarketHealth)
         assert result.current_ltv is None
         assert result.ltv_usage_percent is None
+        assert result.health_factor == 0.0
+        assert result.is_critical
+
+    def test_zero_oracle_price_with_debt_is_critical(self):
+        reader, ctx = _make_morpho_reader()
+
+        ctx.call.side_effect = sequenced(
+            [
+                encode(["uint256", "uint128", "uint128"], [0, 100, 1000]),
+                encode(
+                    ["uint128", "uint128", "uint128", "uint128", "uint128", "uint128"],
+                    [2000, 2000, 1000, 1000, 1700000000, 0],
+                ),
+                encode(
+                    ["address", "address", "address", "address", "uint256"],
+                    [TOKEN_A, TOKEN_B, ORACLE_ADDR, IRM_ADDR, 860000000000000000],
+                ),
+                encode(["uint256"], [0]),
+            ]
+        )
+
+        result = _compute_morpho_market_health(
+            ctx, reader, MORPHO_MARKET_ID, VAULT_ADDR, 14, "MORPHO"
+        )
+
+        assert isinstance(result, LendingMarketHealth)
+        assert result.current_ltv is None
+        assert result.health_factor == 0.0
+        assert result.is_critical
+
+    def test_dust_collateral_flooring_the_limit_to_zero_is_critical(self):
+        reader, ctx = _make_morpho_reader()
+
+        ctx.call.side_effect = sequenced(
+            [
+                encode(["uint256", "uint128", "uint128"], [0, 500 * 10**6, 1]),
+                encode(
+                    ["uint128", "uint128", "uint128", "uint128", "uint128", "uint128"],
+                    [2000, 2000 * 10**6, 1000, 1000 * 10**6, 1700000000, 0],
+                ),
+                encode(
+                    ["address", "address", "address", "address", "uint256"],
+                    [TOKEN_A, TOKEN_B, ORACLE_ADDR, IRM_ADDR, 860000000000000000],
+                ),
+                encode(["uint256"], [ORACLE_PRICE_SCALE]),
+            ]
+        )
+
+        result = _compute_morpho_market_health(
+            ctx, reader, MORPHO_MARKET_ID, VAULT_ADDR, 14, "MORPHO"
+        )
+
+        assert isinstance(result, LendingMarketHealth)
+        assert result.current_ltv is not None
+        assert result.health_factor == 0.0
+        assert result.is_critical
+
+    def test_limit_floors_twice_as_morpho_does(self):
+        # 2 * 0.9 floors to 1, then 1 * 0.86 floors to 0; one combined floor gives 1.
+        reader, ctx = _make_morpho_reader()
+
+        ctx.call.side_effect = sequenced(
+            [
+                encode(["uint256", "uint128", "uint128"], [0, 10**6, 2]),
+                encode(
+                    ["uint128", "uint128", "uint128", "uint128", "uint128", "uint128"],
+                    [2000, 2000 * 10**6, 1000, 1000 * 10**6, 1700000000, 0],
+                ),
+                encode(
+                    ["address", "address", "address", "address", "uint256"],
+                    [TOKEN_A, TOKEN_B, ORACLE_ADDR, IRM_ADDR, 860000000000000000],
+                ),
+                encode(["uint256"], [9 * ORACLE_PRICE_SCALE // 10]),
+            ]
+        )
+
+        result = _compute_morpho_market_health(
+            ctx, reader, MORPHO_MARKET_ID, VAULT_ADDR, 14, "MORPHO"
+        )
+
+        assert isinstance(result, LendingMarketHealth)
+        assert result.health_factor == 0.0
+
+    def test_debt_on_a_zero_lltv_market_is_critical_without_usage(self):
+        reader, ctx = _make_morpho_reader()
+
+        ctx.call.side_effect = sequenced(
+            [
+                encode(["uint256", "uint128", "uint128"], [0, 500 * 10**6, 1000]),
+                encode(
+                    ["uint128", "uint128", "uint128", "uint128", "uint128", "uint128"],
+                    [2000, 2000 * 10**6, 1000, 1000 * 10**6, 1700000000, 0],
+                ),
+                encode(
+                    ["address", "address", "address", "address", "uint256"],
+                    [TOKEN_A, TOKEN_B, ORACLE_ADDR, IRM_ADDR, 0],
+                ),
+                encode(["uint256"], [ORACLE_PRICE_SCALE]),
+            ]
+        )
+
+        result = _compute_morpho_market_health(
+            ctx, reader, MORPHO_MARKET_ID, VAULT_ADDR, 14, "MORPHO"
+        )
+
+        assert isinstance(result, LendingMarketHealth)
+        assert result.ltv_usage_percent is None
+        assert result.health_factor == 0.0
+        assert result.is_critical
 
     def test_position_read_failure_is_unreadable(self):
         reader, ctx = _make_morpho_reader()
@@ -1162,6 +1271,8 @@ def _euler_ctx(
     debt: int = EULER_DEBT,
     unit: str = EULER_USD_UNIT_OF_ACCOUNT,
     liquidity_reverts: bool = False,
+    adjusted: int = EULER_ADJUSTED_COLLATERAL,
+    liability: int = EULER_LIABILITY,
 ) -> MagicMock:
     """Answer the TESS sub-account's reads; only the borrow vault is its controller."""
 
@@ -1177,7 +1288,7 @@ def _euler_ctx(
                 raise ContractLogicError("E_NoLiability")
             return encode(
                 ["address[]", "uint256[]", "uint256"],
-                [[EULER_COLLATERAL], [EULER_ADJUSTED_COLLATERAL], EULER_LIABILITY],
+                [[EULER_COLLATERAL], [adjusted], liability],
             )
         if selector == _SELECTOR_UNIT_OF_ACCOUNT:
             return encode(["address"], [unit])
@@ -1241,6 +1352,19 @@ class TestEulerV2Health:
         assert result.total_collateral_usd is None
         assert result.total_debt_usd is None
 
+    @pytest.mark.parametrize("adjusted", [0, EULER_ADJUSTED_COLLATERAL])
+    def test_debt_valued_at_zero_is_unreadable_not_no_debt(self, adjusted):
+        result = _compute_euler_v2_account_health(
+            _euler_ctx(adjusted=adjusted, liability=0),
+            EULER_ACCOUNT,
+            MarketId(11),
+            "EULER_V2",
+        )
+
+        assert isinstance(result, UnreadableLendingMarket)
+        assert result.substrate_id == EULER_SUB_ACCOUNT
+        assert result.reason == "debt is valued at zero"
+
     def test_failed_debt_read_is_unreadable(self):
         ctx = MagicMock()
         ctx.call.side_effect = Web3RPCError("rpc down")

{ipor_fusion-4.2.0.dev18 → ipor_fusion-4.2.0.dev23}/tests/test_logs.py

@@ -1,5 +1,6 @@
 """Adaptive eth_getLogs paging (`core.logs`) against a fake provider."""
 
+import socket
 import threading
 from collections.abc import Callable
 from typing import Any
@@ -366,17 +367,40 @@ class TestNoPartialResults:
         with pytest.raises(KeyError):
             get_logs_adaptive(_web3(eth), ADDRESS, TOPICS, chunk_hint=100)
 
-    def test_error_names_no_provider_url(self):
-        provider_url = "https://rpc.example/v2/SECRETKEY"
-        eth = _FakeEth(
-            reject=lambda _l, _h: Web3RPCError(f"bad for url: {provider_url}")
-        )
+    @pytest.mark.parametrize(
+        "quoted",
+        ["https://rpc.example/v2/SECRETKEY", "/v2/SECRETKEY?apikey=SECRETKEY"],
+    )
+    def test_error_names_no_provider_url(self, quoted):
+        eth = _FakeEth(reject=lambda _l, _h: Web3RPCError(f"bad for url: {quoted}"))
 
         with pytest.raises(LogScanError) as info:
             get_logs_adaptive(_web3(eth), ADDRESS, TOPICS, 999, 1_000)
 
         assert "SECRETKEY" not in str(info.value)
 
+    def test_refused_connection_names_no_request_target(self, monkeypatch):
+        monkeypatch.setattr(logs_mod, "_TRANSIENT_RETRIES", 0)
+        with socket.socket() as probe:
+            probe.bind(("127.0.0.1", 0))
+            port = probe.getsockname()[1]
+
+        def reject(_low: int, _high: int) -> Exception | None:
+            try:
+                requests.post(
+                    f"http://127.0.0.1:{port}/v2/SECRETKEY?apikey=SECRETKEY", timeout=1
+                )
+            except requests.ConnectionError as exc:
+                return exc
+            return None
+
+        with pytest.raises(LogScanError) as info:
+            get_logs_adaptive(
+                _web3(_FakeEth(reject=reject)), ADDRESS, TOPICS, 999, 1_000
+            )
+
+        assert "SECRETKEY" not in str(info.value)
+
     def test_is_an_ipor_fusion_error(self):
         assert issubclass(LogScanError, IporFusionError)
 

{ipor_fusion-4.2.0.dev18 → ipor_fusion-4.2.0.dev23}/tests/test_mcp_models.py

@@ -33,6 +33,7 @@ from ipor_fusion import (
     ExecutePreflightJson,
     Fee,
     FuseAddress,
+    IporFusionMarkets,
     MarketId,
     MetaMorphoV1Json,
     MetaMorphoV2Json,
@@ -72,6 +73,12 @@ from ipor_fusion.mcp.models import (
     VaultValidationResponse,
     WithdrawManagerDetails,
 )
+from ipor_fusion.readers.lending_health import (
+    LendingMarketHealth as LendingMarketHealthRow,
+)
+from ipor_fusion.readers.lending_health import (
+    VaultLendingHealth,
+)
 from ipor_fusion.readers.oracle_mapping import (
     OracleAsset,
     OracleMapping,
@@ -79,6 +86,7 @@ from ipor_fusion.readers.oracle_mapping import (
     OraclePrice,
 )
 from ipor_fusion.types import Period, RoleId
+from ipor_fusion.vault_info import _lending_health_json
 
 
 def _amount(raw: int = 0, formatted: str = "0") -> AmountJson:
@@ -1307,3 +1315,26 @@ class TestModelsImportGraph:
             }
         )
         assert offending == []
+
+
+def test_debt_without_ltv_passes_through_the_published_model() -> None:
+    row = LendingMarketHealthRow(
+        protocol="morpho",
+        market_id=IporFusionMarkets.MORPHO,
+        market_name="MORPHO",
+        current_ltv=None,
+        max_ltv=0.86,
+        health_factor=0.0,
+        total_collateral_usd=None,
+        total_debt_usd=None,
+        ltv_usage_percent=None,
+        substrate_id="0x" + "ab" * 32,
+    )
+
+    document = _lending_health_json(VaultLendingHealth(markets=[row]))
+    model = mcp_models.LendingHealth.model_validate(document)
+
+    market = model.markets[0]
+    assert market.health_factor == 0.0
+    assert market.current_ltv is None
+    assert market.is_critical