-
Notifications
You must be signed in to change notification settings - Fork 1
Add bank deletion support #2
Copy link
Copy link
Open
Labels
area: memoryCore memory engineCore memory engineenhancementNew feature or requestNew feature or requestgood first issueGood for newcomersGood for newcomershelp wantedExtra attention is neededExtra attention is neededup-for-grabsListed on upforgrabs.netListed on upforgrabs.net
Description
Activity
Metadata
Metadata
Assignees
Labels
area: memoryCore memory engineCore memory engineenhancementNew feature or requestNew feature or requestgood first issueGood for newcomersGood for newcomershelp wantedExtra attention is neededExtra attention is neededup-for-grabsListed on upforgrabs.netListed on upforgrabs.net
Summary
Banks can be created with
createBank()and listed withlistBanks(), but there's no way to delete a bank and all its associated data.What needs to happen
deleteBank(bankId: string)method to theEngramclassMemoryStorefirst, then expose throughEngramFiles to look at
src/store/store.ts— MemoryStore class, see existingcreateBank()anddeleteFact()patternssrc/engram.ts— Engram orchestrator, exposes store methodssrc/store/schema.ts— table definitions to understand relationshipstests/unit/store.test.ts— add tests hereAcceptance criteria
engram.deleteBank('my-bank')removes all scoped data