Added scope type for "inside" (#2776)

Inside/interior today is a modifier and will stay so in the future, but
we are also introducing interior as a scope type. This will allow us to
do `"take inside"` when on a if statement, python function and so on.

It now takes the closest of the nearest tree-sitter interior _or_
surrounding pair. This is most useful for Python; in other languages
with {} statement-blocks, the braces would already be easy to target,
but now "take inside" behaves intuitively there as well.

The scm syntax `@scope.interior` have been replaced by `@interior` as
its own scope type.

Fixes #1046

## Checklist

- [x] 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

## Release notes

You can now use "inside" to refer to the inside body of a language scope
(function, if statement, etc). This is especially useful for Python,
because previously you had to think about what scope you were targeting
-- "take inside" now just works.

It also works nicely for targeting the inside of C-style statements that
don't have `{}` braces, like single line conditionals or loops.

In Python, this also means that you can now target the inside of the
function ("take inside") even while the cursor is in a related location
(such as in its name). Note that this behavior currently only exists for
Python; we plan to port it to other languages as well since it is
useful.

---------

Co-authored-by: Phil Cohen <[email protected]>

Author: AndreasArvidsson

cursorless-talon/src/spoken_forms.py

@@ -161,6 +161,7 @@ def handle_new_values(csv_name: str, values: list[SpokenFormEntry]):
                 "textFragment",
                 "disqualifyDelimiter",
                 "pairDelimiter",
+                "interior",
             ],
             default_list_name="scope_type",
         ),

data/fixtures/recorded/implicitExpansion/clearCoreToken.yml

@@ -17,8 +17,4 @@ initialState:
     - anchor: {line: 0, character: 1}
       active: {line: 0, character: 1}
   marks: {}
-finalState:
-  documentContents: ()
-  selections:
-    - anchor: {line: 0, character: 1}
-      active: {line: 0, character: 1}
+thrownError: {name: NoContainingScopeError}

data/fixtures/recorded/modifiers/changeInside.yml

@@ -0,0 +1,26 @@
+languageId: python
+command:
+  version: 7
+  spokenForm: change inside
+  action:
+    name: clearAndSetSelection
+    target:
+      type: primitive
+      modifiers:
+        - {type: interiorOnly}
+  usePrePhraseSnapshot: false
+initialState:
+  documentContents: |-
+    def testing():
+        pass
+  selections:
+    - anchor: {line: 0, character: 0}
+      active: {line: 0, character: 0}
+  marks: {}
+finalState:
+  documentContents: |-
+    def testing():
+        
+  selections:
+    - anchor: {line: 1, character: 4}
+      active: {line: 1, character: 4}

data/fixtures/recorded/modifiers/interior/changeInside.yml

@@ -0,0 +1,22 @@
+languageId: html
+command:
+  version: 7
+  spokenForm: change inside
+  action:
+    name: clearAndSetSelection
+    target:
+      type: primitive
+      modifiers:
+        - {type: interiorOnly}
+  usePrePhraseSnapshot: false
+initialState:
+  documentContents: <div>content</div>
+  selections:
+    - anchor: {line: 0, character: 1}
+      active: {line: 0, character: 1}
+  marks: {}
+finalState:
+  documentContents: <>content</div>
+  selections:
+    - anchor: {line: 0, character: 1}
+      active: {line: 0, character: 1}

data/fixtures/recorded/modifiers/interior/changeInside2.yml

@@ -0,0 +1,22 @@
+languageId: html
+command:
+  version: 7
+  spokenForm: change inside
+  action:
+    name: clearAndSetSelection
+    target:
+      type: primitive
+      modifiers:
+        - {type: interiorOnly}
+  usePrePhraseSnapshot: false
+initialState:
+  documentContents: <div>content</div>
+  selections:
+    - anchor: {line: 0, character: 9}
+      active: {line: 0, character: 9}
+  marks: {}
+finalState:
+  documentContents: <div></div>
+  selections:
+    - anchor: {line: 0, character: 5}
+      active: {line: 0, character: 5}

data/fixtures/recorded/modifiers/interior/changeInside3.yml

@@ -0,0 +1,22 @@
+languageId: plaintext
+command:
+  version: 7
+  spokenForm: change inside
+  action:
+    name: clearAndSetSelection
+    target:
+      type: primitive
+      modifiers:
+        - {type: interiorOnly}
+  usePrePhraseSnapshot: false
+initialState:
+  documentContents: (hello)
+  selections:
+    - anchor: {line: 0, character: 3}
+      active: {line: 0, character: 3}
+  marks: {}
+finalState:
+  documentContents: ()
+  selections:
+    - anchor: {line: 0, character: 1}
+      active: {line: 0, character: 1}

data/fixtures/recorded/modifiers/interior/changeInside4.yml

@@ -0,0 +1,22 @@
+languageId: html
+command:
+  version: 7
+  spokenForm: change inside
+  action:
+    name: clearAndSetSelection
+    target:
+      type: primitive
+      modifiers:
+        - {type: interiorOnly}
+  usePrePhraseSnapshot: false
+initialState:
+  documentContents: <div>content</div> (hello)
+  selections:
+    - anchor: {line: 0, character: 8}
+      active: {line: 0, character: 23}
+  marks: {}
+finalState:
+  documentContents: <div>)
+  selections:
+    - anchor: {line: 0, character: 5}
+      active: {line: 0, character: 5}

data/fixtures/recorded/modifiers/interior/changeInsideElement.yml

@@ -0,0 +1,24 @@
+languageId: html
+command:
+  version: 7
+  spokenForm: change inside element
+  action:
+    name: clearAndSetSelection
+    target:
+      type: primitive
+      modifiers:
+        - {type: interiorOnly}
+        - type: containingScope
+          scopeType: {type: xmlElement}
+  usePrePhraseSnapshot: false
+initialState:
+  documentContents: <div>content</div>
+  selections:
+    - anchor: {line: 0, character: 1}
+      active: {line: 0, character: 1}
+  marks: {}
+finalState:
+  documentContents: <div></div>
+  selections:
+    - anchor: {line: 0, character: 5}
+      active: {line: 0, character: 5}

data/fixtures/scopes/html/element.scope

@@ -1,14 +1,10 @@
-<div id="root"></div>
+<div id="root">text</div>
 ---
 
 [Content] =
 [Removal] =
-[Domain] = 0:0-0:21
-  >---------------------<
-0| <div id="root"></div>
-
-[Interior] = 0:15-0:15
-                 ><
-0| <div id="root"></div>
+[Domain] = 0:0-0:25
+  >-------------------------<
+0| <div id="root">text</div>
 
 [Insertion delimiter] = "\n"

data/fixtures/scopes/html/interior.element.scope

@@ -0,0 +1,40 @@
+<div> text </div>
+---
+
+[#1 Content] =
+[#1 Removal] = 0:1-0:4
+   >---<
+0| <div> text </div>
+
+[#1 Domain] = 0:0-0:5
+  >-----<
+0| <div> text </div>
+
+[#1 Insertion delimiter] = " "
+
+
+[#2 Content] = 0:6-0:10
+        >----<
+0| <div> text </div>
+
+[#2 Removal] = 0:5-0:11
+       >------<
+0| <div> text </div>
+
+[#2 Domain] = 0:0-0:17
+  >-----------------<
+0| <div> text </div>
+
+[#2 Insertion delimiter] = " "
+
+
+[#3 Content] =
+[#3 Removal] = 0:13-0:16
+               >---<
+0| <div> text </div>
+
+[#3 Domain] = 0:11-0:17
+             >------<
+0| <div> text </div>
+
+[#3 Insertion delimiter] = " "