Repository navigation
feat(mothership): let an agent chat list its owner's other live chats for history recall - #9238
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
[Medium impact] The PR appears safe to merge; no actionable issues remain from this review. SummaryAdds a way for agent and plan chats to list their owner’s other live chats in the same workspace or organization.
Both earlier, unnumbered findings are addressed: real-database query coverage was added, and the unused import was removed. No new actionable issues were found. Tests were not run during this review.
|
|
148bb73 to
dd5b2bb
Compare
|
@greptile-apps review |
|
@cubic-dev-ai review |
@waleedlatif1 I have started the AI code review. It will take a few minutes to complete. |
… for history recall
… mock in the recall test
dd5b2bb to
65fa970
Compare
|
@greptile-apps review |
|
@cubic-dev-ai review |
@waleedlatif1 I have started the AI code review. It will take a few minutes to complete. |
Summary
recall_chatscontrol kind andPOST /api/mothership/chats/recallso an agent chat can list its owner's other live Mothership chats (id, title, updated time), newest firstcopilot.use, owner-readable in a workspace, build permission in an organization, and plan chats hidden unless plan mode is enabledmothershipchat in agent or plan mode may askaccepts, so older Sims never receive it and the worker reports it as unsupportedchat-recall.ts,sim-transport.ts)Product update (if relevant)
A new chat can recall what the user said in their other chats ("the ledger I pasted yesterday") instead of asking them to paste it again. The worker reads only chats on this list, intersected with its own records, and frames what it reads as a record of another conversation.
Type of Change
Testing
Companion: simstudioai/mothership#759
Checklist