Skip to content

fix(openclaw-plugin): extend default Phase 2 commit wait timeout#1415

Merged
qin-ctx merged 1 commit intovolcengine:mainfrom
yeyitech:fix/issue-1405-compact-timeout
Apr 14, 2026
Merged

fix(openclaw-plugin): extend default Phase 2 commit wait timeout#1415
qin-ctx merged 1 commit intovolcengine:mainfrom
yeyitech:fix/issue-1405-compact-timeout

Conversation

@yeyitech
Copy link
Copy Markdown
Contributor

@yeyitech yeyitech commented Apr 13, 2026

Summary

  • increase the default client-side Phase 2 commit wait timeout from 120s to 300s for commitSession(wait=true)
  • reuse the same 300s default for the plugin's background Phase 2 polling logs
  • add a regression test proving slow Phase 2 completion after ~200s no longer gets reported as timeout

Testing

  • npm test -- --run tests/ut/client.test.ts tests/ut/context-engine-compact.test.ts
  • npm test

Closes #1405

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Apr 13, 2026

CLA assistant check
All committers have signed the CLA.

@github-actions
Copy link
Copy Markdown

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

🎫 Ticket compliance analysis ✅

1405 - Fully compliant

Compliant requirements:

  • Default Phase 2 commit wait timeout increased from 120s to 300s in both client poll and background polling
  • Added regression test for slow Phase 2 completion
⏱️ 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

PR Code Suggestions ✨

No code suggestions found for the PR.

@qin-ctx qin-ctx merged commit 054c998 into volcengine:main Apr 14, 2026
2 checks passed
@github-project-automation github-project-automation bot moved this from Backlog to Done in OpenViking project Apr 14, 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.

[Bug]: 执行compact压缩时,默认的超时时间太短,导致压缩超时直接失败。

3 participants