@devaloop/devalang 0.0.1-alpha.15 → 0.0.1-alpha.16-hotfix.0

package/rust/core/preprocessor/resolver/loop_.rs

@@ -1,121 +1,320 @@
-use std::collections::HashMap;
-
-use crate::{
-    core::{
-        parser::statement::{ Statement, StatementKind },
-        preprocessor::{
-            module::Module,
-            resolver::{ driver::resolve_statement, value::resolve_value },
-        },
-        shared::value::Value,
-        store::global::GlobalStore,
-    },
-    utils::logger::Logger,
-};
-
-pub fn resolve_loop(
-    stmt: &Statement,
-    module: &Module,
-    path: &str,
-    global_store: &mut GlobalStore
-) -> Statement {
-    let logger = Logger::new();
-
-    let resolved_value = resolve_value(&stmt.value, module, global_store);
-
-    let Value::Map(value_map) = &resolved_value else {
-        return error_stmt(&logger, module, stmt, "Expected a map for loop value");
-    };
-
-    let mut resolved_map: HashMap<String, Value> = HashMap::new();
-    for (key, val) in value_map {
-        resolved_map.insert(key.clone(), resolve_value(val, module, global_store));
-    }
-
-    // Foreach form takes precedence if present
-    if let (Some(Value::Identifier(var_name)), Some(array_val)) = (resolved_map.get("foreach"), resolved_map.get("array")) {
-        // Resolve array elements
-        let resolved_array = match array_val {
-            Value::Array(items) => Value::Array(items.iter().map(|v| resolve_value(v, module, global_store)).collect()),
-            other => resolve_value(other, module, global_store),
-        };
-
-        let body_value = match resolved_map.get("body") {
-            Some(Value::Block(stmts)) => {
-                let resolved = stmts
-                    .iter()
-                    .map(|s| resolve_statement(s, module, path, global_store))
-                    .collect();
-                Value::Block(resolved)
-            }
-            _ => {
-                error_value(&logger, module, stmt, "Invalid or missing loop body");
-                Value::Block(vec![])
-            }
-        };
-
-        let mut final_map = HashMap::new();
-        final_map.insert("foreach".to_string(), Value::Identifier(var_name.clone()));
-        final_map.insert("array".to_string(), resolved_array);
-        final_map.insert("body".to_string(), body_value);
-
-        return Statement { kind: StatementKind::Loop, value: Value::Map(final_map), ..stmt.clone() };
-    }
-
-    let iterator_value = match resolved_map.get("iterator") {
-        Some(Value::Number(n)) => Value::Number(*n),
-        Some(other) => {
-            error_value(
-                &logger,
-                module,
-                stmt,
-                &format!("Loop iterator must be a number, found: {:?}", other)
-            );
-            Value::Number(1.0)
-        }
-        None => {
-            error_value(&logger, module, stmt, "Missing 'iterator' in loop");
-            Value::Number(1.0)
-        }
-    };
-
-    let body_value = match resolved_map.get("body") {
-        Some(Value::Block(stmts)) => {
-            let resolved = stmts
-                .iter()
-                .map(|s| resolve_statement(s, module, path, global_store))
-                .collect();
-            Value::Block(resolved)
-        }
-        _ => {
-            error_value(&logger, module, stmt, "Invalid or missing loop body");
-            Value::Block(vec![])
-        }
-    };
-
-    let mut final_map = HashMap::new();
-    final_map.insert("iterator".to_string(), iterator_value);
-    final_map.insert("body".to_string(), body_value);
-
-    Statement {
-        kind: StatementKind::Loop,
-        value: Value::Map(final_map),
-        ..stmt.clone()
-    }
-}
-
-fn error_value(logger: &Logger, module: &Module, stmt: &Statement, msg: &str) {
-    let stacktrace = format!("{}:{}:{}", module.path, stmt.line, stmt.column);
-    logger.log_error_with_stacktrace(msg, &stacktrace);
-}
-
-fn error_stmt(logger: &Logger, module: &Module, stmt: &Statement, msg: &str) -> Statement {
-    error_value(logger, module, stmt, msg);
-    Statement {
-        kind: StatementKind::Error {
-            message: msg.to_string(),
-        },
-        value: Value::Null,
-        ..stmt.clone()
-    }
-}
+use std::collections::HashMap;
+
+use crate::{
+    core::{
+        parser::statement::{Statement, StatementKind},
+        preprocessor::{
+            module::Module,
+            resolver::{driver::resolve_statement, value::resolve_value},
+        },
+        shared::value::Value,
+        store::global::GlobalStore,
+    },
+    utils::logger::Logger,
+};
+
+pub fn resolve_loop(
+    stmt: &Statement,
+    module: &Module,
+    path: &str,
+    global_store: &mut GlobalStore,
+) -> Statement {
+    let logger = Logger::new();
+
+    let resolved_value = resolve_value(&stmt.value, module, global_store);
+
+    let Value::Map(value_map) = &resolved_value else {
+        return error_stmt(&logger, module, stmt, "Expected a map for loop value");
+    };
+
+    let mut resolved_map: HashMap<String, Value> = HashMap::new();
+    for (key, val) in value_map {
+        resolved_map.insert(key.clone(), resolve_value(val, module, global_store));
+    }
+
+    // Foreach form takes precedence if present
+    if let (Some(Value::Identifier(var_name)), Some(array_val)) =
+        (resolved_map.get("foreach"), resolved_map.get("array"))
+    {
+        // Normalize array_val into an iterable Array
+        let resolved_array = match array_val {
+            Value::Array(items) => Value::Array(
+                items
+                    .iter()
+                    .map(|v| resolve_value(v, module, global_store))
+                    .collect(),
+            ),
+            Value::Number(n) => {
+                // Iterate 0..n-1
+                let count = (*n).max(0.0) as usize;
+                let mut items = Vec::with_capacity(count);
+                for i in 0..count {
+                    items.push(Value::Number(i as f32));
+                }
+                Value::Array(items)
+            }
+            Value::String(s) => {
+                // Try to parse a simple comma-separated list: "a,b,c" -> ["a","b","c"]
+                // If numeric string: iterate 0..n-1
+                if let Ok(n) = s.parse::<f32>() {
+                    let count = n.max(0.0) as usize;
+                    let mut items = Vec::with_capacity(count);
+                    for i in 0..count {
+                        items.push(Value::Number(i as f32));
+                    }
+                    Value::Array(items)
+                } else if s.contains(',') {
+                    let parts: Vec<Value> = s
+                        .split(',')
+                        .map(|p| Value::String(p.trim().to_string()))
+                        .collect();
+                    Value::Array(parts)
+                } else {
+                    // Fallback: iterate characters
+                    let parts: Vec<Value> =
+                        s.chars().map(|c| Value::String(c.to_string())).collect();
+                    Value::Array(parts)
+                }
+            }
+            Value::Identifier(name) => {
+                // Resolve identifier from module variables (already resolved map above)
+                let v = if let Some(v) = module.variable_table.get(name) {
+                    v.clone()
+                } else {
+                    Value::Null
+                };
+                match v {
+                    Value::Array(items) => Value::Array(
+                        items
+                            .iter()
+                            .map(|v| resolve_value(v, module, global_store))
+                            .collect(),
+                    ),
+                    Value::Number(n) => {
+                        let count = n.max(0.0) as usize;
+                        let mut items = Vec::with_capacity(count);
+                        for i in 0..count {
+                            items.push(Value::Number(i as f32));
+                        }
+                        Value::Array(items)
+                    }
+                    Value::String(s) => {
+                        if let Ok(n) = s.parse::<f32>() {
+                            let count = n.max(0.0) as usize;
+                            let mut items = Vec::with_capacity(count);
+                            for i in 0..count {
+                                items.push(Value::Number(i as f32));
+                            }
+                            Value::Array(items)
+                        } else if s.contains(',') {
+                            let parts: Vec<Value> = s
+                                .split(',')
+                                .map(|p| Value::String(p.trim().to_string()))
+                                .collect();
+                            Value::Array(parts)
+                        } else {
+                            let parts: Vec<Value> =
+                                s.chars().map(|c| Value::String(c.to_string())).collect();
+                            Value::Array(parts)
+                        }
+                    }
+                    other => {
+                        error_value(
+                            &logger,
+                            module,
+                            stmt,
+                            &format!(
+                                "Foreach identifier '{}' resolves to unsupported value: {:?}",
+                                name, other
+                            ),
+                        );
+                        Value::Array(vec![])
+                    }
+                }
+            }
+            other => {
+                // Resolve and normalize if possible
+                let v = resolve_value(other, module, global_store);
+                match v {
+                    Value::Array(items) => Value::Array(items),
+                    Value::Number(n) => {
+                        let count = n.max(0.0) as usize;
+                        let mut items = Vec::with_capacity(count);
+                        for i in 0..count {
+                            items.push(Value::Number(i as f32));
+                        }
+                        Value::Array(items)
+                    }
+                    Value::String(s) => {
+                        if let Ok(n) = s.parse::<f32>() {
+                            let count = n.max(0.0) as usize;
+                            let mut items = Vec::with_capacity(count);
+                            for i in 0..count {
+                                items.push(Value::Number(i as f32));
+                            }
+                            Value::Array(items)
+                        } else if s.contains(',') {
+                            let parts: Vec<Value> = s
+                                .split(',')
+                                .map(|p| Value::String(p.trim().to_string()))
+                                .collect();
+                            Value::Array(parts)
+                        } else {
+                            let parts: Vec<Value> =
+                                s.chars().map(|c| Value::String(c.to_string())).collect();
+                            Value::Array(parts)
+                        }
+                    }
+                    other => {
+                        error_value(
+                            &logger,
+                            module,
+                            stmt,
+                            &format!("Unsupported foreach array value: {:?}", other),
+                        );
+                        Value::Array(vec![])
+                    }
+                }
+            }
+        };
+
+        let body_value = match resolved_map.get("body") {
+            Some(Value::Block(stmts)) => {
+                let resolved = stmts
+                    .iter()
+                    .map(|s| resolve_statement(s, module, path, global_store))
+                    .collect();
+                Value::Block(resolved)
+            }
+            _ => {
+                error_value(&logger, module, stmt, "Invalid or missing loop body");
+                Value::Block(vec![])
+            }
+        };
+
+        let mut final_map = HashMap::new();
+        final_map.insert("foreach".to_string(), Value::Identifier(var_name.clone()));
+        final_map.insert("array".to_string(), resolved_array);
+        final_map.insert("body".to_string(), body_value);
+
+        return Statement {
+            kind: StatementKind::Loop,
+            value: Value::Map(final_map),
+            ..stmt.clone()
+        };
+    }
+
+    let iterator_value = match resolved_map.get("iterator") {
+        Some(Value::Number(n)) => Value::Number(*n),
+        Some(Value::String(s)) => {
+            if let Ok(n) = s.parse::<f32>() {
+                Value::Number(n)
+            } else {
+                error_value(
+                    &logger,
+                    module,
+                    stmt,
+                    &format!("Loop iterator string not numeric: '{}'", s),
+                );
+                Value::Number(1.0)
+            }
+        }
+        Some(Value::Identifier(name)) => {
+            // Try resolving from module vars (may be number or numeric string)
+            if let Some(v) = module.variable_table.get(name) {
+                match v {
+                    Value::Number(n) => Value::Number(*n),
+                    Value::String(s) => {
+                        if let Ok(n) = s.parse::<f32>() {
+                            Value::Number(n)
+                        } else {
+                            error_value(
+                                &logger,
+                                module,
+                                stmt,
+                                &format!(
+                                    "Loop iterator '{}' resolves to non-numeric string: '{}'",
+                                    name, s
+                                ),
+                            );
+                            Value::Number(1.0)
+                        }
+                    }
+                    other => {
+                        error_value(
+                            &logger,
+                            module,
+                            stmt,
+                            &format!(
+                                "Loop iterator '{}' resolves to non-number: {:?}",
+                                name, other
+                            ),
+                        );
+                        Value::Number(1.0)
+                    }
+                }
+            } else {
+                error_value(
+                    &logger,
+                    module,
+                    stmt,
+                    &format!("Loop iterator identifier '{}' not found", name),
+                );
+                Value::Number(1.0)
+            }
+        }
+        Some(other) => {
+            error_value(
+                &logger,
+                module,
+                stmt,
+                &format!("Loop iterator must be a number, found: {:?}", other),
+            );
+            Value::Number(1.0)
+        }
+        None => {
+            error_value(&logger, module, stmt, "Missing 'iterator' in loop");
+            Value::Number(1.0)
+        }
+    };
+
+    let body_value = match resolved_map.get("body") {
+        Some(Value::Block(stmts)) => {
+            let resolved = stmts
+                .iter()
+                .map(|s| resolve_statement(s, module, path, global_store))
+                .collect();
+            Value::Block(resolved)
+        }
+        _ => {
+            error_value(&logger, module, stmt, "Invalid or missing loop body");
+            Value::Block(vec![])
+        }
+    };
+
+    let mut final_map = HashMap::new();
+    final_map.insert("iterator".to_string(), iterator_value);
+    final_map.insert("body".to_string(), body_value);
+
+    Statement {
+        kind: StatementKind::Loop,
+        value: Value::Map(final_map),
+        ..stmt.clone()
+    }
+}
+
+fn error_value(logger: &Logger, module: &Module, stmt: &Statement, msg: &str) {
+    let stacktrace = format!("{}:{}:{}", module.path, stmt.line, stmt.column);
+    logger.log_error_with_stacktrace(msg, &stacktrace);
+}
+
+fn error_stmt(logger: &Logger, module: &Module, stmt: &Statement, msg: &str) -> Statement {
+    error_value(logger, module, stmt, msg);
+    Statement {
+        kind: StatementKind::Error {
+            message: msg.to_string(),
+        },
+        value: Value::Null,
+        ..stmt.clone()
+    }
+}

package/rust/core/preprocessor/resolver/mod.rs

@@ -1,15 +1,15 @@
-pub mod driver;
-
-pub mod value;
-
-pub mod trigger;
-pub mod loop_;
-pub mod bank;
-pub mod tempo;
-pub mod group;
-pub mod condition;
-pub mod spawn;
-pub mod call;
-pub mod synth;
-pub mod let_;
-pub mod function;
+pub mod driver;
+
+pub mod value;
+
+pub mod bank;
+pub mod call;
+pub mod condition;
+pub mod function;
+pub mod group;
+pub mod let_;
+pub mod loop_;
+pub mod spawn;
+pub mod synth;
+pub mod tempo;
+pub mod trigger;

package/rust/core/preprocessor/resolver/spawn.rs

@@ -1,73 +1,76 @@
-use crate::{
-    core::{
-        parser::statement::{ Statement, StatementKind },
-        preprocessor::module::Module,
-        shared::value::Value,
-        store::global::GlobalStore,
-    },
-    utils::logger::{ Logger, LogLevel },
-};
-
-pub fn resolve_spawn(
-    stmt: &Statement,
-    name: String,
-    args: Vec<Value>,
-    module: &Module,
-    _path: &str,
-    global_store: &mut GlobalStore
-) -> Statement {
-    let logger = Logger::new();
-
-    // ✅ Si c'est une fonction
-    if let Some(func) = global_store.functions.functions.get(&name) {
-        let mut resolved_map = std::collections::HashMap::new();
-        resolved_map.insert("name".to_string(), Value::String(name.clone()));
-        resolved_map.insert("args".to_string(), Value::Array(args.clone()));
-        resolved_map.insert("body".to_string(), Value::Block(func.body.clone()));
-
-        return Statement {
-            kind: StatementKind::Spawn { name, args },
-            value: Value::Map(resolved_map),
-            ..stmt.clone()
-        };
-    }
-
-    // ✅ Si c'est un group dans les variables
-    if let Some(variable) = global_store.variables.variables.get(&name) {
-        if let Value::Statement(stmt_box) = variable {
-            if let StatementKind::Group = stmt_box.kind {
-                if let Value::Map(map) = &stmt_box.value {
-                    if let Some(Value::Block(body)) = map.get("body") {
-                        let mut resolved_map = std::collections::HashMap::new();
-                        resolved_map.insert("identifier".to_string(), Value::String(name.clone()));
-                        resolved_map.insert("args".to_string(), Value::Array(args.clone()));
-                        resolved_map.insert("body".to_string(), Value::Block(body.clone()));
-
-                        return Statement {
-                            kind: StatementKind::Spawn { name, args },
-                            value: Value::Map(resolved_map),
-                            ..stmt.clone()
-                        };
-                    }
-                }
-            }
-        }
-    }
-
-    // ❌ Sinon erreur
-    let stacktrace = format!("{}:{}:{}", module.path, stmt.line, stmt.column);
-    logger.log_message(
-        LogLevel::Error,
-        &format!("Function or group '{}' not found for spawn\n  → at {stacktrace}", name)
-    );
-
-    Statement {
-        kind: StatementKind::Error {
-            message: format!("Function or group '{}' not found for spawn", name),
-        },
-        value: Value::Null,
-        ..stmt.clone()
-    }
-}
-
-// (removed unused helpers get_group_body, error_stmt)
+use crate::{
+    core::{
+        parser::statement::{Statement, StatementKind},
+        preprocessor::module::Module,
+        shared::value::Value,
+        store::global::GlobalStore,
+    },
+    utils::logger::{LogLevel, Logger},
+};
+
+pub fn resolve_spawn(
+    stmt: &Statement,
+    name: String,
+    args: Vec<Value>,
+    module: &Module,
+    _path: &str,
+    global_store: &mut GlobalStore,
+) -> Statement {
+    let logger = Logger::new();
+
+    // If it's a function
+    if let Some(func) = global_store.functions.functions.get(&name) {
+        let mut resolved_map = std::collections::HashMap::new();
+        resolved_map.insert("name".to_string(), Value::String(name.clone()));
+        resolved_map.insert("args".to_string(), Value::Array(args.clone()));
+        resolved_map.insert("body".to_string(), Value::Block(func.body.clone()));
+
+        return Statement {
+            kind: StatementKind::Spawn { name, args },
+            value: Value::Map(resolved_map),
+            ..stmt.clone()
+        };
+    }
+
+    // If it's a group stored in variables
+    if let Some(variable) = global_store.variables.variables.get(&name) {
+        if let Value::Statement(stmt_box) = variable {
+            if let StatementKind::Group = stmt_box.kind {
+                if let Value::Map(map) = &stmt_box.value {
+                    if let Some(Value::Block(body)) = map.get("body") {
+                        let mut resolved_map = std::collections::HashMap::new();
+                        resolved_map.insert("identifier".to_string(), Value::String(name.clone()));
+                        resolved_map.insert("args".to_string(), Value::Array(args.clone()));
+                        resolved_map.insert("body".to_string(), Value::Block(body.clone()));
+
+                        return Statement {
+                            kind: StatementKind::Spawn { name, args },
+                            value: Value::Map(resolved_map),
+                            ..stmt.clone()
+                        };
+                    }
+                }
+            }
+        }
+    }
+
+    // Otherwise, log an error
+    let stacktrace = format!("{}:{}:{}", module.path, stmt.line, stmt.column);
+    logger.log_message(
+        LogLevel::Error,
+        &format!(
+            "Function or group '{}' not found for spawn\n  → at {stacktrace}",
+            name
+        ),
+    );
+
+    Statement {
+        kind: StatementKind::Error {
+            message: format!("Function or group '{}' not found for spawn", name),
+        },
+        value: Value::Null,
+        ..stmt.clone()
+    }
+}
+
+// (removed unused helpers get_group_body, error_stmt)