Issue Station
Sign In
Tools hereToolbox

hermes - ✅(Solved) Fix [Bug]: Mattermost mobile users are unable to send commands. [1 pull requests, 1 comments, 2 participants]

Official PRs (…)
ON THIS PAGE

Recommended Tools

×6

Utilities matched from this issue’s tags and category — try them while you read without losing context.

GitHub issue graph ai analysis

Paste a GitHub issue URL. We fetch that issue, discover linked issues from bodies/comments/timeline, collect linked pull requests, and produce a structured English report.

The report is written in English Markdown for sharing and archival.

Helpful · Quick feedback

Loading…
GitHub stats
NousResearch/hermes-agent#25184•Fetched 2026-05-14 03:48:11
View on GitHub
Comments
1
Participants
2
Timeline
5
Reactions
0
Author
JoshuaBowermanJoshuaBowerman
Participants
alt-glitchalt-glitchJoshuaBowermanJoshuaBowerman
Timeline (top)
labeled ×3commented ×1cross-referenced ×1

Error Message

Additional Logs / Traceback (optional)

Not much to root cause here. The error message is pretty clear.

Root Cause

Root Cause Analysis (optional)

Fix Action

Fixed

PR fix notes

PR #25187: fix(gateway): Mattermost Thread and Command Handling Fixes

Description (problem / solution / changelog)

What does this PR do?

This PR changes the mattermost thread handling to properly choose the thread's root id. It also changes the mattermost command handling code to accept commands preceded by a single space character.

Related Issue

Fixes https://github.com/NousResearch/hermes-agent/issues/25181 Fixes https://github.com/NousResearch/hermes-agent/issues/25184

Type of Change

<!-- Check the one that applies. -->
  • 🐛 Bug fix (non-breaking change that fixes an issue)
  • ✨ New feature (non-breaking change that adds functionality)
  • 🔒 Security fix
  • 📝 Documentation update
  • ✅ Tests (adding or improving test coverage)
  • ♻️ Refactor (no behavior change)
  • 🎯 New skill (bundled or hub)

Changes Made

<!-- List the specific changes. Include file paths for code changes. -->
  • gateway/platforms/base.py - uses thread_id if available.
  • gateway/platforms/mattermost.py - Allows prefixed ' ' character in commands for mattermost.
  • tests/gateway/test_mattermost.py - Tests for changed behaviour.

How to Test

  1. Run hermes gateway with mattermost configured and threading enabled.
  2. Post /help to a channel
  3. You should see the proper help command output.
  4. Post /help as a reply to the bot reply.
  5. You should get the same output again.

Checklist

<!-- Complete these before requesting review. -->

Code

  • I've read the Contributing Guide
  • My commit messages follow Conventional Commits (fix(scope):, feat(scope):, etc.)
  • I searched for existing PRs to make sure this isn't a duplicate
  • My PR contains only changes related to this fix/feature (no unrelated commits)
  • I've run pytest tests/ -q and all tests pass
  • I've added tests for my changes (required for bug fixes, strongly encouraged for features)
  • I've tested on my platform: <!-- e.g. Ubuntu 24.04, macOS 15.2, Windows 11 -->

Changed files

  • gateway/platforms/base.py (modified, +7/-0)
  • gateway/platforms/mattermost.py (modified, +10/-1)
  • tests/gateway/test_mattermost.py (modified, +202/-0)

Code Example

Not doing that. I know the issue, I will create a PR.

---
RAW_BUFFERClick to expand / collapse

Bug Description

The mattermost mobile app prevents sending messages with a starting / character.

<img width="1344" height="679" alt="Image" src="https://github.com/user-attachments/assets/45f86f51-8f59-48f5-95e8-db966147c7e4" />

Steps to Reproduce

Try to run a command from the mattermost mobile app.

Expected Behavior

Either you should be able to send commands with the preceding /. Or you should be able to prefix a ' ' character as suggested by the app and the gateway should accept that as a command.

Actual Behavior

You can't post that message.

Affected Component

Gateway (Telegram/Discord/Slack/WhatsApp)

Messaging Platform (if gateway-related)

No response

Debug Report

Not doing that. I know the issue, I will create a PR.

Operating System

Arch

Python Version

3.11.15

Hermes Version

v0.13.0

Additional Logs / Traceback (optional)

Root Cause Analysis (optional)

Not much to root cause here. The error message is pretty clear.

Proposed Fix (optional)

I'm going to change the mattermost command handling to process commands preceded with a single space character.

Are you willing to submit a PR for this?

  • I'd like to fix this myself and submit a PR

Vote matrix · Quick signals

Works
Did the solution work? Tap to confirm.
Easy Fix
Was it a quick fix?
Time Saver
Did it save you time?
Blocking
Was it severely blocking?
Common Issue
Are others likely hitting this too?
Flaky / Intermittent
Is it intermittent?
Verified / Reproducible
Can you reproduce it reliably?
Loading…
#configuration error#environment variable#network issue#logging issue#authentication issue

Still need to ship something?

×6

Another batch ranked right after the header list — different links, same matching logic.

Back to top recommendations

TRENDING

hermes - ✅(Solved) Fix [Bug]: Mattermost mobile users are unable to send commands. [1 pull requests, 1 comments, 2 participants]