Skip to content

fix(compression): show engine preview output - #4128

Merged
diegosouzapw merged 1 commit into
cheaper-inference:release/v3.8.29from
megamen32:fix/context-compression-preview
Jun 18, 2026
Merged

diegosouzapw merged 1 commit into
cheaper-inference:release/v3.8.29from
megamen32:fix/context-compression-preview

Conversation

@megamen32

Copy link
Copy Markdown
Contributor

Summary

  • render compression preview original and compressed text, not only token counters
  • render returned preview diff segments so engine pages such as /dashboard/context/ccr show the actual preview content
  • add a regression test covering original/compressed/diff visibility from /api/compression/preview

Base

Created from the latest upstream branch by committer date: diegosouzapw/OmniRoute@release/v3.8.29 (cf716e97d).

Tests

  • npx vitest run tests/unit/ui/engineConfigPage.test.tsx
  • npx eslint src/shared/components/compression/EngineConfigPage.tsx tests/unit/ui/engineConfigPage.test.tsx

Notes

  • npx tsc --noEmit --pretty false currently fails on existing repository-wide errors unrelated to this change (stale .build/next/types route export checks, opencode-plugin, executor typings, etc.).

@megamen32
megamen32 requested a review from diegosouzapw as a code owner June 18, 2026 00:55
@gemini-code-assist

Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@diegosouzapw
diegosouzapw merged commit 677d447 into cheaper-inference:release/v3.8.29 Jun 18, 2026
4 checks passed
@diegosouzapw

Copy link
Copy Markdown
Collaborator

Thanks, @megamen32! 🙏 Good catch — the compression preview was throwing away the most useful part of the API response. Verified end-to-end: /api/compression/preview already returns original/compressed/diff: segments (route.ts), and your renderDiffSegment matches the real { type, text } segment shape from diffHelper.ts, so it renders correctly against live data — not just the mock. Regression test covers it. Merged into release/v3.8.29; ships in the upcoming release. 🚀

@diegosouzapw diegosouzapw mentioned this pull request Jun 19, 2026
tkgo11 pushed a commit to tkgo11/OmniRoute that referenced this pull request Sep 23, 2026
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