docs: Walrus Memory lifecycle and console guides (BEDU-904)#486
Open
reemsabawi-mystenlabs wants to merge 8 commits into
Open
docs: Walrus Memory lifecycle and console guides (BEDU-904)#486reemsabawi-mystenlabs wants to merge 8 commits into
reemsabawi-mystenlabs wants to merge 8 commits into
Conversation
Add the lifecycle/console pages under the Walrus Memory docs spine: - Tracking Agent-Owned Blobs and Storage (fundamentals/architecture): object model for agent-owned blobs, a blob-lifecycle state diagram, three ways to discover what an agent owns, and expiry/renewal tracking. - Manage your memory (guides): browse/search by meaning, namespace organization, epoch renewal, deletion paths, and index rebuild. - Sign in as a returning user (guides): first-sign-in delegate-key recovery, restore-based memory rediscovery, the re-upload (no migration) expectation, and onchain wallet-visibility caveats. Wire all three into docs.json navigation. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016DHZxHQuhv7oxhnTs598fo
Remove the Mermaid state diagram; the lifecycle is covered by the prose steps that follow. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016DHZxHQuhv7oxhnTs598fo
Collaborator
Style Guide AuditAll 3 file(s) pass the style guide audit. |
Style-guide word preference (may -> might) in the two lifecycle guides. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016DHZxHQuhv7oxhnTs598fo
Address review findings on the lifecycle/console guides: - restore().total is bounded by `limit` (default 10, newest-first) and is the number of blobs inspected in that call, not an authoritative count of the namespace. Reworded in tracking-agent-storage, manage-your-memory, and returning-user; point to an onchain query for an exact count. - The search index lives in the relayer's database (owner + namespace), not the browser. A returning user on a new device recalls existing memories as soon as they hold a delegate key; restore is for a lost/incomplete relayer index, not routine device switches. - Dashboard renewal is not shipped: frame epoch renewal as a storage-layer Blob extend today, with the one-click dashboard control noted as planned. Tie preview to the shipped delete panel and add the SDK recall path. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016DHZxHQuhv7oxhnTs598fo
Collaborator
Author
|
Thanks — all three fixed in
Generated by Claude Code |
Second round of review fixes: - restore has no pagination cursor: a repeat call at the same limit re-inspects the same newest blobs and returns nothing new. Reworded the three pages to say rerun with a progressively higher limit, and explain why. - tracking-agent-storage no longer presents renewal as a shipped dashboard control; the one-click control is marked planned, storage-layer extend is the path today. - manage-your-memory no longer describes a dashboard memory-list/preview/ delete panel as shipped; it leads with the SDK recall path and defers the dashboard delete/preview flow to the existing Delete old memories guide. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016DHZxHQuhv7oxhnTs598fo
Collaborator
Author
|
Fixed in
Generated by Claude Code |
jessiemongeon1
requested changes
Jul 22, 2026
- Apply reviewer suggestions verbatim (intro, Blob-object sentence, renew paragraph, heading renames). - Bold table headers; number the subheadings under "Three ways..."; number the delete-options list. - Remove the now auto-generated Related links footers from all three pages. - Drop "this page"/"this guide" preambles for AEO/GEO. - Rename headings to be concise (Renew memories, Delete memories, Rebuild the index, Browse and search) and clarify two returning-user headings. - Title-case the guide page titles; convert "Before you begin" to a Prerequisites section with a list. - Tighten wordy sentences. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016DHZxHQuhv7oxhnTs598fo
3 tasks
- returning-user: rename the confusing "What your wallet reveals onchain" heading to "Onchain privacy" (the actual line-74 comment). - manage-your-memory: remove the redundant "Search by meaning" subheading and simplify the confusing browse intro (line-25 comment). - Convert the clear "X is <verb>ed by Y" passives to active voice across all three pages, per the style guide. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016DHZxHQuhv7oxhnTs598fo
Apply the audit bot's suggested active rephrasings in manage-your-memory: "the memory is gone" -> "disappears", "might be missing rows" -> "might lack rows", "blobs already indexed" -> "blobs the relayer already indexed". Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016DHZxHQuhv7oxhnTs598fo
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
Adds the lifecycle/console pages of the Walrus Memory docs spine (BEDU-904) and wires them into navigation.
fundamentals/architecture/tracking-agent-storage.mdBlobobject, storage resource, rebuildable vector entry); three ways to discover what an agent owns (capture at write time, query onchain by owner + namespace viarestore, read theBlobobject); expiry/renewal tracking; ownership in sponsored writes.guides/manage-your-memory.mdrecall) and preview; namespace organization; epoch renewal; both deletion paths (cross-linking the existing delete guides); rebuilding a stale index withrestore.guides/returning-user.mdrestore-based rediscovery; the "you own your data, re-upload — no migration" expectation; onchain wallet-visibility caveats.Navigation (
docs/docs.json): the two guides join the Guides group; the tracking page joins Fundamentals → Architecture next tofunding-storage.Notes for reviewers
packages/sdk/src/(recall({query,limit}),restore→{restored,skipped,total},rememberBulkAndWait→results[].status/blob_id). The returning-user recovery flow matches the dashboard's actualisRecoveringExistingAccount/isNewAccountstates inapps/app.Blobextend, the existing delete guides) rather than inventing specific buttons. The guides target the MemWal dashboard (apps/app→ memory.walrus.xyz).docs.jsonvalidated and all internal links resolve.🤖 Generated with Claude Code
https://claude.ai/code/session_016DHZxHQuhv7oxhnTs598fo
Generated by Claude Code