fix(review): dismiss stale bot request changes

[Terry-Mao]

Summary

• map PR review REJECT/APPROVE follow-up behavior so stale bot-authored REQUEST_CHANGES reviews can be dismissed on later APPROVE
• add REVIEW_BOT_LOGIN wiring and docs for custom review bot identities
• cover dismiss behavior and workflow env wiring in delivered tests

Validation

• python3 -m unittest test_post_pr_review test_review_workflow_dispatch
• python3 -m unittest discover -s .github/aicodingflow-tests
• local review-pr-local: APPROVE

Refs #151

[github-actions]

Found 0 critical, 1 important, 0 suggestions. The new stale-review dismissal path needs a stricter author check before merge.

[github-actions]

Applied requested changes on feat/review-reject-pr-flow-151.

Summary:

Summary

• require stale request-changes dismissal to match both the configured review bot login and a bot identity
• add regression coverage so a configured human login is not eligible for stale bot review dismissal

Validation

• python3 -m unittest test_post_pr_review

#162

[Terry-Mao]

@kratos-ci-bot /review

[github-actions]

Found: 0 critical, 1 important, 0 suggestions. The stale-review cleanup should stay best-effort; otherwise an APPROVE run can fail after posting the review.

[github-actions]

Applied requested changes on feat/review-reject-pr-flow-151.

Summary:

Summary

• skip stale bot request-changes cleanup when listing PR reviews fails
• keep the current PR review publishing flow from failing on best-effort cleanup
• add regression coverage for transient review-listing failures

Validation

• python3 -m unittest discover -s .github/aicodingflow-tests -p test_post_pr_review.py

#162

[github-actions]

Found: 0 critical, 1 important, 0 suggestions. The stale request-changes dismissal can miss reviews because the GitHub reviews list is paginated; please fetch all review pages or request a sufficiently bounded complete set before deciding dismissal is done.

[github-actions]

No blocking findings. The stale bot REQUEST_CHANGES dismissal is scoped to later APPROVE verdicts for non-member code PRs, only dismisses reviews authored by the configured bot login, handles review-listing/dismissal failures without interrupting review posting, and the workflow/docs/tests cover the new REVIEW_BOT_LOGIN wiring.