Lizandro GuarnizoandClaude Sonnet 5
d5fbdf6d97
Por qué no aparecía la factura: la única herramienta de venta del bot era adjuntar_factura, y exige un documento pendiente en el chat. Al dictarle los datos por texto el modelo no tenía con qué guardar — y como el prompt tampoco se lo prohibía, informó "guardada exitosamente" sobre algo que nunca se creó. /api/v2/facturas ya permitía crearlas, pero el bot no llama a la API REST: usa su propio registro de herramientas, así que solo puede hacer lo que ese registro expone. El hueco estaba ahí, no en la API. crear_factura toma cliente_id y monto (obligatorios), número, concepto y fechas. Devuelve el monto realmente guardado y el id, que es lo que el prompt le exige informar en vez de repetir lo que dijo el usuario. Las descripciones de las dos herramientas se remiten entre sí para que el modelo no elija la equivocada: con documento adjunto va adjuntar_factura, con datos dictados va crear_factura. El test verifica esa distinción, porque elegir mal reproduce exactamente el fallo original. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
46 lines
1.5 KiB
Go
46 lines
1.5 KiB
Go
package services
|
|
|
|
import (
|
|
"strings"
|
|
"testing"
|
|
)
|
|
|
|
// Registrar una factura dictando los datos no era posible: la única
|
|
// herramienta de venta era adjuntar_factura, que exige un archivo pendiente.
|
|
// Sin una alternativa, el modelo no tenía con qué guardar y el usuario recibía
|
|
// un "guardado exitosamente" sobre algo que nunca se creó.
|
|
func TestExisteHerramientaParaFacturaSinAdjunto(t *testing.T) {
|
|
tools := agentTools()
|
|
|
|
nombres := map[string]agentTool{}
|
|
for _, tl := range tools {
|
|
nombres[tl.Function.Name] = tl
|
|
}
|
|
|
|
crear, ok := nombres["crear_factura"]
|
|
if !ok {
|
|
t.Fatal("falta crear_factura: no habría forma de registrar una factura de venta sin archivo adjunto")
|
|
}
|
|
if _, ok := nombres["adjuntar_factura"]; !ok {
|
|
t.Error("adjuntar_factura debe seguir existiendo para el caso con documento")
|
|
}
|
|
|
|
// La descripción tiene que distinguirlas, o el modelo elige la equivocada
|
|
// y falla por falta de adjunto — que es justo el problema original.
|
|
d := strings.ToLower(crear.Function.Description)
|
|
if !strings.Contains(d, "sin archivo adjunto") {
|
|
t.Error("la descripción debe dejar claro que no necesita adjunto")
|
|
}
|
|
if !strings.Contains(d, "adjuntar_factura") {
|
|
t.Error("la descripción debe remitir a adjuntar_factura cuando sí hay documento")
|
|
}
|
|
|
|
// cliente_id y monto son obligatorios: sin ellos la factura no sirve.
|
|
req := strings.Join(crear.Function.Parameters.Required, ",")
|
|
for _, campo := range []string{"cliente_id", "monto"} {
|
|
if !strings.Contains(req, campo) {
|
|
t.Errorf("%q debería ser obligatorio en crear_factura", campo)
|
|
}
|
|
}
|
|
}
|