fix(release): try preventing race conditions (#911)

See projen/projen#3763 for more detail. We decided to first put the solution in place on a controlled set of projects. If it proves successful, we will push it to projen itself.

src/projects/jsii.ts

@@ -122,6 +122,8 @@ export class Cdk8sTeamJsiiProject extends cdk.JsiiProject {
     this.deps.removeDependency('@types/node^16');
     this.deps.addDependency('@types/[email protected]', DependencyType.BUILD);
 
+    node.limitReleaseConcurrency(this);
+
   }
 }
 

src/projects/node.ts

@@ -1,5 +1,5 @@
 import * as deepmerge from 'deepmerge';
-import { DependencyType, ReleasableCommits, javascript } from 'projen';
+import { DependencyType, JsonPatch, Project, ReleasableCommits, javascript } from 'projen';
 import { NodeProject, NodeProjectOptions, UpgradeDependencies, UpgradeDependenciesOptions, UpgradeDependenciesSchedule } from 'projen/lib/javascript';
 import { Backport } from '../components/backport/backport';
 import { CodeOfConductMD } from '../components/code-of-conduct/code-of-conduct';
@@ -185,6 +185,8 @@ export class Cdk8sTeamNodeProject extends javascript.NodeProject {
     if (options.backport ?? false) {
       new Backport(this, { branches: options.backportBranches, repoName });
     }
+
+    limitReleaseConcurrency(this);
   }
 }
 
@@ -302,3 +304,15 @@ export function addComponents(project: NodeProject, repoName: string, options: C
   }
 
 }
+
+// see https://github.com/projen/projen/pull/3763
+export function limitReleaseConcurrency(project: Project) {
+
+  const regex = /^\.github\/workflows\/release/;
+  const releaseWorkflows = project.files.filter(f => regex.test(f.path));
+
+  for (const releaseWorkflow of releaseWorkflows) {
+    const objectFile = project.tryFindObjectFile(releaseWorkflow.path)!;
+    objectFile.patch(JsonPatch.add('/concurrency', { 'group': '${{ github.workflow }}', 'cancel-in-progress': false }));
+  }
+}

src/projects/typescript.ts

@@ -86,5 +86,7 @@ export class Cdk8sTeamTypeScriptProject extends typescript.TypeScriptProject {
     this.deps.removeDependency('@types/node');
     this.deps.addDependency('@types/[email protected]', DependencyType.BUILD);
 
+    node.limitReleaseConcurrency(this);
+
   }
 }