Skip to content

fix(compression): add explicit auxiliary context_length config - #10072

Closed
zkytech wants to merge 1 commit into
NousResearch:mainfrom
zkytech:fix/aux-compression-context-length
Closed

fix(compression): add explicit auxiliary context_length config#10072
zkytech wants to merge 1 commit into
NousResearch:mainfrom
zkytech:fix/aux-compression-context-length

Conversation

@zkytech

@zkytech zkytech commented Apr 15, 2026

Copy link
Copy Markdown

Summary

  • add an explicit auxiliary.compression.context_length config key to the default config
  • honor that override during compression feasibility checks before metadata probing/fallbacks
  • document the new key in cli-config.yaml.example and cover it with a regression test

Closes #8810.

Context

This PR keeps the scope intentionally narrow: it only adds the explicit auxiliary compression context override requested in #8810.

I checked related work before opening this PR:

Changes

  • run_agent.py: read auxiliary.compression.context_length and forward it to get_model_context_length(..., config_context_length=...) during _check_compression_model_feasibility()
  • hermes_cli/config.py: add auxiliary.compression.context_length to DEFAULT_CONFIG
  • cli-config.yaml.example: document the new config key for custom OpenAI-compatible endpoints
  • tests/run_agent/test_compression_feasibility.py: add regression coverage that the explicit override is forwarded and used by the warning path

Testing

  • source venv/bin/activate && python -m pytest tests/run_agent/test_compression_feasibility.py -q

Contribution Guide Notes

  • reviewed CONTRIBUTING.md before submitting
  • checked for related issues/PRs first and linked them here
  • kept the patch scoped to the requested config addition

@zkytech

zkytech commented Apr 15, 2026

Copy link
Copy Markdown
Author

Closing this PR because already includes this functionality.

@zkytech zkytech closed this Apr 15, 2026
@zkytech
zkytech deleted the fix/aux-compression-context-length branch April 15, 2026 05:04
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.

Feature: add explicit auxiliary.compression.context_length config

1 participant