.

Author: SimonCropp

src/Verify/Serialization/Scrubbers/VerifierSettings.cs

@@ -2,7 +2,10 @@
 
 public static partial class VerifierSettings
 {
-    static VerifierSettings() =>
+    static VerifierSettings()
+    {
+        InitBuiltInTypedConverters();
+
         MemberConverter<Exception, string>(
             _ => _.StackTrace,
             (_, value) =>
@@ -14,4 +17,5 @@ static VerifierSettings() =>
 
                 return ScrubStackTrace.Scrub(value);
             });
+    }
 }

src/Verify/Splitters/Settings_Typed.cs

@@ -2,7 +2,16 @@
 
 public static partial class VerifierSettings
 {
-    static List<TypeConverter> typedConverters = [];
+    static List<TypeConverter> typedConverters = null!;
+
+    static void InitBuiltInTypedConverters() =>
+        typedConverters =
+        [
+            // StringBuilder - use "txt" extension
+            new(
+                (target, _) => Task.FromResult(new ConversionResult(null, "txt", (StringBuilder) target)),
+                (target, _) => target is StringBuilder)
+        ];
 
     internal static bool TryGetTypedConverter<T>(
         T target,
@@ -37,7 +46,8 @@ public static void RegisterFileConverter<T>(
     {
         InnerVerifier.ThrowIfVerifyHasBeenRun();
         var converter = new TypeConverter((target, context) => conversion((T) target, context), DefaultCanConvert(canConvert));
-        typedConverters.Add(converter);
+        // Insert at beginning so user converters take precedence over built-in converters
+        typedConverters.Insert(0, converter);
     }
 
     public static void RegisterFileConverter(
@@ -56,7 +66,8 @@ public static void RegisterFileConverter(
     {
         InnerVerifier.ThrowIfVerifyHasBeenRun();
         var converter = new TypeConverter(conversion, canConvert);
-        typedConverters.Add(converter);
+        // Insert at beginning so user converters take precedence over built-in converters
+        typedConverters.Insert(0, converter);
     }
 
     static CanConvert DefaultCanConvert<T>(CanConvert<T>? canConvert)

src/Verify/Verifier/InnerVerifier_Inner.cs

@@ -94,7 +94,7 @@ async Task<VerifyResult> VerifyInner(object? root, Func<Task>? cleanup, IEnumera
         return (cleanup, resultTargets);
     }
 
-    async Task<(List<ResolvedTarget> targets, Func<Task> cleanup, bool applyScrubbers, List<object> converterInfos)> ResolveTarget(Target target)
+    async Task<(List<ResolvedTarget> targets, Func<Task> cleanup, bool applyScrubbers, List<object> converterInfos)> ResolveTarget(Target target, string? skipConverterForExtension = null)
     {
         var cleanup = () => Task.CompletedTask;
         var results = new List<ResolvedTarget>();
@@ -105,13 +105,23 @@ async Task<VerifyResult> VerifyInner(object? root, Func<Task>? cleanup, IEnumera
         {
             if (target.TryGetStream(out var stream))
             {
-                if (target.PerformConversion)
+                // Skip converter if same extension (prevents infinite recursion)
+                if (target.PerformConversion && target.Extension != skipConverterForExtension)
                 {
                     var (streamResults, streamCleanup, infos) = await ResolveStream(stream, target.Extension, target.Name);
                     return (streamResults, streamCleanup, true, infos);
                 }
 
-                results.Add(new(target.Extension, stream, target.Name));
+                // Direct conversion without converter
+                stream.MoveToStart();
+                if (FileExtensions.IsTextExtension(target.Extension))
+                {
+                    results.Add(new(target.Extension, await stream.ReadStringBuilderWithFixedLines(), target.Name));
+                }
+                else
+                {
+                    results.Add(new(target.Extension, stream, target.Name));
+                }
             }
             else if (target.TryGetStringBuilder(out var sb))
             {
@@ -180,13 +190,6 @@ async Task<VerifyResult> VerifyInner(object? root, Func<Task>? cleanup, IEnumera
             throw new("Use Verify(IEnumerable<T> targets, string extension)");
         }
 
-        // Handle StringBuilder - apply scrubbers
-        if (data is StringBuilder sb2)
-        {
-            results.Add(new("txt", sb2, target.Name));
-            return (results, cleanup, true, converterInfos);
-        }
-
         // Handle string - check for JSON appenders first (matches TryGetRootTarget behavior)
         if (data is string str)
         {
@@ -286,9 +289,10 @@ async Task<VerifyResult> VerifyInner(object? root, Func<Task>? cleanup, IEnumera
             }
 
             // Recursively resolve converted targets (they may contain objects)
+            // Pass extension to prevent infinite recursion for same-extension targets
             foreach (var convTarget in converted)
             {
-                var (resolved, resolveCleanup, _, nestedInfos) = await ResolveTarget(convTarget);
+                var (resolved, resolveCleanup, _, nestedInfos) = await ResolveTarget(convTarget, extension);
                 cleanup += resolveCleanup;
                 results.AddRange(resolved);
                 converterInfos.AddRange(nestedInfos);