Fix autocomplete repo match (#1218)

* make repo_name ; make autocomplete and folder queries case insensitive; remove repo.display_name()

* use stringified repo_ref in prompt

Author: ggordonhall

server/bleep/src/agent/prompts.rs

@@ -100,7 +100,8 @@ pub fn system<'a>(paths: impl IntoIterator<Item = &'a RepoPath>) -> String {
     if iter.peek().is_some() {
         s.push_str("\n## PATHS ##\nindex, repo, path\n");
         for (i, path) in iter.enumerate() {
-            let repo = path.repo.display_name();
+            let repo = &path.repo;
+            let repo = &format!("{repo}");
             let path = &path.path;
             s.push_str(&format!("{}, {}, {}\n", i, repo, path));
         }

server/bleep/src/agent/symbol.rs

@@ -65,7 +65,7 @@ impl Agent {
                     .to_string(),
                 token_info_request: TokenInfoRequest {
                     relative_path: chunk.repo_path.path.clone(),
-                    repo_ref: chunk.repo_path.repo.display_name(),
+                    repo_ref: chunk.repo_path.repo.indexed_name(),
                     branch: None,
                     start: range.start.byte,
                     end: range.end.byte,

server/bleep/src/indexes/schema.rs

@@ -80,7 +80,7 @@ impl File {
 
         let repo_disk_path = builder.add_text_field("repo_disk_path", STRING);
         let repo_ref = builder.add_text_field("repo_ref", STRING | STORED);
-        let repo_name = builder.add_text_field("repo_name", trigram.clone());
+        let repo_name = builder.add_text_field("repo_name", STRING | STORED);
         let relative_path = builder.add_text_field("relative_path", trigram.clone());
 
         let content = builder.add_text_field("content", trigram.clone());

server/bleep/src/query/compiler.rs

@@ -255,7 +255,7 @@ pub fn case_permutations(s: &str) -> impl Iterator<Item = CompactString> {
 
     // Make sure not to overflow. The end condition is a mask with the highest bit set, and we use
     // `u32` masks.
-    debug_assert!(chars.len() <= 31);
+    debug_assert!(chars.len() <= 5);
 
     let num_chars = chars.len();
 

server/bleep/src/repo.rs

@@ -97,23 +97,14 @@ impl RepoRef {
 
     pub fn indexed_name(&self) -> String {
         // Local repos indexed as: dirname
-        // Github repos indexed as: github.com/org/repo
+        // Github repos indexed as: org/repo
         match self.backend {
             Backend::Local => Path::new(&self.name)
                 .file_name()
                 .expect("last component is `..`")
                 .to_string_lossy()
                 .into(),
-            Backend::Github => format!("{}", self),
-        }
-    }
-
-    pub fn display_name(&self) -> String {
-        match self.backend {
-            // org_name/repo_name
             Backend::Github => self.name.to_owned(),
-            // repo_name
-            Backend::Local => self.indexed_name(),
         }
     }
 

server/bleep/src/webserver/autocomplete.rs

@@ -52,6 +52,10 @@ pub(super) async fn handle(
     let queries = parser::parse(&api_params.q)
         .map_err(Error::user)?
         .into_iter()
+        .map(|q| parser::Query {
+            case_sensitive: Some(true),
+            ..q
+        })
         .map(|mut q| {
             let keywords = &["lang:", "path:", "repo:"];
 

server/bleep/src/webserver/file.rs

@@ -100,6 +100,7 @@ pub(super) async fn folder(
         repo: Some(parser::Literal::from(&params.repo_ref.indexed_name())),
         path: Some(parser::Literal::from(params.path.to_string_lossy())),
         branch: params.branch.map(|b| parser::Literal::from(&b)),
+        case_sensitive: Some(true),
         ..Default::default()
     };
 

server/bleep/src/webserver/repos.rs

@@ -128,7 +128,7 @@ impl From<(&RepoRef, &Repository)> for Repo {
 
         Repo {
             provider: key.backend(),
-            name: key.display_name(),
+            name: key.indexed_name(),
             repo_ref: key.clone(),
             sync_status: repo.pub_sync_status.clone(),
             local_duplicates: vec![],