Migrate java scopes

packages/cursorless-engine/src/languages/getTextFragmentExtractor.ts

@@ -132,10 +132,6 @@ const textFragmentExtractors: Record<
     "html",
     htmlStringTextFragmentExtractor,
   ),
-  java: constructDefaultTextFragmentExtractor(
-    "java",
-    constructHackedStringTextFragmentExtractor("java"),
-  ),
   javascript: constructDefaultTextFragmentExtractor(
     "javascript",
     typescriptStringTextFragmentExtractor,

packages/cursorless-engine/src/languages/java.ts

@@ -2,7 +2,6 @@ import {
   createPatternMatchers,
   argumentMatcher,
   leadingMatcher,
-  conditionMatcher,
   trailingMatcher,
   matcher,
   cascadingMatcher,
@@ -13,61 +12,9 @@ import { patternFinder } from "../util/nodeFinders";
 import { NodeMatcherAlternative } from "../typings/Types";
 import { SimpleScopeTypeType } from "@cursorless/common";
 
-// Generated by the following command:
-// > curl https://raw.githubusercontent.com/tree-sitter/tree-sitter-java/master/src/node-types.json | jq '[.[] | select(.type == "statement" or .type == "declaration") | .subtypes[].type]'
-const STATEMENT_TYPES = [
-  "annotation_type_declaration",
-  "class_declaration",
-  "enum_declaration",
-  "import_declaration",
-  "interface_declaration",
-  "module_declaration",
-  "package_declaration",
-  "assert_statement",
-  "break_statement",
-  "continue_statement",
-  "declaration",
-  "do_statement",
-  "enhanced_for_statement",
-  "expression_statement",
-  "for_statement",
-  "if_statement",
-  "labeled_statement",
-  "local_variable_declaration",
-  "return_statement",
-  "switch_expression",
-  "synchronized_statement",
-  "throw_statement",
-  "try_statement",
-  "try_with_resources_statement",
-  "while_statement",
-  "yield_statement",
-
-  // exceptions
-  // ";",
-  // "block",
-  "switch_statement",
-  "method_declaration",
-  "constructor_declaration",
-  "field_declaration",
-];
-
 const nodeMatchers: Partial<
   Record<SimpleScopeTypeType, NodeMatcherAlternative>
 > = {
-  statement: STATEMENT_TYPES,
-  class: "class_declaration",
-  className: "class_declaration[name]",
-  ifStatement: "if_statement",
-  string: "string_literal",
-  comment: ["line_comment", "block_comment", "comment"],
-  anonymousFunction: "lambda_expression",
-  list: "array_initializer",
-  functionCall: [
-    "method_invocation",
-    "object_creation_expression",
-    "explicit_constructor_invocation",
-  ],
   functionCallee: cascadingMatcher(
     matcher(
       patternFinder("method_invocation"),
@@ -82,14 +29,12 @@ const nodeMatchers: Partial<
       childRangeSelector(["argument_list", ";"], []),
     ),
   ),
-  map: "block",
   name: [
     "*[declarator][name]",
     "assignment_expression[left]",
     "*[name]",
     "formal_parameter.identifier!",
   ],
-  namedFunction: ["method_declaration", "constructor_declaration"],
   type: trailingMatcher([
     "generic_type.type_arguments.type_identifier",
     "generic_type.type_identifier",
@@ -100,10 +45,6 @@ const nodeMatchers: Partial<
     "formal_parameter[type]",
     "method_declaration[type]",
   ]),
-  functionName: [
-    "method_declaration.identifier!",
-    "constructor_declaration.identifier!",
-  ],
   value: leadingMatcher(
     [
       "*[declarator][value]",
@@ -113,10 +54,7 @@ const nodeMatchers: Partial<
     ],
     ["=", "+=", "-=", "*=", "/=", "%=", "&=", "|=", "^=", "<<=", ">>="],
   ),
-  condition: conditionMatcher("*[condition]"),
   argumentOrParameter: argumentMatcher("formal_parameters", "argument_list"),
-  branch: ["switch_block_statement_group", "switch_rule"],
-  switchStatementSubject: "switch_expression[condition][0]",
 };
 
 export default createPatternMatchers(nodeMatchers);

packages/cursorless-vscode-e2e/src/suite/fixtures/recorded/languages/java/changeBranch.yml

@@ -0,0 +1,42 @@
+languageId: java
+command:
+  version: 6
+  spokenForm: change branch
+  action:
+    name: clearAndSetSelection
+    target:
+      type: primitive
+      modifiers:
+        - type: containingScope
+          scopeType: {type: branch}
+  usePrePhraseSnapshot: true
+initialState:
+  documentContents: |
+    public class MyClass {
+      public void myFunk() {
+        if (true) {
+        }
+        else if (false) {
+        }
+        else {
+        }
+      }
+    }
+  selections:
+    - anchor: {line: 2, character: 4}
+      active: {line: 2, character: 4}
+  marks: {}
+finalState:
+  documentContents: |
+    public class MyClass {
+      public void myFunk() {
+
+        else if (false) {
+        }
+        else {
+        }
+      }
+    }
+  selections:
+    - anchor: {line: 2, character: 4}
+      active: {line: 2, character: 4}

packages/cursorless-vscode-e2e/src/suite/fixtures/recorded/languages/java/changeBranch2.yml

@@ -0,0 +1,42 @@
+languageId: java
+command:
+  version: 6
+  spokenForm: change branch
+  action:
+    name: clearAndSetSelection
+    target:
+      type: primitive
+      modifiers:
+        - type: containingScope
+          scopeType: {type: branch}
+  usePrePhraseSnapshot: true
+initialState:
+  documentContents: |
+    public class MyClass {
+      public void myFunk() {
+        if (true) {
+        }
+        else if (false) {
+        }
+        else {
+        }
+      }
+    }
+  selections:
+    - anchor: {line: 4, character: 4}
+      active: {line: 4, character: 4}
+  marks: {}
+finalState:
+  documentContents: |
+    public class MyClass {
+      public void myFunk() {
+        if (true) {
+        }
+
+        else {
+        }
+      }
+    }
+  selections:
+    - anchor: {line: 4, character: 4}
+      active: {line: 4, character: 4}

packages/cursorless-vscode-e2e/src/suite/fixtures/recorded/languages/java/changeBranch3.yml

@@ -0,0 +1,42 @@
+languageId: java
+command:
+  version: 6
+  spokenForm: change branch
+  action:
+    name: clearAndSetSelection
+    target:
+      type: primitive
+      modifiers:
+        - type: containingScope
+          scopeType: {type: branch}
+  usePrePhraseSnapshot: true
+initialState:
+  documentContents: |
+    public class MyClass {
+      public void myFunk() {
+        if (true) {
+        }
+        else if (false) {
+        }
+        else {
+        }
+      }
+    }
+  selections:
+    - anchor: {line: 6, character: 4}
+      active: {line: 6, character: 4}
+  marks: {}
+finalState:
+  documentContents: |
+    public class MyClass {
+      public void myFunk() {
+        if (true) {
+        }
+        else if (false) {
+        }
+
+      }
+    }
+  selections:
+    - anchor: {line: 6, character: 4}
+      active: {line: 6, character: 4}

packages/cursorless-vscode-e2e/src/suite/fixtures/recorded/languages/java/changeBranch4.yml

@@ -0,0 +1,33 @@
+languageId: java
+command:
+  version: 6
+  spokenForm: change branch
+  action:
+    name: clearAndSetSelection
+    target:
+      type: primitive
+      modifiers:
+        - type: containingScope
+          scopeType: {type: branch}
+  usePrePhraseSnapshot: true
+initialState:
+  documentContents: |-
+    public class MyClass {
+        public void myFunk() {
+            true ? 1 : 2
+        }
+    }
+  selections:
+    - anchor: {line: 2, character: 15}
+      active: {line: 2, character: 15}
+  marks: {}
+finalState:
+  documentContents: |-
+    public class MyClass {
+        public void myFunk() {
+            true ?  : 2
+        }
+    }
+  selections:
+    - anchor: {line: 2, character: 15}
+      active: {line: 2, character: 15}

packages/cursorless-vscode-e2e/src/suite/fixtures/recorded/languages/java/changeBranch5.yml

@@ -0,0 +1,33 @@
+languageId: java
+command:
+  version: 6
+  spokenForm: change branch
+  action:
+    name: clearAndSetSelection
+    target:
+      type: primitive
+      modifiers:
+        - type: containingScope
+          scopeType: {type: branch}
+  usePrePhraseSnapshot: true
+initialState:
+  documentContents: |-
+    public class MyClass {
+        public void myFunk() {
+            true ? 1 : 2
+        }
+    }
+  selections:
+    - anchor: {line: 2, character: 19}
+      active: {line: 2, character: 19}
+  marks: {}
+finalState:
+  documentContents: |-
+    public class MyClass {
+        public void myFunk() {
+            true ? 1 : 
+        }
+    }
+  selections:
+    - anchor: {line: 2, character: 19}
+      active: {line: 2, character: 19}

packages/cursorless-vscode-e2e/src/suite/fixtures/recorded/languages/java/changeCondition.yml

@@ -0,0 +1,43 @@
+languageId: java
+command:
+  version: 6
+  spokenForm: change condition
+  action:
+    name: clearAndSetSelection
+    target:
+      type: primitive
+      modifiers:
+        - type: containingScope
+          scopeType: {type: condition}
+  usePrePhraseSnapshot: true
+initialState:
+  documentContents: |
+    public class MyClass {
+      public void myFunk() {
+        if (true) {
+        }
+        else if (false) {
+        }
+        else {
+        }
+      }
+    }
+  selections:
+    - anchor: {line: 2, character: 4}
+      active: {line: 2, character: 4}
+  marks: {}
+finalState:
+  documentContents: |
+    public class MyClass {
+      public void myFunk() {
+        if () {
+        }
+        else if (false) {
+        }
+        else {
+        }
+      }
+    }
+  selections:
+    - anchor: {line: 2, character: 8}
+      active: {line: 2, character: 8}