Move default spoken forms to json file (#1738)

` 2713 passing (6s)`

Fixes #1737

## 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)
- [x] I have not broken the cheatsheet

---------

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

Author: 3 people

cursorless-talon/src/actions/actions.py

@@ -1,20 +1,62 @@
-from talon import Module, actions, app
+from talon import Module, actions
 
-from ..csv_overrides import init_csv_and_watch_changes
 from ..targets.target_types import CursorlessTarget, ImplicitDestination
-from .actions_callback import callback_action_defaults, callback_action_map
-from .actions_simple import (
-    no_wait_actions,
-    no_wait_actions_post_sleep,
-    simple_action_defaults,
-)
 from .bring_move import BringMoveTargets
+from .call import cursorless_call_action
 from .execute_command import cursorless_execute_command_action
+from .homophones import cursorless_homophones_action
 
 mod = Module()
 
 
-mod.list("cursorless_experimental_action", "Experimental actions")
+mod.list(
+    "cursorless_simple_action",
+    desc="Cursorless internal: simple actions",
+)
+
+mod.list(
+    "cursorless_callback_action",
+    desc="Cursorless internal: actions implemented via a callback function",
+)
+
+mod.list(
+    "cursorless_custom_action",
+    desc="Cursorless internal: user-defined custom actions",
+)
+
+mod.list(
+    "cursorless_experimental_action",
+    desc="Cursorless internal: experimental actions",
+)
+
+ACTION_LIST_NAMES = [
+    "simple_action",
+    "callback_action",
+    "paste_action",
+    "bring_move_action",
+    "swap_action",
+    "wrap_action",
+    "insert_snippet_action",
+    "reformat_action",
+    "experimental_action",
+]
+
+callback_actions = {
+    "callAsFunction": cursorless_call_action,
+    "findInDocument": actions.user.private_cursorless_find,
+    "nextHomophone": cursorless_homophones_action,
+}
+
+# Don't wait for these actions to finish, usually because they hang on some kind of user interaction
+no_wait_actions = [
+    "generateSnippet",
+    "rename",
+]
+
+# These are actions that we don't wait for, but still want to have a post action sleep
+no_wait_actions_post_sleep = {
+    "rename": 0.3,
+}
 
 
 @mod.capture(
@@ -42,8 +84,8 @@ def cursorless_action_or_ide_command(m) -> dict:
 class Actions:
     def cursorless_command(action_name: str, target: CursorlessTarget):
         """Perform cursorless command on target"""
-        if action_name in callback_action_map:
-            callback_action_map[action_name](target)
+        if action_name in callback_actions:
+            callback_actions[action_name](target)
         elif action_name in ["replaceWithTarget", "moveToTarget"]:
             actions.user.cursorless_bring_move(
                 action_name, BringMoveTargets(target, ImplicitDestination())
@@ -79,33 +121,3 @@ def private_cursorless_action_or_ide_command(
             actions.user.cursorless_command(value, target)
         elif type == "ide_command":
             actions.user.cursorless_ide_command(value, target)
-
-
-default_values = {
-    "simple_action": simple_action_defaults,
-    "callback_action": callback_action_defaults,
-    "paste_action": {"paste": "pasteFromClipboard"},
-    "bring_move_action": {"bring": "replaceWithTarget", "move": "moveToTarget"},
-    "swap_action": {"swap": "swapTargets"},
-    "wrap_action": {"wrap": "wrapWithPairedDelimiter", "repack": "rewrap"},
-    "insert_snippet_action": {"snippet": "insertSnippet"},
-    "reformat_action": {"format": "applyFormatter"},
-}
-
-
-ACTION_LIST_NAMES = list(default_values.keys()) + ["experimental_action"]
-
-
-def on_ready() -> None:
-    init_csv_and_watch_changes("actions", default_values)
-    init_csv_and_watch_changes(
-        "experimental/experimental_actions",
-        {
-            "experimental_action": {
-                "-from": "experimental.setInstanceReference",
-            }
-        },
-    )
-
-
-app.register("ready", on_ready)

cursorless-talon/src/actions/actions_callback.py

@@ -1,6 +1,5 @@
 from talon import Module, actions
 
-from ..paired_delimiter import PairedDelimiter
 from ..targets.target_types import CursorlessTarget
 
 mod = Module()
@@ -11,7 +10,7 @@
 @mod.action_class
 class Actions:
     def private_cursorless_wrap_with_paired_delimiter(
-        action_name: str, target: CursorlessTarget, paired_delimiter: PairedDelimiter
+        action_name: str, target: CursorlessTarget, paired_delimiter: list[str]
     ):
         """Execute Cursorless wrap/rewrap with paired delimiter action"""
         if action_name == "rewrap":
@@ -20,8 +19,8 @@ def private_cursorless_wrap_with_paired_delimiter(
         actions.user.private_cursorless_command_and_wait(
             {
                 "name": action_name,
-                "left": paired_delimiter.left,
-                "right": paired_delimiter.right,
+                "left": paired_delimiter[0],
+                "right": paired_delimiter[1],
                 "target": target,
             }
         )

cursorless-talon/src/actions/actions_custom.py

@@ -2,19 +2,17 @@
 from dataclasses import dataclass
 from typing import Any
 
-from talon import Context, Module
+from talon import Module
 
 from ..targets.range_target import RangeConnective
 
 mod = Module()
-ctx = Context()
 
 mod.list("cursorless_line_direction", desc="Supported directions for line modifier")
 
 
 @dataclass
 class CustomizableTerm:
-    defaultSpokenForm: str
     cursorlessIdentifier: str
     type: str
     formatter: Callable
@@ -23,17 +21,12 @@ class CustomizableTerm:
 # NOTE: Please do not change these dicts.  Use the CSVs for customization.
 # See https://www.cursorless.org/docs/user/customization/
 directions = [
-    CustomizableTerm(
-        "row", "lineNumberModulo100", "modulo100", lambda number: number - 1
-    ),
-    CustomizableTerm("up", "lineNumberRelativeUp", "relative", lambda number: -number),
-    CustomizableTerm(
-        "down", "lineNumberRelativeDown", "relative", lambda number: number
-    ),
+    CustomizableTerm("lineNumberModulo100", "modulo100", lambda number: number - 1),
+    CustomizableTerm("lineNumberRelativeUp", "relative", lambda number: -number),
+    CustomizableTerm("lineNumberRelativeDown", "relative", lambda number: number),
 ]
 
 directions_map = {d.cursorlessIdentifier: d for d in directions}
-DEFAULT_DIRECTIONS = {d.defaultSpokenForm: d.cursorlessIdentifier for d in directions}
 
 
 @mod.capture(