chore: wip

Author: chrisbbreuer

src/buddy.ts

@@ -199,7 +199,8 @@ export class Buddy {
               return new Promise((resolve, reject) => {
                 const child = spawn(command, args, { stdio: 'pipe' })
                 child.on('close', (code) => {
-                  if (code === 0) resolve()
+                  if (code === 0)
+                    resolve()
                   else reject(new Error(`Git command failed with code ${code}`))
                 })
                 child.on('error', reject)
@@ -210,6 +211,7 @@ export class Buddy {
             await runGitCommand('git', ['checkout', 'main'])
             await runGitCommand('git', ['reset', '--hard', 'HEAD'])
             await runGitCommand('git', ['clean', '-fd'])
+            // eslint-disable-next-line no-console
             console.log(`🧹 Reset to clean main state before generating updates for ${group.name}`)
           }
           catch (error) {

src/git/github-provider.ts

@@ -59,7 +59,8 @@ export class GitHubProvider implements GitProvider {
         if (nonWorkflowFiles.length > 0) {
           console.log(`📝 Committing ${nonWorkflowFiles.length} non-workflow files...`)
           files = nonWorkflowFiles
-        } else {
+        }
+        else {
           console.warn(`⚠️ All files are workflow files. No files will be committed in this PR.`)
           console.warn(`💡 To update workflow files, consider using a GitHub App with appropriate permissions.`)
           return // Exit early if no non-workflow files to commit
@@ -162,7 +163,8 @@ export class GitHubProvider implements GitProvider {
         if (nonWorkflowFiles.length > 0) {
           console.log(`📝 Committing ${nonWorkflowFiles.length} non-workflow files...`)
           files = nonWorkflowFiles
-        } else {
+        }
+        else {
           console.warn(`⚠️ All files are workflow files. No files will be committed in this PR.`)
           console.warn(`💡 To update workflow files, consider using a GitHub App with appropriate permissions.`)
           return // Exit early if no non-workflow files to commit

src/pr/pr-generator.ts

@@ -206,7 +206,8 @@ export class PullRequestGenerator {
           const sourceUrl = this.getComposerSourceUrl(packageInfo.repository.url, update.name)
           // Format: [packageName](repoUrl) ([source](sourceUrl)) - matching npm format
           packageCell = `[${update.name}](${sourceUrl}) ([source](${sourceUrl}))`
-        } else {
+        }
+        else {
           // Fallback to Packagist page only
           packageCell = `[${update.name}](https://packagist.org/packages/${encodeURIComponent(update.name)})`
         }
@@ -259,8 +260,6 @@ export class PullRequestGenerator {
       body += `\n`
     }
 
-
-
     // GitHub Actions table (simplified, without badges, deduplicated)
     if (uniqueGithubActionsUpdates.length > 0) {
       body += `### GitHub Actions\n\n`

src/registry/registry-client.ts

@@ -630,7 +630,8 @@ export class RegistryClient {
               if (showData.versions) {
                 availableVersions = showData.versions // This is already an array of version strings
               }
-            } catch (error) {
+            }
+            catch (error) {
               console.warn(`Failed to get available versions for ${pkg.name}, using latest only:`, error)
               availableVersions = [latestVersion]
             }
@@ -676,7 +677,7 @@ export class RegistryClient {
   /**
    * Find the best patch, minor, and major updates for a package
    */
-  private async findBestUpdates(currentVersion: string, availableVersions: string[], constraint: string): Promise<{ version: string, type: 'patch' | 'minor' | 'major' }[]> {
+  private async findBestUpdates(currentVersion: string, availableVersions: string[], _constraint: string): Promise<{ version: string, type: 'patch' | 'minor' | 'major' }[]> {
     const { getUpdateType } = await import('../utils/helpers')
     const candidates: { version: string, type: 'patch' | 'minor' | 'major' }[] = []
 
@@ -714,11 +715,13 @@ export class RegistryClient {
         if (!bestPatch || this.compareVersions(version, bestPatch) > 0) {
           bestPatch = version
         }
-      } else if (updateType === 'minor' && versionParts.major === currentParts.major) {
+      }
+      else if (updateType === 'minor' && versionParts.major === currentParts.major) {
         if (!bestMinor || this.compareVersions(version, bestMinor) > 0) {
           bestMinor = version
         }
-      } else if (updateType === 'major') {
+      }
+      else if (updateType === 'major') {
         if (!bestMajor || this.compareVersions(version, bestMajor) > 0) {
           bestMajor = version
         }
@@ -745,16 +748,16 @@ export class RegistryClient {
   private parseVersion(version: string): { major: number, minor: number, patch: number } | null {
     // Remove 'v' prefix and any pre-release identifiers
     const cleanVersion = version.replace(/^v/, '').split('-')[0].split('+')[0]
-    const parts = cleanVersion.split('.').map(p => parseInt(p, 10))
+    const parts = cleanVersion.split('.').map(p => Number.parseInt(p, 10))
 
-    if (parts.length < 2 || parts.some(p => isNaN(p))) {
+    if (parts.length < 2 || parts.some(p => Number.isNaN(p))) {
       return null
     }
 
     return {
       major: parts[0] || 0,
       minor: parts[1] || 0,
-      patch: parts[2] || 0
+      patch: parts[2] || 0,
     }
   }
 
@@ -766,10 +769,13 @@ export class RegistryClient {
     const parseA = this.parseVersion(a)
     const parseB = this.parseVersion(b)
 
-    if (!parseA || !parseB) return 0
+    if (!parseA || !parseB)
+      return 0
 
-    if (parseA.major !== parseB.major) return parseA.major - parseB.major
-    if (parseA.minor !== parseB.minor) return parseA.minor - parseB.minor
+    if (parseA.major !== parseB.major)
+      return parseA.major - parseB.major
+    if (parseA.minor !== parseB.minor)
+      return parseA.minor - parseB.minor
     return parseA.patch - parseB.patch
   }
 
@@ -972,11 +978,13 @@ export class RegistryClient {
         if (!bestPatch || this.compareVersions(version, bestPatch) > 0) {
           bestPatch = version
         }
-      } else if (updateType === 'minor' && versionParts.major === currentParts.major) {
+      }
+      else if (updateType === 'minor' && versionParts.major === currentParts.major) {
         if (!bestMinor || this.compareVersions(version, bestMinor) > 0) {
           bestMinor = version
         }
-      } else if (updateType === 'major') {
+      }
+      else if (updateType === 'major') {
         if (!bestMajor || this.compareVersions(version, bestMajor) > 0) {
           bestMajor = version
         }

test/composer-non-major-pr.test.ts

@@ -177,8 +177,8 @@ describe('Composer Non-Major PR', () => {
           metadata: {
             name: 'monolog/monolog',
             repository: 'https://github.com/Seldaek/monolog',
-            description: 'Sends your logs to files, sockets, inboxes, databases and various web services'
-          }
+            description: 'Sends your logs to files, sockets, inboxes, databases and various web services',
+          },
         },
         {
           name: 'monolog/monolog', // Duplicate package
@@ -190,8 +190,8 @@ describe('Composer Non-Major PR', () => {
           metadata: {
             name: 'monolog/monolog',
             repository: 'https://github.com/Seldaek/monolog',
-            description: 'Sends your logs to files, sockets, inboxes, databases and various web services'
-          }
+            description: 'Sends your logs to files, sockets, inboxes, databases and various web services',
+          },
         },
         {
           name: 'phpunit/phpunit',
@@ -237,8 +237,8 @@ describe('Composer Non-Major PR', () => {
           metadata: {
             name: 'monolog/monolog',
             repository: 'https://github.com/Seldaek/monolog',
-            description: 'Sends your logs to files, sockets, inboxes, databases and various web services'
-          }
+            description: 'Sends your logs to files, sockets, inboxes, databases and various web services',
+          },
         },
       ],
       updateType: 'minor',
@@ -250,11 +250,11 @@ describe('Composer Non-Major PR', () => {
 
     // Should format like npm: [packageName](repoUrl) ([source](sourceUrl))
     expect(prBody).toContain('[monolog/monolog](https://github.com/Seldaek/monolog/tree/master) ([source](https://github.com/Seldaek/monolog/tree/master))')
-    
+
     // Should have the same columns as npm dependencies
     expect(prBody).toContain('| Package | Change | Age | Adoption | Passing | Confidence |')
     expect(prBody).toContain('|---|---|---|---|---|---|')
-    
+
     // Should have confidence badges
     expect(prBody).toContain('[![age](https://developer.mend.io/api/mc/badges/age/packagist/')
     expect(prBody).toContain('[![adoption](https://developer.mend.io/api/mc/badges/adoption/packagist/')