fix(analyzer): don't raise COA1001 for ItemResult pass to object parameter

Author: ascott18

src/IntelliTect.Coalesce.Analyzer.Tests/Coalesce1001_SimplifyItemResultTests.cs

@@ -665,6 +665,23 @@ public Task GetResult()
             """);
     }
 
+    [Fact]
+    public async Task ArgumentToObjectParameter_NoWarning()
+    {
+        await VerifyAnalyzerAsync("""
+            public class TestClass
+            {
+                public object GetResult()
+                {
+                    string error = "error";
+                    return BadRequest(new ItemResult(error));
+                }
+
+                private object BadRequest(object obj) => obj;
+            }
+            """);
+    }
+
     [Fact]
     public async Task ItemResultGeneric_PropertyInitializer_ErrorMessageWithObjectNull_ReportsInfo()
     {

src/IntelliTect.Coalesce.Analyzer/Analyzers/Coalesce1001_SimplifyItemResult.cs

@@ -291,14 +291,29 @@ initializer.Parent is IVariableDeclaratorOperation varOp &&
             return false;
         }
 
-        return parent switch
+        // Sometimes the object creation is wrapped in a conversion operation
+        // (e.g., when passing ItemResult to a parameter of type object)
+        if (parent is IConversionOperation conversion)
         {
-            // Don't suggest when the ItemResult is used as an argument to another method
-            // where implicit conversion might not work if the target method is generic
-            IArgumentOperation => false,
+            parent = conversion.Parent;
+        }
 
-            // For all other cases, suggest simplification
-            _ => true
-        };
+        if (parent is IArgumentOperation argOp)
+        {
+            if (argOp.Parameter?.OriginalDefinition.Type.TypeKind == TypeKind.TypeParameter)
+            {
+                // Don't suggest simplification for generic type parameters
+                // E.g., Task.FromResult(new ItemResult(error))
+                return false;
+            }
+
+            // Only suggest simplification if the parameter is ItemResult
+            if (argOp.Parameter?.Type is { } type && !IsItemResultType(type))
+            {
+                return false;
+            }
+        }
+
+        return true;
     }
 }