fix: massage depName if missing (#28831)

renovatebot · May 4, 2024 · 5f1286f · 5f1286f
1 parent 74d2bfb
commit 5f1286f
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 2 deletions.
diff --git a/lib/workers/repository/extract/manager-files.spec.ts b/lib/workers/repository/extract/manager-files.spec.ts
@@ -52,13 +52,13 @@ describe('workers/repository/extract/manager-files', () => {
       fileMatch.getMatchingFiles.mockReturnValue(['Dockerfile']);
       fs.readLocalFile.mockResolvedValueOnce('some content');
       html.extractPackageFile = jest.fn(() => ({
-        deps: [{}, { replaceString: 'abc' }],
+        deps: [{}, { replaceString: 'abc', packageName: 'p' }],
       })) as never;
       const res = await getManagerPackageFiles(managerConfig);
       expect(res).toEqual([
         {
           packageFile: 'Dockerfile',
-          deps: [{}, { replaceString: 'abc' }],
+          deps: [{}, { replaceString: 'abc', packageName: 'p', depName: 'p' }],
         },
       ]);
     });

diff --git a/lib/workers/repository/extract/manager-files.ts b/lib/workers/repository/extract/manager-files.ts
@@ -9,6 +9,18 @@ import type { PackageFile } from '../../../modules/manager/types';
 import { readLocalFile } from '../../../util/fs';
 import type { WorkerExtractConfig } from '../../types';
 
+function massageDepNames(packageFiles: PackageFile[] | null): void {
+  if (packageFiles) {
+    for (const packageFile of packageFiles) {
+      for (const dep of packageFile.deps) {
+        if (dep.packageName && !dep.depName) {
+          dep.depName = dep.packageName;
+        }
+      }
+    }
+  }
+}
+
 export async function getManagerPackageFiles(
   config: WorkerExtractConfig,
 ): Promise<PackageFile[] | null> {
@@ -35,6 +47,7 @@ export async function getManagerPackageFiles(
       config,
       fileList,
     );
+    massageDepNames(allPackageFiles);
     return allPackageFiles;
   }
   const packageFiles: PackageFile[] = [];
@@ -58,5 +71,6 @@ export async function getManagerPackageFiles(
       logger.debug(`${packageFile} has no content`);
     }
   }
+  massageDepNames(packageFiles);
   return packageFiles;
 }