hermes - 💡(How to fix) Fix Bug: memory tool replace/remove operations fail — old_text parameter not recognized

Bug Description

The memory tool's replace and remove actions fail because the old_text parameter is not correctly mapped to the underlying implementation. Only the add action works.

Steps to Reproduce

1. Use the memory tool with action=replace and provide old_text parameter
2. Observe the error: "old_text is required"
3. Try with old_string instead (matching the patch tool convention)
4. Observe the error: "content is required" (different error, still broken)
5. Try action=remove with old_string
6. Observe the error: "old_text is required"

Concrete Examples

Expected Behavior

replace and remove should work when old_text is provided, as defined in the tool schema.

Actual Behavior

Neither old_text nor old_string parameter names are correctly passed to the underlying handler. The parameter mapping appears broken — the schema defines old_text/new_text but the implementation may expect old_string/new_string (similar to the patch tool), or vice versa.

Root Cause Analysis

The tool schema declares parameters old_text and new_text, but the parameter names are not correctly forwarded to the backend logic. This is likely a parameter name mismatch between the tool schema definition and the handler function signature.

Workaround

Use action=add to write new entries. replace and remove are currently unusable.

Environment

• Hermes Agent: latest (installed via install.sh)
• Memory provider: built-in (default)
• Platform: Linux