Skip to content

docs(openclaw): refresh OpenViking operator guidance#2570

Open
Mijamind719 wants to merge 1 commit into
codex/openclaw-recall-target-typesfrom
codex/openclaw-docs-skill-refresh
Open

docs(openclaw): refresh OpenViking operator guidance#2570
Mijamind719 wants to merge 1 commit into
codex/openclaw-recall-target-typesfrom
codex/openclaw-docs-skill-refresh

Conversation

@Mijamind719

Copy link
Copy Markdown
Collaborator

Summary

Stacked PR 5/5 extracted from #2386, based on #2569.

This refreshes user/operator guidance after the smaller runtime PRs:

  • adds a focused openviking-context-database operator skill
  • exposes the skill through openclaw.plugin.json
  • includes the skill in install-manifest.json optional files
  • updates README/README_CN for recall trace, recall target types, and default-hidden add_resource

This intentionally avoids importing the large #2386 documentation bundle wholesale.

Validation

  • npm test -- --run tests/ut/manifest-contracts.test.ts tests/ut/setup-command.test.ts tests/ut/config.test.ts
  • npm run typecheck
  • npm test
  • npm run build

@github-actions

Copy link
Copy Markdown

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

🎫 Ticket compliance analysis 🔶

2569 - Partially compliant

Compliant requirements:

  • Update plugin manifest config hints

Non-compliant requirements:

  • Add configurable recall target types (runtime changes not in this PR)
  • Keep deprecated recallResources (runtime changes not in this PR)
  • Update auto-recall and default memory_recall (runtime changes not in this PR)
  • Allow memory_recall to override targets with resourceTypes (runtime changes not in this PR)
  • Add --recall-target-types to non-interactive setup (runtime changes not in this PR)
  • Update tests (not in this PR)

2386 - Partially compliant

Compliant requirements:

  • Add/update docs (README, README_CN, new operator skill)
  • Update plugin meta information (openclaw.plugin.json, install-manifest.json)

Non-compliant requirements:

  • Sync runtime code (not in this PR)
  • Add/update tests (not in this PR)
  • Exclude TOS release logic (not applicable here)
⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🏅 Score: 95
🧪 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.

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

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

1 participant