docs(sdk): document loadNdjson; retire stale version claims - #18
Merged
Merged
Conversation
…laims - Bulk-load section gains og.loadNdjson() (server 0.9.0+): raw application/x-ndjson body, strict bounded graph batch, envelope shapes, 413 bound, fork-if-missing rule. - Retry table adds loadNdjson and corrects the append row: a 0.9.0+ server rejects duplicate ids with a key conflict instead of absorbing them. - The intro note now points at Server compatibility instead of hardcoding a 0.7.x target.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
README catch-up for the 0.9.0 sync (#17): documents
og.loadNdjson()with envelope shapes and bounds, adds it to the retry table, corrects theappendretry row for 0.9.0 key-conflict semantics, and stops hardcoding a 0.7.x server target in favor of the Server compatibility section. npm shows this README from the next publish; GitHub immediately.Greptile Summary
The README now documents the 0.9.0
loadNdjson()API and removes a stale fixed-version compatibility claim.loadNdjson()to the retry guidance and clarifies version-dependent append behavior.Confidence Score: 5/5
The documentation-only PR appears safe to merge with no actionable correctness or security issues identified.
The new method signature, request encoding, envelope examples, default merge mode, size-limit behavior, and retry guidance agree with the SDK and server contracts.
Important Files Changed
Reviews (1): Last reviewed commit: "docs(sdk): document loadNdjson; retire t..." | Re-trigger Greptile