Skip to content

fix(openclaw-plugin): implement context engine ingest/compact contract#1032

Open
mobilebarn wants to merge 1 commit intovolcengine:mainfrom
mobilebarn:fix/context-engine-contract
Open

fix(openclaw-plugin): implement context engine ingest/compact contract#1032
mobilebarn wants to merge 1 commit intovolcengine:mainfrom
mobilebarn:fix/context-engine-contract

Conversation

@mobilebarn
Copy link
Copy Markdown

Summary

  • ingest() now writes user/assistant messages to the OV session via addSessionMessage instead of returning a no-op { ingested: false }
  • ingestBatch() delegates to ingest() for each message, accumulating the ingested count
  • compact() commits the OV session as a fallback when delegated compaction is unavailable, ensuring the memory extraction pipeline still runs even without runtime compaction

Test plan

  • Verify /compact command works when OpenViking is used as a context engine (previously returned no-op)
  • Verify messages are ingested into OV sessions during normal conversation flow
  • Verify batch ingestion correctly delegates to single-message ingest
  • Verify compact fallback commits the OV session and logs appropriately
  • Verify heartbeat and non-user/assistant messages are still skipped in ingest

Closes #833

- ingest() now writes messages to the OV session instead of no-op
- ingestBatch() delegates to ingest() for each message
- compact() commits the OV session as fallback when delegated
  compaction is unavailable, ensuring memory extraction still runs

Closes volcengine#833
@github-actions
Copy link
Copy Markdown

Failed to generate code suggestions for PR

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

Labels

None yet

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

[Question]: 安装启用openviking以后 传统的/compact命令就无法使用了吗?那我想要压缩上下文怎么办?

1 participant