feat(ai-sdk): re-export 7-tool surface by Dhravya · Pull Request #1433 · supermemoryai/supermemory

Dhravya · 2026-08-08T02:40:00Z

Summary

Re-export full tool set from @supermemory/tools/ai-sdk
Add unit tests for tool re-exports

Stacked on #1432

Test plan

bun run test:unit in packages/ai-sdk

Made with Cursor

…ools Thin @supermemory/ai-sdk package now mirrors the canonical 7-tool surface with updated tests. Co-authored-by: Cursor <cursoragent@cursor.com>

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

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

Review Complete

Load review tools
Get PR diff and analyze changes
Check existing review comments
Review code for bugs, security issues, and logic errors
Submit review

Overview: This PR replaces inline tool implementations in @supermemory/ai-sdk with re-exports from @supermemory/tools/ai-sdk, expanding the surface from 2 tools to 7.

Issues found:

Cursor Bugbot correctly identified the includeFullDocs parameter bug — it's defined in the schema but not passed to client.search(). However, this bug exists in the base branch (feat/tools-seven-tool-parity), not in this PR. This PR correctly re-exports whatever the shared package provides. The fix belongs in #1432.

The re-export itself is correct, CI workflow changes properly detect changes in both packages, and the publish workflow's retry loop for npm dependency availability is a sensible pattern.

Score: 9/10

Clean re-export with good test coverage and documentation updates. No bugs introduced by this PR.

|

cursor

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 9c3f84b. Configure here.}

cursor · 2026-08-08T02:41:19Z

+export {
+	supermemoryTools,
+	searchMemoriesTool,
+	addMemoryTool,


Search ignores includeFullDocs

Medium Severity

Replacing the inline tools with re-exports from @supermemory/tools/ai-sdk changes searchMemories: the tool schema still accepts includeFullDocs, but the shared implementation never passes it to client.search, unlike the removed @supermemory/ai-sdk code that forwarded it to search. Callers and models cannot control full-document inclusion in results.

^{Reviewed by Cursor Bugbot for commit 9c3f84b. Configure here.}

graphite-app · 2026-08-08T02:41:49Z

+	const testApiKey = process.env.SUPERMEMORY_API_KEY as string
+	const testOpenAIKey = process.env.OPENAI_API_KEY as string


The variables testApiKey and testOpenAIKey use type assertions (as string) to cast string | undefined to string. According to the style guide rule on Type assertions and casting, unnecessary type assertions should be avoided and type annotations should be used instead of assertions for object literals. While this is not an object literal, the spirit of the rule is to prefer type annotations over assertions. More importantly, using as string here is unsafe — it suppresses the undefined possibility without a runtime check. Consider using a non-null assertion only after a guard, or restructuring so the type is narrowed properly (e.g., via a runtime check before assignment).

Suggested change

const testApiKey = process.env.SUPERMEMORY_API_KEY as string

const testOpenAIKey = process.env.OPENAI_API_KEY as string

const testApiKey = process.env.SUPERMEMORY_API_KEY ?? (() => { throw new Error("SUPERMEMORY_API_KEY is not set") })()

const testOpenAIKey = process.env.OPENAI_API_KEY ?? (() => { throw new Error("OPENAI_API_KEY is not set") })()

Spotted by Graphite (based on custom rule: TypeScript style guide (Google))

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

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

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	`dbafb2d`	Aug 24 2026, 04:23 PM

cloudflare-workers-and-pages · 2026-08-08T02:47:48Z

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	`dbafb2d`	Commit Preview URL Branch Preview URL	Aug 24 2026, 04:25 PM

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

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.

claude

Overview: This PR replaces inline tool implementations in @supermemory/ai-sdk with re-exports from @supermemory/tools/ai-sdk, adding 5 new tools to the surface.

Issues found:

Cursor Bugbot correctly identified that includeFullDocs is accepted in the searchMemories schema but not passed to client.search() in packages/tools/src/ai-sdk.ts:59-65. I verified this — the parameter is destructured on line 55 but never used. This bug exists in the base branch (feat/tools-seven-tool-parity) and should be fixed in PR #1432, not here.

The re-export itself is correct — this PR simply surfaces whatever the shared package provides.

Score: 8/10

The code change is straightforward and the test coverage looks good. Fix the includeFullDocs bug in the base PR before merging this stack.

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

claude

Overview: This PR replaces inline tool implementations in @supermemory/ai-sdk with re-exports from @supermemory/tools/ai-sdk, expanding the surface from 2 tools to 7.

Issues found:

Cursor Bugbot correctly identified the includeFullDocs parameter bug — it's accepted in the schema but never passed to client.search() in packages/tools/src/ai-sdk.ts:54. I verified this: line 54 destructures { informationToGet, limit } but omits includeFullDocs, even though lines 33-42 define it in the input schema.

However, this bug exists in the base branch (feat/tools-seven-tool-parity / PR #1432), not in this PR. This PR correctly re-exports whatever the shared package provides. The fix belongs in #1432, not here.

The re-export itself is correct, the CI workflow changes are solid, and the publish workflow's retry loop for npm dependency availability is a sensible pattern.

Score: 9/10

Clean re-export with good test coverage and documentation updates. The one outstanding item is fixing the includeFullDocs bug in the base PR before merging this stack.

…xports

claude

Overview: This PR replaces inline tool implementations in @supermemory/ai-sdk with re-exports from @supermemory/tools/ai-sdk, expanding the surface from 2 tools to 7.

Issues found:

Cursor Bugbot correctly identified the includeFullDocs parameter bug — it's defined in the schema (lines 33-42 of packages/tools/src/ai-sdk.ts) but not passed to client.search() in the execute function (line 54). However, this bug exists in the base branch (feat/tools-seven-tool-parity), not in this PR. This PR correctly re-exports whatever the shared package provides. The fix belongs in #1432.

The re-export itself is correct, CI workflow changes properly detect changes in both packages, and the publish workflow's retry loop for npm dependency availability is a sensible pattern.

Score: 9/10

Clean re-export with good test coverage and documentation updates. No bugs introduced by this PR. The one item to track is fixing includeFullDocs in the base PR (#1432) before merging the stack.

feat(ai-sdk): re-export full Supermemory tool set from @supermemory/t…

9c3f84b

…ools Thin @supermemory/ai-sdk package now mirrors the canonical 7-tool surface with updated tests. Co-authored-by: Cursor <cursoragent@cursor.com>

Dhravya mentioned this pull request Aug 8, 2026

fix(python-sdks): v4 API migration for integration packages #1434

Open

1 task

graphite-app Bot assigned Dhravya Aug 8, 2026

cursor Bot reviewed Aug 8, 2026

View reviewed changes

graphite-app Bot reviewed Aug 8, 2026

View reviewed changes

Dhravya force-pushed the feat/ai-sdk-tool-reexports branch from acb8dc0 to 9c3f84b Compare August 8, 2026 02:47

This was referenced Aug 8, 2026

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

Open

feat(openai-sdk-python): 7-tool parity #1430

Open

chore(ci): Python SDK pytest workflow #1431

Open

This was referenced Aug 8, 2026

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

Open

docs(skills): refresh Supermemory skill for 7-tool parity #1435

Open

chore(ci): wire JS tools unit tests into CI #1436

Open

feat(sdk-playground): interactive SDK chat playground #1437

Open

claude Bot reviewed Aug 8, 2026

View reviewed changes

graphite-app Bot reviewed Aug 14, 2026

View reviewed changes

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

Dhravya force-pushed the feat/ai-sdk-tool-reexports branch from 84e2149 to 9c3f84b 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

Dhravya assigned ved015 Aug 18, 2026

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

github-actions Bot and others added 5 commits August 18, 2026 15:20

chore: update bun.lock

b69a28d

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

chore: merge latest stacked changes

8a1fae3

fix(ai-sdk): make seven-tool release publishable

e6ac260

docs(ai-sdk): clarify multi-step memory tools

5f9a417

chore: merge latest stacked tools fixes

ace3a76

chore: merge latest stacked tools fixes

a1a6209

claude Bot reviewed Aug 24, 2026

View reviewed changes

Merge branch 'feat/tools-seven-tool-parity' into feat/ai-sdk-tool-ree…

dbafb2d

…xports

ved015 approved these changes Aug 24, 2026

View reviewed changes

claude Bot reviewed Aug 24, 2026

View reviewed changes

This was referenced Aug 25, 2026

fix: deduplicate @supermemory/ai-sdk into @supermemory/tools/ai-sdk #1251

Closed

fix(tools): deduplicate @supermemory/ai-sdk into @supermemory/tools/a… #1256

Closed

feat(ai-sdk): re-export @supermemory/tools/ai-sdk #1402

Closed

Navigation Menu

Uh oh!

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

feat(ai-sdk): re-export 7-tool surface#1433
Dhravya wants to merge 8 commits into
feat/tools-seven-tool-parityfrom
feat/ai-sdk-tool-reexports

Dhravya commented Aug 8, 2026

Uh oh!

claude Bot commented Aug 8, 2026 •

edited

Loading

Uh oh!

cursor Bot left a comment

Uh oh!

cursor Bot Aug 8, 2026

Uh oh!

graphite-app Bot Aug 8, 2026

Uh oh!

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

edited

Loading

Uh oh!

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

edited

Loading

Uh oh!

Dhravya commented Aug 8, 2026 •

edited

Loading

Uh oh!

claude Bot left a comment

Uh oh!

Uh oh!

claude Bot left a comment

Uh oh!

claude Bot left a comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

		const testApiKey = process.env.SUPERMEMORY_API_KEY as string
		const testOpenAIKey = process.env.OPENAI_API_KEY as string

Uh oh!

Conversation

Dhravya commented Aug 8, 2026

Summary

Test plan

Uh oh!

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

Uh oh!

Review Complete

Uh oh!

cursor Bot left a comment

Choose a reason for hiding this comment

Uh oh!

cursor Bot Aug 8, 2026

Choose a reason for hiding this comment

Search ignores includeFullDocs

Uh oh!

graphite-app Bot Aug 8, 2026

Choose a reason for hiding this comment

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!

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!

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!

claude Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

claude Bot left a comment

Choose a reason for hiding this comment

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

claude Bot commented Aug 8, 2026 •

edited

Loading

cloudflare-workers-and-pages 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