oed-cli 0.1.1__tar.gz → 0.1.2__tar.gz

{oed_cli-0.1.1/src/oed_cli.egg-info → oed_cli-0.1.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: oed-cli
-Version: 0.1.1
+Version: 0.1.2
 Summary: oed — openEuler Infra command line. Auto-discovered, AI-friendly.
 Author: oed-cli contributors
 License: Apache-2.0
@@ -97,7 +97,7 @@ pip install -e .
 Verify:
 
 ```bash
-oed --version       # → oed, version 0.2.0
+oed --version       # → oed, version 0.1.2
 ```
 
 ---

{oed_cli-0.1.1 → oed_cli-0.1.2}/README.md

@@ -63,7 +63,7 @@ pip install -e .
 Verify:
 
 ```bash
-oed --version       # → oed, version 0.2.0
+oed --version       # → oed, version 0.1.2
 ```
 
 ---

{oed_cli-0.1.1 → oed_cli-0.1.2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "oed-cli"
-version = "0.1.1"
+version = "0.1.2"
 description = "oed — openEuler Infra command line. Auto-discovered, AI-friendly."
 readme = "README.md"
 requires-python = ">=3.10"

{oed_cli-0.1.1 → oed_cli-0.1.2}/src/oed_cli/__init__.py

@@ -2,5 +2,5 @@
 
 from __future__ import annotations
 
-__version__ = "0.2.0"
+__version__ = "0.1.2"
 __all__ = ["__version__"]

{oed_cli-0.1.1 → oed_cli-0.1.2}/src/oed_cli/invoke.py

@@ -34,6 +34,12 @@ def _fill_path(template: str, params: dict[str, Any]) -> tuple[str, list[str]]:
 
     Returns the rendered path and the list of placeholder names that were
     not provided, so the caller can raise a precise error.
+
+    Huawei APIG marks required path params with a trailing ``+`` in the
+    template (``/t/{id+}``); the ``+`` is a spec-side marker that never
+    appears in the real request URL or in the declared
+    ``parameters[].name``. Strip it so the lookup matches what the
+    caller actually provided under ``--<flag>`` / ``--params``.
     """
 
     missing: list[str] = []
@@ -48,13 +54,14 @@ def _fill_path(template: str, params: dict[str, Any]) -> tuple[str, list[str]]:
         if name_end == -1:
             parts.append(head + sep + chunk)
             break
-        name = chunk[:name_end]
+        raw_name = chunk[:name_end]
+        name = raw_name.rstrip("+")
         parts.append(head)
         if name in params:
             parts.append(str(params[name]))
         else:
             missing.append(name)
-            parts.append("{" + name + "}")
+            parts.append("{" + raw_name + "}")
         rest = chunk[name_end + 1 :]
     return "".join(parts), missing
 

{oed_cli-0.1.1 → oed_cli-0.1.2/src/oed_cli.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: oed-cli
-Version: 0.1.1
+Version: 0.1.2
 Summary: oed — openEuler Infra command line. Auto-discovered, AI-friendly.
 Author: oed-cli contributors
 License: Apache-2.0
@@ -97,7 +97,7 @@ pip install -e .
 Verify:
 
 ```bash
-oed --version       # → oed, version 0.2.0
+oed --version       # → oed, version 0.1.2
 ```
 
 ---

{oed_cli-0.1.1 → oed_cli-0.1.2}/tests/test_dynamic.py

@@ -526,6 +526,49 @@ def test_call_operation_missing_path_param_raises_user_error(patched):
         call_operation(op)  # no id
 
 
+def test_call_operation_strips_trailing_plus_in_path_template(patched):
+    """Huawei APIG marks required path params with ``{name+}`` in the path
+    template (``/t/{id+}``). The declared ``parameters[].name`` is plain
+    ``id`` — the ``+`` is a spec-side marker. ``oed forum getTopic --id 19308``
+    must therefore resolve the user's ``--id`` value into the rendered URL,
+    not complain that ``id+`` is missing."""
+
+    from oed_cli.dynamic import operations_table
+    from oed_cli.invoke import call_operation
+
+    spec = {
+        "openapi": "3.0.1",
+        "paths": {
+            "/t/{id+}": {
+                "get": {
+                    "operationId": "getTopic",
+                    "parameters": [
+                        {"in": "path", "name": "id",
+                         "required": True, "schema": {"type": "string"}},
+                    ],
+                }
+            }
+        },
+    }
+    table = operations_table(spec, "forum", base_url="https://apig.osinfra.cn")
+    op = table["getTopic"]
+
+    payload = call_operation(op, params={"id": "19308"})
+    assert patched["captured"]["url"].endswith("/t/19308")
+    assert payload["ok"] is True
+    assert payload["status"] == 200
+
+    # Missing-param error must use the clean name (what the user types),
+    # not the APIG-side ``id+`` form.
+    from oed_cli.errors import UserError
+    try:
+        call_operation(op)
+    except UserError as exc:
+        assert exc.to_dict()["message"] == "missing path params: ['id']"
+    else:
+        pytest.fail("expected UserError for missing path param")
+
+
 # ---------- main.py dispatch ----------