Skip to content

Commit 6076635

Browse files
committed
fix(memory): emit real scoringTimeMs + vectorSearchTimeMs diagnostics
Previously these fields were hardcoded to 0 in three places (CognitiveMemoryManager.retrieve success + weak-hit paths, plus an AgentMemory shim). The shim's zero is correct (standalone Memory doesn't separate stages), but the cognitive path was lying. Fix: MemoryStore.query now returns a timings object with vectorSearchMs (wall time around the vectorStore.query loop) and scoringMs (wall time around scoreAndRankTraces + detectPartiallyRetrieved). CognitiveMemoryManager threads these into diagnostics instead of 0. agentos-bench latencyBreakdown.scoring no longer reports n=0 for every case.
1 parent 62ad19d commit 6076635

9 files changed

Lines changed: 56 additions & 22 deletions

File tree

dist/memory/CognitiveMemoryManager.d.ts.map

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/memory/CognitiveMemoryManager.js

Lines changed: 5 additions & 5 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/memory/CognitiveMemoryManager.js.map

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/memory/retrieval/store/MemoryStore.d.ts

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -72,6 +72,15 @@ export declare class MemoryStore {
7272
query(queryText: string, currentMood: PADState, options?: CognitiveRetrievalOptions): Promise<{
7373
scored: ScoredMemoryTrace[];
7474
partial: PartiallyRetrievedTrace[];
75+
/**
76+
* Per-stage wall-clock timings. Surfaced so
77+
* {@link CognitiveMemoryManager} can populate its diagnostics
78+
* with real numbers instead of the former 0-placeholder.
79+
*/
80+
timings: {
81+
vectorSearchMs: number;
82+
scoringMs: number;
83+
};
7584
}>;
7685
/**
7786
* Record that a memory was accessed (retrieved).

dist/memory/retrieval/store/MemoryStore.d.ts.map

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)