Remove all polyfill emission from source generator

The generator should never emit polyfills into consumer projects.
Instead it detects platform capabilities and generates appropriate code:

- Remove CallerArgumentExpressionAttribute polyfill emission; the
  supportsCallerArgExpr flag now checks both C# 10+ language version
  AND that the attribute exists in the compilation, falling back to
  file/line dispatch when unavailable
- Remove [ModuleInitializer] from generated registration code entirely
- Replace [ModuleInitializer] in ReactiveUI benchmarks with explicit
  static constructor calls using an atomic bool guard pattern
- Update snapshot tests

Author: glennawatson

src/ReactiveUI.Binding.SourceGenerators/BindingGenerator.cs

@@ -22,43 +22,15 @@ public class BindingGenerator : IIncrementalGenerator
     /// <inheritdoc/>
     public void Initialize(IncrementalGeneratorInitializationContext context)
     {
-        // Detect C# language version for CallerArgumentExpression support (C# 10+)
-        var supportsCallerArgExpr = context.ParseOptionsProvider.Select(static (opts, _) =>
-            opts is CSharpParseOptions csharpOpts
-            && csharpOpts.LanguageVersion >= LanguageVersion.CSharp10);
-
-        // Conditionally emit CallerArgumentExpression polyfill when C# 10+ but attribute is missing
-        var needsPolyfill = supportsCallerArgExpr
+        // Detect whether the consumer project supports CallerArgumentExpression (C# 10+ AND attribute available).
+        // When supported, invocation generators use expression-text dispatch; otherwise file/line dispatch.
+        var supportsCallerArgExpr = context.ParseOptionsProvider
             .Combine(context.CompilationProvider)
             .Select(static (data, _) =>
-                data.Left && data.Right.GetTypeByMetadataName(
-                    Constants.CallerArgumentExpressionAttributeMetadataName) is null);
-
-        context.RegisterSourceOutput(needsPolyfill, static (ctx, needs) =>
-        {
-            if (needs)
-            {
-                ctx.AddSource(
-                    "CallerArgumentExpressionAttribute.g.cs",
-                    Constants.CallerArgumentExpressionAttributeSource);
-            }
-        });
-
-        // Conditionally emit ModuleInitializer polyfill for runtimes that don't include it (e.g. .NET Framework)
-        var needsModuleInitPolyfill = context.CompilationProvider
-            .Select(static (compilation, _) =>
-                compilation.GetTypeByMetadataName(
-                    Constants.ModuleInitializerAttributeMetadataName) is null);
-
-        context.RegisterSourceOutput(needsModuleInitPolyfill, static (ctx, needs) =>
-        {
-            if (needs)
-            {
-                ctx.AddSource(
-                    "ModuleInitializerAttribute.g.cs",
-                    Constants.ModuleInitializerAttributeSource);
-            }
-        });
+                data.Left is CSharpParseOptions csharpOpts
+                && csharpOpts.LanguageVersion >= LanguageVersion.CSharp10
+                && data.Right.GetTypeByMetadataName(
+                    Constants.CallerArgumentExpressionAttributeMetadataName) is not null);
 
         // Pipeline A: Shared type detection
         // One pass: sets flags for IRO, INPC, WpfDP, WinUIDP, KVO, etc.

src/ReactiveUI.Binding.SourceGenerators/Constants.cs

@@ -45,43 +45,4 @@ internal static class Constants
     internal const string CallerFilePathAttributeName = "System.Runtime.CompilerServices.CallerFilePathAttribute";
     internal const string CallerLineNumberAttributeName = "System.Runtime.CompilerServices.CallerLineNumberAttribute";
     internal const string CallerArgumentExpressionAttributeMetadataName = "System.Runtime.CompilerServices.CallerArgumentExpressionAttribute";
-    internal const string ModuleInitializerAttributeMetadataName = "System.Runtime.CompilerServices.ModuleInitializerAttribute";
-
-    /// <summary>
-    /// Polyfill source for CallerArgumentExpressionAttribute when targeting runtimes that don't include it.
-    /// Only emitted when C# 10+ is detected and the attribute is not already available.
-    /// </summary>
-    internal const string CallerArgumentExpressionAttributeSource = """
-        // <auto-generated/>
-        #pragma warning disable
-        namespace System.Runtime.CompilerServices
-        {
-            [global::System.AttributeUsage(global::System.AttributeTargets.Parameter, AllowMultiple = false, Inherited = false)]
-            internal sealed class CallerArgumentExpressionAttribute : global::System.Attribute
-            {
-                public CallerArgumentExpressionAttribute(string parameterName)
-                {
-                    ParameterName = parameterName;
-                }
-
-                public string ParameterName { get; }
-            }
-        }
-        """;
-
-    /// <summary>
-    /// Polyfill source for ModuleInitializerAttribute when targeting runtimes that don't include it
-    /// (e.g. .NET Framework 4.x). Only emitted when the attribute is not already available.
-    /// </summary>
-    internal const string ModuleInitializerAttributeSource = """
-        // <auto-generated/>
-        #pragma warning disable
-        namespace System.Runtime.CompilerServices
-        {
-            [global::System.AttributeUsage(global::System.AttributeTargets.Method, Inherited = false)]
-            internal sealed class ModuleInitializerAttribute : global::System.Attribute
-            {
-            }
-        }
-        """;
 }

src/ReactiveUI.Binding.SourceGenerators/Generators/RegistrationGenerator.cs

@@ -13,7 +13,7 @@ namespace ReactiveUI.Binding.SourceGenerators.Generators;
 
 /// <summary>
 /// Consolidates all per-kind fallback generator results into a single RegisterSourceOutput.
-/// Generates all ICreatesObservableForProperty implementations and one [ModuleInitializer] class.
+/// Generates all ICreatesObservableForProperty implementations and a registration class.
 /// </summary>
 internal static class RegistrationGenerator
 {
@@ -58,7 +58,7 @@ internal static IncrementalValueProvider<ImmutableArray<ObservableTypeInfo>> Con
     }
 
     /// <summary>
-    /// Generates the consolidated registration output: all per-kind binder classes + [ModuleInitializer].
+    /// Generates the consolidated registration output: all per-kind binder classes.
     /// </summary>
     /// <param name="context">The source production context.</param>
     /// <param name="allTypes">All detected observable type infos across all notification kinds.</param>
@@ -93,7 +93,6 @@ internal static class __GeneratedBinderRegistration
                     /// <summary>
                     /// Registers all generated binders with the Splat service locator.
                     /// </summary>
-                    [global::System.Runtime.CompilerServices.ModuleInitializer]
                     internal static void Initialize()
                     {
                         // Generated binder registrations will be added here in future phases.

src/benchmarks/ReactiveUI.Binding.Benchmarks.ReactiveUI/ModuleInitializer.cs

@@ -2,25 +2,31 @@
 // ReactiveUI Association Incorporated licenses this file to you under the MIT license.
 // See the LICENSE file in the project root for full license information.
 
-using System.Runtime.CompilerServices;
-
 using ReactiveUI.Builder;
 
 using Splat;
 
 namespace ReactiveUI.Binding.Benchmarks;
 
 /// <summary>
-/// Module initializer that configures ReactiveUI before any benchmarks run.
+/// One-shot initializer that configures ReactiveUI before any benchmarks run.
+/// Call <see cref="EnsureInitialized"/> from each benchmark class's static constructor.
 /// </summary>
 internal static class ModuleInitializer
 {
+    private static int _initialized;
+
     /// <summary>
-    /// Initializes ReactiveUI using the builder pattern.
+    /// Ensures ReactiveUI is initialized exactly once, regardless of how many
+    /// benchmark classes call this method.
     /// </summary>
-    [ModuleInitializer]
-    internal static void Initialize()
+    internal static void EnsureInitialized()
     {
+        if (Interlocked.Exchange(ref _initialized, 1) != 0)
+        {
+            return;
+        }
+
         ModeDetector.OverrideModeDetector(new BenchmarkModeDetector());
         RxAppBuilder.CreateReactiveUIBuilder()
             .WithCoreServices()

src/benchmarks/ReactiveUI.Binding.Benchmarks.ReactiveUI/Polyfills/ModuleInitializerAttribute.cs

@@ -24,6 +24,8 @@ public class ReactiveUIBindingBenchmark
     private BenchmarkViewModel _source = null!;
     private BenchmarkView _target = null!;
 
+    static ReactiveUIBindingBenchmark() => ModuleInitializer.EnsureInitialized();
+
     /// <summary>
     /// Sets up fresh source and target objects before each benchmark iteration.
     /// </summary>

src/benchmarks/ReactiveUI.Binding.Benchmarks.ReactiveUI/ReactiveUIBindingBenchmark.cs

@@ -25,6 +25,8 @@ public class ReactiveUIObservationBenchmark
 
     private BenchmarkViewModel _vm = null!;
 
+    static ReactiveUIObservationBenchmark() => ModuleInitializer.EnsureInitialized();
+
     /// <summary>
     /// Sets up a fresh view model before each benchmark iteration.
     /// </summary>

src/benchmarks/ReactiveUI.Binding.Benchmarks.ReactiveUI/ReactiveUIObservationBenchmark.cs

@@ -14,7 +14,6 @@ internal static class __GeneratedBinderRegistration
         /// <summary>
         /// Registers all generated binders with the Splat service locator.
         /// </summary>
-        [global::System.Runtime.CompilerServices.ModuleInitializer]
         internal static void Initialize()
         {
             // Generated binder registrations will be added here in future phases.

src/tests/ReactiveUI.Binding.SourceGenerators.Tests/BindGeneratorTests.MultipleBindings#GeneratedBinderRegistration.g.verified.cs

@@ -14,7 +14,6 @@ internal static class __GeneratedBinderRegistration
         /// <summary>
         /// Registers all generated binders with the Splat service locator.
         /// </summary>
-        [global::System.Runtime.CompilerServices.ModuleInitializer]
         internal static void Initialize()
         {
             // Generated binder registrations will be added here in future phases.

src/tests/ReactiveUI.Binding.SourceGenerators.Tests/BindGeneratorTests.SingleProperty_StringToString#GeneratedBinderRegistration.g.verified.cs

@@ -14,7 +14,6 @@ internal static class __GeneratedBinderRegistration
         /// <summary>
         /// Registers all generated binders with the Splat service locator.
         /// </summary>
-        [global::System.Runtime.CompilerServices.ModuleInitializer]
         internal static void Initialize()
         {
             // Generated binder registrations will be added here in future phases.