memory: add descriptions to the top-level array parameters of create_entities, create_relations, add_observations and delete_observations

Aberta Para iniciantes
  • Algo errado?
#5,416 1 comentário 0 reações 0 responsáveis Ver no GitHub

Mantenedores costumam responder em até 1 dia

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
2/5
Tempo estimado
1-3 horas
Facilidade para iniciantes
88/100
Tipo de issue
Bug
Clareza
Claramente especificada
Status de atividade
Ativa
Stack de tecnologia
typescript
Domínio
api

Direção de pesquisa

Comece em src/memory/index.ts e compare os quatro parâmetros de array com delete_entities.entities, que já tem uma descrição. Verifique os testes existentes para tools/list e adicione cobertura para garantir que todas as propriedades de entrada de nível superior tenham uma descrição não vazia. O trabalho estará concluído quando as quatro descrições estiverem presentes, o teste passar e o mcp-doctor não reportar avisos SCHEMA_PROP_DESC_MISSING.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

enhancement v2
Which server?

memory (@modelcontextprotocol/server-memory)

Protocol era

Modern (2026-07-28)

MCP client

No response

The problem

Four tools expose a top-level array parameter without a description in their input schema: create_entities.entities, create_relations.relations, add_observations.observations and delete_observations.deletions. Nested fields are described, and delete_entities already describes its top-level entityNames array ("An array of entity names to delete"), so these four are inconsistent. Clients and models see only the property name in tools/list. Found with a schema linter (mcp-doctor): 0 errors, 4 warnings (SCHEMA_PROP_DESC_MISSING) for exactly these four. Present in 2.0.1 and on v2/main (72ad22c).

Solution you have in mind (optional)

Prompt (per CONTRIBUTING.md): In src/memory/index.ts, add .describe(...) to the top-level array parameter in the inputSchema of four tools, following delete_entities' style:

  • create_entities.entities: "An array of entities to create"
  • create_relations.relations: "An array of relations to create"
  • add_observations.observations: "An array of entities and the observations to add to each"
  • delete_observations.deletions: "An array of entities and the observations to delete from each"
    Do not change output schemas or behavior. Add a test asserting every top-level input property of every tool in tools/list has a non-empty description.
Which MCP feature does it demonstrate? (optional)

Tools: input schema property descriptions in tools/list (https://modelcontextprotocol.io/specification/2025-06-18/server/tools).

Alternatives or workarounds you have tried

None on the server side. Clients and models currently see only the property names (entities, relations, observations, deletions) for these four parameters.

Already built it locally?

Not built locally. The suggested prompt is in the proposed solution above. Verification: npx @tonyweckstrom/mcp-doctor -- npx -y @modelcontextprotocol/server-memory currently reports 4 SCHEMA_PROP_DESC_MISSING warnings; after the change it should report 0.

Before you submit
  • I searched existing issues and this is not a duplicate.
  • This is a request for this repository or one of its servers, not a new server, the MCP specification or an SDK.
Linguagem predominante
TypeScript
Estrelas
91.6k
Forks
11.9k
Merge médio
4h 31min
PRs com merge (30d)
236

Preparar o ambiente

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de modelcontextprotocol/servers

Todas as issues de modelcontextprotocol/servers

Issues semelhantes

Mais issues de TypeScript

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.