Skip to content

feat(api): update API spec from langfuse/langfuse 270f036#1521

Merged
hassiebp merged 2 commits intomainfrom
api-spec-bot-270f036
Feb 11, 2026
Merged

feat(api): update API spec from langfuse/langfuse 270f036#1521
hassiebp merged 2 commits intomainfrom
api-spec-bot-270f036

Conversation

@langfuse-bot
Copy link
Collaborator

@langfuse-bot langfuse-bot commented Feb 10, 2026

Important

Adds observation_id filter to get() in ScoreV2Client and AsyncScoreV2Client for filtering scores by observation IDs.

  • Behavior:
    • Adds observation_id parameter to get() in ScoreV2Client and AsyncScoreV2Client to filter scores by observation IDs.
    • Updates API documentation in reference.md to include observation_id parameter description.
  • Code:
    • Modifies get() method in score_v_2/client.py to accept observation_id and include it in request parameters.
    • Updates docstrings in score_v_2/client.py to document observation_id parameter usage.

This description was created by Ellipsis for 3720157. You can customize this summary. It will automatically update as commits are pushed.

Disclaimer: Experimental PR review

Greptile Overview

Greptile Summary

This PR updates the generated Langfuse API client and reference docs to expose a new observation_id filter on client.score_v_2.get() (sync + async). The client now forwards this as the observationId query parameter to GET /api/public/v2/scores, and the API reference is updated to document the new argument.

No correctness, integration, or import-order issues were found in the changeset; the update is consistent across sync/async clients and matches the existing codegen patterns.

Confidence Score: 5/5

  • This PR is safe to merge with minimal risk.
  • Changes are small, consistent across sync/async clients, and only add an optional query parameter plus matching documentation; no breaking signature changes or integration wiring changes were introduced.
  • No files require special attention

Important Files Changed

Filename Overview
langfuse/api/reference.md Adds observation_id parameter documentation for client.score_v_2.get(); change matches existing reference format.
langfuse/api/resources/score_v_2/client.py Adds optional observation_id filter to sync/async ScoreV2Client.get() and passes it as observationId query param; imports remain at module top.

Sequence Diagram

sequenceDiagram
    participant U as User
    participant SDK as langfuse-python SDK
    participant S as ScoreV2Client/AsyncScoreV2Client
    participant API as Langfuse API

    U->>SDK: client.score_v_2.get(..., observation_id=...)
    SDK->>S: get(observation_id=...)
    S->>API: GET /api/public/v2/scores?observationId=...
    API-->>S: 2xx GetScoresResponse JSON
    S-->>SDK: parse_obj_as(GetScoresResponse)
    SDK-->>U: GetScoresResponse
Loading

Context used:

  • Rule from dashboard - Move imports to the top of the module instead of placing them within functions or methods. (source)

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

2 files reviewed, no comments

Edit Code Review Agent Settings | Greptile

@sumerman sumerman enabled auto-merge (squash) February 11, 2026 09:22
@hassiebp hassiebp disabled auto-merge February 11, 2026 09:44
@hassiebp hassiebp merged commit bd0fafc into main Feb 11, 2026
7 of 12 checks passed
@hassiebp hassiebp deleted the api-spec-bot-270f036 branch February 11, 2026 09:44
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.

3 participants