Skip to content

docs(openclaw-plugin): list openviking_tool_result_read/search/list in plugin tool tables (#2058)#2295

Merged
qin-ctx merged 2 commits into
volcengine:mainfrom
r266-tech:docs-openclaw-plugin-tool-result-tools
May 29, 2026
Merged

docs(openclaw-plugin): list openviking_tool_result_read/search/list in plugin tool tables (#2058)#2295
qin-ctx merged 2 commits into
volcengine:mainfrom
r266-tech:docs-openclaw-plugin-tool-result-tools

Conversation

@r266-tech
Copy link
Copy Markdown
Contributor

What

The examples/openclaw-plugin README tool tables list 8 tools, but openclaw.plugin.json registers 11 — the externalized-tool-result drilldown trio added in #2058 (openviking_tool_result_read / _search / _list) has never appeared in the tables. This adds the three missing rows to both README.md and README_CN.md so the canonical tool tables match the actual registered/exposed surface.

Grounding

  • examples/openclaw-plugin/openclaw.plugin.json registers all 11 tools, including openviking_tool_result_read/search/list.
  • examples/openclaw-plugin/index.ts registers the three as first-class tools with descriptions: read = "Restore the full original content of a tool result that was externalized by OpenViking" (L1606); search = "Search inside an externalized tool result for a keyword" (L1703); list = "List externalized tool results for the current session" (L1807).
  • The sibling drilldown tools ov_archive_search / ov_archive_expand are already in the table, so the three new rows follow the existing convention rather than introducing a new one.

Scope

Strictly the tool tables (the exhaustive canonical reference of registered tools). The curated "Tools and Expandability" prose subset is left untouched — it already omits some registered tools by design. Pure docs, dual-doc EN + zh.

@github-actions
Copy link
Copy Markdown

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

🎫 Ticket compliance analysis ✅

2058 - Fully compliant

Compliant requirements:

  • Added openviking_tool_result_read / openviking_tool_result_search / openviking_tool_result_list to README.md
  • Added the same three tools to README_CN.md
  • Tool tables now match the registered tool surface
⏱️ 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 156afc4 into volcengine:main May 29, 2026
1 check passed
@github-project-automation github-project-automation Bot moved this from Backlog to Done in OpenViking project May 29, 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