[rafaumeu]

Summary

Fixes #8137 — combo retry redundantly recompresses identical bodies across targets because the memo cache key includes model, which changes every retry attempt.

Problem

makeMemoKey() always included model and supportsVision in the cache key. This is correct for the lite engine (which strips data:image URLs based on vision support), but wrong for model-independent engines like rtk, caveman, and stacked pipelines without lite.

In the combo retry loop, the body and config are identical across targets — only the model changes. Including model in the key produced a fresh cache miss on every retry, re-running the full compression pipeline 5-8x per request instead of serving the cached result.

Fix

makeMemoKey() now conditionally includes model + supportsVision only when the compression pipeline actually uses a vision-dependent engine:

• lite — yes (image-URL stripping)
• standard — yes (lite → caveman pipeline)
• stacked with a lite step — yes
• rtk, caveman, stacked without lite — no (model-independent)

New helper: usesVisionDependentEngine(mode, config) classifies modes.

Test plan

• All 32 existing result-memo tests pass
• 5 new tests:
  • rtk mode: same key across different models
  • caveman mode: same key across different models
  • stacked-without-lite: same key across different models
  • stacked-with-lite: different keys across different models (still vision-dependent)
  • lite mode: model + supportsVision still affect key (existing regression test preserved)
• No LSP diagnostics introduced

[gemini-code-assist]

Caution

The consumer version of Gemini Code Assist on GitHub has been sunset. All code review activity has officially ceased.

[mergify]

⚠️ The sha of the head commit of this PR conflicts with #8179. Mergify cannot evaluate rules on this PR. Once #8179 is merged or closed, Mergify will resume processing this PR. ⚠️

[diegosouzapw]

Thanks, @rafaumeu! 🙏 Merged — compression memo key is now model-independent for non-vision engines. result-memo tests green.

[diegosouzapw]

Thanks, @rafaumeu! 🙏 Merged — compression memo key is now model-independent for non-vision engines. Rebased onto the current release tip (dropped the stale bundled .env.example/README noise, kept only the resultMemo change). 32/32 tests green.