docs(managed_agents): link Memory cookbook to Agent Memory docs#587
docs(managed_agents): link Memory cookbook to Agent Memory docs#587gaganb-ant wants to merge 1 commit intomainfrom
Conversation
…ies links - Hyperlink the Memory intro to the Agent Memory docs page - Update the Memory stores API reference link to the platform.claude.com URL - Remove the 'Other notebooks in this series' section from the summary
Notebook ChangesThis PR modifies the following notebooks: 📓
|
There was a problem hiding this comment.
PR Review
Recommendation: REQUEST_CHANGES
Summary
Small documentation polish on CMA_remember_user_preferences.ipynb: adds an inline hyperlink on "Memory" in the intro, updates the Learn More link to platform.claude.com, and removes the "Other notebooks in this series" block. The intent is good, but the link migration is incomplete, leaving the notebook with three different base URLs for the same Memory docs page.
Actionable Feedback (2 items)
-
CMA_remember_user_preferences.ipynb(in cell withclient.beta.memory_stores.memory_versions.list(...), line 575) — The inline link[Memory API reference](https://docs.anthropic.com/en/docs/managed-agents/memory)was not updated. After this PR the same page is referenced via bothdocs.anthropic.comandplatform.claude.comwithin the same file. Update this link tohttps://platform.claude.com/docs/en/managed-agents/memoryto match the other two Memory links. -
CMA_remember_user_preferences.ipynb(in "Learn more" section, line 625) — The anchor link[Session resources](https://docs.anthropic.com/en/docs/managed-agents/memory#attach-a-memory-store-to-a-session)still usesdocs.anthropic.com. Consider updating the base toplatform.claude.comfor consistency (i.e.https://platform.claude.com/docs/en/managed-agents/memory#attach-a-memory-store-to-a-session).
Detailed Review
Link consistency
After this PR there are four references to Memory docs in the file, split across two different base URLs:
| Line | URL |
|---|---|
| 12 (intro, added) | platform.claude.com/docs/en/managed-agents/memory |
| 575 (inline in Audit cell, unchanged) | docs.anthropic.com/en/docs/managed-agents/memory |
| 624 (Learn More, updated) | platform.claude.com/docs/en/managed-agents/memory |
| 625 (Learn More anchor, unchanged) | docs.anthropic.com/en/docs/managed-agents/memory#... |
A partial migration is worse than either consistent state. The fix is to update lines 575 and 625 to use platform.claude.com as well.
Removal of "Other notebooks in this series"
Neither of the two removed cross-links (to CMA_iterate_fix_failing_tests.ipynb and CMA_operate_in_production.ipynb) linked back to this notebook, so removing them does not strand any reader arriving via a back-link. The removal is defensible and simplifies the conclusion section.
Intro hyperlink
Adding [Memory](https://platform.claude.com/docs/en/managed-agents/memory) in the intro is a useful improvement — it gives readers an immediate path to the reference docs without scrolling to Learn More.
Positive Notes
- The core notebook content, code, and explanations are untouched and remain high quality.
- The rationale for the changes is clearly articulated in the PR description.
Small polish pass on the Memory cookbook (
CMA_remember_user_preferences.ipynb):platform.claude.comURL.Generated by Claude Code