assistant2: Fix `\\` appearing for paths in file context picker (#27528)

Bennet Bo Fenner created 9 months ago

Closes #ISSUE

Release Notes:

- N/A

Change summary

crates/assistant2/src/context_picker/file_context_picker.rs | 11 +++++-
1 file changed, 8 insertions(+), 3 deletions(-)

Detailed changes

crates/assistant2/src/context_picker/file_context_picker.rs

@@ -282,7 +282,10 @@ pub fn render_file_context_entry(
     cx: &App,
 ) -> Stateful<Div> {
     let (file_name, directory) = if path == Path::new("") {
-        (SharedString::from(path_prefix.clone()), None)
+        (
+            SharedString::from(path_prefix.trim_end_matches('/').to_string()),
+            None,
+        )
     } else {
         let file_name = path
             .file_name()
@@ -291,8 +294,10 @@ pub fn render_file_context_entry(
             .to_string()
             .into();
 
-        let mut directory = format!("{}/", path_prefix);
-
+        let mut directory = path_prefix.to_string();
+        if !directory.ends_with('/') {
+            directory.push('/');
+        }
         if let Some(parent) = path.parent().filter(|parent| parent != &Path::new("")) {
             directory.push_str(&parent.to_string_lossy());
             directory.push('/');