Item: Ignore unmatched starting delimiter (#1559)

## Checklist

- [x] I have added
[tests](https://www.cursorless.org/docs/contributing/test-case-recorder/)
- [ ] I have updated the
[docs](https://github.com/cursorless-dev/cursorless/tree/main/docs) and
[cheatsheet](https://github.com/cursorless-dev/cursorless/tree/main/cursorless-talon/src/cheatsheet)
- [ ] I have not broken the cheatsheet

---------

Co-authored-by: Pokey Rule <[email protected]>

Author: AndreasArvidsson

packages/cursorless-engine/src/processTargets/modifiers/ItemStage/tokenizeRange.ts

@@ -128,7 +128,11 @@ export function joinLexemesBySkippingMatchingPairs(lexemes: string[]) {
     }
 
     // Starting delimiter found
-    else if (leftToRightMap[lexeme] != null) {
+    // Make sure that there is a matching closing delimiter
+    else if (
+      leftToRightMap[lexeme] != null &&
+      lexemes.indexOf(leftToRightMap[lexeme], index + 1) > -1
+    ) {
       openingDelimiter = lexeme;
       closingDelimiter = leftToRightMap[lexeme];
       delimiterBalance = 1;

packages/cursorless-vscode-e2e/src/suite/fixtures/recorded/itemTextual/clearItem20.yml

@@ -0,0 +1,31 @@
+languageId: plaintext
+command:
+  version: 5
+  spokenForm: clear item
+  action: {name: clearAndSetSelection}
+  targets:
+    - type: primitive
+      modifiers:
+        - type: containingScope
+          scopeType: {type: collectionItem}
+  usePrePhraseSnapshot: true
+initialState:
+  documentContents: |-
+    values = [
+        111, // don't ...
+        222,
+        333
+    ]
+  selections:
+    - anchor: {line: 2, character: 4}
+      active: {line: 2, character: 4}
+  marks: {}
+finalState:
+  documentContents: |-
+    values = [
+        111, ,
+        333
+    ]
+  selections:
+    - anchor: {line: 1, character: 9}
+      active: {line: 1, character: 9}