feat(api): update API spec from langfuse/langfuse 43a10b5#1717
feat(api): update API spec from langfuse/langfuse 43a10b5#1717langfuse-bot wants to merge 1 commit into
Conversation
|
@claude review |
There was a problem hiding this comment.
LGTM, docstring-only update to auto-generated API client code.
Extended reasoning...
Overview
This PR updates docstrings in two auto-generated Fern API client files (langfuse/api/sessions/client.py and langfuse/api/sessions/raw_client.py) to recommend the v2 observations endpoint as an alternative for new data extraction workflows. The changes are limited to the docstrings of list and get methods (both sync and async variants) — no logic, parameters, return types, request paths, or error handling are modified.
Security risks
None. These are documentation-only changes inside auto-generated client code. No request construction, authentication, or response parsing was touched.
Level of scrutiny
Minimal. These files are auto-generated by Fern from the upstream API spec, and the diff is purely textual within docstrings. The changes match the stated PR purpose ("update API spec from langfuse/langfuse 43a10b5").
Other factors
The bug hunting system found no issues. There are no outstanding reviewer comments to address.
Greptile Summary
This PR syncs the auto-generated API client from the upstream
langfuse/langfusespec (commit43a10b5), updating only docstrings in the sessions client classes. No functional or behavioral changes are introduced.list(get sessions) docstring in all four client classes now carries a deprecation notice recommending the v2 observations endpoint with a bounded time range instead of the legacy sessions endpoint.get(get a session) docstring now points large-session and new extraction workflows toGET /api/public/v2/observationswith asessionIdfilter, replacing the older suggestion to useGET /api/public/traces?sessionId=<sessionId>.Confidence Score: 5/5
Safe to merge — all changes are docstring-only updates with no impact on runtime behavior.
The entire diff consists of docstring updates across four auto-generated client classes. No logic, parameters, return types, or imports were touched, so there is no risk of regression.
No files require special attention.
Sequence Diagram
%%{init: {'theme': 'neutral'}}%% sequenceDiagram participant Caller participant SessionsClient participant LangfuseAPI Note over SessionsClient: list() — legacy endpoint Caller->>SessionsClient: list(page, limit, ...) SessionsClient->>LangfuseAPI: GET /api/public/sessions LangfuseAPI-->>SessionsClient: PaginatedSessions SessionsClient-->>Caller: PaginatedSessions Note over SessionsClient: Recommended alternative (v2) Caller->>LangfuseAPI: "GET /api/public/v2/observations?fromStartTime=from&toStartTime=to" LangfuseAPI-->>Caller: Observations (group by sessionId) Note over SessionsClient: get() — legacy endpoint Caller->>SessionsClient: get(session_id) SessionsClient->>LangfuseAPI: "GET /api/public/sessions/{session_id}" LangfuseAPI-->>SessionsClient: SessionWithTraces (traces unpaginated) SessionsClient-->>Caller: SessionWithTraces Note over SessionsClient: Recommended alternative (v2) Caller->>LangfuseAPI: "GET /api/public/v2/observations?filter=sessionId filter&fromStartTime=from" LangfuseAPI-->>Caller: Paginated observations%%{init: {'theme': 'base', 'themeVariables': {"darkMode": true, "background": "#0d1117", "primaryColor": "#21262d", "primaryTextColor": "#e6edf3", "primaryBorderColor": "#8b949e", "lineColor": "#8b949e", "textColor": "#e6edf3", "edgeLabelBackground": "#161b22", "actorBkg": "#21262d", "actorBorder": "#8b949e", "actorTextColor": "#e6edf3", "actorLineColor": "#8b949e", "signalColor": "#8b949e", "signalTextColor": "#e6edf3", "noteBkgColor": "#373320", "noteBorderColor": "#d4a72c", "noteTextColor": "#f0e6c0", "labelBoxBkgColor": "#21262d", "labelBoxBorderColor": "#8b949e", "labelTextColor": "#e6edf3", "loopTextColor": "#e6edf3", "activationBkgColor": "#30363d", "activationBorderColor": "#8b949e"}}}%% sequenceDiagram participant Caller participant SessionsClient participant LangfuseAPI Note over SessionsClient: list() — legacy endpoint Caller->>SessionsClient: list(page, limit, ...) SessionsClient->>LangfuseAPI: GET /api/public/sessions LangfuseAPI-->>SessionsClient: PaginatedSessions SessionsClient-->>Caller: PaginatedSessions Note over SessionsClient: Recommended alternative (v2) Caller->>LangfuseAPI: "GET /api/public/v2/observations?fromStartTime=from&toStartTime=to" LangfuseAPI-->>Caller: Observations (group by sessionId) Note over SessionsClient: get() — legacy endpoint Caller->>SessionsClient: get(session_id) SessionsClient->>LangfuseAPI: "GET /api/public/sessions/{session_id}" LangfuseAPI-->>SessionsClient: SessionWithTraces (traces unpaginated) SessionsClient-->>Caller: SessionWithTraces Note over SessionsClient: Recommended alternative (v2) Caller->>LangfuseAPI: "GET /api/public/v2/observations?filter=sessionId filter&fromStartTime=from" LangfuseAPI-->>Caller: Paginated observationsReviews (1): Last reviewed commit: "feat(api): update API spec from langfuse..." | Re-trigger Greptile