Skip to content
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
18 changes: 18 additions & 0 deletions .github/scripts/bot-gfi-assign-on-comment.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,9 @@
// Posts a comment if the issue is already assigned.
// All other validation and additional GFI comments are handled by other existing bots which can be refactored with time.

const fs = require('fs');
const path = require('path');

const GOOD_FIRST_ISSUE_LABEL = 'Good First Issue';
const UNASSIGNED_GFI_SEARCH_URL =
'https://github.com/hiero-ledger/hiero-sdk-python/issues?q=is%3Aissue%20state%3Aopen%20label%3A%22Good%20First%20Issue%22%20no%3Aassignee';
Expand Down Expand Up @@ -201,6 +204,21 @@ module.exports = async ({ github, context }) => {
});

console.log('[gfi-assign] Assignment completed successfully');

// Chain mentor assignment after successful GFI assignment
try {
const assignMentor = require('./bot-mentor-assignment.js');
await assignMentor({ github, context });
console.log('[gfi-assign] Mentor assignment chained successfully');
} catch (error) {
console.error('[gfi-assign] Mentor assignment failed but user assignment succeeded:', {
message: error.message,
status: error.status,
issueNumber: context.payload.issue?.number,
assignee: requesterUsername,
});
// Don't throw error - user assignment was successful
}
} catch (error) {
console.error('[gfi-assign] Error:', {
message: error.message,
Expand Down
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -87,6 +87,7 @@ This changelog is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.
- Added workflow to prevent assigning intermediate issues to contributors without prior Good First Issue completion (#1143).
- Added `Client.from_env()` and network-specific factory methods (e.g., `Client.for_testnet()`) to simplify client initialization and reduce boilerplate. [[#1251](https://github.com/hiero-ledger/hiero-sdk-python/issues/1251)]
- Improved unit test coverage for `TransactionId` class, covering parsing logic, hashing, and scheduled transactions.
- Chained Good First Issue assignment with mentor assignment to bypass GitHub's anti-recursion protection - mentor assignment now occurs immediately after successful user assignment in the same workflow execution. (#1369)

### Changed
- Added global review instructions to CodeRabbit configuration to limit reviews to issue/PR scope and prevent scope creep [#1373]
Expand Down
Loading