feat: option to efficiently use horizontal space

Author: pankgeorg

pretty/formatter.test.ts

@@ -5,9 +5,9 @@ import { assertEquals } from "@std/assert/equals";
 import { assertMatch } from "@std/assert/match";
 import { assertStringIncludes } from "@std/assert/string-includes";
 import {
-  type CategoryColorMap,
-  getPrettyFormatter,
-  prettyFormatter,
+    type CategoryColorMap,
+    getPrettyFormatter,
+    prettyFormatter,
 } from "./formatter.ts";
 
 const test = suite(import.meta);
@@ -829,3 +829,30 @@ test("properties set to true", () => {
     "Deno" in globalThis ? 'bar: "baz"' : "bar: 'baz'",
   );
 });
+
+test("newLine set to true", () => {
+  const formatter = getPrettyFormatter({
+    properties: true,
+    colors: false,
+    messageNewLine: true,
+    inspectOptions: { colors: false },
+  });
+
+  const record = createLogRecord("info", ["test"], ["FooBar"], Date.now(), {
+    foo: "bar",
+    bar: "baz",
+  });
+  const result = formatter(record);
+  console.error(result)
+  // Should contain multiple lines due to wrapping
+  const lines = result.split("\n");
+  assertEquals(lines.length, 5); // Normal log line + 1 for horizontal space + formatted properties + newline
+  assertEquals(
+    lines[2].trim(),
+    "Deno" in globalThis ? 'foo: "bar"' : "foo: 'bar'",
+  );
+  assertEquals(
+    lines[3].trim(),
+    "Deno" in globalThis ? 'bar: "baz"' : "bar: 'baz'",
+  );
+});

pretty/formatter.ts

@@ -1,11 +1,11 @@
+import { inspect, type InspectOptions } from "#util";
 import {
-  getLogLevels,
-  type LogLevel,
-  type LogRecord,
-  type TextFormatter,
-  type TextFormatterOptions,
+    getLogLevels,
+    type LogLevel,
+    type LogRecord,
+    type TextFormatter,
+    type TextFormatterOptions,
 } from "@logtape/logtape";
-import { inspect, type InspectOptions } from "#util";
 import { getOptimalWordWrapWidth } from "./terminal.ts";
 import { truncateCategory, type TruncationStrategy } from "./truncate.ts";
 import { getDisplayWidth, stripAnsi } from "./wcwidth.ts";
@@ -476,7 +476,15 @@ export interface PrettyFormatterOptions
    * @default `"rgb(148,163,184)"` (light slate gray)
    */
   readonly messageColor?: Color;
-
+  /**
+   * Visual style applied to save some horizontal space
+   *
+   * Controls whether the message will start after the category, or whether
+   * it will wrap below the categories, where the categories appear as a heading
+   *
+   * @default false
+   */
+  readonly messageNewLine?: boolean;
   /**
    * Visual style applied to log message text.
    *
@@ -672,6 +680,7 @@ export function getPrettyFormatter(
     categoryTruncate = "middle",
     messageColor = "rgb(148,163,184)",
     messageStyle = "dim",
+    messageNewLine = false,
     colors: useColors = true,
     align = true,
     inspectOptions = {},
@@ -822,6 +831,7 @@ export function getPrettyFormatter(
     let message = "";
     const messageColorCode = useColors ? colorToAnsi(messageColor) : "";
     const messageStyleCode = useColors ? styleToAnsi(messageStyle) : "";
+    const messageStart = messageNewLine ? "\n" : "";
     const messagePrefix = useColors
       ? `${messageStyleCode}${messageColorCode}`
       : "";
@@ -932,12 +942,14 @@ export function getPrettyFormatter(
       );
 
       let result =
-        `${formattedTimestamp}${formattedIcon} ${paddedLevel} ${paddedCategory} ${formattedMessage}`;
-      const indentWidth = getDisplayWidth(
-        stripAnsi(
-          `${formattedTimestamp}${formattedIcon} ${paddedLevel} ${paddedCategory} `,
-        ),
-      );
+        `${formattedTimestamp}${formattedIcon} ${paddedLevel} ${paddedCategory} ${messageStart}${formattedMessage}`;
+      const indentWidth = !messageNewLine
+        ? getDisplayWidth(
+          stripAnsi(
+            `${formattedTimestamp}${formattedIcon} ${paddedLevel} ${paddedCategory} `,
+          ),
+        )
+        : 4;
 
       // Apply word wrapping if enabled, or if there are multiline interpolated values
       if (wordWrapEnabled || message.includes("\n")) {
@@ -961,12 +973,12 @@ export function getPrettyFormatter(
       return result + "\n";
     } else {
       let result =
-        `${formattedTimestamp}${formattedIcon} ${formattedLevel} ${formattedCategory} ${formattedMessage}`;
-      const indentWidth = getDisplayWidth(
+        `${formattedTimestamp}${formattedIcon} ${formattedLevel} ${formattedCategory} ${messageStart}${formattedMessage}`;
+      const indentWidth = !messageNewLine ?getDisplayWidth(
         stripAnsi(
           `${formattedTimestamp}${formattedIcon} ${formattedLevel} ${formattedCategory} `,
         ),
-      );
+      ): 4;
 
       // Apply word wrapping if enabled, or if there are multiline interpolated values
       if (wordWrapEnabled || message.includes("\n")) {
@@ -1002,7 +1014,7 @@ function formatProperties(
   let result = "";
   for (const prop in record.properties) {
     const propValue = record.properties[prop];
-    const pad = indentWidth - getDisplayWidth(prop) - 2;
+    const pad = Math.max(6, indentWidth - getDisplayWidth(prop) - 2);
     result += "\n" + wrapText(
       `${" ".repeat(pad)}${useColors ? DIM : ""}${prop}:${
         useColors ? RESET : ""