test(logging): verify standalone bootstrap writes memos.log (#2147) (#2150)

* fix(local-plugin): call initLogger in bootstrapMemoryCoreFull

`bootstrapMemoryCoreFull` in `apps/memos-local-plugin/core/pipeline/memory-core.ts`
never invoked `initLogger(config, home)` after `loadConfig(home)`. The logger
stayed in the `bootstrapConsoleOnly()` fallback (console + memory buffer +
SSE broadcast only), so the `FileRotatingTransport` / `JsonlEventsTransport`
sinks were never wired and `MEMOS_HOME/logs/` (memos.log, error.log,
audit.log, llm.jsonl, perf.jsonl, events.jsonl) was left empty even though
`config.logging.file.enabled` defaults to true.

The fix adds `initLogger` to the logger import and calls it after the config
is resolved and before the first `rootLogger.child(...)` call. `initLogger`
is idempotent and swaps the active root in place, so existing child handles
keep working.

A new regression test `tests/unit/pipeline/bootstrap-init-logger.test.ts`
boots the pipeline against a tmp home, emits a marker line, and asserts
that `memos.log` is created on disk and contains the marker. It fails on
the previous code (ENOENT for memos.log) and passes with this fix.

Fixes #2147

* style(hermes): format daemon manager

---------

Co-authored-by: MemOS AutoDev <autodev@memtensor.local>
Co-authored-by: jiachengzhen <jiacz@memtensor.cn>

Author: 3 people

apps/memos-local-plugin/adapters/hermes/memos_provider/daemon_manager.py

@@ -215,7 +215,6 @@ def _rebuild_if_stale() -> bool:
         return False
 
 
-
 def ensure_bridge_running(*, probe_only: bool = False) -> bool:
     """Return True when the bridge is (or can be) operational.
 

apps/memos-local-plugin/tests/unit/pipeline/bootstrap-init-logger.test.ts

@@ -0,0 +1,74 @@
+/**
+ * Regression test for issue #2147:
+ * The standalone bridge must request logger initialization after config is
+ * resolved so file transports (memos.log, error.log, audit.log, llm.jsonl,
+ * perf.jsonl, events.jsonl) are created on disk. Before the fix, the bridge
+ * left the logger in `bootstrapConsoleOnly()` mode, so `home.logsDir` remained
+ * empty even though `config.logging.file.enabled` defaults to `true`.
+ *
+ * The assertion is deliberately tight: emit a distinctive marker line
+ * through `rootLogger.child(...)` after bootstrap, flush the logger, then
+ * confirm the marker landed in `memos.log`. Any regression that leaves the
+ * console-only sinks in place would make `memos.log` either missing or
+ * marker-free.
+ */
+
+import { afterEach, describe, expect, it } from "vitest";
+import { promises as fs } from "node:fs";
+import { join } from "node:path";
+
+import { makeTmpHome, type TmpHomeContext } from "../../helpers/tmp-home.js";
+import { bootstrapMemoryCoreFull } from "../../../core/pipeline/memory-core.js";
+import {
+  initTestLogger,
+  rootLogger,
+  shutdownLogger,
+} from "../../../core/logger/index.js";
+import type { MemoryCore } from "../../../agent-contract/memory-core.js";
+
+describe("bootstrapMemoryCoreFull -> initLogger", () => {
+  let home: TmpHomeContext | null = null;
+  let core: MemoryCore | null = null;
+
+  afterEach(async () => {
+    if (core) await core.shutdown();
+    await shutdownLogger();
+    if (home) await home.cleanup();
+    core = null;
+    home = null;
+    initTestLogger();
+  });
+
+  it("wires file transports when initLogging is set", async () => {
+    home = await makeTmpHome({ agent: "openclaw" });
+
+    // Sanity check: fixture leaves logsDir empty before bootstrap.
+    const before = await fs.readdir(home.home.logsDir);
+    expect(before).toEqual([]);
+
+    const result = await bootstrapMemoryCoreFull({
+      agent: "openclaw",
+      home: home.home,
+      config: home.config,
+      pkgVersion: "issue-2147-test",
+      initLogging: true,
+    });
+    core = result.core;
+
+    // Distinctive marker so we can grep memos.log without depending on
+    // whichever config warnings bootstrap emitted.
+    const marker = "issue-2147.marker.line.abcdef";
+    rootLogger.child({ channel: "core.pipeline.bootstrap" }).info(marker, {
+      probe: "bootstrap-init-logger",
+    });
+    await rootLogger.flush();
+
+    const memosLogPath = join(home.home.logsDir, "memos.log");
+    const stat = await fs.stat(memosLogPath);
+    expect(stat.isFile()).toBe(true);
+    expect(stat.size).toBeGreaterThan(0);
+
+    const text = await fs.readFile(memosLogPath, "utf8");
+    expect(text).toContain(marker);
+  });
+});