feat(release-notes)!: support configurable fetching stage

docs/usage/configuration-options.md

@@ -923,7 +923,14 @@ A similar one could strip leading `v` prefixes:
 
 ## fetchReleaseNotes
 
-Set this to `false` if you want to disable release notes fetching.
+Use this config option to configure release notes fetching.
+The available options are:
+
+- `off` - disable release notes fetching
+- `branch` - fetch release notes while creating/updating branch
+- `pr`(default) - fetches release notes while creating/updating pull-request
+
+It is not recommended to set fetchReleaseNotes=branch unless you are embedding release notes in commit information, because it results in a performance decrease.
 
 Renovate can fetch release notes when they are hosted on one of these platforms:
 

lib/config/migrations/custom/fetch-release-notes-migration.spec.ts

@@ -0,0 +1,22 @@
+import { FetchReleaseNotesMigration } from './fetch-release-notes-migration';
+
+describe('config/migrations/custom/fetch-release-notes-migration', () => {
+  it('migrates', () => {
+    expect(FetchReleaseNotesMigration).toMigrate(
+      {
+        fetchReleaseNotes: false as never,
+      },
+      {
+        fetchReleaseNotes: 'off',
+      }
+    );
+    expect(FetchReleaseNotesMigration).toMigrate(
+      {
+        fetchReleaseNotes: true as never,
+      },
+      {
+        fetchReleaseNotes: 'pr',
+      }
+    );
+  });
+});

lib/config/migrations/custom/fetch-release-notes-migration.ts

@@ -0,0 +1,12 @@
+import is from '@sindresorhus/is';
+import { AbstractMigration } from '../base/abstract-migration';
+
+export class FetchReleaseNotesMigration extends AbstractMigration {
+  override readonly propertyName = 'fetchReleaseNotes';
+
+  override run(value: unknown): void {
+    if (is.boolean(value)) {
+      this.rewrite(value ? 'pr' : 'off');
+    }
+  }
+}

lib/config/migrations/migrations-service.ts

@@ -20,6 +20,7 @@ import { DepTypesMigration } from './custom/dep-types-migration';
 import { DryRunMigration } from './custom/dry-run-migration';
 import { EnabledManagersMigration } from './custom/enabled-managers-migration';
 import { ExtendsMigration } from './custom/extends-migration';
+import { FetchReleaseNotesMigration } from './custom/fetch-release-notes-migration';
 import { GoModTidyMigration } from './custom/go-mod-tidy-migration';
 import { HostRulesMigration } from './custom/host-rules-migration';
 import { IgnoreNodeModulesMigration } from './custom/ignore-node-modules-migration';
@@ -148,6 +149,7 @@ export class MigrationsService {
     DatasourceMigration,
     RecreateClosedMigration,
     StabilityDaysMigration,
+    FetchReleaseNotesMigration,
   ];
 
   static run(originalConfig: RenovateConfig): RenovateConfig {

lib/config/options/index.ts

@@ -2551,9 +2551,10 @@ const options: RenovateOptions[] = [
   },
   {
     name: 'fetchReleaseNotes',
-    description: 'Controls if release notes are fetched.',
-    type: 'boolean',
-    default: true,
+    description: 'Controls if and when release notes are fetched.',
+    type: 'string',
+    allowedValues: ['off', 'branch', 'pr'],
+    default: 'pr',
     cli: false,
     env: false,
   },

lib/config/types.ts

@@ -258,7 +258,7 @@ export interface RenovateConfig
   vulnerabilitySeverity?: string;
   regexManagers?: RegExManager[];
 
-  fetchReleaseNotes?: boolean;
+  fetchReleaseNotes?: FetchReleaseNotesOptions;
   secrets?: Record<string, string>;
 
   constraints?: Record<string, string>;
@@ -299,6 +299,8 @@ export type UpdateType =
   | 'bump'
   | 'replacement';
 
+type FetchReleaseNotesOptions = 'off' | 'branch' | 'pr';
+
 export type MatchStringsStrategy = 'any' | 'recursive' | 'combination';
 
 export type MergeStrategy =

lib/workers/repository/update/branch/index.spec.ts

@@ -3,7 +3,6 @@ import {
   getConfig,
   git,
   mocked,
-  mockedFunction,
   partial,
   platform,
   scm,
@@ -34,7 +33,6 @@ import * as _mergeConfidence from '../../../../util/merge-confidence';
 import * as _sanitize from '../../../../util/sanitize';
 import * as _limits from '../../../global/limits';
 import type { BranchConfig, BranchUpgradeConfig } from '../../../types';
-import { needsChangelogs } from '../../changelog';
 import type { ResultWithPr } from '../pr';
 import * as _prWorker from '../pr';
 import * as _prAutomerge from '../pr/automerge';
@@ -816,9 +814,8 @@ describe('workers/repository/update/branch/index', () => {
         ignoreTests: true,
         prCreation: 'not-pending',
         commitBody: '[skip-ci]',
-        fetchReleaseNotes: true,
+        fetchReleaseNotes: 'branch',
       } satisfies BranchConfig;
-      mockedFunction(needsChangelogs).mockReturnValueOnce(true);
       scm.getBranchCommit.mockResolvedValue('123test'); //TODO:not needed?
       expect(await branchWorker.processBranch(inconfig)).toEqual({
         branchExists: true,

lib/workers/repository/update/branch/index.ts

@@ -34,7 +34,7 @@ import { toMs } from '../../../../util/pretty-time';
 import * as template from '../../../../util/template';
 import { isLimitReached } from '../../../global/limits';
 import type { BranchConfig, BranchResult, PrBlockedBy } from '../../../types';
-import { embedChangelog, needsChangelogs } from '../../changelog';
+import { embedChangelogs } from '../../changelog';
 import { ensurePr } from '../pr';
 import { checkAutoMerge } from '../pr/automerge';
 import { setArtifactErrorStatus } from './artifacts';
@@ -482,6 +482,10 @@ export async function processBranch(
       } else {
         logger.debug('No updated lock files in branch');
       }
+      if (config.fetchReleaseNotes === 'branch') {
+        await embedChangelogs(config.upgrades);
+      }
+
       const postUpgradeCommandResults = await executePostUpgradeCommands(
         config
       );
@@ -540,14 +544,6 @@ export async function processBranch(
 
       // compile commit message with body, which maybe needs changelogs
       if (config.commitBody) {
-        if (
-          config.fetchReleaseNotes &&
-          needsChangelogs(config, ['commitBody'])
-        ) {
-          // we only need first upgrade, the others are only needed on PR update
-          // we add it to first, so PR fetch can skip fetching for that update
-          await embedChangelog(config.upgrades[0]);
-        }
         // changelog is on first upgrade
         config.commitMessage = `${config.commitMessage!}\n\n${template.compile(
           config.commitBody,

lib/workers/repository/update/pr/index.spec.ts

@@ -101,7 +101,7 @@ describe('workers/repository/update/pr/index', () => {
         platform.createPr.mockResolvedValueOnce(pr);
         limits.isLimitReached.mockReturnValueOnce(true);
 
-        config.fetchReleaseNotes = true;
+        config.fetchReleaseNotes = 'pr';
 
         const res = await ensurePr(config);
 
@@ -844,13 +844,13 @@ describe('workers/repository/update/pr/index', () => {
           bodyFingerprint: fingerprint(
             generatePrBodyFingerprintConfig({
               ...config,
-              fetchReleaseNotes: true,
+              fetchReleaseNotes: 'pr',
             })
           ),
           lastEdited: new Date('2020-01-20T00:00:00Z').toISOString(),
         };
         prCache.getPrCache.mockReturnValueOnce(cachedPr);
-        const res = await ensurePr({ ...config, fetchReleaseNotes: true });
+        const res = await ensurePr({ ...config, fetchReleaseNotes: 'pr' });
         expect(res).toEqual({
           type: 'with-pr',
           pr: existingPr,
@@ -877,13 +877,13 @@ describe('workers/repository/update/pr/index', () => {
           bodyFingerprint: fingerprint(
             generatePrBodyFingerprintConfig({
               ...config,
-              fetchReleaseNotes: true,
+              fetchReleaseNotes: 'pr',
             })
           ),
           lastEdited: new Date('2020-01-20T00:00:00Z').toISOString(),
         };
         prCache.getPrCache.mockReturnValueOnce(cachedPr);
-        const res = await ensurePr({ ...config, fetchReleaseNotes: true });
+        const res = await ensurePr({ ...config, fetchReleaseNotes: 'pr' });
         expect(res).toEqual({
           type: 'with-pr',
           pr: {

lib/workers/repository/update/pr/index.ts

@@ -233,7 +233,7 @@ export async function ensurePr(
     }`;
   }
 
-  if (config.fetchReleaseNotes) {
+  if (config.fetchReleaseNotes === 'pr') {
     // fetch changelogs when not already done;
     await embedChangelogs(upgrades);
   }

lib/workers/repository/updates/branchify.spec.ts

@@ -124,7 +124,7 @@ describe('workers/repository/updates/branchify', () => {
     });
 
     it('no fetch changelogs', async () => {
-      config.fetchReleaseNotes = false;
+      config.fetchReleaseNotes = 'off';
       flattenUpdates.mockResolvedValueOnce([
         {
           depName: 'foo',
@@ -155,7 +155,7 @@ describe('workers/repository/updates/branchify', () => {
     });
 
     it('fetch changelogs if required', async () => {
-      config.fetchReleaseNotes = true;
+      config.fetchReleaseNotes = 'pr';
       config.repoIsOnboarded = true;
       mockedFunction(_changelog.needsChangelogs).mockReturnValueOnce(true);
       flattenUpdates.mockResolvedValueOnce([

lib/workers/repository/updates/branchify.ts

@@ -72,9 +72,9 @@ export async function branchifyUpgrades(
       })
       .reverse();
 
-    if (config.fetchReleaseNotes && config.repoIsOnboarded) {
-      const branches = branchUpgrades[branchName].filter((upg) =>
-        needsChangelogs(upg)
+    if (config.fetchReleaseNotes !== 'off' && config.repoIsOnboarded) {
+      const branches = branchUpgrades[branchName].filter(
+        (upg) => needsChangelogs(upg) || config.fetchReleaseNotes === 'branch'
       );
       if (branches.length) {
         logger.warn(