fix: Add null checks for reflection calls (#1442)

Closes #1420

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Opus 4.5 <[email protected]>

Author: thomhurst

src/ModularPipelines/Engine/ModuleExecutor.cs

@@ -487,15 +487,25 @@ private async Task<IModuleResult> ExecuteTypedModule(
         var resultType = module.ResultType;
 
         // Use reflection to call the generic ExecuteAsync method
-        var executeMethod = typeof(IModuleExecutionPipeline).GetMethod(nameof(IModuleExecutionPipeline.ExecuteAsync))!
-            .MakeGenericMethod(resultType);
+        var executeMethodInfo = typeof(IModuleExecutionPipeline).GetMethod(nameof(IModuleExecutionPipeline.ExecuteAsync))
+            ?? throw new InvalidOperationException($"Method '{nameof(IModuleExecutionPipeline.ExecuteAsync)}' not found on type '{nameof(IModuleExecutionPipeline)}'.");
 
-        var task = (Task) executeMethod.Invoke(_executionPipeline, new object[] { module, executionContext, moduleContext, cancellationToken })!;
+        var executeMethod = executeMethodInfo.MakeGenericMethod(resultType);
+
+        var invokeResult = executeMethod.Invoke(_executionPipeline, new object[] { module, executionContext, moduleContext, cancellationToken })
+            ?? throw new InvalidOperationException($"Invocation of '{nameof(IModuleExecutionPipeline.ExecuteAsync)}' returned null.");
+
+        var task = (Task)invokeResult;
         await task;
 
         // Get the result from the completed task
-        var resultProperty = task.GetType().GetProperty("Result")!;
-        return (IModuleResult) resultProperty.GetValue(task)!;
+        var resultProperty = task.GetType().GetProperty("Result")
+            ?? throw new InvalidOperationException($"Property 'Result' not found on task type '{task.GetType().Name}'.");
+
+        var resultValue = resultProperty.GetValue(task)
+            ?? throw new InvalidOperationException($"Property 'Result' returned null for task type '{task.GetType().Name}'.");
+
+        return (IModuleResult)resultValue;
     }
 
     private async Task<IDisposable> WaitForParallelLimiter(Type moduleType)

src/ModularPipelines/Engine/OptionsProvider.cs

@@ -39,7 +39,15 @@ public OptionsProvider(IPipelineServiceContainerWrapper pipelineServiceContainer
 
         foreach (var option in types.Select(t => _serviceProvider.GetService(typeof(IOptions<>).MakeGenericType(t))))
         {
-            yield return option!.GetType().GetProperty("Value", BindingFlags.Public | BindingFlags.Instance)!.GetValue(option);
+            if (option is null)
+            {
+                continue;
+            }
+
+            var valueProperty = option.GetType().GetProperty("Value", BindingFlags.Public | BindingFlags.Instance)
+                ?? throw new InvalidOperationException($"Property 'Value' not found on type '{option.GetType().Name}'.");
+
+            yield return valueProperty.GetValue(option);
         }
     }
 }