Skip to content

fix(python): community fixes for the openai, cartesia and pipecat wrappers - #1799

Open
MaheshtheDev wants to merge 14 commits into
mainfrom
mtd/python-community-fixes
Open

MaheshtheDev wants to merge 14 commits into
mainfrom
mtd/python-community-fixes

Conversation

@MaheshtheDev

@MaheshtheDev MaheshtheDev commented Oct 12, 2026 •

Copy link
Copy Markdown
Member

Bundles 10 community fixes for the Python wrappers, each commit kept with its original author. Ships supermemory-openai-sdk 1.0.10, supermemory-cartesia 0.1.5 and supermemory-pipecat 0.1.5.

Co-Authored-By: abhinav7x94 204053250+abhinav7x94@users.noreply.github.com
Co-Authored-By: shamAnimates 145093437+shamAnimates@users.noreply.github.com
Co-Authored-By: aloktripathi1 142301030+aloktripathi1@users.noreply.github.com
Co-Authored-By: Cintu07 178455858+Cintu07@users.noreply.github.com
Co-Authored-By: Atharvsinh-codez 201381760+Atharvsinh-codez@users.noreply.github.com
Co-Authored-By: abhay-codes07 182421137+abhay-codes07@users.noreply.github.com
Co-Authored-By: nkootstra 545768+nkootstra@users.noreply.github.com
Co-Authored-By: Agnik47 140933190+Agnik47@users.noreply.github.com

@mintlify

mintlify Bot commented Oct 12, 2026 •

Copy link
Copy Markdown
Contributor

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated
supermemory 🟢 Ready View Preview Oct 12, 2026, 11:06 PM

💡 Tip: Enable Automations to automatically generate PRs for you.

@MaheshtheDev MaheshtheDev changed the title fix(openai-sdk-python): isolate middleware clients (#1503) fix(python): community fixes for the openai, cartesia and pipecat wrappers Oct 12, 2026

Copy link
Copy Markdown
Member Author

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.

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Oct 12, 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 a1a0ea6 Oct 13 2026, 02:26 AM

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Oct 12, 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 a1a0ea6 Commit Preview URL

Branch Preview URL
Oct 13 2026, 02:27 AM

@polylane polylane Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested observability improvements for code this pull request changes. They are advisory and never affect the production-impact verdict.

To apply several suggestions in one commit, open Files changed and select Add suggestion to batch.

Comment thread packages/cartesia-sdk-python/src/supermemory_cartesia/agent.py
@polylane

polylane Bot commented Oct 12, 2026 •

Copy link
Copy Markdown
Contributor

Note

Production impact unlikely.

The new commit a1a0ea6 only trims comments and docstrings, and every changed Python file is AST-identical to the previous head, so no runtime path changed. No deployed resource loads these client SDKs, and the two flagged Workers logged 0 errors over 72h.

View the full analysis →

Also considered · 3 refuted
  • Refuted · OpenAI middleware facade rework degrades a deployed consumer · No consumer exists in the workspace, and both flagged Workers are TypeScript with zero error rows over 72h (14,169 requests for supermemory-app, 2,825,465 for supermemory-mcp).
  • Refuted · Moving the supermemory <5 cap onto the dependency breaks installs · The constraint binds only external pip installers, and it matches the two sibling wrappers already on main.
  • Refuted · The final commit trims comments but changes runtime behaviour · The executable syntax tree is identical between the previous head and this head for every changed Python file once docstrings are stripped, so no runtime path differs.
Analysed against 2 Cloudflare Workers and 1 repository

View in Polylane Disable reviews

Polylane analysed a1a0ea6 for production impact. You can ask follow-ups by mentioning @polylane in a comment.

Rate this review: 👍 helpful · 👎 not helpful · 😕 confusing

Previous verdicts (3)
Head Verdict Analysis
b5ae2c7 Production impact unlikely analysis
111bedb Production impact unlikely analysis
20afa0a Production impact unlikely analysis

The SDK stream() helper bypassed the memory middleware; it now runs through the wrapped create. The <5 cap from #1774 was on a keyword line in cartesia and pipecat, so it now sits on the real dependency.
@socket-security

socket-security Bot commented Oct 13, 2026 •

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addedpypi/​supermemory@​3.62.0100100100100100

View full report

@polylane polylane Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested observability improvements for code this pull request changes. They are advisory and never affect the production-impact verdict.

To apply several suggestions in one commit, open Files changed and select Add suggestion to batch.

Comment thread packages/cartesia-sdk-python/src/supermemory_cartesia/agent.py Outdated

@capy-ai capy-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Re-reviewed the bounded fix delta. The official sync and async chat.completions.stream helpers now retain memory injection, sync persistence, native stream manager/response closing, and shared-client tenant isolation in real-SDK mock-transport tests. Fresh wheel/sdist checks and declared Supermemory <5 dependency resolution pass.

@capy-ai capy-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Reviewed the follow-up delta from the tested 111bedb head: only the Cartesia guarded-error log text changes; OpenAI/Pipecat source and dependency fixes are unchanged. Native Cartesia tests and the changed guard/output check pass. Renewing approval for this exact head.

@polylane polylane Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested observability improvements for code this pull request changes. They are advisory and never affect the production-impact verdict.

To apply several suggestions in one commit, open Files changed and select Add suggestion to batch.

Comment thread packages/cartesia-sdk-python/src/supermemory_cartesia/agent.py

@capy-ai capy-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Reviewed the final comment/docstring cleanup delta. Executable ASTs match the approved parent across all 14 changed Python files; tests and dependency metadata are unchanged in behavior. Positive real-SDK sync/async stream, tenant-isolation and native export controls pass on this exact source.

This branch was successfully deployed

1 active (outdated) deployment
staging - apps/docs — 20afa0a2 Deployed Oct 12, 2026 by mintlify[bot]
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.

8 participants