[BUG] Go Ollama adapter does not send tool_name on tool result messages
Los mantenedores suelen responder en 1 día
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 68/100
Línea de trabajo
Lee go/adk/pkg/models/ollama_adk.go, especialmente convertGenaiContentsToOllamaMessages, y compara cómo maneja las respuestas de funciones con _convert_content_to_ollama_messages en el archivo de referencia de Python. Añade una prueba para las respuestas de get_weather y get_time que compruebe que cada mensaje de herramienta lleva el nombre de su función; el trabajo estará terminado cuando ambos valores de ToolName coincidan con esos nombres.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Bug Description
The Go Ollama adapter does not send tool_name on tool result messages. In convertGenaiContentsToOllamaMessages, each function response becomes a message with only role: tool and content. The Python Ollama adapter sets tool_name from the function response name. Ollama uses this field when it builds the prompt. For example, the Gemma 4 renderer shows a result with no tool_name as coming from an "unknown" tool.
Steps To Reproduce
- Build a history with a model turn that calls
get_weatherandget_time, then a user turn with both function responses. - Pass it to
convertGenaiContentsToOllamaMessages. - Both
toolmessages have an emptyToolName.
Expected Behavior
Each tool message carries the name of the tool that produced it, the same as the Python runtime.
Additional Context
File: go/adk/pkg/models/ollama_adk.go. Python reference: _convert_content_to_ollama_messages in python/packages/kagent-adk/src/kagent/adk/models/_ollama.py. I have a small fix with a test ready.
- Lenguaje dominante
- Go
- Estrellas
- 4.1k
- Forks
- 859
- Merge medio
- 1 d 8 h
- PR fusionados (30 d)
- 169
Preparar el entorno
Inicia el contenedor de desarrollo del proyecto en tu navegador, con tu propia cuenta de GitHub.
- 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 kagent-dev/kagent
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
kagent-dev/kagent#3428 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
[BUG]Posiblemente ocupada @helloxjade la tomó hace 1 día. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
kagent-dev/kagent#3385 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
kagent-dev/kagent#3247 ·
Los mantenedores suelen responder en 1 día
-
[BUG] Ollama ModelConfig options: integer coercion crashes on decimal strings, and boolean options only accept exact "true"Posiblemente ocupada @piyush182004 la tomó hace 10 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
kagent-dev/kagent#3245 · 2 comentarios · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
ui: useAgentInstances(true) turns an all_creators refusal into a page errorQuizá libre de nuevo @suryakant-source la tomó hace 10 días y no hay ningún pull request abierto. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
kagent-dev/kagent#3212 · 1 comentario ·
Los mantenedores suelen responder en 1 día
Todos los issues de kagent-dev/kagent
Issues similares
-
🐛 tunnel diag: Docker log collection deadlocks when the container's stderr log exceeds 64 KiBAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
cloudflare/cloudflared#1783 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
orchestration.migration.data.tasks.retry values are ignored by the 8.8 data migration jobPosiblemente ocupada @irisfield la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
camunda/camunda-platform-helm#7773 ·
Los mantenedores suelen responder en 1 día
-
Add missing unit tests for IsAdmissionPolicyName and IsCertManagerResourceNamePosiblemente ocupada @a1h8 la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
kubernetes-sigs/security-profiles-operator#3644 ·
Los mantenedores suelen responder en 1 día