add semantically mass-forget memories endpoint to docs#1170
Conversation
|
Vorflux skipped this auto review because this account has reached its Auto Review daily review limit (20/20). You can change this in Auto Review Settings: https://us1.vorflux.com/supermemory/settings?section=pull-requests |
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
How to use the Graphite Merge QueueAdd the label Main to this PR to add it to the merge queue. You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has enabled the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. This stack of pull requests is managed by Graphite. Learn more about stacking. |
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| ✅ Deployment successful! View logs |
supermemory-mcp | d2eab9d | Jun 27 2026, 07:28 PM |
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
supermemory-app | d2eab9d | Commit Preview URL | Jun 27 2026, 07:29 PM |
|
Claude finished @sohamd22's task in 1m 59s —— View job Review Complete
Summary: This is a clean docs-only PR. The Forget Memory docs were corrected to match the actual SDK usage (verified in Score: 10/10 — No bugs, security issues, or logic errors found. |
There was a problem hiding this comment.
Overview: Docs-only PR that corrects the Forget Memory endpoint documentation (from non-existent POST /v4/memories/{id}/forget to actual DELETE /v4/memories with body params) and adds documentation for the new semantic bulk-forget endpoint (POST /v4/memories/forget-matching).
Issues found: None — this looks good to ship.
I verified the corrected Forget Memory docs match the SDK usage pattern in apps/mcp/src/client.ts:179-228, where this.client.memories.forget() takes {id | content, containerTag} parameters. The new Forget Matching docs are well-structured with appropriate warnings about the destructive nature of bulk operations, dryRun support, and safety bounds (threshold, maxForget).
Score: 10/10

Docs: add Forget Matching endpoint + fix stale Forget Memory docs
What this does
POST /v4/memories/forget-matchingendpoint — semantic/promptable mass-forget. CoversdryRun(preview),threshold/maxForgetsafety bounds, the request/response shape, andforgetBatchId.This PR is docs-only. The existing forget endpoint's behavior/contract is untouched — the previous docs were simply wrong and described a route that has never existed:
POST /v4/memories/{id}/forgetDELETE /v4/memories{ id | content, containerTag, reason? }The handler (
forgetMemoryinapps/api/src/routes/v4/memories/handlers.ts) was not modified — this just makes the docs reflect reality.Also