Beta

[5hojib]

Summary by Sourcery

Enhancements:

• Remove the "refresh status" and "overview status" buttons, simplifying the status interface.

[sourcery-ai]

Reviewer's Guide by Sourcery

This pull request refactors the status command, removing the overview page and associated logic. It also updates the default branch to 'beta' and fixes a bug in the shortener logic.

Sequence diagram for simplified status command flow

Loading

sequenceDiagram
    participant User
    participant Bot
    participant TaskDict

    User->>Bot: Send status query
    Bot->>TaskDict: Get task data
    alt Next/Previous page
        Bot->>Bot: Update page number
    else Status change
        Bot->>TaskDict: Update task status
    end
    Bot->>User: Return status message with navigation buttons

Class diagram for bot commands update

Loading

classDiagram
    class BotCommands {
        +StartCommand: str
        +MirrorCommand: list
        +YtdlCommand: list
        +LeechCommand: list
    }
    note for BotCommands "StartCommand simplified to 'start'"

State diagram for URL shortener logic

Loading

stateDiagram-v2
    [*] --> CheckShorteners
    CheckShorteners --> ReturnOriginal: No shorteners available
    CheckShorteners --> AttemptShorten: Shorteners available
    AttemptShorten --> ReturnShortened: Success
    AttemptShorten --> RetryAttempt: Failure
    RetryAttempt --> AttemptShorten: Retry < 4
    RetryAttempt --> ReturnOriginal: Retry >= 4
    ReturnShortened --> [*]
    ReturnOriginal --> [*]

File-Level Changes

Change	Details	Files
Removed the overview page from the status command.	Removed the 'ov' case from the status_pages function. Removed the logic for displaying the overview page. Removed the button for the overview page.	bot/modules/status.py
Updated the status message to remove the overview button.	Removed the overview button from the status message. Added a check to avoid creating a button maker if there are no buttons to add.	bot/helper/ext_utils/status_utils.py
Fixed a bug in the shortener logic.	Added a check to return the original URL if the shorteners list is empty.	bot/helper/aeon_utils/shorteners.py
Updated the default branch to 'beta'.	Changed the default value of UPSTREAM_BRANCH to 'beta'.	update.py
Removed the index from the start command.	Changed the StartCommand to 'start'.	bot/helper/telegram_helper/bot_commands.py

---

Tips and commands

Interacting with Sourcery

• Trigger a new review: Comment @sourcery-ai review on the pull request.
• Continue discussions: Reply directly to Sourcery's review comments.
• Generate a GitHub issue from a review comment: Ask Sourcery to create an
  issue from a review comment by replying to it.
• Generate a pull request title: Write @sourcery-ai anywhere in the pull
  request title to generate a title at any time.
• Generate a pull request summary: Write @sourcery-ai summary anywhere in
  the pull request body to generate a PR summary at any time. You can also use
  this command to specify where the summary should be inserted.

Customizing Your Experience

Access your dashboard to:

• Enable or disable review features such as the Sourcery-generated pull request
  summary, the reviewer's guide, and others.
• Change the review language.
• Add, remove or edit custom review instructions.
• Adjust other review settings.

Getting Help

• Contact our support team for questions or feedback.
• Visit our documentation for detailed guides and information.
• Keep in touch with the Sourcery team by following us on X/Twitter, LinkedIn or GitHub.

[sourcery-ai]

Hey @5hojib - I've reviewed your changes - here's some feedback:

Overall Comments:

• Please provide documentation explaining the rationale behind removing the status overview feature ('ov' command). This seems like a significant reduction in monitoring capabilities that should be justified.
• The PR title and description are too vague. Please add a proper description explaining the purpose and impact of these changes.

Here's what I looked at during the review

• 🟡 General issues: 1 issue found
• 🟢 Security: all looks good
• 🟢 Testing: all looks good
• 🟢 Complexity: all looks good
• 🟢 Documentation: all looks good

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.}

[sourcery-ai]

issue (bug_risk): This line creates a refresh button that won't work since its handler was removed from status.py

The refresh button handler (data[2] == "ref" case) was removed from status.py, but this code still tries to create the refresh button. This will lead to a non-functional UI element.