Automattic · zinigor · Nov 14, 2023 · Oct 26, 2023 · Oct 26, 2023 · Oct 27, 2023
diff --git a/.github/files/lint-project-structure.sh b/.github/files/lint-project-structure.sh
@@ -72,8 +72,6 @@ for PROJECT in projects/*; do
 	fi
 done
 
-ROOT_PACKAGE_JSON_ENGINES="$(jq '.engines | .pnpm |= empty' package.json)"
-
 for PROJECT in projects/*/*; do
 	SLUG="${PROJECT#projects/}"
 	TYPE="${SLUG%/*}"
@@ -97,25 +95,6 @@ for PROJECT in projects/*/*; do
 		echo "::error file=$PROJECT/.gitattributes::$PROJECT/.github/ should have git attribute export-ignore."
 	fi
 
-	# - package.json engines should match monorepo root package.json engines
-	if [[ -e "$PROJECT/package.json" ]]; then
-		PACKAGE_JSON_ENGINES="$(jq '.engines' "$PROJECT/package.json")"
-		if [[ "$PACKAGE_JSON_ENGINES" != "$ROOT_PACKAGE_JSON_ENGINES" ]]; then
-			EXIT=1
-			LINE=$(jq --stream --arg obj "$PACKAGE_JSON_ENGINES" 'if length == 1 then .[0][:-1] else .[0] end | if . == ["engines"] then input_line_number - ( $obj | gsub( "[^\n]"; "" ) | length ) else empty end' "$PROJECT/package.json")
-			if [[ -n "$LINE" ]]; then
-				echo "---" # Bracket message containing newlines for better visibility in GH's logs.
-				echo "::error file=$PROJECT/package.json,line=$LINE::Engines must match those in the monorepo root package.json (but omitting \"pnpm\").%0A  \"engines\": ${ROOT_PACKAGE_JSON_ENGINES//$'\n'/%0A  }"
-				echo "---"
-			else
-				LINE=$(wc -l < "$PROJECT/package.json")
-				echo "---" # Bracket message containing newlines for better visibility in GH's logs.
-				echo "::error file=$PROJECT/package.json,line=$LINE::Engines must be specified, matching those in the monorepo root package.json (but omitting \"pnpm\").%0A  \"engines\": ${ROOT_PACKAGE_JSON_ENGINES//$'\n'/%0A  }"
-				echo "---"
-			fi
-		fi
-	fi
-
 	# - package.json for js modules should look like a library to renovate.
 	if [[ "$PROJECT" == projects/js-packages/* && -e "$PROJECT/package.json" ]]; then
 		! IFS= read -r INDEX < <( ls -- "$PROJECT"/index.{js,jsx,cjs,mjs,ts,tsx,d.ts} 2>/dev/null )

diff --git a/.github/versions.sh b/.github/versions.sh
@@ -1,7 +1,7 @@
 # Versions of tools to use in CI.
 PHP_VERSION=8.2
 COMPOSER_VERSION=2.6.5
-NODE_VERSION=18.17.0
+NODE_VERSION=20.8.1
 PNPM_VERSION=8.6.8
 
 # Other useful version numbers.

diff --git a/.nvmrc b/.nvmrc
@@ -1 +1 @@
-18.17.0
+20.8.1
diff --git a/docs/monorepo.md b/docs/monorepo.md
@@ -311,7 +311,6 @@ See PCYsg-xsv-p2#mirror-repo-secrets for details.
 
 Note the following will also be done by the build process:
 
-* In `package.json`, the `.engines` will be deleted. If there is a `.publish_engines`, it will be renamed to `.engines`.
 * Entries will be prepended to `.npmignore` to ignore `.github` and `composer.json` during the NPM publish. This file will be created if not present.
 
 Before you create the first release tag, you may want to check out the mirror and run `npm publish --dry-run` to ensure that only the files you want published will be published.

diff --git a/docs/yarn-upgrade.md b/docs/yarn-upgrade.md
diff --git a/package.json b/package.json
@@ -34,8 +34,7 @@
 		"jetpack-js-tools": "workspace:*"
 	},
 	"engines": {
-		"node": "^18.13.0",
-		"pnpm": "^8.6.2",
-		"yarn": "use pnpm instead - see docs/yarn-upgrade.md"
+		"node": "^20.8.1",
+		"pnpm": "^8.6.2"
 	}
 }
diff --git a/projects/github-actions/repo-gardening/changelog/update-node-20 b/projects/github-actions/repo-gardening/changelog/update-node-20
@@ -0,0 +1,5 @@
+Significance: patch
+Type: removed
+Comment: Remove "engines" from package.json. No change to the project itself; note the build formerly removed this before mirroring.
+
+
diff --git a/projects/github-actions/repo-gardening/package.json b/projects/github-actions/repo-gardening/package.json
@@ -25,9 +25,5 @@
 	},
 	"scripts": {
 		"build": "ncc build src/index.js -o dist --source-map --license licenses.txt"
-	},
-	"engines": {
-		"node": "^18.13.0",
-		"yarn": "use pnpm instead - see docs/yarn-upgrade.md"
 	}
 }
diff --git a/projects/github-actions/required-review/changelog/update-node-20 b/projects/github-actions/required-review/changelog/update-node-20
@@ -0,0 +1,5 @@
+Significance: patch
+Type: removed
+Comment: Remove "engines" from package.json. No change to the project itself; note the build formerly removed this before mirroring.
+
+
diff --git a/projects/github-actions/required-review/package.json b/projects/github-actions/required-review/package.json
@@ -17,9 +17,5 @@
 	},
 	"scripts": {
 		"build": "ncc build src/main.js -o dist --source-map --license licenses.txt"
-	},
-	"engines": {
-		"node": "^18.13.0",
-		"yarn": "use pnpm instead - see docs/yarn-upgrade.md"
 	}
 }
diff --git a/projects/github-actions/test-results-to-slack/changelog/update-node-20 b/projects/github-actions/test-results-to-slack/changelog/update-node-20
@@ -0,0 +1,5 @@
+Significance: patch
+Type: removed
+Comment: Remove "engines" from package.json. No change to the project itself; note the build formerly removed this before mirroring.
+
+
diff --git a/projects/github-actions/test-results-to-slack/package.json b/projects/github-actions/test-results-to-slack/package.json
@@ -28,9 +28,5 @@
 	"scripts": {
 		"build": "ncc build src/index.js -o dist --source-map --license licenses.txt",
 		"test": "jest --config=tests/jest.config.js --verbose --runInBand"
-	},
-	"engines": {
-		"node": "^18.13.0",
-		"yarn": "use pnpm instead - see docs/yarn-upgrade.md"
 	}
 }
diff --git a/projects/js-packages/ai-client/changelog/update-node-20 b/projects/js-packages/ai-client/changelog/update-node-20
@@ -0,0 +1,5 @@
+Significance: patch
+Type: removed
+Comment: Remove "engines" from package.json. No change to the project itself; note the build formerly removed this before mirroring.
+
+
diff --git a/projects/js-packages/ai-client/package.json b/projects/js-packages/ai-client/package.json
@@ -28,10 +28,6 @@
 		"jest-environment-jsdom": "29.5.0",
 		"typescript": "5.0.4"
 	},
-	"engines": {
-		"node": "^18.13.0",
-		"yarn": "use pnpm instead - see docs/yarn-upgrade.md"
-	},
 	"exports": {
 		".": "./index.ts"
 	},

diff --git a/projects/js-packages/analytics/changelog/update-node-20 b/projects/js-packages/analytics/changelog/update-node-20
@@ -0,0 +1,5 @@
+Significance: patch
+Type: removed
+Comment: Remove "engines" from package.json. No change to the project itself; note the build formerly removed this before mirroring.
+
+
diff --git a/projects/js-packages/analytics/package.json b/projects/js-packages/analytics/package.json
@@ -1,6 +1,6 @@
 {
 	"name": "@automattic/jetpack-analytics",
-	"version": "0.1.27",
+	"version": "0.1.28-alpha",
 	"description": "Jetpack Analytics Package",
 	"homepage": "https://github.com/Automattic/jetpack/tree/HEAD/projects/js-packages/analytics/#readme",
 	"bugs": {
@@ -25,9 +25,5 @@
 	},
 	"scripts": {
 		"test": "NODE_OPTIONS=--experimental-vm-modules jest"
-	},
-	"engines": {
-		"node": "^18.13.0",
-		"yarn": "use pnpm instead - see docs/yarn-upgrade.md"
 	}
 }
diff --git a/projects/js-packages/api/changelog/update-node-20 b/projects/js-packages/api/changelog/update-node-20
@@ -0,0 +1,5 @@
+Significance: patch
+Type: removed
+Comment: Remove "engines" from package.json. No change to the project itself; note the build formerly removed this before mirroring.
+
+
diff --git a/projects/js-packages/api/package.json b/projects/js-packages/api/package.json
@@ -27,9 +27,5 @@
 	},
 	"scripts": {
 		"test": "NODE_OPTIONS=--experimental-vm-modules jest"
-	},
-	"engines": {
-		"node": "^18.13.0",
-		"yarn": "use pnpm instead - see docs/yarn-upgrade.md"
 	}
 }
diff --git a/projects/js-packages/babel-plugin-replace-textdomain/changelog/update-node-20 b/projects/js-packages/babel-plugin-replace-textdomain/changelog/update-node-20
@@ -0,0 +1,5 @@
+Significance: patch
+Type: removed
+Comment: Remove "engines" from package.json. No change to the project itself; note the build formerly removed this before mirroring.
+
+
diff --git a/projects/js-packages/babel-plugin-replace-textdomain/package.json b/projects/js-packages/babel-plugin-replace-textdomain/package.json
@@ -1,6 +1,6 @@
 {
 	"name": "@automattic/babel-plugin-replace-textdomain",
-	"version": "1.0.30",
+	"version": "1.0.31-alpha",
 	"description": "A Babel plugin to replace the textdomain in gettext-style function calls.",
 	"homepage": "https://github.com/Automattic/jetpack/tree/HEAD/projects/js-packages/babel-plugin-replace-textdomain/#readme",
 	"bugs": {
@@ -24,10 +24,6 @@
 		"babel-plugin-tester": "11.0.4",
 		"jest": "29.4.3"
 	},
-	"engines": {
-		"node": "^18.13.0",
-		"yarn": "use pnpm instead - see docs/yarn-upgrade.md"
-	},
 	"exports": {
 		".": "./src/index.js"
 	}

diff --git a/projects/js-packages/base-styles/changelog/update-node-20 b/projects/js-packages/base-styles/changelog/update-node-20
@@ -0,0 +1,5 @@
+Significance: patch
+Type: removed
+Comment: Remove "engines" from package.json. No change to the project itself; note the build formerly removed this before mirroring.
+
+
diff --git a/projects/js-packages/base-styles/package.json b/projects/js-packages/base-styles/package.json
@@ -21,9 +21,5 @@
 	},
 	"devDependencies": {
 		"@wordpress/base-styles": "4.36.0"
-	},
-	"engines": {
-		"node": "^18.13.0",
-		"yarn": "use pnpm instead - see docs/yarn-upgrade.md"
 	}
 }
diff --git a/projects/js-packages/boost-score-api/changelog/update-node-20 b/projects/js-packages/boost-score-api/changelog/update-node-20
@@ -0,0 +1,5 @@
+Significance: patch
+Type: removed
+Comment: Remove "engines" from package.json. No change to the project itself; note the build formerly removed this before mirroring.
+
+
diff --git a/projects/js-packages/boost-score-api/package.json b/projects/js-packages/boost-score-api/package.json
@@ -34,10 +34,6 @@
 		"webpack": "5.76.0",
 		"webpack-cli": "4.9.1"
 	},
-	"engines": {
-		"node": "^18.13.0",
-		"yarn": "use pnpm instead - see docs/yarn-upgrade.md"
-	},
 	"exports": {
 		".": {
 			"jetpack:src": "./src/index.ts",

diff --git a/projects/js-packages/components/changelog/update-node-20 b/projects/js-packages/components/changelog/update-node-20
@@ -0,0 +1,5 @@
+Significance: patch
+Type: removed
+Comment: Remove "engines" from package.json. No change to the project itself; note the build formerly removed this before mirroring.
+
+
diff --git a/projects/js-packages/components/package.json b/projects/js-packages/components/package.json
@@ -82,9 +82,5 @@
 		"build": "pnpm run compile-ts",
 		"compile-ts": "tsc --pretty",
 		"test": "NODE_OPTIONS=--experimental-vm-modules jest"
-	},
-	"engines": {
-		"node": "^18.13.0",
-		"yarn": "use pnpm instead - see docs/yarn-upgrade.md"
 	}
 }
diff --git a/projects/js-packages/config/changelog/update-node-20 b/projects/js-packages/config/changelog/update-node-20
@@ -0,0 +1,5 @@
+Significance: patch
+Type: removed
+Comment: Remove "engines" from package.json. No change to the project itself; note the build formerly removed this before mirroring.
+
+
diff --git a/projects/js-packages/config/package.json b/projects/js-packages/config/package.json
@@ -1,6 +1,6 @@
 {
 	"name": "@automattic/jetpack-config",
-	"version": "0.1.22",
+	"version": "0.1.23-alpha",
 	"description": "Handles Jetpack global configuration shared across all packages",
 	"homepage": "https://github.com/Automattic/jetpack/tree/HEAD/projects/js-packages/config/#readme",
 	"bugs": {
@@ -20,10 +20,6 @@
 		"jest": "29.4.3",
 		"jest-environment-jsdom": "29.4.3"
 	},
-	"engines": {
-		"node": "^18.13.0",
-		"yarn": "use pnpm instead - see docs/yarn-upgrade.md"
-	},
 	"exports": {
 		".": "./src/index.js"
 	},

diff --git a/projects/js-packages/connection/changelog/update-node-20 b/projects/js-packages/connection/changelog/update-node-20
@@ -0,0 +1,5 @@
+Significance: patch
+Type: removed
+Comment: Remove "engines" from package.json. No change to the project itself; note the build formerly removed this before mirroring.
+
+
diff --git a/projects/js-packages/connection/package.json b/projects/js-packages/connection/package.json
@@ -58,9 +58,5 @@
 	],
 	"scripts": {
 		"test": "NODE_OPTIONS=--experimental-vm-modules jest"
-	},
-	"engines": {
-		"node": "^18.13.0",
-		"yarn": "use pnpm instead - see docs/yarn-upgrade.md"
 	}
 }
diff --git a/projects/js-packages/eslint-changed/changelog/update-node-20 b/projects/js-packages/eslint-changed/changelog/update-node-20
@@ -0,0 +1,5 @@
+Significance: patch
+Type: removed
+Comment: Remove "engines" from package.json. No change to the project itself; note the build formerly removed this before mirroring.
+
+
diff --git a/projects/js-packages/eslint-changed/package.json b/projects/js-packages/eslint-changed/package.json
@@ -1,6 +1,6 @@
 {
 	"name": "@automattic/eslint-changed",
-	"version": "2.0.7",
+	"version": "2.0.8-alpha",
 	"description": "Run eslint on files, but only report warnings and errors from lines that were changed.",
 	"homepage": "https://github.com/Automattic/jetpack/tree/HEAD/projects/js-packages/eslint-changed/#readme",
 	"type": "module",
@@ -30,9 +30,5 @@
 	},
 	"peerDependencies": {
 		"eslint": ">=7.0.0"
-	},
-	"engines": {
-		"node": "^18.13.0",
-		"yarn": "use pnpm instead - see docs/yarn-upgrade.md"
 	}
 }
diff --git a/projects/js-packages/eslint-changed/src/cli.js b/projects/js-packages/eslint-changed/src/cli.js
@@ -6,7 +6,7 @@ import { Command } from 'commander';
 import { ESLint } from 'eslint';
 import parseDiff from 'parse-diff';
 
-const APP_VERSION = '2.0.7';
+const APP_VERSION = '2.0.8-alpha';
 
 /**
  * Create a Commander instance.

diff --git a/projects/js-packages/eslint-config-target-es/changelog/update-node-20 b/projects/js-packages/eslint-config-target-es/changelog/update-node-20
@@ -0,0 +1,5 @@
+Significance: patch
+Type: removed
+Comment: Remove "engines" from package.json. No change to the project itself; note the build formerly removed this before mirroring.
+
+