Skip to content

tools: refine contributor guidance workflow - #65745

Open
panva wants to merge 4 commits into
nodejs:mainfrom
panva:first-time-contributor-bot-token
Open

tools: refine contributor guidance workflow#65745
panva wants to merge 4 commits into
nodejs:mainfrom
panva:first-time-contributor-bot-token

Conversation

@panva

@panva panva commented Sep 2, 2026

Copy link
Copy Markdown
Member

Use the Node.js GitHub bot token for association checks and route the welcome and caution messages independently. Any outside contributor (other than dependabot) regardless of being associated as first time or not goes through agentscan (it's a signal we should surface to aid moderation).

Also labels either agentscan:community-flagged , agentscan:automated-account , or agentscan:mixed-signals accordingly.

Followup to #65592, #65533, and #65648. See #65648 (comment)

Use the Node.js GitHub bot token for association checks and route the
welcome and caution messages independently.

Signed-off-by: Filip Skokan <panva.ip@gmail.com>
@panva panva added the fast-track PRs proposed for a shorter-than-standard waiting period before landing. label Sep 2, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/actions

@nodejs-github-bot nodejs-github-bot added the meta Issues and PRs related to the general management of the project. label Sep 2, 2026
@panva panva removed the fast-track PRs proposed for a shorter-than-standard waiting period before landing. label Sep 2, 2026
@nodejs nodejs deleted a comment from github-actions Bot Sep 2, 2026

@mcollina mcollina left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm

@panva panva added the author ready PRs with CI started, the required approvals, and no outstanding review comments. label Sep 2, 2026
@MikeMcC399 MikeMcC399 removed their assignment Sep 2, 2026
@panva panva unassigned jasnell and trivikr Sep 2, 2026
Comment thread .github/workflows/contributor-guidance.yml
@panva

panva commented Sep 2, 2026

Copy link
Copy Markdown
Member Author

It's intentionally a bit wide at the moment and can be narrowed down when it runs a few times. The workflow is a new one that can be disabled if need be, it won't disrupt existing workflows.

flowchart TD
    opened["PR opened"] --> gate{"Free event pre-filter"}

    gate -->|"FIRST_TIMER / FIRST_TIME_CONTRIBUTOR / CONTRIBUTOR / NONE"| resolve["Resolve association using GH_USER_TOKEN"]
    gate -->|"Anything else, Dependabot, or a rerun"| skip["Skip without allocating a runner"]

    resolve -->|"FIRST_TIMER / FIRST_TIME_CONTRIBUTOR"| first["First-time contributor"]
    resolve -->|"CONTRIBUTOR / NONE after retries"| external["External contributor without a welcome"]
    resolve -->|"Trusted association"| stop["Stop without scanning or commenting"]
    resolve -->|"API error / unexpected value"| failure["Fail resolver; no scan or comment"]

    first --> firstScan["Run AgentScan"]
    firstScan -->|"Successful positive signal"| welcomeCaution["Apply labels and post welcome + caution"]
    firstScan -->|"No signal or scan failure"| welcome["Post welcome only"]

    external --> externalScan["Run AgentScan"]
    externalScan -->|"Successful positive signal"| caution["Apply labels and post caution only"]
    externalScan -->|"No signal or scan failure"| noComment["No comment"]
Loading

@bnb bnb 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.

love to see further AgentScan integration <3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs with CI started, the required approvals, and no outstanding review comments. meta Issues and PRs related to the general management of the project.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants