Conversation
Remove mem0 MCP integration entirely from the project: - Delete Curator agent (curator.md) and all mem0 tool references - Clean all 8 agent templates, 7 command files, docs, skills, tests - Remove ACE learning system references from plugin metadata - Delete obsolete files (tier3_integration_report.json, apply_tier3_deltas.py, playbook.json.backup, requirements-semantic.txt) - Update __init__.py to remove curator generator and mem0 allowlist Simplify map-review to always use thorough mode: - Remove BIG/SMALL mode selection menu - Always present top 4 issues per section Fix hook errors causing noise during agent work: - block-secrets.py: add SAFE_PATH_PREFIXES to stop blocking .claude/ framework files - safety-guardrails.py: narrow "token"/"password" patterns to file extensions only, add .claude/ dirs to safe prefixes, fix missing execute permission - settings.local.json: remove duplicate workflow-context-injector.py (matcher "*", timeout 1s) and duplicate end-of-turn.sh Stop hook All templates synced via make sync-templates. 635 passed, 1 skipped, 0 failures.
Curator agent was removed in the fixes branch (f9a6f3f) but the template files were left behind. Clean up both .claude/ and src/mapify_cli/templates/ copies.
|
Caution Review failedThe pull request is closed. ℹ️ Recent review infoConfiguration used: defaults Review profile: CHILL Plan: Pro 📒 Files selected for processing (88)
📝 WalkthroughWalkthroughThis pull request removes the ACE Learning System and Curator agent from the MAP Framework, reducing the agent roster from 12 to 11. The changes eliminate all mem0 MCP integration, including pattern storage, tiered search, and deduplication workflows. The MEM0_SEARCH step is renamed to CONTEXT_SEARCH, and documentation is updated throughout to reflect the simplified architecture. Changes
Estimated code review effort🎯 4 (Complex) | ⏱️ ~75 minutes The changes span 90+ files with extensive scope, but the nature of edits is highly repetitive and consistent (systematic removal of mem0 references, Curator agent, and related workflows across the codebase). While individually each file change is straightforward, the breadth of the change set and potential for overlooked references in cascading systems demands careful verification of completeness and cross-file consistency. Possibly related PRs
Poem
✨ Finishing Touches
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
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
1 participant
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
.claude/agents/curator.mdandsrc/mapify_cli/templates/agents/curator.mdf9a6f3f(fixes branch)insert_tier3_bullets.sql,curator_operations_retry.json)Test plan
Summary by CodeRabbit
New Features
Changes
Removed