-
Notifications
You must be signed in to change notification settings - Fork 53
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Langfuse integration to app #954
base: main
Are you sure you want to change the base?
Conversation
Co-Authored-By: hirotaka.miyagi@route06.co.jp <hirotaka.miyagi@route06.co.jp>
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
4 Skipped Deployments
|
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
Updates to Preview Branch (devin/1742543457-langfuse-integration) ↗︎
Tasks are run on every commit but only new migration files are pushed.
View logs for this Workflow Run ↗︎. |
CI Feedback 🧐(Feedback updated until commit b40e732)A test triggered by this PR failed. Here is an AI-generated analysis of the failure:
|
This comment was marked as off-topic.
This comment was marked as off-topic.
Co-Authored-By: hirotaka.miyagi@route06.co.jp <hirotaka.miyagi@route06.co.jp>
Co-Authored-By: hirotaka.miyagi@route06.co.jp <hirotaka.miyagi@route06.co.jp>
Co-Authored-By: hirotaka.miyagi@route06.co.jp <hirotaka.miyagi@route06.co.jp>
Co-Authored-By: hirotaka.miyagi@route06.co.jp <hirotaka.miyagi@route06.co.jp>
Co-Authored-By: hirotaka.miyagi@route06.co.jp <hirotaka.miyagi@route06.co.jp>
LANGFUSE_BASE_URL="https://cloud.langfuse.com" | ||
LANGFUSE_PUBLIC_KEY="" | ||
LANGFUSE_SECRET_KEY="" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
I will merge next Monday! |
Issue
Why is this change needed?
What would you like reviewers to focus on?
Testing Verification
What was done
pr_agent:summary
Detailed Changes
pr_agent:walkthrough
Additional Notes
Implements Langfuse integration in the app to track LLM calls, similar to how it's implemented in migration-web.
Link to Devin run: https://app.devin.ai/sessions/5ff355a737c8445bb7b88133c09a62c5
Requested by: hirotaka.miyagi@route06.co.jp