Repository navigation
Memory hooks: a small schema gap, and an offer to help with a Guardian for them #546
vgudur-dev
started this conversation in
Ideas
Replies: 0 comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Hi all, and thank you for ACS. I work on agent memory security (I lead OWASP Agent Memory Guard), so I read the two memory hooks closely. Two things:
A small spec point.
docs/spec/instrument/hooks.mdlists TTL in thememoryStorepayload, butspecification/v0.1.0/hooks/memory-store.jsonhas no TTL or expiry property, only free-formmetadata. Would you like an issue or a small PR to align them? An explicit expiry field would help, because it lets a Guardian limit how long an entry may live.An offer. The AGT reference implementation evaluates
steps/toolCallRequestandsteps/toolCallResult. I'd be glad to contribute a small Python Guardian that evaluatessteps/memoryStoreandsteps/memoryContextRetrieval, using Agent Memory Guard as the policy engine, with a few conformance vectors for those two hooks. Examples: a write that looks harmless but changes later behaviour, and a stored entry that should be denied when it is read back.This sits next to #507 (memory lineage), which covers provenance; I'd follow whatever that settles.
I know the current priority is the runnable Guardian and the interoperability evidence, so please tell me whether this fits now or should wait for v0.2.0. Happy to follow whatever process you prefer.
All reactions