Description

delete_all(user_id=...) is meant to delete only memories matching the provided filters. However, after deleting individual memories it called self.vector_store.reset(), which drops the entire
collection/table — destroying ALL memories regardless of the filter.

The async version of delete_all() already omits this call, confirming it was unintentional in the sync path.

Fixes #3785

Type of change

• Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

• Unit Test

Added regression assertion vector_store.reset.assert_not_called() to the existing test_delete_all test. Both parametrizations (v1.0/v1.1) pass.

Checklist:

• My code follows the style guidelines of this project
• I have performed a self-review of my own code
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes

Maintainer Checklist

• closes Bug: memories table was dropped by this reset call #3785
• Made sure Checks passed