ipor-fusion 4.0.1.dev2__tar.gz → 4.0.1.dev5__tar.gz

{ipor_fusion-4.0.1.dev2 → ipor_fusion-4.0.1.dev5}/MIGRATION_4.md

@@ -1130,12 +1130,12 @@ are listed with a link.
 - **The CLI and the MCP tools take the same arguments.**
   - **MCP inputs:** the vault tools' `chain_id` and every `block_number` are
     optional and omitted means auto-detected / latest; `market_morpho_blue` and
-    `market_meta_morpho` still require a positive `chain_id`. The `0` sentinel is
-    gone: an optional `0` is rejected with an error naming the argument to omit (it
-    is never read as chain or block 0), so a client that passed `0` omits the
-    argument instead; the market tools' required `chain_id` rejects `0` too and
-    needs the real chain id. The CLI refuses `--block-number 0` and `--chain-id 0`
-    the same way. `server_info(changelog_since=)` is `server_info(since=)` as in
+    `market_meta_morpho` still require a positive `chain_id`. A `chain_id` of `0`
+    no longer means auto-detection: it is rejected with an error naming the
+    argument to omit, and the market tools' required `chain_id` needs the real
+    chain id. A `block_number` of `0` still means the latest block, like an
+    omitted one (genesis never holds a vault). The CLI follows the same rules:
+    `--chain-id 0` is refused, `--block-number 0` reads the latest block. `server_info(changelog_since=)` is `server_info(since=)` as in
     `fusion changelog --since`, and `vault_oracle_mapping` takes `max_depth` like
     `--max-depth`.
   - **Chain auto-detection:** the vault commands and tools use the saved vault's

{ipor_fusion-4.0.1.dev2 → ipor_fusion-4.0.1.dev5}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: ipor-fusion
-Version: 4.0.1.dev2
+Version: 4.0.1.dev5
 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.1.dev2 → ipor_fusion-4.0.1.dev5}/pyproject.toml

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

{ipor_fusion-4.0.1.dev2 → ipor_fusion-4.0.1.dev5}/src/ipor_fusion/cli/market_cmd.py

@@ -87,7 +87,7 @@ def market() -> None:
     "block_number",
     type=BLOCK_NUMBER,
     default=None,
-    help="Block number for on-chain reads (default: latest).",
+    help="Block number for on-chain reads (0 or omitted: latest).",
 )
 @click.option(
     "--no-api",

{ipor_fusion-4.0.1.dev2 → ipor_fusion-4.0.1.dev5}/src/ipor_fusion/cli/options.py

@@ -109,24 +109,18 @@ CHAIN = ChainType()
 
 
 class BlockNumberType(click.IntRange):
-    """A positive block number. 0 is refused rather than read as block 0:
-    some callers still pass it meaning "latest"."""
+    """A block number, 0 meaning the latest block like an omitted option:
+    callers fill optional integers with 0, and genesis never holds a vault."""
 
     name = "block"
 
     def __init__(self) -> None:
-        super().__init__(min=1)
+        super().__init__(min=0)
 
     def convert(
         self, value: object, param: click.Parameter | None, ctx: click.Context | None
-    ) -> int:
-        if value in (0, "0"):
-            self.fail(
-                "0 is not a block selector: omit --block-number for the latest block",
-                param,
-                ctx,
-            )
-        return super().convert(value, param, ctx)
+    ) -> int | None:
+        return super().convert(value, param, ctx) or None
 
 
 BLOCK_NUMBER = BlockNumberType()

{ipor_fusion-4.0.1.dev2 → ipor_fusion-4.0.1.dev5}/src/ipor_fusion/cli/preflight_cmd.py

@@ -53,7 +53,10 @@ def _read_calldata(calldata: str) -> str:
     "--chain-id", type=CHAIN, default=None, help="Chain ID or name (e.g. 1, ethereum)."
 )
 @click.option(
-    "--block-number", type=BLOCK_NUMBER, default=None, help="Block (default: latest)."
+    "--block-number",
+    type=BLOCK_NUMBER,
+    default=None,
+    help="Block (0 or omitted: latest).",
 )
 @click.option(
     "--simulate/--no-simulate",

{ipor_fusion-4.0.1.dev2 → ipor_fusion-4.0.1.dev5}/src/ipor_fusion/cli/vault_cmd.py

@@ -188,7 +188,7 @@ vault.add_command(list_vaults, "ls")
     "--block-number",
     type=BLOCK_NUMBER,
     default=None,
-    help="Block number (default: latest).",
+    help="Block number (0 or omitted: latest).",
 )
 @click.option(
     "--json", "json_output", is_flag=True, default=False, help="Output as JSON."
@@ -233,7 +233,7 @@ def info(
     "--block-number",
     type=BLOCK_NUMBER,
     default=None,
-    help="Block number (default: latest).",
+    help="Block number (0 or omitted: latest).",
 )
 @click.option(
     "--json", "json_output", is_flag=True, default=False, help="Output as JSON."
@@ -294,7 +294,7 @@ def validate(
     "--block-number",
     type=BLOCK_NUMBER,
     default=None,
-    help="Block number (default: latest).",
+    help="Block number (0 or omitted: latest).",
 )
 @click.option(
     "--json", "json_output", is_flag=True, default=False, help="Output as JSON."
@@ -355,7 +355,7 @@ def role_accounts(
     "--block-number",
     type=BLOCK_NUMBER,
     default=None,
-    help="Block number (default: latest).",
+    help="Block number (0 or omitted: latest).",
 )
 @click.option(
     "--max-depth",

{ipor_fusion-4.0.1.dev2 → ipor_fusion-4.0.1.dev5}/tests/test_cli_mcp_parity.py

@@ -166,18 +166,31 @@ def test_a_negative_block_is_rejected() -> None:
     load.assert_not_called()
 
 
+@pytest.mark.parametrize(("raw", "expected"), [("0", None), ("123", 123)])
+def test_cli_block_zero_reads_the_latest_block(raw: str, expected: int | None) -> None:
+    with patch(
+        "ipor_fusion.cli.vault_cmd._build_ctx",
+        autospec=True,
+        side_effect=RuntimeError("stop"),
+    ) as build_ctx:
+        CliRunner().invoke(
+            cli, ["vault", "info", _VAULT, "--chain-id", "1", "--block-number", raw]
+        )
+    assert build_ctx.call_args.args[3] == expected
+
+
 @pytest.mark.parametrize(
     "args",
     [
-        ["vault", "info", _VAULT, "--block-number", "0"],
-        ["vault", "validate", _VAULT, "--block-number", "0"],
-        ["market", "morpho-blue", _MARKET, "--chain", "1", "--block", "0"],
+        ["vault", "info", _VAULT, "--block-number", "-1"],
+        ["vault", "validate", _VAULT, "--block-number", "-1"],
+        ["market", "morpho-blue", _MARKET, "--chain", "1", "--block", "-1"],
     ],
 )
-def test_cli_refuses_block_zero(args: list[str]) -> None:
+def test_cli_rejects_a_negative_block(args: list[str]) -> None:
     result = CliRunner().invoke(cli, args)
     assert result.exit_code == 2
-    assert "omit --block-number" in result.output
+    assert "-1 is not in the range x>=0" in result.output
 
 
 def test_cli_refuses_chain_zero() -> None: