Skip to content

fix: standardize Langfuse endpoint on LANGFUSE_BASE_URL - #351

Merged
norberia merged 2 commits into
mainfrom
codex/langfuse-base-url
Sep 10, 2026
Merged

norberia merged 2 commits into
mainfrom
codex/langfuse-base-url

Conversation

@norberia

@norberia norberia commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Use the official LANGFUSE_BASE_URL environment variable for Chat Assistant tracing and forwarding into newly created GitHub Deploy Devboxes. Update Helm values, environment examples, Turbo configuration, tests, and migration notes together.

This is a breaking configuration rename: LANGFUSE_HOST is no longer read or forwarded. Deployments must rename the variable and recreate existing Devboxes to apply the new environment. Historical changelog entries retain their original names.

Validation:

  • 47 focused tests passed, including rejection of the old variable and Devbox forwarding coverage.
  • bun typecheck passed.
  • bun lint passed. The removed-variable regression test has a scoped Turbo lint exemption because the old variable intentionally is not declared in turbo.json.
  • bun check passed (1,660 files).
  • git diff --check passed.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 10, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-09-10T09:53:20.527888Z efbec24 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@norberia
norberia merged commit eabe1a6 into main Sep 10, 2026
8 checks passed
@norberia
norberia deleted the codex/langfuse-base-url branch September 11, 2026 08:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant