Add Langfuse integration to app #954

devin-ai-integration · 2025-03-21T07:53:19Z

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: [email protected]

Co-Authored-By: [email protected] <[email protected]>

changeset-bot · 2025-03-21T07:53:23Z

⚠️ No Changeset found

Latest commit: f3a7ea9

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

vercel · 2025-03-21T07:53:23Z

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

Name	Status	Preview	Comments	Updated (UTC)
liam-app	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Mar 25, 2025 0:57am
liam-docs	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Mar 25, 2025 0:57am
liam-erd-sample	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Mar 25, 2025 0:57am

3 Skipped Deployments

Name	Status	Updated (UTC)
test-liam-app	⬜️ Ignored (Inspect)	Mar 25, 2025 0:57am
test-liam-docs	⬜️ Ignored (Inspect)	Mar 25, 2025 0:57am
test-liam-erd-sample	⬜️ Ignored (Inspect)	Mar 25, 2025 0:57am

devin-ai-integration · 2025-03-21T07:53:25Z

🤖 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

supabase · 2025-03-21T07:53:47Z

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

Deployments	Status	Updated
Database	✅	Tue, 25 Mar 2025 00:49:21 UTC
Services	✅	Tue, 25 Mar 2025 00:49:21 UTC
APIs	✅	Tue, 25 Mar 2025 00:49:21 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	✅	Tue, 25 Mar 2025 00:49:34 UTC
Migrations	✅	Tue, 25 Mar 2025 00:49:35 UTC
Seeding	✅	Tue, 25 Mar 2025 00:49:35 UTC
Edge Functions	✅	Tue, 25 Mar 2025 00:49:35 UTC

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

qodo-merge-pro-for-open-source · 2025-03-21T07:53:59Z

CI Feedback 🧐

(Feedback updated until commit `85fa919`)

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 install command could not be executed with the `--frozen-lockfile` flag. The lockfile (pnpm-lock.yaml) is out of date compared to the package.json file in the frontend/apps/app directory. Specifically, the package `langfuse-langchain` version `3.36.0` is present in the package.json but not in the lockfile. This mismatch prevents the installation from proceeding in frozen lockfile mode, which is typically enforced in CI environments.
Relevant error logs: 1: ##[group]Operating System 2: Ubuntu ... 167: Cache Size: ~388 MB (406427495 B) 168: [command]/usr/bin/tar -xf /home/runner/work/_temp/446f3d6e-bbb4-48f3-b752-c3fe6d26dfd8/cache.tzst -P -C /home/runner/work/liam/liam --use-compress-program unzstd 169: Cache restored successfully 170: Cache restored from key: node-cache-Linux-x64-pnpm-981845ee42f88851b38a0e5b5459c41fdd02ed05a02542dfeb8e5613a229895b 171: ##[group]Run pnpm install --frozen-lockfile --prefer-offline 172: �[36;1mpnpm install --frozen-lockfile --prefer-offline�[0m 173: shell: /usr/bin/bash --noprofile --norc -e -o pipefail {0} 174: env: 175: PNPM_HOME: /home/runner/setup-pnpm/node_modules/.bin 176: ##[endgroup] 177: Scope: all 12 workspace projects 178: ERR_PNPM_OUTDATED_LOCKFILE Cannot install with "frozen-lockfile" because pnpm-lock.yaml is not up to date with frontend/apps/app/package.json 179: 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" 180: Failure reason: 181: 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.2.3","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.2.3","react":"18.3.1","react-dom":"18","ts-pattern":"5.4.0","valibot":"^1.0.0-beta.5"}) 182: ##[error]Process completed with exit code 1. 183: Post job cleanup.

Co-Authored-By: [email protected] <[email protected]>

frontend/apps/app/.env.template

Co-Authored-By: [email protected] <[email protected]>

hoshinotsuyoshi · 2025-03-21T11:26:03Z

.env.template

+LANGFUSE_BASE_URL="https://cloud.langfuse.com"
+LANGFUSE_PUBLIC_KEY=""
+LANGFUSE_SECRET_KEY=""


I confirmed it works

pnpm --filter @liam-hq/app trigger:dev

hoshinotsuyoshi

LGTM!

MH4GF · 2025-03-21T11:36:05Z

I will merge next Monday!

MH4GF · 2025-03-25T00:44:18Z

to devin: Please resolve conflict.

Co-Authored-By: [email protected] <[email protected]>

Add Langfuse integration to app

b40e732

Co-Authored-By: [email protected] <[email protected]>

This comment was marked as off-topic.

Sign in to view

devin-ai-integration bot and others added 2 commits March 21, 2025 07:55

Update pnpm-lock.yaml for langfuse-langchain

6a1579b

Co-Authored-By: [email protected] <[email protected]>

Update docs/packages-license.md

1a0cce9

devin-ai-integration bot marked this pull request as ready for review March 21, 2025 07:59

devin-ai-integration bot requested a review from a team as a code owner March 21, 2025 07:59

devin-ai-integration bot requested review from hoshinotsuyoshi, FunamaYukina, junkisai, MH4GF, NoritakaIkeda and sasamuku and removed request for a team March 21, 2025 07:59

vercel bot deployed to Preview – liam-app March 21, 2025 08:18 View deployment

devin-ai-integration bot marked this pull request as draft March 21, 2025 08:18

vercel bot deployed to Preview – liam-erd-sample March 21, 2025 08:19 View deployment

Add environment settings to Langfuse integration

021db83

Co-Authored-By: [email protected] <[email protected]>

devin-ai-integration bot marked this pull request as ready for review March 21, 2025 08:22

MH4GF reviewed Mar 21, 2025

View reviewed changes

frontend/apps/app/.env.template Outdated Show resolved Hide resolved

Move Langfuse environment variables to root .env.template

35c4181

Co-Authored-By: [email protected] <[email protected]>

vercel bot deployed to Preview – liam-app March 21, 2025 08:47 View deployment

vercel bot deployed to Preview – liam-erd-sample March 21, 2025 08:48 View deployment

devin-ai-integration bot and others added 2 commits March 21, 2025 09:11

Merge main into feature branch and resolve conflicts

d0b2d84

Co-Authored-By: [email protected] <[email protected]>

Fix formatting in processGenerateReview.ts

a9004c8

Co-Authored-By: [email protected] <[email protected]>

vercel bot deployed to Preview – liam-erd-sample March 21, 2025 09:22 View deployment

vercel bot deployed to Preview – liam-app March 21, 2025 09:25 View deployment

hoshinotsuyoshi reviewed Mar 21, 2025

View reviewed changes

hoshinotsuyoshi approved these changes Mar 21, 2025

View reviewed changes

Resolve merge conflict in pnpm-lock.yaml

85fa919

Co-Authored-By: [email protected] <[email protected]>

vercel bot had a problem deploying to Preview – liam-erd-sample March 25, 2025 00:45 Failure

vercel bot had a problem deploying to Preview – liam-docs March 25, 2025 00:46 Failure

vercel bot had a problem deploying to Preview – liam-app March 25, 2025 00:46 Failure

Update pnpm-lock.yaml with langfuse-langchain dependency

f3a7ea9

Co-Authored-By: [email protected] <[email protected]>

vercel bot deployed to Preview – liam-erd-sample March 25, 2025 00:49 View deployment

vercel bot deployed to Preview – liam-app March 25, 2025 00:53 View deployment

vercel bot deployed to Preview – liam-docs March 25, 2025 00:57 View deployment

MH4GF added this pull request to the merge queue Mar 25, 2025

Merged via the queue into main with commit b3e9c11 Mar 25, 2025
21 checks passed

MH4GF deleted the devin/1742543457-langfuse-integration branch March 25, 2025 03:50

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add Langfuse integration to app #954

Add Langfuse integration to app #954

devin-ai-integration bot commented Mar 21, 2025

changeset-bot bot commented Mar 21, 2025 •

edited

Loading

vercel bot commented Mar 21, 2025 •

edited

Loading

devin-ai-integration bot commented Mar 21, 2025

supabase bot commented Mar 21, 2025 •

edited

Loading

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

edited

Loading

This comment was marked as off-topic.

hoshinotsuyoshi Mar 21, 2025

hoshinotsuyoshi left a comment

MH4GF commented Mar 21, 2025

MH4GF commented Mar 25, 2025

Add Langfuse integration to app #954

Add Langfuse integration to app #954

Conversation

devin-ai-integration bot commented Mar 21, 2025

Issue

Why is this change needed?

What would you like reviewers to focus on?

Testing Verification

What was done

Detailed Changes

Additional Notes

changeset-bot bot commented Mar 21, 2025 • edited Loading

⚠️ No Changeset found

vercel bot commented Mar 21, 2025 • edited Loading

devin-ai-integration bot commented Mar 21, 2025

🤖 Devin AI Engineer

supabase bot commented Mar 21, 2025 • edited Loading

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

CI Feedback 🧐

(Feedback updated until commit 85fa919)

This comment was marked as off-topic.

hoshinotsuyoshi Mar 21, 2025

Choose a reason for hiding this comment

hoshinotsuyoshi left a comment

Choose a reason for hiding this comment

MH4GF commented Mar 21, 2025

MH4GF commented Mar 25, 2025

changeset-bot bot commented Mar 21, 2025 •

edited

Loading

vercel bot commented Mar 21, 2025 •

edited

Loading

supabase bot commented Mar 21, 2025 •

edited

Loading

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

edited

Loading

(Feedback updated until commit `85fa919`)