Skip to content

test: cover before_reset without context engine#1318

Closed
haosenwang1018 wants to merge 4 commits intovolcengine:mainfrom
haosenwang1018:test/cover-before-reset-without-context-engine
Closed

test: cover before_reset without context engine#1318
haosenwang1018 wants to merge 4 commits intovolcengine:mainfrom
haosenwang1018:test/cover-before-reset-without-context-engine

Conversation

@haosenwang1018
Copy link
Copy Markdown
Contributor

Summary

  • add a focused plugin test for before_reset when the OpenViking context-engine was registered but never instantiated
  • verify that reset remains a no-op in that case instead of attempting a commit through a missing engine reference
  • strengthen the hook/context-engine compatibility coverage around the recent OpenClaw integration fixes

Testing

  • git diff --check

@CLAassistant
Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 9, 2026

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

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

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 9, 2026

PR Code Suggestions ✨

No code suggestions found for the PR.

@qin-ctx
Copy link
Copy Markdown
Collaborator

qin-ctx commented Apr 10, 2026

pr有重复哈,先关闭了

@qin-ctx qin-ctx closed this Apr 10, 2026
@github-project-automation github-project-automation bot moved this from Backlog to Done in OpenViking project Apr 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants