Migrate text fragment to scm

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

@@ -37,7 +37,7 @@ export function getNodeMatcher(
   const matcher = matchers[scopeTypeType];
 
   if (matcher == null) {
-    return notSupported;
+    return notSupported(scopeTypeType);
   }
 
   if (includeSiblings) {

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

@@ -1,3 +1,7 @@
+import { SimpleScopeTypeType } from "@cursorless/common";
+import type { SyntaxNode } from "web-tree-sitter";
+import { NodeMatcherAlternative } from "../typings/Types";
+import { patternFinder } from "../util/nodeFinders";
 import {
   ancestorChainNodeMatcher,
   argumentMatcher,
@@ -9,11 +13,6 @@ import {
   patternMatcher,
   trailingMatcher,
 } from "../util/nodeMatchers";
-import { NodeMatcherAlternative, SelectionWithEditor } from "../typings/Types";
-import { SimpleScopeTypeType } from "@cursorless/common";
-import type { SyntaxNode } from "web-tree-sitter";
-import { getNodeRange } from "../util/nodeSelectors";
-import { patternFinder } from "../util/nodeFinders";
 
 // Generated by the following command:
 // curl https://raw.githubusercontent.com/tree-sitter/tree-sitter-ruby/1ebfdb288842dae5a9233e2509a135949023dd82/src/node-types.json  \
@@ -161,7 +160,6 @@ const nodeMatchers: Partial<
       1,
     ),
   ),
-  string: "string",
   functionName: ["method[name]", "singleton_method[name]"],
   anonymousFunction: cascadingMatcher(
     patternMatcher("lambda", "do_block"),
@@ -194,14 +192,3 @@ const nodeMatchers: Partial<
   collectionItem: argumentMatcher(...mapTypes, ...listTypes),
 };
 export const patternMatchers = createPatternMatchers(nodeMatchers);
-
-export function stringTextFragmentExtractor(
-  node: SyntaxNode,
-  _selection: SelectionWithEditor,
-) {
-  if (node.type === "string_content" || node.type === "heredoc_content") {
-    return getNodeRange(node);
-  }
-
-  return null;
-}

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

@@ -146,7 +146,6 @@ const nodeMatchers: Partial<
     ],
     1,
   ),
-  string: ["raw_string_literal", "string_literal"],
   condition: cascadingMatcher(
     patternMatcher("while_expression[condition]", "if_expression[condition]"),
     matcher(
@@ -159,7 +158,6 @@ const nodeMatchers: Partial<
   ),
   functionCall: ["call_expression", "macro_invocation", "struct_expression"],
   functionCallee: "call_expression[function]",
-  comment: ["line_comment", "block_comment"],
   list: ["array_expression", "tuple_expression"],
   collectionItem: argumentMatcher(
     "array_expression",

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

@@ -21,9 +21,6 @@ const nodeMatchers: Partial<
     "trait_definition[name]",
   ],
 
-  string: ["interpolated_string_expression", "string"],
-  comment: "comment",
-
   // list.size(), does not count foo.size (field_expression), or foo size (postfix_expression)
   functionCall: "call_expression",
   namedFunction: "function_definition",

packages/cursorless-engine/src/processTargets/modifiers/surroundingPair/index.ts

@@ -1,11 +1,9 @@
 import {
   ComplexSurroundingPairName,
-  Selection,
   SurroundingPairScopeType,
 } from "@cursorless/common";
 import type { SyntaxNode } from "web-tree-sitter";
 import { LanguageDefinitions } from "../../../languages/LanguageDefinitions";
-import getTextFragmentExtractor from "../../../languages/getTextFragmentExtractor";
 import { Target } from "../../../typings/target.types";
 import { SurroundingPairTarget } from "../../targets";
 import { getContainingScopeTarget } from "../getContainingScopeTarget";
@@ -117,21 +115,7 @@ function processSurroundingPairCore(
       return containingScope?.[0].contentRange;
     }
 
-    // Then try to use the legacy text fragment extractor if it exists
-    const textFragmentExtractor = getTextFragmentExtractor(document.languageId);
-
-    if (textFragmentExtractor == null) {
-      // If the text fragment extractor doesn't exist, or if it explicitly is
-      // set to `null`, then we just use text-based algorithm on entire document
-      return document.range;
-    }
-
-    const selectionWithEditor = {
-      editor,
-      selection: new Selection(range.start, range.end),
-    };
-
-    return textFragmentExtractor(node, selectionWithEditor);
+    return document.range;
   })();
 
   if (textFragmentRange != null) {

packages/cursorless-engine/src/util/nodeMatchers.ts

@@ -169,12 +169,11 @@ export function cascadingMatcher(...matchers: NodeMatcher[]): NodeMatcher {
   };
 }
 
-export const notSupported: NodeMatcher = (
-  _selection: SelectionWithEditor,
-  _node: SyntaxNode,
-) => {
-  throw new Error("Node type not supported");
-};
+export function notSupported(scopeTypeType: SimpleScopeTypeType): NodeMatcher {
+  return (_selection: SelectionWithEditor, _node: SyntaxNode) => {
+    throw new Error(`Node type '${scopeTypeType}' not supported`);
+  };
+}
 
 export function createPatternMatchers(
   nodeMatchers: Partial<Record<SimpleScopeTypeType, NodeMatcherAlternative>>,

queries/ruby.scm

@@ -29,3 +29,10 @@
     name: (_) @className
   ) @_.domain
 ) @_.iteration
+
+(string) @string
+
+[
+  (string_content)
+  (heredoc_content)
+] @textFragment

queries/rust.scm

@@ -2,3 +2,13 @@
   (if_expression)
   (if_let_expression)
 ] @ifStatement
+
+[
+  (raw_string_literal)
+  (string_literal)
+] @string @textFragment
+
+[
+  (line_comment)
+  (block_comment)
+] @comment @textFragment

queries/scala.scm

@@ -1 +1,8 @@
 (if_expression) @ifStatement
+
+[
+  (string)
+  (interpolated_string_expression)
+] @string @textFragment
+
+(comment) @comment @textFragment