a2aproject / a2aproject/a2a-tck

[Bug]: inconsistent snake_case usage for REST

Abierto
#97 0 comentarios 0 reacciones 0 asignados Ver en GitHub
Lenguaje dominante
Python
Estrellas
50
Forks
40
Merge medio
7 d 1 h
PR fusionados (30 d)
1

Descripción

### What happened?

Tests for REST transport use snake_case for fields by converting request payloads via [`convert_a2a_message_to_protobuf_json`](https://github.com/a2aproject/a2a-tck/blob/b03fefcae9767dad0e978e11573192f74252dfe3/tck/message_utils.py#L52). Response conversion uses [`convert_protobuf_response_to_a2a_json`](https://github.com/a2aproject/a2a-tck/blob/b03fefcae9767dad0e978e11573192f74252dfe3/tck/message_utils.py#L133C5-L133C42) as all the validations operate on camelCase. However this isn't done in [`send_streaming_message`](https://github.com/a2aproject/a2a-tck/blob/b03fefcae9767dad0e978e11573192f74252dfe3/tck/transport/rest_client.py#L311-L312) while assertions still use camelCase (i.e. [here](https://github.com/a2aproject/a2a-tck/blob/b03fefcae9767dad0e978e11573192f74252dfe3/tests/optional/capabilities/test_streaming_methods.py#L131)).

The proposal is to use camelCase due to:

1. v0.3.0 doesn't mention casing explicitly for REST, however in [this example](https://a2a-protocol.org/v0.3.0/specification/#72-messagestream) response payload in the REST tab is shown in camelCase.

1. v1.0 draft explicitly specifies using camelCase everywhere: [5.5. JSON Field Naming Convention](https://a2a-protocol.org/latest/specification/#55-json-field-naming-convention).

1. SDKs already use camelCase (at lest it's the case for Java, Python and Go).

### Relevant log output

```shell

```

### Code of Conduct

- [x] I agree to follow this project's Code of Conduct

Guía de contribución

Abrir la guía de contribución

Evaluación

Este issue todavía no se ha evaluado.

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.