Sitelet https://github.com/nodejs/node/pull/65533
Skip to content

tools: welcome first-time contributors - #65533

Open
panva wants to merge 3 commits into
nodejs:mainfrom
panva:first-time-contributor-welcome
Open

tools: welcome first-time contributors#65533
panva wants to merge 3 commits into
nodejs:mainfrom
panva:first-time-contributor-welcome

Conversation

@panva

@panva panva commented Aug 25, 2026

Copy link
Copy Markdown
Member

Add contribution guidance and an advisory AgentScan warning for first-time pull request authors.

@panva
tools: welcome first-time contributors
1ee89c0 
Add contribution guidance and an advisory AgentScan warning for
first-time pull request authors.

Signed-off-by: Filip Skokan <panva.ip@gmail.com>
@panva panva added discuss Issues opened for discussions and feedbacks. tools Issues and PRs related to the tools directory. labels Aug 25, 2026
@nodejs-github-bot

nodejs-github-bot commented Aug 25, 2026

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 Aug 25, 2026
jasnell
jasnell approved these changes Aug 25, 2026
View reviewed changes

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

Worth a try to see if it's effective

@panva panva added the author ready PRs that have at least one approval, no outstanding review comments, and a CI started. label Aug 25, 2026
MikeMcC399

This comment was marked as resolved.

Sign in to view

@panva

panva commented Aug 25, 2026 •
edited
Loading

Copy link
Copy Markdown
Member Author

Perhaps flagging multiple PRs from FIRST_TIME_CONTRIBUTOR could be added to this workflow or to a follow-on change?

It can be a prose in the text. I wouldn't run expensive policy queries in this workflow. Please make a suggestion.

@MikeMcC399

MikeMcC399 commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Perhaps flagging multiple PRs from FIRST_TIME_CONTRIBUTOR could be added to this workflow or to a follow-on change?

It can be a prose in the text. I wouldn't run expensive policy queries in this workflow. Please make a suggestion.

Thanks for the response! The text is fine for the first contribution, and if they have read the guidance, they shouldn't be submitting multiple PRs anyway.

I hadn't considered the cost of additional queries and I understand your concern.

So I'm not going to suggest a change after all!

@panva panva added commit-queue Add this label to land a pull request using GitHub Actions. and removed commit-queue Add this label to land a pull request using GitHub Actions. labels Aug 25, 2026
@Renegade334

Renegade334 commented Aug 25, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

Example run: nodejs/node-auto-test#100 (comment)

Example with positive agentscan: nodejs/node-auto-test#101 (comment)
@panva I think the indentation looks a bit janky?

(sorry for accidentally closing, was on the wrong tab 😞)

@Renegade334 Renegade334 reopened this Aug 25, 2026
@panva

panva commented Aug 25, 2026

Copy link
Copy Markdown
Member Author

Example with positive agentscan: nodejs/node-auto-test#101 (comment)

I'll reflow the caution block

@panva

panva commented Aug 25, 2026

Copy link
Copy Markdown
Member Author

@Renegade334 can you test again?

@Renegade334

Renegade334 commented Aug 25, 2026

Copy link
Copy Markdown
Member

nodejs/node-auto-test#102 (comment)

LGTM!

@panva panva added the commit-queue Add this label to land a pull request using GitHub Actions. label Aug 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jasnell jasnell jasnell approved these changes
@legendecas legendecas legendecas approved these changes
@Renegade334 Renegade334 Renegade334 approved these changes
@MikeMcC399 MikeMcC399 MikeMcC399 approved these changes

Assignees

No one assigned

Labels

author ready PRs that have at least one approval, no outstanding review comments, and a CI started. commit-queue Add this label to land a pull request using GitHub Actions. discuss Issues opened for discussions and feedbacks. meta Issues and PRs related to the general management of the project. tools Issues and PRs related to the tools directory.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@panva @nodejs-github-bot @MikeMcC399 @Renegade334 @jasnell @legendecas