Skip to content

feat(stream): Created base handler to share functionality between handlers, mocked logger functionality#465

Merged
Murike merged 12 commits intomainfrom
feature/parity-stream
Jan 27, 2026
Merged

feat(stream): Created base handler to share functionality between handlers, mocked logger functionality#465
Murike merged 12 commits intomainfrom
feature/parity-stream

Conversation

@Murike
Copy link
Contributor

@Murike Murike commented Jan 19, 2026

User description

(TS SDK Parity Effort) Missing - Streaming Support

[sc-43778] - https://app.shortcut.com/galileo/story/43778/ts-sdk-parity-effort-missing-streaming-support

Description

  • Added base class to share functionality between handlers (OpenAI, Langchain)
  • Added more support for error management

Obs.: The Galileo Logger PR implemented several features for stream mode support. Current PR reviewed endpoints and prepared shared features the streaming operation of OpenAI and Langchain handlers can use. Future implementations can use and expand these shared functionalities too, the current implementation is limited due to differences on how OpenAI and Langchain work.




Generated description

Below is a concise technical summary of the changes proposed in this PR:
Establishes a foundational streaming architecture by introducing a BaseStreamingAdapter and specialized utilities for metrics tracking and span finalization. Centralizes error handling and common JSON types to ensure consistency across different LLM provider implementations.

TopicDetails
Streaming Core Provides a shared framework for LLM streaming through the BaseStreamingAdapter and metrics tracking components.
Modified files (9)
  • src/types/streaming-adapter.types.ts
  • src/utils/galileo-logger.ts
  • src/utils/streaming/base-streaming-adapter.ts
  • src/utils/streaming/finalizer.ts
  • src/utils/streaming/index.ts
  • src/utils/streaming/metrics.ts
  • tests/utils/streaming/base-streaming-adapter.test.ts
  • tests/utils/streaming/finalizer.test.ts
  • tests/utils/streaming/metrics.test.ts
Latest Contributors(2)
UserCommitDate
Murikefeat-logger-Improved-b...January 19, 2026
juandrucker@rungalileo.iofeat-support-ingesting...July 25, 2025
Error & Type Utils Standardizes error mapping via ErrorManager and consolidates core JSON type definitions.
Modified files (6)
  • src/types/base.types.ts
  • src/types/logging/logger.types.ts
  • src/types/logging/span.types.ts
  • src/types/logging/step.types.ts
  • src/utils/errors.ts
  • src/utils/task-handler.ts
Latest Contributors(2)
UserCommitDate
Murikefeat-logger-Improved-b...January 19, 2026
juandrucker@rungalileo.iofeat-support-ingesting...July 25, 2025
This pull request is reviewed by Baz. Review like a pro on (Baz).

@Murike Murike changed the title feature(stream): Created base handler to share functionality between handlers, mocked logger functionality feat(stream): Created base handler to share functionality between handlers, mocked logger functionality Jan 19, 2026
@Murike Murike force-pushed the feature/parity-stream branch from a08415a to c0bc8f4 Compare January 20, 2026 21:42
@Murike
Copy link
Contributor Author

Murike commented Jan 21, 2026

Excessive number of "formatting" commits was for troubleshooting problem in local pre-commit. Issue was resolved cleaning cache, commands below:

pre-commit clean 
rm -rf ~/.cache/pre-commit

@Murike Murike merged commit 92a9e90 into main Jan 27, 2026
8 checks passed
@Murike Murike deleted the feature/parity-stream branch January 27, 2026 13:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants