McpServerToolCreateOptions miss MarshalResult option.
まだ誰も着手していません。
評価
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 初心者へのやさしさ
- 68/100
- issue の種類
- 機能追加
- 明瞭さ
- おおむね明確
- 活発さ
- 活発
- 技術スタック
- csharp
調査の方向性
McpServerToolCreateOptions、AIFunctionFactoryOptions、McpServerTool.Create、DeriveOptions、CreateAIFunctionFactoryOptions から始めて、作成設定がどのように渡されるかを追跡します。ツール作成オプションを通じて MarshalResult を構成できる、サポートされた方法を追加し、その後、構成した callback が reflection なしで AIFunctionFactoryOptions に到達することを確認します。
索引モデルが issue の本文から書いたものです。
説明
Is your feature request related to a problem? Please describe.
i need MarshalResult for AIFunctionFactoryOptions, but the CreateAIFunctionFactoryOptions assign the MarshalResult=new ValueTask(result).
i without any solution to reset it, i just can to create a AIFunction,then use the AIFunction to create McpServerTool.
but i will loss many settings from DeriveOptions and CreateAIFunctionFactoryOptions(private methods).
Describe the solution you'd like
please add a MarhsalResult option in McpServerToolCreateOptions, then i can initialize the AIFunctionFactoryOptions.MarshalResult.
or add a Action< AIFunctionFactoryOptions > initailizeAIFunctionOptions, then call after CreateAIFunctionFactoryOptions, give me a chance to update the aifunction options.
public new static AIFunctionMcpServerResource Create(MethodInfo method, object? target, McpServerResourceCreateOptions? options
, Action<AIFunctionFactoryOptions> initializeAIFunctionFactoryOptions = null)
{
Throw.IfNull(method, "method");
options = DeriveOptions(method, options);
var aioptions = CreateAIFunctionFactoryOptions(method, options);
initializeAIFunctionFactoryOptions?.Invoke(aioptions);
return Create(AIFunctionFactory.Create(method, target, aioptions), options);
}
public new static AIFunctionMcpServerTool Create(MethodInfo method, object? target, McpServerToolCreateOptions? options
, Action<AIFunctionFactoryOptions> initializeAIFunctionFactoryOptions = null)
{
Throw.IfNull(method, "method");
options = DeriveOptions(method, options);
var aioptions = CreateAIFunctionFactoryOptions(method, options);
initializeAIFunctionFactoryOptions?.Invoke(aioptions);
return Create(AIFunctionFactory.Create(method, target, aioptions), options);
}`
public new static AIFunctionMcpServerPrompt Create(MethodInfo method, object? target, McpServerPromptCreateOptions? options
, Action<AIFunctionFactoryOptions> initializeAIFunctionFactoryOptions = null)
{
Throw.IfNull(method, "method");
options = DeriveOptions(method, options);
var aioptions = CreateAIFunctionFactoryOptions(method, options);
initializeAIFunctionFactoryOptions?.Invoke(aioptions);
return Create(AIFunctionFactory.Create(method, target, aioptions), options);
}
Describe alternatives you've considered
currently , i just can use reflection to invoke DeriveOptions and CreateAIFunctionFactoryOptions,
but this was not stable in the future version. please add a standard MarshalResult options for my feature.
var createoptions = new McpServerToolCreateOptions()
{
Name = mcpname,
SerializerOptions = serializeroptions,
};
createoptions = (McpServerToolCreateOptions)DummyInstance.Value.Tool_DeriveOptions.Invoke(null, [mcpmethod, createoptions]);
var aifuncoptions = (AIFunctionFactoryOptions)DummyInstance.Value.Tool_CreateAIFunctionFactoryOptions.Invoke(null, [mcpmethod, createoptions]);
aifuncoptions.MarshalResult = OnToolMarshalResult;
var aifunc = AIFunctionFactory.Create(mcpmethod, instance, aifuncoptions);
var tool = McpServerTool.Create(aifunc, createoptions);
log.Info($"McpTool 已建立 - tool:{tool}。\r\n{mcpmethod}");
tools.Add(tool);
- 主要言語
- C#
- スター
- 4.5k
- フォーク
- 814
- 平均マージ
- 9日 19時間
- マージ済み PR(30日)
- 4
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
modelcontextprotocol/csharp-sdk のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
modelcontextprotocol/csharp-sdk#1867 ·
-
難易度 1/5 1時間未満 初心者へのやさしさ 88/100
modelcontextprotocol/csharp-sdk#1840 · コメント 1 件 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
modelcontextprotocol/csharp-sdk#1836 ·
-
enhancement needs confirmation
難易度 2/5 1〜3時間 初心者へのやさしさ 64/100
modelcontextprotocol/csharp-sdk#678 · コメント 1 件 ·
-
enhancement needs confirmation P3 ready for work
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
modelcontextprotocol/csharp-sdk#515 · コメント 6 件 · リアクション 3 件 ·
modelcontextprotocol/csharp-sdk の issue をすべて見る
似ている issue
-
bug
難易度 1/5 1時間未満 初心者へのやさしさ 90/100
-
Type: enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
apache/arrow-adbc#4809 ·
-
type/automation type/tech-debt
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
microsoft/vscode-azurefunctions#5197 · コメント 1 件 ·
-
難易度 1/5 1時間未満 初心者へのやさしさ 85/100
microsoft/microsoft-ui-reactor#1274 ·