[codex] fix Telegram uploads and repeated progress edits#264
Open
GSL-R wants to merge 3 commits into
Open
Conversation
Owner
|
Reviewed against current
Neither blocks the grammY |
Contributor
Author
|
Addressed both review notes in 642e85e:
Local verification:
The refreshed PR Actions run currently fails in 12 unrelated prompt-size, AGY quota, quota API, and shared-skill assertions. The latest upstream main runs are failing in the same period as well; the Telegram tests pass in this PR run. |
lidge-jun
added a commit
that referenced
this pull request
Jul 6, 2026
…ick PR #264) Cherry-picked 3 commits from GSL-R:fix/telegram-upload-progress-loop: - route Node streaming/multipart bodies through node-fetch - keep IPv4 HTTPS agent for file uploads - consume pending tool-status snapshots exactly once - focused tests for Node Readable uploads and status coalescing
lidge-jun
added a commit
that referenced
this pull request
Jul 6, 2026
- cherry-pick PR #264 (telegram uploads/progress edits) into dev - bump PSC-006 prompt size budget 35000 → 36000 (diagram-file additions) - update server_api.md route/endpoint counts (+1 for /api/widgets) - realign diagram-iframe-expand-contract string matches to P1 refactor - realign orchestration timeout/dispatch contract tests - realign manager folder-panel line-count and session contract tests - realign orc-state-route diagnostics contract test npm test: 5392 tests, 5374 pass, 0 fail, 18 skipped
lidge-jun
added a commit
that referenced
this pull request
Jul 6, 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
node-fetchinstead of JSON-stringifying them in the IPv4 Telegram fetch adapterReadableuploads and status update coalescingRoot cause
The custom IPv4 fetch adapter only handled strings and JSON-compatible bodies. grammY builds file uploads as a Node
Readablemultipart stream, sosendPhotowas serializing that stream as JSON and failing with a network-style error.Separately, the Telegram tool-status updater never cleared
pendingStatusTextafter a successful flush. Itsfinallyblock therefore scheduled the sameeditMessageTextforever during long agent runs, generating unnecessary Telegram traffic and delaying real deliveries.Impact
sendPhoto,sendDocument, andsendVoicecan retain their multipart bodiesValidation
npx tscsendPhotothrough/api/channel/sendsucceeded with the generated dream image after restart