feat(ng-dev): add support for verifying merge-mode reset before caretaker handoff

Author: josephperrott

.github/local-actions/branch-manager/main.js

@@ -7,12 +7,10 @@
  */
 
 import {Argv, CommandModule} from 'yargs';
-
 import {addGithubTokenOption} from '../../utils/git/github-yargs.js';
-
 import {updateCaretakerTeamViaPrompt} from './update-github-team.js';
-
-export interface CaretakerHandoffOptions {}
+import {assertValidCaretakerConfig, getConfig} from '../../utils/config.js';
+import {verifyMergeMode} from './verify-merge-mode.js';
 
 /** Builds the command. */
 function builder(argv: Argv) {
@@ -21,11 +19,18 @@ function builder(argv: Argv) {
 
 /** Handles the command. */
 async function handler() {
+  // TODO(josephperrott): require merge mode to be defined in all caretaker configs.
+  const {
+    caretaker: {repositoryMergeMode},
+  } = await getConfig([assertValidCaretakerConfig]);
+  if (repositoryMergeMode && !(await verifyMergeMode(repositoryMergeMode))) {
+    return;
+  }
   await updateCaretakerTeamViaPrompt();
 }
 
 /** yargs command module for assisting in handing off caretaker.  */
-export const HandoffModule: CommandModule<{}, CaretakerHandoffOptions> = {
+export const HandoffModule: CommandModule<{}, {}> = {
   handler,
   builder,
   command: 'handoff',

ng-dev/caretaker/handoff/cli.ts

@@ -0,0 +1,43 @@
+/**
+ * @license
+ * Copyright Google LLC
+ *
+ * Use of this source code is governed by an MIT-style license that can be
+ * found in the LICENSE file at https://angular.io/license
+ */
+
+import {RepositoryMergeModes} from '../../utils/config';
+import {bold, green, Log, red} from '../../utils/logging';
+import {Prompt} from '../../utils/prompt';
+import {getRepoConfigValue} from '../config/get';
+import {setRepoConfigValue} from '../config/set';
+
+export async function verifyMergeMode(expectedMode: RepositoryMergeModes): Promise<boolean> {
+  const currentMergeMode = await getRepoConfigValue('merge-mode');
+  if (currentMergeMode === expectedMode) {
+    return true;
+  }
+
+  Log.info(
+    `The repository is currently set to ${bold(currentMergeMode)} and needs to be reset before handoff`,
+  );
+  if (
+    await Prompt.confirm({
+      message: `Would you like to reset this to ${expectedMode}`,
+      default: true,
+    })
+  ) {
+    try {
+      await setRepoConfigValue('merge-mode', expectedMode);
+      Log.info(`${green('✔')} Successfuly set merge-mode to ${expectedMode}`);
+      return true;
+    } catch (err) {
+      Log.info(`${red('✘')} Failed to update merge-mode`);
+      Log.info(err);
+      return false;
+    }
+  }
+  // User chose not to reset merge-mode
+  Log.info('Aborting...');
+  return false;
+}

ng-dev/caretaker/handoff/verify-merge-mode.ts

@@ -61,6 +61,9 @@ export interface GoogleSyncConfig {
   separateFilePatterns: string[];
 }
 
+/** The available merge modes for repositories to utilize for normal usage. */
+export type RepositoryMergeModes = 'team-only' | 'caretaker-only';
+
 export interface CaretakerConfig {
   /** Github queries showing a snapshot of pulls/issues caretakers need to monitor. */
   githubQueries?: {name: string; query: string}[];
@@ -74,6 +77,11 @@ export interface CaretakerConfig {
    * The configuration file is expected to be valid JSONC and match {@see GoogleSyncConfig}.
    */
   g3SyncConfigPath?: string;
+  /**
+   * The merge mode to use for the repository, either allowing only the caretaker to perform
+   * merges, or allowing all team members to do so.
+   */
+  repositoryMergeMode?: RepositoryMergeModes;
 }
 
 /** The filename expected for creating the ng-dev config. */