GeneralManager 0.86.1__tar.gz → 0.86.2__tar.gz

{generalmanager-0.86.1/src/GeneralManager.egg-info → generalmanager-0.86.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: GeneralManager
-Version: 0.86.1
+Version: 0.86.2
 Summary: Modular Django-based data management framework with ORM, GraphQL, fine-grained permissions, rule validation, calculations and caching.
 Author-email: Tim Kleindick <tkleindick@yahoo.de>
 License: MIT License

{generalmanager-0.86.1 → generalmanager-0.86.2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "GeneralManager"
-version = "0.86.1"
+version = "0.86.2"
 description = "Modular Django-based data management framework with ORM, GraphQL, fine-grained permissions, rule validation, calculations and caching."
 readme = "README.md"
 authors = [{ name = "Tim Kleindick", email = "tkleindick@yahoo.de" }]

{generalmanager-0.86.1 → generalmanager-0.86.2/src/GeneralManager.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: GeneralManager
-Version: 0.86.1
+Version: 0.86.2
 Summary: Modular Django-based data management framework with ORM, GraphQL, fine-grained permissions, rule validation, calculations and caching.
 Author-email: Tim Kleindick <tkleindick@yahoo.de>
 License: MIT License

{generalmanager-0.86.1 → generalmanager-0.86.2}/src/general_manager/cache/data_change_context.py

@@ -162,3 +162,44 @@ def may_reuse_data_change_transaction(database_alias: str) -> bool:
         and not owner.caller_in_atomic_block
         and is_data_change_operation_authorized(database_alias)
     )
+
+
+@dataclass
+class DataChangeOutcome:
+    """Private per-call outcome; unknown/custom mutations remain changes."""
+
+    interface: object
+    unchanged: bool = False
+
+
+_operation_outcome: ContextVar[DataChangeOutcome | None] = ContextVar(
+    "data_change_operation_outcome", default=None
+)
+
+
+@contextmanager
+def suspend_data_change_outcome() -> Iterator[None]:
+    """Keep nested signal/lifecycle receivers outside their parent's outcome."""
+    token = _operation_outcome.set(None)
+    try:
+        yield
+    finally:
+        _operation_outcome.reset(token)
+
+
+@contextmanager
+def capture_data_change_outcome(interface: object) -> Iterator[DataChangeOutcome]:
+    """Isolate the wrapped operation from pre-signals and nested mutations."""
+    outcome = DataChangeOutcome(interface)
+    token = _operation_outcome.set(outcome)
+    try:
+        yield outcome
+    finally:
+        _operation_outcome.reset(token)
+
+
+def mark_data_change_unchanged(interface: object) -> None:
+    """Accept a proven no-op only for the interface owned by this call."""
+    outcome = _operation_outcome.get()
+    if outcome is not None and outcome.interface is interface:
+        outcome.unchanged = True

{generalmanager-0.86.1 → generalmanager-0.86.2}/src/general_manager/cache/signals.py

@@ -15,8 +15,10 @@ from django.utils.connection import ConnectionDoesNotExist
 from general_manager.cache.data_change_context import (
     DataChangeTransactionScope,
     authorize_data_change_operation,
+    capture_data_change_outcome,
     own_data_change_transaction,
     record_data_change_phase,
+    suspend_data_change_outcome,
 )
 from general_manager.as_of import reject_historical_mutation
 from general_manager.logging import get_logger
@@ -66,7 +68,9 @@ def data_change(
     clears run-scoped ORM bucket/index caches, emits `pre_data_change`, invokes
     the wrapped callable, then emits `post_data_change` with the changed
     instance, previous instance, action name, identification, and copied
-    `_old_values` payload. Both signals receive a per-call `change_context` and
+    `_old_values` payload. Proven unchanged built-in ORM updates skip only the
+    post-change signal; pre-change preparation and transaction cleanup still run.
+    Both signals receive a per-call `change_context` and
     the manager interface's database alias. ORM-backed signal dispatch and the
     mutation share one transaction. GraphQL warm-up requeue keys collected
     during signal handling are drained only after the outermost active
@@ -101,6 +105,7 @@ def data_change(
     )
 
     @wraps(decorator_source)
+    @suspend_data_change_outcome()
     def perform_change(*args: P.args, **kwargs: P.kwargs) -> R:
         """
         Emit pre_data_change and post_data_change signals around the wrapped function call.
@@ -252,7 +257,12 @@ def data_change(
                     if is_orm_backed
                     else nullcontext()
                 )
-                with operation_context:
+                with (
+                    operation_context,
+                    capture_data_change_outcome(
+                        getattr(instance_before, "_interface", None)
+                    ) as outcome,
+                ):
                     mutation_started = (
                         perf_counter() if transaction_scope is not None else None
                     )
@@ -282,15 +292,16 @@ def data_change(
                 if identification is None:
                     identification = pre_identification
 
-                post_data_change.send(
-                    sender=sender,
-                    instance=instance,
-                    previous_instance=instance_before,
-                    identification=identification,
-                    action=action,
-                    old_relevant_values=old_relevant_values,
-                    **signal_kwargs,
-                )
+                if not outcome.unchanged:
+                    post_data_change.send(
+                        sender=sender,
+                        instance=instance,
+                        previous_instance=instance_before,
+                        identification=identification,
+                        action=action,
+                        old_relevant_values=old_relevant_values,
+                        **signal_kwargs,
+                    )
                 if instance_before is not None:
                     try:
                         delattr(instance_before, "_old_values")

{generalmanager-0.86.1 → generalmanager-0.86.2}/src/general_manager/interface/capabilities/orm/mutations.py

@@ -11,7 +11,10 @@ from django.contrib.auth import get_user_model
 from django.db import DEFAULT_DB_ALIAS, models, router, transaction
 from django.db.models import NOT_PROVIDED
 
-from general_manager.cache.data_change_context import owns_data_change_transaction
+from general_manager.cache.data_change_context import (
+    mark_data_change_unchanged,
+    owns_data_change_transaction,
+)
 from general_manager.interface.capabilities.base import CapabilityName
 from general_manager.interface.capabilities.builtin import BaseCapability
 from general_manager.interface.capabilities.orm_utils.payload_normalizer import (
@@ -20,6 +23,7 @@ from general_manager.interface.capabilities.orm_utils.payload_normalizer import
 from general_manager.interface.capabilities.orm_utils.update_state import (
     changed_many_to_many,
     is_unchanged_update,
+    record_skipped_update_save,
     track_update,
 )
 from general_manager.interface.utils.database_interface_protocols import (
@@ -225,6 +229,7 @@ class OrmMutationCapability(BaseCapability):
                     object.__setattr__(instance, "changed_by_id", creator_id)
                 instance.full_clean()
                 if is_unchanged_update(instance):
+                    record_skipped_update_save(instance)
                     return instance.pk
                 if batch is not None and database_alias is None:
                     if (
@@ -606,7 +611,7 @@ class OrmUpdateCapability(BaseCapability):
                     changed_many,
                     history_comment,
                     database_alias=database_alias,
-                ):
+                ) as update_state:
                     instance = mutation.assign_simple_attributes(
                         interface_instance.__class__, instance, normalized_simple
                     )
@@ -624,6 +629,32 @@ class OrmUpdateCapability(BaseCapability):
                     history_comment=history_comment,
                 )
             discard_orm_instance_cache(interface_instance.__class__, pk)
+            if update_state is not None and update_state.save_skipped:
+                # A custom wrapper may write after delegating to a built-in
+                # method. Only the complete standard path can prove a no-op.
+                from general_manager.interface.base_interface import InterfaceBase
+
+                standard_methods = (
+                    (interface_instance.update, InterfaceBase.update),
+                    (self.update, OrmUpdateCapability.update),
+                    (
+                        mutation.assign_simple_attributes,
+                        OrmMutationCapability.assign_simple_attributes,
+                    ),
+                    (
+                        mutation.save_with_history,
+                        OrmMutationCapability.save_with_history,
+                    ),
+                    (
+                        mutation.apply_many_to_many,
+                        OrmMutationCapability.apply_many_to_many,
+                    ),
+                )
+                if all(
+                    getattr(method, "__func__", None) is standard
+                    for method, standard in standard_methods
+                ):
+                    mark_data_change_unchanged(interface_instance)
             return {"id": pk}
 
         return call_with_observability(

{generalmanager-0.86.1 → generalmanager-0.86.2}/src/general_manager/interface/capabilities/orm_utils/update_state.py

@@ -16,10 +16,11 @@ from django.db.models.fields.files import FieldFile
 from general_manager.measurement.measurement_field import MeasurementField
 
 
-@dataclass(frozen=True)
+@dataclass
 class _UpdateState:
     instance: models.Model
     values: dict[str, object]
+    save_skipped: bool = False
 
 
 _update_state: ContextVar[_UpdateState | None] = ContextVar(
@@ -47,7 +48,7 @@ def track_update(
     history_comment: str | None,
     *,
     database_alias: str | None = None,
-) -> Iterator[None]:
+) -> Iterator[_UpdateState | None]:
     """Scope the original row to its save, including nested mutation calls.
 
     The save compares after full_clean(), so validation and changes made by
@@ -85,7 +86,7 @@ def track_update(
             state = _UpdateState(instance, values)
     token = _update_state.set(state)
     try:
-        yield
+        yield state
     finally:
         _update_state.reset(token)
 
@@ -107,6 +108,13 @@ def is_unchanged_update(instance: models.Model) -> bool:
     return True
 
 
+def record_skipped_update_save(instance: models.Model) -> None:
+    """Record the built-in save decision without classifying custom wrappers."""
+    state = _update_state.get()
+    if state is not None and state.instance is instance:
+        state.save_skipped = True
+
+
 def changed_many_to_many(
     instance: models.Model,
     values: dict[str, list[object]],

{generalmanager-0.86.1 → generalmanager-0.86.2}/src/general_manager/workflow/signal_bridge.py

@@ -29,6 +29,8 @@ _WORKFLOW_SIGNAL_BRIDGE_KEY = "WORKFLOW_SIGNAL_BRIDGE"
 _DISPATCH_UID = "general_manager_workflow_signal_bridge"
 
 _RESERVED_KEYS = {
+    "change_context",
+    "database_alias",
     "creator_id",
     "history_comment",
     "identification",
@@ -88,8 +90,8 @@ def _manager_change_to_event(
         for key, value in kwargs.items()
         if key not in _RESERVED_KEYS and not key.startswith("_")
     }
-    if action in {"create", "update"} and not relevant_fields:
-        return None
+    # The mutation outcome controls no-op suppression. Empty input can still
+    # create defaults, change values in clean(), or request an audit touch.
 
     identification = kwargs.get("identification")
     if identification is None: