Move Clone to GeneratBindingPropertyResolvers

Author: exyi

src/Framework/Framework/Compilation/Binding/GeneralBindingPropertyResolvers.cs

@@ -142,7 +142,8 @@ public SimplePathExpressionBindingProperty FormatSimplePath(IBinding binding)
                 // if contains api parameter, can't use this as a path
                 if (js.Expression.DescendantNodes().Any(n => n.TryGetAnnotation(out ViewModelInfoAnnotation? vmInfo) && vmInfo.ExtensionParameter is RestApiRegistrationHelpers.ApiExtensionParameter apiParameter))
                     throw new Exception($"Can't get a path expression for command binding from binding that is using rest api.");
-                return new(js.Expression.FormatParametrizedScript());
+                var expr = js.Expression.Clone(); // need to Clone, because js.Expression is frozen and not parethesized
+                return new(expr.FormatParametrizedScript());
             }
             else if (binding.GetProperty<KnockoutExpressionBindingProperty>(ErrorHandlingMode.ReturnNull) is { } expr)
             {

src/Framework/Framework/Compilation/Javascript/Ast/JsAstHelpers.cs

@@ -73,8 +73,6 @@ public static ParametrizedCode FormatParametrizedScript(this JsNode node, bool n
         {
             if (node is null)
                 throw new ArgumentNullException(nameof(node));
-            // FixParenthesis modifies the tree in-place; if the node is frozen we must clone it first
-            if (node.IsFrozen) node = node.Clone();
             node = node.FixParenthesis();
             var visitor = new JsFormattingVisitor(niceMode, indent);
             node.AcceptVisitor(visitor);