Aron Gates e47dcf37ff fix: address code review feedback for data retention PR ... Critical: - Fix BookmarkMenu crash: restore optional chaining on conversation - Fix migration hazard: backward-compatible sidebar filter that also checks expiredAt for documents without isTemporary field Major: - Add logging to getRetentionExpiry error path, align with tools.js - Add tests for retentionMode: ALL in saveConvo and saveMessage - Fix share route: apply expiredAt for temporary chats too by querying the conversation's isTemporary flag server-side - Add assertions for getRetentionExpiry mocks in process tests Minor: - Fix ChatRoute isTemporaryChat to be strictly boolean via Boolean() - Fix stale test description (expired -> temporary) - Comment out retentionMode default in example yaml - Simplify verbose if/else to isTemporary === true - Add compound index on { user: 1, isTemporary: 1 } - Remove narrating comment from process.spec.js Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> (cherry picked from commit 6bad535f90)		2026-05-10 14:06:39 -04:00
..
__tests__	fix: address code review feedback for data retention PR	2026-05-10 14:06:39 -04:00
crud.js	🔐 feat: Mint Code API Auth Tokens (#13028)	2026-05-09 16:09:10 -04:00
crud.spec.js	🔐 feat: Mint Code API Auth Tokens (#13028)	2026-05-09 16:09:10 -04:00
index.js
process.js	feat: extend data retention to files, tool calls, and shared links	2026-05-10 14:06:12 -04:00
process.spec.js	fix: address code review feedback for data retention PR	2026-05-10 14:06:39 -04:00