chore: several improvements

Author: chrisbbreuer

src/buddy.ts

@@ -1,3 +1,4 @@
+/* eslint-disable no-console */
 import type {
   BuddyBotConfig,
   DashboardData,
@@ -126,6 +127,12 @@ export class Buddy {
 
       // Determine if we have workflow permissions (BUDDY_BOT_TOKEN has full permissions)
       const hasWorkflowPermissions = !!process.env.BUDDY_BOT_TOKEN
+      if (process.env.BUDDY_BOT_TOKEN) {
+        console.log('✅ BUDDY_BOT_TOKEN detected - workflow permissions enabled')
+      }
+      else {
+        console.log('⚠️ BUDDY_BOT_TOKEN not found - workflow permissions disabled')
+      }
 
       // Initialize GitHub provider
       const gitProvider = new GitHubProvider(

src/git/github-provider.ts

@@ -54,7 +54,7 @@ export class GitHubProvider implements GitProvider {
       if (workflowFiles.length > 0 && !this.hasWorkflowPermissions) {
         console.warn(`⚠️ Detected ${workflowFiles.length} workflow file(s). These require elevated permissions.`)
         console.warn(`⚠️ Workflow files: ${workflowFiles.map(f => f.path).join(', ')}`)
-        console.warn(`ℹ️ Workflow files will be skipped in this commit. You need to set BUDDY_BOT_TOKEN (with workflow permissions) in the repository settings.`)
+        console.warn(`ℹ️ Workflow files will be skipped in this commit. BUDDY_BOT_TOKEN not detected or lacks workflow permissions.`)
 
         // If we have non-workflow files, commit just those
         if (nonWorkflowFiles.length > 0) {
@@ -63,7 +63,7 @@ export class GitHubProvider implements GitProvider {
         }
         else {
           console.warn(`⚠️ All files are workflow files. No files will be committed in this PR.`)
-          console.warn(`💡 To update workflow files, you need to set BUDDY_BOT_TOKEN (with workflow permissions) in the repository settings.`)
+          console.warn(`💡 To update workflow files, ensure BUDDY_BOT_TOKEN is set with workflow:write permissions.`)
           return // Exit early if no non-workflow files to commit
         }
       }

src/pr/pr-generator.ts

@@ -66,15 +66,44 @@ export class PullRequestGenerator {
     // Add update type labels
     if (group.updateType === 'major') {
       labels.push('major')
-    } else if (group.updateType === 'minor') {
+    }
+    else if (group.updateType === 'minor') {
       labels.push('minor')
-    } else if (group.updateType === 'patch') {
+    }
+    else if (group.updateType === 'patch') {
       labels.push('patch')
     }
 
+    // Add package type labels based on what types of packages are being updated
+    const npmCount = group.updates.filter(u =>
+      u.file === 'package.json' || u.file.endsWith('/package.json') || u.file.endsWith('\\package.json'),
+    ).length
+
+    const composerCount = group.updates.filter(u =>
+      u.file.endsWith('composer.json') || u.file.endsWith('composer.lock'),
+    ).length
+
+    const systemCount = group.updates.filter(u =>
+      (u.file.includes('.yaml') || u.file.includes('.yml')) && !u.file.includes('.github/workflows/'),
+    ).length
+
+    const actionsCount = group.updates.filter(u =>
+      u.file.includes('.github/workflows/'),
+    ).length
+
+    // Add package type labels
+    if (npmCount > 0)
+      labels.push('npm')
+    if (composerCount > 0)
+      labels.push('composer')
+    if (systemCount > 0)
+      labels.push('system')
+    if (actionsCount > 0)
+      labels.push('github-actions')
+
     // For single package updates, add the specific package name as a label
     if (group.updates.length === 1) {
-      const update = group.updates[0]
+      // const update = group.updates[0]
       labels.push(group.title) // This will be like "chore(deps): update dependency stripe to 18.4.0"
     }
 
@@ -118,9 +147,18 @@ export class PullRequestGenerator {
         dependencyType: u.dependencyType,
         currentVersion: u.currentVersion,
         newVersion: u.newVersion,
+        updateType: u.updateType,
       })),
     })
 
+    // Early validation - warn about empty updates
+    if (group.updates.length === 0) {
+      this.log('⚠️ WARNING: No updates in group - this will result in sparse PR body', {
+        groupName: group.name,
+        groupTitle: group.title,
+      })
+    }
+
     // Count different types of updates
     const packageJsonCount = group.updates.filter(u =>
       u.file === 'package.json' || u.file.endsWith('/package.json') || u.file.endsWith('\\package.json'),