Merge pull request #40 from asynkron/codex/fix-history-compaction-bug

Fix history compactor log metrics

Author: rogeralsing

src/agent/historyCompactor.js

@@ -123,9 +123,12 @@ export class HistoryCompactor {
       content: `${DEFAULT_SUMMARY_PREFIX}\n${summarizedText}`.trim(),
     };
 
+    const originalHistoryLength = history.length;
     history.splice(firstContentIndex, entriesToCompactCount, compactedEntry);
+    // Log the before/after lengths explicitly so the compaction impact is clear.
     this._log('log', '[history-compactor] Compacted history entries.', {
-      originalEntries: entriesToCompactCount,
+      entriesCompacted: entriesToCompactCount,
+      originalHistoryLength,
       resultingHistoryLength: history.length,
     });
 

tests/unit/historyCompactor.test.js

@@ -56,6 +56,14 @@ describe('HistoryCompactor', () => {
     expect(logger.log).toHaveBeenCalledWith(
       '[history-compactor] Compacted summary:\nCondensed summary.',
     );
+    expect(logger.log).toHaveBeenCalledWith(
+      '[history-compactor] Compacted history entries.',
+      {
+        entriesCompacted: 2,
+        originalHistoryLength: 5,
+        resultingHistoryLength: 4,
+      },
+    );
 
     const [payload, options] = responsesCreate.mock.calls[0];
     expect(payload.model).toBe('test-model');