fix(ng-dev): remove renovate target label update during release

The logic to update the renovate config during the release to switch the target labels from `target: rc` to `target: patch` has been removed. Instead, a static `target: automation` label is added to the default renovate preset.

This simplifies the release process by removing the need to update the renovate config during the release.

Author: alan-agius4

ng-dev/release/publish/BUILD.bazel

@@ -18,7 +18,6 @@ ts_project(
         "//ng-dev:node_modules/semver",
         "//ng-dev:node_modules/typed-graphqlify",
         "//ng-dev/commit-message",
-        "//ng-dev/pr/common/labels",
         "//ng-dev/pr/merge",
         "//ng-dev/release/build",
         "//ng-dev/release/config",

ng-dev/release/publish/actions.ts

@@ -43,8 +43,6 @@ import {promptToInitiatePullRequestMerge} from './prompt-merge.js';
 import {Prompt} from '../../utils/prompt.js';
 import {PnpmVersioning} from './pnpm-versioning.js';
 import {Commit} from '../../utils/git/octokit-types.js';
-import {updateRenovateConfigTargetLabels} from './actions/renovate-config-updates.js';
-import {targetLabels} from '../../pr/common/labels/target.js';
 
 /** Interface describing a Github repository. */
 export interface GithubRepo {
@@ -567,19 +565,6 @@ export abstract class ReleaseAction {
 
     const filesToCommit: string[] = [workspaceRelativeChangelogPath];
 
-    if (version.patch === 0 && version.prerelease.length === 0) {
-      // Switch the renovate labels for `target: rc` to `target: patch`
-      const renovateConfigPath = await updateRenovateConfigTargetLabels(
-        this.projectDir,
-        targetLabels['TARGET_RC'].name,
-        targetLabels['TARGET_PATCH'].name,
-      );
-
-      if (renovateConfigPath) {
-        filesToCommit.push(renovateConfigPath);
-      }
-    }
-
     await this.createCommit(commitMessage, filesToCommit);
     Log.info(green(`  ✓   Created changelog cherry-pick commit for: "${version}".`));
 

ng-dev/release/publish/actions/renovate-config-updates.ts

@@ -2,7 +2,6 @@ import {existsSync} from 'node:fs';
 import {green, Log} from '../../../utils/logging.js';
 import {join} from 'node:path';
 import {writeFile, readFile} from 'node:fs/promises';
-import {targetLabels} from '../../../pr/common/labels/target.js';
 
 /**
  * Updates the `renovate.json` configuration file to include a new base branch.
@@ -37,92 +36,8 @@ export async function updateRenovateConfig(
 
   configJson['baseBranchPatterns'] = ['main', newBranchName];
 
-  updateRenovateTargetLabel(
-    configJson,
-    targetLabels['TARGET_PATCH'].name,
-    targetLabels['TARGET_RC'].name,
-  );
   await writeFile(renovateConfigPath, JSON.stringify(configJson, undefined, 2));
 
   Log.info(green(`  ✓   Updated Renovate config.`));
   return renovateConfigPath;
 }
-
-/**
- * Updates a specific target label in the `renovate.json` configuration file.
- * This function specifically targets and replaces one label with another within the `packageRules`.
- *
- * @param projectDir - The path to the project directory.
- * @param fromLabel - The label name to be replaced.
- * @param toLabel - The new label name to replace `fromLabel` with.
- * @returns A promise that resolves to the path of the modified `renovate.json` file if updated,
- * or `null` if the file was not found or the `baseBranchPatterns` array has an unexpected format.
- */
-export async function updateRenovateConfigTargetLabels(
-  projectDir: string,
-  fromLabel: string,
-  toLabel: string,
-): Promise<string | null> {
-  const renovateConfigPath = join(projectDir, 'renovate.json');
-  if (!existsSync(renovateConfigPath)) {
-    Log.warn(`  ✘   Skipped updating Renovate config as it was not found.`);
-
-    return null;
-  }
-
-  const config = await readFile(renovateConfigPath, 'utf-8');
-  const configJson = JSON.parse(config) as Record<string, unknown>;
-
-  // Check baseBranchPatterns just in case, though this function's primary focus is labels
-  const baseBranchPatterns = configJson['baseBranchPatterns'];
-  if (!Array.isArray(baseBranchPatterns) || baseBranchPatterns.length !== 2) {
-    Log.warn(
-      `  ✘   Skipped updating Renovate config: "baseBranchPatterns" must contain exactly 2 branches.`,
-    );
-
-    return null;
-  }
-
-  if (updateRenovateTargetLabel(configJson, fromLabel, toLabel)) {
-    await writeFile(renovateConfigPath, JSON.stringify(configJson, undefined, 2));
-    Log.info(green(`  ✓   Updated target label in Renovate config.`));
-
-    return renovateConfigPath;
-  } else {
-    Log.info(green(`  ✓   No changes to target labels in Renovate config.`));
-    return null;
-  }
-}
-
-/**
- * Updates a specific target label within the `packageRules` of a Renovate configuration.
- *
- * @param configJson - The parsed JSON object of the Renovate configuration.
- * @param fromLabel - The label name to be replaced.
- * @param toLabel - The new label name to replace `fromLabel` with.
- * @returns `true` is the label has been updated, otherwise `false`.
- */
-function updateRenovateTargetLabel(
-  configJson: Record<string, unknown>,
-  fromLabel: string,
-  toLabel: string,
-): boolean {
-  if (!Array.isArray(configJson['packageRules'])) {
-    return false;
-  }
-
-  let updated = false;
-  for (const rule of configJson['packageRules']) {
-    if (!Array.isArray(rule.addLabels)) {
-      continue;
-    }
-
-    const idx = (rule.addLabels as string[]).findIndex((x) => x === fromLabel);
-    if (idx >= 0) {
-      rule.addLabels[idx] = toLabel;
-      updated = true;
-    }
-  }
-
-  return updated;
-}

ng-dev/release/publish/test/common.spec.ts

@@ -617,11 +617,9 @@ describe('common release action logic', () => {
           'packageRules': [
             {
               'matchBaseBranches': ['main'],
-              'addLabels': ['target: minor'],
             },
             {
               'matchBaseBranches': ['!main'],
-              'addLabels': ['target: rc'],
             },
           ],
         }),

renovate-presets/default.json5

@@ -16,7 +16,7 @@
   semanticCommits: 'enabled',
   semanticCommitScope: '',
   semanticCommitType: 'build',
-  labels: ['area: build & ci', 'action: merge'],
+  labels: ['area: build & ci', 'action: merge', 'target: automation'],
 
   lockFileMaintenance: {
     enabled: true,