Skip to content

[Intermediate]: Exclude team members to get notified with reminder to /assign on Good First Issues #1367

@exploreriii

Description

@exploreriii

🧩 Intermediate Contributors

This issue is intended for contributors who already have some familiarity with the
Hiero Python SDK codebase and contribution workflow.

You should feel comfortable:

  • navigating existing source code and examples
  • understanding SDK concepts without step-by-step guidance
  • following the standard PR workflow without additional onboarding

If this is your very first contribution to the project, we recommend starting with a few
Good First Issues before working on this one.

🐞 Problem Description

Anyone posting under a good first issue, saying something other than /assign, gets a reminder to ask to get /assign

function buildAssignReminder(username) {
return `${ASSIGN_REMINDER_MARKER}
👋 Hi @${username}!

If you’d like to work on this Good First Issue, just comment:

/assign

and you’ll be automatically assigned. Feel free to ask questions here if anything is unclear!`;
}

We should not send the message if the user is a team member.

💡 Expected Solution

Disable the comment for assign reminder if the user is a team member (has triage, write) permissions.

Don't post in that case.

🧠 Implementation Notes

We have similar examples in other workflows that document how to do this

Edit
.github/scripts/bot-gfi-assign-on-comment.js

For the reminder, line 70 which is called in line 147
add a check before
if in team member, skip

team member can be identified

github.rest.repos.checkCollaborator({
  owner,
  repo,
  username,
});

and checking it returns 204

then craft some kind of skip when they are team members:

const isTeamMember = await isRepoCollaborator({
    github,
    owner,
    repo,
    username: comment.user.login,
});

if (isTeamMember) {
    console.log('[gfi-assign] Skip reminder: commenter is collaborator');
    return;
}

Once this is completed, please test the issue on your fork.
Read:
docs/sdk_developers/training/testing_forks.md

Merge your fix to your main on your fork
Enable issues in settings
Create an issue label called 'Good First Issue'
create a good first issue
post "i want to be assigned"
wait to see if you get the assigned reminder (you shouldnt you'll be a team member on your fork)

create a second test, ask a friend to comment /assign, they should get an assigned reminder

✅ Acceptance Criteria

To merge this issue, the pull request must:

  • Fully address the problem described above
  • Follow existing project conventions and patterns
  • Include tests or example updates where appropriate
  • Pass all CI checks
  • Include a valid changelog entry
  • Use DCO and GPG-signed commits

📚 Additional Context or Resources

Metadata

Metadata

Assignees

No one assigned

    Labels

    github_actionsPull requests that update Github Actions code.intermediaterequires some knowledge of the codebase with some defined steps to implement or examplesjavascriptUses Javascript programming language

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions