Conversation
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.
Summary
This hotfix restores
/telegram-setupon Windows/QEMU-style environments where nativefetchcan fail during bot-token validation.It also tightens Threaded Mode terminology across release docs so
Threaded Modenames the mode,threadnames the Telegram UI surface, and BotFather is only described as the bot configuration tool.Why
Issue #83 exposed that setup-time
getMevalidation did not use the same fallback-aware Telegram transport as normal Bot API calls. That meant a Windows/QEMU host could fail withExtension "command:telegram-setup" error: fetch failedbeforetelegram.jsonwas saved, even though later API paths already had IPv4 fallback support.User-visible behavior
/telegram-setuptoken validation retries native transport failures through the existing IPv4 fallback path.Changed areas
lib/telegram-api.ts: routes bot identity lookup through fallback-aware transport.lib/setup.ts: reports setup token-check transport failures as UI errors.tests/telegram-api.test.ts,tests/setup.test.ts: cover setup identity fallback and non-crashing setup failure behavior.@llblab/pi-telegramto0.18.1and records the Windows setup hotfix.Risk Notes
No config migration required. The transport change is scoped to setup-time
getMevalidation and reuses the same fallback helper already used by JSON calls, uploads, and downloads.Validation
npm run validate— passed (typecheck, tests, audit, pack dry-run)repo_context_check /home/llb/.pi/agent/extensions/pi-telegram— passed, 0 errors / 0 warnings