Skip to content
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

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

devin-ai-integration[bot]
Copy link
Contributor

Issue

  • resolve:

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

Co-Authored-By: hirotaka.miyagi@route06.co.jp <hirotaka.miyagi@route06.co.jp>
Copy link

changeset-bot bot commented Mar 21, 2025

⚠️ No Changeset found

Latest commit: a9004c8

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copy link

vercel bot commented Mar 21, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
liam-app 🛑 Canceled (Inspect) Mar 21, 2025 9:25am
liam-erd-sample ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 21, 2025 9:25am
4 Skipped Deployments
Name Status Preview Comments Updated (UTC)
liam-docs ⬜️ Ignored (Inspect) Visit Preview Mar 21, 2025 9:25am
test-liam-app ⬜️ Ignored (Inspect) Mar 21, 2025 9:25am
test-liam-docs ⬜️ Ignored (Inspect) Mar 21, 2025 9:25am
test-liam-erd-sample ⬜️ Ignored (Inspect) Mar 21, 2025 9:25am

Copy link
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add "(aside)" to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

Copy link

supabase bot commented Mar 21, 2025

Updates to Preview Branch (devin/1742543457-langfuse-integration) ↗︎

Deployments Status Updated
Database Fri, 21 Mar 2025 09:14:37 UTC
Services Fri, 21 Mar 2025 09:14:37 UTC
APIs Fri, 21 Mar 2025 09:14:37 UTC

Tasks are run on every commit but only new migration files are pushed.
Close and reopen this PR if you want to apply changes from existing seed or migration files.

Tasks Status Updated
Configurations Fri, 21 Mar 2025 09:14:37 UTC
Migrations Fri, 21 Mar 2025 09:14:37 UTC
Seeding Fri, 21 Mar 2025 09:14:37 UTC
Edge Functions Fri, 21 Mar 2025 09:14:37 UTC

View logs for this Workflow Run ↗︎.
Learn more about Supabase for Git ↗︎.

Copy link
Contributor

qodo-merge-pro-for-open-source bot commented Mar 21, 2025

CI Feedback 🧐

(Feedback updated until commit b40e732)

A test triggered by this PR failed. Here is an AI-generated analysis of the failure:

Action: frontend-ci

Failed stage: Run /./.github/actions/pnpm-setup [❌]

Failure summary:

The action failed because the pnpm-lock.yaml file is out of sync with the
frontend/apps/app/package.json file. The CI is running with the --frozen-lockfile flag (which is
default in CI environments), but the lockfile doesn't match the dependencies specified in
package.json.

Specifically, the error shows that the package langfuse-langchain version 3.36.0 is present in
package.json but not in the lockfile. This indicates that someone added this dependency but didn't
update the lockfile.

Relevant error logs:
1:  ##[group]Operating System
2:  Ubuntu
...

164:  Cache Size: ~400 MB (419869780 B)
165:  [command]/usr/bin/tar -xf /home/runner/work/_temp/29164e07-c36f-4177-86ed-79778941490a/cache.tzst -P -C /home/runner/work/liam/liam --use-compress-program unzstd
166:  Cache restored successfully
167:  Cache restored from key: node-cache-Linux-x64-pnpm-54390409b596918c23cf4d15277b25e541acf18027242b5ca51fb4345c16cf14
168:  ##[group]Run pnpm install --frozen-lockfile --prefer-offline
169:  �[36;1mpnpm install --frozen-lockfile --prefer-offline�[0m
170:  shell: /usr/bin/bash --noprofile --norc -e -o pipefail {0}
171:  env:
172:  PNPM_HOME: /home/runner/setup-pnpm/node_modules/.bin
173:  ##[endgroup]
174:  Scope: all 12 workspace projects
175:  ERR_PNPM_OUTDATED_LOCKFILE  Cannot install with "frozen-lockfile" because pnpm-lock.yaml is not up to date with frontend/apps/app/package.json
176:  Note that in CI environments this setting is true by default. If you still need to run install in such cases, use "pnpm install --no-frozen-lockfile"
177:  Failure reason:
178:  specifiers in the lockfile ({"@langchain/core":"0.3.42","@langchain/openai":"0.4.4","@liam-hq/db":"workspace:*","@liam-hq/db-structure":"workspace:*","@liam-hq/erd-core":"workspace:*","@liam-hq/ui":"workspace:*","@next/third-parties":"15.1.5","@octokit/auth-app":"6.1.3","@octokit/rest":"20.0.2","@prisma/client":"6.4.1","@sentry/nextjs":"8","@trigger.dev/build":"3.3.17","@trigger.dev/sdk":"3.3.17","cheerio":"1.0.0","date-fns":"4.1.0","dotenv":"16.4.7","flags":"3.1.1","minimatch":"10.0.1","next":"15.1.2","react":"18.3.1","react-dom":"18","ts-pattern":"5.4.0","valibot":"^1.0.0-beta.5","@biomejs/biome":"1.9.3","@octokit/openapi-types":"24.0.0","@prisma/nextjs-monorepo-workaround-plugin":"6.5.0","@types/gtag.js":"0.0.20","@types/node":"22.9.0","@types/react":"18","@types/react-dom":"18","esbuild":"0.25.1","trigger.dev":"3.3.17","typed-css-modules":"0.9.1","typescript":"5","vitest":"3.0.8"}) don't match specs in package.json ({"@biomejs/biome":"1.9.3","@octokit/openapi-types":"24.0.0","@prisma/nextjs-monorepo-workaround-plugin":"6.5.0","@types/gtag.js":"0.0.20","@types/node":"22.9.0","@types/react":"18","@types/react-dom":"18","esbuild":"0.25.1","trigger.dev":"3.3.17","typed-css-modules":"0.9.1","typescript":"5","vitest":"3.0.8","@langchain/core":"0.3.42","@langchain/openai":"0.4.4","@liam-hq/db":"workspace:*","@liam-hq/db-structure":"workspace:*","@liam-hq/erd-core":"workspace:*","@liam-hq/ui":"workspace:*","@next/third-parties":"15.1.5","@octokit/auth-app":"6.1.3","@octokit/rest":"20.0.2","@prisma/client":"6.4.1","@sentry/nextjs":"8","@trigger.dev/build":"3.3.17","@trigger.dev/sdk":"3.3.17","cheerio":"1.0.0","date-fns":"4.1.0","dotenv":"16.4.7","flags":"3.1.1","langfuse-langchain":"3.36.0","minimatch":"10.0.1","next":"15.1.2","react":"18.3.1","react-dom":"18","ts-pattern":"5.4.0","valibot":"^1.0.0-beta.5"})
179:  ##[error]Process completed with exit code 1.
180:  Post job cleanup.

This comment was marked as off-topic.

devin-ai-integration bot and others added 2 commits March 21, 2025 07:55
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>
@devin-ai-integration devin-ai-integration bot marked this pull request as ready for review March 21, 2025 08:22
Co-Authored-By: hirotaka.miyagi@route06.co.jp <hirotaka.miyagi@route06.co.jp>
devin-ai-integration bot and others added 2 commits March 21, 2025 09:11
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>
Comment on lines +6 to +8
LANGFUSE_BASE_URL="https://cloud.langfuse.com"
LANGFUSE_PUBLIC_KEY=""
LANGFUSE_SECRET_KEY=""
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I confirmed it works

pnpm --filter @liam-hq/app trigger:dev
スクリーンショット 2025-03-21 20 24 58

Copy link
Member

@hoshinotsuyoshi hoshinotsuyoshi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@MH4GF
Copy link
Member

MH4GF commented Mar 21, 2025

I will merge next Monday!

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.

2 participants