fix: preserve object and array spread syntax in template bindings (#260)

* fix: preserve object and array spread syntax in template bindings

Object and array spread expressions in Angular template bindings were
silently dropped during compilation, e.g. `[prop]="{ ...base, key: val }"``
would emit `{ key: val }` with the spread lost entirely.

The fix spans the full compilation pipeline:
- ingest: handle LiteralMapKey::Spread when converting AST→IR
- ir/expression: add spreads: Vec<bool> parallel field to all literal
  map/array IR types to track which entries are spreads
- resolve_names: propagate spreads through name resolution
- pure_literal_structures: emit spread entries in pure function wrappers
- reify/ir_expression: emit spread entries when lowering IR→output AST
- reify/angular_expression: handle SpreadElement in LiteralArray arm
- emit: fix convert_ast_for_pure_function_body to handle spreads in
  both LiteralMap and LiteralArray
- output/ast: clone_in and chaining preserve is_spread on LiteralMapEntry
- output/oxc_converter: handle SpreadProperty in object expressions
- defer_configs, pipe_variadic: maintain spreads parallel-vec invariant

Adds 9 snapshot tests covering object spread, array spread, multiple
spreads, spread-with-pipe, spread-at-end, chained bindings, and arrow
function body spread. Adds TS NAPI tests for both object and array spread.

* fix: include spread metadata in pure function pooling key

Pooled pure-function bodies for `[a]` and `[...a]` (and object equivalents)
collided in the constant pool because the key generation for
DerivedLiteralArray/DerivedLiteralMap ignored the `spreads` parallel array,
causing one binding to silently inherit the other's runtime semantics.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

* style: apply cargo fmt to integration test assertions

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: ashley-hunter

crates/oxc_angular_compiler/src/class_debug_info/compiler.rs

@@ -53,36 +53,36 @@ fn internal_compile_class_debug_info<'a>(
     let mut entries = Vec::new_in(allocator);
 
     // className
-    entries.push(LiteralMapEntry {
-        key: Ident::from("className"),
-        value: literal_string_atom(allocator, debug_info.class_name.clone()),
-        quoted: false,
-    });
+    entries.push(LiteralMapEntry::new(
+        Ident::from("className"),
+        literal_string_atom(allocator, debug_info.class_name.clone()),
+        false,
+    ));
 
     // Include filePath and lineNumber only if filePath is set
     // (matching Angular's behavior - if filePath is null, downstream consumers
     // will typically ignore lineNumber as well)
     if let Some(file_path) = &debug_info.file_path {
-        entries.push(LiteralMapEntry {
-            key: Ident::from("filePath"),
-            value: literal_string_atom(allocator, file_path.clone()),
-            quoted: false,
-        });
-
-        entries.push(LiteralMapEntry {
-            key: Ident::from("lineNumber"),
-            value: literal_number(allocator, debug_info.line_number),
-            quoted: false,
-        });
+        entries.push(LiteralMapEntry::new(
+            Ident::from("filePath"),
+            literal_string_atom(allocator, file_path.clone()),
+            false,
+        ));
+
+        entries.push(LiteralMapEntry::new(
+            Ident::from("lineNumber"),
+            literal_number(allocator, debug_info.line_number),
+            false,
+        ));
     }
 
     // Include forbidOrphanRendering only if it's true (to reduce generated code)
     if debug_info.forbid_orphan_rendering {
-        entries.push(LiteralMapEntry {
-            key: Ident::from("forbidOrphanRendering"),
-            value: literal_bool(allocator, true),
-            quoted: false,
-        });
+        entries.push(LiteralMapEntry::new(
+            Ident::from("forbidOrphanRendering"),
+            literal_bool(allocator, true),
+            false,
+        ));
     }
 
     let debug_info_object = OutputExpression::LiteralMap(Box::new_in(

crates/oxc_angular_compiler/src/class_metadata/builders.rs

@@ -65,11 +65,7 @@ pub fn build_decorator_metadata_array<'a>(
         };
 
         // Add "type" entry
-        map_entries.push(LiteralMapEntry {
-            key: Ident::from("type"),
-            value: type_expr,
-            quoted: false,
-        });
+        map_entries.push(LiteralMapEntry::new(Ident::from("type"), type_expr, false));
 
         // Add "args" entry if the decorator has arguments
         if let Expression::CallExpression(call) = &decorator.expression
@@ -84,14 +80,14 @@ pub fn build_decorator_metadata_array<'a>(
             }
 
             if !args.is_empty() {
-                map_entries.push(LiteralMapEntry {
-                    key: Ident::from("args"),
-                    value: OutputExpression::LiteralArray(Box::new_in(
+                map_entries.push(LiteralMapEntry::new(
+                    Ident::from("args"),
+                    OutputExpression::LiteralArray(Box::new_in(
                         LiteralArrayExpr { entries: args, source_span: None },
                         allocator,
                     )),
-                    quoted: false,
-                });
+                    false,
+                ));
             }
         }
 
@@ -156,22 +152,18 @@ pub fn build_ctor_params_metadata<'a>(
             ))
         });
 
-        map_entries.push(LiteralMapEntry {
-            key: Ident::from("type"),
-            value: type_expr,
-            quoted: false,
-        });
+        map_entries.push(LiteralMapEntry::new(Ident::from("type"), type_expr, false));
 
         // Extract decorators from the parameter
         let param_decorators = extract_angular_decorators_from_param(param);
         if !param_decorators.is_empty() {
             let decorators_array =
                 build_decorator_metadata_array(allocator, &param_decorators, source_text);
-            map_entries.push(LiteralMapEntry {
-                key: Ident::from("decorators"),
-                value: decorators_array,
-                quoted: false,
-            });
+            map_entries.push(LiteralMapEntry::new(
+                Ident::from("decorators"),
+                decorators_array,
+                false,
+            ));
         }
 
         param_entries.push(OutputExpression::LiteralMap(Box::new_in(
@@ -258,11 +250,7 @@ pub fn build_prop_decorators_metadata<'a>(
         let decorators_array =
             build_decorator_metadata_array(allocator, &angular_decorators, source_text);
 
-        prop_entries.push(LiteralMapEntry {
-            key: prop_name,
-            value: decorators_array,
-            quoted: false,
-        });
+        prop_entries.push(LiteralMapEntry::new(prop_name, decorators_array, false));
     }
 
     if prop_entries.is_empty() {