feat: batch ops, stale UX, review_days default#9
Merged
Conversation
There was a problem hiding this comment.
Pull request overview
This PR enhances the AI Memory Protocol’s maintenance/triage workflow by increasing the default review cadence, adding batch operations for deprecations, and improving stale-memory review UX across both the CLI and MCP server.
Changes:
- Increase default
review_daysfrom 30 → 90 (RST directive generation, CLI defaults, MCP tool schema defaults). - Add batch deprecation support (
ids/--ids) and MCP-side optional auto-rebuild (rebuild). - Extend stale-memory listing with optional body previews and bulk renewal of
review_afterdates.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 11 comments.
| File | Description |
|---|---|
src/ai_memory_protocol/rst.py |
Updates the default review window used when generating new memory directives. |
src/ai_memory_protocol/mcp_server.py |
Updates MCP schemas/defaults and adds batch deprecate + stale UX/renew handler logic. |
src/ai_memory_protocol/cli.py |
Adds CLI batch deprecate and stale UX/renew flags; updates default review-days. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…lidate empty ids, conditional rebuild message
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
rebuildoption tomemory_deprecateMCP tool (matchesmemory_addbehavior)idsparam (comma-separated) for deprecating multiple memories in one callmemory_stale --body: show truncated body preview (first 200 chars) inline during triagememory_stale --renew N: bulk-renew review_after dates on all stale memories by N daysTest plan
memory stale --bodyshows body previewsmemory stale --renew 90renews datesmemory deprecate --ids ID1,ID2batch deprecates