feat(api): update API spec from langfuse/langfuse b8db8b6#831
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
@claude review |
There was a problem hiding this comment.
Auto-generated API spec update — doc-only changes (deprecation notices, clarifications, character limit doc).
Extended reasoning...
Overview
This PR is an auto-generated Fern API spec update across 8 files in packages/core/src/api/api/resources/. Every change in the diff is a JSDoc/comment update — no type signatures, no enum values, no client logic, no endpoints added or removed. Touched areas: blob storage integration types (clarifying how exportFieldGroups now applies to all sources and documenting a new 2026-06-22 cloud deprecation gate), dataset item id (255-char limit note), v2 scores client (deprecation notices pointing to v3), and v3 scores client (expanded field-group documentation).
Security risks
None. All changes are to TypeScript doc comments in auto-generated client code; no runtime behavior, no auth/crypto/permissions code, no user input handling.
Level of scrutiny
Low. These are mechanical Fern-generated comment updates that mirror upstream spec changes. The only thing to verify is that no type/enum/signature drifted unexpectedly, which a quick scan of the diff confirms — only JSDoc lines changed.
Other factors
The bug hunting system found no issues. The PR follows the established feat(api): update API spec from langfuse/langfuse <sha> pattern used for routine spec syncs in this repo. No reviewer comments outstanding.
There was a problem hiding this comment.
LGTM — auto-generated API spec sync; only JSDoc/type comment changes, no runtime logic.
Extended reasoning...
Overview
This is an automated API spec sync from langfuse/langfuse@b8db8b6 regenerating Fern-generated TypeScript client files. All 8 changed files start with the This file was auto-generated by Fern header. The diff is purely JSDoc/comment updates plus a 255-char length note on CreateDatasetItemRequest.id. No runtime logic, no exported types, no enum values, and no method signatures change.
Security risks
None. There is no authentication, crypto, input validation, or network-facing logic touched. The changes are confined to documentation strings and a comment about a length constraint that the server enforces.
Level of scrutiny
Low. These files are codegen output, regenerated from the upstream OpenAPI spec; reviewing them by hand has limited value beyond confirming the diff is documentation-only. The only items worth checking are the two Greptile flags (P1 core removal, P2 exportFieldGroups nullability), both of which the maintainer (niklassemmler) already addressed inline confirming the server still accepts core and that no breaking change exists.
Other factors
The bug-hunting system found no bugs. The maintainer has already engaged with the only substantive feedback and resolved both inline threads. Nothing else in the diff is ambiguous or requires human judgment.
Greptile Summary
This is an automated API spec sync from
langfuse/langfuse@b8db8b6, updating TypeScript type definitions and JSDoc comments across the blob storage integration and scores resources — no runtime logic is changed.GET /api/public/v1/scores(list and by-ID) are now marked deprecated in the JSDoc, directing users to v3; v3's field-group documentation was expanded to enumerate exactly which fields each group adds.exportFieldGroupsnow applies to all export sources (includingLEGACY_TRACES_OBSERVATIONS); a new Cloud-only integration deprecation gate (effective 2026-06-22) is added alongside the existing project-level gate.CreateDatasetItemRequest.idnow documents the 255-character max length.Confidence Score: 4/5
Safe to merge for documentation; one backward-incompatible parameter change warrants a changelog note.
The
fields=coreremoval from the allowed values list inGetScoresV3Requestis a subtle breaking change for any consumer who was passingcoreexplicitly per the old spec — the server returns HTTP 400 for unknown group names.packages/core/src/api/api/resources/scoresV3/client/requests/GetScoresV3Request.ts and packages/core/src/api/api/resources/blobStorageIntegrations/types/BlobStorageIntegrationResponse.ts
Sequence Diagram
sequenceDiagram participant SDK as JS SDK Consumer participant v1 as GET /v1/scores participant v3 as GET /v3/scores SDK->>v1: getMany() or get(scoreId) note over v1: Deprecated, not available on Langfuse v4+ v1-->>SDK: scores response SDK->>v3: "getMany with fields=core" note over v3: core removed from allowed values v3-->>SDK: HTTP 400 unknown group name SDK->>v3: "getMany with fields=details,subject,annotation" note over v3: Core fields always returned automatically v3-->>SDK: scores with core plus requested groupsPrompt To Fix All With AI
Reviews (1): Last reviewed commit: "feat(api): update API spec from langfuse..." | Re-trigger Greptile