Skip to content

docs(benchmark/locomo): add mem0/supermemory/claudecode to README directory tree#2005

Merged
qin-ctx merged 1 commit into
volcengine:mainfrom
r266-tech:docs-locomo-readme-add-3-missing-subdirs
May 13, 2026
Merged

docs(benchmark/locomo): add mem0/supermemory/claudecode to README directory tree#2005
qin-ctx merged 1 commit into
volcengine:mainfrom
r266-tech:docs-locomo-readme-add-3-missing-subdirs

Conversation

@r266-tech
Copy link
Copy Markdown
Contributor

Summary

The top-level benchmark/locomo/README.md directory tree under ## 目录结构 enumerates only vikingbot/ + openclaw/, but the directory actually contains 5 active subdirs. The other three accumulated as merged drift:

The top README opens with "本目录包含 LoCoMo(Long-Term Conversation Memory)评测脚本" (i.e., describes the whole benchmark/locomo/ directory), so listing 2/5 actively-maintained eval paths is a real discoverability gap for users browsing the dir from the repo root.

Scope of this patch

Minimal additive — 3 new tree entries plus the box-drawing prefix flip on the openclaw/ block (└──/ → ├──/│ ) now that openclaw/ is no longer the last child. Each new entry is a one-liner pointing readers at the subdir's own README:

├── mem0/               # mem0 评测脚本(详见 mem0/README.md)
├── supermemory/        # Supermemory 评测脚本(详见 supermemory/README.md)
└── claudecode/         # Claude Code 评测脚本(详见 claudecode/README.md)

The subdir READMEs (mem0 4.7 KB, supermemory 5.5 KB, claudecode 5.3 KB) remain authoritative for per-mode prerequisites / ingest / eval / stats — this patch does not duplicate that content into the top README. Only the missing tree entries are added to restore the canonical entry point for directory browsing.

Note on in-flight PRs

Two open PRs would extend this directory further; both touch only files inside their own new subdirs (no conflict with this README change):

If either lands, a follow-up 1-line tree update will be needed. This PR only adds the 3 entries whose source PRs are already merged on main.

Test plan

  • Render check: the box-drawing tree remains a valid 5-entry single tree, code fence boundaries unchanged, no other section touched.
  • No internal anchors / cross-references in the rest of the file rely on openclaw/ being the last tree entry (grep confirms none).

@github-actions
Copy link
Copy Markdown

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🏅 Score: 100
🧪 No relevant tests
🔒 No security concerns identified
✅ No TODO sections
🔀 No multiple PR themes
⚡ No major issues detected

@github-actions
Copy link
Copy Markdown

PR Code Suggestions ✨

No code suggestions found for the PR.

@qin-ctx qin-ctx merged commit b71b9a0 into volcengine:main May 13, 2026
1 check passed
@github-project-automation github-project-automation Bot moved this from Backlog to Done in OpenViking project May 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants