fix(tools): ship 12 community fixes as 2.4.0

apps/docs/integrations/claude-memory.mdx

@@ -183,7 +183,7 @@ Paths are normalized for storage: `/memories/preferences` is stored as `--memori
 ```typescript
 {
   command: "rename",
-  path: "/memories/old-name.txt",
+  old_path: "/memories/old-name.txt",
   new_path: "/memories/new-name.txt"
 }
 ```

apps/mcp/src/server/client/index.test.ts

@@ -0,0 +1,66 @@
+import { afterEach, describe, expect, it, vi } from "vitest"
+import { SupermemoryClient } from "./index"
+
+const API_URL = "https://api.example.com"
+
+describe("SupermemoryClient.getDocuments", () => {
+	afterEach(() => {
+		vi.restoreAllMocks()
+		vi.unstubAllGlobals()
+	})
+
+	function stubFetch() {
+		const fetchMock = vi.fn().mockResolvedValue(
+			Response.json({
+				documents: [],
+				pagination: {
+					currentPage: 1,
+					limit: 200,
+					totalItems: 0,
+					totalPages: 0,
+				},
+			}),
+		)
+		vi.stubGlobal("fetch", fetchMock)
+		return fetchMock
+	}
+
+	it("cancels through a caller-provided signal", async () => {
+		const fetchMock = stubFetch()
+		const controller = new AbortController()
+
+		await new SupermemoryClient("sm_test_key", "user_1", API_URL).getDocuments(
+			["user_1"],
+			1,
+			200,
+			{ signal: controller.signal },
+		)
+
+		const [, init] = fetchMock.mock.calls[0] as [string, RequestInit]
+		controller.abort()
+		expect(init.signal?.aborted).toBe(true)
+	})
+
+	it("keeps the timeout when a caller-provided signal is present", async () => {
+		const timeoutController = new AbortController()
+		const timeoutSpy = vi
+			.spyOn(AbortSignal, "timeout")
+			.mockReturnValue(timeoutController.signal)
+		const fetchMock = stubFetch()
+
+		await new SupermemoryClient("sm_test_key", "user_1", API_URL).getDocuments(
+			["user_1"],
+			1,
+			200,
+			{ signal: new AbortController().signal },
+		)
+
+		expect(timeoutSpy).toHaveBeenCalledWith(30_000)
+
+		const [, init] = fetchMock.mock.calls[0] as [string, RequestInit]
+		// Firing only the timeout leg aborts the request: a caller signal adds
+		// cancellation, it does not remove the 30s bound.
+		timeoutController.abort()
+		expect(init.signal?.aborted).toBe(true)
+	})
+})

apps/mcp/src/server/client/index.ts

@@ -339,7 +339,14 @@ export class SupermemoryClient {
 		options?: { signal?: AbortSignal },
 	): Promise<DocumentsApiResponse> {
 		try {
-			const signal = options?.signal ?? AbortSignal.timeout(FETCH_TIMEOUT_MS)
+			// Compose rather than choose: a caller-supplied signal must add
+			// cancellation on top of the timeout, not replace it.
+			const signal = options?.signal
+				? AbortSignal.any([
+						options.signal,
+						AbortSignal.timeout(FETCH_TIMEOUT_MS),
+					])
+				: AbortSignal.timeout(FETCH_TIMEOUT_MS)
 			const response = await fetch(`${this.apiUrl}/v3/documents/documents`, {
 				method: "POST",
 				headers: {

packages/tools/README.md

@@ -263,7 +263,7 @@ The `MemoryPromptData` object provides:
 
 #### OpenAI Middleware with Supermemory
 
-The `withSupermemory` function creates an OpenAI client with SuperMemory middleware automatically injected:
+The `withSupermemory` function creates an isolated middleware facade around an OpenAI client. It does not mutate the supplied client, so a shared base client can safely be wrapped with different user or conversation options:
 
 ```typescript
 import { withSupermemory } from "@supermemory/tools/openai"

packages/tools/package.json

@@ -1,14 +1,14 @@
 {
   "name": "@supermemory/tools",
   "type": "module",
-  "version": "2.3.0",
+  "version": "2.4.0",
   "description": "Memory tools for AI SDK, OpenAI, Voltagent and Mastra with supermemory",
   "scripts": {
     "build": "tsdown",
     "dev": "tsdown --watch --ignore-watch .turbo",
     "check-types": "tsc --noEmit",
     "test": "vitest --testTimeout 100000",
-    "test:unit": "vitest run --testTimeout 100000 src/tools-shared.test.ts src/tool-operations.test.ts src/claude-memory.test.ts test/with-supermemory/unit.test.ts test/with-supermemory/conversation-conversion.test.ts test/openai-middleware.unit.test.ts test/mastra/unit.test.ts test/voltagent.unit.test.ts",
+    "test:unit": "vitest run --testTimeout 100000 src/tools-shared.test.ts src/tool-operations.test.ts src/claude-memory.test.ts src/shared/memory-client.test.ts test/with-supermemory/unit.test.ts test/with-supermemory/conversation-conversion.test.ts test/openai-middleware.unit.test.ts test/mastra/unit.test.ts test/voltagent.unit.test.ts",
     "test:watch": "vitest --watch --testTimeout 100000"
   },
   "dependencies": {

packages/tools/src/ai-sdk.ts

@@ -10,6 +10,7 @@ import {
 	clampSearchLimit,
 	deleteDocumentByIdentifier,
 	getContainerTags,
+	resolveConfiguredContainerTag,
 } from "./tools-shared"
 import { forgetMemoryRequest } from "./shared/forget-memory"
 import type { SupermemoryToolsConfig } from "./types"
@@ -146,7 +147,7 @@ export const getProfileTool = (
 		}),
 		execute: async ({ containerTag, query }) => {
 			try {
-				const tag = containerTag || containerTags[0]
+				const tag = resolveConfiguredContainerTag(containerTags, containerTag)
 
 				const response = await client.profile({
 					containerTag: tag,
@@ -199,7 +200,7 @@ export const documentListTool = (
 		execute: async ({ containerTag, limit, page }) => {
 			try {
 				const scopeTags: [string, ...string[]] = containerTag
-					? [containerTag]
+					? [resolveConfiguredContainerTag(containerTags, containerTag)]
 					: containerTags
 
 				const response = await client.documents.list({
@@ -339,12 +340,12 @@ export const memoryForgetTool = (
 					}
 				}
 
-				const tag = containerTag || containerTags[0]
+				const tag = resolveConfiguredContainerTag(containerTags, containerTag)
 
 				await forgetMemoryRequest(
 					apiKey,
 					{
-						containerTag: tag as string,
+						containerTag: tag,
 						...(memoryId && { id: memoryId }),
 						...(memoryContent && { content: memoryContent }),
 						...(reason && { reason }),