feat: append plugin name to logs (#205)

Author: sapphi-red

packages/vite/src/node/__tests__/build.spec.ts

@@ -886,8 +886,7 @@ test.skip('adjust worker build error for worker.format', async () => {
   expect.unreachable()
 })
 
-// rolldown does not append plugin name to the message automatically
-describe.skip('onRollupLog', () => {
+describe('onRollupLog', () => {
   const pluginName = 'rollup-plugin-test'
   const msgInfo = 'This is the INFO message.'
   const msgWarn = 'This is the WARN message.'
@@ -980,7 +979,7 @@ describe.skip('onRollupLog', () => {
       },
     })
     expect(onLogInfo).toBeCalledWith(
-      expect.objectContaining({ message: `[plugin ${pluginName}] ${msgInfo}` }),
+      expect.objectContaining({ message: msgInfo, plugin: pluginName }),
     )
   })
 
@@ -994,7 +993,7 @@ describe.skip('onRollupLog', () => {
       },
     })
     expect(onWarn).toBeCalledWith(
-      expect.objectContaining({ message: `[plugin ${pluginName}] ${msgWarn}` }),
+      expect.objectContaining({ message: msgWarn, plugin: pluginName }),
     )
   })
 

packages/vite/src/node/build.ts

@@ -1134,23 +1134,29 @@ export function onRollupLog(
       }
     }
 
+    // append plugin name to align with Rollup's behavior
+    let message = logging.message
+    if (logging.plugin) {
+      message = `[plugin ${logging.plugin}] ${message}`
+    }
+
     switch (logLeveling) {
       case 'info':
-        environment.logger.info(logging.message)
+        environment.logger.info(message)
         return
       case 'warn':
-        environment.logger.warn(colors.yellow(logging.message))
+        environment.logger.warn(colors.yellow(message))
         return
       case 'error':
-        environment.logger.error(colors.red(logging.message))
+        environment.logger.error(colors.red(message))
         return
       case 'debug':
-        debugLogger?.(logging.message)
+        debugLogger?.(message)
         return
       default:
         logLeveling satisfies never
         // fallback to info if a unknown log level is passed
-        environment.logger.info(logging.message)
+        environment.logger.info(message)
         return
     }
   }