fix(angular): strip TS types from RawSource, add expression-body fallback, thread source_text through property decorators

Three reviewer fixes:

1. P1 - RawSource now strips TypeScript type annotations via
   parse-transform-codegen pipeline, preventing invalid JS output
   like `(dep: Dep) => { ... }` in generated code.

2. Medium - Expression-body arrows with unsupported inner expressions
   (e.g., `() => someUnsupportedExpr`) now fall back to RawSource
   instead of returning None.

3. P2 - Thread source_text through property_decorators.rs so
   @input({transform}), @ViewChild, @ContentChild arguments with
   complex expressions are preserved via RawSource fallback.

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

Author: Brooooooklyn

crates/oxc_angular_compiler/src/component/decorator.rs

@@ -241,7 +241,7 @@ pub fn extract_component_metadata<'a>(
         extract_constructor_deps(allocator, class, import_map, has_superclass);
 
     // Extract inputs from @Input decorators on class members
-    metadata.inputs = extract_input_metadata(allocator, class);
+    metadata.inputs = extract_input_metadata(allocator, class, source_text);
 
     // Extract outputs from @Output decorators on class members
     metadata.outputs = extract_output_metadata(allocator, class);

crates/oxc_angular_compiler/src/component/transform.rs

@@ -1636,11 +1636,11 @@ pub fn transform_angular_file(
                     let template = allocator.alloc_str(&template_string);
                     // 4.5 Extract view queries from the class (for @ViewChild/@ViewChildren)
                     // These need to be passed to compile_component_full so predicates can be pooled
-                    let view_queries = extract_view_queries(allocator, class);
+                    let view_queries = extract_view_queries(allocator, class, Some(source));
 
                     // 4.6 Extract content queries from the class (for @ContentChild/@ContentChildren)
                     // Signal-based queries (contentChild(), contentChildren()) are also detected here
-                    let content_queries = extract_content_queries(allocator, class);
+                    let content_queries = extract_content_queries(allocator, class, Some(source));
 
                     // Collect content query property names for .d.ts generation
                     // (before content_queries is moved into compile_component_full)

crates/oxc_angular_compiler/src/directive/decorator.rs

@@ -167,7 +167,7 @@ pub fn extract_directive_metadata<'a>(
     }
 
     // Extract @Input/@Output/@HostBinding/@HostListener from class members
-    builder = builder.extract_from_class(allocator, class);
+    builder = builder.extract_from_class(allocator, class, source_text);
 
     // Detect if ngOnChanges lifecycle hook is implemented
     // Similar to Angular's: const usesOnChanges = members.some(member => ...)

crates/oxc_angular_compiler/src/directive/metadata.rs

@@ -410,9 +410,15 @@ impl<'a> R3DirectiveMetadataBuilder<'a> {
     ///
     /// # Returns
     /// The builder with all extracted metadata added.
-    pub fn extract_from_class(mut self, allocator: &'a Allocator, class: &'a Class<'a>) -> Self {
+    pub fn extract_from_class(
+        mut self,
+        allocator: &'a Allocator,
+        class: &'a Class<'a>,
+        source_text: Option<&'a str>,
+    ) -> Self {
         // Extract inputs from @Input decorators
-        let inputs = super::property_decorators::extract_input_metadata(allocator, class);
+        let inputs =
+            super::property_decorators::extract_input_metadata(allocator, class, source_text);
         for input in inputs {
             self = self.add_input(input);
         }
@@ -424,13 +430,15 @@ impl<'a> R3DirectiveMetadataBuilder<'a> {
         }
 
         // Extract view queries from @ViewChild/@ViewChildren
-        let view_queries = super::property_decorators::extract_view_queries(allocator, class);
+        let view_queries =
+            super::property_decorators::extract_view_queries(allocator, class, source_text);
         for query in view_queries {
             self = self.add_view_query(query);
         }
 
         // Extract content queries from @ContentChild/@ContentChildren
-        let content_queries = super::property_decorators::extract_content_queries(allocator, class);
+        let content_queries =
+            super::property_decorators::extract_content_queries(allocator, class, source_text);
         for query in content_queries {
             self = self.add_query(query);
         }
@@ -597,7 +605,7 @@ mod tests {
         let builder = R3DirectiveMetadataBuilder::new(&allocator)
             .name(Ident::from("TestDirective"))
             .r#type(OutputAstBuilder::variable(&allocator, Ident::from("TestDirective")))
-            .extract_from_class(&allocator, class.unwrap());
+            .extract_from_class(&allocator, class.unwrap(), Some(code));
 
         let metadata = builder.build();
         assert!(metadata.is_some());
@@ -635,7 +643,7 @@ mod tests {
         let builder = R3DirectiveMetadataBuilder::new(&allocator)
             .name(Ident::from("TestDirective"))
             .r#type(OutputAstBuilder::variable(&allocator, Ident::from("TestDirective")))
-            .extract_from_class(&allocator, class.unwrap());
+            .extract_from_class(&allocator, class.unwrap(), Some(code));
 
         let metadata = builder.build();
         assert!(metadata.is_some());
@@ -668,7 +676,7 @@ mod tests {
         let builder = R3DirectiveMetadataBuilder::new(&allocator)
             .name(Ident::from("TestComponent"))
             .r#type(OutputAstBuilder::variable(&allocator, Ident::from("TestComponent")))
-            .extract_from_class(&allocator, class.unwrap());
+            .extract_from_class(&allocator, class.unwrap(), Some(code));
 
         let metadata = builder.build();
         assert!(metadata.is_some());
@@ -701,7 +709,7 @@ mod tests {
         let builder = R3DirectiveMetadataBuilder::new(&allocator)
             .name(Ident::from("TestComponent"))
             .r#type(OutputAstBuilder::variable(&allocator, Ident::from("TestComponent")))
-            .extract_from_class(&allocator, class.unwrap());
+            .extract_from_class(&allocator, class.unwrap(), Some(code));
 
         let metadata = builder.build();
         assert!(metadata.is_some());
@@ -734,7 +742,7 @@ mod tests {
         let builder = R3DirectiveMetadataBuilder::new(&allocator)
             .name(Ident::from("TestDirective"))
             .r#type(OutputAstBuilder::variable(&allocator, Ident::from("TestDirective")))
-            .extract_from_class(&allocator, class.unwrap());
+            .extract_from_class(&allocator, class.unwrap(), Some(code));
 
         let metadata = builder.build();
         assert!(metadata.is_some());
@@ -769,7 +777,7 @@ mod tests {
         let builder = R3DirectiveMetadataBuilder::new(&allocator)
             .name(Ident::from("TestDirective"))
             .r#type(OutputAstBuilder::variable(&allocator, Ident::from("TestDirective")))
-            .extract_from_class(&allocator, class.unwrap());
+            .extract_from_class(&allocator, class.unwrap(), Some(code));
 
         let metadata = builder.build();
         assert!(metadata.is_some());
@@ -807,7 +815,7 @@ mod tests {
         let builder = R3DirectiveMetadataBuilder::new(&allocator)
             .name(Ident::from("TestComponent"))
             .r#type(OutputAstBuilder::variable(&allocator, Ident::from("TestComponent")))
-            .extract_from_class(&allocator, class.unwrap());
+            .extract_from_class(&allocator, class.unwrap(), Some(code));
 
         let metadata = builder.build();
         assert!(metadata.is_some());
@@ -837,7 +845,7 @@ mod tests {
         let builder = R3DirectiveMetadataBuilder::new(&allocator)
             .name(Ident::from("EmptyDirective"))
             .r#type(OutputAstBuilder::variable(&allocator, Ident::from("EmptyDirective")))
-            .extract_from_class(&allocator, class.unwrap());
+            .extract_from_class(&allocator, class.unwrap(), Some(code));
 
         let metadata = builder.build();
         assert!(metadata.is_some());
@@ -870,7 +878,7 @@ mod tests {
             .name(Ident::from("TestDirective"))
             .r#type(OutputAstBuilder::variable(&allocator, Ident::from("TestDirective")))
             .add_input(R3InputMetadata::simple(Ident::from("existingInput")))
-            .extract_from_class(&allocator, class.unwrap());
+            .extract_from_class(&allocator, class.unwrap(), Some(code));
 
         let metadata = builder.build();
         assert!(metadata.is_some());