Make range explicit for stored targets (#1529)

Fixes #1418

## Checklist

- [ ] 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

Author: AndreasArvidsson

packages/cursorless-engine/src/core/commandRunner/selectionToStoredTarget.ts

@@ -14,5 +14,5 @@ export const selectionToStoredTarget = (selection: SelectionWithEditor) =>
     editor: selection.editor,
     isReversed: selection.selection.isReversed,
     contentRange: selection.selection,
-    hasExplicitRange: !selection.selection.isEmpty,
+    hasExplicitRange: true,
   });

packages/cursorless-vscode-e2e/src/suite/fixtures/recorded/actions/cloneArgue.yml

@@ -46,9 +46,9 @@ finalState:
       hasExplicitRange: true
 ide:
   flashes:
-    - range:
+    - style: justAdded
+      range:
         type: character
         start: {line: 0, character: 26}
         end: {line: 0, character: 37}
-      style: justAdded
 fullTargets: [{type: primitive, mark: {type: cursor}, modifiers: [{type: containingScope, scopeType: {type: argumentOrParameter}}]}]

packages/cursorless-vscode-e2e/src/suite/fixtures/recorded/actions/cloneToken.yml

@@ -36,9 +36,9 @@ finalState:
       hasExplicitRange: true
 ide:
   flashes:
-    - range:
+    - style: justAdded
+      range:
         type: character
         start: {line: 0, character: 12}
         end: {line: 0, character: 17}
-      style: justAdded
 fullTargets: [{type: primitive, mark: {type: cursor}, modifiers: [{type: containingScope, scopeType: {type: token}}]}]

packages/cursorless-vscode-e2e/src/suite/fixtures/recorded/actions/cloneToken2.yml

@@ -36,9 +36,9 @@ finalState:
       hasExplicitRange: true
 ide:
   flashes:
-    - range:
+    - style: justAdded
+      range:
         type: character
         start: {line: 0, character: 6}
         end: {line: 0, character: 11}
-      style: justAdded
 fullTargets: [{type: primitive, mark: {type: cursor}, modifiers: [{type: containingScope, scopeType: {type: token}}]}]

packages/cursorless-vscode-e2e/src/suite/fixtures/recorded/actions/cloneUpArgue.yml

@@ -46,9 +46,9 @@ finalState:
       hasExplicitRange: true
 ide:
   flashes:
-    - range:
+    - style: justAdded
+      range:
         type: character
         start: {line: 0, character: 13}
         end: {line: 0, character: 24}
-      style: justAdded
 fullTargets: [{type: primitive, mark: {type: cursor}, modifiers: [{type: containingScope, scopeType: {type: argumentOrParameter}}]}]

packages/cursorless-vscode-e2e/src/suite/fixtures/recorded/actions/cloneUpToken.yml

@@ -36,9 +36,9 @@ finalState:
       hasExplicitRange: true
 ide:
   flashes:
-    - range:
+    - style: justAdded
+      range:
         type: character
         start: {line: 0, character: 6}
         end: {line: 0, character: 11}
-      style: justAdded
 fullTargets: [{type: primitive, mark: {type: cursor}, modifiers: [{type: containingScope, scopeType: {type: token}}]}]

packages/cursorless-vscode-e2e/src/suite/fixtures/recorded/actions/cloneUpToken2.yml

@@ -36,9 +36,9 @@ finalState:
       hasExplicitRange: true
 ide:
   flashes:
-    - range:
+    - style: justAdded
+      range:
         type: character
         start: {line: 0, character: 0}
         end: {line: 0, character: 5}
-      style: justAdded
 fullTargets: [{type: primitive, mark: {type: cursor}, modifiers: [{type: containingScope, scopeType: {type: token}}]}]

packages/cursorless-vscode-e2e/src/suite/fixtures/recorded/actions/drinkThis.yml

@@ -30,5 +30,5 @@ finalState:
         start: {line: 1, character: 2}
         end: {line: 1, character: 2}
       isReversed: false
-      hasExplicitRange: false
+      hasExplicitRange: true
 fullTargets: [{type: primitive, mark: {type: cursor}, modifiers: []}]

packages/cursorless-vscode-e2e/src/suite/fixtures/recorded/actions/insertEmptyLines/dropThis.yml

@@ -25,9 +25,9 @@ finalState:
         start: {line: 1, character: 0}
         end: {line: 1, character: 0}
       isReversed: false
-      hasExplicitRange: false
+      hasExplicitRange: true
 ide:
   flashes:
-    - range: {type: line, start: 0, end: 0}
-      style: justAdded
+    - style: justAdded
+      range: {type: line, start: 0, end: 0}
 fullTargets: [{type: primitive, mark: {type: cursor}, modifiers: []}]

packages/cursorless-vscode-e2e/src/suite/fixtures/recorded/actions/insertEmptyLines/dropThis10.yml

@@ -29,9 +29,9 @@ finalState:
         start: {line: 2, character: 0}
         end: {line: 2, character: 0}
       isReversed: false
-      hasExplicitRange: false
+      hasExplicitRange: true
 ide:
   flashes:
-    - range: {type: line, start: 1, end: 1}
-      style: justAdded
+    - style: justAdded
+      range: {type: line, start: 1, end: 1}
 fullTargets: [{type: primitive, mark: {type: cursor}, modifiers: []}]