Conversation
- Add two-paragraph technical pitch above the fold with 3-line quick start - Update tool count 37→38 (memory_verify), function count 50→52, test count 573 - Add memory_verify to MCP tools table, mem::verify and mem::cascade-update to functions table - Add citation provenance and cascading staleness to capabilities table - Add cross-agent sync and memory trust to comparison table - Update API endpoint count 93→95, version 0.6.0→0.6.1
📝 WalkthroughWalkthroughREADME.md documentation update that reorganizes navigation, adds detailed descriptions of memory persistence and cross-agent behavior, expands feature comparison tables with new capabilities (citation provenance, cascading staleness, cross-agent sync, memory trust), and increments version counts and inventory (MCP tools 37→38, API endpoints 93→95, functions 50→52, tests 551→573, health version 0.6.0→0.6.1). New MCP tool Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
README.md (1)
29-37: Optional copy polish: repeated sentence openings reduce readability.At Line 29 and Line 31, multiple consecutive sentences begin with the same word (“You” / “Your”). Consider light rewording for rhythm.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@README.md` around lines 29 - 37, The opening paragraph has several consecutive sentences starting with the same word (“You” and “Your” around the description of agentmemory and its benefits); rewrite those sentences in the README paragraph so the rhythm varies—e.g., convert one or two “You …” sentences into passive or subject-alternating forms and rephrase a “Your agents…” sentence to start with the benefit or feature (mentioning agentmemory, CLAUDE.md, .cursorrules, or the triple-stream retrieval) to break repetition while preserving meaning and metrics.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@README.md`:
- Line 623: The README has inconsistent endpoint counts: the header states "95
endpoints" while the "Supported Agents REST" section states "93 REST endpoints";
update the latter to match the actual count (use the same value as the header,
95) and verify the number against src/api.ts to ensure accuracy, then adjust the
text in the "Supported Agents REST" section (the phrase "93 REST endpoints") so
both places show the same endpoint count.
---
Nitpick comments:
In `@README.md`:
- Around line 29-37: The opening paragraph has several consecutive sentences
starting with the same word (“You” and “Your” around the description of
agentmemory and its benefits); rewrite those sentences in the README paragraph
so the rhythm varies—e.g., convert one or two “You …” sentences into passive or
subject-alternating forms and rephrase a “Your agents…” sentence to start with
the benefit or feature (mentioning agentmemory, CLAUDE.md, .cursorrules, or the
triple-stream retrieval) to break repetition while preserving meaning and
metrics.
| ## API | ||
|
|
||
| 93 endpoints on port `3111` (87 core + 6 MCP protocol). Protected endpoints require `Authorization: Bearer <secret>` when `AGENTMEMORY_SECRET` is set. The table below shows a representative subset; see `src/api.ts` for the full endpoint list. | ||
| 95 endpoints on port `3111` (89 core + 6 MCP protocol). Protected endpoints require `Authorization: Bearer <secret>` when `AGENTMEMORY_SECRET` is set. The table below shows a representative subset; see `src/api.ts` for the full endpoint list. |
There was a problem hiding this comment.
Endpoint count is still inconsistent across sections.
Line 623 says 95 endpoints, but the Supported Agents REST section still states 93 REST endpoints (Line 148). Please align both to avoid user confusion.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@README.md` at line 623, The README has inconsistent endpoint counts: the
header states "95 endpoints" while the "Supported Agents REST" section states
"93 REST endpoints"; update the latter to match the actual count (use the same
value as the header, 95) and verify the number against src/api.ts to ensure
accuracy, then adjust the text in the "Supported Agents REST" section (the
phrase "93 REST endpoints") so both places show the same endpoint count.
1 participant
Summary
Test plan
Summary by CodeRabbit
Documentation