mcp: mock schema is invalid GraphQL, so schema_* tests pass on a parse error
Los mantenedores suelen responder en 1 día
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 35/100
Línea de trabajo
Read internal/mcp/client/mock_client.go and the schema_* subtests in internal/mcp/server_test.go. First inspect the mock schema parse failure and how the tests handle tool errors. Define the missing GraphQL types and make the tests fail on tool errors; done when the schema parses and the tests verify successful tool results. An open linked pull request (#822) is already addressing this work.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
The mock schema in internal/mcp/client/mock_client.go references about 35 types it never defines, for example TeamEnvironment, TeamConnection and Time. Parsing fails with Undefined type TeamEnvironment.
The schema_* subtests in internal/mcp/server_test.go therefore only ever receive the parse error. They still pass because they check for a substring or a non-empty result, and the error text matches. For example, schema_list_queries looks for "me", which appears in "schema".
Fix: define the missing types in the mock, and make the tests fail when a tool returns an error.
- Lenguaje dominante
- Go
- Estrellas
- 2
- Forks
- 5
- Merge medio
- 3 h 49 min
- PR fusionados (30 d)
- 34
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de nais/cli
-
Dokumentasjon på nais defaultsAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
-
Dificultad 3/5 1-2 días Aptitud para principiantes 56/100
nais/cli#827 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
Dificultad 3/5 1-2 días Aptitud para principiantes 57/100
Los mantenedores suelen responder en 1 día
-
mcp: schema_list_queries returns an array as structuredContent, and its outputSchema is invalidPosiblemente ocupada @Starefossen la tomó hace 2 días. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 68/100
Los mantenedores suelen responder en 1 día
-
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
Los mantenedores suelen responder en 1 día
Issues similares
-
area:recipes event:contribfest format:human-first source:agent status:needs-triage type:recipe
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
cadence-workflow/cadence-ai-samples#140 ·
Los mantenedores suelen responder en 1 día
-
ux
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
evcc-io/evcc#35605 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
project-setup
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
Los mantenedores suelen responder en 1 día
-
bug report
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
Los mantenedores suelen responder en 1 día
-
type:bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
nightgauge/nightgauge#2732 ·
Los mantenedores suelen responder en 1 día