Skip to content

docs: fix drift in Graph Viewer - #269

Draft
jack-arturo wants to merge 1 commit into
mainfrom
docs/audit-graph-viewer-overview-20260728
Draft

docs: fix drift in Graph Viewer#269
jack-arturo wants to merge 1 commit into
mainfrom
docs/audit-graph-viewer-overview-20260728

Conversation

@jack-arturo

Copy link
Copy Markdown
Member

Automated docs-accuracy audit of docs/graph-viewer/overview. Draft because the snapshot-cap fix rewrites an explanatory paragraph.

Claim Current state Fix Evidence
"arrow keys to traverse nodes, Enter to select, / to focus search, ? for shortcuts help" Neither Enter nor / is bound. The registered keys are ArrowUp/Down/Left/Right, Tab, Escape, p, r, R, ,, l, ? — and the app's own help panel adds Shift+arrows for Z-axis movement. Replaced the sentence with the real shortcut table, matching the in-app help automem-graph-viewer@c6338d1:src/hooks/useKeyboardNavigation.ts#L153-L258, #L281-L291
Force layout table lists 4 parameters: Charge strength, Link distance, Center gravity, Collision radius ForceConfig has 5 fields. linkStrength (0.1–1.0, default 0.5) was missing entirely, and the field is named centerStrength, not "center gravity". Added the missing parameter, corrected the name, and added the documented range/default columns src/lib/types.ts#L130-L136
"The overview loads the most important memories first (default 2,000 nodes)." 2,000 is right, but the framing is stale. The server's min(limit, 2000) cap was lifted (automem PR #141), so MAX_SNAPSHOT = 2000 is now a client-side limit set by the main-thread physics wall — and it is user-overridable via ?cap= or localStorage.automem_snapshot_cap, which the page never mentions. Rewrote the paragraph around the client-side ceiling and documented the escape hatch src/api/client.ts#L161-L191

Verified against: verygoodplugins/automem-graph-viewer@c6338d1
Verified against: verygoodplugins/automem@8ff266e (for the /viewer compatibility routes)

Checked and found correct, left untouched: the StatsBar "In view" / "In store" scope language, the four clustering modes (ClusterMode = 'type' | 'tags' | 'semantic' | 'entity' | 'none', with semantic = connected components over edges of strength ≥ 0.5), single-level expansion undo in useExpandableGraph, the SearchBar debounce + recent-search history, the /graph/neighbors expand path, "no environment variable required" for hand tracking, the npm run dev:all iPhone bridge, and every component and hook filename in the architecture tree.

One near-miss worth recording: I flagged "Referenced entities (people, places, etc.)" as wrong, since places isn't one of AutoMem's headline entity categories — then found automem/api/recall.py#L1355 iterating ("people", "places", "organizations"). It's correct; left alone.

Questions

  • The architecture tree omits the top-level src/types/ and src/utils/ directories (and main.tsx, index.css). Since it already abridges hooks/ with a ..., I read it as a curated overview rather than a literal listing and left it alone — but if it's meant to be complete, those two directories should be added.

Unverified

None — every code-anchored claim on this page resolved to current source.

Follow-ups

Beyond what's fixed here:

  1. "Nodes are colored and sized by memory type, importance, or custom attributes." The "custom attributes" clause has no obvious counterpart in palette.ts or the display settings; worth either substantiating or dropping.
  2. Semantic clustering's 0.5 strength threshold is undocumented. "Strongly-connected relationship clusters" is accurate but the concrete cutoff would be more useful.
  3. useTagCloud / TagCloud, useFilterChips, EdgeParticles, and the hand recording/playback hooks are shipped features with no mention on this page.

Generated by Claude Code

Verified against automem-graph-viewer@c6338d13.

- keyboard shortcut list documented Enter/slash bindings that do not exist
- force layout table omitted linkStrength and mislabeled centerStrength
- the 2,000-node ceiling is a client-side physics limit, not the server cap

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01XQCAKtAw2NRatRuYVEfDcv
@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying automem-website with  Cloudflare Pages  Cloudflare Pages

Latest commit: b091001
Status: ✅  Deploy successful!
Preview URL: https://6b364e05.automem-website.pages.dev
Branch Preview URL: https://docs-audit-graph-viewer-over-23lj.automem-website.pages.dev

View logs

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants