PAIR-code / PAIR-code/deliberate-lab
Adjust system messages to better record 'left the chat' information
Nobody has claimed this yet.
- Dominant language
- TypeScript
- Stars
- 96
- Forks
- 40
- Avg merge
- 2d 11h
- Merged PRs (30d)
- 20
Description
In #866, we added a conditional to make sure agent participants don't respond to UserType.SYSTEM messages where the sender ID matches because right now, those messages are always "[Participant A] left the chat."
However, this may not always be the case. We should instead capture "this is a 'left the chat' message" in the system message.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the conditional introduced in PR #866 and trace where UserType.SYSTEM messages are created and consumed. Confirm how the message type distinguishes a participant leaving the chat, then verify that agent participants no longer rely on matching sender IDs and that other system messages retain their intended behavior.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- backend
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 48/100