Skip to content

feat: add non-propagate option for memory patch#386

Merged
evan-ak merged 3 commits intomainfrom
feat/patch_propagation
Mar 17, 2026
Merged

feat: add non-propagate option for memory patch#386
evan-ak merged 3 commits intomainfrom
feat/patch_propagation

Conversation

@evan-ak
Copy link
Copy Markdown
Collaborator

@evan-ak evan-ak commented Mar 17, 2026

📝 Pull Request Summary

Please provide a short summary explaining the purpose of this PR.


✅ What does this PR do?

  • Adds a propagate: bool = True parameter to create_memory_item, update_memory_item, and delete_memory_item in both PatchMixin and CRUDMixin.
  • When propagate=False, the memory item is created/updated/deleted as usual, but category summary re-generation is skipped.

🤔 Why is this change needed?

  • In some workflows (e.g. bulk imports or migrations), patching category summaries on every individual memory write is unnecessary and expensive. This option lets callers opt out of propagation when they know category summaries will be rebuilt separately or don't need updating.

🔍 Type of Change

Please check what applies:

  • Bug fix
  • New feature
  • Documentation update
  • Refactor / cleanup
  • Other (please explain)

✅ PR Quality Checklist

  • PR title follows the conventional format (feat:, fix:, docs:)
  • Changes are limited in scope and easy to review
  • Documentation updated where applicable
  • No breaking changes (or clearly documented)
  • Related issues or discussions linked

📌 Optional

  • Screenshots or examples added (if applicable)
  • Edge cases considered
  • Follow-up tasks mentioned

@evan-ak evan-ak merged commit 3b67458 into main Mar 17, 2026
3 checks passed
@evan-ak evan-ak deleted the feat/patch_propagation branch March 17, 2026 07:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant