portolan-python 0.1.6__tar.gz → 0.1.8__tar.gz

{portolan_python-0.1.6 → portolan_python-0.1.8}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: portolan-python
-Version: 0.1.6
+Version: 0.1.8
 Summary: A lightweight Python implementation of the Portolan specification.
 Author: Portolan contributors
 License: Apache-2.0

{portolan_python-0.1.6 → portolan_python-0.1.8}/docs/distribution.md

@@ -8,8 +8,8 @@ preferred package index for published versions.
 Push a tag that matches the version in `pyproject.toml`:
 
 ```bash
-git tag v0.1.6
-git push origin v0.1.6
+git tag v0.1.8
+git push origin v0.1.8
 ```
 
 The release workflow runs the tests, builds both distributions, checks their
@@ -25,7 +25,7 @@ Install the wheel attached to a release:
 
 ```bash
 python -m pip install \
-  https://github.com/jemacchi/portolan-python/releases/download/v0.1.6/portolan_python-0.1.6-py3-none-any.whl
+  https://github.com/jemacchi/portolan-python/releases/download/v0.1.8/portolan_python-0.1.8-py3-none-any.whl
 ```
 
 This path needs no package index account.
@@ -35,7 +35,7 @@ This path needs no package index account.
 Install a published version from PyPI with:
 
 ```bash
-python -m pip install portolan-python==0.1.6
+python -m pip install portolan-python==0.1.8
 ```
 
 Omit the version constraint to install the latest published version.

{portolan_python-0.1.6 → portolan_python-0.1.8}/docs/examples.md

@@ -10,14 +10,14 @@ GitHub:
 
 ```bash
 python -m pip install \
-  https://github.com/jemacchi/portolan-python/releases/download/v0.1.6/portolan_python-0.1.6-py3-none-any.whl
+  https://github.com/jemacchi/portolan-python/releases/download/v0.1.8/portolan_python-0.1.8-py3-none-any.whl
 ```
 
 You can also install the source at a specific tag:
 
 ```bash
 python -m pip install \
-  "portolan-python @ git+https://github.com/jemacchi/portolan-python.git@v0.1.6"
+  "portolan-python @ git+https://github.com/jemacchi/portolan-python.git@v0.1.8"
 ```
 
 ## List collections and assets

{portolan_python-0.1.6 → portolan_python-0.1.8}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "portolan-python"
-version = "0.1.6"
+version = "0.1.8"
 description = "A lightweight Python implementation of the Portolan specification."
 readme = "README.md"
 requires-python = ">=3.10"

{portolan_python-0.1.6 → portolan_python-0.1.8}/src/portolan/registry.py

@@ -3,6 +3,7 @@
 from __future__ import annotations
 
 import json
+import math
 import os
 import shutil
 import tempfile
@@ -31,6 +32,14 @@ class RegistryCatalogEntry:
     url: str
     title: str | None = None
     status: str | None = None
+    bbox: tuple[float, float, float, float] | None = None
+    licenses: tuple[str, ...] = ()
+    collection_count: int | None = None
+    feature_count: int | None = None
+    total_size_bytes: int | None = None
+    updated: str | None = None
+    logo_url: str | None = None
+    failure_reason: str | None = None
 
 
 def load_registry_entries(
@@ -52,18 +61,33 @@ def load_registry_entries(
         registry_id = link.get("portolan_registry:id")
         if not isinstance(href, str) or not isinstance(registry_id, str):
             continue
+        resolved_url = urljoin(registry_url, href)
+        if not _is_remote_url(resolved_url):
+            continue
         status = link.get("portolan_registry:status")
         if status != "valid" and not include_stale:
             continue
         if catalog_ids is not None and registry_id not in catalog_ids:
             continue
         title = link.get("title")
+        licenses = link.get("portolan_registry:licenses")
+        logo = link.get("portolan_registry:logo")
         entries.append(
             RegistryCatalogEntry(
                 id=registry_id,
-                url=urljoin(registry_url, href),
+                url=resolved_url,
                 title=title if isinstance(title, str) else None,
                 status=status if isinstance(status, str) else None,
+                bbox=_horizontal_bbox(link.get("bbox")),
+                licenses=tuple(sorted(key for key in licenses if isinstance(key, str)))
+                if isinstance(licenses, dict)
+                else (),
+                collection_count=_optional_int(link.get("portolan_registry:collection_count")),
+                feature_count=_optional_int(link.get("portolan_registry:feature_count")),
+                total_size_bytes=_optional_int(link.get("portolan_registry:total_size_bytes")),
+                updated=_optional_str(link.get("portolan_registry:updated")),
+                logo_url=_optional_str(logo.get("href")) if isinstance(logo, dict) else None,
+                failure_reason=_optional_str(link.get("portolan_registry:failure_reason")),
             )
         )
         if limit is not None and len(entries) >= limit:
@@ -71,6 +95,33 @@ def load_registry_entries(
     return entries
 
 
+def _optional_int(value: object) -> int | None:
+    return value if isinstance(value, int) and not isinstance(value, bool) else None
+
+
+def _optional_str(value: object) -> str | None:
+    return value if isinstance(value, str) and value else None
+
+
+def _horizontal_bbox(value: object) -> tuple[float, float, float, float] | None:
+    if not isinstance(value, (list, tuple)) or len(value) < 4 or len(value) % 2:
+        return None
+    if not all(
+        isinstance(coordinate, (int, float))
+        and not isinstance(coordinate, bool)
+        and math.isfinite(coordinate)
+        for coordinate in value
+    ):
+        return None
+    half = len(value) // 2
+    return (float(value[0]), float(value[1]), float(value[half]), float(value[half + 1]))
+
+
+def _is_remote_url(url: str) -> bool:
+    parsed = urlparse(url)
+    return parsed.scheme in {"http", "https"} and bool(parsed.netloc)
+
+
 def download_registry_catalog(
     catalog_url: str,
     output_dir: Path,

{portolan_python-0.1.6 → portolan_python-0.1.8}/tests/test_registry_api.py

@@ -40,8 +40,19 @@ def test_load_registry_entries_filters_to_valid_children() -> None:
                 "rel": "child",
                 "href": "https://example.test/a/catalog.json",
                 "title": "A",
+                "bbox": [-71.0, -35.0, -70.0, -34.0],
                 "portolan_registry:id": "catalog-a",
                 "portolan_registry:status": "valid",
+                "portolan_registry:licenses": {"ODbL-1.0": 2, "CC-BY-4.0": 1},
+                "portolan_registry:collection_count": 3,
+                "portolan_registry:feature_count": 42,
+                "portolan_registry:total_size_bytes": 2048,
+                "portolan_registry:updated": "2026-10-08T00:00:00Z",
+                "portolan_registry:logo": {
+                    "href": "https://example.test/a/logo.png",
+                    "type": "image/png",
+                },
+                "portolan_registry:failure_reason": None,
             },
             {
                 "rel": "child",
@@ -59,6 +70,48 @@ def test_load_registry_entries_filters_to_valid_children() -> None:
     assert [(entry.id, entry.url, entry.title, entry.status) for entry in entries] == [
         ("catalog-a", "https://example.test/a/catalog.json", "A", "valid")
     ]
+    assert entries[0].bbox == (-71.0, -35.0, -70.0, -34.0)
+    assert entries[0].licenses == ("CC-BY-4.0", "ODbL-1.0")
+    assert entries[0].collection_count == 3
+    assert entries[0].feature_count == 42
+    assert entries[0].total_size_bytes == 2048
+    assert entries[0].updated == "2026-10-08T00:00:00Z"
+    assert entries[0].logo_url == "https://example.test/a/logo.png"
+    assert entries[0].failure_reason is None
+
+
+def test_load_registry_entries_ignores_malformed_optional_metadata() -> None:
+    registry = {
+        "links": [
+            {
+                "rel": "child",
+                "href": "https://example.test/a/catalog.json",
+                "bbox": [True, -35.0, -70.0, -34.0],
+                "portolan_registry:id": "catalog-a",
+                "portolan_registry:status": "valid",
+                "portolan_registry:licenses": ["not", "a", "mapping"],
+                "portolan_registry:collection_count": True,
+                "portolan_registry:feature_count": "many",
+                "portolan_registry:total_size_bytes": 1.5,
+                "portolan_registry:updated": 7,
+                "portolan_registry:logo": {"href": 9},
+                "portolan_registry:failure_reason": [],
+            }
+        ]
+    }
+
+    (entry,) = load_registry_entries(
+        "https://registry.test/catalogs.json", fetch_json=lambda url: registry
+    )
+
+    assert entry.bbox is None
+    assert entry.licenses == ()
+    assert entry.collection_count is None
+    assert entry.feature_count is None
+    assert entry.total_size_bytes is None
+    assert entry.updated is None
+    assert entry.logo_url is None
+    assert entry.failure_reason is None
 
 
 def test_load_registry_entries_resolves_relative_child_urls() -> None:
@@ -125,6 +178,12 @@ def test_load_registry_entries_skips_invalid_link_shapes() -> None:
             {"rel": "item", "href": "https://example.test/item.json"},
             {"rel": "child", "href": "https://example.test/missing-id/catalog.json"},
             {"rel": "child", "href": 5, "portolan_registry:id": "bad-href"},
+            {
+                "rel": "child",
+                "href": "javascript:alert(1)",
+                "portolan_registry:id": "unsafe-scheme",
+                "portolan_registry:status": "valid",
+            },
             {
                 "rel": "child",
                 "href": "https://example.test/no-status/catalog.json",

{portolan_python-0.1.6 → portolan_python-0.1.8}/uv.lock

@@ -468,7 +468,7 @@ wheels = [
 
 [[package]]
 name = "portolan-python"
-version = "0.1.6"
+version = "0.1.8"
 source = { editable = "." }
 
 [package.optional-dependencies]