Skip to content

feat(observability): прокидывать имя модели в Langfuse generation - #52

Merged
Tarasusrus merged 1 commit into
mainfrom
feature/langfuse-model-name
May 7, 2026
Merged

feat(observability): прокидывать имя модели в Langfuse generation#52
Tarasusrus merged 1 commit into
mainfrom
feature/langfuse-model-name

Conversation

@Tarasusrus

Copy link
Copy Markdown
Owner

Summary

  • agent.Service получает llmModel через WithLLMModel и передаёт в StartGeneration (3 точки в service.go) вместо пустой строки.
  • Без этого Langfuse не матчит pricing/cost — графики Cost не растут.

Changes

  • internal/agent/service.go: поле llmModel + WithLLMModel + использование в трёх StartGeneration вызовах (llm.ask, llm.iter%d, llm.final)
  • cmd/app/main.go: .WithLLMModel(cfg.LLM.ChatModel) в цепочке конструктора
  • .env.example: комментарий что LLM_CHAT_MODEL также используется как model в Langfuse generation; должно совпадать с Langfuse model entry

Использование

В .env выставить имя совпадающее с Langfuse model entry. Например, если в Langfuse заведена запись deepseek-v4-flash:
`LLM_CHAT_MODEL=deepseek-v4-flash`

DeepSeek API принимает алиасы (deepseek-chat / deepseek-v4-flash) — оба роутятся в одну модель.

Closes simpleAI-96n.

Test plan

  • go build / go vet clean
  • go test ./... PASS
  • Ручная проверка: после merge установить LLM_CHAT_MODEL=deepseek-v4-flash, отправить запрос, увидеть в Langfuse generation с непустым model и привязанным cost

🤖 Generated with Claude Code

agent.Service получает llmModel через WithLLMModel(model) и передаёт
в StartGeneration вместо пустой строки. main.go прокидывает
cfg.LLM.ChatModel. Без этого Langfuse не матчит pricing/cost.

Имя модели должно совпадать с записью в Langfuse settings (Models →
Custom). Например, для DeepSeek API принимает deepseek-chat, но
дашборд может показывать display name deepseek-v4-flash — в этом
случае выставить LLM_CHAT_MODEL=deepseek-v4-flash и завести такую же
запись в Langfuse.

Closes simpleAI-96n.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@Tarasusrus
Tarasusrus merged commit 6c22be7 into main May 7, 2026
2 checks passed
@Tarasusrus
Tarasusrus deleted the feature/langfuse-model-name branch May 7, 2026 01:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant