A bit of clean up and improvements

stack-core/src/engine.rs

@@ -115,10 +115,9 @@ impl Engine {
             }
             Ok(context)
           } else {
-            context.stack_push(Expr {
-              kind: ExprKind::Error(Error::new("unknown function".into())),
-              info: None,
-            })?;
+            context.stack_push(
+              ExprKind::Error(Error::new("unknown function".into())).into(),
+            )?;
 
             Ok(context)
           }

stack-core/src/expr.rs

@@ -13,6 +13,15 @@ pub struct Expr {
   pub info: Option<ExprInfo>,
 }
 
+impl From<ExprKind> for Expr {
+  fn from(value: ExprKind) -> Self {
+    Self {
+      info: None,
+      kind: value,
+    }
+  }
+}
+
 impl PartialEq for Expr {
   #[inline]
   fn eq(&self, other: &Self) -> bool {
@@ -108,6 +117,26 @@ impl ExprKind {
       x => x,
     }
   }
+
+  pub fn type_of(&self) -> &str {
+    match self {
+      ExprKind::Nil => "nil",
+      ExprKind::Error(_) => "error",
+
+      ExprKind::Boolean(_) => "boolean",
+      ExprKind::Integer(_) => "integer",
+      ExprKind::Float(_) => "float",
+      ExprKind::String(_) => "string",
+
+      ExprKind::Symbol(_) => "symbol",
+
+      ExprKind::Lazy(_) => "lazy",
+      ExprKind::List(_) => "list",
+      ExprKind::Record(_) => "record",
+
+      ExprKind::Fn(_) => "function",
+    }
+  }
 }
 
 impl PartialEq for ExprKind {
@@ -270,10 +299,7 @@ impl fmt::Display for ExprKind {
             .chain(core::iter::repeat(", "))
             .zip(x.iter())
             .try_for_each(|(sep, (key, value))| {
-              let key = Expr {
-                info: None,
-                kind: ExprKind::Symbol(*key),
-              };
+              let key: Expr = ExprKind::Symbol(*key).into();
               write!(f, "{sep}{key:#}: {value:#}")
             })?;