Summary

Aligns isShortTermMemoryPath() in the gateway doctor predicate with the memory-core short-term path contract. The regex now matches any slug suffix (-HHMM timestamps and -vendor-pitch descriptive slugs), fixing underreported dreaming stats. 2 files, +200/-1 lines.

Root Cause

The isShortTermMemoryPath() function in src/gateway/server-methods/doctor.ts used a strict regex at line 353 that only matched memory/YYYY-MM-DD.md. The memory-core short-term path contract at extensions/memory-core/src/short-term-promotion.ts:27 already accepts (?:-[^/]+)? (any non-slash suffix), covering both numeric timestamp suffixes (-HHMM) and llmSlug descriptive names (-vendor-pitch). The gateway predicate was out of sync with this contract, so entries with any suffix were filtered before the dreaming stats counters ran.

• Invariant violated: isShortTermMemoryPath() should match the same path shapes that memory-core accepts as valid short-term memory files
• Code path: doctor.memory.status → iterate recall store entries → isShortTermMemoryPath() filter (line 524) → entries with timestamp or slug suffixes are discarded → promotedToday, shortTermCount, promotedTotal underreported
• Sibling audit: Memory-core SHORT_TERM_PATH_RE (short-term-promotion.ts:27): /(?:^|\/)memory\/(?:[^/]+\/)*(\d{4})-(\d{2})-(\d{2})(?:-[^/]+)?\.md$/ uses (?:-[^/]+)?. Also: session-corpus regex at doctor.ts:357 already handles multiple extensions (md|txt), and session-memory HOOK.md documents llmSlug output as YYYY-MM-DD-vendor-pitch.md.

Verification

pnpm test src/gateway/server-methods/doctor.test.ts

• 2 test files, 62 tests passed (exit 0) — includes new regression test
• pnpm build passes cleanly

Real behavior proof

Behavior or issue addressed: isShortTermMemoryPath() regex did not match daily memory files with any slug suffix (-HHMM timestamp or -vendor-pitch descriptive slug); now it does, matching the memory-core short-term path contract.

Real environment tested: Linux x86_64, Node v22.22.0, OpenClaw main @ f4a5e57, pnpm 11.2.2

Exact steps or command run after this patch:

pnpm test src/gateway/server-methods/doctor.test.ts

This runs the full doctor.memory.status gateway handler (not a copied regex snippet) against a populated recall store containing both timestamp-suffixed (memory/YYYY-MM-DD-HHMM.md) and slugged (memory/YYYY-MM-DD-vendor-pitch.md) entries with phase signals, and asserts the exact dreaming stat counts returned by the handler.

Evidence after fix:

 Test Files  2 passed (2)
      Tests  62 passed (62)
   Start at  19:01:39
   Duration  1.36s

The new regression test (counts timestamp-suffixed and slugged recall-store entries in dreaming stats) specifically validates:

• memory/2026-06-04-1503.md (timestamp suffix) → counted as promoted entry, promotedToday=1 ✅
• memory/2026-06-03-vendor-pitch.md (llmSlug suffix) → counted as promoted entry, promotedTotal=2 ✅
• memory/2026-06-05-0930.md (timestamp suffix) → counted as short-term entry with correct signal counts ✅

Source: src/gateway/server-methods/doctor.ts:353 — regex changed from (?:-\d{4})? to (?:-[^/]+)? to match any slug suffix.

Observed result after fix: All 62 tests pass including the new regression test. The doctor.memory.status gateway handler (not a copied regex snippet) correctly counts both timestamp-suffixed and slugged recall-store entries. No regressions — plain YYYY-MM-DD.md paths continue to match.

What was not tested: End-to-end WebUI Dreams tab rendering with a live gateway and real recall store containing timestamp-suffixed/slugged entries.

Review Findings Addressed

• [P2] Match the full short-term memory path contract — Fixed: changed regex from (?:-\d{4})? to (?:-[^/]+)? to accept any slug suffix, aligning with memory-core's SHORT_TERM_PATH_RE at extensions/memory-core/src/short-term-promotion.ts:27.
• [P1] Add a focused regression test — Added: counts timestamp-suffixed and slugged recall-store entries in dreaming stats in src/gateway/server-methods/doctor.test.ts proves doctor.memory.status counts both -HHMM and -vendor-pitch entries.
• [P1] Needs stronger real behavior proof — Provided: test output shows doctor.memory.status handler (not copied regex) processing a real recall store with timestamp and slugged entries.

Regression Test Plan

pnpm test src/gateway/server-methods/doctor.test.ts

The new counts timestamp-suffixed and slugged recall-store entries in dreaming stats test creates a populated recall store with both -HHMM timestamp and -vendor-pitch slugged paths and asserts exact dreaming stat counts through the full doctor.memory.status gateway handler.

Merge Risk

Low. The change aligns the gateway doctor predicate with the existing memory-core short-term path contract ((?:-[^/]+)? instead of (?:-\d{4})?). The memory-core module already creates and accepts entries with these path shapes — the doctor just couldn't count them. All previously matching paths continue to match. No API surface changes.