openepd 7.1.0__py3-none-any.whl → 7.2.0__py3-none-any.whl

openepd/__version__.py

@@ -13,4 +13,4 @@
 #  See the License for the specific language governing permissions and
 #  limitations under the License.
 #
-VERSION = "7.1.0"
+VERSION = "7.2.0"

openepd/api/average_dataset/generic_estimate_sync_api.py

@@ -135,7 +135,8 @@ class GenericEstimateApi(BaseApiMethodGroup):
         """
         ge_id = ge.id
         if not ge_id:
-            raise ValueError("The ID must be set to edit a GenericEstimate.")
+            msg = "The ID must be set to edit a GenericEstimate."
+            raise ValueError(msg)
 
         response = self._client.do_request(
             "put",

openepd/api/average_dataset/industry_epd_sync_api.py

@@ -90,7 +90,8 @@ class IndustryEpdApi(BaseApiMethodGroup):
         """
         iepd_id = iepd.id
         if not iepd_id:
-            raise ValueError("The ID must be set to edit a IndustryEpd.")
+            msg = "The ID must be set to edit a IndustryEpd."
+            raise ValueError(msg)
 
         response = self._client.do_request(
             "put",

openepd/api/base_sync_client.py

@@ -14,14 +14,15 @@
 #  limitations under the License.
 #
 __all__ = (
-    "HttpStreamReader",
-    "SyncHttpClient",
+    "USER_AGENT_DEFAULT",
+    "BaseApiMethodGroup",
     "DoRequest",
+    "HttpStreamReader",
     "RetryHandler",
-    "BaseApiMethodGroup",
-    "USER_AGENT_DEFAULT",
+    "SyncHttpClient",
 )
 
+from collections.abc import Callable
 import datetime
 from functools import partial, wraps
 from io import IOBase
@@ -29,7 +30,7 @@ import logging
 import random
 import shutil
 import time
-from typing import IO, Any, BinaryIO, Callable, Final, NamedTuple
+from typing import IO, Any, BinaryIO, Final, NamedTuple
 
 import requests
 from requests import PreparedRequest, Response, Session, Timeout
@@ -180,7 +181,7 @@ class SyncHttpClient:
         self._throttler = Throttler(rate_per_sec=requests_per_sec)
         self._throttle_retry_timeout: float = (
             float(throttle_retry_timeout)
-            if isinstance(throttle_retry_timeout, (float, int))
+            if isinstance(throttle_retry_timeout, float | int)
             else throttle_retry_timeout.total_seconds()
         )
         self.user_agent = user_agent
@@ -412,7 +413,8 @@ class SyncHttpClient:
 
         response.raise_for_status()
         # This can't be handled by static checker because of the dynamic nature of the raise_for_status method
-        raise RuntimeError("This line should never be reached")
+        msg = "This line should never be reached"
+        raise RuntimeError(msg)
 
     def _get_url_for_request(self, path_or_url: str) -> str:
         """
@@ -471,7 +473,7 @@ class SyncHttpClient:
                     exception = e
 
                 if exception or response.status_code == requests_codes.service_unavailable:
-                    secs = random.randint(60, 60 * 5)
+                    secs = random.randint(60, 60 * 5)  # noqa: S311
                     logger.warning(
                         "%s %s is unavailable. Attempts left: %s. Waiting %s seconds...",
                         method,

openepd/api/common.py

@@ -13,12 +13,12 @@
 #  See the License for the specific language governing permissions and
 #  limitations under the License.
 #
-from collections.abc import Iterable
+from collections.abc import Callable, Iterable, Iterator
 from contextlib import contextmanager
 from datetime import datetime, timedelta
 import threading
 from time import sleep
-from typing import Callable, Generic, Iterator, cast
+from typing import Generic, cast
 
 from requests import Response
 
@@ -126,20 +126,26 @@ class StreamingListResponse(Iterable[TOpenEpdObject], Generic[TOpenEpdObject]):
         :return: list of items on the page
         """
         if page_num <= 0:
-            raise ValueError("Page number must be positive")
+            msg = "Page number must be positive"
+            raise ValueError(msg)
         if self.__current_page != page_num or force_reload:
             self.__recent_response = self.__fetch_handler(page_num, self.__page_size)
             self.__current_page = page_num
         if self.__recent_response is None:
-            raise RuntimeError("Response is empty, this should not happen, check if fetch_handler is compatible")
+            msg = "Response is empty, this should not happen, check if fetch_handler is compatible"
+            raise RuntimeError(msg)
         if self.__recent_response.payload is None:
-            raise ValueError("Response does not contain payload")
+            msg = "Response does not contain payload"
+            raise ValueError(msg)
         if not isinstance(self.__recent_response.payload, list):
-            raise ValueError("Response does not contain a list")
+            msg = "Response does not contain a list"
+            raise ValueError(msg)
         if self.__recent_response.meta is None:
-            raise ValueError("Response does not contain meta")
+            msg = "Response does not contain meta"
+            raise ValueError(msg)
         if not isinstance(self.__recent_response.meta, PagingMetaMixin):
-            raise ValueError("Response does not contain paging meta")
+            msg = "Response does not contain paging meta"
+            raise ValueError(msg)
         return self.__recent_response.payload
 
     def get_paging_meta(self) -> PagingMeta:
@@ -152,7 +158,8 @@ class StreamingListResponse(Iterable[TOpenEpdObject], Generic[TOpenEpdObject]):
         """
         paging_meta = cast(PagingMetaMixin, self.get_meta()).paging
         if paging_meta is None:
-            raise ValueError("Response does not contain paging meta")
+            msg = "Response does not contain paging meta"
+            raise ValueError(msg)
         return paging_meta
 
     def get_meta(self) -> MetaCollectionDto:
@@ -206,8 +213,7 @@ class StreamingListResponse(Iterable[TOpenEpdObject], Generic[TOpenEpdObject]):
         self.goto_page(start_from_page)
         while True:
             items = self.goto_page(self.current_page)
-            for x in items:
-                yield x
+            yield from items
             if not self.has_next_page():
                 return  # no more pages
             else:

openepd/api/dto/common.py

@@ -63,7 +63,7 @@ TMetaExtension = TypeVar("TMetaExtension", bound=MetaExtensionBase)
 
 class MetaCollectionDto(BaseOpenEpdApiModel, Generic[TMetaExtension]):
     """
-    This class is intended to be used as a container for different meta objects.
+    Use this class as a container for different meta objects.
 
     From a specific controller, you should return a specific subclass of MetaCollectionDto and appropriate mixins. This
     would allow to retain type information to generate schema for meta section of response.

openepd/api/epd/sync_api.py

@@ -173,7 +173,8 @@ class EpdApi(BaseApiMethodGroup):
         """
         epd_id = epd.id
         if not epd_id:
-            raise ValueError("The EPD ID must be set to edit an EPD.")
+            msg = "The EPD ID must be set to edit an EPD."
+            raise ValueError(msg)
         response = self._client.do_request(
             "put",
             f"/epds/{encode_path_param(epd_id)}",

openepd/api/org/__init__.py

@@ -0,0 +1,15 @@
+#
+#  Copyright 2025 by C Change Labs Inc. www.c-change-labs.com
+#
+#  Licensed under the Apache License, Version 2.0 (the "License");
+#  you may not use this file except in compliance with the License.
+#  You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+#  Unless required by applicable law or agreed to in writing, software
+#  distributed under the License is distributed on an "AS IS" BASIS,
+#  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+#  See the License for the specific language governing permissions and
+#  limitations under the License.
+#

openepd/api/org/sync_api.py

@@ -0,0 +1,79 @@
+#
+#  Copyright 2025 by C Change Labs Inc. www.c-change-labs.com
+#
+#  Licensed under the Apache License, Version 2.0 (the "License");
+#  you may not use this file except in compliance with the License.
+#  You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+#  Unless required by applicable law or agreed to in writing, software
+#  distributed under the License is distributed on an "AS IS" BASIS,
+#  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+#  See the License for the specific language governing permissions and
+#  limitations under the License.
+#
+from typing import Literal, overload
+
+from requests import Response
+
+from openepd.api.base_sync_client import BaseApiMethodGroup
+from openepd.api.utils import encode_path_param
+from openepd.model.org import Org, OrgRef
+
+
+class OrgApi(BaseApiMethodGroup):
+    """API methods for Orgs."""
+
+    @overload
+    def create(self, to_create: Org, with_response: Literal[True]) -> tuple[OrgRef, Response]: ...
+
+    @overload
+    def create(self, to_create: Org, with_response: Literal[False] = False) -> OrgRef: ...
+
+    def create(self, to_create: Org, with_response: bool = False) -> OrgRef | tuple[OrgRef, Response]:
+        """
+        Create a new organization.
+
+        :param to_create: Organization to create
+        :param with_response: if True, return a tuple of (OrgRef, Response), otherwise return only OrgRef
+        :return: Organization reference or Organization reference with HTTP Response object depending on parameter
+        :raise ValidationError: if given object Org is invalid
+        """
+        response = self._client.do_request("post", "/orgs", json=to_create.to_serializable())
+        content = response.json()
+        ref = OrgRef.model_validate(content)
+        if with_response:
+            return ref, response
+        return ref
+
+    @overload
+    def edit(self, to_edit: Org, with_response: Literal[True]) -> tuple[OrgRef, Response]: ...
+
+    @overload
+    def edit(self, to_edit: Org, with_response: Literal[False] = False) -> OrgRef: ...
+
+    def edit(self, to_edit: Org, with_response: bool = False) -> OrgRef | tuple[OrgRef, Response]:
+        """
+        Edit an organization.
+
+        :param to_edit: Organization to edit
+        :param with_response: if True, return a tuple of (OrgRef, Response), otherwise return only Org
+        :return: Organization reference or Organization reference with HTTP Response object depending on parameter
+        :raise ValueError: if the organization web_domain is not set
+        """
+        entity_id = to_edit.web_domain
+        if not entity_id:
+            msg = "The organization web_domain must be set to edit an organization."
+            raise ValueError(msg)
+        response = self._client.do_request(
+            "put",
+            f"/orgs/{encode_path_param(entity_id)}",
+            json=to_edit.to_serializable(exclude_unset=True, exclude_defaults=True, by_alias=True),
+        )
+        response.raise_for_status()
+        content = response.json()
+        ref = OrgRef.model_validate(content)
+        if with_response:
+            return ref, response
+        return ref

openepd/api/pcr/sync_api.py

@@ -13,7 +13,12 @@
 #  See the License for the specific language governing permissions and
 #  limitations under the License.
 #
+from typing import Literal, overload
+
+from requests import Response
+
 from openepd.api.base_sync_client import BaseApiMethodGroup
+from openepd.api.utils import encode_path_param
 from openepd.model.pcr import Pcr, PcrRef
 
 
@@ -42,3 +47,33 @@ class PcrApi(BaseApiMethodGroup):
         """
         pcr_ref_obj = self._client.do_request("post", "/pcrs", json=pcr.to_serializable()).json()
         return PcrRef.model_validate(pcr_ref_obj)
+
+    @overload
+    def edit(self, to_edit: Pcr, with_response: Literal[True]) -> tuple[PcrRef, Response]: ...
+
+    @overload
+    def edit(self, to_edit: Pcr, with_response: Literal[False] = False) -> PcrRef: ...
+
+    def edit(self, to_edit: Pcr, with_response: bool = False) -> PcrRef | tuple[PcrRef, Response]:
+        """
+        Edit a pcr.
+
+        :param to_edit: Pcr to edit
+        :param with_response: if True, return a tuple of (PcrRef, Response), otherwise return only PcrRef
+        :return: Pcr reference or Pcr reference with HTTP Response object depending on parameter
+        :raise ValueError: if the pcr ID is not set
+        """
+        entity_id = to_edit.id
+        if not entity_id:
+            msg = "The pcr ID must be set to edit a pcr."
+            raise ValueError(msg)
+        response = self._client.do_request(
+            "put",
+            f"/pcrs/{encode_path_param(entity_id)}",
+            json=to_edit.to_serializable(exclude_unset=True, exclude_defaults=True, by_alias=True),
+        )
+        content = response.json()
+        ref = PcrRef.model_validate(content)
+        if with_response:
+            return ref, response
+        return ref

openepd/api/plant/__init__.py

@@ -0,0 +1,15 @@
+#
+#  Copyright 2025 by C Change Labs Inc. www.c-change-labs.com
+#
+#  Licensed under the Apache License, Version 2.0 (the "License");
+#  you may not use this file except in compliance with the License.
+#  You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+#  Unless required by applicable law or agreed to in writing, software
+#  distributed under the License is distributed on an "AS IS" BASIS,
+#  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+#  See the License for the specific language governing permissions and
+#  limitations under the License.
+#

openepd/api/plant/sync_api.py

@@ -0,0 +1,79 @@
+#
+#  Copyright 2025 by C Change Labs Inc. www.c-change-labs.com
+#
+#  Licensed under the Apache License, Version 2.0 (the "License");
+#  you may not use this file except in compliance with the License.
+#  You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+#  Unless required by applicable law or agreed to in writing, software
+#  distributed under the License is distributed on an "AS IS" BASIS,
+#  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+#  See the License for the specific language governing permissions and
+#  limitations under the License.
+#
+from typing import Literal, overload
+
+from requests import Response
+
+from openepd.api.base_sync_client import BaseApiMethodGroup
+from openepd.api.utils import encode_path_param
+from openepd.model.org import Plant, PlantRef
+
+
+class PlantApi(BaseApiMethodGroup):
+    """API methods for Plants."""
+
+    @overload
+    def create(self, to_create: Plant, with_response: Literal[True]) -> tuple[PlantRef, Response]: ...
+
+    @overload
+    def create(self, to_create: Plant, with_response: Literal[False] = False) -> PlantRef: ...
+
+    def create(self, to_create: Plant, with_response: bool = False) -> PlantRef | tuple[PlantRef, Response]:
+        """
+        Create a new plant.
+
+        :param to_create: Plant to create
+        :param with_response: if True, return a tuple of (PlantRef, Response), otherwise return only PlantRef
+        :return: Plant reference or Plant reference with HTTP Response object depending on parameter
+        :raise ValidationError: if given object Plant is invalid
+        """
+        response = self._client.do_request("post", "/plants", json=to_create.to_serializable())
+        content = response.json()
+        ref = PlantRef.model_validate(content)
+        if with_response:
+            return ref, response
+        return ref
+
+    @overload
+    def edit(self, to_edit: Plant, with_response: Literal[True]) -> tuple[PlantRef, Response]: ...
+
+    @overload
+    def edit(self, to_edit: Plant, with_response: Literal[False] = False) -> PlantRef: ...
+
+    def edit(self, to_edit: Plant, with_response: bool = False) -> PlantRef | tuple[PlantRef, Response]:
+        """
+        Edit a plant.
+
+        :param to_edit: Plant to edit
+        :param with_response: if True, return a tuple of (PlantRef, Response), otherwise return only PlantRef
+        :return: Plant reference or Plant reference with HTTP Response object depending on parameter
+        :raise ValueError: if the plant ID is not set
+        """
+        entity_id = to_edit.id
+        if not entity_id:
+            msg = "The plant ID must be set to edit a plant."
+            raise ValueError(msg)
+        response = self._client.do_request(
+            "put",
+            f"/plants/{encode_path_param(entity_id)}",
+            json=to_edit.to_serializable(exclude_unset=True, exclude_defaults=True, by_alias=True),
+        )
+        response.raise_for_status()
+        content = response.json()
+        ref = PlantRef.model_validate(content)
+        if with_response:
+            return ref, response
+        return ref

openepd/api/standard/__init__.py

@@ -0,0 +1,15 @@
+#
+#  Copyright 2025 by C Change Labs Inc. www.c-change-labs.com
+#
+#  Licensed under the Apache License, Version 2.0 (the "License");
+#  you may not use this file except in compliance with the License.
+#  You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+#  Unless required by applicable law or agreed to in writing, software
+#  distributed under the License is distributed on an "AS IS" BASIS,
+#  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+#  See the License for the specific language governing permissions and
+#  limitations under the License.
+#

openepd/api/standard/sync_api.py

@@ -0,0 +1,79 @@
+#
+#  Copyright 2025 by C Change Labs Inc. www.c-change-labs.com
+#
+#  Licensed under the Apache License, Version 2.0 (the "License");
+#  you may not use this file except in compliance with the License.
+#  You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+#  Unless required by applicable law or agreed to in writing, software
+#  distributed under the License is distributed on an "AS IS" BASIS,
+#  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+#  See the License for the specific language governing permissions and
+#  limitations under the License.
+#
+from typing import Literal, overload
+
+from requests import Response
+
+from openepd.api.base_sync_client import BaseApiMethodGroup
+from openepd.api.utils import encode_path_param
+from openepd.model.standard import Standard, StandardRef
+
+
+class StandardApi(BaseApiMethodGroup):
+    """API methods for Standards."""
+
+    @overload
+    def create(self, to_create: Standard, with_response: Literal[True]) -> tuple[StandardRef, Response]: ...
+
+    @overload
+    def create(self, to_create: Standard, with_response: Literal[False] = False) -> StandardRef: ...
+
+    def create(self, to_create: Standard, with_response: bool = False) -> StandardRef | tuple[StandardRef, Response]:
+        """
+        Create a new standard.
+
+        :param to_create: Standard to create
+        :param with_response: if True, return a tuple of (StandardRef, Response), otherwise return only StandardRef
+        :return: Standard reference or Standard reference with HTTP Response object depending on parameter
+        :raise ValidationError: if given object Standard is invalid
+        """
+        response = self._client.do_request("post", "/standards", json=to_create.to_serializable())
+        content = response.json()
+        ref = StandardRef.model_validate(content)
+        if with_response:
+            return ref, response
+        return ref
+
+    @overload
+    def edit(self, to_edit: Standard, with_response: Literal[True]) -> tuple[StandardRef, Response]: ...
+
+    @overload
+    def edit(self, to_edit: Standard, with_response: Literal[False] = False) -> StandardRef: ...
+
+    def edit(self, to_edit: Standard, with_response: bool = False) -> StandardRef | tuple[StandardRef, Response]:
+        """
+        Edit a standard.
+
+        :param to_edit: Standard to edit
+        :param with_response: if True, return a tuple of (StandardRef, Response), otherwise return only StandardRef
+        :return: Standard reference or Standard reference with HTTP Response object depending on parameter
+        :raise ValueError: if the standard short_name is not set
+        """
+        entity_id = to_edit.short_name
+        if not entity_id:
+            msg = "The standard short_name must be set to edit a standard."
+            raise ValueError(msg)
+        response = self._client.do_request(
+            "put",
+            f"/standards/{encode_path_param(entity_id)}",
+            json=to_edit.to_serializable(exclude_unset=True, exclude_defaults=True, by_alias=True),
+        )
+        response.raise_for_status()
+        content = response.json()
+        ref = StandardRef.model_validate(content)
+        if with_response:
+            return ref, response
+        return ref

openepd/api/sync_client.py

@@ -22,7 +22,10 @@ from openepd.api.average_dataset.industry_epd_sync_api import IndustryEpdApi
 from openepd.api.base_sync_client import SyncHttpClient, TokenAuth
 from openepd.api.category.sync_api import CategoryApi
 from openepd.api.epd.sync_api import EpdApi
+from openepd.api.org.sync_api import OrgApi
 from openepd.api.pcr.sync_api import PcrApi
+from openepd.api.plant.sync_api import PlantApi
+from openepd.api.standard.sync_api import StandardApi
 
 
 class OpenEpdApiClientSync:
@@ -41,6 +44,9 @@ class OpenEpdApiClientSync:
         self._http_client = SyncHttpClient(base_url, auth=auth, **kwargs)
         self.__epd_api: EpdApi | None = None
         self.__pcr_api: PcrApi | None = None
+        self.__org_api: OrgApi | None = None
+        self.__plant_api: PlantApi | None = None
+        self.__standard_api: StandardApi | None = None
         self.__category_api: CategoryApi | None = None
         self.__generic_estimate_api: GenericEstimateApi | None = None
         self.__industry_epd_api: IndustryEpdApi | None = None
@@ -59,6 +65,27 @@ class OpenEpdApiClientSync:
             self.__pcr_api = PcrApi(self._http_client)
         return self.__pcr_api
 
+    @property
+    def orgs(self) -> OrgApi:
+        """Get the Org API."""
+        if self.__org_api is None:
+            self.__org_api = OrgApi(self._http_client)
+        return self.__org_api
+
+    @property
+    def plants(self) -> PlantApi:
+        """Get the Plant API."""
+        if self.__plant_api is None:
+            self.__plant_api = PlantApi(self._http_client)
+        return self.__plant_api
+
+    @property
+    def standards(self) -> StandardApi:
+        """Get the Standard API."""
+        if self.__standard_api is None:
+            self.__standard_api = StandardApi(self._http_client)
+        return self.__standard_api
+
     @property
     def categories(self) -> CategoryApi:
         """Get the Category API."""

openepd/bundle/base.py

@@ -14,8 +14,9 @@
 #  limitations under the License.
 #
 import abc
+from collections.abc import Callable, Iterator, Sequence
 import csv
-from typing import IO, Callable, Iterator, Self, Sequence, Type
+from typing import IO, Self
 
 from openepd.bundle.model import AssetInfo, AssetType, BundleManifest, RelType
 from openepd.model.base import TOpenEpdObject
@@ -70,7 +71,7 @@ class BaseBundleReader(BundleMixin, metaclass=abc.ABCMeta):
     def __enter__(self) -> Self:
         return self
 
-    def __exit__(self, type, value, traceback):
+    def __exit__(self, type, value, traceback):  # noqa: A002
         self.close()
 
     @abc.abstractmethod
@@ -133,7 +134,7 @@ class BaseBundleReader(BundleMixin, metaclass=abc.ABCMeta):
         pass
 
     @abc.abstractmethod
-    def read_object_asset(self, obj_class: Type[TOpenEpdObject], asset_ref: AssetRef) -> TOpenEpdObject:
+    def read_object_asset(self, obj_class: type[TOpenEpdObject], asset_ref: AssetRef) -> TOpenEpdObject:
         """Read an object asset by given reference."""
         pass
 
@@ -171,7 +172,7 @@ class BaseBundleWriter(BundleMixin, metaclass=abc.ABCMeta):
     def __enter__(self) -> Self:
         return self
 
-    def __exit__(self, type, value, traceback):
+    def __exit__(self, type, value, traceback):  # noqa: A002
         self.close()
 
     @abc.abstractmethod

openepd/bundle/reader.py

@@ -13,10 +13,11 @@
 #  See the License for the specific language governing permissions and
 #  limitations under the License.
 #
+from collections.abc import Callable, Iterator, Sequence
 import csv
 import io
 from os import PathLike
-from typing import IO, Callable, Iterator, Sequence, Type, cast
+from typing import IO, cast
 import zipfile
 
 from openepd.bundle.base import AssetFilter, AssetRef, BaseBundleReader
@@ -95,7 +96,8 @@ class DefaultBundleReader(BaseBundleReader):
         with self._bundle_archive.open("toc", "r") as toc_stream:
             toc_reader = csv.DictReader(io.TextIOWrapper(toc_stream, encoding="utf-8"), dialect="toc")
             if not toc_reader.fieldnames or len(toc_reader.fieldnames) < len(self._TOC_FIELDS):
-                raise ValueError("The bundle file is not valid. TOC reading error: wrong number of fields")
+                msg = "The bundle file is not valid. TOC reading error: wrong number of fields"
+                raise ValueError(msg)
 
     def root_assets_iter(
         self,
@@ -148,17 +150,21 @@ class DefaultBundleReader(BaseBundleReader):
         """Read the blob asset."""
         asset = self.get_asset_by_ref(asset_ref)
         if asset is None:
-            raise ValueError("Asset not found")
+            msg = "Asset not found"
+            raise ValueError(msg)
         return self._bundle_archive.open(asset.ref, "r")
 
-    def read_object_asset(self, obj_class: Type[TOpenEpdObject], asset_ref: AssetRef) -> TOpenEpdObject:
+    def read_object_asset(self, obj_class: type[TOpenEpdObject], asset_ref: AssetRef) -> TOpenEpdObject:
         """Read the object asset."""
         asset = self.get_asset_by_ref(asset_ref)
         if asset is None:
-            raise ValueError("Asset not found")
+            msg = "Asset not found"
+            raise ValueError(msg)
         if obj_class.get_asset_type() is None:
-            raise ValueError(f"Target object {obj_class.__name__} is not supported asset")
+            msg = f"Target object {obj_class.__name__} is not supported asset"
+            raise ValueError(msg)
         if asset.type != obj_class.get_asset_type():
-            raise ValueError(f"Asset type mismatch. Expected {obj_class.get_asset_type()}, got {asset.type}")
+            msg = f"Asset type mismatch. Expected {obj_class.get_asset_type()}, got {asset.type}"
+            raise ValueError(msg)
         with self._bundle_archive.open(asset.ref, "r") as asset_stream:
             return obj_class.model_validate_json(asset_stream.read())