Skip to content

Fix Telegram callback button acknowledgements - #10

Merged
quickerup merged 1 commit into
mainfrom
codex/fix-endless-loop-button-functionality
Jul 19, 2026
Merged

Fix Telegram callback button acknowledgements#10
quickerup merged 1 commit into
mainfrom
codex/fix-endless-loop-button-functionality

Conversation

@quickerup

@quickerup quickerup commented Jul 19, 2026

Copy link
Copy Markdown
Owner

Motivation

  • Telegram inline buttons (workflow browser selection) were left showing a persistent loading spinner because the callback query was not acknowledged, causing a poor UX that needs fixing.

Description

  • After opening the workflow browser for wf_sel: callbacks we now call answerCallbackQuery to acknowledge the button, and answerCallbackQuery was changed to accept an options object and include provided fields (e.g. text, show_alert) when calling the Telegram API in worker/src/index.js.

Testing

  • Ran node --check worker/src/index.js and git diff --check, both checks passed.

Codex Task

Summary by CodeRabbit

  • Bug Fixes
    • Workflow selections in Telegram now provide immediate confirmation after being chosen.
    • Callback responses support additional options for more flexible interaction handling.

@quickerup
quickerup merged commit ead37f5 into main Jul 19, 2026
1 check was pending
@coderabbitai

coderabbitai Bot commented Jul 19, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 2aa7b829-bf07-4bc4-84e6-6935ae551c6d

📥 Commits

Reviewing files that changed from the base of the PR and between a387380 and f32ba0a.

📒 Files selected for processing (1)
  • worker/src/index.js

📝 Walkthrough

Walkthrough

The worker now acknowledges wf_sel:* Telegram callbacks with “Workflow opened.” and generalizes answerCallbackQuery to accept optional Telegram response fields.

Changes

Telegram callback acknowledgements

Layer / File(s) Summary
Generalize callback responses
worker/src/index.js
answerCallbackQuery accepts an optional options object and includes its fields in the Telegram API request body.
Acknowledge workflow selection
worker/src/index.js
wf_sel:* handling sends a “Workflow opened.” callback confirmation before continuing.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related PRs

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/fix-endless-loop-button-functionality

Comment @coderabbitai help to get the list of available commands.

@quickerup
quickerup deleted the codex/fix-endless-loop-button-functionality branch July 19, 2026 06:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant