Skip to content

perf(memory): double guidelines load in multi-chunk summarization path #1805

@bug-ops

Description

@bug-ops

Summary

In crates/zeph-core/src/agent/context/summarization.rs, summarize_messages_with_budget loads compression guidelines at line 1303, and then summarize_messages internally loads them again at ~line 367 when called via the multi-chunk fallback at line 1355. This results in two SQLite queries for the same data.

Fix

Load guidelines once in summarize_messages_with_budget and pass them as a parameter to summarize_messages, or cache them at the call site.

Not a correctness issue; acceptable for now.

Identified by code reviewer (REVIEW-10) during ACON compression guidelines PR (#1647) review.

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or requestmemoryzeph-memory crate (SQLite)performancePerformance improvements

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions