Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Add Telegram Team features #1033

Merged
merged 2 commits into from
Dec 14, 2024

Conversation

azep-ninja
Copy link
Contributor

Relates to:

PR 957

Risks

Low: Additional settings to character.json types are low level changes, and should not break any core functionality. Additionally telegram messageManager.ts and file additions are specific to the new team settings, so risk is low.

Background

What does this PR do?

Enhances the Telegram bot's message handling to support:

  1. Team-based agent interactions - allowing multiple agents to coexist and coordinate responses
  2. More natural conversation flow in team contexts
  3. Improved interest management for multi-agent scenarios

What kind of change is this?

Features (non-breaking change which adds functionality)
Improvements (misc. changes to existing features)

Documentation changes needed?

My changes require a change to the project documentation.

Add documentation for team-based interaction features

Testing

Where should a reviewer start?

  1. Review changes in messageManager.ts (Telegram) particularly:
  • Team logic handling in handleMessage
  • Updated and _shouldRespond methods
  • New configuration option handling

Detailed testing steps

  1. Test single agent mentions-only mode:
  • Verify agent only responds to direct mentions
  • Verify no responses to non-mention messages
  1. Test team functionality:
  • Configure multiple agents with isPartOfTeam: true
  • Verify only mentioned agent responds to direct mentions
  • Verify team leader handles team-wide requests
  • Verify proper conversation handoff between agents
  • Verify natural conversation flow (no duplicate responses)
  1. Edge cases:
  • Test with incorrect/missing team configuration
  • Test with mixed team/non-team agents
  • Test conversation handling with multiple users

ninja_dev - Discord Username

@odilitime odilitime changed the base branch from main to develop December 13, 2024 16:37
@odilitime
Copy link
Collaborator

Fix the linter issues and we'll merge it

@odilitime odilitime deleted the branch elizaOS:develop December 13, 2024 19:07
@odilitime odilitime closed this Dec 13, 2024
@odilitime odilitime reopened this Dec 13, 2024
@azep-ninja
Copy link
Contributor Author

@odilitime ready to go!

@monilpat monilpat merged commit 7ad8141 into elizaOS:develop Dec 14, 2024
4 checks passed
@azep-ninja azep-ninja deleted the telegram-team-feature branch December 15, 2024 21:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants