fix: remove the pdk and projen bins from the virtual envs as we want …

…to use npx (#766)
aws · Apr 18, 2024 · 62f9229 · 62f9229
1 parent c5517bc
commit 62f9229
Show file tree

Hide file tree

Showing 4 changed files with 110 additions and 3 deletions.
diff --git a/.../infrastructure/test/projects/python/__snapshots__/infrastructure-py-project.test.ts.snap b/.../infrastructure/test/projects/python/__snapshots__/infrastructure-py-project.test.ts.snap
diff --git a/packages/monorepo/src/components/nx-configurator.ts b/packages/monorepo/src/components/nx-configurator.ts
@@ -197,9 +197,14 @@ export class NxConfigurator extends Component implements INxProjectCore {
       ["install", "install:ci"].forEach((t) => {
         const task = project.tasks.tryFind(t);
 
-        // Setup env
-        const cmd = "poetry env use python$PYTHON_VERSION";
-        task?.steps[0]?.exec !== cmd && task?.prependExec(cmd);
+        // Setup env and ensure the projen & pdk bins are removed from the venv as we always want to use the npx variant
+        [
+          "rm -f `poetry env info -p`/bin/projen `poetry env info -p`/bin/pdk",
+          "poetry env use python$PYTHON_VERSION",
+        ].forEach(
+          (cmd) =>
+            !task?.steps.find((s) => s.exec === cmd) && task?.prependExec(cmd)
+        );
 
         const pythonVersion = project.deps.tryGetDependency("python")?.version;
         task!.env(

diff --git a/packages/monorepo/test/__snapshots__/monorepo.test.ts.snap b/packages/monorepo/test/__snapshots__/monorepo.test.ts.snap
diff --git a/packages/type-safe-api/test/project/__snapshots__/type-safe-api-project.test.ts.snap b/packages/type-safe-api/test/project/__snapshots__/type-safe-api-project.test.ts.snap