snippet edit support no longer experimental

Author: BenjaminBrienen

Cargo.lock

@@ -1,12 +1,7 @@
 //! Settings for tweaking assists.
-//!
-//! The fun thing here is `SnippetCap` -- this type can only be created in this
-//! module, and we use to statically check that we only produce snippet
-//! assists if we are allowed to.
 
 use hir::FindPathConfig;
 use ide_db::{
-    SnippetCap,
     assists::ExprFillDefaultMode,
     imports::{import_assets::ImportPathConfig, insert_use::InsertUseConfig},
     rename::RenameConfig,
@@ -16,7 +11,7 @@ use crate::AssistKind;
 
 #[derive(Clone, Debug, PartialEq, Eq)]
 pub struct AssistConfig {
-    pub snippet_cap: Option<SnippetCap>,
+    pub snippet_edit_support: bool,
     pub allowed: Option<Vec<AssistKind>>,
     pub insert_use: InsertUseConfig,
     pub prefer_no_std: bool,

crates/ide-assists/src/assist_config.rs

@@ -53,8 +53,8 @@ pub(crate) fn add_label_to_loop(acc: &mut Assists, ctx: &AssistContext<'_>) -> O
             ];
             editor.insert_all(Position::before(&loop_kw), elements);
 
-            if let Some(cap) = ctx.config.snippet_cap {
-                editor.add_annotation(label.syntax(), builder.make_placeholder_snippet(cap));
+            if ctx.config.snippet_edit_support {
+                editor.add_annotation(label.syntax(), builder.make_placeholder_snippet());
             }
 
             let loop_body = loop_expr.loop_body().and_then(|it| it.stmt_list());
@@ -94,8 +94,8 @@ fn insert_label_after_token(
     let elements = vec![make.whitespace(" ").into(), label.syntax().clone().into()];
     editor.insert_all(Position::after(token), elements);
 
-    if let Some(cap) = ctx.config.snippet_cap {
-        editor.add_annotation(label.syntax(), builder.make_placeholder_snippet(cap));
+    if ctx.config.snippet_edit_support {
+        editor.add_annotation(label.syntax(), builder.make_placeholder_snippet());
     }
 }
 

crates/ide-assists/src/handlers/add_label_to_loop.rs

@@ -200,7 +200,7 @@ fn add_missing_impl_members_inner(
             first_new_item = assoc_item_list.assoc_items().next();
         }
 
-        if let Some(cap) = ctx.config.snippet_cap {
+        if ctx.config.snippet_edit_support {
             let mut placeholder = None;
             if let DefaultMethods::No = mode
                 && let Some(ast::AssocItem::Fn(func)) = &first_new_item
@@ -211,10 +211,10 @@ fn add_missing_impl_members_inner(
             }
 
             if let Some(macro_call) = placeholder {
-                let placeholder = edit.make_placeholder_snippet(cap);
+                let placeholder = edit.make_placeholder_snippet();
                 editor.add_annotation(macro_call.syntax(), placeholder);
             } else if let Some(first_new_item) = first_new_item {
-                let tabstop = edit.make_tabstop_before(cap);
+                let tabstop = edit.make_tabstop_before();
                 editor.add_annotation(first_new_item.syntax(), tabstop);
             };
         };

crates/ide-assists/src/handlers/add_missing_impl_members.rs

@@ -278,22 +278,22 @@ pub(crate) fn add_missing_match_arms(acc: &mut Assists, ctx: &AssistContext<'_>)
             arms_edit.add_comma_after_last_arm(ctx, &make, &editor);
             arms_edit.append_arms(&missing_arms, &make, &editor);
 
-            if let Some(cap) = ctx.config.snippet_cap {
+            if ctx.config.snippet_edit_support {
                 if let Some(it) = missing_arms
                     .first()
                     .and_then(|arm| arm.syntax().descendants().find_map(ast::WildcardPat::cast))
                 {
-                    editor.add_annotation(it.syntax(), builder.make_placeholder_snippet(cap));
+                    editor.add_annotation(it.syntax(), builder.make_placeholder_snippet());
                 }
 
                 for arm in &missing_arms {
                     if let Some(expr) = arm.expr() {
-                        editor.add_annotation(expr.syntax(), builder.make_placeholder_snippet(cap));
+                        editor.add_annotation(expr.syntax(), builder.make_placeholder_snippet());
                     }
                 }
 
                 if let Some(arm) = missing_arms.last() {
-                    editor.add_annotation(arm.syntax(), builder.make_tabstop_after(cap));
+                    editor.add_annotation(arm.syntax(), builder.make_tabstop_after());
                 }
             }
 

crates/ide-assists/src/handlers/add_missing_match_arms.rs

@@ -112,10 +112,10 @@ pub(crate) fn add_turbo_fish(acc: &mut Assists, ctx: &AssistContext<'_>) -> Opti
                             placeholder_ty.syntax().clone().into(),
                         ];
                         editor.insert_all(Position::after(pat.syntax()), elements);
-                        if let Some(cap) = ctx.config.snippet_cap {
+                        if ctx.config.snippet_edit_support {
                             editor.add_annotation(
                                 placeholder_ty.syntax(),
-                                builder.make_placeholder_snippet(cap),
+                                builder.make_placeholder_snippet(),
                             );
                         }
                     }
@@ -173,9 +173,9 @@ pub(crate) fn add_turbo_fish(acc: &mut Assists, ctx: &AssistContext<'_>) -> Opti
                 }
             };
 
-            if let Some(cap) = ctx.config.snippet_cap {
+            if ctx.config.snippet_edit_support {
                 for arg in fish_head.generic_args() {
-                    editor.add_annotation(arg.syntax(), builder.make_placeholder_snippet(cap));
+                    editor.add_annotation(arg.syntax(), builder.make_placeholder_snippet());
                 }
             }
             builder.add_file_edits(ctx.vfs_file_id(), editor);

crates/ide-assists/src/handlers/add_turbo_fish.rs

@@ -94,11 +94,11 @@ pub(crate) fn convert_from_to_tryfrom(acc: &mut Assists, ctx: &AssistContext<'_>
                 make.ty_alias(None, "Error", None, None, None, Some((make.ty("()"), None)));
             let error_type = ast::AssocItem::TypeAlias(error_type_alias);
 
-            if let Some(cap) = ctx.config.snippet_cap
+            if ctx.config.snippet_edit_support
                 && let ast::AssocItem::TypeAlias(type_alias) = &error_type
                 && let Some(ty) = type_alias.ty()
             {
-                let placeholder = builder.make_placeholder_snippet(cap);
+                let placeholder = builder.make_placeholder_snippet();
                 editor.add_annotation(ty.syntax(), placeholder);
             }
 

crates/ide-assists/src/handlers/convert_from_to_tryfrom.rs

@@ -192,10 +192,10 @@ fn edit_tuple_assignment(
         .and_then(ast::RecordPatField::for_field_name)
         .is_some_and(|field| field.colon_token().is_none());
 
-    if let Some(cap) = ctx.config.snippet_cap {
+    if ctx.config.snippet_edit_support {
         // place cursor on first tuple name
         if let Some(ast::Pat::IdentPat(first_pat)) = tuple_pat.fields().next() {
-            let annotation = edit.make_tabstop_before(cap);
+            let annotation = edit.make_tabstop_before();
             editor.add_annotation(
                 first_pat.name().expect("first ident pattern should have a name").syntax(),
                 annotation,

crates/ide-assists/src/handlers/destructure_tuple_binding.rs

@@ -134,7 +134,7 @@ pub(crate) fn extract_expressions_from_format_string(
             let new_tt = make.token_tree(tt_delimiter, new_tt_bits);
             editor.replace(tt.syntax(), new_tt.syntax());
 
-            if let Some(cap) = ctx.config.snippet_cap {
+            if ctx.config.snippet_edit_support {
                 // Add placeholder snippets over placeholder args
                 for pos in placeholder_indexes {
                     // Skip the opening delimiter
@@ -145,7 +145,7 @@ pub(crate) fn extract_expressions_from_format_string(
                     };
 
                     if stdx::always!(placeholder.kind() == T![_]) {
-                        let annotation = edit.make_placeholder_snippet(cap);
+                        let annotation = edit.make_placeholder_snippet();
                         editor.add_annotation(placeholder, annotation);
                     }
                 }
@@ -154,7 +154,7 @@ pub(crate) fn extract_expressions_from_format_string(
                 if let Some(NodeOrToken::Token(literal)) =
                     new_tt.token_trees_and_tokens().nth(1 + format_string_index)
                 {
-                    let annotation = edit.make_tabstop_after(cap);
+                    let annotation = edit.make_tabstop_after();
                     editor.add_annotation(literal, annotation);
                 }
             }

crates/ide-assists/src/handlers/extract_expressions_from_format_string.rs

@@ -177,10 +177,10 @@ pub(crate) fn extract_function(acc: &mut Assists, ctx: &AssistContext<'_>) -> Op
 
             let fn_def = format_function(ctx, module, &fun, old_indent).clone_for_update();
 
-            if let Some(cap) = ctx.config.snippet_cap
+            if ctx.config.snippet_edit_support
                 && let Some(name) = fn_def.name()
             {
-                builder.add_tabstop_before(cap, name);
+                builder.add_tabstop_before(name);
             }
 
             // FIXME: wrap non-adt types