Tool annotation default values are not applied
Les mainteneurs répondent en général sous 2 jours
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 48/100
- Type d'issue
- Bug
- Clarté
- Plutôt claire
- Activité
- Calme
- Stack technique
- csharp
- Domaine
- backend-api-design
Piste de recherche
Commencez dans src/ModelContextProtocol.Core/Server/AIFunctionMcpServerTool.cs autour des lignes 165-188, en comparant les vérifications de nullabilité des champs internes avec les getters des propriétés publiques décrits dans l’issue. Reproduisez les annotations d’outil générées et déterminez si des valeurs null ou des valeurs par défaut documentées sont attendues. La tâche est terminée lorsque le comportement choisi est cohérent et vérifié pour les annotations concernées.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
I was taking a try at the tool annotations. According to the code XML comments, there are default values for them. I was expecting that those default annotations are attached. But it's not, so I took some investigation on source.
The code checks nullability on internal fields instead of public properties. The public properties getter will use the default values while internal fields won't. So the default values are not applied.
Is this intended? Seems a choice between null annotation vs default values annotation.
- Langage dominant
- C#
- Étoiles
- 4.5k
- Forks
- 814
- Merge moyen
- 9 j 19 h
- PR mergées (30 j)
- 4
Préparer son environnement
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de modelcontextprotocol/csharp-sdk
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
modelcontextprotocol/csharp-sdk#1867 ·
Les mainteneurs répondent en général sous 2 jours
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 88/100
modelcontextprotocol/csharp-sdk#1840 · 1 commentaire ·
Les mainteneurs répondent en général sous 2 jours
-
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
modelcontextprotocol/csharp-sdk#1836 ·
Les mainteneurs répondent en général sous 2 jours
-
enhancement needs confirmation
Difficulté 2/5 1-3 heures Accessibilité débutants 64/100
modelcontextprotocol/csharp-sdk#678 · 1 commentaire ·
Les mainteneurs répondent en général sous 2 jours
-
enhancement needs confirmation P3 ready for work
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
modelcontextprotocol/csharp-sdk#515 · 6 commentaires · 3 réactions ·
Les mainteneurs répondent en général sous 2 jours
Toutes les issues de modelcontextprotocol/csharp-sdk
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
microsoft/onnxruntime-genai#2633 ·
Les mainteneurs répondent en général sous 1 jour
-
python triage
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
microsoft/semantic-kernel#14491 ·
Les mainteneurs répondent en général sous 2 jours
-
area:jobads-cv FE mvp P3
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
klasolsson81/jobbliggaren#1878 ·
Les mainteneurs répondent en général sous 1 jour
-
bug help wanted
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
pnp/pnpcore#1878 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
SubtitleEdit/subtitleedit#15331 · 2 commentaires ·
Les mainteneurs répondent en général sous 1 jour