Fix: Passing property to mutable component argument

fastn-js/src/component_invocation.rs

@@ -44,7 +44,7 @@ pub enum ElementKind {
 #[derive(Debug)]
 pub struct InstantiateComponent {
     pub component: InstantiateComponentData,
-    pub arguments: Vec<(String, fastn_js::SetPropertyValue)>,
+    pub arguments: Vec<(String, fastn_js::SetPropertyValue, bool)>,
     pub parent: String,
     pub inherited: String,
     pub should_return: bool,
@@ -66,7 +66,7 @@ pub enum InstantiateComponentData {
 impl InstantiateComponent {
     pub fn new(
         component_name: &str,
-        arguments: Vec<(String, fastn_js::SetPropertyValue)>,
+        arguments: Vec<(String, fastn_js::SetPropertyValue, bool)>,
         parent: &str,
         inherited: &str,
         should_return: bool,
@@ -86,7 +86,7 @@ impl InstantiateComponent {
 
     pub fn new_with_definition(
         component_definition: fastn_js::SetPropertyValue,
-        arguments: Vec<(String, fastn_js::SetPropertyValue)>,
+        arguments: Vec<(String, fastn_js::SetPropertyValue, bool)>,
         parent: &str,
         inherited: &str,
         should_return: bool,

fastn-js/src/to_js.rs

@@ -259,8 +259,16 @@ impl fastn_js::InstantiateComponent {
                 text("{")
                     .append(
                         pretty::RcDoc::intersperse(
-                            self.arguments.iter().map(|(k, v)| {
-                                format!("{}: {}", fastn_js::utils::name_to_js_(k), v.to_js())
+                            self.arguments.iter().map(|(k, value, is_mutable)| {
+                                format!(
+                                    "{}: {}",
+                                    fastn_js::utils::name_to_js_(k),
+                                    if *is_mutable {
+                                        format!("fastn.mutable({})", value.to_js())
+                                    } else {
+                                        value.to_js()
+                                    }
+                                )
                             }),
                             comma().append(space()),
                         )

ftd/src/js/utils.rs

@@ -257,7 +257,7 @@ pub(crate) fn get_set_property_values_for_provided_component_properties(
     component_properties: &[ftd::interpreter::Property],
     line_number: usize,
     has_rive_components: &mut bool,
-) -> Option<Vec<(String, fastn_js::SetPropertyValue)>> {
+) -> Option<Vec<(String, fastn_js::SetPropertyValue, bool)>> {
     use itertools::Itertools;
 
     // Attempt to retrieve component or web component arguments
@@ -280,6 +280,7 @@ pub(crate) fn get_set_property_values_for_provided_component_properties(
                                 has_rive_components,
                                 false,
                             ),
+                            v.mutable,
                         )
                     })
                 })

ftd/t/js/65-counter.ftd

@@ -0,0 +1,26 @@
+-- counter:
+$count: 10
+
+/-- counter: 10
+
+-- component counter:
+caption integer $count: 20
+
+-- ftd.row:
+border-width.px: 2
+padding.px: 20
+spacing.fixed.px: 20
+background.solid if { counter.count % 2 == 0 }: yellow
+border-radius.px: 5
+
+-- ftd.text: ➕
+$on-click$: $ftd.increment-by($a=$counter.count, v=1)
+
+-- ftd.integer: $counter.count
+
+-- ftd.text: ➖
+$on-click$: $ftd.increment-by($a=$counter.count, v=-1)
+
+-- end: ftd.row
+
+-- end: counter