chore(ci): wire JS tools unit tests into CI - #1436
Conversation
Wire @supermemory/tools and @supermemory/ai-sdk unit tests into CI and update workspace turbo config and lockfile. Co-authored-by: Cursor <cursoragent@cursor.com>
Reviewing PR #1436
Overview: Adds unit test jobs for JS SDK packages (memory-graph, tools, ai-sdk) to CI workflow with corresponding turbo.json task definitions. Issues found: Cursor Bugbot's finding is valid. The export { supermemoryTools, ... } from "@supermemory/tools/ai-sdk"This maps to Fix options:
- name: Build tools package
run: bun run --cwd packages/tools build
- name: Run JS SDK unit tests
run: |
bun run --cwd packages/memory-graph test
bun run --cwd packages/tools test:unit
bun run --cwd packages/ai-sdk test:unit
- name: Run JS SDK unit tests
run: bunx turbo run test:unit --filter='@supermemory/memory-graph' --filter='@supermemory/tools' --filter='@supermemory/ai-sdk'Score: 8/10 — Real issue that will cause CI failures on clean checkout, but straightforward to fix. |
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| ✅ Deployment successful! View logs |
supermemory-mcp | 05c0566 | Aug 08 2026, 02:51 AM |
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| ❌ Deployment failed View logs |
supermemory-app | 78269ff | Aug 08 2026, 02:48 AM |
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
There was a problem hiding this comment.
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 78269ff. Configure here.
| run: | | ||
| bun run --cwd packages/memory-graph test | ||
| bun run --cwd packages/tools test:unit | ||
| bun run --cwd packages/ai-sdk test:unit |
There was a problem hiding this comment.
AI SDK tests skip tools build
Medium Severity
The new CI step runs packages/ai-sdk test:unit without building @supermemory/tools first. That test loads ./tools, which re-exports from @supermemory/tools/ai-sdk via exports to dist/, and dist/ is not in the repo, so a clean checkout is likely to fail when resolving those modules.
Reviewed by Cursor Bugbot for commit 78269ff. Configure here.
There was a problem hiding this comment.
Overview: Adds unit test jobs for JS SDK packages (memory-graph, tools, ai-sdk) to CI workflow with corresponding turbo.json task definitions.
Issues found: None — this looks good to ship.
Verification:
- ✅ All referenced test scripts exist in their respective package.json files
- ✅
packages/memory-graphhastest→vitest run - ✅
packages/toolshastest:unit→ runs specific unit test files - ✅
packages/ai-sdkhastest:unit→vitest run src/tools.unit.test.ts - ✅ turbo.json task configuration is correct (
testdepends on^build,test:unitis standalone)
Score: 10/10
d8216c1 to
78269ff
Compare
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>



Summary
ci.ymlturbo.jsonand rootpackage.jsonscriptsbun.lockStacked on #1435
Test plan
Made with Cursor