poelis-sdk 0.3.2__tar.gz → 0.3.3__tar.gz

{poelis_sdk-0.3.2 → poelis_sdk-0.3.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: poelis-sdk
-Version: 0.3.2
+Version: 0.3.3
 Summary: Official Python SDK for Poelis
 Project-URL: Homepage, https://poelis.com
 Project-URL: Source, https://github.com/PoelisTechnologies/poelis-python-sdk

{poelis_sdk-0.3.2 → poelis_sdk-0.3.3}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "poelis-sdk"
-version = "0.3.2"
+version = "0.3.3"
 description = "Official Python SDK for Poelis"
 readme = "README.md"
 requires-python = ">=3.11"

{poelis_sdk-0.3.2 → poelis_sdk-0.3.3}/src/poelis_sdk/browser.py

@@ -140,39 +140,77 @@ class _Node:
             self._props_loaded_at = time.time()
             return self._props_cache
         # Try direct properties(itemId: ...) first; fallback to searchProperties
-        q = (
+        # Attempt 1: query with parsedValue support
+        q_parsed = (
             "query($iid: ID!) {\n"
             "  properties(itemId: $iid) {\n"
             "    __typename\n"
-            "    ... on NumericProperty { integerPart exponent category }\n"
-            "    ... on TextProperty { value }\n"
+            "    ... on NumericProperty { category value parsedValue }\n"
+            "    ... on TextProperty { value parsedValue }\n"
             "    ... on DateProperty { value }\n"
             "  }\n"
             "}"
         )
         try:
-            r = self._client._transport.graphql(q, {"iid": self._id})
+            r = self._client._transport.graphql(q_parsed, {"iid": self._id})
             r.raise_for_status()
             data = r.json()
             if "errors" in data:
-                raise RuntimeError(data["errors"])  # trigger fallback
+                raise RuntimeError(data["errors"])  # try value-only shape
             self._props_cache = data.get("data", {}).get("properties", []) or []
             self._props_loaded_at = time.time()
         except Exception:
-            q2 = (
-                "query($iid: ID!, $limit: Int!, $offset: Int!) {\n"
-                "  searchProperties(q: \"*\", itemId: $iid, limit: $limit, offset: $offset) {\n"
-                "    hits { id workspaceId productId itemId propertyType name category value owner }\n"
+            # Attempt 2: value-only, legacy compatible
+            q_value_only = (
+                "query($iid: ID!) {\n"
+                "  properties(itemId: $iid) {\n"
+                "    __typename\n"
+                "    ... on NumericProperty { category value }\n"
+                "    ... on TextProperty { value }\n"
+                "    ... on DateProperty { value }\n"
                 "  }\n"
                 "}"
             )
-            r2 = self._client._transport.graphql(q2, {"iid": self._id, "limit": 100, "offset": 0})
-            r2.raise_for_status()
-            data2 = r2.json()
-            if "errors" in data2:
-                raise RuntimeError(data2["errors"])  # propagate
-            self._props_cache = data2.get("data", {}).get("searchProperties", {}).get("hits", []) or []
-            self._props_loaded_at = time.time()
+            try:
+                r = self._client._transport.graphql(q_value_only, {"iid": self._id})
+                r.raise_for_status()
+                data = r.json()
+                if "errors" in data:
+                    raise RuntimeError(data["errors"])  # trigger fallback to search
+                self._props_cache = data.get("data", {}).get("properties", []) or []
+                self._props_loaded_at = time.time()
+            except Exception:
+                # Fallback to searchProperties
+                q2_parsed = (
+                    "query($iid: ID!, $limit: Int!, $offset: Int!) {\n"
+                    "  searchProperties(q: \"*\", itemId: $iid, limit: $limit, offset: $offset) {\n"
+                    "    hits { id workspaceId productId itemId propertyType name category value parsedValue owner }\n"
+                    "  }\n"
+                    "}"
+                )
+                try:
+                    r2 = self._client._transport.graphql(q2_parsed, {"iid": self._id, "limit": 100, "offset": 0})
+                    r2.raise_for_status()
+                    data2 = r2.json()
+                    if "errors" in data2:
+                        raise RuntimeError(data2["errors"])  # try minimal
+                    self._props_cache = data2.get("data", {}).get("searchProperties", {}).get("hits", []) or []
+                    self._props_loaded_at = time.time()
+                except Exception:
+                    q2_min = (
+                        "query($iid: ID!, $limit: Int!, $offset: Int!) {\n"
+                        "  searchProperties(q: \"*\", itemId: $iid, limit: $limit, offset: $offset) {\n"
+                        "    hits { id workspaceId productId itemId propertyType name category value owner }\n"
+                        "  }\n"
+                        "}"
+                    )
+                    r3 = self._client._transport.graphql(q2_min, {"iid": self._id, "limit": 100, "offset": 0})
+                    r3.raise_for_status()
+                    data3 = r3.json()
+                    if "errors" in data3:
+                        raise RuntimeError(data3["errors"])  # propagate
+                    self._props_cache = data3.get("data", {}).get("searchProperties", {}).get("hits", []) or []
+                    self._props_loaded_at = time.time()
         return self._props_cache
 
     def _props_key_map(self) -> Dict[str, Dict[str, Any]]:
@@ -419,6 +457,10 @@ class _PropWrapper:
     @property
     def value(self) -> Any:  # type: ignore[override]
         p = self._raw
+        # Use parsedValue if available (new backend feature)
+        if "parsedValue" in p:
+            return p["parsedValue"]
+        # Fallback to legacy parsing logic for backward compatibility
         # searchProperties shape
         if "numericValue" in p and p.get("numericValue") is not None:
             return p["numericValue"]

poelis_sdk-0.3.3/src/poelis_sdk/models.py

@@ -0,0 +1,130 @@
+from __future__ import annotations
+
+from typing import Optional, Any, Union, List
+
+from pydantic import BaseModel, Field
+
+"""Pydantic models for SDK resources."""
+
+
+class Product(BaseModel):
+    """Product resource representation."""
+
+    id: str = Field(min_length=1)
+    name: str = Field(min_length=1)
+    workspaceId: Optional[str] = None
+    code: Optional[str] = None
+    description: Optional[str] = None
+
+
+class PaginatedProducts(BaseModel):
+    """Paginated response for products list."""
+
+    data: list[Product]
+    limit: int
+    offset: int
+
+
+class PropertyValue(BaseModel):
+    """Base class for property values with typed access."""
+    
+    raw_value: str = Field(alias="value")
+    parsed_value: Optional[Any] = Field(alias="parsedValue", default=None)
+    
+    @property
+    def value(self) -> Any:
+        """Get the properly typed value, falling back to raw string if parsing failed."""
+        return self.parsed_value if self.parsed_value is not None else self.raw_value
+
+
+class NumericProperty(BaseModel):
+    """Numeric property representation."""
+    
+    id: str = Field(min_length=1)
+    item_id: str = Field(alias="itemId", min_length=1)
+    position: float
+    name: str = Field(min_length=1)
+    value: str
+    category: str
+    display_unit: Optional[str] = Field(alias="displayUnit", default=None)
+    owner: str = Field(min_length=1)
+    type: str = Field(min_length=1)
+    parsed_value: Optional[Union[int, float, List[Any], str]] = Field(alias="parsedValue", default=None)
+    
+    @property
+    def typed_value(self) -> Union[int, float, List[Any], str]:
+        """Get the properly typed value, falling back to raw string if parsing failed."""
+        return self.parsed_value if self.parsed_value is not None else self.value
+
+
+class TextProperty(BaseModel):
+    """Text property representation."""
+    
+    id: str = Field(min_length=1)
+    item_id: str = Field(alias="itemId", min_length=1)
+    position: float
+    name: str = Field(min_length=1)
+    value: str
+    owner: str = Field(min_length=1)
+    type: str = Field(min_length=1)
+    parsed_value: Optional[Union[int, float, List[Any], str]] = Field(alias="parsedValue", default=None)
+    
+    @property
+    def typed_value(self) -> Union[int, float, List[Any], str]:
+        """Get the properly typed value, falling back to raw string if parsing failed."""
+        return self.parsed_value if self.parsed_value is not None else self.value
+
+
+class DateProperty(BaseModel):
+    """Date property representation."""
+    
+    id: str = Field(min_length=1)
+    item_id: str = Field(alias="itemId", min_length=1)
+    position: float
+    name: str = Field(min_length=1)
+    value: str
+    owner: str = Field(min_length=1)
+    type: str = Field(min_length=1)
+    parsed_value: Optional[str] = Field(alias="parsedValue", default=None)
+    
+    @property
+    def typed_value(self) -> str:
+        """Get the properly typed value, falling back to raw string if parsing failed."""
+        return self.parsed_value if self.parsed_value is not None else self.value
+
+
+class PropertySearchResult(BaseModel):
+    """Property search result with unified fields across all property types."""
+    
+    id: str = Field(min_length=1)
+    workspace_id: str = Field(alias="workspaceId", min_length=1)
+    product_id: str = Field(alias="productId", min_length=1)
+    item_id: str = Field(alias="itemId", min_length=1)
+    property_type: str = Field(alias="propertyType", min_length=1)
+    name: str = Field(min_length=1)
+    category: Optional[str] = None
+    display_unit: Optional[str] = Field(alias="displayUnit", default=None)
+    value: Any  # Raw value from GraphQL
+    parsed_value: Optional[Union[int, float, List[Any], str]] = Field(alias="parsedValue", default=None)
+    owner: str = Field(min_length=1)
+    created_by: str = Field(alias="createdBy", min_length=1)
+    created_at: str = Field(alias="createdAt", min_length=1)
+    updated_at: str = Field(alias="updatedAt", min_length=1)
+    
+    @property
+    def typed_value(self) -> Union[int, float, List[Any], str]:
+        """Get the properly typed value, falling back to raw string if parsing failed."""
+        return self.parsed_value if self.parsed_value is not None else self.value
+
+
+class PropertySearchResponse(BaseModel):
+    """Response for property search queries."""
+    
+    query: str
+    hits: List[PropertySearchResult]
+    total: int
+    limit: int
+    offset: int
+    processing_time_ms: int = Field(alias="processingTimeMs")
+
+

{poelis_sdk-0.3.2 → poelis_sdk-0.3.3}/src/poelis_sdk/search.py

@@ -54,7 +54,7 @@ class SearchClient:
             "query($q: String!, $ws: ID, $pid: ID, $iid: ID, $ptype: String, $cat: String, $limit: Int!, $offset: Int!, $sort: String) {\n"
             "  searchProperties(q: $q, workspaceId: $ws, productId: $pid, itemId: $iid, propertyType: $ptype, category: $cat, limit: $limit, offset: $offset, sort: $sort) {\n"
             "    query total limit offset processingTimeMs\n"
-            "    hits { id workspaceId productId itemId propertyType name category value owner }\n"
+            "    hits { id workspaceId productId itemId propertyType name category value parsedValue owner }\n"
             "  }\n"
             "}"
         )

poelis_sdk-0.3.3/tests/test_typed_properties.py

@@ -0,0 +1,212 @@
+"""Tests for typed property values functionality."""
+
+from __future__ import annotations
+
+from typing import TYPE_CHECKING
+
+from poelis_sdk.models import NumericProperty, TextProperty, DateProperty, PropertySearchResult
+from poelis_sdk.browser import _PropWrapper
+
+if TYPE_CHECKING:
+    from _pytest.capture import CaptureFixture  # noqa: F401
+    from _pytest.fixtures import FixtureRequest  # noqa: F401
+    from _pytest.logging import LogCaptureFixture  # noqa: F401
+    from _pytest.monkeypatch import MonkeyPatch  # noqa: F401
+    from pytest_mock.plugin import MockerFixture  # noqa: F401
+
+
+class TestTypedPropertyModels:
+    """Test Pydantic models for typed property values."""
+
+    def test_numeric_property_with_parsed_value(self) -> None:
+        """Test NumericProperty model with parsedValue field."""
+        prop_data = {
+            "id": "prop1",
+            "itemId": "item1",
+            "position": 1.0,
+            "name": "Weight",
+            "value": "42.5",
+            "category": "Physical",
+            "displayUnit": "kg",
+            "owner": "user1",
+            "type": "numeric",
+            "parsedValue": 42.5
+        }
+        
+        prop = NumericProperty(**prop_data)
+        assert prop.typed_value == 42.5
+        assert isinstance(prop.typed_value, float)
+        assert prop.value == "42.5"  # Raw value remains string
+
+    def test_numeric_property_without_parsed_value(self) -> None:
+        """Test NumericProperty model without parsedValue field (fallback)."""
+        prop_data = {
+            "id": "prop1",
+            "itemId": "item1",
+            "position": 1.0,
+            "name": "Weight",
+            "value": "42.5",
+            "category": "Physical",
+            "displayUnit": "kg",
+            "owner": "user1",
+            "type": "numeric"
+        }
+        
+        prop = NumericProperty(**prop_data)
+        assert prop.typed_value == "42.5"  # Falls back to raw string
+        assert prop.value == "42.5"
+
+    def test_text_property_with_parsed_array(self) -> None:
+        """Test TextProperty model with parsed array value."""
+        prop_data = {
+            "id": "prop2",
+            "itemId": "item1",
+            "position": 2.0,
+            "name": "Tags",
+            "value": '["tag1", "tag2", "tag3"]',
+            "owner": "user1",
+            "type": "text",
+            "parsedValue": ["tag1", "tag2", "tag3"]
+        }
+        
+        prop = TextProperty(**prop_data)
+        assert prop.typed_value == ["tag1", "tag2", "tag3"]
+        assert isinstance(prop.typed_value, list)
+        assert prop.value == '["tag1", "tag2", "tag3"]'  # Raw value remains string
+
+    def test_text_property_with_parsed_integer(self) -> None:
+        """Test TextProperty model with parsed integer value."""
+        prop_data = {
+            "id": "prop3",
+            "itemId": "item1",
+            "position": 3.0,
+            "name": "Count",
+            "value": "100",
+            "owner": "user1",
+            "type": "text",
+            "parsedValue": 100
+        }
+        
+        prop = TextProperty(**prop_data)
+        assert prop.typed_value == 100
+        assert isinstance(prop.typed_value, int)
+        assert prop.value == "100"  # Raw value remains string
+
+    def test_date_property_with_parsed_value(self) -> None:
+        """Test DateProperty model with parsedValue field."""
+        prop_data = {
+            "id": "prop4",
+            "itemId": "item1",
+            "position": 4.0,
+            "name": "Created Date",
+            "value": "2024-01-15",
+            "owner": "user1",
+            "type": "date",
+            "parsedValue": "2024-01-15"
+        }
+        
+        prop = DateProperty(**prop_data)
+        assert prop.typed_value == "2024-01-15"
+        assert isinstance(prop.typed_value, str)
+        assert prop.value == "2024-01-15"
+
+    def test_property_search_result_with_parsed_value(self) -> None:
+        """Test PropertySearchResult model with parsedValue field."""
+        result_data = {
+            "id": "prop5",
+            "workspaceId": "ws1",
+            "productId": "prod1",
+            "itemId": "item1",
+            "propertyType": "numeric",
+            "name": "Temperature",
+            "category": "Environmental",
+            "displayUnit": "°C",
+            "value": "25.3",
+            "parsedValue": 25.3,
+            "owner": "user1",
+            "createdBy": "user1",
+            "createdAt": "2024-01-15T10:00:00Z",
+            "updatedAt": "2024-01-15T10:00:00Z"
+        }
+        
+        result = PropertySearchResult(**result_data)
+        assert result.typed_value == 25.3
+        assert isinstance(result.typed_value, float)
+        assert result.value == "25.3"  # Raw value remains string
+
+
+class TestPropWrapper:
+    """Test _PropWrapper class with typed values."""
+
+    def test_prop_wrapper_with_parsed_value(self) -> None:
+        """Test _PropWrapper with parsedValue field."""
+        prop_data = {
+            "id": "prop1",
+            "name": "Weight",
+            "value": "42.5",
+            "parsedValue": 42.5,
+            "category": "Physical"
+        }
+        
+        wrapper = _PropWrapper(prop_data)
+        assert wrapper.value == 42.5
+        assert isinstance(wrapper.value, float)
+        assert wrapper.category == "Physical"
+
+    def test_prop_wrapper_without_parsed_value_fallback(self) -> None:
+        """Test _PropWrapper fallback when parsedValue is not available."""
+        prop_data = {
+            "id": "prop1",
+            "name": "Weight",
+            "value": "42.5",
+            "category": "Physical"
+        }
+        
+        wrapper = _PropWrapper(prop_data)
+        assert wrapper.value == "42.5"  # Falls back to raw value
+        assert isinstance(wrapper.value, str)
+        assert wrapper.category == "Physical"
+
+    def test_prop_wrapper_with_parsed_array(self) -> None:
+        """Test _PropWrapper with parsed array value."""
+        prop_data = {
+            "id": "prop2",
+            "name": "Tags",
+            "value": '["tag1", "tag2"]',
+            "parsedValue": ["tag1", "tag2"],
+            "category": "Metadata"
+        }
+        
+        wrapper = _PropWrapper(prop_data)
+        assert wrapper.value == ["tag1", "tag2"]
+        assert isinstance(wrapper.value, list)
+        assert wrapper.category == "Metadata"
+
+    def test_prop_wrapper_legacy_numeric_parsing(self) -> None:
+        """Test _PropWrapper with legacy numeric parsing (integerPart/exponent)."""
+        prop_data = {
+            "id": "prop3",
+            "name": "Legacy Numeric",
+            "integerPart": 42,
+            "exponent": 1,
+            "category": "Legacy"
+        }
+        
+        wrapper = _PropWrapper(prop_data)
+        assert wrapper.value == 420  # 42 * 10^1
+        assert isinstance(wrapper.value, int)
+        assert wrapper.category == "Legacy"
+
+    def test_prop_wrapper_legacy_search_properties(self) -> None:
+        """Test _PropWrapper with legacy searchProperties format."""
+        prop_data = {
+            "id": "prop4",
+            "name": "Search Numeric",
+            "numericValue": 123.45,
+            "category": "Search"
+        }
+        
+        wrapper = _PropWrapper(prop_data)
+        assert wrapper.value == 123.45
+        assert isinstance(wrapper.value, float)
+        assert wrapper.category == "Search"

{poelis_sdk-0.3.2 → poelis_sdk-0.3.3}/uv.lock

@@ -538,7 +538,7 @@ wheels = [
 
 [[package]]
 name = "poelis-sdk"
-version = "0.3.2"
+version = "0.3.3"
 source = { editable = "." }
 dependencies = [
     { name = "build" },

poelis_sdk-0.3.2/src/poelis_sdk/models.py

@@ -1,27 +0,0 @@
-from __future__ import annotations
-
-from typing import Optional
-
-from pydantic import BaseModel, Field
-
-"""Pydantic models for SDK resources."""
-
-
-class Product(BaseModel):
-    """Product resource representation."""
-
-    id: str = Field(min_length=1)
-    name: str = Field(min_length=1)
-    workspaceId: Optional[str] = None
-    code: Optional[str] = None
-    description: Optional[str] = None
-
-
-class PaginatedProducts(BaseModel):
-    """Paginated response for products list."""
-
-    data: list[Product]
-    limit: int
-    offset: int
-
-