fix: use pnpm deploy to pack against to ensure all bundled dependenci…

…es are included fix #755
aws · Apr 15, 2024 · 8d80469 · 8d80469
1 parent aad17d9
commit 8d80469
Show file tree

Hide file tree

Showing 39 changed files with 77 additions and 221 deletions.
diff --git a/.projen/tasks.json b/.projen/tasks.json
diff --git a/packages/aws-arch/.projen/tasks.json b/packages/aws-arch/.projen/tasks.json
diff --git a/packages/cdk-graph-plugin-diagram/.projen/tasks.json b/packages/cdk-graph-plugin-diagram/.projen/tasks.json
diff --git a/...s/cdk-graph-plugin-diagram/test/graphviz/__image_snapshots__/config/compact.png b/...s/cdk-graph-plugin-diagram/test/graphviz/__image_snapshots__/config/compact.png
diff --git a/...s/cdk-graph-plugin-diagram/test/graphviz/__image_snapshots__/config/default.png b/...s/cdk-graph-plugin-diagram/test/graphviz/__image_snapshots__/config/default.png
diff --git a/...s/cdk-graph-plugin-diagram/test/graphviz/__image_snapshots__/config/verbose.png b/...s/cdk-graph-plugin-diagram/test/graphviz/__image_snapshots__/config/verbose.png
diff --git a/...s/cdk-graph-plugin-diagram/test/graphviz/__image_snapshots__/dot/all-stages.png b/...s/cdk-graph-plugin-diagram/test/graphviz/__image_snapshots__/dot/all-stages.png
diff --git a/...es/cdk-graph-plugin-diagram/test/graphviz/__image_snapshots__/dot/dev-stage.png b/...es/cdk-graph-plugin-diagram/test/graphviz/__image_snapshots__/dot/dev-stage.png
diff --git a/.../cdk-graph-plugin-diagram/test/graphviz/__image_snapshots__/dot/multi-stack.png b/.../cdk-graph-plugin-diagram/test/graphviz/__image_snapshots__/dot/multi-stack.png
diff --git a/...s/cdk-graph-plugin-diagram/test/graphviz/__image_snapshots__/dot/prod-stage.png b/...s/cdk-graph-plugin-diagram/test/graphviz/__image_snapshots__/dot/prod-stage.png
diff --git a/...cdk-graph-plugin-diagram/test/graphviz/__image_snapshots__/dot/single-stack.png b/...cdk-graph-plugin-diagram/test/graphviz/__image_snapshots__/dot/single-stack.png
diff --git a/...gin-diagram/test/graphviz/__image_snapshots__/filter/filter-cfntype-exclude.png b/...gin-diagram/test/graphviz/__image_snapshots__/filter/filter-cfntype-exclude.png
diff --git a/...in-diagram/test/graphviz/__image_snapshots__/filter/filter-nodetype-exclude.png b/...in-diagram/test/graphviz/__image_snapshots__/filter/filter-nodetype-exclude.png
diff --git a/...in-diagram/test/graphviz/__image_snapshots__/filter/filter-nodetype-include.png b/...in-diagram/test/graphviz/__image_snapshots__/filter/filter-nodetype-include.png
diff --git a/...ugin-diagram/test/graphviz/__image_snapshots__/pdk-integ/pdk-integ-pipeline.png b/...ugin-diagram/test/graphviz/__image_snapshots__/pdk-integ/pdk-integ-pipeline.png
diff --git a/...-graph-plugin-diagram/test/graphviz/__image_snapshots__/theme/dark-services.png b/...-graph-plugin-diagram/test/graphviz/__image_snapshots__/theme/dark-services.png
diff --git a/...k-graph-plugin-diagram/test/graphviz/__image_snapshots__/theme/dark-verbose.png b/...k-graph-plugin-diagram/test/graphviz/__image_snapshots__/theme/dark-verbose.png
diff --git a/packages/cdk-graph-plugin-diagram/test/graphviz/__image_snapshots__/theme/dark.png b/packages/cdk-graph-plugin-diagram/test/graphviz/__image_snapshots__/theme/dark.png
diff --git a/...graph-plugin-diagram/test/graphviz/__image_snapshots__/theme/light-services.png b/...graph-plugin-diagram/test/graphviz/__image_snapshots__/theme/light-services.png
diff --git a/...-graph-plugin-diagram/test/graphviz/__image_snapshots__/theme/light-verbose.png b/...-graph-plugin-diagram/test/graphviz/__image_snapshots__/theme/light-verbose.png
diff --git a/...ages/cdk-graph-plugin-diagram/test/graphviz/__image_snapshots__/theme/light.png b/...ages/cdk-graph-plugin-diagram/test/graphviz/__image_snapshots__/theme/light.png
diff --git a/packages/cdk-graph-plugin-threat-composer/.projen/tasks.json b/packages/cdk-graph-plugin-threat-composer/.projen/tasks.json
diff --git a/packages/cdk-graph/.projen/tasks.json b/packages/cdk-graph/.projen/tasks.json
diff --git a/packages/cloudscape-react-ts-website/.projen/tasks.json b/packages/cloudscape-react-ts-website/.projen/tasks.json
diff --git a/packages/identity/.projen/tasks.json b/packages/identity/.projen/tasks.json
diff --git a/packages/infrastructure/.projen/tasks.json b/packages/infrastructure/.projen/tasks.json
diff --git a/packages/monorepo/.projen/tasks.json b/packages/monorepo/.projen/tasks.json
diff --git a/packages/monorepo/package.json b/packages/monorepo/package.json
diff --git a/packages/monorepo/scripts/monorepo/pnpm/link-bundled-transitive-deps.js b/packages/monorepo/scripts/monorepo/pnpm/link-bundled-transitive-deps.js
diff --git a/packages/monorepo/src/projects/typescript/monorepo-ts.ts b/packages/monorepo/src/projects/typescript/monorepo-ts.ts
@@ -547,24 +547,6 @@ export class MonorepoTsProject
       this.linkLocalWorkspaceBins();
     }
 
-    if (this.package.packageManager === NodePackageManager.PNPM) {
-      // PNPM hoisting hides transitive bundled dependencies which results in
-      // transitive dependencies being packed incorrectly.
-      this.subprojects.forEach((subProject) => {
-        if (
-          NodePackageUtils.isNodeProject(subProject) &&
-          getBundledDeps(subProject).length
-        ) {
-          const pkgFolder = path.relative(this.root.outdir, subProject.outdir);
-          // Create a symlink in the sub-project node_modules for all transitive deps
-          // before running "package" task
-          subProject.packageTask.prependExec(
-            `monorepo.pnpm-link-bundled-transitive-deps ${pkgFolder}`
-          );
-        }
-      });
-    }
-
     this.subprojects.forEach((subProject) => {
       if (NodePackageUtils.isNodeProject(subProject)) {
         // Remove any subproject .npmrc files since only the root one matters

diff --git a/packages/pdk-nag/.projen/tasks.json b/packages/pdk-nag/.projen/tasks.json