chore(ci): Python SDK pytest workflow - #1431
Conversation
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| ✅ Deployment successful! View logs |
supermemory-mcp | 4ae703d | Sep 01 2026, 05:59 AM |
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
supermemory-app | 4ae703d | Commit Preview URL Branch Preview URL |
Sep 01 2026, 06:01 AM |
|
Claude finished @ved015's task in 1m 52s —— View job Code Review Complete
Summary: Reviewed 91 changed files covering CI workflow, Python SDK v4 migration, browser extension improvements, MCP authentication, and web app enhancements. No bugs or security issues found — the code is solid and ready to ship (9/10). |
![claude[bot]](https://avatars.githubusercontent.com/in/1236702?s=60&v=4){kind=small}
There was a problem hiding this comment.
Overview: Adds a GitHub Actions workflow to run pytest for four Python SDK packages, along with Python SDK v4 API migration, browser extension memory serialization improvements, MCP API key authentication, and various web app enhancements.
Issues found: None — this looks good to ship.
The code changes are well-structured:
- CI workflow properly tests across Python versions (3.9-3.13) with dependency matrices for minimum/locked/current SDK versions
- Python SDK correctly migrates to v4 API with proper deprecation warnings for
include_full_docs, addsadditionalProperties: falseto tool schemas for stricter validation, and implements document delete scope checking - MCP auth adds API key validation with reasonable cache limits (1000 entries, 60s TTL)
- Middleware properly scopes the
?view=bypass to/pathname only (security improvement) - Browser extension fixes memory serialization by using JSON instead of comma-delimited strings, preventing parsing issues with memories containing commas
Score: 9/10
Minor note: The PR title says "Python SDK pytest workflow" but includes substantial changes beyond CI. Consider updating the PR description to reflect the full scope for reviewers.
Merge activity
|
006a3f5 to
3bc0dd5
Compare
1160c46 to
348483d
Compare
Run pytest for openai-sdk-python, agent-framework, cartesia, and pipecat packages on PRs touching those paths. Co-authored-by: Cursor <cursoragent@cursor.com>
3bc0dd5 to
4ae703d
Compare
## 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](https://cursor.com)
2 participants
Summary
.github/workflows/ci-python.ymlto run pytest for the four Python SDK packagesStacked on #1430
Test plan
Made with Cursor