Repository navigation
feat(cli): send Telegram text without attachments (refs #518) - #3104
Merged
Merged
Conversation
Owner
Author
Stale-base triage: unrelated deltaCompared PR head The independent architecture review classified the delta as UNRELATED to this CLI change:
Keep the current head and its 29-test/review evidence; no branch refresh, rebuild or duplicate CI is required for this delta. This does not waive full-tier CI: every required check must still actually succeed on the exact PR head before merge. |
|
This was referenced Oct 18, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.



What changed
Adds
telegram-send-message --text <TEXT>with the existing bot selectors, allowing rooms and coordinators to send text without an attachment or active Telegram bridge. It preserves the original text, rejects empty input and a conservative 4096 UTF-16-unit limit, and uses controlled errors without printing message content or bot credentials.The image command remains compatible; the CLI reference documents the new verb.
Why
The existing CLI required an image or file even for a short status update. Closes #518.
How I tested
cargo +1.98.1 check --locked -p agentscommander --all-targets: passed.BC0C786F9B12EE0C2B078655A82DD82389C89BE54AF6856A2E915F01869F374F.Anything reviewers should know
Local validation covered Windows debug artifacts at
2168d1526d40491d137e429ac170f3f1b9fab748, including main through01456f151b3b61cfc55709b30c49984146966f0e. The initial Windows CI failure repeated on the same commit: the new test introduced an extra process-construction route rejected by the existing isolation source guard. The test now reuses the existing isolated runner; all cases and assertions are retained, and the unchanged guard passes locally. Both independent reviewers approved this correction and integration.Success/failure behavior uses synthetic senders; the real error-discarding helper was reviewed structurally. Full CI is requested with
ci:full; every required check must succeed on the exact PR head before merge.