build,meta: rearrange and narrow git ignore rules

PR-URL: https://github.com/nodejs/node/pull/27954
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>

.gitignore

@@ -1,9 +1,13 @@
-# Whitelist dotfiles
+# Please take extra care not to add rules that will make tracked files ignored.
+# To test that, run `git ls-files -i --exclude-standard`.
+# To check why a specific file is ignored, run `git check-ignore -v <filepath>`.
+
+# === Rule for dotfiles ===
+# Ignore all dotfiles.
 .*
+# Exclude specific dotfiles that we want to track.
 !deps/**/.*
 !test/fixtures/**/.*
-!tools/node_modules/**/.*
-!tools/doc/node_modules/**/.*
 !.clang-format
 !.editorconfig
 !.eslintignore
@@ -19,109 +23,108 @@
 !.eslintrc.yaml
 !.cpplint
 
-core
+# === Rules for root dir ===
-vgcore.*
+/core
-v8*.log
+/vgcore.*
-perf.data
+/v8*.log
-perf.data.old
+/perf.data
-tags
+/perf.data.old
-*.pyc
+/tags
-doc/api.xml
+/doc/api.xml
-tmp/
+/node
-test/tmp*/
+/node_g
-iojs
+/*.exe
-iojs_g
+/*.swp
-node
-node_g
-*.swp
-icu_config.gypi
-node_trace.*.log
-coverage/
-!**/node_modules/**/coverage
-
 /out
+/*.txt
 
-# various stuff that VC++ produces/uses and is not in /out
+# === Rules for artifacts of `./configure` ===
+/icu_config.gypi
+/config.gypi
+/config.status
+/config_fips.gypi
+
+# === Rules for MSVS artifacts ===
 /Debug
 /Release
-!doc/blog/**
 *.sln
-!nodemsi.sln
 *.suo
-*.vcxproj
+*.vcxproj*
-!custom_actions.vcxproj
-*.vcxproj.user
-*.vcxproj.filters
 UpgradeLog*.XML
 _UpgradeReport_Files/
 *.sdf
 *.opensdf
-*.VC.db
+*.VC.*
-*.VC.opendb
+*.wixobj
-/*.exe
-
-/config.mk
-/config.gypi
-/config.status
-/config_fips.gypi
-*-nodegyp*
-/gyp-mac-tool
-/npm.wxs
-/tools/msvs/npm.wixobj
 /tools/msvs/genfiles/
+/npm.wxs
+# Exclude MSVS files used for .msi file generation
+!tools/msvs/msi/custom_actions.vcxproj
+!tools/msvs/msi/nodemsi.sln
+
+# === Rules for GYP artifacts ===
+*-nodegyp*
+/gyp-*-tool
 /test/addons/??_*/
-email.md
+/*.mk
-deps/v8-*
+
-deps/icu
+# === Rules for other build artifacts ===
-deps/icu*.zip
+/email.md
-deps/icu*.tgz
+/deps/v8-*
-deps/icu-tmp
+/deps/icu
+/deps/icu*.zip
+/deps/icu*.tgz
+/deps/icu-tmp
 ./node_modules
-android-toolchain/
+/android-toolchain/
-
 # generated by gyp on Windows
-deps/openssl/openssl.props
+/deps/openssl/openssl.props
-deps/openssl/openssl.targets
+/deps/openssl/openssl.targets
-deps/openssl/openssl.xml
+/deps/openssl/openssl.xml
-
 # generated by gyp on android
 /*.target.mk
 /*.host.mk
-deps/openssl/openssl.target.mk
+/deps/openssl/openssl.target.mk
-deps/zlib/zlib.target.mk
+/deps/zlib/zlib.target.mk
 
-!deps/npm/node_modules
+# === Rules for release artifacts ===
-# not needed and causes issues for distro packagers
-deps/npm/node_modules/.bin/
-
-# build/release artifacts
 /*.tar.*
 /*.pkg
 /SHASUMS*.txt*
 
-# api docs artifacts
+# === Rules for `node_modules` ===
-tools/doc/node_modules
+!**/node_modules/**
+/node_modules
+/tools/doc/node_modules
+/tools/clang-format/node_modules
 
-# clang-format artifacts
+# === Rules for test artifacts ===
-tools/clang-format/node_modules
+/*.tap
-
-# test artifacts
-icu_config.gypi
-*.tap
 /*.xml
+/node_trace.*.log
+# coverage related
+/gcovr
+/build
 
-# Xcode workspaces and project folders
+# === Rules for XCode artifacts ===
 *.xcodeproj
 *.xcworkspace
+*.pbxproj
 
-# libuv book and GitHub template
+# === Rules for files in `/deps` ===
-deps/uv/.github/
+# Exclude all files in the vendored `npm`.
-deps/uv/docs/code/
+!/deps/npm/node_modules
-deps/uv/docs/src/guide/
+# These are not needed and causes issues for distro packagers.
-
+/deps/npm/node_modules/.bin/
-# do not override V8's .gitignore
+# Respect V8's .gitignore
 !deps/v8/**
-# ignore VS compiler output unhandled by V8's .gitignore
+# Ignore the libuv book and GitHub templates
-deps/v8/third_party/eu-strip/
+/deps/uv/.github/
+/deps/uv/docs/code/
+/deps/uv/docs/src/guide/
 
+# === Global Rules ===
+# Keep last to avoid being excluded
+*.pyc
+__pycache__
 .DS_Store