impr (hql): parser refactor, nested for loop type checking, tidying repo (#428)

## Description
<!-- Provide a brief description of the changes in this PR -->

## Related Issues
<!-- Link to any related issues using #issue_number -->

Closes #

## Checklist when merging to main
<!-- Mark items with "x" when completed -->

- [ ] No compiler warnings (if applicable)
- [ ] Code is formatted with `rustfmt`
- [ ] No useless or dead code (if applicable)
- [ ] Code is easy to understand
- [ ] Doc comments are used for all functions, enums, structs, and
fields (where appropriate)
- [ ] All tests pass
- [ ] Performance has not regressed (assuming change was not to fix a
bug)
- [ ] Version number has been updated in `helix-cli/Cargo.toml` and
`helixdb/Cargo.toml`

## Additional Notes
<!-- Add any additional information that would be helpful for reviewers
-->

Author: xav-db

README.md

@@ -1,7 +1,7 @@
 <div align="center">
 
 <picture>
-  <img src="/docs/full_logo.png" alt="HelixDB Logo">
+  <img src="/assets/full_logo.png" alt="HelixDB Logo">
 </picture>
 
 <b>HelixDB</b>: a database built from scratch to be the storage backend for any AI application.

docs/full_logo.png renamed to assets/full_logo.png

@@ -7,9 +7,12 @@ use futures_util::StreamExt;
 use helix_db::{
     helix_engine::traversal_core::config::Config,
     helixc::{
-        analyzer::analyzer::analyze,
+        analyzer::analyze,
         generator::{Source as GeneratedSource, tsdisplay::ToTypeScript},
-        parser::helix_parser::{Content, HelixParser, HxFile, Source},
+        parser::{
+            HelixParser,
+            types::{Content, HxFile, Source},
+        },
     },
     utils::styled_string::StyledString,
 };
@@ -956,12 +959,7 @@ pub fn copy_repo_dir_for_build(src: &std::path::Path, dst: &std::path::Path) ->
         if let Some(file_name) = entry.file_name().to_str()
             && matches!(
                 file_name,
-                ".git"
-                    | ".gitignore"
-                    | ".github"
-                    | ".DS_Store"
-                    | "target"
-                    | "docs"
+                ".git" | ".gitignore" | ".github" | ".DS_Store" | "target" | "docs"
             )
         {
             continue;

docs/icon-1.png renamed to assets/icon-1.png

@@ -63,7 +63,7 @@ traversal           = { (start_node | start_edge | search_vector | start_vector)
 id_traversal        = { identifier ~ ((step+ ~ last_step?) | last_step) }
 anonymous_traversal = { "_"  ~ ((step+ ~ last_step?) | last_step)? }
 step                = { "::" ~ (graph_step | order_by | where_step | closure_step | object_step | exclude_field | count | ID | range_step | AddE) }
-last_step           = { "::" ~ (bool_operations | update) }
+last_step           = { "::" ~ (bool_operations | update | first) }
 // change this for loop to be able to take traversals etc in the future. 
 for_loop            = { "FOR" ~ for_argument ~ "IN" ~ identifier ~ "{" ~ query_body ~ "}" }
 for_argument        = { object_access | object_destructuring | identifier }
@@ -194,6 +194,7 @@ ID           = { "ID" }
 update_field = { identifier ~ ":" ~ (evaluates_to_anything | anonymous_traversal) }
 update       = { "UPDATE" ~ "(" ~ "{" ~ update_field ~ ("," ~ update_field)* ~ "}" ~ ")" }
 drop = { "DROP" ~ evaluates_to_anything }
+first = { "FIRST" }
 
 // ---------------------------------------------------------------------
 // Vector steps

helix-cli/src/utils.rs

@@ -431,7 +431,7 @@ impl BM25Flatten for HashMap<String, Value> {
             .fold(String::with_capacity(self.len() * 4), |mut s, (k, v)| {
                 s.push_str(k);
                 s.push(' ');
-                s.push_str(&v.to_string());
+                s.push_str(&v.inner_stringify());
                 s.push(' ');
                 s
             })

helix-db/mod.rs

@@ -178,9 +178,6 @@ pub mod macros {
     #[macro_export]
     macro_rules! traversal_remapping {
         ($remapping_vals:expr, $var_name:expr, $should_spread:expr, $new_name:expr => $traversal:expr) => {{
-            // TODO: ref?
-            // Apply remappings to the nested traversal result
-
             let nested_return_value = ReturnValue::from_traversal_value_array_with_mixin(
                 $traversal,
                 $remapping_vals.borrow_mut(),