fix: set ROSETTA_MAX_WORKER_COUNT in pack.sh (#16738)

Remove global environment variables that are set in buildspec.yml files
as these interfere with more granular settings set in pack.sh, and
instead move all settings there.

The max heap size (8G) configured in pack.sh was overridden by the one
set in buildspec.yml because the last time the option is passed wins,
and pack.sh _prepends_ to `NODE_OPTIONS`.


----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*

buildspec-pr.yaml

@@ -2,12 +2,6 @@ version: 0.2
 
 # This buildspec is intended to be used by GitHub PR builds.
 
-env:
-  variables:
-    # Globally allow node to use a lot of memory
-    NODE_OPTIONS: --max-old-space-size=4096
-    JSII_ROSETTA_MAX_WORKER_COUNT: 8
-
 phases:
   install:
     commands:

buildspec.yaml

@@ -2,12 +2,6 @@ version: 0.2
 
 # This buildspec is intended to be run by CodePipeline builds.
 
-env:
-  variables:
-    # Globally allow node to use a lot of memory
-    NODE_OPTIONS: --max-old-space-size=4096
-    JSII_ROSETTA_MAX_WORKER_COUNT: 8
-
 phases:
   install:
     commands:

pack.sh

@@ -5,6 +5,7 @@
 set -eu
 export PATH=$PWD/node_modules/.bin:$PATH
 export NODE_OPTIONS="--max-old-space-size=8192 ${NODE_OPTIONS:-}"
+export JSII_ROSETTA_MAX_WORKER_COUNT="${JSII_ROSETTA_MAX_WORKER_COUNT:-8}"
 root=$PWD
 
 # Get version and changelog file name (these require that .versionrc.json would have been generated)