Merge branch 'main' into migrateNotebookCell

Author: AndreasArvidsson

cursorless-everywhere-talon/cursorless_everywhere_talon.py

@@ -14,6 +14,8 @@
 tag: user.cursorless_everywhere_talon
 """
 
+ctx.tags = ["user.cursorless"]
+
 
 @ctx.action_class("user")
 class UserActions:

cursorless-everywhere-talon/cursorless_everywhere_talon_browser.py

@@ -0,0 +1,84 @@
+from talon import Context, Module, actions
+
+from .cursorless_everywhere_types import (
+    EditorEdit,
+    EditorState,
+    SelectionOffsets,
+)
+
+mod = Module()
+
+mod.tag(
+    "cursorless_everywhere_talon_browser",
+    desc="Enable RPC to browser extension when using cursorless everywhere in Talon",
+)
+
+ctx = Context()
+ctx.matches = r"""
+tag: user.cursorless_everywhere_talon_browser
+"""
+
+RPC_COMMAND = "talonCommand"
+
+
+@ctx.action_class("user")
+class Actions:
+    def cursorless_everywhere_get_editor_state() -> EditorState:
+        command = {
+            "id": "getEditorState",
+        }
+        res = rpc_get(command)
+        if use_fallback(res):
+            return actions.next()
+        return res
+
+    def cursorless_everywhere_set_selections(
+        selections: list[SelectionOffsets],  # pyright: ignore [reportGeneralTypeIssues]
+    ):
+        command = {
+            "id": "setSelections",
+            "selections": [
+                js_object_to_python_dict(s, ["anchor", "active"])
+                for s in js_array_to_python_list(selections)
+            ],
+        }
+        res = rpc_get(command)
+        if use_fallback(res):
+            actions.next(selections)
+
+    def cursorless_everywhere_edit_text(
+        edit: EditorEdit,  # pyright: ignore [reportGeneralTypeIssues]
+    ):
+        command = {
+            "id": "editText",
+            "text": edit["text"],
+            "changes": [
+                js_object_to_python_dict(c, ["text", "rangeOffset", "rangeLength"])
+                for c in js_array_to_python_list(edit["changes"])
+            ],
+        }
+        res = rpc_get(command)
+        if use_fallback(res):
+            actions.next(edit)
+
+
+def rpc_get(command: dict):
+    return actions.user.run_rpc_command_get(RPC_COMMAND, command)
+
+
+def use_fallback(result: dict) -> bool:
+    return result.get("fallback", False)
+
+
+def js_array_to_python_list(array) -> list:
+    result = []
+    for i in range(array.length):
+        result.append(array[i])
+    return result
+
+
+def js_object_to_python_dict(object, keys: list[str]) -> dict:
+    result = {}
+    for key in keys:
+        result[key] = object[key]
+    return result

cursorless-talon/.github/PULL_REQUEST_TEMPLATE.md

@@ -0,0 +1 @@
+Please file pull requests to the cursorless-talon subdirectory in the https://github.com/cursorless-dev/cursorless repo

cursorless-talon/src/check_community_repo.py

@@ -0,0 +1,37 @@
+from talon import app, registry
+
+required_captures = [
+    "number_small",
+    "user.any_alphanumeric_key",
+    "user.formatters",
+    "user.ordinals_small",
+]
+
+required_actions = [
+    "user.homophones_get",
+    "user.reformat_text",
+]
+
+
+def on_ready():
+    missing_captures = [
+        capture for capture in required_captures if capture not in registry.captures
+    ]
+    missing_actions = [
+        action for action in required_actions if action not in registry.actions
+    ]
+    errors = []
+    if missing_captures:
+        errors.append(f"Missing captures: {', '.join(missing_captures)}")
+    if missing_actions:
+        errors.append(f"Missing actions: {', '.join(missing_actions)}")
+    if errors:
+        print("Cursorless community requirements:")
+        print("\n".join(errors))
+        app.notify(
+            "Cursorless: Please install the community repository",
+            body="https://github.com/talonhub/community",
+        )
+
+
+app.register("ready", on_ready)

cursorless-talon/src/cursorless.talon

@@ -45,5 +45,5 @@ tutorial (previous | last): user.private_cursorless_tutorial_previous()
 tutorial restart: user.private_cursorless_tutorial_restart()
 tutorial resume: user.private_cursorless_tutorial_resume()
 tutorial (list | close): user.private_cursorless_tutorial_list()
-tutorial <user.private_cursorless_number_small>:
-    user.private_cursorless_tutorial_start_by_number(private_cursorless_number_small)
+tutorial <number_small>:
+    user.private_cursorless_tutorial_start_by_number(number_small)

cursorless-talon/src/get_grapheme_spoken_form_entries.py

@@ -2,7 +2,6 @@
 import typing
 from collections import defaultdict
 from typing import Iterator, Mapping
-from uu import Error
 
 from talon import app, registry, scope
 
@@ -55,7 +54,7 @@ def generate_lists_from_capture(capture_name) -> Iterator[str]:
     try:
         # NB: [-1] because the last capture is the active one
         rule = registry.captures[capture_name][-1].rule.rule
-    except Error:
+    except Exception:
         app.notify("Error constructing spoken forms for graphemes")
         print(f"Error getting rule for capture {capture_name}")
         return
@@ -86,7 +85,7 @@ def get_id_to_talon_list(list_name: str) -> dict[str, str]:
     try:
         # NB: [-1] because the last list is the active one
         return typing.cast(dict[str, str], registry.lists[list_name][-1]).copy()
-    except Error:
+    except Exception:
         app.notify(f"Error getting list {list_name}")
         return {}
 

cursorless-talon/src/marks/lines_number.py

@@ -31,13 +31,13 @@ class CustomizableTerm:
 
 @mod.capture(
     rule=(
-        "{user.cursorless_line_direction} <user.private_cursorless_number_small> "
-        "[<user.cursorless_range_connective> <user.private_cursorless_number_small>]"
+        "{user.cursorless_line_direction} <number_small> "
+        "[<user.cursorless_range_connective> <number_small>]"
     )
 )
 def cursorless_line_number(m) -> LineNumber:
     direction = directions_map[m.cursorless_line_direction]
-    numbers: list[int] = m.private_cursorless_number_small_list
+    numbers: list[int] = m.number_small_list
     anchor = create_line_number_mark(direction.type, direction.formatter(numbers[0]))
     if len(numbers) > 1:
         active = create_line_number_mark(

cursorless-talon/src/modifiers/ordinal_scope.py

@@ -13,24 +13,24 @@
 @mod.capture(
     rule="<user.ordinals_small> | [<user.ordinals_small>] {user.cursorless_last_modifier}"
 )
-def ordinal_or_last(m) -> int:
+def cursorless_ordinal_or_last(m) -> int:
     """An ordinal or the word 'last'"""
     if m[-1] == "last":
         return -getattr(m, "ordinals_small", 1)
     return m.ordinals_small - 1
 
 
 @mod.capture(
-    rule="<user.ordinal_or_last> [<user.cursorless_range_connective> <user.ordinal_or_last>] <user.cursorless_scope_type>"
+    rule="<user.cursorless_ordinal_or_last> [<user.cursorless_range_connective> <user.cursorless_ordinal_or_last>] <user.cursorless_scope_type>"
 )
 def cursorless_ordinal_range(m) -> dict[str, Any]:
     """Ordinal range"""
     anchor = create_ordinal_scope_modifier(
-        m.cursorless_scope_type, m.ordinal_or_last_list[0]
+        m.cursorless_scope_type, m.cursorless_ordinal_or_last_list[0]
     )
-    if len(m.ordinal_or_last_list) > 1:
+    if len(m.cursorless_ordinal_or_last_list) > 1:
         active = create_ordinal_scope_modifier(
-            m.cursorless_scope_type, m.ordinal_or_last_list[1]
+            m.cursorless_scope_type, m.cursorless_ordinal_or_last_list[1]
         )
         range_connective: RangeConnective = m.cursorless_range_connective
         return {
@@ -47,7 +47,7 @@ def cursorless_ordinal_range(m) -> dict[str, Any]:
     rule=(
         "[{user.cursorless_every_scope_modifier}] "
         "({user.cursorless_first_modifier} | {user.cursorless_last_modifier}) "
-        "<user.private_cursorless_number_small> <user.cursorless_scope_type_plural>"
+        "<number_small> <user.cursorless_scope_type_plural>"
     ),
 )
 def cursorless_first_last(m) -> dict[str, Any]:
@@ -57,13 +57,13 @@ def cursorless_first_last(m) -> dict[str, Any]:
         return create_ordinal_scope_modifier(
             m.cursorless_scope_type_plural,
             0,
-            m.private_cursorless_number_small,
+            m.number_small,
             is_every,
         )
     return create_ordinal_scope_modifier(
         m.cursorless_scope_type_plural,
-        -m.private_cursorless_number_small,
-        m.private_cursorless_number_small,
+        -m.number_small,
+        m.number_small,
         is_every,
     )
 

cursorless-talon/src/modifiers/relative_scope.py

@@ -31,28 +31,28 @@ def cursorless_relative_scope_singular(m) -> dict[str, Any]:
 
 
 @mod.capture(
-    rule="[{user.cursorless_every_scope_modifier}] <user.cursorless_relative_direction> <user.private_cursorless_number_small> <user.cursorless_scope_type_plural>"
+    rule="[{user.cursorless_every_scope_modifier}] <user.cursorless_relative_direction> <number_small> <user.cursorless_scope_type_plural>"
 )
 def cursorless_relative_scope_plural(m) -> dict[str, Any]:
     """Relative previous/next plural scope. `next three funks`"""
     return create_relative_scope_modifier(
         m.cursorless_scope_type_plural,
         1,
-        m.private_cursorless_number_small,
+        m.number_small,
         m.cursorless_relative_direction,
         hasattr(m, "cursorless_every_scope_modifier"),
     )
 
 
 @mod.capture(
-    rule="[{user.cursorless_every_scope_modifier}] <user.private_cursorless_number_small> <user.cursorless_scope_type_plural> [{user.cursorless_forward_backward_modifier}]"
+    rule="[{user.cursorless_every_scope_modifier}] <number_small> <user.cursorless_scope_type_plural> [{user.cursorless_forward_backward_modifier}]"
 )
 def cursorless_relative_scope_count(m) -> dict[str, Any]:
     """Relative count scope. `three funks`"""
     return create_relative_scope_modifier(
         m.cursorless_scope_type_plural,
         0,
-        m.private_cursorless_number_small,
+        m.number_small,
         getattr(m, "cursorless_forward_backward_modifier", "forward"),
         hasattr(m, "cursorless_every_scope_modifier"),
     )

cursorless-talon/src/number_small.py

@@ -1,46 +1,20 @@
 """
-This file allows us to use a custom `number_small` capture.  See #1021 for more
-info.
+DEPRECATED @ 2024-12-21
+This file allows us to use a custom `number_small` capture.  See #1021 for more info.
 """
 
-from talon import Context, Module
+from talon import Module, app, registry
 
 mod = Module()
-mod.tag(
-    "cursorless_custom_number_small",
-    "This tag causes Cursorless to use the global <number_small> capture",
-)
 
-ctx = Context()
-ctx.matches = """
-not tag: user.cursorless_custom_number_small
-"""
-
-
-@mod.capture(rule="<number_small>")
-def private_cursorless_number_small(m) -> int:
-    return m.number_small
-
-
-digit_list = "zero one two three four five six seven eight nine".split()
-teens = "ten eleven twelve thirteen fourteen fifteen sixteen seventeen eighteen nineteen".split()
-tens = "twenty thirty forty fifty sixty seventy eighty ninety".split()
+mod.tag("cursorless_custom_number_small", "DEPRECATED!")
 
-number_small_list = [*digit_list, *teens]
-for ten in tens:
-    number_small_list.append(ten)
-    number_small_list.extend(f"{ten} {digit}" for digit in digit_list[1:])
-number_small_map = {n: i for i, n in enumerate(number_small_list)}
 
-mod.list("private_cursorless_number_small", desc="List of small numbers")
-# FIXME: Remove type ignore once Talon supports list types
-# See https://github.com/talonvoice/talon/issues/654
-ctx.lists["self.private_cursorless_number_small"] = number_small_map.keys()  # pyright: ignore [reportArgumentType]
+def on_ready():
+    if "user.cursorless_custom_number_small" in registry.tags:
+        print(
+            "WARNING tag: 'user.cursorless_custom_number_small' is deprecated and should not be used anymore, as Cursorless now uses community number_small"
+        )
 
 
-@ctx.capture(
-    "user.private_cursorless_number_small",
-    rule="{user.private_cursorless_number_small}",
-)
-def override_private_cursorless_number_small(m) -> int:
-    return number_small_map[m.private_cursorless_number_small]
+app.register("ready", on_ready)