fix: align Agent Memory docs with current API spec#3332
Merged
cmilesb merged 1 commit intoMay 15, 2026
Conversation
Fix 8 inconsistencies between the hand-written documentation and the
Agent Memory OpenAPI spec (MemoryDataPlaneServer):
api-examples.md:
- Fix curl path: /v1/caches/ -> /v1/stores/ (wrong path segment)
- Fix Store ID description typo ("Store ID ID of your cache")
- Fix AddSessionEvent endpoint: /session-memory -> /session-memory/events
- Fix createdAt format: Unix ms -> ISO 8601 date-time (per MOD-15623)
- Fix duplicate namespace row in filter table (second should be topics)
- Add missing memoryType filter to the table
- Fix createdAt filter data type: integer -> string (ISO 8601)
create-service.md:
- Fix Long-term TTL description (was copy-pasted from database selector)
Verified against the Smithy-generated OpenAPI spec from langcache main
(commit fd8b6c2b).
Co-authored-by: Cursor <cursoragent@cursor.com>
|
|
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
3 participants
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.
Summary
Fixes 8 inconsistencies between the hand-written Agent Memory documentation pages and the current OpenAPI spec (verified against the Smithy-generated spec from langcache main
fd8b6c2b).Fixes in
api-examples.md/v1/caches/instead of/v1/stores//events(/session-memory->/session-memory/events)createdAtuses Unix ms (1778076906498) instead of ISO 8601 ("2026-05-02T18:15:06Z") per MOD-15623namespacerow; second should betopicsmemoryTypefiltercreatedAtfilter data type listed asintegerinstead ofstring (ISO 8601)Fixes in
create-service.mdTest plan
Made with Cursor
Note
Low Risk
Low risk documentation-only updates, but they correct previously broken endpoints and payload formats that could mislead users if incorrect.
Overview
Aligns
api-examples.mdwith the current Agent Memory API by fixing the base path (/v1/stores), correcting the session event endpoint to/session-memory/events, updatingcreatedAtexamples/filters to ISO-8601 strings, and fixing/expanding the long-term memory search filter table (addstopicsandmemoryType, removes duplicate row).Clarifies
create-service.mdby correcting the Long-term TTL description to describe long-term memory retention rather than unrelated database selection text.Reviewed by Cursor Bugbot for commit f32e3ab. Bugbot is set up for automated code reviews on this repo. Configure here.