Merge branch 'main' into javaItem

AndreasArvidsson · web-flow · commit 17d3117c32a9 · 2024-11-18T08:50:05.000+01:00
diff --git a/cursorless-talon/src/cheatsheet/sections/modifiers.py b/cursorless-talon/src/cheatsheet/sections/modifiers.py
@@ -64,7 +64,7 @@ def get_modifiers():
             "variations": [
                 {
                     "spokenForm": complex_modifiers["extendThroughStartOf"],
-                    "description": "Extend through start of line",
+                    "description": "Extend through start of line/pair",
                 },
                 {
                     "spokenForm": f"{complex_modifiers['extendThroughStartOf']} <modifier>",
@@ -78,7 +78,7 @@ def get_modifiers():
             "variations": [
                 {
                     "spokenForm": complex_modifiers["extendThroughEndOf"],
-                    "description": "Extend through end of line",
+                    "description": "Extend through end of line/pair",
                 },
                 {
                     "spokenForm": f"{complex_modifiers['extendThroughEndOf']} <modifier>",
diff --git a/data/fixtures/recorded/languages/typescript/takeItem.yml b/data/fixtures/recorded/languages/typescript/takeItem.yml
@@ -23,5 +23,5 @@ finalState:
 
     const value = { a: 1, b: 2, c: 3 };
   selections:
-    - anchor: {line: 1, character: 0}
-      active: {line: 1, character: 35}
+    - anchor: {line: 1, character: 6}
+      active: {line: 1, character: 34}
diff --git a/data/fixtures/scopes/dart/class.scope b/data/fixtures/scopes/dart/class.scope
@@ -0,0 +1,15 @@
+class MyClass {
+
+}
+---
+
+[Content] =
+[Removal] =
+[Domain] = 0:0-2:1
+  >---------------
+0| class MyClass {
+1|
+2| }
+   -<
+
+[Insertion delimiter] = "\n\n"
diff --git a/data/fixtures/scopes/dart/className.scope b/data/fixtures/scopes/dart/className.scope
@@ -0,0 +1,29 @@
+class MyClass {
+    
+}
+---
+
+[Content] = 0:6-0:13
+        >-------<
+0| class MyClass {
+
+[Removal] = 0:6-0:14
+        >--------<
+0| class MyClass {
+
+[Leading delimiter] = 0:5-0:6
+       >-<
+0| class MyClass {
+
+[Trailing delimiter] = 0:13-0:14
+               >-<
+0| class MyClass {
+
+[Domain] = 0:0-2:1
+  >---------------
+0| class MyClass {
+1|     
+2| }
+   -<
+
+[Insertion delimiter] = " "
diff --git a/data/fixtures/scopes/dart/ifStatement.scope b/data/fixtures/scopes/dart/ifStatement.scope
@@ -0,0 +1,19 @@
+foo() {
+  if (true) { }
+}
+---
+
+[Content] =
+[Domain] = 1:2-1:15
+    >-------------<
+1|   if (true) { }
+
+[Removal] = 1:0-1:15
+  >---------------<
+1|   if (true) { }
+
+[Leading delimiter] = 1:0-1:2
+  >--<
+1|   if (true) { }
+
+[Insertion delimiter] = "\n"
diff --git a/data/fixtures/scopes/dart/list.scope b/data/fixtures/scopes/dart/list.scope
@@ -0,0 +1,17 @@
+final list = [1,2,3];
+---
+
+[Content] =
+[Domain] = 0:13-0:20
+               >-------<
+0| final list = [1,2,3];
+
+[Removal] = 0:12-0:20
+              >--------<
+0| final list = [1,2,3];
+
+[Leading delimiter] = 0:12-0:13
+              >-<
+0| final list = [1,2,3];
+
+[Insertion delimiter] = " "
diff --git a/data/fixtures/scopes/dart/map.scope b/data/fixtures/scopes/dart/map.scope
@@ -0,0 +1,25 @@
+  final map = {
+    'hi': 1,
+  };
+---
+
+[Content] =
+[Domain] = 0:14-2:3
+                >-
+0|   final map = {
+1|     'hi': 1,
+2|   };
+   ---<
+
+[Removal] = 0:13-2:3
+               >--
+0|   final map = {
+1|     'hi': 1,
+2|   };
+   ---<
+
+[Leading delimiter] = 0:13-0:14
+               >-<
+0|   final map = {
+
+[Insertion delimiter] = " "
diff --git a/data/fixtures/scopes/javascript.core/collectionItem.unenclosed.scope b/data/fixtures/scopes/javascript.core/collectionItem.unenclosed.scope
@@ -0,0 +1,33 @@
+let foo, bar;
+---
+
+[#1 Content] =
+[#1 Domain] = 0:4-0:7
+      >---<
+0| let foo, bar;
+
+[#1 Removal] = 0:4-0:9
+      >-----<
+0| let foo, bar;
+
+[#1 Trailing delimiter] = 0:7-0:9
+         >--<
+0| let foo, bar;
+
+[#1 Insertion delimiter] = ", "
+
+
+[#2 Content] =
+[#2 Domain] = 0:9-0:12
+           >---<
+0| let foo, bar;
+
+[#2 Removal] = 0:7-0:12
+         >-----<
+0| let foo, bar;
+
+[#2 Leading delimiter] = 0:7-0:9
+         >--<
+0| let foo, bar;
+
+[#2 Insertion delimiter] = ", "
diff --git a/data/fixtures/scopes/rust/string.singleLine.scope b/data/fixtures/scopes/rust/string.singleLine.scope
@@ -0,0 +1,30 @@
+"aaa"
+r#"bbb"#
+r##"ccc"##
+---
+
+[#1 Content] =
+[#1 Removal] =
+[#1 Domain] = 0:0-0:5
+  >-----<
+0| "aaa"
+
+[#1 Insertion delimiter] = " "
+
+
+[#2 Content] =
+[#2 Removal] =
+[#2 Domain] = 1:0-1:8
+  >--------<
+1| r#"bbb"#
+
+[#2 Insertion delimiter] = " "
+
+
+[#3 Content] =
+[#3 Removal] =
+[#3 Domain] = 2:0-2:10
+  >----------<
+2| r##"ccc"##
+
+[#3 Insertion delimiter] = " "
diff --git a/data/fixtures/scopes/rust/textFragment.string.singleLine.scope b/data/fixtures/scopes/rust/textFragment.string.singleLine.scope
@@ -0,0 +1,30 @@
+"aaa"
+r#"bbb"#
+r##"ccc"##
+---
+
+[#1 Content] =
+[#1 Removal] =
+[#1 Domain] = 0:1-0:4
+   >---<
+0| "aaa"
+
+[#1 Insertion delimiter] = " "
+
+
+[#2 Content] =
+[#2 Removal] =
+[#2 Domain] = 1:3-1:6
+     >---<
+1| r#"bbb"#
+
+[#2 Insertion delimiter] = " "
+
+
+[#3 Content] =
+[#3 Removal] =
+[#3 Domain] = 2:4-2:7
+      >---<
+2| r##"ccc"##
+
+[#3 Insertion delimiter] = " "
diff --git a/data/playground/dart.dart b/data/playground/dart.dart
@@ -0,0 +1,22 @@
+foo() {
+  final hi = '';
+  if (hi.isNotEmpty) {
+
+    //
+  }
+
+
+  else if (true) {
+    //
+  }
+
+  final list = [1, 2, 3];
+
+  final map = {
+    'hi': 1,
+  };
+}
+
+class Something{
+  
+}
diff --git a/packages/cheatsheet/src/lib/sampleSpokenFormInfos/defaults.json b/packages/cheatsheet/src/lib/sampleSpokenFormInfos/defaults.json
@@ -796,7 +796,7 @@
           "variations": [
             {
               "spokenForm": "tail",
-              "description": "Extend through end of line"
+              "description": "Extend through end of line/pair"
             },
             {
               "spokenForm": "tail <modifier>",
@@ -810,7 +810,7 @@
           "variations": [
             {
               "spokenForm": "head",
-              "description": "Extend through start of line"
+              "description": "Extend through start of line/pair"
             },
             {
               "spokenForm": "head <modifier>",
diff --git a/packages/common/package.json b/packages/common/package.json
@@ -23,7 +23,7 @@
     "@types/js-yaml": "^4.0.9",
     "@types/lodash-es": "4.17.12",
     "@types/mocha": "^10.0.9",
-    "cross-spawn": "7.0.3",
+    "cross-spawn": "7.0.5",
     "fast-check": "3.22.0",
     "js-yaml": "^4.1.0",
     "mocha": "^10.7.3"
diff --git a/packages/common/src/scopeSupportFacets/dart.ts b/packages/common/src/scopeSupportFacets/dart.ts
@@ -0,0 +1,13 @@
+import type { LanguageScopeSupportFacetMap } from "./scopeSupportFacets.types";
+import { ScopeSupportFacetLevel } from "./scopeSupportFacets.types";
+
+// eslint-disable-next-line @typescript-eslint/no-unused-vars
+const { supported, unsupported, notApplicable } = ScopeSupportFacetLevel;
+
+export const dartScopeSupport: LanguageScopeSupportFacetMap = {
+  ifStatement: supported,
+  list: supported,
+  map: supported,
+  class: supported,
+  className: supported,
+};
diff --git a/packages/common/src/scopeSupportFacets/javascript.ts b/packages/common/src/scopeSupportFacets/javascript.ts
@@ -112,6 +112,8 @@ export const javascriptCoreScopeSupport: LanguageScopeSupportFacetMap = {
   "value.return": supported,
   "value.return.lambda": supported,
   "value.field": supported,
+
+  "collectionItem.unenclosed": supported,
 };
 
 export const javascriptJsxScopeSupport: LanguageScopeSupportFacetMap = {
diff --git a/packages/common/src/scopeSupportFacets/languageScopeSupport.ts b/packages/common/src/scopeSupportFacets/languageScopeSupport.ts
@@ -4,6 +4,7 @@ import { clojureScopeSupport } from "./clojure";
 import { cppScopeSupport } from "./cpp";
 import { csharpScopeSupport } from "./csharp";
 import { cssScopeSupport } from "./css";
+import { dartScopeSupport } from "./dart";
 import { goScopeSupport } from "./go";
 import { htmlScopeSupport } from "./html";
 import { javaScopeSupport } from "./java";
@@ -35,6 +36,7 @@ export const languageScopeSupport: StringRecord<LanguageScopeSupportFacetMap> =
     cpp: cppScopeSupport,
     csharp: csharpScopeSupport,
     css: cssScopeSupport,
+    dart: dartScopeSupport,
     go: goScopeSupport,
     html: htmlScopeSupport,
     java: javaScopeSupport,
diff --git a/packages/common/src/scopeSupportFacets/rust.ts b/packages/common/src/scopeSupportFacets/rust.ts
@@ -7,4 +7,6 @@ const { supported, unsupported, notApplicable } = ScopeSupportFacetLevel;
 export const rustScopeSupport: LanguageScopeSupportFacetMap = {
   ifStatement: supported,
   disqualifyDelimiter: supported,
+  "string.singleLine": supported,
+  "textFragment.string.singleLine": supported,
 };
diff --git a/packages/cursorless-engine/src/processTargets/modifiers/ContainingScopeStage.ts b/packages/cursorless-engine/src/processTargets/modifiers/ContainingScopeStage.ts
@@ -45,23 +45,39 @@ export class ContainingScopeStage implements ModifierStage {
         .run(target);
     }
 
-    const containingScope = getContainingScopeTarget(
+    const containingScopes = getContainingScopeTarget(
       target,
       scopeHandler,
       ancestorIndex,
     );
-
-    if (containingScope == null) {
-      if (scopeType.type === "collectionItem") {
-        // For `collectionItem`, fall back to generic implementation
-        return this.modifierStageFactory
+    if (scopeType.type === "collectionItem") {
+      // For `collectionItem`, combine with generic implementation
+      try {
+        const legacyScopes = this.modifierStageFactory
           .getLegacyScopeStage(this.modifier)
           .run(target);
+        if (containingScopes == null) {
+          return legacyScopes;
+        }
+        if (containingScopes.length === 1 && legacyScopes.length === 1) {
+          const containingRange = containingScopes[0].contentRange;
+          const legacyRange = legacyScopes[0].contentRange;
+          if (
+            containingRange.contains(legacyRange) &&
+            !containingRange.isRangeEqual(legacyRange)
+          ) {
+            return legacyScopes;
+          }
+        }
+      } catch (_ex) {
+        // Do nothing
       }
+    }
 
+    if (containingScopes == null) {
       throw new NoContainingScopeError(this.modifier.scopeType.type);
     }
 
-    return containingScope;
+    return containingScopes;
   }
 }
diff --git a/packages/cursorless-org-docs/src/docs/user/README.md b/packages/cursorless-org-docs/src/docs/user/README.md
@@ -310,6 +310,8 @@ The modifiers `"head"` and `"tail"` can be used to expand a target through the b
 - `"take head air"`: selects the mark through to start of the line
 - `"take tail air"`: selects the mark through to the end of the line
 
+When inside a single-line surrounding pair (eg parentheses, brackets, etc) the head/tail modifier will only expand to the interior of that pair instead of the whole line. You can explicitly say `"head line"` or `"tail line"` to get the line behavior.
+
 When followed by a modifier, they will expand their input to the start or end of the given modifier range. For example:
 
 - `"take head funk"`: select from the cursor the start of the containing function
diff --git a/packages/cursorless-org-docs/src/docs/user/languages/dart.mdx b/packages/cursorless-org-docs/src/docs/user/languages/dart.mdx
@@ -0,0 +1,5 @@
+import Language from "./Language";
+
+# dart
+
+<Language languageId="dart"></Language>
diff --git a/packages/test-harness/package.json b/packages/test-harness/package.json
@@ -40,7 +40,7 @@
     "@types/mocha": "^10.0.9",
     "@types/tail": "2.2.3",
     "@vscode/test-electron": "^2.4.1",
-    "cross-spawn": "7.0.3",
+    "cross-spawn": "7.0.5",
     "mocha": "^10.7.3"
   },
   "types": "./out/runners/extensionTestsNeovim.d.ts",
diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml
diff --git a/queries/dart.scm b/queries/dart.scm
diff --git a/queries/javascript.core.scm b/queries/javascript.core.scm
diff --git a/queries/rust.scm b/queries/rust.scm

Original file line number	Diff line number	Diff line change
`@@ -64,7 +64,7 @@ def get_modifiers():`
`64`	`64`	`"variations": [`
`65`	`65`	`{`
`66`	`66`	`"spokenForm": complex_modifiers["extendThroughStartOf"],`
`67`		`- "description": "Extend through start of line",`
	`67`	`+ "description": "Extend through start of line/pair",`
`68`	`68`	`},`
`69`	`69`	`{`
`70`	`70`	`"spokenForm": f"{complex_modifiers['extendThroughStartOf']} <modifier>",`
`@@ -78,7 +78,7 @@ def get_modifiers():`
`78`	`78`	`"variations": [`
`79`	`79`	`{`
`80`	`80`	`"spokenForm": complex_modifiers["extendThroughEndOf"],`
`81`		`- "description": "Extend through end of line",`
	`81`	`+ "description": "Extend through end of line/pair",`
`82`	`82`	`},`
`83`	`83`	`{`
`84`	`84`	`"spokenForm": f"{complex_modifiers['extendThroughEndOf']} <modifier>",`