fix: Update BingTextSearch to return strongly-typed KernelSearchResults<BingWebPage> per PR #13318

- Change ITextSearch<BingWebPage>.GetSearchResultsAsync to return KernelSearchResults<BingWebPage> instead of <object>
- Add strongly-typed helper GetResultsAsBingWebPageAsync returning IAsyncEnumerable<BingWebPage>
- Update tests to expect strongly-typed results (no manual casting needed)
- Fix duplicate XML comment issue

This aligns with PR #13318 interface type safety improvements and eliminates wasteful casting.

Author: alzarei

dotnet/src/Plugins/Plugins.UnitTests/Web/Bing/BingTextSearchTests.cs

@@ -372,7 +372,7 @@ public async Task GenericGetSearchResultsAsyncWithSnippetContainsFilterProducesC
             Skip = 0,
             Filter = page => page.Snippet!.Contains("semantic")
         };
-        KernelSearchResults<object> result = await textSearch.GetSearchResultsAsync("What is the Semantic Kernel?", searchOptions);
+        KernelSearchResults<BingWebPage> result = await textSearch.GetSearchResultsAsync("What is the Semantic Kernel?", searchOptions);
 
         // Assert - Verify LINQ Snippet.Contains() converted to Bing's inbody: operator
         var requestUris = this._messageHandlerStub.RequestUris;
@@ -852,19 +852,17 @@ public async Task GenericGetSearchResultsAsyncFilterTranslationPreservesBingWebP
             Skip = 0,
             Filter = page => page.Language == "en" && page.DisplayUrl!.Contains("microsoft")
         };
-        KernelSearchResults<object> result = await textSearch.GetSearchResultsAsync("technology", searchOptions);
+        KernelSearchResults<BingWebPage> result = await textSearch.GetSearchResultsAsync("technology", searchOptions);
 
         // Assert - Verify BingWebPage objects have all expected properties
         Assert.NotNull(result);
         Assert.NotNull(result.Results);
         var resultList = await result.Results.ToListAsync();
         Assert.Equal(10, resultList.Count);
-        foreach (var obj in resultList)
+        foreach (var page in resultList)
         {
-            Assert.NotNull(obj);
-            Assert.IsType<BingWebPage>(obj);
-            var page = (BingWebPage)obj;
-            // Verify key properties are populated
+            Assert.NotNull(page);
+            // Verify key properties are populated - now strongly typed, no cast needed!
             Assert.NotNull(page.Name);
             Assert.NotNull(page.Url);
             Assert.NotNull(page.Snippet);

dotnet/src/Plugins/Plugins.Web/Bing/BingTextSearch.cs

@@ -94,10 +94,18 @@ Task<KernelSearchResults<TextSearchResult>> ITextSearch<BingWebPage>.GetTextSear
     }
 
     /// <inheritdoc/>
-    Task<KernelSearchResults<object>> ITextSearch<BingWebPage>.GetSearchResultsAsync(string query, TextSearchOptions<BingWebPage>? searchOptions, CancellationToken cancellationToken)
+    async Task<KernelSearchResults<BingWebPage>> ITextSearch<BingWebPage>.GetSearchResultsAsync(string query, TextSearchOptions<BingWebPage>? searchOptions, CancellationToken cancellationToken)
     {
         var legacyOptions = searchOptions != null ? ConvertToLegacyOptions(searchOptions) : new TextSearchOptions();
-        return this.GetSearchResultsAsync(query, legacyOptions, cancellationToken);
+
+        BingSearchResponse<BingWebPage>? searchResponse = await this.ExecuteSearchAsync(query, legacyOptions, cancellationToken).ConfigureAwait(false);
+
+        long? totalCount = legacyOptions.IncludeTotalCount ? searchResponse?.WebPages?.TotalEstimatedMatches : null;
+
+        return new KernelSearchResults<BingWebPage>(
+            this.GetResultsAsBingWebPageAsync(searchResponse, cancellationToken),
+            totalCount,
+            GetResultsMetadata(searchResponse));
     }
 
     #region private
@@ -452,7 +460,7 @@ private async Task<HttpResponseMessage> SendGetRequestAsync(string query, TextSe
     }
 
     /// <summary>
-    /// Return the search results as instances of <see cref="BingWebPage"/>.
+    /// Return the search results as instances of <see cref="object"/>.
     /// </summary>
     /// <param name="searchResponse">Response containing the web pages matching the query.</param>
     /// <param name="cancellationToken">Cancellation token</param>
@@ -470,6 +478,25 @@ private async IAsyncEnumerable<object> GetResultsAsWebPageAsync(BingSearchRespon
         }
     }
 
+    /// <summary>
+    /// Return the search results as strongly-typed <see cref="BingWebPage"/> instances.
+    /// </summary>
+    /// <param name="searchResponse">Response containing the web pages matching the query.</param>
+    /// <param name="cancellationToken">Cancellation token</param>
+    private async IAsyncEnumerable<BingWebPage> GetResultsAsBingWebPageAsync(BingSearchResponse<BingWebPage>? searchResponse, [EnumeratorCancellation] CancellationToken cancellationToken)
+    {
+        if (searchResponse is null || searchResponse.WebPages is null || searchResponse.WebPages.Value is null)
+        {
+            yield break;
+        }
+
+        foreach (var webPage in searchResponse.WebPages.Value)
+        {
+            yield return webPage;
+            await Task.Yield();
+        }
+    }
+
     /// <summary>
     /// Return the search results as instances of <see cref="TextSearchResult"/>.
     /// </summary>