Closed Bug 2022894 Opened 1 month ago Closed 1 month ago

Memories list shows empty entries in the applied memories popover

Categories

(Core :: Machine Learning: Frontend, defect)

Desktop
Unspecified
defect

Tracking

()

RESOLVED DUPLICATE of bug 2022892
Tracking Status
firefox150 --- affected

People

(Reporter: echa, Assigned: echa)

References

(Blocks 1 open bug)

Details

(Whiteboard: [genai][chat])

ChatMessage.addTokens adds the raw memory ID to memoriesApplied, but never sets _pendingMemoryIds.

Later on, ChatConversation.receiveResponse checks _pendingMemoryIds?.length to decide whether it should resolve those IDs into full memory objects using getMemoriesByID. Because _pendingMemoryIds is never populated, that resolution step never runs.

The fix is in ChatMessage.sys.mjs: addTokens should also populate _pendingMemoryIds so the IDs actually get resolved.

Looks like this double-submitted

Status: NEW → RESOLVED
Closed: 1 month ago
Duplicate of bug: 2022892
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.