feat(tools): 7-tool parity and description refresh by Dhravya · Pull Request #1432 · supermemoryai/supermemory

Dhravya · 2026-08-08T02:39:53Z

Summary

Refresh canonical tool descriptions in tools-shared.ts
Align OpenAI and AI SDK tool bindings with 7-tool surface
Export TOOL_DESCRIPTIONS / PARAMETER_DESCRIPTIONS from package index

Stacked on #1431

Test plan

bun run test:unit in packages/tools

Made with Cursor

Update shared tool descriptions (proactive search, documentAdd guidance), export TOOL_DESCRIPTIONS from package index, and align OpenAI/AI SDK tool schemas with memoryForget and document operations. Co-authored-by: Cursor <cursoragent@cursor.com>

cloudflare-workers-and-pages · 2026-08-08T02:40:05Z

Deploying with Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status	Name	Latest Commit	Updated (UTC)
✅ Deployment successful! View logs	supermemory-mcp	`7153801`	Aug 22 2026, 06:47 AM

claude · 2026-08-08T02:40:15Z

Claude finished @ved015's task in 1m 57s —— View job

Code Review Complete

Loading review tools
Fetching PR diff and existing review comments
Analyzing changes for bugs, security issues, and logic errors
Submitting review

Summary: Reviewed the 7-tool parity and SDK v3→v4 migration changes. The implementation is solid with improved container-tag scoping for document deletions.

One issue to address: restore the missing test assertion expect(stored).not.toContain("line3") (already flagged by Graphite) before merging.

Score: 9/10

cloudflare-workers-and-pages · 2026-08-08T02:41:52Z

Deploying with Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status	Name	Latest Commit	Preview URL	Updated (UTC)
✅ Deployment successful! View logs	supermemory-app	`7153801`	Commit Preview URL Branch Preview URL	Aug 22 2026, 06:49 AM

graphite-app · 2026-08-08T02:45:13Z

+	it.each(["$&", "$'", "$`", "$$"])(
+		"stores %s literally instead of expanding it as a replacement pattern",
+		async (dollarSequence) => {
+			const result = await tool.handleCommand({
+				command: "str_replace",
+				path: FILE_PATH,
+				old_str: "line3",
+				new_str: `price is ${dollarSequence} today`,
+			})
+
+			expect(result.success).toBe(true)
+			expect(addMock).toHaveBeenCalledTimes(1)
+			const stored = addMock.mock.calls[0]?.[0]?.content as string
+			expect(stored).toContain(`price is ${dollarSequence} today`)
+		},
+	)


Test regression: the assertion expect(stored).not.toContain("line3") was removed during reformatting. The test now only verifies the new string was added but doesn't verify the old string was actually replaced. This weakens test coverage and won't catch if str_replace fails to remove the old content.

expect(stored).toContain(`price is ${dollarSequence} today`) expect(stored).not.toContain("line3") // Add this back

Suggested change

it.each(["$&", "$'", "$`", "$$"])(

"stores %s literally instead of expanding it as a replacement pattern",

async (dollarSequence) => {

const result = await tool.handleCommand({

command: "str_replace",

path: FILE_PATH,

old_str: "line3",

new_str: `price is ${dollarSequence} today`,

})

expect(result.success).toBe(true)

expect(addMock).toHaveBeenCalledTimes(1)

const stored = addMock.mock.calls[0]?.[0]?.content as string

expect(stored).toContain(`price is ${dollarSequence} today`)

},

)

it.each(["$&", "$'", "$`", "$$"])(

"stores %s literally instead of expanding it as a replacement pattern",

async (dollarSequence) => {

const result = await tool.handleCommand({

command: "str_replace",

path: FILE_PATH,

old_str: "line3",

new_str: `price is ${dollarSequence} today`,

})

expect(result.success).toBe(true)

expect(addMock).toHaveBeenCalledTimes(1)

const stored = addMock.mock.calls[0]?.[0]?.content as string

expect(stored).toContain(`price is ${dollarSequence} today`)

expect(stored).not.toContain("line3") // Add this back

},

)

Spotted by Graphite

Is this helpful? React 👍 or 👎 to let us know.

Dhravya · 2026-08-08T02:47:50Z

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.
Learn more

How to use the Graphite Merge Queue

Add 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.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

socket-security · 2026-08-18T15:20:41Z

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff	Package	Supply Chain Security	Vulnerability	Quality	Maintenance	License
	npm/supermemory@4.25.4

View full report

claude

Overview: This PR upgrades the supermemory SDK from v3 to v4, aligns 7 tool bindings with the new API surface, and refreshes tool descriptions for clarity.

Issues found: The missing test assertion (expect(stored).not.toContain("line3")) was already flagged by Graphite — this is a real test coverage regression that should be restored. The other Graphite comments (unused includeFullDocs, implicit any type, Biome formatting) are linter/style issues.

The core implementation changes are solid:

The deleteDocumentByIdentifier logic properly resolves IDs within scoped container tags before deletion, preventing cross-scope data access
The container tag validation correctly rejects empty strings
The getFileDocument refactor to use document listing + GET instead of search is more reliable for exact-match file operations
Error handling for ambiguous document identifiers is appropriate

Score: 9/10 — Clean logic, improved scoping safety. Restore the dropped test assertion before merging.

- Replace `as any` with typed assertion in claude-memory.ts - Apply Biome formatting fixes Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

graphite-app Bot assigned Dhravya Aug 8, 2026

Dhravya mentioned this pull request Aug 8, 2026

feat(ai-sdk): re-export 7-tool surface #1433

Open

1 task

graphite-app Bot reviewed Aug 8, 2026

View reviewed changes

Dhravya force-pushed the feat/tools-seven-tool-parity branch from 222bda0 to d5937b8 Compare August 8, 2026 02:47

Dhravya mentioned this pull request Aug 8, 2026

fix(openai-sdk-python): migrate to v4 Supermemory APIs #1429

Open

1 task

graphite-app Bot reviewed Aug 14, 2026

View reviewed changes

Comment thread packages/tools/src/ai-sdk.ts

Comment thread packages/tools/src/claude-memory.ts Outdated

Comment thread packages/tools/src/tools-shared.ts Outdated

Dhravya force-pushed the feat/tools-seven-tool-parity branch from 34d7599 to d5937b8 Compare August 18, 2026 15:18

This was referenced Aug 18, 2026

feat(tools): SDK-level cross-source memory deduplication #1531

Open

feat(python-sdks): SDK-level cross-source memory deduplication #1532

Open

feat(sdk-playground): reflect SDK-owned memory block in debug view #1533

Open

chore: update bun.lock

6103019

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Dhravya assigned ved015 Aug 18, 2026

Dhravya requested a review from MaheshtheDev August 18, 2026 15:20

ved015 added 2 commits August 22, 2026 12:10

chore: merge latest stacked changes

d04bb3b

fix(tools): harden seven-tool parity

d21d661

claude Bot reviewed Aug 22, 2026

View reviewed changes

fix(tools): resolve lint and format errors

7153801

- Replace `as any` with typed assertion in claude-memory.ts - Apply Biome formatting fixes Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Navigation Menu

Uh oh!

feat(tools): 7-tool parity and description refresh - #1432

feat(tools): 7-tool parity and description refresh#1432
Dhravya wants to merge 5 commits into
chore/ci-python-sdk-testsfrom
feat/tools-seven-tool-parity

Dhravya commented Aug 8, 2026

Uh oh!

cloudflare-workers-and-pages Bot commented Aug 8, 2026 •

edited

Loading

Uh oh!

claude Bot commented Aug 8, 2026 •

edited

Loading

Uh oh!

cloudflare-workers-and-pages Bot commented Aug 8, 2026 •

edited

Loading

Uh oh!

graphite-app Bot Aug 8, 2026

Uh oh!

Dhravya commented Aug 8, 2026 •

edited

Loading

Uh oh!

Uh oh!

Uh oh!

Uh oh!

socket-security Bot commented Aug 18, 2026 •

edited

Loading

Uh oh!

claude Bot left a comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

Dhravya commented Aug 8, 2026

Summary

Test plan

Uh oh!

cloudflare-workers-and-pages Bot commented Aug 8, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Deploying with Cloudflare Workers

Uh oh!

claude Bot commented Aug 8, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Code Review Complete

Uh oh!

cloudflare-workers-and-pages Bot commented Aug 8, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Deploying with Cloudflare Workers

Uh oh!

graphite-app Bot Aug 8, 2026

Choose a reason for hiding this comment

Uh oh!

Dhravya commented Aug 8, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

How to use the Graphite Merge Queue

Uh oh!

Uh oh!

Uh oh!

Uh oh!

socket-security Bot commented Aug 18, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

claude Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

cloudflare-workers-and-pages Bot commented Aug 8, 2026 •

edited

Loading

claude Bot commented Aug 8, 2026 •

edited

Loading

cloudflare-workers-and-pages Bot commented Aug 8, 2026 •

edited

Loading

Dhravya commented Aug 8, 2026 •

edited

Loading

socket-security Bot commented Aug 18, 2026 •

edited

Loading