fix(openai): add check for metadata NotGiven

[hassiebp]

Important

Fixes metadata validation in _get_langfuse_data_from_kwargs to handle NotGiven type in langfuse/openai.py.

• Behavior:
  • In langfuse/openai.py, _get_langfuse_data_from_kwargs now checks if metadata is an instance of NotGiven before validating it as a dictionary.
  • Prevents TypeError when metadata is not provided in kwargs.

^{This description was created by for 0959b2b. You can customize this summary. It will automatically update as commits are pushed.}

Greptile Summary

Disclaimer: Experimental PR review

Improves OpenAI SDK compatibility by fixing metadata validation to handle NotGiven type, particularly important for OpenAI Agent SDK integration.

• Modified langfuse/openai.py to properly handle NotGiven sentinel value in metadata parameter validation
• Prevents incorrect validation errors when metadata is legitimately NotGiven by OpenAI SDK
• Enhances compatibility with OpenAI Agent SDK and similar features that use NotGiven as a valid state

[greptile-apps]

LGTM

_{1 file reviewed, no comments
Edit PR Review Bot Settings | Greptile}

[ellipsis-dev]

The new check for NotGiven is good; consider adding an inline comment explaining why NotGiven is skipped in type validation.

Suggested change

	and not isinstance(metadata, NotGiven)
	and not isinstance(metadata, NotGiven) # NotGiven is a sentinel for missing values, not a dict