fix: resolve the lockFileMaintenance not working

renovatebot · Nov 21, 2024 · 0877eb2 · 0877eb2
1 parent fa947a8
commit 0877eb2
Show file tree

Hide file tree

Showing 13 changed files with 16 additions and 17 deletions.
diff --git a/lib/modules/manager/bundler/artifacts.ts b/lib/modules/manager/bundler/artifacts.ts
@@ -93,7 +93,7 @@ export async function updateArtifacts(
 
     const commands: string[] = [];
 
-    if (config.isLockFileMaintenance) {
+    if (config.updateType === 'lockFileMaintenance') {
       commands.push('bundler lock --update');
     } else {
       const bundlerUpgraded = updatedDeps
@@ -289,7 +289,7 @@ export async function updateArtifacts(
     if (
       recursionLimit > 0 &&
       resolveMatches.length &&
-      !config.isLockFileMaintenance
+      config.updateType !== 'lockFileMaintenance'
     ) {
       logger.debug(
         { resolveMatches, updatedDeps },

diff --git a/lib/modules/manager/composer/artifacts.spec.ts b/lib/modules/manager/composer/artifacts.spec.ts
@@ -725,8 +725,8 @@ describe('modules/manager/composer/artifacts', () => {
         updatedDeps: [],
         newPackageFileContent: '{}',
         config: {
+          updateType:"lockFileMaintenance",
           ...config,
-          isLockFileMaintenance: true,
         },
       }),
     ).toEqual([

diff --git a/lib/modules/manager/composer/artifacts.ts b/lib/modules/manager/composer/artifacts.ts
@@ -113,6 +113,7 @@ export async function updateArtifacts({
 
   const file = Json.pipe(PackageFile).parse(newPackageFileContent);
 
+  const isLockFileMaintenance = config.updateType === 'lockFileMaintenance';
   const lockFileName = packageFileName.replace(regEx(/\.json$/), '.lock');
   const lockfile = await z
     .string()
@@ -174,7 +175,7 @@ export async function updateArtifacts({
 
     const cmd = 'composer';
     let args: string;
-    if (config.isLockFileMaintenance) {
+    if (isLockFileMaintenance) {
       args = 'update';
     } else {
       args =

diff --git a/lib/modules/manager/gradle/artifacts.ts b/lib/modules/manager/gradle/artifacts.ts
@@ -161,7 +161,7 @@ export async function updateArtifacts({
   }
 
   if (
-    config.isLockFileMaintenance &&
+    config.updateType === 'lockFileMaintenance' &&
     (!isGradleBuildFile(packageFileName) ||
       dirname(packageFileName) !== dirname(gradlewFile))
   ) {
@@ -205,7 +205,7 @@ export async function updateArtifacts({
         .join(' ')}`;
 
       if (
-        config.isLockFileMaintenance === true ||
+        config.updateType === 'lockFileMaintenance' ||
         !updatedDeps.length ||
         isGcvPropsFile(packageFileName)
       ) {

diff --git a/lib/modules/manager/jsonnet-bundler/artifacts.ts b/lib/modules/manager/jsonnet-bundler/artifacts.ts
@@ -48,7 +48,7 @@ export async function updateArtifacts(
   };
 
   try {
-    if (config.isLockFileMaintenance) {
+    if (config.updateType === 'lockFileMaintenance') {
       await exec('jb update', execOptions);
     } else {
       const dependencyUrls = updatedDeps.map(dependencyUrl);

diff --git a/lib/modules/manager/nix/artifacts.spec.ts b/lib/modules/manager/nix/artifacts.spec.ts
@@ -34,7 +34,6 @@ const dockerAdminConfig = {
 process.env.CONTAINERBASE = 'true';
 
 const config: UpdateArtifactsConfig = {};
-const lockMaintenanceConfig = { ...config, isLockFileMaintenance: true };
 const updateInputCmd = `nix \
 --extra-experimental-features 'nix-command flakes' \
 flake lock --update-input nixpkgs`;
@@ -294,7 +293,7 @@ describe('modules/manager/nix/artifacts', () => {
       packageFileName: 'flake.nix',
       updatedDeps: [{ depName: 'nixpkgs' }],
       newPackageFileContent: '{}',
-      config: lockMaintenanceConfig,
+      config: { updateType:"lockFileMaintenance",...config },
     });
 
     expect(res).toEqual([

diff --git a/lib/modules/manager/nix/artifacts.ts b/lib/modules/manager/nix/artifacts.ts
@@ -17,6 +17,8 @@ export async function updateArtifacts({
 }: UpdateArtifact): Promise<UpdateArtifactsResult[] | null> {
   const lockFileName = packageFileName.replace(regEx(/\.nix$/), '.lock');
   const existingLockFileContent = await readLocalFile(lockFileName, 'utf8');
+  const isLockFileMaintenance = config.updateType === 'lockFileMaintenance';
+
   if (!existingLockFileContent) {
     logger.debug('No flake.lock found');
     return null;
@@ -35,7 +37,7 @@ export async function updateArtifacts({
     cmd += `--extra-access-tokens github.com=${token} `;
   }
 
-  if (config.isLockFileMaintenance) {
+  if (isLockFileMaintenance) {
     cmd += 'flake update';
   } else {
     const inputs = updatedDeps

diff --git a/lib/modules/manager/nuget/artifacts.spec.ts b/lib/modules/manager/nuget/artifacts.spec.ts
@@ -189,7 +189,7 @@ describe('modules/manager/nuget/artifacts', () => {
         newPackageFileContent: '{}',
         config: {
           ...config,
-          isLockFileMaintenance: true,
+          updateType:"lockFileMaintenance"
         },
       }),
     ).toEqual([

diff --git a/lib/modules/manager/nuget/artifacts.ts b/lib/modules/manager/nuget/artifacts.ts
@@ -138,7 +138,7 @@ export async function updateArtifacts({
   }
 
   try {
-    if (updatedDeps.length === 0 && config.isLockFileMaintenance !== true) {
+    if (updatedDeps.length === 0 && config.updateType !== 'lockFileMaintenance') {
       logger.debug(
         `Not updating lock file because no deps changed and no lock file maintenance.`,
       );

diff --git a/lib/modules/manager/pip-compile/artifacts.ts b/lib/modules/manager/pip-compile/artifacts.ts
@@ -110,7 +110,7 @@ export async function updateArtifacts({
     try {
       await writeLocalFile(inputFileName, newInputContent);
       // TODO(not7cd): use --upgrade option instead deleting
-      if (config.isLockFileMaintenance) {
+      if (config.updateType === 'lockFileMaintenance') {
         await deleteLocalFile(outputFileName);
       }
       const compileArgs = extractHeaderCommand(existingOutput, outputFileName);

diff --git a/lib/modules/manager/pipenv/artifacts.ts b/lib/modules/manager/pipenv/artifacts.ts
@@ -135,7 +135,7 @@ export async function updateArtifacts({
   }
   try {
     await writeLocalFile(pipfileName, newPipfileContent);
-    if (config.isLockFileMaintenance) {
+    if (config.updateType === 'lockFileMaintenance') {
       await deleteLocalFile(lockFileName);
     }
     const cmd = 'pipenv lock';

diff --git a/lib/modules/manager/types.ts b/lib/modules/manager/types.ts
@@ -26,7 +26,6 @@ export interface ExtractConfig extends CustomExtractConfig {
 }
 
 export interface UpdateArtifactsConfig {
-  isLockFileMaintenance?: boolean;
   constraints?: Record<string, string>;
   composerIgnorePlatformReqs?: string[];
   goGetDirs?: string[];
@@ -175,7 +174,6 @@ export interface Upgrade<T = Record<string, any>> extends PackageDependency<T> {
   newVersion?: string;
   updateType?: UpdateType;
   version?: string;
-  isLockFileMaintenance?: boolean;
   isRemediation?: boolean;
   isVulnerabilityAlert?: boolean;
   vulnerabilitySeverity?: string;

diff --git a/lib/workers/repository/updates/flatten.ts b/lib/workers/repository/updates/flatten.ts
@@ -148,7 +148,6 @@ export async function flattenUpdates(
           packageFileConfig.lockFileMaintenance,
         );
         lockFileConfig.updateType = 'lockFileMaintenance';
-        lockFileConfig.isLockFileMaintenance = true;
         lockFileConfig = await applyPackageRules(
           lockFileConfig,
           'lock-file-maintenance-merge',