Added iteration scope for python arguments (#2757)

Fixes #2729

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

Author: AndreasArvidsson

data/fixtures/scopes/python/name.argument.actual.iteration.scope

@@ -0,0 +1,16 @@
+aaa(bbb=0, ccc=1)
+---
+
+[#1 Range] =
+[#1 Domain] = 0:0-0:17
+  >-----------------<
+0| aaa(bbb=0, ccc=1)
+
+
+[#2 Range] = 0:4-0:16
+      >------------<
+0| aaa(bbb=0, ccc=1)
+
+[#2 Domain] = 0:3-0:17
+     >--------------<
+0| aaa(bbb=0, ccc=1)

data/fixtures/scopes/python/name.argument.actual.scope

@@ -0,0 +1,29 @@
+aaa(bbb=0, ccc=1)
+---
+
+[#1 Content] =
+[#1 Removal] = 0:4-0:7
+      >---<
+0| aaa(bbb=0, ccc=1)
+
+[#1 Domain] = 0:4-0:9
+      >-----<
+0| aaa(bbb=0, ccc=1)
+
+[#1 Insertion delimiter] = " "
+
+
+[#2 Content] =
+[#2 Removal] = 0:11-0:14
+             >---<
+0| aaa(bbb=0, ccc=1)
+
+[#2 Leading delimiter] = 0:10-0:11
+            >-<
+0| aaa(bbb=0, ccc=1)
+
+[#2 Domain] = 0:11-0:16
+             >-----<
+0| aaa(bbb=0, ccc=1)
+
+[#2 Insertion delimiter] = " "

data/fixtures/scopes/python/value.argument.actual.iteration.scope

@@ -0,0 +1,16 @@
+aaa(bbb=0, ccc=1)
+---
+
+[#1 Range] =
+[#1 Domain] = 0:0-0:17
+  >-----------------<
+0| aaa(bbb=0, ccc=1)
+
+
+[#2 Range] = 0:4-0:16
+      >------------<
+0| aaa(bbb=0, ccc=1)
+
+[#2 Domain] = 0:3-0:17
+     >--------------<
+0| aaa(bbb=0, ccc=1)

data/fixtures/scopes/python/value.argument.actual.scope

@@ -0,0 +1,39 @@
+aaa(bbb=0, ccc=1)
+---
+
+[#1 Content] = 0:8-0:9
+          >-<
+0| aaa(bbb=0, ccc=1)
+
+[#1 Removal] = 0:7-0:9
+         >--<
+0| aaa(bbb=0, ccc=1)
+
+[#1 Leading delimiter] = 0:7-0:8
+         >-<
+0| aaa(bbb=0, ccc=1)
+
+[#1 Domain] = 0:4-0:9
+      >-----<
+0| aaa(bbb=0, ccc=1)
+
+[#1 Insertion delimiter] = " "
+
+
+[#2 Content] = 0:15-0:16
+                 >-<
+0| aaa(bbb=0, ccc=1)
+
+[#2 Removal] = 0:14-0:16
+                >--<
+0| aaa(bbb=0, ccc=1)
+
+[#2 Leading delimiter] = 0:14-0:15
+                >-<
+0| aaa(bbb=0, ccc=1)
+
+[#2 Domain] = 0:11-0:16
+             >-----<
+0| aaa(bbb=0, ccc=1)
+
+[#2 Insertion delimiter] = " "

packages/common/src/scopeSupportFacets/python.ts

@@ -7,10 +7,16 @@ export const pythonScopeSupport: LanguageScopeSupportFacetMap = {
   "name.foreach": supported,
   "name.resource": supported,
   "name.resource.iteration": supported,
+  "name.argument.actual": supported,
+  "name.argument.actual.iteration": supported,
+
   "value.foreach": supported,
   "value.yield": supported,
   "value.resource": supported,
   "value.resource.iteration": supported,
+  "value.argument.actual": supported,
+  "value.argument.actual.iteration": supported,
+
   namedFunction: supported,
   anonymousFunction: supported,
   disqualifyDelimiter: supported,

packages/common/src/scopeSupportFacets/scopeSupportFacetInfos.ts

@@ -442,6 +442,16 @@ export const scopeSupportFacetInfos: Record<
     scopeType: "name",
     isIteration: true,
   },
+  "name.argument.actual": {
+    description: "The name of a (keyword) argument in a function call",
+    scopeType: "name",
+  },
+  "name.argument.actual.iteration": {
+    description:
+      "Iteration scope of the names of the actual parameters of a function call; should be the whole arguments list",
+    scopeType: "name",
+    isIteration: true,
+  },
   "name.argument.formal": {
     description: "The name of a parameter in a function declaration",
     scopeType: "name",
@@ -556,7 +566,7 @@ export const scopeSupportFacetInfos: Record<
     isIteration: true,
   },
   "value.argument.formal": {
-    description: "The value of a parameter in a function declaration",
+    description: "The value of a (keyword) argument in a function declaration",
     scopeType: "value",
   },
   "value.argument.formal.iteration": {
@@ -565,6 +575,16 @@ export const scopeSupportFacetInfos: Record<
     scopeType: "value",
     isIteration: true,
   },
+  "value.argument.actual": {
+    description: "The value of a argument in a function call",
+    scopeType: "value",
+  },
+  "value.argument.actual.iteration": {
+    description:
+      "Iteration scope of the values of the actual parameters of a function call; should be the whole arguments list",
+    scopeType: "value",
+    isIteration: true,
+  },
   "value.argument.formal.method": {
     description: "The value of a parameter in a class method declaration",
     scopeType: "value",

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

@@ -115,6 +115,8 @@ export const scopeSupportFacets = [
   "name.field",
   "name.resource",
   "name.resource.iteration",
+  "name.argument.actual",
+  "name.argument.actual.iteration",
   "name.argument.formal",
   "name.argument.formal.iteration",
   "name.argument.formal.method",
@@ -141,6 +143,8 @@ export const scopeSupportFacets = [
   "value.yield",
   "value.resource",
   "value.resource.iteration",
+  "value.argument.actual",
+  "value.argument.actual.iteration",
   "value.argument.formal",
   "value.argument.formal.iteration",
   "value.argument.formal.method",

queries/python.scm

@@ -626,9 +626,9 @@
 ) @argumentOrParameter.iteration.domain
 
 (argument_list
-  "(" @argumentOrParameter.iteration.start.endOf
-  ")" @argumentOrParameter.iteration.end.startOf
-) @argumentOrParameter.iteration.domain
+  "(" @argumentOrParameter.iteration.start.endOf @name.iteration.start.endOf @value.iteration.start.endOf
+  ")" @argumentOrParameter.iteration.end.startOf @name.iteration.end.startOf @value.iteration.end.startOf
+) @argumentOrParameter.iteration.domain @name.iteration.domain @value.iteration.domain
 
 (call
   (generator_expression