Skip to content

fix: keep connector available during Telegram startup#639

Open
roymeshulam wants to merge 1 commit into
TraderAlice:devfrom
roymeshulam:fix/telegram-connector-startup
Open

fix: keep connector available during Telegram startup#639
roymeshulam wants to merge 1 commit into
TraderAlice:devfrom
roymeshulam:fix/telegram-connector-startup

Conversation

@roymeshulam

Copy link
Copy Markdown

Problem

Telegram Connector startup could block before opening its health endpoint. On this Ubuntu 24 host, the bundled grammY runtime failed setMyCommands with a valid bot token, while the native installed runtime succeeded.

Proposal

  • bind Connector health before external adapter initialization
  • retain degraded adapter health instead of reducing it to a generic unavailable state
  • bound Telegram startup work
  • leave grammY and its retry plugin external to the Connector CJS bundle

Tradeoffs

The connector runtime now relies on its declared grammY dependencies being present beside the packaged service; the existing release/package flow should preserve declared production dependencies. This avoids the observed bundled-runtime incompatibility.

Verification

  • pnpm -F @traderalice/connector-service test
  • pnpm -F @traderalice/connector-service typecheck
  • npx tsc --noEmit
  • pnpm test
  • live local check: valid Telegram token reached awaiting_link after restart; /link completed successfully.

@vercel

vercel Bot commented Jul 16, 2026

Copy link
Copy Markdown

@roymeshulam is attempting to deploy a commit to the luokerenx4's Team Team on Vercel.

A member of the Team first needs to authorize it.

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