feat: add configurable pack command

packages/jsii-pacmak/bin/jsii-pacmak.ts

@@ -153,6 +153,13 @@ import { VERSION_DESC } from '../lib/version';
       default: undefined,
       hidden: true,
     })
+    .option('npm-pack-command', {
+      type: 'string',
+      desc: 'Configure a custom command to execute when packaging i.e. pnpm pack --pack-destination {{destDir}}. Bundle must be written to {{destDir}} which will be substituted at runtime.',
+      defaultDescription: 'npm pack is used.',
+      default: 'npm pack --pack-destination {{destDir}}',
+      hidden: true,
+    })
     .option('validate-assemblies', {
       type: 'boolean',
       desc: 'Whether jsii assemblies should be validated. This can be expensive and is skipped by default.',

packages/jsii-pacmak/lib/index.ts

@@ -74,7 +74,9 @@ export async function pacmak({
 
   await timers.recordAsync('npm pack', () => {
     logging.info('Packaging NPM bundles');
-    return Promise.all(modulesToPackageFlat.map((m) => m.npmPack()));
+    return Promise.all(
+      modulesToPackageFlat.map((m) => m.npmPack(argv['npm-pack-command'])),
+    );
   });
 
   await timers.recordAsync('load jsii', () => {

packages/jsii-pacmak/lib/packaging.ts

@@ -52,15 +52,19 @@ export class JsiiModule {
   /**
    * Prepare an NPM package from this source module
    */
-  public async npmPack() {
+  public async npmPack(
+    packCommand = 'npm pack --pack-destination {{destDir}}',
+  ) {
     this._tarball = await Scratch.make(async (tmpdir) => {
-      // Quoting (JSON-stringifying) the module directory in order to avoid
-      // problems if there are spaces or other special characters in the path.
-      const args = ['pack', JSON.stringify(this.moduleDirectory)];
+      const args = [
+        ...packCommand.split(' ').map((c) => c.replace(/{{destDir}}/g, tmpdir)),
+      ];
       if (logging.level >= logging.LEVEL_VERBOSE) {
         args.push('--loglevel=verbose');
       }
-      const out = await shell('npm', args, { cwd: tmpdir });
+      const out = await shell(args[0], args.slice(1), {
+        cwd: this.moduleDirectory,
+      });
       // Take only the last line of npm pack which should contain the
       // tarball name. otherwise, there can be a lot of extra noise there
       // from scripts that emit to STDOUT.

packages/jsii-pacmak/package.json

@@ -65,7 +65,8 @@
     "jsii": "^0.0.0",
     "jsii-build-tools": "^0.0.0",
     "jsii-calc": "^3.20.120",
-    "pyright": "^1.1.296"
+    "pyright": "^1.1.296",
+    "pnpm": "^7.30.0"
   },
   "keywords": [
     "jsii",

packages/jsii-pacmak/test/build-test.sh

@@ -81,3 +81,7 @@ done
 clean_dists
 echo "Testing ALL-AT-ONCE build."
 ${pacmak} ${OPTS} -v --no-parallel $packagedirs
+
+clean_dists
+echo "Testing pnpm."
+${pacmak} ${OPTS} -v --no-parallel --npm-pack-command='yarn run pnpm pack --pack-destination {{destDir}}' $packagedirs