chore(aws-cdk-lib): Run pre and post package steps for aws-cdk-lib from pack.sh (aws#25239)

Until this PR, the pre and post package steps defined in `aws-cdk-lib`'s `package.json` were not actually executed during the pack step of the pipeline. The ultimate purpose of this change is to make sure that the `.js` source files are minified in the release pipeline, resulting in a smaller overall package size. 

----

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

Author: madeline-k

pack.sh

@@ -38,16 +38,24 @@ function lerna_scopes() {
 
 scripts/run-rosetta.sh --infuse --pkgs-from $TMPDIR/jsii.txt
 
+# Execute any pre-package steps for the jsii modules here:
+echo "Running aws-cdk-lib pre-package"
+npx lerna run --scope aws-cdk-lib package -- --pre-only
+
 # Jsii packaging (all at once using jsii-pacmak)
 echo "Packaging jsii modules" >&2
 $PACMAK \
   --verbose \
   $(cat $TMPDIR/jsii.txt)
 
+# Execute any post-package steps for the jsii modules here:
+echo "Running aws-cdk-lib post-package"
+npx lerna run --scope aws-cdk-lib package -- --post-only
+
 # Non-jsii packaging, which means running 'package' in every individual
 # module
 echo "Packaging non-jsii modules" >&2
-lerna run $(lerna_scopes $(cat $TMPDIR/nonjsii.txt)) --sort --concurrency=1 --stream package
+npx lerna run $(lerna_scopes $(cat $TMPDIR/nonjsii.txt)) --sort --concurrency=1 --stream package
 
 # Finally rsync all 'dist' directories together into a global 'dist' directory
 for dir in $(find packages -name dist | grep -v node_modules | grep -v run-wrappers); do

packages/aws-cdk-lib/package.json

@@ -56,8 +56,12 @@
     ]
   },
   "cdk-package": {
-    "pre": "/bin/bash ./scripts/minify-sources.sh",
-    "post": "node ./scripts/verify-stripped-exp.js"
+    "pre": [
+      "/bin/bash ./scripts/minify-sources.sh"
+    ],
+    "post": [
+      "ts-node ./scripts/verify-stripped-exp.ts"
+    ]
   },
   "pkglint": {
     "exclude": [

scripts/check-pack-prerequisites.sh

@@ -52,8 +52,8 @@ app_min="1.8.0"
 check_which $app $app_min
 app_v=$(${app} -version 2>&1)
 echo -e "Checking javac version... \c"
-# 1.8
-if [ $(echo $app_v | grep -c -E "1\.8\.[0-9].*") -eq 1 ]
+# javac >= 1.8
+if [ $(echo $app_v | grep -c -E "1\.[89]\.[0-9].*") -eq 1 ] || [ $(echo $app_v | grep -c -E "[2-9]\.[0-9]+\.[0-9]+.*") -eq 1 ] || [ $(echo $app_v | grep -c -E "[1-9][0-9]+\.[0-9]+\.[0-9]+.*") -eq 1 ]
 then
     echo "Ok"
 else

tools/@aws-cdk/cdk-build-tools/bin/cdk-package.ts

@@ -22,10 +22,24 @@ async function main() {
       default: require.resolve('jsii-pacmak/bin/jsii-pacmak'),
       defaultDescription: 'jsii-pacmak provided by node dependencies',
     })
+    .option('pre-only', { type: 'boolean', default: false, desc: 'run pre package steps only' })
+    .option('post-only', { type: 'boolean', default: false, desc: 'run post package steps only' })
     .argv;
 
+  if (args['pre-only'] && args['post-only']) {
+    throw new Error('You can set a maxiumum of one of --pre-only and --post-only flags to true. Pick one.');
+  }
+
   const options = cdkPackageOptions();
 
+  if (args['post-only']) {
+    if (options.post) {
+      const commands = options.post.join(' && ');
+      await shell([commands], { timers });
+    }
+    return;
+  }
+
   const outdir = 'dist';
 
   // if this is a private module, don't package
@@ -34,6 +48,14 @@ async function main() {
     return;
   }
 
+  if (options.pre ) {
+    const commands = options.pre.join(' && ');
+    await shell([commands], { timers });
+  }
+  if (args['pre-only']) {
+    return;
+  }
+
   // If we need to shrinkwrap this, do so now.
   if (options.shrinkWrap) {
     await yarnCling.generateShrinkwrap({
@@ -65,11 +87,11 @@ async function main() {
   }
 
   if (options.post) {
-    await shell(options.post, { timers });
+    const commands = options.post.join(' && ');
+    await shell([commands], { timers });
   }
 }
 
-
 main().then(() => {
   buildTimer.end();
   process.stdout.write(`Package complete. ${timers.display()}\n`);

tools/@aws-cdk/cdk-build-tools/lib/package-info.ts

@@ -142,14 +142,14 @@ export interface CDKBuildOptions {
   };
 
   /**
-   * An optional command (formatted as a list of strings) to run before building
+   * Optional commands (formatted as a list of strings, which will be joined together with the && operator) to run before building
    *
    * (Typically a code generator)
    */
   pre?: string[];
 
   /**
-   * An optional command (formatted as a list of strings) to run after building
+   * Optional commands (formatted as a list of strings, which will be joined together with the && operator) to run after building
    *
    * (Schema generator for example)
    */
@@ -190,8 +190,13 @@ export interface CDKPackageOptions {
    */
   shrinkWrap?: boolean;
 
+  /**
+   * Optional commands (formatted as a list of strings, which will be joined together with the && operator) to run before packaging
+   */
+  pre?: string[];
+
   /*
-   * An optional command (formatted as a list of strings) to run after packaging
+   * Optional commands (formatted as a list of strings, which will be joined together with the && operator) to run after packaging
   */
   post?: string[];