chore: remove enabling local export mode via env

[hassiebp]

Important

Remove local export mode activation via environment variable, replacing it with a constructor parameter in LangfuseCoreStateless.

• Behavior:
  • Remove enabling local export mode via LANGFUSE_JS_SDK_LOCAL_EVENT_EXPORT_ENABLED environment variable.
  • Add _isLocalEventExportEnabled parameter to LangfuseCoreOptions in types.ts.
  • Update LangfuseCoreStateless constructor to use _isLocalEventExportEnabled parameter.
• Functions:
  • Rename _shutdownAdmin() to _exportLocalEvents() in index.ts.
  • Replace adminIngestionEvents with localEventExportMap in index.ts.
• Tests:
  • Update tests in langfuse.flush.spec.ts and langfuse-integration-langchain.spec.ts to use _isLocalEventExportEnabled parameter instead of environment variable.

^{This description was created by for 4a0e37e. 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	Nov 20, 2024 10:04am

[greptile-apps]

Disclaimer: Experimental PR review

PR Summary

Simplified local event export configuration in langfuse-js by replacing environment variables with direct constructor parameters.

• Replaced LANGFUSE_JS_SDK_LOCAL_EVENT_EXPORT_ENABLED env var with _isLocalEventExportEnabled boolean flag in CallbackHandler constructor
• Renamed adminIngestionEvents to localEventExportMap and _shutdownAdmin to _exportLocalEvents for clearer intent
• Added validation logic to ensure _projectId is provided when _isLocalEventExportEnabled is true
• Updated test cases to use new configuration approach instead of managing environment variables

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