Vercel AI SDK backfilling by sreedharsreeram · Pull Request #769 · supermemoryai/supermemory

sreedharsreeram · 2026-03-06T14:53:05Z

No description provided.

cloudflare-workers-and-pages · 2026-03-06T14:53:12Z

Deploying with 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	`579ef03`	Apr 14 2026, 03:21 PM

packages/tools/src/ai-sdk.ts

MaheshtheDev

@sreedharsreeram please update the package version and update the documentation for this packages with these new tools

sreedharsreeram · 2026-03-09T23:40:45Z

Vercel AI SDK backfilling #769 👈 (View in Graphite)
main

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.

sreedharsreeram · 2026-03-09T23:40:59Z

@sreedharsreeram please update the package version and update the documentation for this packages with these new tools

Done! Please review

MaheshtheDev · 2026-03-11T02:19:09Z

@sreedharsreeram please fix merge conflicts

packages/tools/src/vercel/index.ts

entelligence-ai-pr-reviews · 2026-04-01T01:35:04Z

Confidence Score: 2/5 - Changes Needed

Not safe to merge — the new withSupermemory signature in packages/tools/src/vercel/index.ts introduces a breaking API change by inserting conversationId as a required positional parameter between containerTag and options, which means all existing callers silently pass their options object as the conversationId string argument with no type error at the call site. This PR does deliver meaningful new functionality — hybrid and document-based RAG search modes, four new AI SDK tool factories, and updated documentation — but the backward-incompatible signature change without a major version bump or deprecation path is a correctness issue that will silently corrupt behavior for existing consumers.

Key Findings:

In packages/tools/src/vercel/index.ts, inserting conversationId as a required positional parameter between containerTag and options is a breaking change: existing callers using withSupermemory(model, containerTag, options) will silently pass their options object as conversationId, causing runtime misbehavior (wrong conversationId, options ignored) with no TypeScript error at the call site depending on overload resolution.
The new SearchMode type ('memories' | 'hybrid' | 'documents') and searchLimit addition to SupermemoryBaseOptions in types.ts appear well-scoped, but the routing logic in buildMemoriesText that switches on searchMode should be verified to handle undefined/default cases gracefully to avoid silent fallthrough.
The PR bumps to v1.5.0 (minor version), but introducing a positional parameter break in a public API warrants a major version bump under semver — shipping this as a minor version understates the compatibility impact for downstream consumers.

Files requiring special attention

packages/tools/src/vercel/index.ts
packages/tools/src/memory-client.ts
packages/tools/src/types.ts

packages/tools/src/vercel/index.ts

entelligence-ai-pr-reviews

🔍 Duplicate Code Detected

Found 2 duplicate function(s) in this PR. Consider consolidating to reduce code duplication.

packages/tools/src/ai-sdk.ts

claude · 2026-04-01T01:56:17Z

Claude finished @sreedharsreeram's task in 1m 36s —— View job

PR Review: Vercel AI SDK backfilling

Fetch and analyze the PR diff
Review key changed files for bugs/security issues
Check if previously flagged issues have been resolved
Submit review

Summary: This PR adds hybrid/documents search modes for RAG via a new /v4/search endpoint and refactors withSupermemory to use an options object pattern.

Verdict: Ready to ship (9/10)

All previously flagged issues have been addressed:

The ReferenceError bug with memoriesResponse scoping is fixed — rawSearchResults is now properly declared at function scope
The API change to an options object pattern is fully documented with updated README examples and tests
No security issues or logic bugs found

See my full review for details.

claude

Overview: Adds hybrid/documents search modes for RAG and makes conversationId a required parameter.

Issues found:

Critical: ReferenceError in memory-client.ts:329 — memoriesResponse is referenced outside its scope. When useHybridSearch is true (hybrid/documents mode), this will throw a ReferenceError at runtime since memoriesResponse is only defined in the else branch.
Documentation mismatch — All README examples use the old 3-argument signature withSupermemory(model, containerTag, options) but the API now requires 4 arguments with conversationId as required positional parameter.
Breaking change flagged by other reviewers — The withSupermemory signature change from (model, containerTag, options?) to (model, containerTag, conversationId, options?) is a breaking change that should warrant a major version bump (2.0.0), not minor (1.5.0).

Score: 5/10

The hybrid search feature looks well-designed, but the ReferenceError will crash any usage of hybrid/documents mode. This needs to be fixed before merge.

packages/tools/README.md

packages/tools/src/vercel/index.ts

entelligence-ai-pr-reviews · 2026-04-01T01:58:43Z

Confidence Score: 1/5 - Blocking Issues

Not safe to merge — the withSupermemory function signature change in packages/tools/src/vercel/index.ts introduces a silent, destructive breaking change: promoting conversationId to a required positional 3rd argument means all existing callers using the 3-arg form withSupermemory(model, tag, options) will silently pass their WrapVercelLanguageModelOptions object as the conversationId string parameter, corrupting the conversation ID with [object Object] and discarding all option overrides without any error or warning. The PR does add valuable SearchMode and searchLimit capabilities with a sensible hybrid/documents branch in buildMemoriesText, but the API breakage is critical enough to block merge. This same concern was flagged in a previous unresolved review comment and remains unaddressed, confirming the issue is known but not fixed.

Key Findings:

In packages/tools/src/vercel/index.ts, inserting conversationId: string as the 3rd positional argument between containerTag and options is a breaking API change: callers on the prior 3-arg signature will pass their options object where a string is expected, silently coercing it to [object Object] as the conversationId and losing all option configuration with no runtime error.
The breaking change is doubly dangerous because TypeScript may not catch it at call sites if conversationId can accept any or if the options object is typed loosely, meaning the corruption can reach production silently across all existing integrations.
A pre-existing unresolved review comment from a prior review cycle explicitly called out this same conversationId positional-argument breakage in withSupermemory, and the issue has not been resolved before this re-review, indicating the risk is acknowledged but the fix (e.g., keeping conversationId inside WrapVercelLanguageModelOptions or making it optional with a default) has not been applied.

Files requiring special attention

packages/tools/src/vercel/index.ts
packages/tools/src/memory-client.ts
packages/tools/src/types.ts

packages/tools/src/shared/memory-client.ts

packages/tools/src/vercel/index.ts

entelligence-ai-pr-reviews · 2026-04-01T02:01:16Z

Confidence Score: 1/5 - Blocking Issues

Not safe to merge — this PR introduces two critical runtime-breaking bugs that will cause immediate failures in production. In memory-client.ts, memoriesResponse is declared with const inside an else block, making it inaccessible in the subsequent promptData construction when the if (useHybridSearch && queryText) branch executes, which will throw a ReferenceError at runtime. Simultaneously, in vercel/index.ts, inserting conversationId as a new required positional parameter between containerTag and options silently breaks every existing caller of withSupermemory(model, containerTag, options) — their options object is now consumed as conversationId, causing type mismatches and incorrect behavior with no compile-time error for JavaScript callers. While the PR's goal of adding hybrid search capabilities and a SearchMode union type is a meaningful feature addition, these two defects must be resolved before any merge can be considered.

Key Findings:

In memory-client.ts, memoriesResponse declared with const inside the else block is out of scope when the hybrid search if branch runs, causing a guaranteed ReferenceError: memoriesResponse is not defined crash on any call that takes the hybrid search path.
In vercel/index.ts, adding conversationId as a positional parameter between containerTag and options is a breaking API change — all existing callers invoking withSupermemory(model, containerTag, options) will silently pass their options object as the conversationId string argument, leading to wrong behavior and hard-to-debug runtime errors.
Both bugs are in the core execution paths of the new feature, meaning the hybrid search functionality is effectively non-functional and the Vercel middleware refactor is backwards-incompatible as written.
4 previous unresolved comment(s) likely resolved in latest diff (score-only signal; thread status unchanged)

Files requiring special attention

packages/tools/src/shared/memory-client.ts
packages/tools/src/vercel/index.ts

packages/tools/src/shared/memory-client.ts

claude

Overview: Adds hybrid/documents search modes for RAG and makes conversationId a required parameter.

Issues found:

Critical: ReferenceError in memory-client.ts:335 — memoriesResponse is referenced outside its scope. When useHybridSearch is true (hybrid/documents mode), this will throw a ReferenceError at runtime since memoriesResponse is only defined in the else block at line 274.
Documentation mismatch — All README examples (lines 193, 220) use the signature without the now-required conversationId field. Since options is now required with conversationId: string, examples like withSupermemory(openai("gpt-4"), "user-123", { mode: "full" }) will fail TypeScript compilation.
Breaking change — Making options required and conversationId required within it is a breaking change. TypeScript will catch this at compile time (not a silent failure), but it warrants a major version bump (2.0.0) rather than minor (1.5.0).

Score: 4/10

The hybrid search feature is well-designed, but the ReferenceError will crash any usage of hybrid/documents mode. This must be fixed before merge.

packages/tools/src/shared/memory-client.ts

entelligence-ai-pr-reviews · 2026-04-01T02:18:47Z

Confidence Score: 1/5 - Blocking Issues

Not safe to merge — this PR contains a confirmed runtime ReferenceError in memory-client.ts where memoriesResponse is declared inside an else block but referenced at line 328 outside that block, meaning the hybrid search code path crashes unconditionally. Beyond this new critical bug, four unresolved correctness issues in packages/tools/src/vercel/index.ts document that promoting conversationId to a required positional parameter between containerTag and options constitutes a breaking API change that will silently break all existing callers without a major version bump. The PR does introduce genuinely useful functionality — hybrid search mode, configurable searchLimit, and SearchMode typing — but the scope of breakage makes this unshippable in its current state.

Key Findings:

memoriesResponse is scoped inside the else branch in buildMemoriesText (or equivalent function in memory-client.ts) but read unconditionally at line 328 after the if/else block, producing a ReferenceError at runtime on every hybrid search invocation — the core new feature of this PR is broken.
In packages/tools/src/vercel/index.ts, conversationId was removed from WrapVercelLanguageModelOptions and made a required positional 3rd argument to withSupermemory, breaking all existing callers using the previous 2-arg or options-bag form without a corresponding major version change to signal the breaking API contract.
The duplicate unresolved comments about withSupermemory's signature change indicate this breaking change concern was raised in a previous review cycle and was never resolved, suggesting the author has not addressed reviewer feedback on a correctness-critical issue.
Releasing as 1.5.0 (minor bump) is inconsistent with the breaking signature change to withSupermemory, which requires a major version bump under semver; downstream consumers relying on automated updates could have their builds broken silently.
2 previous unresolved comment(s) likely resolved in latest diff (score-only signal; thread status unchanged)

Files requiring special attention

packages/tools/src/shared/memory-client.ts
packages/tools/src/vercel/index.ts
packages/tools/src/shared/types.ts

entelligence-ai-pr-reviews · 2026-04-01T02:27:06Z

Confidence Score: 4/5 - Mostly Safe

Likely safe but review recommended — this PR introduces SearchMode typing and supermemoryHybridSearch in memory-client.ts with a clear branching strategy in buildMemoriesText, and the automated review found no flagged issues across the 6 reviewed files. The refactoring of conversationId to optional and the addition of formatSearchResults as a helper appear well-scoped, though 2 of 8 changed files were not reviewed, leaving a small blind spot. The PR is largely clean and achieves its stated goal of hybrid search support without introducing obvious regressions.

Key Findings:

No automated review comments were generated across 6 of 8 reviewed files, suggesting the core logic in memory-client.ts, types.ts, and the Vercel middleware refactor is structurally sound.
The branching in buildMemoriesText between profile-based and hybrid search paths is a meaningful control-flow change — while no bugs were flagged, this kind of conditional dispatch warrants human review to confirm edge cases (e.g., when searchMode defaults or is undefined) are handled correctly.
2 files out of 8 changed were not covered by the automated review, meaning there is incomplete visibility into the full scope of changes — this is a minor but real gap in confidence.
Making conversationId optional is a subtle interface contract change that could silently affect downstream consumers if they relied on it being required — worth confirming no callers are broken.

Files requiring special attention

memory-client.ts
types.ts

claude

Overview: Adds hybrid/documents search modes for RAG and makes conversationId required.

Good news: The ReferenceError bug flagged by earlier reviewers has been fixed in this version. The code now properly declares rawSearchResults at function scope (line 216), avoiding the scoping issue where memoriesResponse was referenced outside its else block.

Issues found:

Documentation examples missing required field — All new examples in README.md and user-profiles.mdx are missing the now-required conversationId field. Users following the docs will get TypeScript errors. See inline comments for suggested fixes.
Breaking API change — Making options required and conversationId required within it changes the public API signature from withSupermemory(model, tag, options?) to withSupermemory(model, tag, options) where options must include conversationId. This is technically a breaking change that would warrant a major version bump (2.0.0) under strict semver. However, TypeScript will catch this at compile time (not a silent failure), so the impact is manageable.

Score: 8/10

The hybrid search feature is well-implemented and the previous scoping bug has been fixed. The only blocking issues are the documentation examples that won't compile. Once those are updated, this is good to ship.

packages/tools/README.md

apps/docs/ai-sdk/user-profiles.mdx

entelligence-ai-pr-reviews · 2026-04-01T04:18:07Z

Confidence Score: 4/5 - Mostly Safe

Safe to merge — this PR cleanly introduces hybrid search (RAG) capability and conversation-scoped memory enforcement to @supermemory/tools v1.5.0 with no identified logic bugs, security concerns, or runtime issues. The new SearchMode type, supermemoryHybridSearch function, and SearchResultItem/SearchResponse interfaces appear to be well-scoped additions. No review comments were generated across the reviewed files, suggesting the implementation is structurally sound, though 4 of 10 changed files were not reviewed, leaving a small blind spot.

Key Findings:

supermemoryHybridSearch and the refactored buildMemoriesText in memory-client.ts introduce new code paths (hybrid/document search) that were not flagged for issues, suggesting the branching logic is straightforward and consistent with existing patterns.
The new SearchMode type ('memories' | 'hybrid' | 'documents') and searchLimit option added to SupermemoryBaseOptions in types.ts are additive and non-breaking changes, which reduces regression risk for existing consumers.
Coverage was 6/10 changed files — the 4 unreviewed files (likely documentation and/or test files) represent a minor blind spot but do not constitute a blocking concern given the nature of the PR.
No pre-existing unresolved comments were identified, meaning this PR does not compound any outstanding technical debt.

Files requiring special attention

memory-client.ts
types.ts

packages/tools/src/shared/memory-client.ts

claude

Overview: Adds hybrid/documents search modes for RAG via a new /v4/search endpoint, enabling search across both memories AND document chunks.

Issues found: None — this looks good to ship.

Clarifications on previous review concerns:

ReferenceError (FIXED): The scoping bug where memoriesResponse was referenced outside its else block has been properly fixed. The code now declares rawSearchResults at function scope (memory-client.ts:216-219), so it's always accessible regardless of which branch executes.
Breaking API change (NOT an issue): The withSupermemory signature remains (model, containerTag, options?) with conversationId staying optional inside WrapVercelLanguageModelOptions (vercel/index.ts:17). The only change is a new runtime validation: if addMemory: "always" is set, then conversationId is required (vercel/index.ts:131-138). This is defensive programming — TypeScript will catch it at compile time if you have proper types, and runtime throws a clear error message. This is not a breaking change for existing callers.
Documentation (CORRECT): The hybrid search examples correctly omit conversationId when not using addMemory: "always". Examples that do use addMemory: "always" now include conversationId.

What's well done:

Clean separation between hybrid search path and legacy profile search path in buildMemoriesText
Proper error handling in both supermemoryHybridSearch and supermemoryProfileSearch
Good TypeScript types for SearchMode and SearchResultItem
Clear JSDoc documentation throughout
Appropriate minor version bump (1.5.0) for additive, non-breaking features

Score: 9/10

entelligence-ai-pr-reviews · 2026-04-02T22:52:23Z

Confidence Score: 4/5 - Mostly Safe

Likely safe but review recommended — this PR introduces supermemoryHybridSearch, the SearchMode union type, and searchMode/searchLimit fields on SupermemoryBaseOptions cleanly, with no critical issues flagged by automated review. The new /v4/search endpoint integration appears straightforward, and the conversationId enforcement aligns with the stated v1.5.0 goals. However, only 6 of 10 changed files were reviewed, meaning portions of the implementation — potentially including the Vercel AI SDK adapter wiring and documentation updates — were not fully analyzed, leaving a residual gap in confidence.

Key Findings:

No logic bugs, null-pointer risks, or security issues were identified in the reviewed files; the SearchResultItem/SearchResponse interfaces provide typed boundaries around the /v4/search response, reducing runtime surprises.
The searchMode field defaults and searchLimit optionality were not flagged as problematic, but the absence of validation (e.g., guarding against invalid SearchMode strings at runtime if values come from external config) is worth a quick manual check.
4 of 10 changed files went unreviewed, including likely the Vercel AI SDK backfill adapter itself — the primary feature of this PR — meaning the automated analysis has limited visibility into the core integration path.
Documentation-only files (READMEs, changelogs) appear to be part of the diff; these carry no runtime risk but should be spot-checked to ensure version numbers and API signatures match the actual implementation.

Files requiring special attention

packages/tools/src/shared/memory-client.ts
packages/tools/src/shared/types.ts
packages/tools/src/vercel-ai/index.ts

Prefix unused variables with underscore and fix formatting in memory-client. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Break long conditionals across multiple lines and use single quotes for string containing escaped double quotes. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

claude

Overview: Adds hybrid/documents search modes for RAG via a new /v4/search endpoint, enabling search across both memories AND document chunks.

Issues found: None — this looks good to ship.

Clarifications on previous review concerns:

ReferenceError (FIXED): The scoping bug where memoriesResponse was referenced outside its else block has been properly fixed. The code now declares rawSearchResults at function scope (memory-client.ts:216-219), and it's correctly assigned in both the hybrid search branch (line 255) and the profile search branch (line 339).
Breaking API change (NOT an issue): The withSupermemory signature remains (model, containerTag, options?) with conversationId staying optional inside WrapVercelLanguageModelOptions (vercel/index.ts:17). The new runtime validation at lines 131-138 only throws if addMemory: "always" is set without conversationId — this is correct defensive programming, not a breaking change. Existing callers are unaffected.
Documentation: Updated correctly with new searchMode and searchLimit options. Examples that use addMemory: "always" now include conversationId.

What's well done:

Clean separation between hybrid search path and legacy profile search path in buildMemoriesText
Proper error handling in both supermemoryHybridSearch and supermemoryProfileSearch
Good TypeScript types for SearchMode and SearchResultItem
Clear JSDoc documentation throughout
Appropriate minor version bump (1.5.0) for additive, non-breaking features

Score: 9/10

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

claude

Overview: Adds hybrid/documents search modes for RAG via a new /v4/search endpoint and refactors the withSupermemory API to use an options object pattern.

Issues found:

Breaking API change without README update — The withSupermemory signature changed from (model, containerTag, options?) to ({ model, containerTag, customId, ...options }). The README still shows the old positional argument API, which will cause runtime errors for anyone following the docs.
addMemory default changed from "never" to "always" — This is a significant behavior change. Existing users who upgrade will suddenly have all conversations saved to memory without opting in. This should either be reverted to maintain backwards compatibility or explicitly documented as a breaking change.
customId is now required — Previously conversationId was optional. Now customId is required in the options object. This needs to be reflected in all documentation.

What's working well:

The ReferenceError bug flagged by earlier reviewers has been properly fixed (rawSearchResults is now declared at function scope)
Clean implementation of hybrid search with proper TypeScript types
Good test coverage for the new caching behavior

Score: 7/10

The hybrid search feature is well-implemented, but the undocumented breaking changes and the addMemory default flip need to be addressed before merge. Either update all README examples to match the new API, or consider a major version bump to signal the breaking changes.

packages/tools/src/vercel/index.ts

packages/tools/README.md

packages/tools/src/vercel/index.ts

socket-security · 2026-04-14T14:41:05Z

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

Diff	Package	Supply Chain Security	Vulnerability	Quality	Maintenance	License
	@mastra/core@1.24.1
	@ai-sdk/openai@2.0.102
	@anthropic-ai/sdk@0.65.0
	@ai-sdk/provider@3.0.8
	@ai-sdk/anthropic@2.0.74
	vitest@3.2.4
	@total-typescript/tsconfig@1.0.4
	zod@3.23.8 ⏵ 4.3.6	⁺³
	supermemory@3.14.0
	tsdown@0.14.2
	dotenv@16.6.1
	ai@5.0.173

View full report

claude

Overview: Adds hybrid/documents search modes for RAG via a new /v4/search endpoint and refactors withSupermemory to use an options object pattern.

Issues found: None — this looks good to ship.

Clarifications on previous review concerns:

ReferenceError (FIXED): The scoping bug where memoriesResponse was referenced outside its else block has been properly fixed. The code now declares rawSearchResults at function scope (memory-client.ts:216-219) and correctly assigns it in both the hybrid search branch (line 255) and the profile search branch (line 339).
Breaking API change (Documented): The withSupermemory signature changed from positional args (model, containerTag, options?) to an options object ({ model, containerTag, customId, ...options }). This is a breaking change, but:
- All tests have been updated to use the new pattern
- README documentation reflects the new API signature
- The options object pattern is cleaner and more extensible
addMemory default: The default is "always" as documented in the README under "Automatic Memory Capture". This is set explicitly in vercel/index.ts:155.

What's well done:

Clean separation between hybrid search path and legacy profile search in buildMemoriesText
Proper error handling with typed responses from supermemoryHybridSearch
Good TypeScript types for SearchMode, SearchResultItem, and SearchResponse
Per-turn memory caching to avoid redundant API calls during tool-call continuations
Comprehensive test coverage for the new options pattern

Score: 9/10

packages/tools/src/vercel/index.ts

claude

Overview: Adds hybrid/documents search modes for RAG via a new /v4/search endpoint and refactors withSupermemory to use an options object pattern with customId as a required field.

Issues found: None — this looks good to ship.

Clarifications on previous review concerns:

ReferenceError (FIXED): The scoping bug where memoriesResponse was referenced outside its else block has been properly fixed. The code now declares rawSearchResults at function scope (memory-client.ts:216-219) and correctly assigns it in both the hybrid search branch (line 255) and the profile search branch (line 339).
Breaking API change (Documented): The withSupermemory signature changed from positional args to an options object { model, containerTag, customId, ...options }. This is a breaking change, but:
- README documentation has been fully updated to show the new pattern
- All tests have been updated to use the new API
- Proper validation with clear error messages for missing customId
Documentation: README correctly shows customId as required and documents all new options including searchMode and searchLimit.

What's well done:

Clean separation between hybrid search path and legacy profile search in buildMemoriesText
Proper error handling with typed SearchResponse and SearchResultItem interfaces
Per-turn memory caching via MemoryCache to avoid redundant API calls during tool-call continuations
Comprehensive test coverage for the new options pattern

Score: 9/10

claude

Overview: Adds hybrid/documents search modes for RAG via a new /v4/search endpoint and refactors withSupermemory to use an options object pattern with customId as a required field.

Issues found: None — this looks good to ship.

Verification of previously flagged concerns:

ReferenceError (FIXED): The scoping bug where memoriesResponse was referenced outside its else block has been properly fixed. The code now declares rawSearchResults at function scope (memory-client.ts:216-219) and correctly assigns it in both the hybrid search branch (line 255) and the profile search branch (line 339). The promptData construction at line 345 uses this function-scoped variable.
Breaking API change (Documented): The withSupermemory signature changed from positional args (model, containerTag, options?) to an options object { model, containerTag, customId, ...options }. This is properly reflected in:
- README.md with all examples updated
- All integration tests using the new pattern
- docs-test files updated
- Proper validation with clear error messages for missing customId
Hybrid search implementation: Clean separation between hybrid search path (uses /v4/search) and legacy profile search path (uses /v4/profile) in buildMemoriesText. Error handling is consistent across both paths.

What's well done:

Per-turn memory caching via MemoryCache to avoid redundant API calls during tool-call continuations
Proper TypeScript types for SearchMode, SearchResultItem, and SearchResponse
Comprehensive test coverage for the new options pattern
Clear JSDoc documentation throughout

Score: 9/10

MaheshtheDev · 2026-04-14T20:27:04Z

packages/tools/package.json

    "@ai-sdk/anthropic": "^2.0.25",
    "@ai-sdk/openai": "^2.0.23",
    "ai": "^5.0.29",
-    "lru-cache": "^11.2.6",


@sreedharsreeram

may i ask why this is removed?

graphite-app bot requested a review from Dhravya March 6, 2026 14:53

graphite-app bot assigned sreedharsreeram Mar 6, 2026

sentry bot reviewed Mar 6, 2026

View reviewed changes

packages/tools/src/ai-sdk.ts Outdated Show resolved Hide resolved

sentry bot reviewed Mar 6, 2026

View reviewed changes

packages/tools/src/ai-sdk.ts Outdated Show resolved Hide resolved

sreedharsreeram requested a review from MaheshtheDev March 6, 2026 15:11

MaheshtheDev reviewed Mar 8, 2026

View reviewed changes

sreedharsreeram force-pushed the back-filling-vercel branch from 5eb9bde to 6bb8c68 Compare March 9, 2026 23:40

sreedharsreeram requested a review from MaheshtheDev March 10, 2026 23:28

sreedharsreeram force-pushed the back-filling-vercel branch from 6bb8c68 to 3bdf7d3 Compare April 1, 2026 01:32

entelligence-ai-pr-reviews bot reviewed Apr 1, 2026

View reviewed changes

packages/tools/src/vercel/index.ts Outdated Show resolved Hide resolved

sentry bot reviewed Apr 1, 2026

View reviewed changes

packages/tools/src/vercel/index.ts Outdated Show resolved Hide resolved

entelligence-ai-pr-reviews bot reviewed Apr 1, 2026

View reviewed changes

packages/tools/src/ai-sdk.ts Show resolved Hide resolved

packages/tools/src/ai-sdk.ts Show resolved Hide resolved

claude bot reviewed Apr 1, 2026

View reviewed changes

packages/tools/README.md Show resolved Hide resolved

entelligence-ai-pr-reviews bot reviewed Apr 1, 2026

View reviewed changes

packages/tools/src/vercel/index.ts Outdated Show resolved Hide resolved

packages/tools/src/vercel/index.ts Outdated Show resolved Hide resolved

entelligence-ai-pr-reviews bot reviewed Apr 1, 2026

View reviewed changes

packages/tools/src/shared/memory-client.ts Show resolved Hide resolved

packages/tools/src/vercel/index.ts Outdated Show resolved Hide resolved

sentry bot reviewed Apr 1, 2026

View reviewed changes

packages/tools/src/shared/memory-client.ts Show resolved Hide resolved

claude bot reviewed Apr 1, 2026

View reviewed changes

entelligence-ai-pr-reviews bot reviewed Apr 1, 2026

View reviewed changes

packages/tools/src/shared/memory-client.ts Show resolved Hide resolved

sentry bot reviewed Apr 1, 2026

View reviewed changes

packages/tools/src/shared/memory-client.ts Show resolved Hide resolved

claude bot reviewed Apr 1, 2026

View reviewed changes

packages/tools/README.md Show resolved Hide resolved

apps/docs/ai-sdk/user-profiles.mdx Show resolved Hide resolved

sentry bot reviewed Apr 1, 2026

View reviewed changes

packages/tools/src/shared/memory-client.ts Show resolved Hide resolved

packages/tools/src/shared/memory-client.ts Show resolved Hide resolved

claude bot reviewed Apr 2, 2026

View reviewed changes

sreedharsreeram and others added 11 commits April 13, 2026 17:56

vercel sdk updated

266d1d8

updated description

be240fb

fixed containerTags bug

4389e3c

fixed documentDelete bug

9155763

fix: resolve biome lint and format errors

d95d7d7

Prefix unused variables with underscore and fix formatting in memory-client. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

fixed bugs

3b29452

fixed bugs

a20929e

fix: format memory-client.ts for biome

c5a03f3

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

fixes

ba20b01

fix: format vercel/index.ts for biome CI

a03800b

Break long conditionals across multiple lines and use single quotes for string containing escaped double quotes. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

fixes

a0d469a

sreedharsreeram force-pushed the back-filling-vercel branch from 730a790 to a0d469a Compare April 14, 2026 00:57

claude bot reviewed Apr 14, 2026

View reviewed changes

updated ai sdk

147a549

sreedharsreeram force-pushed the back-filling-vercel branch from 324179c to 147a549 Compare April 14, 2026 01:47

fix: format files for biome CI

707361d

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

claude bot reviewed Apr 14, 2026

View reviewed changes

packages/tools/src/vercel/index.ts Show resolved Hide resolved

packages/tools/README.md Outdated Show resolved Hide resolved

sentry bot reviewed Apr 14, 2026

View reviewed changes

packages/tools/src/vercel/index.ts Show resolved Hide resolved

fix

ba91680

claude bot reviewed Apr 14, 2026

View reviewed changes

sentry bot reviewed Apr 14, 2026

View reviewed changes

packages/tools/src/vercel/index.ts Show resolved Hide resolved

fixed bug

579ef03

claude bot reviewed Apr 14, 2026

View reviewed changes

MaheshtheDev reviewed Apr 14, 2026

View reviewed changes

Conversation

sreedharsreeram commented Mar 6, 2026

Uh oh!

cloudflare-workers-and-pages bot commented Mar 6, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Deploying with Cloudflare Workers

Uh oh!

Uh oh!

Uh oh!

MaheshtheDev left a comment

Choose a reason for hiding this comment

Uh oh!

sreedharsreeram commented Mar 9, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

How to use the Graphite Merge Queue

Uh oh!

sreedharsreeram commented Mar 9, 2026

Uh oh!

MaheshtheDev commented Mar 11, 2026

Uh oh!

Uh oh!

entelligence-ai-pr-reviews bot commented Apr 1, 2026

Confidence Score: 2/5 - Changes Needed

Uh oh!

Uh oh!

entelligence-ai-pr-reviews bot left a comment

Choose a reason for hiding this comment

🔍 Duplicate Code Detected

Uh oh!

Uh oh!

Uh oh!

claude bot commented Apr 1, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

PR Review: Vercel AI SDK backfilling

Uh oh!

claude bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

entelligence-ai-pr-reviews bot commented Apr 1, 2026

Confidence Score: 1/5 - Blocking Issues

Uh oh!

Uh oh!

Uh oh!

entelligence-ai-pr-reviews bot commented Apr 1, 2026

Confidence Score: 1/5 - Blocking Issues

Uh oh!

Uh oh!

claude bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

entelligence-ai-pr-reviews bot commented Apr 1, 2026

Confidence Score: 1/5 - Blocking Issues

Uh oh!

entelligence-ai-pr-reviews bot commented Apr 1, 2026

Confidence Score: 4/5 - Mostly Safe

Uh oh!

claude bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

entelligence-ai-pr-reviews bot commented Apr 1, 2026

Confidence Score: 4/5 - Mostly Safe

Uh oh!

Uh oh!

Uh oh!

claude bot left a comment

Choose a reason for hiding this comment

Uh oh!

entelligence-ai-pr-reviews bot commented Apr 2, 2026

Confidence Score: 4/5 - Mostly Safe

Uh oh!

claude bot left a comment

Choose a reason for hiding this comment

Uh oh!

cloudflare-workers-and-pages bot commented Mar 6, 2026 •

edited

Loading

sreedharsreeram commented Mar 9, 2026 •

edited

Loading

claude bot commented Apr 1, 2026 •

edited

Loading

socket-security bot commented Apr 14, 2026 •

edited

Loading