-
Notifications
You must be signed in to change notification settings - Fork 2
feat(memory): resolve compress_provider at runtime in compress_context handler #2356
Copy link
Copy link
Closed
Labels
P3Research — medium-high complexityResearch — medium-high complexityenhancementNew feature or requestNew feature or requestmemoryzeph-memory crate (SQLite)zeph-memory crate (SQLite)
Description
Context
PR #2355 added compress_provider config field to [memory.compression] as part of the Focus autonomous compression feature (#2218). However, handle_compress_context() currently uses the primary provider rather than resolving the named provider from the config field.
Required
In crates/zeph-core/src/agent/tool_execution/native.rs (or wherever compress_context is handled):
- Read
compression.compress_providerfrom config - Resolve via
ProviderRegistry::get(name)(or equivalent) - Pass the resolved
AnyProviderto the compaction pipeline instead of the primary provider
Follow the same pattern as admission_provider wiring added in #2355.
Priority
P3 — multi-model design compliance. Functional correctness not affected (falls back to primary provider).
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
P3Research — medium-high complexityResearch — medium-high complexityenhancementNew feature or requestNew feature or requestmemoryzeph-memory crate (SQLite)zeph-memory crate (SQLite)