Move AC_mouse_* alias helpers into a shared module

Sonar flagged 39.8% duplication on the previous commit because the
three single-button click adapters were defined identically in both
action_executor.py and callback_function_executor.py. Generate them
once in utils/executor/mouse_aliases.py via a small factory and have
both executors import the same callables.

Author: JE-Chen

je_auto_control/utils/callback/callback_function_executor.py

@@ -1,4 +1,4 @@
-from typing import Any, Callable, Optional, Tuple
+from typing import Any, Callable
 
 # utils cv2_utils
 from je_auto_control.utils.cv2_utils.screenshot import pil_screenshot
@@ -7,6 +7,9 @@
 from je_auto_control.utils.logging.logging_instance import autocontrol_logger
 # executor
 from je_auto_control.utils.executor.action_executor import execute_action, execute_files
+from je_auto_control.utils.executor.mouse_aliases import (
+    click_mouse_left, click_mouse_middle, click_mouse_right,
+)
 # file process
 from je_auto_control.utils.file_process.get_dir_file_list import get_dir_files_as_list
 # html report
@@ -48,24 +51,6 @@
 from je_auto_control.wrapper.auto_control_screen import screen_size, screenshot
 
 
-def _click_mouse_left(x: Optional[int] = None,
-                      y: Optional[int] = None) -> Tuple[int, int, int]:
-    """Callback adapter: click left mouse button (button is hardcoded)."""
-    return click_mouse("mouse_left", x, y)
-
-
-def _click_mouse_right(x: Optional[int] = None,
-                       y: Optional[int] = None) -> Tuple[int, int, int]:
-    """Callback adapter: click right mouse button (button is hardcoded)."""
-    return click_mouse("mouse_right", x, y)
-
-
-def _click_mouse_middle(x: Optional[int] = None,
-                        y: Optional[int] = None) -> Tuple[int, int, int]:
-    """Callback adapter: click middle mouse button (button is hardcoded)."""
-    return click_mouse("mouse_middle", x, y)
-
-
 class CallbackFunctionExecutor:
     """
     CallbackFunctionExecutor
@@ -79,9 +64,9 @@ def __init__(self):
         # 事件字典，對應字串名稱到實際函式
         self.event_dict: dict = {
             # mouse 滑鼠相關
-            "AC_mouse_left": _click_mouse_left,
-            "AC_mouse_right": _click_mouse_right,
-            "AC_mouse_middle": _click_mouse_middle,
+            "AC_mouse_left": click_mouse_left,
+            "AC_mouse_right": click_mouse_right,
+            "AC_mouse_middle": click_mouse_middle,
             "AC_click_mouse": click_mouse,
             "AC_get_mouse_table": get_mouse_table,
             "AC_get_mouse_position": get_mouse_position,

je_auto_control/utils/executor/action_executor.py

@@ -1,5 +1,5 @@
 import types
-from typing import Any, Callable, Dict, List, Optional, Tuple, Union
+from typing import Any, Callable, Dict, List, Optional, Union
 
 from je_auto_control.utils.exception.exception_tags import (
     action_is_null_error_message, add_command_exception_error_message,
@@ -22,6 +22,9 @@
 from je_auto_control.utils.executor.flow_control import (
     BLOCK_COMMANDS, LoopBreak, LoopContinue,
 )
+from je_auto_control.utils.executor.mouse_aliases import (
+    click_mouse_left, click_mouse_middle, click_mouse_right,
+)
 from je_auto_control.utils.ocr.ocr_engine import (
     click_text as ocr_click_text,
     locate_text_center as ocr_locate_text_center,
@@ -91,24 +94,6 @@ def _vlm_locate_as_list(description: str,
     return None if coords is None else [coords[0], coords[1]]
 
 
-def _click_mouse_left(x: Optional[int] = None,
-                      y: Optional[int] = None) -> Tuple[int, int, int]:
-    """Executor adapter: click left mouse button (button is hardcoded)."""
-    return click_mouse("mouse_left", x, y)
-
-
-def _click_mouse_right(x: Optional[int] = None,
-                       y: Optional[int] = None) -> Tuple[int, int, int]:
-    """Executor adapter: click right mouse button (button is hardcoded)."""
-    return click_mouse("mouse_right", x, y)
-
-
-def _click_mouse_middle(x: Optional[int] = None,
-                        y: Optional[int] = None) -> Tuple[int, int, int]:
-    """Executor adapter: click middle mouse button (button is hardcoded)."""
-    return click_mouse("mouse_middle", x, y)
-
-
 def _history_list_as_dicts(limit: int = 100,
                            source_type: Optional[str] = None) -> List[dict]:
     """Executor adapter: list run history as plain dicts (JSON-friendly)."""
@@ -142,9 +127,9 @@ def __init__(self):
         # 事件字典，對應字串名稱到函式
         self.event_dict: dict = {
             # Mouse 滑鼠相關
-            "AC_mouse_left": _click_mouse_left,
-            "AC_mouse_right": _click_mouse_right,
-            "AC_mouse_middle": _click_mouse_middle,
+            "AC_mouse_left": click_mouse_left,
+            "AC_mouse_right": click_mouse_right,
+            "AC_mouse_middle": click_mouse_middle,
             "AC_click_mouse": click_mouse,
             "AC_get_mouse_table": get_mouse_table,
             "AC_get_mouse_position": get_mouse_position,

je_auto_control/utils/executor/mouse_aliases.py

@@ -0,0 +1,30 @@
+"""Single-button mouse-click aliases shared by the action and callback executors.
+
+Each ``AC_mouse_<button>`` command name needs a callable whose signature is
+``(x=None, y=None)`` so the button itself is implied by the command name and
+not threaded through ``mouse_keycode``. Generating the three callables from
+one factory keeps the registration sites identical across executors.
+"""
+from typing import Callable, Optional, Tuple
+
+from je_auto_control.wrapper.auto_control_mouse import click_mouse
+
+ClickFn = Callable[[Optional[int], Optional[int]], Tuple[int, int, int]]
+
+
+def _make_button_click(button: str) -> ClickFn:
+    def click(x: Optional[int] = None,
+              y: Optional[int] = None) -> Tuple[int, int, int]:
+        return click_mouse(button, x, y)
+
+    click.__name__ = f"click_{button}"
+    click.__qualname__ = click.__name__
+    click.__doc__ = f"Click the {button.replace('_', ' ')} mouse button at (x, y)."
+    return click
+
+
+click_mouse_left: ClickFn = _make_button_click("mouse_left")
+click_mouse_right: ClickFn = _make_button_click("mouse_right")
+click_mouse_middle: ClickFn = _make_button_click("mouse_middle")
+
+__all__ = ["click_mouse_left", "click_mouse_right", "click_mouse_middle"]