fix(openai): support passing in traceName

[hassiebp]

Important

Add support for traceName in Langfuse configuration for OpenAI SDK tracing, defaulting to generationName if not provided.

• Behavior:
  • Support for traceName in LangfuseConfig in observeOpenAI.ts and traceMethod.ts.
  • traceName defaults to generationName if not provided.
• Types:
  • Add traceName to LangfuseNewTraceConfig in types.ts.
• Functions:
  • Update observeOpenAI() to include traceName in config.
  • Update wrapMethod() to use traceName for tracing.

^{This description was created by for 78db7d9. It will automatically update as commits are pushed.}

[vercel]

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

Name	Status	Preview	Updated (UTC)
langfuse-js	✅ Ready (Inspect)	Visit Preview	Dec 16, 2024 10:40am

[greptile-apps]

Disclaimer: Experimental PR review

PR Summary

Adds support for custom trace naming in the OpenAI integration by introducing an optional traceName parameter, allowing more flexible trace identification while maintaining backward compatibility.

• Added optional traceName property to LangfuseNewTraceConfig type in /langfuse/src/openai/types.ts
• Modified observeOpenAI function to prioritize traceName from config, falling back to generationName in /langfuse/src/openai/observeOpenAI.ts
• Updated trace creation in /langfuse/src/openai/traceMethod.ts to use the provided traceName when creating new traces

_{3 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile}

[greptile-apps]

Disclaimer: Experimental PR review

PR Summary

(updates since last review)

No major changes found since last review. The previous review accurately covered the key changes related to the traceName parameter implementation across the relevant files.

_{2 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile}