fix(api): handle multiple file params, fix parse_mode defaults#31
Merged
fix(api): handle multiple file params, fix parse_mode defaults#31
Conversation
added 3 commits
February 12, 2026 00:24
Add CI pipeline with test matrix (Lua 5.3, 5.4, LuaJIT) and lint job. Fix 16 luacheck warnings: unused imports, variable shadowing, style.
Fixes #30 - api.request only processed one file entry via next(file), causing thumbnails to be lost and potentially dropping the primary media when both a file and thumbnail were provided. Changed to iterate all entries with a for loop. Also fixes: - send_reply, input_text_message_content, and send_inline_article using deprecated 'markdown' instead of 'MarkdownV2' for boolean parse_mode - send_poll missing question_parse_mode and question_entities params (added in Bot API 7.3) Adds comprehensive unit tests for all media methods, e2e test suite (busted --run e2e), .gitignore, and bumps version to 3.1-0.
Add comprehensive e2e tests for bot profile (name/description set/get/restore with language_code), scoped commands, admin rights, menu button, webhook opts, forum topic icon stickers, user profile photos, available gifts, sticker sets, invoice links, custom emoji stickers, and error-case coverage for all send methods, chat management, inline queries, games, and passport.
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
1 participant
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
Fixes #30 -
api.requestonly processed one file entry vianext(file), causing thumbnails to be lost and potentially dropping the primary media when both a file and thumbnail were provided.api.requestfile handling fromnext(file)(single entry) toforloop (all entries), fixingsend_video,send_audio,send_document,send_animation, andsend_video_notewhen thumbnails are providedsend_reply,input_text_message_content, andsend_inline_articlewere using deprecated'markdown'instead of'MarkdownV2'for booleanparse_mode = truequestion_parse_modeandquestion_entitiestosend_poll(Bot API 7.3+)busted --run e2e),.gitignore, and.envsupportTest plan