feat: add langfuse openai SDK tracing to openai_provider only by leesf · Pull Request #3433 · agentscope-ai/QwenPaw

leesf · 2026-04-15T09:21:02Z

This pull request improves the integration with Langfuse tracing in the OpenAIProvider by conditionally importing the appropriate AsyncOpenAI client based on environment variables and package availability. This ensures that Langfuse tracing is only enabled when both the secret key is set and the package is installed, and provides clear logging if the setup is incomplete.

Conditional Langfuse integration:

The import of AsyncOpenAI now checks if LANGFUSE_SECRET_KEY is set and if the langfuse package is installed; if both are true, it imports AsyncOpenAI from langfuse.openai, otherwise it falls back to the standard openai client.
If LANGFUSE_SECRET_KEY is set but langfuse is not installed, a warning is logged to inform the user how to enable tracing.

Dependency management:

Added imports for importlib.util and os to support the conditional logic for importing and environment variable checking.

Agent-Logs-Url: https://github.com/leesf/QwenPaw/sessions/866badcb-6274-4c83-8e0e-38cbec1ea35a Co-authored-by: leesf <10128888+leesf@users.noreply.github.com>

Agent-Logs-Url: https://github.com/leesf/QwenPaw/sessions/c126b162-3248-4e4e-a1f0-a78488042930 Co-authored-by: leesf <10128888+leesf@users.noreply.github.com>

Co-authored-by: leesf <10128888+leesf@users.noreply.github.com>

… not installed Agent-Logs-Url: https://github.com/leesf/QwenPaw/sessions/9b75df92-92e2-4a75-b51c-44c9199d116e Co-authored-by: leesf <10128888+leesf@users.noreply.github.com>

Agent-Logs-Url: https://github.com/leesf/QwenPaw/sessions/69d05c99-82cf-45f2-ad1b-dde39b93e7a0 Co-authored-by: leesf <10128888+leesf@users.noreply.github.com>

feat(observability): add optional Langfuse tracing for provider-backed LLM calls

github-actions · 2026-04-15T09:21:15Z

Welcome to QwenPaw! 🐾

Hi @leesf, thank you for your first Pull Request! 🎉

📋 About PR Template

To help maintainers review your PR faster, please make sure to include:

✅ Description - What this PR does and why
✅ Type of Change - Bug fix / Feature / Breaking change / Documentation / Refactoring
✅ Component(s) Affected - Core / Console / Channels / Skills / CLI / Documentation / Tests / CI/CD / Scripts
✅ Checklist:
- Run and pass pre-commit run --all-files
- Run and pass relevant tests (pytest or as applicable)
- Update documentation if needed
✅ Testing - How to test these changes

✅ Local Verification Evidence:

pre-commit run --all-files
# paste summary result

pytest
# paste summary result

Complete PR information helps speed up the review process. You can edit the PR description to add these details.

🙌 Join Developer Community

Thanks so much for your contribution! We'd love to invite you to join the official QwenPaw developer group! You can find the Discord and DingTalk group links under the "Developer Community" section on our docs page:
https://qwenpaw.agentscope.io/docs/community

We truly appreciate your enthusiasm—and look forward to your future contributions! 😊

We'll review your PR soon.

Copilot

Pull request overview

Adds optional Langfuse tracing support to the OpenAIProvider by selecting AsyncOpenAI from langfuse.openai only when LANGFUSE_SECRET_KEY is set and Langfuse is available, otherwise falling back to the standard OpenAI SDK client.

Changes:

Added conditional import logic to choose between langfuse.openai.AsyncOpenAI and openai.AsyncOpenAI.
Added an import-time warning when LANGFUSE_SECRET_KEY is set but Langfuse isn’t installed.
Added supporting imports (importlib.util, os) for environment/package checks.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

+if os.environ.get("LANGFUSE_SECRET_KEY") and importlib.util.find_spec("langfuse"):
+    from langfuse.openai import AsyncOpenAI  # type: ignore[import]
+else:
+    if os.environ.get("LANGFUSE_SECRET_KEY"):
+        logging.warning(
+            "LANGFUSE_SECRET_KEY is set but langfuse is not installed; "
+            "install with `pip install langfuse` to enable tracing"
+        )


Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Copilot

Pull request overview

Adds an optional Langfuse-instrumented OpenAI client to OpenAIProvider so tracing is enabled only when LANGFUSE_SECRET_KEY is set and the Langfuse package is available, otherwise falling back to the standard OpenAI SDK with a warning.

Changes:

Conditionally imports AsyncOpenAI from langfuse.openai when Langfuse tracing is configured.
Falls back to openai.AsyncOpenAI and logs a warning when tracing is partially configured.
Adds os and importlib.util imports to support env/package detection.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

xieyxclack · 2026-04-15T10:06:43Z

@leesf Please take a look at the Copilot comments and make modifications if you find them helpful.

xieyxclack · 2026-04-15T11:15:03Z

And please format the code by pre-commit run --all-files

Agent-Logs-Url: https://github.com/leesf/QwenPaw/sessions/daf93103-1af3-464d-8080-1ef1afcdedb8 Co-authored-by: leesf <10128888+leesf@users.noreply.github.com>

Fix `pre-commit run --all-files` failures in OpenAI provider import/type wiring

… import Agent-Logs-Url: https://github.com/leesf/QwenPaw/sessions/e0dcd6e5-ed11-4cbc-9d2e-a7d5a4ff058c Co-authored-by: leesf <10128888+leesf@users.noreply.github.com>

Agent-Logs-Url: https://github.com/leesf/QwenPaw/sessions/959b4d27-8043-49a5-a557-941a69e4021f Co-authored-by: leesf <10128888+leesf@users.noreply.github.com>

…port Agent-Logs-Url: https://github.com/leesf/QwenPaw/sessions/959b4d27-8043-49a5-a557-941a69e4021f Co-authored-by: leesf <10128888+leesf@users.noreply.github.com>

refactor: replace AsyncOpenAIType with direct AsyncOpenAI conditional import

leesf · 2026-04-16T01:18:29Z

And please format the code by pre-commit run --all-files

Thanks, updated the PR.

Copilot

Pull request overview

This PR adds optional Langfuse tracing support to OpenAIProvider by conditionally importing AsyncOpenAI from langfuse.openai only when LANGFUSE_SECRET_KEY is set and Langfuse is available, otherwise falling back to the standard OpenAI SDK client and logging a warning when tracing is misconfigured.

Changes:

Added conditional import logic to select langfuse.openai.AsyncOpenAI vs openai.AsyncOpenAI.
Added warning logging when LANGFUSE_SECRET_KEY is set but langfuse is not installed.
Added importlib.util and os imports to support the conditional behavior.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

+if os.environ.get("LANGFUSE_SECRET_KEY") and importlib.util.find_spec(
+    "langfuse",
+):
+    from langfuse.openai import AsyncOpenAI  # type: ignore[import]
+else:
+    if os.environ.get("LANGFUSE_SECRET_KEY"):
+        logger.warning(
+            "LANGFUSE_SECRET_KEY is set but langfuse is not installed; "
+            "install with `pip install langfuse` to enable tracing",
+        )
+    from openai import AsyncOpenAI  # pylint: disable=ungrouped-imports


Copilot AI and others added 7 commits April 15, 2026 06:34

Initial plan

514ac7a

feat(observability): add optional Langfuse tracing for LLM calls

e67965e

Agent-Logs-Url: https://github.com/leesf/QwenPaw/sessions/866badcb-6274-4c83-8e0e-38cbec1ea35a Co-authored-by: leesf <10128888+leesf@users.noreply.github.com>

feat: add langfuse openai SDK tracing to openai_provider only

3905fa8

Agent-Logs-Url: https://github.com/leesf/QwenPaw/sessions/c126b162-3248-4e4e-a1f0-a78488042930 Co-authored-by: leesf <10128888+leesf@users.noreply.github.com>

revert: remove all changes except openai_provider.py

a1048d0

Co-authored-by: leesf <10128888+leesf@users.noreply.github.com>

fix(openai_provider): update conditional import to warn when langfuse…

a5c1da5

… not installed Agent-Logs-Url: https://github.com/leesf/QwenPaw/sessions/9b75df92-92e2-4a75-b51c-44c9199d116e Co-authored-by: leesf <10128888+leesf@users.noreply.github.com>

fix(openai_provider): remove duplicate import logging

bd1b62a

Agent-Logs-Url: https://github.com/leesf/QwenPaw/sessions/69d05c99-82cf-45f2-ad1b-dde39b93e7a0 Co-authored-by: leesf <10128888+leesf@users.noreply.github.com>

Merge pull request #1 from leesf/copilot/add-langfuse-observability

c97a669

feat(observability): add optional Langfuse tracing for provider-backed LLM calls

Copilot AI review requested due to automatic review settings April 15, 2026 09:21

github-project-automation bot added this to QwenPaw Apr 15, 2026

github-project-automation bot moved this to Todo in QwenPaw Apr 15, 2026

github-actions bot added the first-time-contributor PR created by a first time contributor label Apr 15, 2026

Copilot AI reviewed Apr 15, 2026

View reviewed changes

Potential fix for pull request finding

8a9f5ec

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Copilot AI review requested due to automatic review settings April 15, 2026 09:44

Copilot started reviewing on behalf of leesf April 15, 2026 09:45 View session

Copilot AI reviewed Apr 15, 2026

View reviewed changes

Comment thread src/qwenpaw/providers/openai_provider.py Outdated

leesf requested a deployment to maintainer-approved April 15, 2026 10:05 — with GitHub Actions Waiting

xieyxclack added the Under Review label Apr 15, 2026

xieyxclack requested a review from qbc2016 April 15, 2026 10:06

Copilot AI and others added 7 commits April 15, 2026 12:21

fix: resolve pre-commit all-files failures in openai provider

7eb7064

Agent-Logs-Url: https://github.com/leesf/QwenPaw/sessions/daf93103-1af3-464d-8080-1ef1afcdedb8 Co-authored-by: leesf <10128888+leesf@users.noreply.github.com>

fix: make pre-commit all-files pass for openai provider imports

d89f73c

Agent-Logs-Url: https://github.com/leesf/QwenPaw/sessions/daf93103-1af3-464d-8080-1ef1afcdedb8 Co-authored-by: leesf <10128888+leesf@users.noreply.github.com>

Merge pull request #2 from leesf/copilot/fix-pre-commit-issues

2371919

Fix `pre-commit run --all-files` failures in OpenAI provider import/type wiring

refactor: replace AsyncOpenAIType with direct AsyncOpenAI conditional…

7df168d

… import Agent-Logs-Url: https://github.com/leesf/QwenPaw/sessions/e0dcd6e5-ed11-4cbc-9d2e-a7d5a4ff058c Co-authored-by: leesf <10128888+leesf@users.noreply.github.com>

fix: apply add-trailing-comma and suppress pylint ungrouped-imports

ca78121

Agent-Logs-Url: https://github.com/leesf/QwenPaw/sessions/959b4d27-8043-49a5-a557-941a69e4021f Co-authored-by: leesf <10128888+leesf@users.noreply.github.com>

fix: suppress pylint ungrouped-imports for conditional AsyncOpenAI im…

7e8a3e3

…port Agent-Logs-Url: https://github.com/leesf/QwenPaw/sessions/959b4d27-8043-49a5-a557-941a69e4021f Co-authored-by: leesf <10128888+leesf@users.noreply.github.com>

Merge pull request #3 from leesf/copilot/update-import-logic-for-openai

fba6311

refactor: replace AsyncOpenAIType with direct AsyncOpenAI conditional import

Copilot AI review requested due to automatic review settings April 16, 2026 01:17

Copilot started reviewing on behalf of leesf April 16, 2026 01:17 View session

Copilot AI reviewed Apr 16, 2026

View reviewed changes

github-actions bot mentioned this pull request Apr 16, 2026

🦞 OpenClaw 生态日报 2026-04-16 gsscsd/big_model_radar#194

Open

leesf temporarily deployed to maintainer-approved April 16, 2026 02:13 — with GitHub Actions Inactive

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: add langfuse openai SDK tracing to openai_provider only#3433

feat: add langfuse openai SDK tracing to openai_provider only#3433
leesf wants to merge 15 commits intoagentscope-ai:mainfrom
leesf:main

leesf commented Apr 15, 2026

Uh oh!

github-actions bot commented Apr 15, 2026

Welcome to QwenPaw! 🐾

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

xieyxclack commented Apr 15, 2026

Uh oh!

xieyxclack commented Apr 15, 2026

Uh oh!

leesf commented Apr 16, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Conversation

leesf commented Apr 15, 2026

Uh oh!

github-actions bot commented Apr 15, 2026

Welcome to QwenPaw! 🐾

📋 About PR Template

🙌 Join Developer Community

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

xieyxclack commented Apr 15, 2026

Uh oh!

xieyxclack commented Apr 15, 2026

Uh oh!

leesf commented Apr 16, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants