Cherry Pick: [SuperEditor] - Fix: Action tag exception when pressing ESC, then typing characters. (Resolves #2177) (#2232)

Author: github-actions[bot]

super_editor/lib/src/default_editor/text.dart

@@ -1429,7 +1429,7 @@ class RemoveTextAttributionsCommand extends EditCommand {
       nodesAndSelections.putIfAbsent(textNode, () => selectionRange);
     }
 
-    // Add attributions.
+    // Remove attributions.
     for (final entry in nodesAndSelections.entries) {
       for (Attribution attribution in attributions) {
         final node = entry.key;

super_editor/lib/src/default_editor/text_tokenizing/action_tags.dart

@@ -256,7 +256,7 @@ class CancelComposingActionTagCommand extends EditCommand {
       AddTextAttributionsCommand(
         documentRange: textNode.selectionBetween(
           composingToken.indexedTag.startOffset,
-          composingToken.indexedTag.endOffset,
+          composingToken.indexedTag.startOffset + 1,
         ),
         attributions: {actionTagCancelledAttribution},
       ),

super_editor/test/super_editor/text_entry/tagging/action_tags_test.dart

@@ -455,13 +455,13 @@ void main() {
         await tester.placeCaretInParagraph("1", 7);
 
         // Start composing a tag.
-        await tester.typeImeText("/");
+        await tester.typeImeText("/stuff");
 
         // Ensure that we're composing.
         var text = SuperEditorInspector.findTextInComponent("1");
         expect(
           text.getAttributedRange({actionTagComposingAttribution}, 7),
-          const SpanRange(7, 7),
+          const SpanRange(7, 12),
         );
 
         // Cancel composing.
@@ -472,7 +472,7 @@ void main() {
         expect(
           text.getAttributionSpansInRange(
             attributionFilter: (attribution) => attribution == actionTagComposingAttribution,
-            range: const SpanRange(0, 7),
+            range: const SpanRange(0, 12),
           ),
           isEmpty,
         );
@@ -482,33 +482,15 @@ void main() {
         );
 
         // Start typing again.
-        await tester.typeImeText("h");
-
-        // Ensure that we didn't start composing again.
-        text = SuperEditorInspector.findTextInComponent("1");
-        expect(text.text, "before /h");
-        expect(
-          text.getAttributionSpansInRange(
-            attributionFilter: (attribution) => attribution == actionTagComposingAttribution,
-            range: const SpanRange(0, 8),
-          ),
-          isEmpty,
-        );
-        expect(
-          text.getAttributedRange({actionTagCancelledAttribution}, 7),
-          const SpanRange(7, 7),
-        );
-
-        // Add a space, cause the tag to end.
         await tester.typeImeText(" ");
 
-        // Ensure that the cancelled tag wasn't submitted, and didn't start composing again.
+        // Ensure that we didn't start composing again.
         text = SuperEditorInspector.findTextInComponent("1");
-        expect(text.text, "before /h ");
+        expect(text.text, "before /stuff ");
         expect(
           text.getAttributionSpansInRange(
             attributionFilter: (attribution) => attribution == actionTagComposingAttribution,
-            range: const SpanRange(0, 9),
+            range: const SpanRange(0, 13),
           ),
           isEmpty,
         );