Bugfix

Author: AndreasArvidsson

data/fixtures/recorded/surroundingPair/textual/changePair2.yml

@@ -0,0 +1,23 @@
+languageId: typescript
+command:
+  version: 7
+  spokenForm: change pair
+  action:
+    name: clearAndSetSelection
+    target:
+      type: primitive
+      modifiers:
+        - type: containingScope
+          scopeType: {type: surroundingPair, delimiter: any}
+  usePrePhraseSnapshot: true
+initialState:
+  documentContents: "[1, 2, \"]\", 3];"
+  selections:
+    - anchor: {line: 0, character: 1}
+      active: {line: 0, character: 1}
+  marks: {}
+finalState:
+  documentContents: ;
+  selections:
+    - anchor: {line: 0, character: 0}
+      active: {line: 0, character: 0}

packages/cursorless-engine/src/processTargets/modifiers/scopeHandlers/SurroundingPairScopeHandler/RangeIterator.ts

@@ -33,7 +33,7 @@ export class RangeIterator<T extends { range: Range }> {
       }
 
       // Search item is before the range. Since the ranges are sorted, we can stop here.
-      if (searchItem.end.isBefore(range.start)) {
+      if (searchItem.end.isBeforeOrEqual(range.start)) {
         return false;
       }