fix: revert eager merge

This reverts commit 296ae02.

advanced_alchemy/service/_async.py

@@ -10,7 +10,6 @@
 
 from sqlalchemy.orm import InstrumentedAttribute
 
-from advanced_alchemy.exceptions import RepositoryError
 from advanced_alchemy.repository._util import model_from_dict
 from advanced_alchemy.repository.typing import ModelT
 
@@ -317,11 +316,6 @@ async def update(
         data = await self.to_model(data, "update")
         if isinstance(id_attribute, InstrumentedAttribute):
             id_attribute = cast("str", id_attribute.description)
-        if item_id is None and self.repository.get_id_attribute_value(item=data, id_attribute=id_attribute) is None:
-            msg = f"Could not identify ID attribute value.  One of the following is required: ``item_id`` or ``data.{id_attribute or self.repository.id_attribute}``"
-            raise RepositoryError(
-                msg,
-            )
         if item_id is not None:
             data = self.repository.set_id_attribute_value(item_id=item_id, item=data, id_attribute=id_attribute)
         return await self.repository.update(

advanced_alchemy/service/_sync.py

@@ -12,7 +12,6 @@
 
 from sqlalchemy.orm import InstrumentedAttribute, Session
 
-from advanced_alchemy.exceptions import RepositoryError
 from advanced_alchemy.repository._util import model_from_dict
 from advanced_alchemy.repository.typing import ModelT
 
@@ -318,11 +317,6 @@ def update(
         data = self.to_model(data, "update")
         if isinstance(id_attribute, InstrumentedAttribute):
             id_attribute = cast("str", id_attribute.description)
-        if item_id is None and self.repository.get_id_attribute_value(item=data, id_attribute=id_attribute) is None:
-            msg = f"Could not identify ID attribute value.  One of the following is required: ``item_id`` or ``data.{id_attribute or self.repository.id_attribute}``"
-            raise RepositoryError(
-                msg,
-            )
         if item_id is not None:
             data = self.repository.set_id_attribute_value(item_id=item_id, item=data, id_attribute=id_attribute)
         return self.repository.update(

tests/integration/test_repository.py

@@ -1550,8 +1550,3 @@ async def test_repo_get_or_create_deprecation(author_repo: AuthorRepository, fir
         existing_obj, existing_created = await maybe_async(author_repo.get_or_create(name="Agatha Christie"))
         assert existing_obj.id == first_author_id
         assert existing_created is False
-
-
-async def test_service_update_no_pk(author_service: AuthorService, first_author_id: Any) -> None:
-    with pytest.raises(RepositoryError):
-        _existing_obj = await maybe_async(author_service.update(data={"name": "Agatha Christie"}))