Skip to content

Commit cee1884

Browse files
RafaelGSSAditi-1400joyeecheung
authored
Apply suggestions from code review
Co-authored-by: Aditi <62544124+Aditi-1400@users.noreply.github.com> Co-authored-by: Joyee Cheung <joyeec9h3@gmail.com>
1 parent 8fa167d commit cee1884

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

doc/contributing/ai-guidelines.md

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,10 @@ When using AI as a coding assistant:
2121

2222
* **Understand the codebase first.** Do not skip familiarizing yourself with
2323
the relevant subsystem. LLMs frequently produce inaccurate descriptions of
24-
Node.js internals — always verify against the actual source.
24+
Node.js internals — always verify against the actual source. When using an AI
25+
tool, ask it to cite the exact source files/PRs/docs it’s relying on, and then
26+
match the claim against that resource to verify if it holds up in the current
27+
code.
2528

2629
* **Own every line you submit.** You are responsible for all code in your
2730
pull request, regardless of how it was generated. Be prepared to explain
@@ -32,7 +35,8 @@ When using AI as a coding assistant:
3235
[commit message guidelines][].
3336

3437
* **Test thoroughly.** AI-generated code must pass the full test suite and
35-
any manually written tests relevant to the change. Do not rely on the LLM
38+
any manually written tests relevant to the change. Existing Tests should not
39+
be removed or modified without human verification. Do not rely on the LLM
3640
to assess correctness.
3741

3842
* **Edit generated comments critically.** LLM-produced comments are often

0 commit comments

Comments
 (0)