refactor(hql): Hql todo resolution (#424)

## Description
<!-- Provide a brief description of the changes in this PR -->
implementing major parser refactor and code tidy up which I think was
much needed. also resolved or removed many of the old todos

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

Closes #426 #425 

## 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

@@ -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-cli/src/utils.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(),