Apply single completion without query

Author: VAWVAW

crates/modalkit/src/editing/buffer/complete.rs

@@ -105,6 +105,11 @@ where
             let adjs = list.replace(&mut self.text, val);
             list.cursor.adjust(&adjs);
             self._adjust_all(adjs, store);
+
+            // apply single completion instantly
+            if list.candidates.len() == 1 {
+                return Ok(None);
+            }
         }
 
         self.completions.insert(gid, list);
@@ -158,6 +163,11 @@ where
             let adjs = list.replace(&mut self.text, val);
             list.cursor.adjust(&adjs);
             self._adjust_all(adjs, store);
+
+            // apply single completion instantly
+            if list.candidates.len() == 1 {
+                return Ok(None);
+            }
         }
 
         self.completions.insert(gid, list);
@@ -219,6 +229,11 @@ where
             let adjs = list.replace(&mut self.text, val);
             list.cursor.adjust(&adjs);
             self._adjust_all(adjs, store);
+
+            // apply single completion instantly
+            if list.candidates.len() == 1 {
+                return Ok(None);
+            }
         }
 
         self.completions.insert(gid, list);
@@ -280,6 +295,11 @@ where
             let adjs = list.replace(&mut self.text, val);
             list.cursor.adjust(&adjs);
             self._adjust_all(adjs, store);
+
+            // apply single completion instantly
+            if list.candidates.len() == 1 {
+                return Ok(None);
+            }
         }
 
         self.completions.insert(gid, list);
@@ -664,21 +684,11 @@ mod tests {
         assert_eq!(ebuf1.get_text(), "foo bar\nbar\n");
         assert_eq!(ebuf1.get_leader(gid), Cursor::new(0, 7));
 
-        // Completing again returns to "b".
-        ebuf1
-            .complete_word(
-                &CompletionStyle::List(next),
-                &CompletionScope::Buffer,
-                &CompletionDisplay::None,
-                ctx!(gid, vwctx, vctx),
-                &mut store,
-            )
-            .unwrap();
-        assert_eq!(ebuf1.get_text(), "foo b\nbar\n");
-        assert_eq!(ebuf1.get_leader(gid), Cursor::new(0, 5));
+        // A single completion is directly applied
+        assert!(!ebuf1.completions.contains_key(&gid));
 
-        // Clear completion list.
-        ebuf1.completions.remove(&gid);
+        type_char!(ebuf1, ' ', gid, vwctx, vctx, store);
+        type_char!(ebuf1, 'b', gid, vwctx, vctx, store);
 
         // Completion with Global scope first yields "bar".
         ebuf1
@@ -690,8 +700,8 @@ mod tests {
                 &mut store,
             )
             .unwrap();
-        assert_eq!(ebuf1.get_text(), "foo bar\nbar\n");
-        assert_eq!(ebuf1.get_leader(gid), Cursor::new(0, 7));
+        assert_eq!(ebuf1.get_text(), "foo bar bar\nbar\n");
+        assert_eq!(ebuf1.get_leader(gid), Cursor::new(0, 11));
 
         // Doing it again yields "baz".
         ebuf1
@@ -703,8 +713,8 @@ mod tests {
                 &mut store,
             )
             .unwrap();
-        assert_eq!(ebuf1.get_text(), "foo baz\nbar\n");
-        assert_eq!(ebuf1.get_leader(gid), Cursor::new(0, 7));
+        assert_eq!(ebuf1.get_text(), "foo bar baz\nbar\n");
+        assert_eq!(ebuf1.get_leader(gid), Cursor::new(0, 11));
 
         // And then returns to "b".
         ebuf1
@@ -716,8 +726,8 @@ mod tests {
                 &mut store,
             )
             .unwrap();
-        assert_eq!(ebuf1.get_text(), "foo b\nbar\n");
-        assert_eq!(ebuf1.get_leader(gid), Cursor::new(0, 5));
+        assert_eq!(ebuf1.get_text(), "foo bar b\nbar\n");
+        assert_eq!(ebuf1.get_leader(gid), Cursor::new(0, 9));
 
         // Go backwards to "baz".
         ebuf1
@@ -729,7 +739,7 @@ mod tests {
                 &mut store,
             )
             .unwrap();
-        assert_eq!(ebuf1.get_text(), "foo baz\nbar\n");
-        assert_eq!(ebuf1.get_leader(gid), Cursor::new(0, 7));
+        assert_eq!(ebuf1.get_text(), "foo bar baz\nbar\n");
+        assert_eq!(ebuf1.get_leader(gid), Cursor::new(0, 11));
     }
 }