fix: bootstrapMemoryCoreFull never calls initLogger — file logs (memos.log, llm.jsonl, audit.log) never written

[7889545]

Pre-submission checklist

• I have searched existing issues and this hasn't been mentioned before
• I have read the project documentation and confirmed this issue doesn't already exist
• This issue is specific to MemOS and not a general software issue

Bug Description

Summary

bootstrapMemoryCoreFull() in core/pipeline/memory-core.ts never calls initLogger(config, home). The logger remains in the bootstrapConsoleOnly() mode (console + memoryBuffer + broadcast only), so all file transports are never created — memos.log, error.log, audit.log, llm.jsonl, perf.jsonl, events.jsonl are all missing.

Root Cause

In memory-core.ts, after loadConfig(home) returns, the code proceeds directly to rootLogger.child(...) without calling initLogger(config, home) first:

// memory-core.ts line ~190
const config = configResult.config;
// ← Missing: initLogger(config, home);
const log = rootLogger.child({
    channel: "core.pipeline.bootstrap",
    ...
});

The initLogger function (from core/logger/index.ts) is responsible for creating FileRotatingTransport instances based on config.logging.file.enabled. Without this call, the logger stays in bootstrap mode.

Meanwhile, config.logging.file.enabled is true by default (defaults.ts line 279), and the config API confirms it's set — but the transports are never instantiated.

Evidence

1. MEMOS_HOME/logs/ directory is empty (no log files at all)
2. The config API returns "logging": { "file": { "enabled": true } }
3. initLogger is exported from core/logger/index.ts but never imported or called in the pipeline bootstrap code
4. The memory buffer transport works (SSE viewer shows logs), confirming the logger itself is functional — only file transports are missing

Impact

• No persistent log files for debugging
• No audit trail (audit.log)
• No LLM call history (llm.jsonl)
• No performance metrics (perf.jsonl)
• Users cannot diagnose issues from logs

Proposed Fix

Add initLogger(config, home) after config is loaded and before first log output:

const config = configResult.config;
// Initialize file transports (logs, audit, llm, perf, events).
initLogger(config, home);
const log = rootLogger.child({
    channel: "core.pipeline.bootstrap",
    ...
});

Also add the import:

import { rootLogger, initLogger } from "../logger/index.js";

Environment

• MemOS: v2.0.20 (memos-local-plugin)
• OS: Windows 11
• Agent: Hermes Agent

[Memtensor-AI]

🤖 AutoDev has picked up this issue and started working on it.

Task ID: 06ddfad15c7f30d6
Working branch: bugfix/autodev-2147-20260723095732420
Target branch: latest dev* branch
Workflow: opsp (analysis → coding → testing → PR)

I will post the PR link here once done. If I need more information, I will ask in the comments.

[Memtensor-AI]

✅ AutoDev task 06ddfad15c7f30d6 completed.

Summary: Fixed issue #2147: bootstrapMemoryCoreFull in apps/memos-local-plugin/core/pipeline/memory-core.ts never called initLogger(config, home) after loadConfig(home), so the logger stayed in bootstrapConsoleOnly() mode and no file transports were ever wired. As a result MEMOS_HOME/logs/ (memos.log, error.log, audit.log, llm.jsonl, perf.jsonl, events.jsonl) remained empty even though config.logging.file.enabled defaults to true.

The patch adds initLogger to the existing logger import and invokes initLogger(config, home) between the config resolution and the first rootLogger.child(...) call. initLogger is idempotent and swaps the active root in place, so pre-existing child handles continue working. A single insertion covers both public entry points because bootstrapMemoryCore delegates to bootstrapMemoryCoreFull.

A new regression test tests/unit/pipeline/bootstrap-init-logger.test.ts boots the pipeline against a tmp home, emits a marker line through rootLogger.child(...) after bootstrap, and asserts that memos.log exists on disk and contains the marker. Confirmed red on the original code (ENOENT for memos.log) and green after the fix. Verification: npx vitest run tests/unit → 1191 passed / 1 skipped / 0 failed across 142 test files; npm run lint (tsc noEmit) → 0 errors.

Branch bugfix/autodev-2147-20260723095732420 pushed to origin; commit eca1d65f. Task file archived to memos-autodev-specs main.

Base branch: main
Branch: bugfix/autodev-2147-20260723095732420
Commit: eca1d65
PR: #2150
Assigned to: @bittergreen
Reviewers: @whipser030, @hijzy