fix(observe): default IO capture on decorated functions

[hassiebp]

Important

observe() in observe.py now defaults IO capture to an environment variable when not explicitly set.

• Behavior:
  • observe() function in observe.py now defaults capture_input and capture_output to None instead of True.
  • If capture_input or capture_output is None, defaults to the value of LANGFUSE_OBSERVE_DECORATOR_IO_CAPTURE_ENABLED environment variable.
• Implementation:
  • Introduces should_capture_input and should_capture_output to determine final capture behavior.
  • Updates _async_observe and _sync_observe to use should_capture_input and should_capture_output instead of direct parameters.

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

Greptile Summary

Disclaimer: Experimental PR review

Modifies the default IO capture behavior in the @observe decorator to be controlled by environment variables instead of hardcoded defaults, improving configuration flexibility.

• Changed capture_input and capture_output defaults in langfuse/_client/observe.py from True to None, deferring to LANGFUSE_OBSERVE_DECORATOR_IO_CAPTURE_ENABLED environment variable
• Enhanced configurability by allowing global IO capture settings through environment variables
• Test coverage confirms functionality with mock traces and observation validation

[greptile-apps]

LGTM

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