Slight tweak

Author: AndreasArvidsson

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

@@ -6,7 +6,14 @@ import type { Range } from "@cursorless/common";
 export class RangeIterator<T extends { range: Range }> {
   private index = 0;
 
-  constructor(public items: T[]) {}
+  constructor(
+    public items: T[],
+    sortItems = false,
+  ) {
+    if (sortItems) {
+      this.items.sort((a, b) => a.range.start.compareTo(b.range.start));
+    }
+  }
 
   contains(searchItem: Range): boolean {
     return this.advance(searchItem);

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

@@ -27,11 +27,14 @@ export function getDelimiterOccurrences(
     "disqualifyDelimiter",
     "textFragment",
   ]);
-  const disqualifyDelimiters = captures?.disqualifyDelimiter ?? [];
-  const textFragments = captures?.textFragment ?? [];
-
-  const disqualifyDelimitersIterator = new RangeIterator(disqualifyDelimiters);
-  const textFragmentsIterator = new RangeIterator(textFragments);
+  const disqualifyDelimitersIterator = new RangeIterator(
+    captures?.disqualifyDelimiter ?? [],
+    true, // Sort items
+  );
+  const textFragmentsIterator = new RangeIterator(
+    captures?.textFragment ?? [],
+    true,
+  );
 
   const delimiterTextToDelimiterInfoMap = Object.fromEntries(
     individualDelimiters.map((individualDelimiter) => [