Fix: Apply history session policies consistently to plugin inputs

Plugin-created objects did not consistently honor the configured session policy, and multi-object loads could prompt repeatedly or consume their decision before an object was accepted. Resolve persisted policies at the insertion boundary and carry one lazy, panel-specific decision across each plugin batch.

	* [FIX] : Apply live general and plugin ask/yes/no settings while preserving explicit per-call choices
	* [NEW] : Add a validated multi-load scope that decides on the first accepted object and leaves empty or cancelled batches untouched
	* [FIX] : Decide before insertion, suppress duplicate prompts, report memory refusals, and debounce signal and image panels independently
	* [CHG] : Batch Test Data plugin loads using the matching data panel
	* [NEW] : Cover plugin policy behavior and add French settings translations

Author: dappham-CODRA

datalab/control/proxy.py

@@ -75,15 +75,39 @@
 
 from collections.abc import Generator
 from contextlib import contextmanager
+from dataclasses import dataclass
+from typing import TYPE_CHECKING, Literal
 
 import guidata.dataset as gds
 import numpy as np
 from sigima import ImageObj, SignalObj
 
+from datalab.config import Conf
 from datalab.control.baseproxy import BaseProxy
 from datalab.control.remote import RemoteClient
 from datalab.utils import qthelpers as qth
 
+if TYPE_CHECKING:
+    from datalab.gui.historysession_ops import SessionBehavior
+    from datalab.gui.main import DLMainWindow
+
+
+@dataclass
+class MultiLoadState:
+    """Track a local proxy multi-object load."""
+
+    panel: Literal["signal", "image"]
+    behavior: SessionBehavior | None
+    decision_applied: bool = False
+
+    def behavior_for(self, panel: Literal["signal", "image"]) -> SessionBehavior | None:
+        """Return the session behavior for an object added to ``panel``."""
+        if panel != self.panel:
+            raise ValueError(
+                f"Cannot add a {panel} object during a {self.panel} multiload session"
+            )
+        return "no" if self.decision_applied else self.behavior
+
 
 class RemoteProxy(RemoteClient):
     """DataLab remote proxy class.
@@ -137,8 +161,20 @@ class LocalProxy(BaseProxy):
 
     Args:
         datalab (DLMainWindow): DLMainWindow instance.
+        input_source: Source of objects added through this proxy.
     """
 
+    def __init__(
+        self,
+        datalab: DLMainWindow | None = None,
+        input_source: Literal["local", "plugin"] = "local",
+    ) -> None:
+        if input_source not in ("local", "plugin"):
+            raise ValueError(f"Invalid local proxy input source: {input_source!r}")
+        super().__init__(datalab)
+        self.input_source = input_source
+        self.multiload_state: MultiLoadState | None = None
+
     def add_signal(
         self,
         title: str,
@@ -150,6 +186,7 @@ def add_signal(
         ylabel: str = "",
         group_id: str = "",
         set_current: bool = True,
+        new_session_behavior: SessionBehavior | None = None,
     ) -> bool:  # pylint: disable=too-many-arguments
         """Add signal data to DataLab.
 
@@ -163,6 +200,7 @@ def add_signal(
             ylabel: Y label. Defaults to ""
             group_id: group id in which to add the signal. Defaults to ""
             set_current: if True, set the added signal as current
+            new_session_behavior: Optional history session creation policy
 
         Returns:
             True if signal was added successfully, False otherwise
@@ -171,9 +209,28 @@ def add_signal(
             ValueError: Invalid xdata dtype
             ValueError: Invalid ydata dtype
         """
-        return self._datalab.add_signal(
-            title, xdata, ydata, xunit, yunit, xlabel, ylabel, group_id, set_current
+        multiload_state = self.multiload_state
+        if multiload_state is None:
+            behavior = new_session_behavior
+            if behavior is None and self.input_source == "plugin":
+                behavior = Conf.proc.history_plugin_new_session_behavior.get()
+        else:
+            behavior = multiload_state.behavior_for("signal")
+        added = self._datalab.add_signal(
+            title,
+            xdata,
+            ydata,
+            xunit,
+            yunit,
+            xlabel,
+            ylabel,
+            group_id,
+            set_current,
+            new_session_behavior=behavior,
         )
+        if added and multiload_state is not None:
+            multiload_state.decision_applied = True
+        return added
 
     def add_image(
         self,
@@ -187,6 +244,7 @@ def add_image(
         zlabel: str = "",
         group_id: str = "",
         set_current: bool = True,
+        new_session_behavior: SessionBehavior | None = None,
     ) -> bool:  # pylint: disable=too-many-arguments
         """Add image data to DataLab.
 
@@ -201,14 +259,22 @@ def add_image(
             zlabel: Z label. Defaults to ""
             group_id: group id in which to add the image. Defaults to ""
             set_current: if True, set the added image as current
+            new_session_behavior: Optional history session creation policy
 
         Returns:
             True if image was added successfully, False otherwise
 
         Raises:
             ValueError: Invalid data dtype
         """
-        return self._datalab.add_image(
+        multiload_state = self.multiload_state
+        if multiload_state is None:
+            behavior = new_session_behavior
+            if behavior is None and self.input_source == "plugin":
+                behavior = Conf.proc.history_plugin_new_session_behavior.get()
+        else:
+            behavior = multiload_state.behavior_for("image")
+        added = self._datalab.add_image(
             title,
             data,
             xunit,
@@ -219,19 +285,81 @@ def add_image(
             zlabel,
             group_id,
             set_current,
+            new_session_behavior=behavior,
         )
+        if added and multiload_state is not None:
+            multiload_state.decision_applied = True
+        return added
 
     def add_object(
-        self, obj: SignalObj | ImageObj, group_id: str = "", set_current: bool = True
-    ) -> None:
+        self,
+        obj: SignalObj | ImageObj,
+        group_id: str = "",
+        set_current: bool = True,
+        new_session_behavior: SessionBehavior | None = None,
+    ) -> bool:
         """Add object to DataLab.
 
         Args:
             obj: Signal or image object
             group_id: group id in which to add the object. Defaults to ""
             set_current: if True, set the added object as current
+            new_session_behavior: Optional history session creation policy
+
+        Returns:
+            True if the object was added successfully, False otherwise
         """
-        self._datalab.add_object(obj, group_id, set_current)
+        multiload_state = self.multiload_state
+        if multiload_state is None:
+            behavior = new_session_behavior
+            if behavior is None and self.input_source == "plugin":
+                behavior = Conf.proc.history_plugin_new_session_behavior.get()
+        else:
+            if isinstance(obj, SignalObj):
+                panel = "signal"
+            elif isinstance(obj, ImageObj):
+                panel = "image"
+            else:
+                raise TypeError(f"Unsupported object type {type(obj)}")
+            behavior = multiload_state.behavior_for(panel)
+        added = self._datalab.add_object(
+            obj, group_id, set_current, new_session_behavior=behavior
+        )
+        if added and multiload_state is not None:
+            multiload_state.decision_applied = True
+        return added
+
+    @contextmanager
+    def multiload_session(
+        self,
+        panel: Literal["signal", "image"],
+        new_session_behavior: SessionBehavior | None = None,
+    ) -> Generator[None, None, None]:
+        """Apply one lazy session decision to a multi-object load.
+
+        Args:
+            panel: Target data panel ("signal" or "image")
+            new_session_behavior: Optional history session creation policy
+
+        Raises:
+            ValueError: If the panel or session behavior is invalid
+            RuntimeError: If another multiload session is already active
+        """
+        if panel not in ("signal", "image"):
+            raise ValueError(f"Invalid data panel: {panel!r}")
+        behavior = new_session_behavior
+        if behavior is None and self.input_source == "plugin":
+            behavior = Conf.proc.history_plugin_multiload_behavior.get()
+        if behavior is not None and behavior not in ("ask", "yes", "no"):
+            raise ValueError(f"Invalid session behavior: {behavior!r}")
+        if self.multiload_state is not None:
+            raise RuntimeError("Nested multiload sessions are not supported")
+        previous_state = self.multiload_state
+        self.multiload_state = MultiLoadState(panel, behavior)
+        try:
+            yield
+        finally:
+            self.multiload_state = previous_state
 
     def calc(self, name: str, param: gds.DataSet | None = None) -> None:
         """Call computation feature ``name``

datalab/gui/historysession_ops.py

@@ -10,7 +10,7 @@
 
 from qtpy import QtWidgets as QW
 
-from datalab.config import _
+from datalab.config import Conf, _
 from datalab.env import execenv
 from datalab.gui.panel.history import chain as hchain
 from datalab.history import HistoryAction, HistorySession, WorkspaceState
@@ -56,7 +56,7 @@ def maybe_start_session_for_input(
     panel_str: str | None = None,
     *,
     load: bool = False,
-    behavior: SessionBehavior = "ask",
+    behavior: SessionBehavior | None = None,
 ) -> bool:
     """Offer to start a new history session before a creation/load is recorded.
 
@@ -71,14 +71,16 @@ def maybe_start_session_for_input(
         load: True when triggered by a file/workspace load, False for an object
          creation. Only affects the prompt wording.
         behavior: Session creation policy: ask, always create ("yes"), or keep
-         the current session ("no").
+         the current session ("no"). Defaults to the live general policy.
 
     Returns:
         True if a new session was created.
 
     Raises:
         ValueError: If ``behavior`` is unsupported.
     """
+    if behavior is None:
+        behavior = Conf.proc.history_new_session_behavior.get()
     if behavior not in SESSION_BEHAVIORS:
         raise ValueError(f"Invalid session behavior: {behavior!r}")
     if not panel.record_mode_enabled or panel.is_replaying():
@@ -96,7 +98,7 @@ def maybe_start_session_for_input(
         return True
     # Debounce: a synchronous burst of creations (plugin/macro) must prompt only
     # once. The guard is reset on the next event-loop turn.
-    if not panel.runtime.execution.start_session_input_prompt():
+    if not panel.runtime.execution.start_session_input_prompt(target_panel_str):
         return False
     if execenv.unattended:
         # Headless runs: honor the accept_dialogs flag (default False -> "No"),

datalab/gui/main.py

@@ -94,6 +94,7 @@
 if TYPE_CHECKING:
     from typing import Literal
 
+    from datalab.gui.historysession_ops import SessionBehavior
     from datalab.gui.panel.base import AbstractPanel, BaseDataPanel
     from datalab.gui.panel.image import ImagePanel
     from datalab.gui.panel.macro import MacroPanel
@@ -2336,36 +2337,51 @@ def import_h5_file(self, filename: str, reset_all: bool | None = None) -> None:
     # (see TODO regarding RemoteClient.add_object method)
     #  @remote_controlled
     def add_object(
-        self, obj: SignalObj | ImageObj, group_id: str = "", set_current=True
-    ) -> None:
+        self,
+        obj: SignalObj | ImageObj,
+        group_id: str = "",
+        set_current=True,
+        new_session_behavior: SessionBehavior | None = None,
+    ) -> bool:
         """Add object - signal or image
 
         Args:
             obj: object to add (signal or image)
             group_id: group ID (optional)
             set_current: True to set the object as current object
+            new_session_behavior: Optional history session creation policy
+
+        Returns:
+            True if the object was added successfully, False otherwise
         """
-        if self.confirm_memory_state():
-            if isinstance(obj, SignalObj):
-                self.signalpanel.add_object(obj, group_id, set_current)
-                panel_str = "signal"
-            elif isinstance(obj, ImageObj):
-                self.imagepanel.add_object(obj, group_id, set_current)
-                panel_str = "image"
-            else:
-                raise TypeError(f"Unsupported object type {type(obj)}")
-            # Record a creation entry so objects added programmatically (plugins,
-            # macros, remote control) appear in the history. ``panel.add_object``
-            # deliberately does not record, so creations entering through this
-            # proxy boundary would otherwise be lost (notably the very first one).
+        if not self.confirm_memory_state():
+            return False
+        if isinstance(obj, SignalObj):
+            panel = self.signalpanel
+            panel_str = "signal"
+        elif isinstance(obj, ImageObj):
+            panel = self.imagepanel
+            panel_str = "image"
+        else:
+            raise TypeError(f"Unsupported object type {type(obj)}")
+        self.historypanel.maybe_start_session_for_input(
+            panel_str=panel_str, behavior=new_session_behavior
+        )
+        panel.add_object(obj, group_id, set_current)
+        # Record a creation entry so objects added programmatically (plugins,
+        # macros, remote control) appear in the history. ``panel.add_object``
+        # deliberately does not record, so creations entering through this
+        # proxy boundary would otherwise be lost (notably the very first one).
+        with self.historypanel.session_prompt_suppressed():
             action = self.historypanel.add_ui_entry(
                 _("New %s") % panel_str,
                 target=panel_str + "panel",
                 method_name="new_object",
                 save_state=False,
             )
-            if action is not None:
-                self.historypanel.register_action_outputs(action, [get_uuid(obj)])
+        if action is not None:
+            self.historypanel.register_action_outputs(action, [get_uuid(obj)])
+        return True
 
     @remote_controlled
     def set_object(self, obj: SignalObj | ImageObj) -> None:
@@ -2436,6 +2452,7 @@ def add_signal(
         ylabel: str = "",
         group_id: str = "",
         set_current: bool = True,
+        new_session_behavior: SessionBehavior | None = None,
     ) -> bool:  # pylint: disable=too-many-arguments
         """Add signal data to DataLab.
 
@@ -2449,6 +2466,7 @@ def add_signal(
             ylabel: Y label. Defaults to ""
             group_id: group id in which to add the signal. Defaults to ""
             set_current: if True, set the added signal as current
+            new_session_behavior: Optional history session creation policy
 
         Returns:
             True if signal was added successfully, False otherwise
@@ -2464,8 +2482,7 @@ def add_signal(
             units=(xunit, yunit),
             labels=(xlabel, ylabel),
         )
-        self.add_object(obj, group_id, set_current)
-        return True
+        return self.add_object(obj, group_id, set_current, new_session_behavior)
 
     # This API mirrors the image metadata accepted by create_image, so the
     # argument count is part of the stable public interface rather than noise.
@@ -2481,6 +2498,7 @@ def add_image(  # pylint: disable=too-many-arguments
         zlabel: str = "",
         group_id: str = "",
         set_current: bool = True,
+        new_session_behavior: SessionBehavior | None = None,
     ) -> bool:
         """Add image data to DataLab.
 
@@ -2495,6 +2513,7 @@ def add_image(  # pylint: disable=too-many-arguments
             zlabel: Z label. Defaults to ""
             group_id: group id in which to add the image. Defaults to ""
             set_current: if True, set the added image as current
+            new_session_behavior: Optional history session creation policy
 
         Returns:
             True if image was added successfully, False otherwise
@@ -2508,8 +2527,7 @@ def add_image(  # pylint: disable=too-many-arguments
             units=(xunit, yunit, zunit),
             labels=(xlabel, ylabel, zlabel),
         )
-        self.add_object(obj, group_id, set_current)
-        return True
+        return self.add_object(obj, group_id, set_current, new_session_behavior)
 
     # ------?
     def __about(self) -> None:  # pragma: no cover