Skip to content

fix(status): show output compression level in cce status - #154

Merged
rajkumarsakthivel merged 2 commits into
mainfrom
fix/status-output-compression-153
Aug 2, 2026
Merged

fix(status): show output compression level in cce status#154
rajkumarsakthivel merged 2 commits into
mainfrom
fix/status-output-compression-153

Conversation

@rajkumarsakthivel

Copy link
Copy Markdown
Member

Summary

Closes #153.

The compression.output config key IS wired into the MCP server (via _apply_output_compression, which appends the compression directive to every tool response). The reporter's claim that it's "parsed but never read" was based on testing only cce search, which shows raw retrieval results to a human, not AI model output.

The real gap: cce status didn't display the active output compression level, so users had no way to verify their setting took effect.

This PR adds an Output line to cce status showing the active level, reading from state.json (where set_output_compression persists runtime changes) with the config default as fallback.

The output compression config key (compression.output) was wired into the
MCP server but cce status didn't display it, making it impossible to verify
the setting took effect. Add the output level line to cce status output,
reading from state.json (which set_output_compression persists to) with
config as fallback.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the cce status CLI output to surface the currently active output compression level (used by the MCP server response directive), using state.json as an override with config fallback so users can verify their setting took effect.

Changes:

  • Add an Output line to cce status showing the current output compression level.
  • Read the output compression level from state.json (runtime override) with config fallback.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/context_engine/cli.py Outdated
Clamp output level to LEVELS (off/lite/standard/max) so corrupted
state.json or invalid config doesn't display a bogus level. Reuse
the project_storage_dir() result for both state.json and stats.json.
@rajkumarsakthivel
rajkumarsakthivel merged commit a1a3383 into main Aug 2, 2026
19 checks passed
@rajkumarsakthivel
rajkumarsakthivel deleted the fix/status-output-compression-153 branch August 2, 2026 20:53
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.

compression.output config key is parsed but never read by the retrieval path

3 participants