Clean expanded paths

This better mimics the behaviour of std::fs::canonicalize without
evaluating symlinks.

Author: joshheinrichs-shopify

Cargo.lock

@@ -18,6 +18,7 @@ hex = "0.4.3"
 ketos = "0.12"
 ketos_derive = "0.12"
 libc = "0.2.48"
+path-clean = "1.0.1"
 regex = "1.11.0"
 serde = "1.0.210"
 serde_derive = "1.0.210"

Cargo.toml

@@ -4,6 +4,7 @@ use crate::{
 };
 use ketos::{Context, Error, FromValueRef, Name, Value};
 use ketos_derive::{ForeignValue, FromValueRef};
+use path_clean::PathClean;
 use std::{
     cell::{Ref, RefCell},
     env,
@@ -76,7 +77,7 @@ fn path_concat(vals: &mut [Value]) -> Result<String, Error> {
         |acc, v| acc.join(<&str as FromValueRef>::from_value_ref(v).unwrap()), // TODO(burke): don't unwrap
     );
 
-    Ok(res.to_string_lossy().to_string())
+    Ok(res.clean().to_string_lossy().to_string())
 }
 
 impl ShadowLang {
@@ -250,8 +251,9 @@ impl ShadowLang {
                         }));
                     }
                 };
+                let cleaned = absolutized.clean();
                 Ok(<String as Into<Value>>::into(
-                    absolutized.to_string_lossy().to_string(),
+                    cleaned.to_string_lossy().to_string(),
                 ))
             })
         });