Use create functions for implicit and base target instead of copy() (#1593)

Fixes #719

## Checklist

- [ ] I have added
[tests](https://www.cursorless.org/docs/contributing/test-case-recorder/)
- [ ] I have updated the
[docs](https://github.com/cursorless-dev/cursorless/tree/main/docs) and
[cheatsheet](https://github.com/cursorless-dev/cursorless/tree/main/cursorless-talon/src/cheatsheet)
- [ ] I have not broken the cheatsheet

---------

Co-authored-by: pre-commit-ci-lite[bot] <117423508+pre-commit-ci-lite[bot]@users.noreply.github.com>

Author: AndreasArvidsson

cursorless-talon/src/actions/actions.py

@@ -1,7 +1,7 @@
 from talon import Module, actions, app
 
 from ..csv_overrides import init_csv_and_watch_changes
-from ..primitive_target import IMPLICIT_TARGET
+from ..primitive_target import create_implicit_target
 from .actions_callback import callback_action_defaults, callback_action_map
 from .actions_simple import (
     no_wait_actions,
@@ -49,7 +49,7 @@ def cursorless_command(action_id: str, target: dict):
                 actions.sleep(no_wait_actions_post_sleep[action_id])
         elif action_id in ["replaceWithTarget", "moveToTarget"]:
             actions.user.cursorless_multiple_target_command(
-                action_id, [target, IMPLICIT_TARGET.copy()]
+                action_id, [target, create_implicit_target()]
             )
         else:
             return actions.user.cursorless_single_target_command(action_id, target)

cursorless-talon/src/actions/call.py

@@ -1,10 +1,10 @@
 from talon import Module, actions
 
-from ..primitive_target import IMPLICIT_TARGET
+from ..primitive_target import create_implicit_target
 
 mod = Module()
 
 
 def run_call_action(target: dict):
-    targets = [target, IMPLICIT_TARGET.copy()]
+    targets = [target, create_implicit_target()]
     actions.user.cursorless_multiple_target_command("callAsFunction", targets)

cursorless-talon/src/actions/move_bring.py

@@ -1,6 +1,6 @@
 from talon import Module
 
-from ..primitive_target import IMPLICIT_TARGET
+from ..primitive_target import create_implicit_target
 
 mod = Module()
 
@@ -15,6 +15,6 @@ def cursorless_move_bring_targets(m) -> list[dict]:
     try:
         target_list += [m.cursorless_positional_target]
     except AttributeError:
-        target_list += [IMPLICIT_TARGET.copy()]
+        target_list += [create_implicit_target()]
 
     return target_list

cursorless-talon/src/actions/swap.py

@@ -1,6 +1,6 @@
 from talon import Module
 
-from ..primitive_target import BASE_TARGET
+from ..primitive_target import create_base_target
 
 mod = Module()
 
@@ -20,6 +20,6 @@ def cursorless_swap_targets(m) -> list[dict]:
     target_list = m.cursorless_target_list
 
     if len(target_list) == 1:
-        target_list = [BASE_TARGET] + target_list
+        target_list = [create_base_target()] + target_list
 
     return target_list

cursorless-talon/src/command.py

@@ -7,7 +7,7 @@
     run_rpc_command_get,
     run_rpc_command_no_wait,
 )
-from .primitive_target import IMPLICIT_TARGET
+from .primitive_target import create_implicit_target
 
 mod = Module()
 
@@ -90,7 +90,7 @@ def cursorless_implicit_target_command(
     ):
         """Execute cursorless command with implicit target"""
         actions.user.cursorless_single_target_command(
-            action, IMPLICIT_TARGET, arg1, arg2, arg3
+            action, create_implicit_target(), arg1, arg2, arg3
         )
 
     def cursorless_multiple_target_command(

cursorless-talon/src/compound_targets.py

@@ -3,7 +3,7 @@
 from talon import Module
 
 from .connective import default_range_connective
-from .primitive_target import IMPLICIT_TARGET
+from .primitive_target import create_implicit_target
 
 mod = Module()
 
@@ -46,7 +46,7 @@ def cursorless_range(m) -> dict[str, Any]:
         return primitive_targets[0]
 
     if len(primitive_targets) == 1:
-        anchor = IMPLICIT_TARGET.copy()
+        anchor = create_implicit_target()
     else:
         anchor = primitive_targets[0]
 

cursorless-talon/src/primitive_target.py

@@ -5,8 +5,13 @@
 
 mod = Module()
 
-BASE_TARGET: dict[str, Any] = {"type": "primitive"}
-IMPLICIT_TARGET = {"type": "implicit"}
+
+def create_base_target() -> dict[str, Any]:
+    return {"type": "primitive"}
+
+
+def create_implicit_target() -> dict[str, Any]:
+    return {"type": "implicit"}
 
 
 @mod.capture(
@@ -17,7 +22,7 @@
 )
 def cursorless_primitive_target(m) -> dict[str, Any]:
     """Supported extents for cursorless navigation"""
-    result = BASE_TARGET.copy()
+    result = create_base_target()
 
     modifiers = [
         *getattr(m, "cursorless_position_list", []),