Fix Telegram rich progress command output#93364
Conversation
|
Codex review: needs real behavior proof before merge. Reviewed June 15, 2026, 12:30 PM ET / 16:30 UTC. Summary PR surface: Source +35, Tests +135. Total +170 across 4 files. Reproducibility: yes. source-level: current main routes Telegram command-output progress through the raw renderer while docs/config say commandText status should hide command text in progress previews. I did not run a live Telegram reproduction in this read-only review. Review metrics: none identified. Merge readiness Overall follows the weaker of proof and patch quality, so missing proof can cap an otherwise strong patch. Rank-up moves:
Proof guidance:
Mantis proof suggestion Risk before merge
Maintainer options:
Next step before merge
Security Review detailsBest possible solution: Land the Telegram fix only after maintainer review and redacted live Telegram proof, then track any shared-channel command-output consistency cleanup separately. Do we have a high-confidence way to reproduce the issue? Yes, source-level: current main routes Telegram command-output progress through the raw renderer while docs/config say commandText status should hide command text in progress previews. I did not run a live Telegram reproduction in this read-only review. Is this the best way to solve the issue? Yes for the Telegram bug: using buildChannelProgressDraftLineForEntry matches the existing tool/item event path and keeps the fix inside the channel renderer boundary. The only unresolved shape question is whether sibling channels should be fixed in a separate shared cleanup. AGENTS.md: found and applied where relevant. Codex review notes: model internal, reasoning high; reviewed against 53541b2141c6. Label changesLabel changes:
Label justifications:
Evidence reviewedPR surface: Source +35, Tests +135. Total +170 across 4 files. View PR surface stats
What I checked:
Likely related people:
What the crustacean ranks mean
Shiny media proof means a screenshot, video, or linked artifact directly shows the changed behavior. Runtime, network, CSP, and security claims still need visible diagnostics. How this review workflow works
|
0f916b2 to
dd010d2
Compare
|
Landed via rebase onto main.
|

Summary
commandText: "status"also hides command titles.Verification
Review