Refactor tests for improved readability and consistency

- Updated formatting in `chat_comprehensive_plan_test.rs` for better alignment and clarity.
- Simplified command arguments in multiple test files for consistency.
- Enhanced assertions in various tests to improve readability.
- Consolidated tool registration calls in `chat_ovsm_executor_integration_test.rs` for cleaner code.
- Streamlined message processing logic in `chat_sendable_textarea_test.rs` and other related tests.
- Removed unnecessary lines and improved logging messages across several test files.
- Ensured consistent use of Rust's idiomatic style throughout the test suite.

Author: 0xrinegade

Cargo.lock

@@ -139,7 +139,6 @@ opt-level = 0
 debug = true
 
 [patch.crates-io]
-curve25519-dalek = { git = "https://github.com/dalek-cryptography/curve25519-dalek", tag = "3.2.0" }
 crunchy = { path = "vendor/crunchy" }
 
 # Custom scripts for development and deployment

Cargo.toml

@@ -61,6 +61,8 @@ sha2 = "0.10"
 base64 = "0.22.1"
 bs58 = "0.5.1"
 hex = "0.4"
+url = "2.5"
+csv = "1.3"
 
 [dev-dependencies]
 proptest = "1.4"

crates/ovsm/Cargo.toml

@@ -241,11 +241,13 @@ impl SExprScanner {
         let text: String = self.source[self.start..self.current].iter().collect();
 
         if is_float {
-            let value: f64 = text.parse()
+            let value: f64 = text
+                .parse()
                 .map_err(|_| Error::ParseError(format!("Invalid float: {}", text)))?;
             self.add_token(TokenKind::Float(value));
         } else {
-            let value: i64 = text.parse()
+            let value: i64 = text
+                .parse()
                 .map_err(|_| Error::ParseError(format!("Invalid integer: {}", text)))?;
             self.add_token(TokenKind::Integer(value));
         }
@@ -256,7 +258,13 @@ impl SExprScanner {
     fn scan_identifier_or_keyword(&mut self) -> Result<()> {
         // In Common Lisp, identifiers can contain *, +, -, /, etc as suffixes
         // First, scan the base identifier
-        while self.peek().is_alphanumeric() || self.peek() == '_' || self.peek() == '-' || self.peek() == '?' || self.peek() == '!' || self.peek() == '&' {
+        while self.peek().is_alphanumeric()
+            || self.peek() == '_'
+            || self.peek() == '-'
+            || self.peek() == '?'
+            || self.peek() == '!'
+            || self.peek() == '&'
+        {
             self.advance();
         }
 
@@ -327,7 +335,8 @@ impl SExprScanner {
 
     fn add_token(&mut self, kind: TokenKind) {
         let lexeme: String = self.source[self.start..self.current].iter().collect();
-        self.tokens.push(Token::new(kind, lexeme, self.line, self.column));
+        self.tokens
+            .push(Token::new(kind, lexeme, self.line, self.column));
     }
 }
 

crates/ovsm/src/lexer/sexpr_scanner.rs

@@ -280,7 +280,7 @@ pub mod tools;
 // Re-export main types
 pub use error::{Error, Result};
 pub use lexer::{SExprScanner, Token, TokenKind};
-pub use parser::{BinaryOp, Expression, SExprParser, Program, Statement, UnaryOp};
+pub use parser::{BinaryOp, Expression, Program, SExprParser, Statement, UnaryOp};
 pub use runtime::{Environment, LispEvaluator, Value};
 pub use tools::{Tool, ToolRegistry};
 

crates/ovsm/src/lib.rs

@@ -7,9 +7,19 @@ mod paren_fixer;
 mod sexpr_parser;
 
 pub use ast::{
-    Argument, BinaryOp, Expression, Program, ProgramMetadata, Statement, UnaryOp,
+    AccumulationClause,
+    Argument,
+    BinaryOp,
+    ConditionClause,
+    ExitClause,
+    Expression,
+    IterationClause,
     // Loop macro structures
-    LoopData, IterationClause, AccumulationClause, ConditionClause, ExitClause,
+    LoopData,
+    Program,
+    ProgramMetadata,
+    Statement,
+    UnaryOp,
 };
 pub use paren_fixer::ParenFixer;
 pub use sexpr_parser::SExprParser;

crates/ovsm/src/parser/mod.rs

@@ -4,8 +4,8 @@
 //! parentheses in OVSM code, similar to error recovery in modern compilers.
 
 use crate::error::Result;
-use crate::Scanner;
 use crate::parser::SExprParser;
+use crate::Scanner;
 
 /// Attempts to automatically fix missing or mismatched parentheses
 pub struct ParenFixer {
@@ -178,10 +178,9 @@ impl ParenFixer {
         let mut close_count = 0;
         let mut in_string = false;
         let mut escape_next = false;
-        let mut in_comment = false;
 
         for line in &self.lines {
-            in_comment = false; // Comments are line-based
+            let mut in_comment = false; // Comments are line-based
 
             for ch in line.chars() {
                 // Handle escape sequences
@@ -360,7 +359,10 @@ impl ParenFixer {
 
         match fixed_result {
             Ok(fixed) => (fixed, report),
-            Err(_) => (self.source.clone(), Some("⚠️  Could not auto-correct parentheses".to_string())),
+            Err(_) => (
+                self.source.clone(),
+                Some("⚠️  Could not auto-correct parentheses".to_string()),
+            ),
         }
     }
 }