Skip to content

chore(hermes): respond to all telegram messages - #2391

Merged
shunkakinoki merged 1 commit into
mainfrom
chore/telegram-require-mention
Aug 16, 2026
Merged

chore(hermes): respond to all telegram messages#2391
shunkakinoki merged 1 commit into
mainfrom
chore/telegram-require-mention

Conversation

@shunkakinoki

@shunkakinoki shunkakinoki commented Aug 16, 2026

Copy link
Copy Markdown
Owner

Sets require_mention: false in the Hermes Telegram config so the agent responds to all messages in group threads (e.g. the みうら thread), not just when mentioned.\n\n- config/hermes/config.tpl.yaml\n- config/hermes/config.template.yaml


Summary by cubic

Hermes Telegram agent now responds to all group messages. Previously it replied only when mentioned; setting telegram.require_mention: false changes behavior to reply to every message in Telegram groups, which may increase noise and token usage.

  • Affects Telegram only; Slack, Discord, and others are unchanged.
  • Regenerate the Hermes config from templates and redeploy to apply. Revert by setting telegram.require_mention: true if noise becomes an issue.

Written for commit 739532f. Summary will update on new commits.

Review in cubic

@pullfrog

pullfrog Bot commented Aug 16, 2026

Copy link
Copy Markdown
Contributor

Run failed. View the logs →

Pullfrog  | Rerun failed job ➔View workflow run | via Pullfrog | Using Claude Opus𝕏

@indent-zero

indent-zero Bot commented Aug 16, 2026

Copy link
Copy Markdown
Contributor

Warning

Indent Zero is shutting down on August 7th. Please migrate over to Indent 2.0 to continue getting PR reviews.

PR Summary

Configures the Hermes Telegram integration to reply to every message in a group thread rather than only when the bot is @mentioned, matching the author's intent for private threads (e.g. the みうら thread). Expected side effect (called out in the PR): higher Telegram traffic and increased LLM token usage.

  • Adds telegram.require_mention: false to config/hermes/config.template.yaml.
  • Mirrors the same key/value in config/hermes/config.tpl.yaml so both templates stay in sync.
  • Reuses the existing require_mention schema key (already set as discord.require_mention: true); scoped to Telegram only — Discord, Slack, Mattermost, and WhatsApp are untouched.

Issues

No issues found.

CI Checks

Waiting for CI checks...

@shunkakinoki
shunkakinoki merged commit c2cb646 into main Aug 16, 2026
25 of 28 checks passed
@shunkakinoki
shunkakinoki deleted the chore/telegram-require-mention branch August 16, 2026 02:53
@coderabbitai

coderabbitai Bot commented Aug 16, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 6cf576d5-2366-444e-88f6-915a5eaf3fd5

📥 Commits

Reviewing files that changed from the base of the PR and between 400a8ff and 739532f.

📒 Files selected for processing (2)
  • config/hermes/config.template.yaml
  • config/hermes/config.tpl.yaml

Disabled knowledge base sources:

  • Linear integration is disabled

You can enable these sources in your CodeRabbit configuration.


📝 Walkthrough

Summary by CodeRabbit

  • New Features
    • Telegram messages can now be processed without requiring a mention.

Walkthrough

Both Hermes Telegram configuration templates now set require_mention to false. Telegram messages can be processed without a user mention.

Changes

Telegram mention configuration

Layer / File(s) Summary
Disable required Telegram mentions
config/hermes/config.template.yaml, config/hermes/config.tpl.yaml
Both configuration templates set Telegram require_mention to false.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Poem

A rabbit hops through Telegram light,
No mention needed to start the night.
Two templates match, their settings agree,
Messages flow freely, as quick as can be.
Hop, hop—configuration harmony!

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/telegram-require-mention

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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.

1 participant