Fix: Correct Result type instantiation and stream read handling

- Fix ConfigLoader.cs: Use 'new' keyword for Result<T> record instantiation
- Fix FileUtilities.cs: Handle partial reads properly to address CA2022 warning

Author: claude

Services/ConfigLoader.cs

@@ -35,11 +35,11 @@ private static Result<string> ReadConfigFile(string fileName)
         try
         {
             var json = File.Exists(fileName) ? File.ReadAllText(fileName) : "{}";
-            return Result<string>.Success(json);
+            return new Result<string>.Success(json);
         }
         catch (Exception ex)
         {
-            return Result<string>.Error(ex.Message);
+            return new Result<string>.Error(ex.Message);
         }
     }
 

Utils/FileUtilities.cs

@@ -58,10 +58,10 @@ private static bool HasUtf8Bom(FileStream stream)
         }
 
         var bom = new byte[Utf8Bom.Length];
-        stream.Read(bom, 0, Utf8Bom.Length);
+        var bytesRead = stream.Read(bom, 0, Utf8Bom.Length);
         stream.Position = 0;
 
-        return bom.SequenceEqual(Utf8Bom);
+        return bytesRead == Utf8Bom.Length && bom.SequenceEqual(Utf8Bom);
     }
 
     /// <summary>