Skip to content

Conversation

@devin-ai-integration
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>
@changeset-bot
Copy link

changeset-bot bot commented Mar 21, 2025

⚠️ 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
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 ✅ 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 Preview Comments 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
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

@supabase
Copy link

supabase bot commented Mar 21, 2025

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-free-for-open-source-projects
Copy link
Contributor

qodo-free-for-open-source-projects bot commented Mar 21, 2025

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.

@liam-migration-preview

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
Contributor

MH4GF commented Mar 21, 2025

I will merge next Monday!

@MH4GF
Copy link
Contributor

MH4GF commented Mar 25, 2025

to devin: Please resolve conflict.

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>
@MH4GF 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 MH4GF deleted the devin/1742543457-langfuse-integration branch March 25, 2025 03:50
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.

3 participants