server: registeredResource.update and registeredResourceTemplate.update desync title and metadata.title when updating metadata alone
Les mainteneurs répondent en général sous 1 jour
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 76/100
- Type d'issue
- Bug
- Clarté
- Clairement spécifiée
- Activité
- Active
- Stack technique
- typescript
- Domaine
- api
Piste de recherche
Commencez dans server/src/server/mcp.ts, au niveau des deux méthodes de mise à jour autour des lignes 783-787 et 824-828. Lisez comment chaque méthode applique updates.metadata et comment les méthodes de liste des ressources combinent title avec les métadonnées. Examinez les tests existants pour les mises à jour des ressources et des modèles enregistrés, puis ajoutez des tests pour les mises à jour portant uniquement sur les métadonnées ; le travail est terminé lorsque le titre et le titre des métadonnées concordent dans les résultats de la liste.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
While checking resource metadata updates in packages/server/src/server/mcp.ts (lines 783-787 and 824-828 on main commit b381559), I noticed _createRegisteredResource and _createRegisteredResourceTemplate copy updates.title into registeredResource.metadata when updates.title !== undefined, since resources/list and resources/templates/list spread ...resource.metadata after title: resource.title.
However, when update({ metadata }) is called without updates.title, registeredResource.metadata = updates.metadata replaces metadata without syncing registeredResource.title (so passing metadata: { title: 'New' } leaves registeredResource.title stale, while passing metadata with title: undefined overwrites title: resource.title with undefined in resources/list). Should update() keep registeredResource.title and registeredResource.metadata.title in sync when updates.metadata is provided? Can put up a small PR if that makes sense!
- Langage dominant
- TypeScript
- Étoiles
- 13.7k
- Forks
- 2.3k
- Merge moyen
- 1 j 2 h
- PR mergées (30 j)
- 60
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
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/typescript-sdk
-
The `maxToolInputElements` refusal lacks the `Input validation error:` prefix that the schema refusal carriesPeut-être pris @barry166 l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
modelcontextprotocol/typescript-sdk#3111 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
v2
Difficulté 2/5 1-3 heures Accessibilité débutants 71/100
modelcontextprotocol/typescript-sdk#3109 ·
Les mainteneurs répondent en général sous 1 jour
-
[v2] server/client 2.4.0 inline fast-uri 3.1.8; four advisories published after release are fixed in 3.1.9Peut-être pris @claude l’a pris aujourd’hui. Ouvertev2
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
modelcontextprotocol/typescript-sdk#3101 ·
Les mainteneurs répondent en général sous 1 jour
-
[v1.x] Protocol.request keeps its abort listener after the request resolved, so a later abort sends notifications/cancelled for an answered request (1.30.0)Peut-être pris Une pull request liée à cette issue est ouverte ou déjà fusionnée. Ouvertev1
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
modelcontextprotocol/typescript-sdk#3090 · 2 commentaires ·
Les mainteneurs répondent en général sous 1 jour
-
[v2] e2e `protocol:timeout:max-total` is flaky over SSE: fake timers advance before the request reaches the serverPeut-être pris @Singmin l’a pris il y a 8 jours. Ouvertev1 v2
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
modelcontextprotocol/typescript-sdk#3045 ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de modelcontextprotocol/typescript-sdk
Issues similaires
-
backend refactor
Difficulté 2/5 1-3 heures Accessibilité débutants 73/100
NIAEFEUP/tts#927 · 1 commentaire · 1 réaction ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
callstackincubator/react-native-harness#296 ·
Les mainteneurs répondent en général sous 1 jour
-
Editor bug with grouped nodesOuvertestatus:in-linear
Difficulté 2/5 1-3 heures Accessibilité débutants 67/100
n8n-io/n8n#42293 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
verified
Difficulté 2/5 1-3 heures Accessibilité débutants 67/100
Les mainteneurs répondent en général sous 5 jours
-
Add inputmode attribute support (forwarded to the inner input)Peut-être à nouveau libre Une pull request pour cette issue a été fermée sans être fusionnée. Ouverteenhancement
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
Neovici/cosmoz-input#467 ·
Les mainteneurs répondent en général sous 1 jour