fix: Multi Params issue with @supermemory/tools#854
fix: Multi Params issue with @supermemory/tools#854MaheshtheDev wants to merge 5 commits intomainfrom
Conversation
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| 🔵 In progress View logs |
supermemory-mcp | 66ce119 | Apr 15 2026, 10:37 PM |
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
supermemory-app | b6a53ae | Commit Preview URL Branch Preview URL |
Apr 15 2026, 10:49 PM |
How to use the Graphite Merge QueueAdd 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. |
|
Claude finished @MaheshtheDev's task in 1m 5s —— View job PR Review CompleteOverview: Fixes proxy/wrapper issue by switching from object spread to ES6 Proxy pattern to preserve prototype-chain properties on wrapped language models. Issues found: None — this looks good to ship. Details:
Score: 9/10 Minor note: The test app changes (switching from OpenAI to Gemini gateway, adding PostHog tracing) are unrelated to the proxy fix. Not a blocker, but worth noting for commit message clarity. |
The lockfile was out of sync with package.json dependencies, causing CI to fail with --frozen-lockfile flag. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub. |
|
Warning Review the following alerts detected in dependencies. According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.
|
There was a problem hiding this comment.
Overview: Fixes proxy/wrapper issue by switching from object spread to ES6 Proxy pattern to preserve prototype-chain properties on wrapped language models.
Issues found: None — this looks good to ship.
Details:
- The core change in
packages/tools/src/vercel/index.tscorrectly solves the problem where{ ...model }would drop prototype/getter properties likeprovider,modelId, andspecificationVersion. TheProxywithReflect.get(target, prop, receiver)properly delegates to the prototype chain. - Unit test added to verify the fix works for prototype-based properties.
- Test app files cleaned up to use package imports (
@supermemory/tools/*) instead of relative paths. - Version bump to 1.4.02 is appropriate for this fix.
Score: 9/10
Minor note: The test app changes (switching from OpenAI to Gemini gateway, adding PostHog tracing) are unrelated to the proxy fix. Not a blocker, but worth noting for commit message clarity.

testing for now
fix proxy issue
upgraded the package