fix

Author: ciaranra

.github/workflows/python-release.yml

@@ -100,18 +100,20 @@ jobs:
             LLVM_SYS_140_PREFIX=$HOME/.pecos/llvm
           CIBW_BEFORE_ALL_LINUX: |
             curl -sSf https://sh.rustup.rs | sh -s -- -y
+            source $HOME/.cargo/env
             dnf install libffi-devel -y
             cargo run --release -p pecos-llvm-utils --bin pecos-llvm -- install --force
           CIBW_REPAIR_WHEEL_COMMAND_LINUX: >
             auditwheel repair -w {dest_dir} {wheel} &&
             pipx run abi3audit --strict --report {wheel}
           # macOS configuration
           CIBW_ENVIRONMENT_MACOS: >
-            PATH=$HOME/.pecos/llvm/bin:$PATH
+            PATH=$HOME/.cargo/bin:$HOME/.pecos/llvm/bin:$PATH
             LLVM_SYS_140_PREFIX=$HOME/.pecos/llvm
             MACOSX_DEPLOYMENT_TARGET=13.2
           CIBW_BEFORE_ALL_MACOS: |
             curl -sSf https://sh.rustup.rs | sh -s -- -y
+            source $HOME/.cargo/env
             rustup update
             cargo run --release -p pecos-llvm-utils --bin pecos-llvm -- install --force
             # Create a codesign wrapper that strips DYLD_LIBRARY_PATH to prevent

crates/pecos-phir-json/src/v0_1/expression.rs

@@ -183,8 +183,7 @@ impl<'a> ExpressionEvaluator<'a> {
     }
 
     /// Converts an expression to a string for caching
-    #[allow(clippy::only_used_in_recursion)]
-    fn expr_to_cache_key(&self, expr: &Expression) -> String {
+    fn expr_to_cache_key(expr: &Expression) -> String {
         match expr {
             Expression::Integer(val) => format!("int:{val}"),
             Expression::Variable(name) => format!("var:{name}"),
@@ -198,7 +197,7 @@ impl<'a> ExpressionEvaluator<'a> {
                         }
                         ArgItem::Integer(val) => write!(&mut key, ",int:{val}").unwrap(),
                         ArgItem::Expression(expr) => {
-                            write!(&mut key, ",expr:{}", self.expr_to_cache_key(expr)).unwrap();
+                            write!(&mut key, ",expr:{}", Self::expr_to_cache_key(expr)).unwrap();
                         }
                     }
                 }
@@ -244,7 +243,7 @@ impl<'a> ExpressionEvaluator<'a> {
         }
 
         // For complex expressions, use caching
-        let cache_key = self.expr_to_cache_key(expr);
+        let cache_key = Self::expr_to_cache_key(expr);
         if let Some(cached_value) = self.expr_cache.get(&cache_key) {
             return Ok(*cached_value);
         }