Skip to content

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

Merged
graphite-app[bot] merged 1 commit into
mainfrom
chore/ci-js-tools-unit-tests
Sep 1, 2026
Merged

chore(ci): wire JS tools unit tests into CI#1436
graphite-app[bot] merged 1 commit into
mainfrom
chore/ci-js-tools-unit-tests

Conversation

@Dhravya

@Dhravya Dhravya commented Aug 8, 2026

Copy link
Copy Markdown
Member

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

@claude

claude Bot commented Aug 8, 2026

Copy link
Copy Markdown

Claude finished @Dhravya's task in 58s —— View job


Reviewing PR #1436

  • Get PR diff and analyze changes
  • Check existing review comments
  • Verify CI workflow configuration
  • Submit review

Overview: Adds Memory Graph unit tests to the CI workflow.

Issues found: None — this looks good to ship.

Verification:

  • ✅ The test script exists in packages/memory-graph/package.jsonvitest run
  • ✅ The step is placed after Memory Graph type checking (logical ordering)
  • ✅ Memory-graph has no build dependencies on tools/ai-sdk packages
  • ✅ The existing Cursor Bugbot comment about "AI SDK tests skip tools build" refers to an earlier version — the current workflow correctly builds tools before AI SDK tests (lines 48-50 run before lines 56-58)

Score: 10/10

@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 7974498 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 7974498 Commit Preview URL

Branch Preview URL
Sep 01 2026, 06:20 AM

@cursor cursor 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.

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

Fix All in Cursor

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

Comment thread .github/workflows/ci.yml Outdated
run: |
bun run --cwd packages/memory-graph test
bun run --cwd packages/tools test:unit
bun run --cwd packages/ai-sdk test:unit

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 78269ff. Configure here.

@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: 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-graph has testvitest run
  • packages/tools has test:unit → runs specific unit test files
  • packages/ai-sdk has test:unitvitest run src/tools.unit.test.ts
  • ✅ turbo.json task configuration is correct (test depends on ^build, test:unit is standalone)

Score: 10/10

Dhravya commented Aug 8, 2026

Copy link
Copy Markdown
Member Author

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 chore/ci-js-tools-unit-tests branch from 0804596 to d51641c 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 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)
@graphite-app
graphite-app Bot force-pushed the chore/ci-js-tools-unit-tests branch from d51641c to 05c4fb6 Compare September 1, 2026 06:02
graphite-app Bot pushed a commit that referenced this pull request Sep 1, 2026
## Summary
- Add `apps/sdk-playground` — chat UI to test TS/Python SDK integrations
- Context panel with document memories, API keys in dashboard, tools reference tab
- Python FastAPI server on port 8792; portless entry in `portless.json`

Stacked on #1436

## Test plan
- [ ] `cd apps/sdk-playground && bun run check-types`
- [ ] `bun run dev` with Supermemory + OpenAI keys in UI
- [ ] Switch SDKs and verify chat + context panel

Made with [Cursor](https://cursor.com)
## 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 force-pushed the docs/supermemory-skill-1417 branch from 7fc2e2c to 4173edb Compare September 1, 2026 06:11
@graphite-app
graphite-app Bot force-pushed the chore/ci-js-tools-unit-tests branch from 05c4fb6 to 7974498 Compare September 1, 2026 06:11
graphite-app Bot pushed a commit that referenced this pull request Sep 1, 2026
## Summary
- Add `apps/sdk-playground` — chat UI to test TS/Python SDK integrations
- Context panel with document memories, API keys in dashboard, tools reference tab
- Python FastAPI server on port 8792; portless entry in `portless.json`

Stacked on #1436

## Test plan
- [ ] `cd apps/sdk-playground && bun run check-types`
- [ ] `bun run dev` with Supermemory + OpenAI keys in UI
- [ ] Switch SDKs and verify chat + context panel

Made with [Cursor](https://cursor.com)
@graphite-app
graphite-app Bot changed the base branch from docs/supermemory-skill-1417 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 7974498 into main Sep 1, 2026
5 of 7 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