From 6f43bcc1677c0528a3936df9efe2bb5b413a281c Mon Sep 17 00:00:00 2001
From: HactarCE <6060305+HactarCE@users.noreply.github.com>
Date: Mon, 27 Oct 2025 18:17:01 -0400
Subject: [PATCH] fix clippy lints

---
 crates/editor/src/code_context_menus.rs |   1 -
 crates/editor/src/editor.rs             | 122 +++++++++++-------------
 2 files changed, 58 insertions(+), 65 deletions(-)

diff --git a/crates/editor/src/code_context_menus.rs b/crates/editor/src/code_context_menus.rs
index c1dd46ff9c53b222972630c7811a14dbd7da6f83..453ad541f54694daa87d46b9d01a790c1a9c3915 100644
--- a/crates/editor/src/code_context_menus.rs
+++ b/crates/editor/src/code_context_menus.rs
@@ -286,7 +286,6 @@ impl CompletionsMenu {
             .map(|(id, completion)| StringMatchCandidate::new(id, completion.label.filter_text()))
             .into_group_map_by(|candidate| completions[candidate.id].match_start)
             .into_iter()
-            .map(|(k, v)| (k, v))
             .collect();
 
         let completions_menu = Self {
diff --git a/crates/editor/src/editor.rs b/crates/editor/src/editor.rs
index 8af70da9651a6dc5735df6a08a6c110d0aaf7b5f..ca0fe99ccec0c4d1e84fbc666f3641956d62d1e7 100644
--- a/crates/editor/src/editor.rs
+++ b/crates/editor/src/editor.rs
@@ -22959,72 +22959,66 @@ fn snippet_completions(
                 is_incomplete = true;
             }
 
-            completions.extend(
-                matches
-                    .iter()
-                    .filter_map(|(string_match, buffer_window_len)| {
-                        let (snippet_index, matching_prefix, _snippet_word_count) =
-                            sorted_snippet_candidates[string_match.candidate_id];
-                        let snippet = &snippets[snippet_index];
-                        let start = buffer_offset - buffer_window_len;
-                        let start = snapshot.anchor_before(start);
-                        let range = start..buffer_anchor;
-                        let lsp_start = to_lsp(&start);
-                        let lsp_range = lsp::Range {
-                            start: lsp_start,
-                            end: lsp_end,
-                        };
-                        Some(Completion {
-                            replace_range: range,
-                            new_text: snippet.body.clone(),
-                            source: CompletionSource::Lsp {
-                                insert_range: None,
-                                server_id: LanguageServerId(usize::MAX),
-                                resolved: true,
-                                lsp_completion: Box::new(lsp::CompletionItem {
-                                    label: snippet.prefix.first().unwrap().clone(),
-                                    kind: Some(CompletionItemKind::SNIPPET),
-                                    label_details: snippet.description.as_ref().map(
-                                        |description| lsp::CompletionItemLabelDetails {
-                                            detail: Some(description.clone()),
-                                            description: None,
-                                        },
-                                    ),
-                                    insert_text_format: Some(InsertTextFormat::SNIPPET),
-                                    text_edit: Some(lsp::CompletionTextEdit::InsertAndReplace(
-                                        lsp::InsertReplaceEdit {
-                                            new_text: snippet.body.clone(),
-                                            insert: lsp_range,
-                                            replace: lsp_range,
-                                        },
-                                    )),
-                                    filter_text: Some(snippet.body.clone()),
-                                    sort_text: Some(char::MAX.to_string()),
-                                    ..lsp::CompletionItem::default()
-                                }),
-                                lsp_defaults: None,
-                            },
-                            label: CodeLabel {
-                                text: matching_prefix.clone(),
-                                runs: Vec::new(),
-                                filter_range: 0..matching_prefix.len(),
-                            },
-                            icon_path: None,
-                            documentation: Some(
-                                CompletionDocumentation::SingleLineAndMultiLinePlainText {
-                                    single_line: snippet.name.clone().into(),
-                                    plain_text: snippet
-                                        .description
-                                        .clone()
-                                        .map(|description| description.into()),
+            completions.extend(matches.iter().map(|(string_match, buffer_window_len)| {
+                let (snippet_index, matching_prefix, _snippet_word_count) =
+                    sorted_snippet_candidates[string_match.candidate_id];
+                let snippet = &snippets[snippet_index];
+                let start = buffer_offset - buffer_window_len;
+                let start = snapshot.anchor_before(start);
+                let range = start..buffer_anchor;
+                let lsp_start = to_lsp(&start);
+                let lsp_range = lsp::Range {
+                    start: lsp_start,
+                    end: lsp_end,
+                };
+                Completion {
+                    replace_range: range,
+                    new_text: snippet.body.clone(),
+                    source: CompletionSource::Lsp {
+                        insert_range: None,
+                        server_id: LanguageServerId(usize::MAX),
+                        resolved: true,
+                        lsp_completion: Box::new(lsp::CompletionItem {
+                            label: snippet.prefix.first().unwrap().clone(),
+                            kind: Some(CompletionItemKind::SNIPPET),
+                            label_details: snippet.description.as_ref().map(|description| {
+                                lsp::CompletionItemLabelDetails {
+                                    detail: Some(description.clone()),
+                                    description: None,
+                                }
+                            }),
+                            insert_text_format: Some(InsertTextFormat::SNIPPET),
+                            text_edit: Some(lsp::CompletionTextEdit::InsertAndReplace(
+                                lsp::InsertReplaceEdit {
+                                    new_text: snippet.body.clone(),
+                                    insert: lsp_range,
+                                    replace: lsp_range,
                                 },
-                            ),
-                            insert_text_mode: None,
-                            confirm: None,
-                            match_start: Some(start),
-                        })
+                            )),
+                            filter_text: Some(snippet.body.clone()),
+                            sort_text: Some(char::MAX.to_string()),
+                            ..lsp::CompletionItem::default()
+                        }),
+                        lsp_defaults: None,
+                    },
+                    label: CodeLabel {
+                        text: matching_prefix.clone(),
+                        runs: Vec::new(),
+                        filter_range: 0..matching_prefix.len(),
+                    },
+                    icon_path: None,
+                    documentation: Some(CompletionDocumentation::SingleLineAndMultiLinePlainText {
+                        single_line: snippet.name.clone().into(),
+                        plain_text: snippet
+                            .description
+                            .clone()
+                            .map(|description| description.into()),
                     }),
-            );
+                    insert_text_mode: None,
+                    confirm: None,
+                    match_start: Some(start),
+                }
+            }));
         }
 
         Ok(CompletionResponse {