[Feat]: Simplification of the client API?
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 65/100
Línea de trabajo
Examine la API del cliente en el módulo spec, centrándose en los métodos que utilizan objetos XXXParams como TaskIdParams. Examine el método sendMessage existente como referencia del patrón. Identifique todos los métodos que podrían simplificarse y, a continuación, cree versiones sobrecargadas que acepten parámetros primitivos y deleguen en las implementaciones existentes. Asegúrese de mantener la compatibilidad con versiones anteriores y pruebe los cambios.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Is your feature request related to a problem? Please describe.
The client API is a thin wrapper around the domain objects in the spec module.
I wonder if we could not make the API a bit simpler while retaining all the capabilities it exposes.
Describe the solution you'd like
For example, a lot of calls wrap their parameters in XXXParams object:
public Task cancelTask(TaskIdParams request)
but TaskIdParams is composed of a taskId and a metadata map.
Could the call simply be:
public Task cancelTask(String taskId, Map<String, Object> metadata) {
return clientTransport.cancelTask(new TaskIdParams(taskId, metadata), request, context);
}
and a default method:
public default Task cancelTask(String taskId) {
return cancelTask(taskId, emptyMap();
}
This way, the TaskIdParams does not leak to the Client API (making it bigger than it needs to be.
The API is already to that for its sendMessage methods that hides the internal MessageSendParams.
This is a pattern that should be applied to other methods as well
Describe alternatives you've considered
No response
Additional context
No response
Code of Conduct
- I agree to follow this project's Code of Conduct
- Lenguaje dominante
- Java
- Estrellas
- 504
- Forks
- 184
- Merge medio
- 1 d 13 h
- PR fusionados (30 d)
- 31
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una 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 a2aproject/a2a-java
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
a2aproject/a2a-java#1012 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
a2aproject/a2a-java#464 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Create a sample that demonstrates how to use a shared contextId across multiple tasksQuizá libre de nuevo @tanish111 la tomó hace 355 días y no hay ningún pull request abierto. Abiertosample
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
a2aproject/a2a-java#375 · 5 comentarios · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
Dificultad 3/5 1-2 días Aptitud para principiantes 55/100
a2aproject/a2a-java#1207 ·
Los mantenedores suelen responder en 1 día
-
Enable checkstyle failOnViolation=truePosiblemente ocupada @ZYZ666-RGB la tomó hace 3 días. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 58/100
a2aproject/a2a-java#1206 · 2 comentarios · 1 asignado ·
Los mantenedores suelen responder en 1 día
Todos los issues de a2aproject/a2a-java
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
openhab/openhab-addons#21921 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
objectionary/eo-graphs#85 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
objectionary/lints#1544 ·
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
objectionary/hone-maven-plugin#1309 ·
Los mantenedores suelen responder en 1 día
-
chore s3
Dificultad 2/5 1-3 horas Aptitud para principiantes 83/100
Los mantenedores suelen responder en 1 día