Skip to content

doc: refactor the AI guidelines - #65269

Merged
nodejs-github-bot merged 3 commits into
nodejs:mainfrom
joyeecheung:consolidate-ai-guidelines
Aug 28, 2026
Merged

doc: refactor the AI guidelines#65269
nodejs-github-bot merged 3 commits into
nodejs:mainfrom
joyeecheung:consolidate-ai-guidelines

Conversation

@joyeecheung

@joyeecheung joyeecheung commented Aug 13, 2026

Copy link
Copy Markdown
Member

The earlier version of the document contains duplications and has become logically disconnected after iteration. This patch wordsmiths it a bit to make it more coherent:

  • Merge policy points that have been repeated in the guidelines into the opening policy (e.g. as pointed out in doc: create ai-guidelines and include to CONTRIBUTING #62105 (comment))
  • Remove redundant taglines.
  • Reword the paragraphs so that they appear more logical:
    -> decision making and responsibility is human-only
    -> AI assistance is not prohibited
    -> disclosure and licensing
    -> verification and review requirement
    -> other off limits items (automation, good first issues)
  • Reword the guidelines to contextualize the suggestions with the use of AI instead of being generic.
  • Change the tone of the document to be more in line with other documents we have (e.g. avoid being too imperative or calling out "you" constantly)

The earlier version of the document contains duplications and
has become logically disconnected after iteration. This patch
wordsmiths it a bit to make it more coherent:

- Merge policy points that have been repeated in the guidelines
  into the opening policy.
- Remove redundant taglines.
- Reword the paragraphs so that they appear more logical:
  -> decision making and accountability is human-only
  -> AI assistance is not prohibited
  -> disclosure and licensing
  -> verification and review requirement
  -> other off limits items (automation, good first issues)
- Reword the guidelines to contextualize the suggestions with
  the use of AI instead of being generic.
- Change the tone of the document to be more in line with other
  documents we have (e.g. avoid being too imperative or calling
  out "you" constantly)

Signed-off-by: Joyee Cheung <joyeec9h3@gmail.com>
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/tsc

@nodejs-github-bot nodejs-github-bot added the doc Issues and PRs related to Node.js documentation. label Aug 13, 2026
Comment thread doc/contributing/ai-guidelines.md Outdated
@joyeecheung
joyeecheung force-pushed the consolidate-ai-guidelines branch from 4b185d4 to 4783c93 Compare August 13, 2026 22:16
Comment thread doc/contributing/ai-guidelines.md Outdated
Comment thread doc/contributing/ai-guidelines.md Outdated
@joyeecheung

Copy link
Copy Markdown
Member Author

@legendecas @mcollina I think I've addressed your comments. Can you take a look again? Thanks!

@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 anyway

Comment thread doc/contributing/ai-guidelines.md
@joyeecheung joyeecheung added commit-queue PRs queued for automated landing through the Commit Queue. commit-queue-squash PRs the Commit Queue should land as one squashed commit. labels Aug 28, 2026
@nodejs-github-bot
nodejs-github-bot merged commit 4215cc3 into nodejs:main Aug 28, 2026
22 checks passed
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Landed in 4215cc3

@nodejs-github-bot nodejs-github-bot removed the commit-queue PRs queued for automated landing through the Commit Queue. label Aug 28, 2026
aduh95 pushed a commit that referenced this pull request Aug 29, 2026
The earlier version of the document contains duplications and
has become logically disconnected after iteration. This patch
wordsmiths it a bit to make it more coherent:

- Merge policy points that have been repeated in the guidelines
  into the opening policy.
- Remove redundant taglines.
- Reword the paragraphs so that they appear more logical:
  -> decision making and accountability is human-only
  -> AI assistance is not prohibited
  -> disclosure and licensing
  -> verification and review requirement
  -> other off limits items (automation, good first issues)
- Reword the guidelines to contextualize the suggestions with
  the use of AI instead of being generic.
- Change the tone of the document to be more in line with other
  documents we have (e.g. avoid being too imperative or calling
  out "you" constantly)

Signed-off-by: Joyee Cheung <joyeec9h3@gmail.com>
PR-URL: #65269
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Aviv Keller <me@aviv.sh>
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
Reviewed-By: Filip Skokan <panva.ip@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

commit-queue-squash PRs the Commit Queue should land as one squashed commit. doc Issues and PRs related to Node.js documentation.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants