@openhoo/hoopilot 2.1.7 → 2.1.8

package/README.md

@@ -379,6 +379,12 @@ Console logs default to pretty output at `info` level:
 hoopilot --log-level info --log-format pretty
 ```
 
+Pretty logs keep common request and diagnostic fields inline for terminal use:
+
+```text
+INFO [16:40:14]: request completed component=server event=http.request.completed method=POST path=/v1/chat/completions status=200 duration=42.37ms stream=true requestId=req-test
+```
+
 For newline-delimited JSON:
 
 ```sh

package/dist/cli.js

@@ -667,6 +667,32 @@ var DEFAULT_LOG_FORMAT = "pretty";
 var DEFAULT_LOG_LEVEL = "info";
 var LOG_FORMATS = ["json", "pretty"];
 var LOG_LEVELS = ["trace", "debug", "info", "warn", "error", "fatal", "silent"];
+var PRETTY_INLINE_FIELDS = [
+  "component",
+  "command",
+  "event",
+  "method",
+  "path",
+  "status",
+  "durationMs",
+  "stream",
+  "route",
+  "requestId",
+  "upstreamPath",
+  "upstreamStatus",
+  "url",
+  "baseUrl",
+  "origin",
+  "currentVersion",
+  "installKind",
+  "latestVersion",
+  "assetName",
+  "count",
+  "plan",
+  "apiBaseUrl",
+  "authStorePath"
+];
+var PRETTY_IGNORED_FIELDS = ["pid", "hostname", "service", ...PRETTY_INLINE_FIELDS];
 var REDACT_PATHS = [
   "apiKey",
   "authorization",
@@ -730,9 +756,11 @@ function createHoopilotLogger(options = {}) {
           // stream's TTY-ness is unknown, so default to no color there.
           colorize: options.colorize ?? (options.stream ? false : process.stdout.isTTY),
           destination: options.stream ?? 1,
-          ignore: "pid,hostname",
+          ignore: PRETTY_IGNORED_FIELDS.join(","),
+          levelFirst: true,
+          messageFormat: formatPrettyMessage,
           singleLine: true,
-          translateTime: "SYS:standard"
+          translateTime: "SYS:HH:MM:ss"
         })
       )
     );
@@ -778,6 +806,45 @@ function errorDetails(error) {
   }
   return { message: String(error) };
 }
+function formatPrettyMessage(log, messageKey) {
+  const message = formatPrettyLogMessage(log[messageKey]);
+  const fields = PRETTY_INLINE_FIELDS.flatMap((field) => {
+    const value = log[field];
+    if (value === void 0) {
+      return [];
+    }
+    return `${prettyFieldLabel(field)}=${formatPrettyFieldValue(field, value)}`;
+  });
+  return fields.length > 0 ? `${message} ${fields.join(" ")}` : message;
+}
+function formatPrettyLogMessage(value) {
+  return typeof value === "string" ? value : formatPrettyValue(value);
+}
+function prettyFieldLabel(field) {
+  return field === "durationMs" ? "duration" : field;
+}
+function formatPrettyFieldValue(field, value) {
+  const formatted = formatPrettyValue(value);
+  return field === "durationMs" && typeof value === "number" ? `${formatted}ms` : formatted;
+}
+function formatPrettyValue(value) {
+  if (typeof value === "number") {
+    return Number.isFinite(value) ? String(value) : JSON.stringify(value);
+  }
+  if (typeof value === "boolean") {
+    return String(value);
+  }
+  if (typeof value === "string") {
+    return isBarePrettyValue(value) ? value : JSON.stringify(value);
+  }
+  if (value === null) {
+    return "null";
+  }
+  return JSON.stringify(value) ?? String(value);
+}
+function isBarePrettyValue(value) {
+  return /^[A-Za-z0-9._~:/?#[\]@!$&'()*+,;=%-]+$/.test(value);
+}
 function isLogFormat(value) {
   return LOG_FORMATS.includes(value);
 }