Auto-generated tile from GitHub (10 skills)
92
94%
Does it follow best practices?
Impact
92%
1.16xAverage score across 44 eval scenarios
Advisory
Suggest reviewing before use
A contributor has been working on a fix for a bug in Node.js's fs module: fs.readdir() was not properly handling the recursive option on certain Linux kernel versions when the directory being scanned contained symbolic links pointing outside the scanned tree. Additionally, a follow-up change updates the documentation to clarify the behavior of the recursive option.
The contributor made the changes but is not familiar with Node.js's commit message style requirements and needs the commit messages properly formatted before submitting a pull request. The Node.js project has strict commit message conventions and validates them automatically.
Your task is to prepare properly formatted commit messages for these two changes, validate them, and document the process.
The changes being committed are:
Change 1 (Bug fix): Modified lib/fs.js and src/node_file.cc to correctly handle symbolic links when recursive: true is used with fs.readdir() on Linux. The fix avoids following symlinks that resolve outside the original directory tree. This closes GitHub issue #52341.
Change 2 (Documentation): Updated doc/api/fs.md to document that fs.readdir() with recursive: true does not follow symbolic links that point outside the scanned directory on any platform, and added a note about the behavior difference with fs.readdirSync(). This is related to pull request #52342.
Produce the following files:
commit-messages.txt — Contains both properly formatted commit messages, separated by a blank line and a --- divider between them.validation-output.txt — The output from running the commit message validator on each message. Run the validator on each message and save all output to this file.process-notes.md — A brief explanation of the commit message format you used and how you validated the messages.evals
scenario-1
scenario-2
scenario-3
scenario-4
scenario-5
scenario-6
scenario-7
scenario-8
scenario-9
scenario-10
scenario-11
scenario-12
scenario-13
scenario-14
scenario-15
scenario-16
scenario-17
scenario-18
scenario-19
scenario-20
scenario-21
scenario-22
scenario-23
scenario-24
scenario-25
scenario-26
scenario-27
scenario-28
scenario-29
scenario-30
scenario-31
scenario-32
scenario-33
scenario-34
scenario-35
scenario-36
scenario-37
scenario-38
scenario-39
scenario-40
scenario-41
scenario-42
scenario-43
scenario-44
skills
documentation
fastify
init
linting-neostandard-eslint9
node
nodejs-core
rules
oauth
octocat
snipgrapher