@gaia-ai/core 0.6.3 → 0.6.4

package/dist/src/core/logger.d.ts

@@ -3,6 +3,14 @@ export type LogSink = {
 } | {
     kind: 'file';
     path: string;
+}
+/**
+ * Tee: pretty stdout AND the durable `log.txt`. The DEFAULT on a TTY since
+ * GAIA-237 — see {@link resolveSink}.
+ */
+ | {
+    kind: 'both';
+    path: string;
 };
 export interface ConductorLogger {
     debug(obj: object, msg?: string): void;
@@ -12,7 +20,16 @@ export interface ConductorLogger {
 }
 /**
  * Pick the log sink. Precedence: explicit CLI override (`sink`) > the
- * GAIA_CONDUCTOR_LOG env var > TTY default (stdout on a TTY, file otherwise).
+ * GAIA_CONDUCTOR_LOG env var > default (`both` on a TTY, `file` otherwise).
+ *
+ * GAIA-237: the TTY default used to be stdout-ONLY, which made the conductor's
+ * own diagnostics non-durable exactly where it normally runs — inside a herdr
+ * pane, which IS a TTY. A well-formed `lifecycle hook failed` record (hook +
+ * worktree + ticket + err) was therefore written nowhere and died with the pane,
+ * so an aborting worktree bootstrap left no trace to find. The default now tees:
+ * the operator keeps pretty stdout AND the record lands in `log.txt`. An
+ * EXPLICIT `stdout` / `file` (CLI flag or GAIA_CONDUCTOR_LOG) keeps its exact
+ * previous meaning — one stream, chosen deliberately.
  */
 export declare function resolveSink(env: NodeJS.ProcessEnv, isTTY: boolean, checkoutRoot: string, override?: string): LogSink;
 /**

package/dist/src/core/logger.js

@@ -3,19 +3,27 @@ import pino from 'pino';
 import pretty from 'pino-pretty';
 /**
  * Pick the log sink. Precedence: explicit CLI override (`sink`) > the
- * GAIA_CONDUCTOR_LOG env var > TTY default (stdout on a TTY, file otherwise).
+ * GAIA_CONDUCTOR_LOG env var > default (`both` on a TTY, `file` otherwise).
+ *
+ * GAIA-237: the TTY default used to be stdout-ONLY, which made the conductor's
+ * own diagnostics non-durable exactly where it normally runs — inside a herdr
+ * pane, which IS a TTY. A well-formed `lifecycle hook failed` record (hook +
+ * worktree + ticket + err) was therefore written nowhere and died with the pane,
+ * so an aborting worktree bootstrap left no trace to find. The default now tees:
+ * the operator keeps pretty stdout AND the record lands in `log.txt`. An
+ * EXPLICIT `stdout` / `file` (CLI flag or GAIA_CONDUCTOR_LOG) keeps its exact
+ * previous meaning — one stream, chosen deliberately.
  */
 export function resolveSink(env, isTTY, checkoutRoot, override) {
-    const fileSink = {
-        kind: 'file',
-        path: join(checkoutRoot, 'log.txt'),
-    };
+    const path = join(checkoutRoot, 'log.txt');
     const choice = override ?? env.GAIA_CONDUCTOR_LOG;
     if (choice === 'stdout')
         return { kind: 'stdout' };
     if (choice === 'file')
-        return fileSink;
-    return isTTY ? { kind: 'stdout' } : fileSink;
+        return { kind: 'file', path };
+    if (choice === 'both')
+        return { kind: 'both', path };
+    return isTTY ? { kind: 'both', path } : { kind: 'file', path };
 }
 /**
  * Pick the log level. Precedence: explicit CLI override (`level`) >
@@ -32,5 +40,24 @@ export function createLogger(opts) {
     if (sink.kind === 'stdout') {
         return pino({ level }, pretty({ colorize: true, sync: true }));
     }
-    return pino({ level }, pino.destination({ dest: sink.path, append: true, mkdir: true }));
+    const file = () => pino.destination({
+        dest: sink.path,
+        append: true,
+        mkdir: true,
+        sync: true,
+    });
+    if (sink.kind === 'file') {
+        return pino({ level }, file());
+    }
+    // 'both' — tee via multistream so the pane stays readable AND the record is
+    // durable (GAIA-237). `level` is enforced by the logger, so each stream takes
+    // everything that got through.
+    // multistream filters per stream and defaults each entry to `info`, which
+    // would silently drop `--log-level debug` from BOTH streams — so each entry
+    // carries the resolved level and the logger stays the single gate.
+    const at = level;
+    return pino({ level }, pino.multistream([
+        { level: at, stream: pretty({ colorize: true, sync: true }) },
+        { level: at, stream: file() },
+    ]));
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gaia-ai/core",
-  "version": "0.6.3",
+  "version": "0.6.4",
   "description": "GAIA surface-agnostic kernel: host contract, split-config helpers, addon preset/discovery machinery, shared primitives.",
   "type": "module",
   "license": "MIT",