Don't prefix with type for info | log

Author: henryStelle

packages/app/src/cli/services/dev/extension/websocket/handlers.test.ts

@@ -280,7 +280,7 @@ describe('handleLogEvent()', () => {
     }
     handleLogEvent(eventData, options)
 
-    expectLogMessageOutput('test-extension', `INFO: Test info message`, options)
+    expectLogMessageOutput('test-extension', `Test info message`, options)
   })
 
   test('outputs log message with parsed JSON array', () => {
@@ -296,7 +296,7 @@ describe('handleLogEvent()', () => {
 
     expectLogMessageOutput(
       'test-extension',
-      outputContent`INFO: Hello world ${outputToken.json({user: 'test'})}`.value,
+      outputContent`Hello world ${outputToken.json({user: 'test'})}`.value,
       options,
     )
   })

packages/app/src/cli/services/dev/extension/websocket/handlers.ts

@@ -73,6 +73,8 @@ const consoleTypeColors = {
   error: (text: string) => outputToken.errorText(text),
 } as const
 
+const typesToIgnore: ReadonlyArray<string> = ['log', 'info'] as const
+
 export function handleLogEvent(
   eventData: {type: string; message: string; extensionName: string},
   options: SetupWebSocketConnectionOptions,
@@ -81,10 +83,14 @@ export function handleLogEvent(
   const formattedMessage = parseLogMessage(message)
 
   const uppercaseType = type.toUpperCase()
-  const colouredType = consoleTypeColors[type as keyof typeof consoleTypeColors]?.(uppercaseType) ?? uppercaseType
+  const coloredType = consoleTypeColors[type as keyof typeof consoleTypeColors]?.(uppercaseType) ?? uppercaseType
+
+  const completeMessage = typesToIgnore.includes(type)
+    ? formattedMessage
+    : outputContent`${coloredType}: ${formattedMessage}`.value
 
   useConcurrentOutputContext({outputPrefix: extensionName, stripAnsi: false}, () => {
-    options.stdout.write(outputContent`${colouredType}: ${formattedMessage}`.value)
+    options.stdout.write(completeMessage)
   })
 }