fix(v2): raise NotFoundError on soft_delete of already-deleted entity

dobrotacreator · claude · dobrotacreator · commit d050e74d6951 · 2026-04-02T11:38:25.000+03:00
Use execute_for_one instead of execute so soft_delete returns 404
when the entity is already soft-deleted, matching v1 behavior.

Co-Authored-By: Claude Opus 4.6 (1M context) &lt;noreply@anthropic.com&gt;
diff --git a/src/notora/v2/services/mixins/delete.py b/src/notora/v2/services/mixins/delete.py
@@ -40,7 +40,7 @@ async def soft_delete(
         actor_id: Any | None = None,
     ) -> None:
         additional_payload = self._apply_updated_by({}, actor_id) or None
-        await self.execute(
+        await self.execute_for_one(
             session,
             self.repo.soft_delete(pk, additional_payload=additional_payload),
         )

Original file line number	Diff line number	Diff line change
`@@ -40,7 +40,7 @@ async def soft_delete(`
`40`	`40`	`actor_id: Any \| None = None,`
`41`	`41`	`) -> None:`
`42`	`42`	`additional_payload = self._apply_updated_by({}, actor_id) or None`
`43`		`- await self.execute(`
	`43`	`+ await self.execute_for_one(`
`44`	`44`	`session,`
`45`	`45`	`self.repo.soft_delete(pk, additional_payload=additional_payload),`
`46`	`46`	`)`