Rename CustomAsyncExtractorInterface to ICustomAsyncExtractor per code review

Co-authored-by: gfs <98900+gfs@users.noreply.github.com>

Author: Copilot

README.md

@@ -167,7 +167,7 @@ catch(OverflowException)
 <br/>
 You can extend RecursiveExtractor with custom extractors to support additional archive or file formats not natively supported. This is useful for formats like MSI, MSP, or other proprietary archive formats.
 
-To create a custom extractor, implement the `CustomAsyncExtractorInterface` interface and register it with the extractor:
+To create a custom extractor, implement the `ICustomAsyncExtractor` interface and register it with the extractor:
 
 ```csharp
 using Microsoft.CST.RecursiveExtractor;
@@ -177,7 +177,7 @@ using System.Collections.Generic;
 using System.Linq;
 
 // Example: Custom extractor for a hypothetical archive format with magic bytes "MYARC"
-public class MyCustomExtractor : CustomAsyncExtractorInterface
+public class MyCustomExtractor : ICustomAsyncExtractor
 {
     private readonly Extractor context;
     private static readonly byte[] MAGIC_BYTES = System.Text.Encoding.ASCII.GetBytes("MYARC");

RecursiveExtractor.Tests/ExtractorTests/CustomExtractorTests.cs

@@ -16,7 +16,7 @@ public class CustomExtractorTests
     /// A simple test custom extractor that extracts files with a specific magic number
     /// For testing purposes, it recognizes files starting with "CUSTOM1"
     /// </summary>
-    private class TestCustomExtractor : CustomAsyncExtractorInterface
+    private class TestCustomExtractor : ICustomAsyncExtractor
     {
         private readonly Extractor context;
         private static readonly byte[] MAGIC_BYTES = System.Text.Encoding.ASCII.GetBytes("CUSTOM1");
@@ -71,7 +71,7 @@ public async IAsyncEnumerable<FileEntry> ExtractAsync(FileEntry fileEntry, Extra
     /// <summary>
     /// A second test custom extractor that recognizes files starting with "CUSTOM2"
     /// </summary>
-    private class SecondTestCustomExtractor : CustomAsyncExtractorInterface
+    private class SecondTestCustomExtractor : ICustomAsyncExtractor
     {
         private readonly Extractor context;
         private static readonly byte[] MAGIC_BYTES = System.Text.Encoding.ASCII.GetBytes("CUSTOM2");

RecursiveExtractor/Extractor.cs

@@ -45,7 +45,7 @@ public Extractor()
         /// Collection of custom extractors that can handle file types not natively supported.
         /// These are checked when a file type is detected as UNKNOWN.
         /// </summary>
-        internal HashSet<CustomAsyncExtractorInterface> CustomExtractors { get; } = new HashSet<CustomAsyncExtractorInterface>();
+        internal HashSet<ICustomAsyncExtractor> CustomExtractors { get; } = new HashSet<ICustomAsyncExtractor>();
 
         /// <summary>
         /// Set up the Default Extractors compatible with this platform.
@@ -109,7 +109,7 @@ public void ClearExtractors()
         /// </summary>
         /// <param name="customExtractor">The custom extractor implementation to add.</param>
         /// <returns>True if the extractor was added, false if it was already present.</returns>
-        public bool AddCustomExtractor(CustomAsyncExtractorInterface customExtractor)
+        public bool AddCustomExtractor(ICustomAsyncExtractor customExtractor)
         {
             if (customExtractor == null)
             {
@@ -123,7 +123,7 @@ public bool AddCustomExtractor(CustomAsyncExtractorInterface customExtractor)
         /// </summary>
         /// <param name="customExtractor">The custom extractor to remove.</param>
         /// <returns>True if the extractor was removed, false if it was not found.</returns>
-        public bool RemoveCustomExtractor(CustomAsyncExtractorInterface customExtractor)
+        public bool RemoveCustomExtractor(ICustomAsyncExtractor customExtractor)
         {
             if (customExtractor == null)
             {
@@ -356,7 +356,7 @@ public async IAsyncEnumerable<FileEntry> ExtractAsync(string filename, byte[] ar
         /// </summary>
         /// <param name="fileEntry">The file entry to check.</param>
         /// <returns>A custom extractor that can handle the file, or null if none found.</returns>
-        private CustomAsyncExtractorInterface? FindMatchingCustomExtractor(FileEntry fileEntry)
+        private ICustomAsyncExtractor? FindMatchingCustomExtractor(FileEntry fileEntry)
         {
             foreach (var customExtractor in CustomExtractors)
             {

RecursiveExtractor/Extractors/CustomAsyncExtractorInterface.cs renamed to RecursiveExtractor/Extractors/ICustomAsyncExtractor.cs

@@ -6,7 +6,7 @@ namespace Microsoft.CST.RecursiveExtractor.Extractors
     /// An interface for custom extractors that can determine if they can handle a given stream.
     /// This allows library users to extend the extractor with support for additional archive types.
     /// </summary>
-    public interface CustomAsyncExtractorInterface : AsyncExtractorInterface
+    public interface ICustomAsyncExtractor : AsyncExtractorInterface
     {
         /// <summary>
         /// Determines if this extractor can extract the given stream based on binary signatures or other criteria.