hermes - 💡(How to fix) Fix English Coach prefix triggers should override domain routing [1 comments, 1 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#15657Fetched 2026-04-26 05:25:56
View on GitHub
Comments
1
Participants
1
Timeline
6
Reactions
0
Participants
Timeline (top)
labeled ×4closed ×1commented ×1

Root Cause

Explicit prefix triggers like polish:, fix:, check:, word:, say:, en:, and zh: should have higher priority than semantic/domain routing. Otherwise learner-facing commands become unreliable whenever the text being corrected mentions technical topics.

Fix Action

Fix / Workaround

Local workaround already applied

The local english-coach skill was patched with this precedence rule.

Code Example

polish: what's the current config's max_turns & approvals.mode

---

When the user message begins with an English Coach trigger like `polish:`, `fix:`, `check:`, `correct:`, `proofread:`, `say:`, `word:`, `words:`, `en:`, or `zh:`, treat everything after the colon as text to process. Do not answer the domain question unless the user asks without an English Coach trigger.
RAW_BUFFERClick to expand / collapse

Skill trigger precedence should override domain-looking text

Bug Description

When a user message starts with an English Coach trigger such as polish:, Hermes can incorrectly route the request to a domain-specific skill if the text after the trigger looks like a config/code/GitHub task.

Example:

polish: what's the current config's max_turns & approvals.mode

Expected behavior: English Coach handles this as a polish request and rewrites the sentence.

Actual behavior: Hermes treated it as a Hermes config question, loaded the Hermes config skill, read ~/.hermes/config.yaml, and answered with the config values.

Why this matters

Explicit prefix triggers like polish:, fix:, check:, word:, say:, en:, and zh: should have higher priority than semantic/domain routing. Otherwise learner-facing commands become unreliable whenever the text being corrected mentions technical topics.

Proposed behavior

If a user message begins with an English Coach trigger followed by :, treat everything after the colon as the text payload for that mode, even if the payload mentions configs, code, commands, GitHub, or other domain-specific terms.

Suggested rule:

When the user message begins with an English Coach trigger like `polish:`, `fix:`, `check:`, `correct:`, `proofread:`, `say:`, `word:`, `words:`, `en:`, or `zh:`, treat everything after the colon as text to process. Do not answer the domain question unless the user asks without an English Coach trigger.

Local workaround already applied

The local english-coach skill was patched with this precedence rule.

Environment

  • Interface: Telegram gateway
  • Affected area: skill routing / trigger precedence

extent analysis

TL;DR

Implement a precedence rule to prioritize English Coach triggers over domain-specific routing when a user message starts with a trigger followed by a colon.

Guidance

  • Identify all English Coach triggers (e.g., polish:, fix:, check:, etc.) and ensure they are handled with higher priority than domain-specific skills.
  • Update the skill routing logic to treat everything after the colon as the text payload for the triggered mode, regardless of domain-specific terms.
  • Verify the fix by testing user messages with English Coach triggers followed by domain-specific text, ensuring the expected behavior is achieved.
  • Consider applying the local workaround already applied to the english-coach skill to other affected areas.

Example

No code snippet is provided as the issue does not contain explicit code references.

Notes

The proposed behavior change may require updates to the Hermes config skill and other domain-specific skills to ensure compatibility with the new precedence rule.

Recommendation

Apply the workaround by implementing the proposed precedence rule, as it addresses the root cause of the issue and ensures reliable handling of learner-facing commands.

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…

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 - 💡(How to fix) Fix English Coach prefix triggers should override domain routing [1 comments, 1 participants]