Skip to content

revert: restore original PythonBot workflow #1826

Merged
exploreriii merged 12 commits into
hiero-ledger:mainfrom
Abhijeet2409:revert-bot-workflow
Feb 21, 2026
Merged

revert: restore original PythonBot workflow #1826
exploreriii merged 12 commits into
hiero-ledger:mainfrom
Abhijeet2409:revert-bot-workflow

Conversation

@Abhijeet2409

Copy link
Copy Markdown
Member

Summary

This PR reverts the PythonBot workflow failure notifier to its original
implementation to restore the previous stable behaviour.

Why

The updated workflow was failing when attempting to post PR comments.
Maintainers requested reverting the workflow to restore stability.

Testing

Fixes #1825

…tability

Signed-off-by: Abhijeet Saharan <abhijeetsaharan2236@gmail.com>
Signed-off-by: Abhijeet Saharan <abhijeetsaharan2236@gmail.com>
Signed-off-by: Abhijeet <abhijeetsaharan2236@gmail.com>
Signed-off-by: Abhijeet <abhijeetsaharan2236@gmail.com>
Signed-off-by: Abhijeet <abhijeetsaharan2236@gmail.com>
Signed-off-by: Abhijeet <abhijeetsaharan2236@gmail.com>
Signed-off-by: Abhijeet Saharan <abhijeetsaharan2236@gmail.com>
Signed-off-by: Abhijeet <abhijeetsaharan2236@gmail.com>
@Abhijeet2409
Abhijeet2409 marked this pull request as ready for review February 19, 2026 19:53
@Abhijeet2409
Abhijeet2409 requested review from a team as code owners February 19, 2026 19:53
@Abhijeet2409

Abhijeet2409 commented Feb 19, 2026

Copy link
Copy Markdown
Member Author

Hi @exploreriii 🙂
This PR restores the original workflow.
The previous revert PR was auto-closed as it wasn’t linked to an issue.
Would appreciate a review.

@coderabbitai

coderabbitai Bot commented Feb 19, 2026

Copy link
Copy Markdown
Contributor

Walkthrough

Deletes the Node.js workflow notifier and replaces it with inline Bash steps in the GitHub Actions workflow; updates workflow inputs, permissions, concurrency, and PR-resolution/posting logic; adds a CHANGELOG entry noting the revert.

Changes

Cohort / File(s) Summary
Deleted notifier script
.github/scripts/bot-workflows.js
Removed the entire Node.js-based workflow failure notifier (DRY_RUN, env validation, gh CLI wrappers, PR resolution, duplicate detection, and posting logic).
Workflow changes
.github/workflows/bot-workflows.yml
Replaced script invocation with inline Bash to resolve PR via gh, check existing comments, and post an idempotent multi-line comment; removed workflow_dispatch inputs and DRY_RUN handling; changed pull-request permission to write and simplified concurrency/condition logic.
Changelog
CHANGELOG.md
Added entry documenting revert to the previous PythonBot workflow behavior (PR #1825).

Sequence Diagram(s)

sequenceDiagram
    participant GHA as GitHub Actions<br/>Workflow
    participant GHCLI as GitHub CLI (gh)
    participant API as GitHub API

    GHA->>GHCLI: Determine PR for HEAD_BRANCH (gh pr list)
    GHCLI->>API: List PRs / query PRs
    API-->>GHCLI: PR number
    GHCLI-->>GHA: PR number

    GHA->>GHCLI: List comments on PR (gh pr view / gh api)
    GHCLI->>API: Get comments
    API-->>GHCLI: Comments list
    GHCLI-->>GHA: Comments data

    alt Identical comment exists
        GHA->>GHA: Skip posting (idempotent)
    else
        GHA->>GHCLI: Post comment on PR (gh api or gh pr comment)
        GHCLI->>API: Create comment
        API-->>GHCLI: Comment created
        GHCLI-->>GHA: Success
    end
Loading

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~45 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately and concisely summarizes the main change—reverting the PythonBot workflow to restore the original implementation, which is the core purpose of this PR.
Description check ✅ Passed The description clearly explains why the revert was necessary (workflow failures when posting PR comments), what testing was performed (tested in fork with successful workflow runs), and references the linked issue (#1825).
Linked Issues check ✅ Passed The PR directly addresses issue #1825 by reverting the PythonBot workflow to its original implementation, restoring stability after recent changes introduced failures.
Out of Scope Changes check ✅ Passed All changes are directly related to reverting the workflow: deletion of the new bot-workflows.js script, simplification of the bot-workflows.yml workflow definition, and documentation of the revert in CHANGELOG.md.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 5

Comment thread .github/workflows/bot-workflows.yml
Comment thread .github/workflows/bot-workflows.yml
Comment thread .github/workflows/bot-workflows.yml
Comment thread .github/workflows/bot-workflows.yml
Comment thread .github/workflows/bot-workflows.yml
@codecov

codecov Bot commented Feb 19, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1826   +/-   ##
=======================================
  Coverage   93.29%   93.29%           
=======================================
  Files         141      141           
  Lines        9118     9118           
=======================================
  Hits         8507     8507           
  Misses        611      611           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Signed-off-by: exploreriii <133720349+exploreriii@users.noreply.github.com>
Signed-off-by: exploreriii <133720349+exploreriii@users.noreply.github.com>
@exploreriii
exploreriii merged commit e75de9d into hiero-ledger:main Feb 21, 2026
20 checks passed
@exploreriii

Copy link
Copy Markdown
Contributor

Thank you! @Abhijeet2409

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.

Restore original PythonBot workflow failure notifier implementation

2 participants