fix(decorator): hoist consts referenced by emitted Ivy definitions

Cargo.toml

@@ -95,6 +95,7 @@ oxc_napi = "0.133"
 oxc_parser = "0.133"
 oxc_semantic = "0.133"
 oxc_span = "0.133"
+oxc_syntax = "0.133"
 oxc_sourcemap = "7.0.0"
 oxc_str = "0.133"
 oxc_transformer = "0.133"

crates/oxc_angular_compiler/Cargo.toml

@@ -18,10 +18,12 @@ doctest = false
 [dependencies]
 oxc_allocator = { workspace = true }
 oxc_ast = { workspace = true }
+oxc_ast_visit = { workspace = true }
 oxc_diagnostics = { workspace = true }
 oxc_parser = { workspace = true }
 oxc_semantic = { workspace = true }
 oxc_span = { workspace = true }
+oxc_syntax = { workspace = true }
 oxc_sourcemap = { workspace = true }
 oxc_str = { workspace = true }
 oxc_transformer = { workspace = true }

crates/oxc_angular_compiler/src/component/mod.rs

@@ -11,6 +11,7 @@ mod cross_file_elision;
 mod decorator;
 mod definition;
 mod dependency;
+mod hoist;
 mod import_elision;
 mod metadata;
 mod namespace_registry;

crates/oxc_angular_compiler/src/component/transform.rs

@@ -27,6 +27,7 @@ use super::decorator::{
     extract_component_metadata, find_component_decorator, find_component_decorator_span,
 };
 use super::definition::{const_value_to_expression, generate_component_definitions};
+use super::hoist::{collect_hoist_edits, program_has_angular_decorated_class};
 use super::import_elision::{ImportElisionAnalyzer, import_elision_edits};
 use super::metadata::{AngularVersion, ComponentMetadata, HostMetadata};
 use super::namespace_registry::NamespaceRegistry;
@@ -2567,6 +2568,33 @@ pub fn transform_angular_file(
         }
     }
 
+    // 5e. TDZ-safe hoisting of top-level bindings referenced by decorator
+    // metadata but declared after the decorated class. Without this, the
+    // emitted `ɵcmp` static field's `ɵɵProvidersFeature` would evaluate the
+    // reference at class-definition time and throw `ReferenceError`. See
+    // issue #287.
+    //
+    // The hoister resolves identifier references through `oxc_semantic` so
+    // a nested-scope shadow of a top-level name can't be mistaken for the
+    // top-level binding itself.
+    //
+    // Gate the Semantic build behind a cheap top-level scan: a real Angular
+    // codebase contains plenty of plain `.ts` helpers, type-only modules, and
+    // services without `@Injectable` that we route through this function. For
+    // those, building a full symbol table just to discover there's nothing to
+    // hoist is pure overhead.
+    if program_has_angular_decorated_class(&parser_ret.program) {
+        // Semantic builder errors (redeclarations, etc.) are intentionally
+        // dropped: the parser already captured syntax errors into
+        // `result.diagnostics` upstream, and Semantic-level diagnostics here
+        // aren't actionable for the hoist pass — we treat the input as
+        // best-effort and rely on the host build to surface genuine errors.
+        // The JIT path (see ~line 1380) follows the same convention.
+        let hoist_semantic =
+            oxc_semantic::SemanticBuilder::new().build(&parser_ret.program).semantic;
+        edits.extend(collect_hoist_edits(&parser_ret.program, source, &hoist_semantic));
+    }
+
     // Apply all edits in one pass
     if options.sourcemap {
         let (code, map) = apply_edits_with_sourcemap(source, edits, path);