Skip to content

🧬 perf: Evolve Compaction Guidance on Warm Turns - #15371

Merged
danny-avila merged 2 commits into
devfrom
danny-avila/evolve-warm-compaction-index
Aug 30, 2026
Merged

danny-avila merged 2 commits into
devfrom
danny-avila/evolve-warm-compaction-index

Conversation

@danny-avila

Copy link
Copy Markdown
Collaborator

Summary

I wired warm Event Actor continuations to the incremental compaction semantic-index snapshot API published in @librechat/agents@3.7.9.

  • Evolve semantic guidance from the bounded persisted snapshot and only the current warm payload.
  • Preserve activity labels for the formatter's existing single pass while keeping provider-message output unchanged.
  • Carry cumulative providedEntryCount telemetry through JSON persistence, HITL resumes, staged approvals, and settled actor state.
  • Reuse the existing actor-state CAS path without adding a storage read, history scan, or persistence mechanism.
  • Default legacy stored projections without providedEntryCount to their retained entry count for rolling compatibility.
  • Keep the graph-facing input backward-compatible by passing only the evolved snapshot entries into createRun.

Change Type

  • New feature (non-breaking change which adds functionality)

Testing

  • Ran the complete AgentClient suite: 186 tests passed.
  • Ran the compaction continuation projection suite: 6 tests passed.
  • Ran the complete conversation persistence suite: 161 tests passed.
  • Verified the real Agents 3.7.9 formatter evolves a bounded base snapshot from one warm delta without changing provider messages.
  • Ran Prettier checks and ESLint across every touched file.
  • Built @librechat/data-schemas successfully and type-checked the changed compaction adapter in isolation.

Test Configuration:

  • Node.js 24.16.0
  • @librechat/agents 3.7.9

Checklist

  • My code adheres to this project's style guidelines
  • I have performed a self-review of my own code
  • I have commented in any complex areas of my code
  • My changes do not introduce new warnings
  • I have written tests demonstrating that my changes are effective or that my feature works
  • Local unit tests pass with my changes
  • Any changes dependent on mine have been merged and published in downstream modules

@danny-avila

Copy link
Copy Markdown
Collaborator Author

@codex review

Please review the current PR head 4ce667b1d99a7088416bf2a47322f694f6640790. Confirm that this exact commit is the reviewed commit and ignore findings that apply only to earlier heads.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Aug 30, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-08-30T19:55:23.593853Z a471a6b Manual request
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Breezy!

Reviewed commit: 4ce667b1d9

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@danny-avila

Copy link
Copy Markdown
Collaborator Author

@codex review

Please review the current PR head a471a6b1b2a133f10415fb8000330159bc3c6ea8. Confirm that this exact commit is the reviewed commit and ignore findings that apply only to earlier heads.

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. 🚀

Reviewed commit: a471a6b1b2

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@danny-avila

Copy link
Copy Markdown
Collaborator Author

CI exposed a real integration regression already present on current dev: #15366 removed request ownership from agent initialization, while #15352 still reads req.config for configured stateful code environments. The focused prerequisite fix is #15373; the existing 88-test initialization suite passes with that correction. I will rebase this PR after the prerequisite lands.

@danny-avila
danny-avila force-pushed the danny-avila/evolve-warm-compaction-index branch from a471a6b to b47be64 Compare August 30, 2026 20:17
@danny-avila
danny-avila force-pushed the danny-avila/evolve-warm-compaction-index branch from b47be64 to f110ac0 Compare August 30, 2026 20:18
@danny-avila
danny-avila merged commit 7533d13 into dev Aug 30, 2026
43 of 44 checks passed
@danny-avila
danny-avila deleted the danny-avila/evolve-warm-compaction-index branch August 30, 2026 21:11
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.

1 participant