Skip to content

Conversation

migeyusu
Copy link

@migeyusu migeyusu commented Sep 3, 2025

… take use of param "count" and "skip" by tool calls.

Motivation and Context

Fixed bug: method 'CreateGetSearchResults' has a bug that it could not take use of param "count" and "skip" by tool calls.

Description

I add following lines to capture params:

if (arguments.TryGetValue("count", out var countObj) && countObj is int countVal and > 0)
            {
                count = countVal;
            }

            if (arguments.TryGetValue("skip", out var skipObj) && skipObj is int skipVal and >= 0)
            {
                skip = skipVal;
            }

Contribution Checklist

… take use of param "count" and "skip" by tool calls.
@migeyusu migeyusu requested a review from a team as a code owner September 3, 2025 06:06
@moonbox3 moonbox3 added the .NET Issue or Pull requests regarding .NET code label Sep 3, 2025
@github-actions github-actions bot changed the title fixed bug in TextSearchExtensions.cs: CreateGetSearchResults will not… .Net: fixed bug in TextSearchExtensions.cs: CreateGetSearchResults will not… Sep 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
.NET Issue or Pull requests regarding .NET code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants