Fix use of license and provider in converter list (#11)

* Fix various lint/type checking issues (#10)
* Fix use of license and provider in converter list

Author: m-mohr

CHANGELOG.md

@@ -8,6 +8,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 ## [Unreleased]
 
 - Converters: `column_filters` allows to inverse the mask
+- Fix use of license and provider in converter list
 - Various small bug fixes and type hint fixes
 
 ## [v0.2.8] - 2025-09-13

tests/test_converters.py

@@ -35,7 +35,7 @@ def test_list_ids_other_module():
 def test_list_all():
     Registry.ignored_datasets = []
     c = Converters()
-    result = c.list_all()
+    result = c.list_all(("short_name", "license", "provider"))
     assert isinstance(result, dict)
     assert len(result.keys()) == 1
     assert "template" in result
@@ -44,6 +44,8 @@ def test_list_all():
     assert result["template"]["short_name"] == "Country, Region, etc."
     assert "license" in result["template"]
     assert result["template"]["license"] == "CC-BY-4.0"
+    assert "provider" in result["template"]
+    assert result["template"]["provider"] == "ABC Corp"
 
 
 def test_load():

vecorel_cli/converters.py

@@ -10,6 +10,7 @@
 from .cli.util import display_pandas_unrestricted
 from .conversion.base import BaseConverter
 from .registry import Registry
+from .vecorel.util import parse_link_str
 
 
 class Converters(BaseCommand):
@@ -59,7 +60,7 @@ def converters(self, providers=False, sources=False, verbose=False, py_package=N
 
         columns = {"short_name": "Short Title", "license": "License"}
         if providers:
-            columns["providers"] = "Provider(s)"
+            columns["provider"] = "Provider"
         if sources:
             columns["sources"] = "Source(s)"
 
@@ -71,7 +72,7 @@ def converters(self, providers=False, sources=False, verbose=False, py_package=N
         display_pandas_unrestricted(None if verbose else self.default_max_colwidth)
 
         if not df.empty:
-            self.info(df)
+            self.info(df.to_string())
         else:
             self.warning("No converters found.")
 
@@ -124,10 +125,8 @@ def list_all(self, keys: Sequence[str] = ("short_name", "license")) -> dict:
 
                     if key == "sources" and isinstance(value, dict):
                         value = ", ".join(list(value.keys()))
-                    elif key == "license" and isinstance(value, dict):
-                        value = value["href"]
-                    elif key == "providers" and isinstance(value, list):
-                        value = ", ".join(list(map(lambda x: x["name"], value)))
+                    elif key in ["license", "provider"] and isinstance(value, str):
+                        value, _ = parse_link_str(value)
 
                     obj[key] = value