fix for layout issues with log messages

Author: wolfeidau

internal/job/shell/logger.go

@@ -86,15 +86,15 @@ func (wl *WriterLogger) Commentf(format string, v ...any) {
 
 func (wl *WriterLogger) Errorf(format string, v ...any) {
 	if wl.Ansi {
-		wl.Printf(ansiColor("🚨 Error: "+format+"\n^^^ +++", "31"), v...)
+		wl.Printf(ansiColor("🚨 Error: "+format, "31")+"\n^^^ +++", v...)
 	} else {
 		wl.Printf("🚨 Error: "+format+"\n^^^ +++", v...)
 	}
 }
 
 func (wl *WriterLogger) Warningf(format string, v ...any) {
 	if wl.Ansi {
-		wl.Printf(ansiColor("⚠️ Warning: "+format+"\n^^^ +++", "33"), v...)
+		wl.Printf(ansiColor("⚠️ Warning: "+format, "33")+"\n^^^ +++", v...)
 	} else {
 		wl.Printf("⚠️ Warning: "+format+"\n^^^ +++", v...)
 	}

internal/job/shell/logger_test.go

@@ -43,6 +43,38 @@ func TestAnsiLogger(t *testing.T) {
 	}
 }
 
+func TestAnsiWithColorsLogger(t *testing.T) {
+	got := &bytes.Buffer{}
+	l := shell.NewWriterLogger(got, true, nil)
+
+	l.Headerf("Testing header: %q", "llamas")
+	l.Printf("Testing print: %q", "llamas")
+	l.Commentf("Testing comment: %q", "llamas")
+	l.Errorf("Testing error: %q", "llamas")
+	l.Warningf("Testing warning: %q", "llamas")
+	l.Promptf("Testing prompt: %q", "llamas")
+
+	want := &bytes.Buffer{}
+
+	fmt.Fprintln(want, `~~~ Testing header: "llamas"`)
+	fmt.Fprintln(want, `Testing print: "llamas"`)
+	fmt.Fprintln(want, `# Testing comment: "llamas"`)
+	fmt.Fprintln(want, `🚨 Error: Testing error: "llamas"`)
+	fmt.Fprintln(want, "^^^ +++")
+	fmt.Fprintln(want, `⚠️ Warning: Testing warning: "llamas"`)
+	fmt.Fprintln(want, "^^^ +++")
+
+	if runtime.GOOS == "windows" {
+		fmt.Fprintln(want, `> Testing prompt: "llamas"`)
+	} else {
+		fmt.Fprintln(want, `$ Testing prompt: "llamas"`)
+	}
+
+	if diff := cmp.Diff(got.String(), want.String()); diff != "" {
+		t.Fatalf("shell.WriterLogger output buffer diff (-got +want):\n%s", diff)
+	}
+}
+
 func TestLoggerStreamer(t *testing.T) {
 	got := &bytes.Buffer{}
 	l := shell.NewWriterLogger(got, false, nil)