Improve external context engine plugin support#8416
Conversation
|
Stephen just confirmed on X that hermes-lcm should be treated as the standalone deliverable/plugin:
Given that, I think the right framing for this PR is:
So from my side, this PR is still intended as host-support cleanup:
In other words: |
|
Pulled one more small host-support commit into this PR. What it adds
Why In particular, it helps close the gap we hit while working on external Validation
Tagging for visibility: @stephenschoettler @teknium1 |
|
Quick correction: I accidentally left vendored repo-shipped LCM files in this branch. That's now removed. Current scope is back to generic host support only:
No bundled plugins/context_engine/lcm payload remains in this PR. Validation rerun after cleanup:
Sorry for the noise. @stephenschoettler @teknium1 |
8f4dd82 to
b5a3f9b
Compare
|
Rebased this cleanly onto current What stayed true after the rewrite:
Local validation on the rewritten branch:
|
|
Follow-up pushed to this PR to close the biggest remaining host-side session-boundary gaps. What changed in this extension:
Why this helps:
Validation run on the updated branch:
|
Summary
context.engineexplicitly selects something elseWhat changed
context.engineexplicitly asks for one/new,/resume, and/branch/compress <focus>backward-compatible with engines that do not accept afocus_topicargumenthermes pluginsso it can be selected from the provider UINon-goals
hermes-lcmplugins/context_engine/Validation
Passed:
python -m pytest tests/run_agent/test_context_engine_plugin_init.py tests/run_agent/test_session_reset_fix.py tests/cli/test_cli_new_session.py tests/hermes_cli/test_plugins_cmd.py tests/agent/test_context_engine.py -q93 passedThat slice includes explicit coverage for: