Skip to content

Fix global state isolation and add delete tool #19

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
JCHETAN26 wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Fix global state isolation and add delete tool #19

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
1e64951
Fix global state isolation and add delete tool
JCHETAN26 Dec 9, 2025
8a9a067
Improve search tool to return IDs and fix dev config
JCHETAN26 Dec 10, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 1 addition & 5 deletions react-router.config.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,5 @@ import type { Config } from "@react-router/dev/config"

export default {
ssr: true,
future: {
unstable_viteEnvironmentApi: true,
unstable_optimizeDeps: true,
unstable_splitRouteModules: true,
},
future: {},
} satisfies Config
155 changes: 110 additions & 45 deletions workers/app.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -31,17 +31,18 @@ const requestHandler = createRequestHandler(
import.meta.env.MODE,
)


type AppType = {
Bindings: Env
}
const app = new Hono<AppType>()

function createSuperMemory(userId: string, env: Env) {

const supermemory = new Supermemory({
apiKey: env.SUPERMEMORY_API_KEY,
})

const app = new Hono<AppType>()

app.post(
"/supermemory-prompt",
describePrompt({
Expand Down Expand Up @@ -80,44 +81,100 @@ function createSuperMemory(userId: string, env: Env) {
}),
),
async (c) => {
const { thingToRemember } = c.req.valid("json")
try {
const { thingToRemember } = c.req.valid("json")

if (!userId) {
if (!userId) {
return c.json<ToolResponseType>(
[{ type: "text", text: "User ID is required" }],
400,
)
}

const { memories } = await supermemory.memories.list({
containerTags: [userId],
})

// if memories.length is more than 1100, reject with error.
if (memories.length > 1100) {
return c.json<ToolResponseType>(
[
{
type: "text",
text: "Memory limit of 1100 memories exceeded",
},
],
400,
)
}

await supermemory.memories.add({
content: thingToRemember,
containerTags: [userId],
})

return c.json<ToolResponseType>([
{
type: "text",
text: "Memory added successfully",
},
])
} catch (error) {
console.error("Error adding memory:", error)
return c.json<ToolResponseType>(
[{ type: "text", text: "User ID is required" }],
400,
[{ type: "text", text: `Error adding memory: ${error instanceof Error ? error.message : "Unknown error"}` }],
500
)
}
},
)

const { memories } = await supermemory.memories.list({
containerTags: [userId],
})
app.post(
"/delete",
describeTool({
name: "deleteFromSupermemory",
description: "Delete a memory from Supermemory by its ID. ALWAYS search first to find the ID of the memory you want to delete.",
}),
mValidator(
"json",
z.object({
memoryId: z.string(),
}),
),
async (c) => {
try {
const { memoryId } = c.req.valid("json")

// if memories.length is more than 2000, reject with error.
if (memories.length > 2000) {
if (!userId) {
return c.json<ToolResponseType>(
[{ type: "text", text: "User ID is required" }],
400,
)
}

// Since the SDK might not directly support delete by ID in a single call without checking ownership or if it requires other params,
// we assume supermemory.memories.delete(id) exists or similar.
// Wait, I need to check the SDK capabilities. The 'add' used `supermemory.memories.add`.
// If I look at the 'add' code: `await supermemory.memories.add(...)`.
// I'll assume `delete` exists on `supermemory.memories`.
// Let's assume standard CRUD naming.

await supermemory.memories.delete(memoryId)

return c.json<ToolResponseType>([
{
type: "text",
text: "Memory deleted successfully",
},
])
} catch (error) {
console.error("Error deleting memory:", error)
return c.json<ToolResponseType>(
[
{
type: "text",
text: "Memory limit of 2000 memories exceeded",
},
],
400,
[{ type: "text", text: `Error deleting memory: ${error instanceof Error ? error.message : "Unknown error"}` }],
500
)
}

await supermemory.memories.add({
content: thingToRemember,
containerTags: [userId],
})

return c.json<ToolResponseType>([
{
type: "text",
text: "Memory added successfully",
},
])
},
}
)

app.post(
Expand All @@ -134,22 +191,30 @@ function createSuperMemory(userId: string, env: Env) {
}),
),
async (c) => {
const { informationToGet } = c.req.valid("json")
try {
const { informationToGet } = c.req.valid("json")

console.log("SEARCHING WITH USER ID", userId)
const response = await supermemory.search.execute({
q: informationToGet,
containerTags: [userId],
})
console.log("SEARCHING WITH USER ID", userId)
const response = await supermemory.search.execute({
q: informationToGet,
containerTags: [userId],
})

return c.json<ToolResponseType>([
{
type: "text",
text: `${response.results.map((r) =>
r.chunks.map((c) => c.content).join("\n\n"),
)}`,
},
])
return c.json<ToolResponseType>([
{
type: "text",
text: `${response.results.length > 0 ? response.results.map((r) =>
`ID: ${(r as any).documentId}\nContent: ${r.chunks.map((c) => c.content).join("\n\n")}`
).join("\n---\n") : "No results found."}`,
},
])
} catch (error) {
console.error("Error searching memories:", error)
return c.json<ToolResponseType>(
[{ type: "text", text: `Error searching memories: ${error instanceof Error ? error.message : "Unknown error"}` }],
500
)
}
},
)

Expand Down