Skip to content

feat: declare vouch-context engine contract#249

Open
claytonlin1110 wants to merge 2 commits into
vouchdev:testfrom
claytonlin1110:feat/vouch-context-engine
Open

feat: declare vouch-context engine contract#249
claytonlin1110 wants to merge 2 commits into
vouchdev:testfrom
claytonlin1110:feat/vouch-context-engine

Conversation

@claytonlin1110

@claytonlin1110 claytonlin1110 commented Jun 17, 2026

Copy link
Copy Markdown

What changed

Adds a vouch-context OpenClaw context engine: Python implementation under src/vouch/openclaw/, a JS plugin entry at adapters/openclaw/vouch-context-engine.mjs, vouch openclaw-rpc CLI bridge, contracts.contextEngines: ["vouch-context"] in openclaw.plugin.json, and context_engines on kb.capabilities.

Why

OpenClaw deployments need a declared context-engine contract so prompts can route through vouch and get back a cited synthesis with salience and hot memory already woven in — the natural counterpart to gbrain's gbrain-context engine.

Closes #228.

What might break

No breaking changes for existing .vouch/ directories or kb.* callers.

  • No on-disk layout changes.
  • No existing kb.* method behaviour changes.
  • kb.capabilities gains an additive context_engines field; clients that ignore unknown fields are unaffected.
  • OpenClaw users who enable the slot must set plugins.slots.contextEngine: "vouch-context" explicitly; default remains the legacy engine.

VEP

Not required — this is an OpenClaw plugin surface and additive capabilities metadata, not a change to the AKBP object model or review-gate protocol.

Tests

  • make check passes locally (lint + mypy + pytest)
  • New / changed behaviour has a test (tests/test_openclaw_context_engine.py, tests/test_openclaw_plugin_manifest.py)
  • CHANGELOG.md updated under ## [Unreleased]

@coderabbitai

coderabbitai Bot commented Jun 17, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: b31b64a4-19da-4e96-a9de-d2abac7bef86

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@plind-junior

Copy link
Copy Markdown
Collaborator

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 91efdfc58f

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/vouch/openclaw/rpc.py
Comment thread adapters/openclaw/vouch-context-engine.mjs
Comment thread src/vouch/openclaw/context_engine.py
Comment thread adapters/openclaw/vouch-context-engine.mjs
Comment thread openclaw.plugin.json
Comment thread openclaw.plugin.json
Comment thread src/vouch/openclaw/context_engine.py
@claytonlin1110 claytonlin1110 force-pushed the feat/vouch-context-engine branch from 91efdfc to 0081245 Compare June 17, 2026 10:51
@claytonlin1110

Copy link
Copy Markdown
Author

@plind-junior please review

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants