fix(manager/regex): depName requirement if using the recursive strategy

docs/usage/configuration-options.md

@@ -2314,7 +2314,7 @@ This can be used to narrow down the search area to prevent multiple matches.
 But the `recursive` strategy still allows the matching of multiple dependencies as described below.
 All matches of the first `matchStrings` pattern are detected, then each of these matches will used as basis be used as the input for the next `matchStrings` pattern, and so on.
 If the next `matchStrings` pattern has multiple matches then it will split again.
-This process will be followed as long there is a match plus a next `matchingStrings` pattern is available or a dependency is detected.
+This process will be followed as long there is a match plus a next `matchingStrings` pattern is available.
 
 Matched groups will be available in subsequent matching layers.
 

lib/modules/manager/regex/index.spec.ts

@@ -419,4 +419,34 @@ describe('modules/manager/regex/index', () => {
     expect(res).toMatchSnapshot();
     expect(res?.deps).toHaveLength(4);
   });
+
+  it('extracts with recursive strategy and without depName', async () => {
+    const config: CustomExtractConfig = {
+      matchStrings: [
+        'jacoco\\s*{[^}]*}',
+        'toolVersion\\s*=\\s*\\"(?<currentValue>\\S*)\\"\\s*',
+      ],
+      matchStringsStrategy: 'recursive',
+      depNameTemplate: 'org.jacoco:jacoco',
+      datasourceTemplate: 'maven',
+    };
+    const res = await extractPackageFile(
+      `
+    jacoco {
+      toolVersion = "0.8.7"
+    }
+    `,
+      'build.gradle.kts',
+      config
+    );
+    expect(res).toMatchObject({
+      deps: [
+        {
+          depName: 'org.jacoco:jacoco',
+          currentValue: '0.8.7',
+          datasource: 'maven',
+        },
+      ],
+    });
+  });
 });

lib/modules/manager/regex/strategies.ts

@@ -3,6 +3,7 @@ import { regEx } from '../../../util/regex';
 import type { CustomExtractConfig, PackageDependency } from '../types';
 import {
   createDependency,
+  isValidDependency,
   mergeExtractionTemplate,
   mergeGroups,
   regexMatchAll,
@@ -22,7 +23,8 @@ export function handleAny(
         config
       )
     )
-    .filter(is.truthy);
+    .filter(is.truthy)
+    .filter(isValidDependency);
 }
 
 export function handleCombination(
@@ -44,7 +46,9 @@ export function handleCombination(
       replaceString: match?.groups?.currentValue ? match[0] : undefined,
     }))
     .reduce((base, addition) => mergeExtractionTemplate(base, addition));
-  return [createDependency(extraction, config)].filter(is.truthy);
+  return [createDependency(extraction, config)]
+    .filter(is.truthy)
+    .filter(isValidDependency);
 }
 
 export function handleRecursive(
@@ -59,21 +63,17 @@ export function handleRecursive(
   );
   // abort if we have no matchString anymore
   if (!regexes[index]) {
-    return [];
+    const result = createDependency(
+      {
+        groups: combinedGroups,
+        replaceString: content,
+      },
+      config
+    );
+    return result ? [result] : [];
   }
   return regexMatchAll(regexes[index], content)
     .flatMap((match) => {
-      // if we have a depName and a currentValue which have the minimal viable definition
-      if (match?.groups?.depName && match?.groups?.currentValue) {
-        return createDependency(
-          {
-            groups: mergeGroups(combinedGroups, match.groups),
-            replaceString: match[0],
-          },
-          config
-        );
-      }
-
       return handleRecursive(
         match[0],
         packageFile,
@@ -82,5 +82,5 @@ export function handleRecursive(
         mergeGroups(combinedGroups, match.groups ?? {})
       );
     })
-    .filter(is.truthy);
+    .filter(isValidDependency);
 }

lib/modules/manager/regex/utils.ts

@@ -97,3 +97,11 @@ export function mergeExtractionTemplate(
     replaceString: addition.replaceString ?? base.replaceString,
   };
 }
+
+export function isValidDependency({
+  depName,
+  currentValue,
+}: PackageDependency): boolean {
+  // check if all the fields are set
+  return Boolean(depName) && Boolean(currentValue);
+}