(#8667) - test-webpack: fail if git working tree is dirty

Co-authored-by: alxndrsn <alxndrsn>

Author: alxndrsn

.github/workflows/ci.yml

@@ -266,8 +266,8 @@ jobs:
       - name: First retry
         id: retry
         if: steps.test.outcome == 'failure'
-        run: ${{ matrix.cmd }}
+        run: git reset --hard && ${{ matrix.cmd }}
         continue-on-error: true
       - name: Second retry
         if: steps.retry.outcome == 'failure'
-        run: ${{ matrix.cmd }}
+        run: git reset --hard && ${{ matrix.cmd }}

bin/test-webpack.sh

@@ -6,6 +6,22 @@
 # Webpack and Browserify to justify it.
 #
 
+# If this script is run _after_ bin/update-package-json-for-publish.js is run,
+# `npm run build` may fail with:
+#
+# > Error: 'default' is not exported by node_modules/inherits/inherits.js
+#
+# To avoid this, fail if this script is run in a non-clean git repo:
+git_diff="$(git diff -- package.json packages/node_modules/*/package.json)"
+if [[ "$git_diff" != "" ]]; then
+  git status --untracked-files=no -- package.json packages/node_modules/*/package.json
+  echo "!!!"
+  echo "!!! Your git working directory has changes to package.json file(s) !!!"
+  echo "!!! Please revert/stage/commit changes, and re-run the command !!!"
+  echo "!!!"
+  exit 1
+fi
+
 npm run build
 npm i webpack@5.66.0 webpack-cli@4.9.2 # do this on-demand to avoid slow installs
 node bin/update-package-json-for-publish.js