Replace `editor::CycleCopilotSuggestions` with `copilot::NextSuggestion`

Antonio Scandurra created

Change summary

assets/keymaps/default.json   | 3 ++-
crates/copilot/src/copilot.rs | 2 +-
crates/editor/src/editor.rs   | 9 ++-------
3 files changed, 5 insertions(+), 9 deletions(-)

Detailed changes

crates/copilot/src/copilot.rs 🔗 

@@ -19,7 +19,7 @@ use util::{
     fs::remove_matching, github::latest_github_release, http::HttpClient, paths, ResultExt,
 };
 
-actions!(copilot, [SignIn, SignOut]);
+actions!(copilot, [SignIn, SignOut, NextSuggestion]);
 
 pub fn init(client: Arc<Client>, cx: &mut MutableAppContext) {
     let copilot = cx.add_model(|cx| Copilot::start(client.http_client(), cx));

crates/editor/src/editor.rs 🔗 

@@ -262,7 +262,6 @@ actions!(
         ToggleSoftWrap,
         RevealInFinder,
         CopyHighlightJson
-        CycleCopilotSuggestions
     ]
 );
 
@@ -391,7 +390,7 @@ pub fn init(cx: &mut MutableAppContext) {
     cx.add_async_action(Editor::rename);
     cx.add_async_action(Editor::confirm_rename);
     cx.add_async_action(Editor::find_all_references);
-    cx.add_action(Editor::cycle_copilot_suggestions);
+    cx.add_action(Editor::next_copilot_suggestion);
 
     hover_popover::init(cx);
     link_go_to_definition::init(cx);
@@ -2801,11 +2800,7 @@ impl Editor {
         Some(())
     }
 
-    fn cycle_copilot_suggestions(
-        &mut self,
-        _: &CycleCopilotSuggestions,
-        cx: &mut ViewContext<Self>,
-    ) {
+    fn next_copilot_suggestion(&mut self, _: &copilot::NextSuggestion, cx: &mut ViewContext<Self>) {
         if self.copilot_state.completions.is_empty() {
             return;
         }