SchemaValidator rejects nested empty JSON objects as arrays
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- Medio día
- Aptitud para principiantes
- 60/100
Línea de trabajo
Start in the SchemaValidator class at its validateAgainstJsonSchema method, and find where it converts an empty root value to stdClass. Run the reproduction from the issue, which decodes '{"parameters":{}}' with the parameters property typed as object, and confirm the nested empty array is rejected. Done means the nested empty value is cast to stdClass when the schema says object, with a regression test for that case and the existing root-level behaviour still passing.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Describe the bug
Nested empty objects in tool arguments are rejected as arrays.
To Reproduce
use Mcp\Capability\Discovery\SchemaValidator;
$data = json_decode(
'{"parameters":{}}',
true,
flags: JSON_THROW_ON_ERROR,
);
$schema = [
'type' => 'object',
'properties' => [
'parameters' => ['type' => 'object'],
],
];
// Invalid type. Expected `object`, but received `array`.
var_dump((new SchemaValidator())->validateAgainstJsonSchema($data, $schema));
Expected behavior
No validation errors. The original JSON value is an object.
Additional context
json_decode($foo, true) converts both {} and [] to an empty PHP array. The SchemaValidator converts an empty root value to stdClass, but nested empty values remain arrays. The empty arrays should probably be converted recursively according to the schema.
- Lenguaje dominante
- PHP
- Estrellas
- 1.6k
- Forks
- 180
- Merge medio
- 3 d 12 min
- PR fusionados (30 d)
- 44
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
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 modelcontextprotocol/php-sdk
-
[Server] Handler type uses bare Closure, hard to decorate RegistryInterface under strict PHPStanAbiertoServer
Dificultad 1/5 Menos de una hora Aptitud para principiantes 78/100
modelcontextprotocol/php-sdk#468 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
Builder::build() silently skips configured file-based discovery when symfony/finder is missing — should fail loudlyPosiblemente ocupada @ousamabenyounes la tomó hace 57 días. Abiertoneeds confirmation needs maintainer action Server
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
modelcontextprotocol/php-sdk#398 · 1 reacción ·
Los mantenedores suelen responder en 1 día
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
modelcontextprotocol/php-sdk#370 ·
Los mantenedores suelen responder en 1 día
-
enhancement
Dificultad 4/5 3-5 días Aptitud para principiantes 52/100
modelcontextprotocol/php-sdk#587 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 18/100
modelcontextprotocol/php-sdk#583 ·
Los mantenedores suelen responder en 1 día
Todos los issues de modelcontextprotocol/php-sdk
Issues similares
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
Los mantenedores suelen responder en 1 día
-
sync-en
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
Los mantenedores suelen responder en 1 día
-
sync-en
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
Los mantenedores suelen responder en 4 días
-
Перевод устарел
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
-
Adapter: Algolia Performance
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100