@ape-egg/vibe-compiler 4.3.27 → 4.3.28

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ape-egg/vibe-compiler",
-  "version": "4.3.27",
+  "version": "4.3.28",
   "type": "module",
   "description": "Optional Rust compiler for Vibe — pre-renders, inlines components, and optimizes iterations. The runtime never needs it.",
   "homepage": "https://vibe.korte.kim",

package/src/compiler/manifest_builder.rs

@@ -7,6 +7,7 @@ use serde_json::Value;
 use std::cell::RefCell;
 use std::collections::{HashMap, BTreeMap};
 use crate::compiler::iteration_optimizer::build_iteration_optimizations;
+use crate::constants::DEFER_ATTR_PREFIX;
 
 pub struct ManifestBuilder {
     binding_regex: Regex,
@@ -127,7 +128,7 @@ impl ManifestBuilder {
             }
 
             NodeData::Element { name, attrs, .. } => {
-                let _tag_name = name.local.to_string();
+                let tag_name = name.local.to_string();
 
                 // Check attribute bindings (both value and name bindings)
                 let mut attr_bindings = BTreeMap::new();
@@ -170,9 +171,25 @@ impl ManifestBuilder {
                 // Store if we have any bindings
                 let has_name_bindings = name_bindings.iter().any(|b| b.is_some());
 
+                // A fetched component binds only src and key, as the runtime's
+                // parse does: every other attribute is a prop substituted into the
+                // fetched template. Bound here, a prop hydrates as a plain
+                // attribute and a falsy one strips itself before the fetch reads
+                // it. Restoration still carries every binding, so a stamped prop
+                // is back in its raw form before the fetch.
+                let attr_value = |n: &str| borrowed_attrs.iter().find(|a| &*a.name.local == n).map(|a| a.value.to_string());
+                let fetched = (tag_name == "component"
+                    || (tag_name == "div" && attr_value("class").is_some_and(|c| c.split_whitespace().any(|c| c == "component"))))
+                    && (attr_value("src").is_some() || attr_value(&format!("{}src", DEFER_ATTR_PREFIX)).is_some());
+                let bound: BTreeMap<String, String> = attr_bindings
+                    .iter()
+                    .filter(|(k, _)| !fetched || ["src", "key"].contains(&k.as_str()))
+                    .map(|(k, v)| (k.clone(), v.clone()))
+                    .collect();
+
                 // Set runtime-compatible fields at node level
-                if !attr_bindings.is_empty() {
-                    manifest_node.attributes = Some(attr_bindings.clone());
+                if !bound.is_empty() {
+                    manifest_node.attributes = Some(bound);
                 }
                 if has_name_bindings {
                     manifest_node.name_bindings = Some(name_bindings.clone());
@@ -667,6 +684,24 @@ mod tests {
         assert_eq!(result, vec!["Hello World"]);
     }
 
+    // A prop on a fetched component is no attribute binding: hydrated as one, a
+    // falsy `open="@[flag]"` strips itself before the fetch reads it. It is
+    // still restored, so the fetch sees it raw.
+    #[test]
+    fn fetched_component_binds_only_src_and_key() {
+        let html = r#"<component src="@[src]" key="@[id]" open="@[flag]"></component><component open="@[flag]"></component>"#;
+        let manifest = ManifestBuilder::new().build_from_html(html, &json!({}), false).unwrap();
+        let body = manifest.children.get("html_0").unwrap().children.get("body_1").unwrap();
+
+        let fetched = body.children.get("component_0").unwrap();
+        assert_eq!(fetched.attributes.as_ref().unwrap().keys().collect::<Vec<_>>(), ["key", "src"]);
+        let restored = fetched.compiled.as_ref().unwrap().restoration.as_ref().unwrap().attributes.as_ref().unwrap();
+        assert_eq!(restored.get("open").unwrap(), "@[flag]");
+
+        let inline = body.children.get("component_1").unwrap();
+        assert_eq!(inline.attributes.as_ref().unwrap().get("open").unwrap(), "@[flag]");
+    }
+
     #[test]
     fn build_multiple_name_bindings() {
         let html = r#"<div test @[theme] @[size]></div>"#;