Add clickable links to open file. FIX: browser security doesn't allow to open file directly

Author: SamarthPyati

src/index.js

@@ -16,17 +16,20 @@ async function search(prompt) {
         body: prompt
     });
 
-    for ([path, rank] of await response.json()) {
-        if (response.json === null) {
-            let item = document.createElement("span");
-            item.appendChild(document.createTextNode("No such token found"));
-            item.appendChild(document.createElement("br"));    
-        }
-
+    const data = await response.json();
+    if (data.length === 0) {
         let item = document.createElement("span");
-        item.appendChild(document.createTextNode(path));
-        item.appendChild(document.createElement("br"));
+        item.textContent = "No results found.";
         results.appendChild(item);
+    } else {
+        for (let [path, rank] of data) {
+            let link = document.createElement("a");
+            link.href = path;
+            link.target = "_blank"; // open in new tab
+            link.textContent = path;
+            results.appendChild(link);
+            results.appendChild(document.createElement("br"));
+        }
     }
 }
 

src/main.rs

@@ -176,6 +176,14 @@ fn append_folder_to_model(dir_path: &Path, model: Arc<Mutex<InMemoryModel>>, pro
             continue 'step;
         }   
 
+        // Get absolute file path 
+        let file_path = fs::canonicalize(&file_path).map_err(|err| {
+            eprintln!("{}: Could not canonicalize path {} as {}", "ERROR".red().bold(),
+            file_path.display().to_string().bright_blue(),
+            err.to_string().red());
+        })?;
+        
+        // Main 
         let mut model = model.lock().unwrap();
         if model.requires_reindexing(&file_path, last_modified)? {
             println!("{}: Indexing {} ...", "INFO".cyan(), file_path_str.bright_cyan());
@@ -299,7 +307,7 @@ fn entry() -> Result<(), ()> {
             // Default address 
             let address = args.next().unwrap_or("127.0.0.1:6969".to_string());
 
-            // TODO: Figure out a better way to append extension ".docsense.json"
+            // IDEATE: Is it fine to place the index file in the folder itself or place in a root dir?
             let mut index_path = Path::new(&dir_path).to_path_buf(); 
             index_path.push(".docsense.json");
 
@@ -351,7 +359,6 @@ fn main() -> io::Result<()> {
     Ok(())
 }
 
-// TODO: Search result must consist of clickable links to open that file or file section
 // TODO: Synonym terms
 // TODO: Add levenstein distance or cosine similarity
 // TODO: Add better document ranker specifically "Okapi BM-25"

src/model.rs

@@ -1,5 +1,5 @@
 use std::{
-    path::{Path, PathBuf}, time::SystemTime
+    fs, path::{Path, PathBuf}, time::SystemTime
 };
 
 use std::collections::HashMap;
@@ -282,6 +282,8 @@ impl Model for InMemoryModel {
         for term in ft.keys() {
             self.gtf.entry(term.to_owned()).and_modify(|x| *x += 1).or_insert(1);
         }
+
+        // Update the Docs table
         self.docs.insert(file_path, Doc { count: term_count, ft: ft , last_modified: last_modified});
         Ok(())
     }

src/server.rs

@@ -12,16 +12,16 @@ pub fn serve_404(request: Request) -> io::Result<()> {
 
 
 pub fn serve_500(request: Request) -> io::Result<()> {
-    return request.respond(Response::from_string("404").with_status_code(StatusCode(500)));
+    return request.respond(Response::from_string("500").with_status_code(StatusCode(500)));
 }
 
 
 pub fn serve_400(request: Request, message: &str) -> io::Result<()> {
-    return request.respond(Response::from_string(format!("400: {message}")).with_status_code(StatusCode(500)));
+    return request.respond(Response::from_string(format!("400: {message}")).with_status_code(StatusCode(400)));
 }
 
 
-pub fn serve_static_file(request: Request, file_path: &str) -> io::Result<()> {
+pub fn serve_static_file(request: Request, file_path: &str, content_type: &str) -> io::Result<()> {
     let html_file = match File::open(Path::new(file_path)) {
         Ok(file) => file, 
         Err(err) => {
@@ -32,8 +32,8 @@ pub fn serve_static_file(request: Request, file_path: &str) -> io::Result<()> {
             return serve_500(request);
         }
     };
-    
-    return request.respond(Response::from_file(html_file));
+    let header = Header::from_bytes("Content-Type", content_type).expect("Should be a valid Content-Type while passing the header.");
+    return request.respond(Response::from_file(html_file).with_header(header));
 }
 
 
@@ -127,11 +127,11 @@ pub fn serve_request(request: Request, model: Arc<Mutex<InMemoryModel>>) -> io::
     match (&request.method(), request.url()) {
 
         (Method::Get, "/") | (Method::Get, "/index.html") => {
-            serve_static_file(request, "src/index.html")?
+            serve_static_file(request, "src/index.html", "text/html; charset=utf-8")?
         }
 
         (Method::Get, "/index.js") => {
-            serve_static_file(request, "src/index.js")?
+            serve_static_file(request, "src/index.js", "text/javascript; charset=utf-8")?
         }
 
         (Method::Post, "/api/search") => {