feat: standardize Rust edition 2024 across workspace

- Update 28 crates to use edition.workspace = true (edition 2024)
- Keep 8 crates on edition 2021 due to set_var/remove_var unsafe requirements:
  terraphim_github_runner_server, terraphim_update, terraphim_multi_agent,
  terraphim_config, terraphim_middleware, terraphim_service,
  terraphim_mcp_server, terraphim_ai_nodejs
- Fix edition 2024 breaking changes:
  - Pattern binding: remove explicit 'ref' in matches (implicit borrowing)
  - Derivable impl: add #[derive(Default)] with #[default] attribute
  - Let-and-return: return expressions directly
  - Unused imports: move to appropriate cfg(test) modules
- Add TuiService tests for search, autocomplete, replace_matches, summarize
- Update handler to use TuiService.search() convenience method
- Fix double-dereference patterns in test code for edition 2024

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: AlexMikhalev

crates/haystack_atlassian/Cargo.toml

@@ -1,7 +1,7 @@
 [package]
 name = "atlassian_haystack"
 version = "1.0.0"
-edition = "2021"
+edition.workspace = true
 license = "MIT"
 
 [dependencies]

crates/haystack_discourse/Cargo.toml

@@ -1,7 +1,7 @@
 [package]
 name = "discourse_haystack"
 version = "1.0.0"
-edition = "2021"
+edition.workspace = true
 authors = ["Terraphim"]
 description = "A CLI client for fetching Discourse posts and messages"
 license = "MIT"

crates/haystack_discourse/src/client.rs

@@ -130,8 +130,8 @@ mod tests {
     use super::*;
     use serde_json::json;
     use wiremock::{
-        matchers::{header, method, path, query_param},
         Mock, MockServer, ResponseTemplate,
+        matchers::{header, method, path, query_param},
     };
 
     fn can_bind_localhost() -> bool {

crates/haystack_jmap/Cargo.toml

@@ -1,7 +1,7 @@
 [package]
 name = "jmap_client"
 version = "1.0.0"
-edition = "2021"
+edition.workspace = true
 license = "MIT"
 
 [dependencies]

crates/terraphim-markdown-parser/Cargo.toml

@@ -1,7 +1,7 @@
 [package]
 name = "terraphim-markdown-parser"
 version = "1.0.0"
-edition = "2021"
+edition.workspace = true
 authors = ["Terraphim Contributors"]
 description = "Terraphim Markdown Parser"
 documentation = "https://terraphim.ai"

crates/terraphim-markdown-parser/src/lib.rs

@@ -2,8 +2,8 @@ use std::collections::HashSet;
 use std::ops::Range;
 use std::str::FromStr;
 
-use markdown::mdast::Node;
 use markdown::ParseOptions;
+use markdown::mdast::Node;
 use terraphim_types::{Document, DocumentType};
 use thiserror::Error;
 use ulid::Ulid;

crates/terraphim-session-analyzer/Cargo.toml

@@ -1,7 +1,7 @@
 [package]
 name = "terraphim-session-analyzer"
 version.workspace = true
-edition = "2021"
+edition.workspace = true
 authors = ["Zestic AI Development Team"]
 description = "Analyze AI coding assistant session logs to identify agent usage patterns"
 documentation = "https://terraphim.ai"

crates/terraphim-session-analyzer/src/kg/query.rs

@@ -1,4 +1,4 @@
-use anyhow::{anyhow, Result};
+use anyhow::{Result, anyhow};
 
 /// Query AST node representing a parsed query
 #[derive(Debug, Clone, PartialEq, Eq)]
@@ -157,7 +157,7 @@ impl ParserState {
     fn parse_not_expression(&mut self) -> Result<QueryNode> {
         if let Some(Token::Not) = self.peek() {
             self.consume(); // consume NOT
-                            // Recursively parse NOT expressions to support "not not X"
+            // Recursively parse NOT expressions to support "not not X"
             let operand = self.parse_not_expression()?;
             Ok(QueryNode::Not(Box::new(operand)))
         } else {
@@ -168,7 +168,7 @@ impl ParserState {
     /// Parse primary terms (concepts or parenthesized expressions)
     fn parse_primary(&mut self) -> Result<QueryNode> {
         match self.peek() {
-            Some(Token::Concept(ref concept)) => {
+            Some(Token::Concept(concept)) => {
                 let concept = concept.clone();
                 self.consume();
                 Ok(QueryNode::Concept(concept))
@@ -494,10 +494,12 @@ mod tests {
         let query = "(deploy and publish";
         let result = QueryParser::parse(query);
         assert!(result.is_err());
-        assert!(result
-            .unwrap_err()
-            .to_string()
-            .contains("closing parenthesis"));
+        assert!(
+            result
+                .unwrap_err()
+                .to_string()
+                .contains("closing parenthesis")
+        );
     }
 
     #[test]

crates/terraphim-session-analyzer/src/lib.rs

@@ -49,14 +49,14 @@ pub mod connectors;
 // Re-export main types for convenience
 pub use analyzer::{Analyzer, SummaryStats};
 pub use models::{
-    get_agent_category, normalize_agent_name, AgentAttribution, AgentInvocation, AgentStatistics,
-    AgentToolCorrelation, AnalyzerConfig, CollaborationPattern, FileOperation, SessionAnalysis,
-    ToolAnalysis, ToolCategory, ToolChain, ToolInvocation, ToolStatistics,
+    AgentAttribution, AgentInvocation, AgentStatistics, AgentToolCorrelation, AnalyzerConfig,
+    CollaborationPattern, FileOperation, SessionAnalysis, ToolAnalysis, ToolCategory, ToolChain,
+    ToolInvocation, ToolStatistics, get_agent_category, normalize_agent_name,
 };
 pub use parser::{SessionParser, TimelineEvent, TimelineEventType};
 pub use patterns::{
-    create_matcher, load_patterns, AhoCorasickMatcher, PatternMatcher, ToolMatch, ToolMetadata,
-    ToolPattern,
+    AhoCorasickMatcher, PatternMatcher, ToolMatch, ToolMetadata, ToolPattern, create_matcher,
+    load_patterns,
 };
 pub use reporter::Reporter;
 

crates/terraphim-session-analyzer/src/main.rs

@@ -15,7 +15,7 @@ use tracing::{info, warn};
 
 use analyzer::Analyzer;
 use parser::SessionParser;
-use patterns::{load_all_patterns, AhoCorasickMatcher, PatternMatcher};
+use patterns::{AhoCorasickMatcher, PatternMatcher, load_all_patterns};
 use reporter::Reporter;
 
 #[derive(Parser)]