Consolidate Bolt PRs: optimize Context lock usage and eliminate redundant ContextValue clones (#59)

* Initial plan

* Optimize Context lookups: inline lock and match on borrowed refs in get_func/get_variable/value

Agent-Logs-Url: https://github.com/ashyanSpada/expression_engine_rs/sessions/7e9ce6a0-1aa8-4510-9420-d73ff44eee82

Co-authored-by: ashyanSpada <22587148+ashyanSpada@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: ashyanSpada <22587148+ashyanSpada@users.noreply.github.com>

Author: Copilot

src/context.rs

@@ -31,17 +31,17 @@ impl Context {
     }
 
     pub fn get_func(&self, name: &str) -> Option<Arc<InnerFunction>> {
-        let value = self.get(name)?;
-        match value {
-            ContextValue::Function(func) => Some(func),
+        let binding = self.0.lock().unwrap();
+        match binding.get(name)? {
+            ContextValue::Function(func) => Some(func.clone()),
             ContextValue::Variable(_) => None,
         }
     }
 
     pub fn get_variable(&self, name: &str) -> Option<Value> {
-        let value = self.get(name)?;
-        match value {
-            ContextValue::Variable(v) => Some(v),
+        let binding = self.0.lock().unwrap();
+        match binding.get(name)? {
+            ContextValue::Variable(v) => Some(v.clone()),
             ContextValue::Function(_) => None,
         }
     }
@@ -51,13 +51,15 @@ impl Context {
     }
 
     pub fn value(&self, name: &str) -> Result<Value> {
-        // Lock is acquired and released inside `get` early to avoid contention during execution
-        let value = self.get(name);
-        match value {
-            Some(ContextValue::Variable(v)) => Ok(v),
-            Some(ContextValue::Function(func)) => func(Vec::new()),
-            None => Ok(Value::None),
-        }
+        let func = {
+            let binding = self.0.lock().unwrap();
+            match binding.get(name) {
+                Some(ContextValue::Variable(v)) => return Ok(v.clone()),
+                Some(ContextValue::Function(func)) => func.clone(),
+                None => return Ok(Value::None),
+            }
+        };
+        func(Vec::new())
     }
 }