Skip to content

fix: match whole command text; refresh readme - #37

Merged
Markeli merged 2 commits into
mainfrom
fix/command-token-matching
Oct 3, 2026
Merged

Markeli merged 2 commits into
mainfrom
fix/command-token-matching

Conversation

@Markeli

@Markeli Markeli commented Oct 3, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Fix: a command matched any message that merely started with its CommandText, so /pinger went to the /ping handler. The command text must now be followed by the end of the message, whitespace or @ (/ping now, /ping@MyBot). The longest matching CommandText wins, and matching is ordinal. Released as 3.0.1.
  • Docs: README features now cover HTTP proxy, queue persistence and multi-step commands as a state machine. Added "How it differs" and a Roadmap pointing to the v4 milestone.

Tests

  • New tests for whole-word matching, longest-match selection and cached-state routing.
  • 113 tests pass on net10.0. net9.0 wasn't run locally because the runtime isn't installed, so CI covers it.

🤖 Generated with Claude Code

Markeli and others added 2 commits October 3, 2026 16:46
/pinger no longer resolves to the /ping handler; the longest matching
command text wins.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Oct 3, 2026

Copy link
Copy Markdown

Release check

Package Version On merge
Curiosus.TelegramBot 3.0.1 🚀 released as 3.0.1

For 89172ed.

@Markeli
Markeli merged commit 2816c45 into main Oct 3, 2026
6 checks passed
@Markeli
Markeli deleted the fix/command-token-matching branch October 3, 2026 14:02
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