Add support for yield statements for "value" in Python (#2215)

Fixes #2214

## 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

packages/common/src/scopeSupportFacets/python.ts

@@ -12,6 +12,7 @@ export const pythonScopeSupport: LanguageScopeSupportFacetMap = {
   "name.resource": supported,
   "name.resource.iteration": supported,
   "value.foreach": supported,
+  "value.yield": supported,
   "value.resource": supported,
   "value.resource.iteration": supported,
 

packages/common/src/scopeSupportFacets/scopeSupportFacetInfos.ts

@@ -299,6 +299,10 @@ export const scopeSupportFacetInfos: Record<
     description: "Value (RHS) of a field in a class / interface",
     scopeType: "value",
   },
+  "value.yield": {
+    description: "Value of a yield statement",
+    scopeType: "value",
+  },
   "value.resource": {
     description: "Value of a 'with' / 'use' / 'using' statement",
     scopeType: "value",

packages/common/src/scopeSupportFacets/scopeSupportFacets.types.ts

@@ -78,6 +78,7 @@ const scopeSupportFacets = [
   "value.return",
   "value.return.lambda",
   "value.field",
+  "value.yield",
   "value.resource",
   "value.resource.iteration",
 

packages/cursorless-vscode-e2e/src/suite/fixtures/scopes/python/value.yield.scope

@@ -0,0 +1,21 @@
+def aaa():
+    yield bbb
+---
+
+[Content] = 1:10-1:13
+1|     yield bbb
+            >---<
+
+[Removal] = 1:9-1:13
+1|     yield bbb
+           >----<
+
+[Leading delimiter] = 1:9-1:10
+1|     yield bbb
+           >-<
+
+[Domain] = 1:4-1:13
+1|     yield bbb
+      >---------<
+
+[Insertion delimiter] = " "

queries/python.scm

@@ -142,6 +142,18 @@
   (_) @value
 ) @_.domain
 
+;;!! yield 1
+;;!        ^
+;;!       xx
+;;!  -------
+;;
+;; NOTE: in tree-sitter, both "yield" and the "1" are children of `yield` but
+;; "yield" is anonymous whereas "1" is named node, so no need to exclude
+;; explicitly
+(yield
+  (_) @value
+) @_.domain
+
 ;;!! with aaa:
 ;;!       ^^^
 ;;!  --------