Skip to content

Clean unused helpers and config paths - #4

Open
HorGonAI wants to merge 1 commit into
horgonai/add-button-keyboard-functionality-to-botfrom
horgonai/optimize-and-clean-up-codebase
Open

Clean unused helpers and config paths#4
HorGonAI wants to merge 1 commit into
horgonai/add-button-keyboard-functionality-to-botfrom
horgonai/optimize-and-clean-up-codebase

Conversation

@HorGonAI

@HorGonAI HorGonAI commented Feb 1, 2026

Copy link
Copy Markdown
Owner

Motivation

  • Remove dead code and unused configuration handling to simplify maintenance and reduce build surface.
  • Avoid duplicated screenshot send logic by centralizing it into a single helper to reduce repetition and potential bugs.
  • Remove stale build reference to a non-existent include directory to prevent confusing CMake setups.

Description

  • Deleted unused helpers and command execution features: removed runCommand, getStatus, buildHelp, the commands map in Config, and the related config parsing/serialization.
  • Replaced repeated screenshot capture + send blocks with a single sendScreenshot helper and updated all call sites to use it.
  • Removed unnecessary includes (<array>, <cstring>) and the unused kMaxMessageLength constant.
  • Simplified the Web App client by removing the unused sendWebAppData function and a redundant Telegram presence check, and removed the include directory entry from CMakeLists.txt.

Testing

  • No automated tests were executed for this change.

Codex Task

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