Skip to content

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

Merged
graphite-app[bot] merged 1 commit into
mainfrom
docs/supermemory-skill-1417
Sep 1, 2026
Merged

docs(skills): refresh Supermemory skill for 7-tool parity#1435
graphite-app[bot] merged 1 commit into
mainfrom
docs/supermemory-skill-1417

Conversation

@Dhravya

@Dhravya Dhravya commented Aug 8, 2026

Copy link
Copy Markdown
Member

Summary

  • Update SKILL.md with proactive search and full 7-tool surface
  • Refresh sdk-guide.md with v4 API examples and tool descriptions

Stacked on #1434

Test plan

  • Review skill content for accuracy

Made with Cursor

@cloudflare-workers-and-pages

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

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  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 4173edb Sep 01 2026, 06:18 AM

@cloudflare-workers-and-pages

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

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  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 4173edb Commit Preview URL

Branch Preview URL
Sep 01 2026, 06:25 AM

@claude

claude Bot commented Aug 8, 2026

Copy link
Copy Markdown

Claude finished @Dhravya's task in 1m 28s —— View job


PR Review Complete

  • Fetch PR diff and existing review comments
  • Review documentation changes for accuracy
  • Check code examples for bugs/security issues
  • Submit review summary

Overview: Documentation update that refreshes Supermemory skill docs for v4 API parity, including the 7-tool surface area, middleware patterns, and updated code examples.

Issues found: None — this looks good to ship.

Score: 10/10

I verified the documentation against the actual implementation in packages/tools/src/ai-sdk.ts and packages/tools/src/vercel/index.ts:

  • All 7 tool names (searchMemories, addMemory, getProfile, documentList, documentAdd, documentDelete, memoryForget) match the implementation
  • The supermemoryTools(apiKey, { containerTags: [...] }) signature is correct
  • The withSupermemory middleware export and usage pattern is accurate
  • Code examples use environment variables for API keys (good security practice)
  • v4 API changes (queryq, updated response shapes, filter syntax) are documented correctly

The documentation is clear, accurate, and provides valuable guidance on when to use tools vs middleware patterns, and how to handle deletion operations properly.

Dhravya commented Aug 8, 2026

Copy link
Copy Markdown
Member Author

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overview: Documentation update that adds comprehensive guidance for Supermemory's 7-tool surface area, including agent tools vs middleware patterns, deletion mechanisms, and SDK examples.

Issues found: None — this looks good to ship.

Score: 10/10

The documentation is clear, accurate, and follows existing patterns. Examples use environment variables properly for API keys, tool naming conventions are consistent between TypeScript (camelCase) and Python (snake_case), and the new sections provide valuable guidance on when to use each tool.

ved015
ved015 previously approved these changes Aug 24, 2026
@graphite-app

graphite-app Bot commented Sep 1, 2026

Copy link
Copy Markdown

Merge activity

@Dhravya
Dhravya force-pushed the fix/python-sdks-v4-api branch from 5462e20 to 96354fa Compare September 1, 2026 05:58
@Dhravya
Dhravya force-pushed the docs/supermemory-skill-1417 branch from ed58861 to 754617b Compare September 1, 2026 05:58
@graphite-app
graphite-app Bot force-pushed the fix/python-sdks-v4-api branch from 96354fa to 6462215 Compare September 1, 2026 06:01
graphite-app Bot pushed a commit that referenced this pull request Sep 1, 2026
## Summary
- Update `SKILL.md` with proactive search and full 7-tool surface
- Refresh `sdk-guide.md` with v4 API examples and tool descriptions

Stacked on #1434

## Test plan
- [ ] Review skill content for accuracy

Made with [Cursor](https://cursor.com)
@graphite-app
graphite-app Bot force-pushed the docs/supermemory-skill-1417 branch from 754617b to 7fc2e2c Compare September 1, 2026 06:01
graphite-app Bot pushed a commit that referenced this pull request Sep 1, 2026
## Summary
- Add tools/ai-sdk unit test jobs to `ci.yml`
- Update `turbo.json` and root `package.json` scripts
- Refresh `bun.lock`

Stacked on #1435

## Test plan
- [ ] CI passes on this branch

Made with [Cursor](https://cursor.com)
## Summary
- Update `SKILL.md` with proactive search and full 7-tool surface
- Refresh `sdk-guide.md` with v4 API examples and tool descriptions

Stacked on #1434

## Test plan
- [ ] Review skill content for accuracy

Made with [Cursor](https://cursor.com)
@graphite-app
graphite-app Bot force-pushed the fix/python-sdks-v4-api branch from 6462215 to c262cc9 Compare September 1, 2026 06:10
@graphite-app
graphite-app Bot force-pushed the docs/supermemory-skill-1417 branch from 7fc2e2c to 4173edb Compare September 1, 2026 06:11
graphite-app Bot pushed a commit that referenced this pull request Sep 1, 2026
## Summary
- Add tools/ai-sdk unit test jobs to `ci.yml`
- Update `turbo.json` and root `package.json` scripts
- Refresh `bun.lock`

Stacked on #1435

## Test plan
- [ ] CI passes on this branch

Made with [Cursor](https://cursor.com)
@graphite-app
graphite-app Bot changed the base branch from fix/python-sdks-v4-api to main September 1, 2026 06:16
@graphite-app
graphite-app Bot dismissed ved015’s stale review September 1, 2026 06:16

The base branch was changed.

@graphite-app
graphite-app Bot merged commit 4173edb into main Sep 1, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants